Skip to content

Deprecate ChooseVersion helper #3436

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

stephenfin
Copy link
Contributor

Resolve a TODO from #3420 and both deprecate the function and migrate our sole user of it away.

@coveralls
Copy link

coveralls commented Jun 19, 2025

Coverage Status

coverage: 63.72% (+0.002%) from 63.718%
when pulling 88e45ae on shiftstack:endpoint-discovery
into bbf1905 on gophercloud:main.

@github-actions github-actions bot added semver:minor Backwards-compatible change backport-v2 This PR will be backported to v2 edit:openstack This PR updates common OpenStack code edit:utils This PR updates utils code labels Jun 19, 2025
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
As noted inline, this uses relative URLs which will be important
shortly.

We also tweak the Magnum one, stripping a lot of irrelevant fields.

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
@stephenfin stephenfin force-pushed the endpoint-discovery branch from 3aaafff to 48c8c06 Compare July 21, 2025 15:33
These can be useful to do service discovery.

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Per the inline deprecation notice.

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
@stephenfin stephenfin force-pushed the endpoint-discovery branch from 48c8c06 to 88e45ae Compare July 21, 2025 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-v2 This PR will be backported to v2 edit:openstack This PR updates common OpenStack code edit:utils This PR updates utils code semver:minor Backwards-compatible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants