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

(OpenStack) InfiniBand support #125

Closed
orviz opened this issue Dec 17, 2018 · 3 comments
Closed

(OpenStack) InfiniBand support #125

orviz opened this issue Dec 17, 2018 · 3 comments

Comments

@orviz
Copy link
Contributor

orviz commented Dec 17, 2018

Short Description of the issue

As part of the requirements of DEEP project, we have implemented infiniband support for OpenStack. As described in [1], this implementation relies on the flavor's metadata (extra_specs) in order to extract this information. Both extra_specs' key and value can be provided at runtime via two new command line arguments.

In the current implementation of cloud-info-provider, infiniband support is available through network_info parameter in the static configuration, but this is VO-wide, not flavor specific. This issue is created to follow the discussion on the best way to integrate DEEP implementation of InfiniBand support.

Might be related with #88

[1] https://github.com/indigo-dc/cloud-info-provider-deep/blob/DEEP/README-DEEP.md

@enolfc
Copy link
Contributor

enolfc commented Dec 20, 2018

I think this is nice addition and by reading the description at the DEEP fork, the implementation seems to be doing the right thing. My only doubt is what are the sensible defaults for the new arguments. I'd suggest to send a PR to bring this in.

@orviz
Copy link
Contributor Author

orviz commented Jan 22, 2021

Addressed in #214

@orviz orviz closed this as completed Jan 22, 2021
@enolfc
Copy link
Contributor

enolfc commented Jan 22, 2021

I think we should close once #124 is merged

@enolfc enolfc reopened this Jan 22, 2021
@enolfc enolfc closed this as completed Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants