Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove obsolete providers #9629

Merged
merged 5 commits into from
Dec 17, 2023
Merged

Conversation

j-xiong
Copy link
Contributor

@j-xiong j-xiong commented Dec 6, 2023

No description provided.

Provider is supported up to 1.20.x

Signed-off-by: Sean Hefty <[email protected]>
Provider is supported up to 1.20.x

Signed-off-by: Sean Hefty <[email protected]>
GNI is supported up to 1.20.x

Signed-off-by: Sean Hefty <[email protected]>
NetworkDirect is now supported by the verbs provider.

Signed-off-by: Sean Hefty <[email protected]>
Signed-off-by: Jianxin Xiong <[email protected]>
@shijin-aws
Copy link
Contributor

Do you need to bump the version to 2.x?

@j-xiong
Copy link
Contributor Author

j-xiong commented Dec 7, 2023

@shijin-aws I don't think so. The API doesn't change.

@shijin-aws
Copy link
Contributor

shijin-aws commented Dec 7, 2023

Since we agreed that we will use main for 2.x development, and only cut 1.x release on demand. I think we need some way to divide which commits should be only for 2.x and won't be part of 1.x when we cut a 1.x branch release in the future. I thought such obsolete provider removal is part of 2.x?

@j-xiong
Copy link
Contributor Author

j-xiong commented Dec 7, 2023

The proposed changes are classified into three categories:
(1) Those don't break API, such as obsolete provider removal.
(2) Those theoretically break API, but practically don't, such as removing APIs that have never been implemented.
(3) Those break API.

We want to delay the creation of 1.x branch as much as possible. The plan is to do that when patches in category (3) are merged.

@shijin-aws
Copy link
Contributor

shijin-aws commented Dec 7, 2023

Thank you. AWS still needs a 1.21 release for a new feature #9525 that is being development in the current main branch. IIUC what you said it should be still be ok to cut 1.x release from main before the (3) happens?

@j-xiong
Copy link
Contributor Author

j-xiong commented Dec 7, 2023

That's right.

@j-xiong
Copy link
Contributor Author

j-xiong commented Dec 8, 2023

@ofiwg/intel-middleware

@garzaran
Copy link

garzaran commented Dec 9, 2023 via email

--disable-efa
--disable-gni
--disable-hook_debug
--disable-mrail
--disable-perf
--disable-psm
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

psm (v1) should have already be removed. so you should be able to remove --disable-psm

@jswaro
Copy link
Contributor

jswaro commented Dec 12, 2023

Why is GNI being removed?

@j-xiong
Copy link
Contributor Author

j-xiong commented Dec 12, 2023

@jswaro The GNI provider is not being actively maintained (the last update was 2 years ago). Please feel free to provide your input here.

@jswaro
Copy link
Contributor

jswaro commented Dec 12, 2023

@jswaro The GNI provider is not being actively maintained (the last update was 2 years ago). Please feel free to provide your input here.

I'll ask around. I don't know if anyone is directly impacted. I haven't received an answer yet.

@j-xiong j-xiong merged commit 52be148 into ofiwg:main Dec 17, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants