Skip to content

Prepare v2.8.0 #3419

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

Merged
merged 1 commit into from
Aug 18, 2025
Merged

Prepare v2.8.0 #3419

merged 1 commit into from
Aug 18, 2025

Conversation

pierreprinetti
Copy link
Member

No description provided.

@pierreprinetti pierreprinetti requested a review from a team June 11, 2025 16:21
@github-actions github-actions bot added v2 This PR targets v2 edit:gophercloud This PR updates common Gophercloud code semver:major Breaking change labels Jun 11, 2025
@pierreprinetti pierreprinetti added semver:minor Backwards-compatible change and removed semver:major Breaking change labels Jun 11, 2025
@kayrus
Copy link
Contributor

kayrus commented Jun 11, 2025

can we wait for #3414?

@pierreprinetti
Copy link
Member Author

can we wait for #3414?

OK!

@pierreprinetti pierreprinetti added the hold Do not merge label Jun 11, 2025
@coveralls
Copy link

coveralls commented Jun 11, 2025

Coverage Status

coverage: 63.673%. remained the same
when pulling 45ade0d on shiftstack:prepare_v2.8.0
into 6332fe9 on gophercloud:v2.

@kayrus
Copy link
Contributor

kayrus commented Jun 13, 2025

and this... #3424

@kayrus
Copy link
Contributor

kayrus commented Jun 13, 2025

and this #3425

@pierreprinetti
Copy link
Member Author

Sure you don't want to leave something for v2.9? 😅

@kayrus
Copy link
Contributor

kayrus commented Jun 13, 2025

Sure you don't want to leave something for v2.9?

I'd like to see main branch problems fixed for successful backports :)

@pierreprinetti
Copy link
Member Author

@kayrus Are we still waiting for any backports?

@kayrus
Copy link
Contributor

kayrus commented Jul 16, 2025

yes, see:
#3462
#3454
#3447
#3424

UPD:

Additionally, several colleagues have experienced issues with the new endpoint discovery logic introduced in 2.7.0. This logic relies solely on the keystone service catalog order, which causes problems when multiple endpoints are available in the catalog.
The current workaround is to override the list of aliases (which are globally modifiable). There have also been multiple related reports in the Terraform OpenStack provider:

To provide a quick fix, I implemented a workaround here: terraform-provider-openstack/utils#1. The endpoint discovery issue should be resolved with #3435. However, fetching microversions during each service client creation seems excessive and introduces unnecessary API calls, particularly for SDK consumers like the Terraform provider. See the detailed description: #3435 (comment).

@github-actions github-actions bot added semver:major Breaking change and removed semver:minor Backwards-compatible change labels Aug 14, 2025
@mandre mandre force-pushed the prepare_v2.8.0 branch 2 times, most recently from 7dd98a5 to e4d0cce Compare August 18, 2025 08:25
@mandre
Copy link
Contributor

mandre commented Aug 18, 2025

The big endpoint discovery change merged in v2 and the CI is as happy as it can be today. It's time to cut v2.8.0.

@mandre mandre removed the hold Do not merge label Aug 18, 2025
@mandre mandre merged commit 0f5d6e5 into gophercloud:v2 Aug 18, 2025
13 of 14 checks passed
@mandre mandre deleted the prepare_v2.8.0 branch August 18, 2025 11:42
@pierreprinetti pierreprinetti added semver:minor Backwards-compatible change and removed semver:major Breaking change labels Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
edit:gophercloud This PR updates common Gophercloud code semver:minor Backwards-compatible change v2 This PR targets v2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants