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

NAS-127697 / 24.04-RC.1 / Allow RDMA capable interfaces to be exposed for regular networking (by bmeagherix) #13299

Merged
merged 2 commits into from
Mar 8, 2024

Conversation

bugclerk
Copy link
Contributor

@bugclerk bugclerk commented Mar 8, 2024

When JBOF support was added, we added all RDMA interfaces to rdma.interface.internal_interfaces. Instead only add those currently in use for JBOF purposes.

Likewise, in rdma.get_link_choices omit any interfaces that have been configured for general networking.

Original PR: #13289
Jira URL: https://ixsystems.atlassian.net/browse/NAS-127697

When JBOF support was added, we added *all* RDMA interfaces to
rdma.interface.internal_interfaces.  Instead only add those currently
in use for JBOF purposes.

Likewise, in rdma.get_link_choices omit any interfaces that have been
configured for general networking.

(cherry picked from commit b2d72ac)
Move rdma.get_configured_interfaces to interface.get_configured_interfaces and
reimplement

(cherry picked from commit f9a330c)
@yocalebo yocalebo merged commit e66f80a into release/24.04-RC.1 Mar 8, 2024
1 of 2 checks passed
@yocalebo yocalebo deleted the NAS-127697-24.04-RC.1 branch March 8, 2024 17:19
@bugclerk
Copy link
Contributor Author

bugclerk commented Mar 8, 2024

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Mar 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants