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

Handle nova api version > 2.87 for hypervisor #3028

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

samuelallan72
Copy link

NOTE: this is based on and tested with v1.5.0 (I'm testing for openstack-exporter/openstack-exporter#268 ). Please let me know if this should be rebased, targeted to v1 branch, etc.? Help appreciated for testing this. :)

Fixes #3026

@github-actions github-actions bot added the semver:patch No API change label Apr 10, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for submitting your first PR! Be sure that we will be looking at it but keep in mind
this sometimes takes a while.
Please let the maintainers know if your PR has not got enough attention after a few days.
If any doubt, please consult our PR tutorial.

@EmilienM
Copy link
Contributor

EmilienM commented May 6, 2024

nice addition, please rebase

@github-actions github-actions bot removed the semver:patch No API change label May 7, 2024
@samuelallan72
Copy link
Author

Thanks @EmilienM , I've rebased it. :)

@github-actions github-actions bot added the semver:patch No API change label May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:patch No API change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nova API > 2.87 breaks fetching cpu_info from hypervisor
2 participants