Skip to content

Prepare v2.8.0 - #3419

Merged
mandre merged 1 commit into
gophercloud:v2from
shiftstack:prepare_v2.8.0
Aug 18, 2025
Merged

mandre merged 1 commit into
gophercloud:v2from
shiftstack:prepare_v2.8.0

Conversation

@pierreprinetti

Copy link
Copy Markdown
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

kayrus commented Jun 11, 2025

Copy link
Copy Markdown
Contributor

can we wait for #3414?

@pierreprinetti

Copy link
Copy Markdown
Member Author

can we wait for #3414?

OK!

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

coveralls commented Jun 11, 2025 •

Copy link
Copy Markdown

Coverage Status

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

@kayrus

kayrus commented Jun 13, 2025

Copy link
Copy Markdown
Contributor

and this... #3424

@kayrus

kayrus commented Jun 13, 2025

Copy link
Copy Markdown
Contributor

and this #3425

@pierreprinetti

Copy link
Copy Markdown
Member Author

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

@kayrus

kayrus commented Jun 13, 2025

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member Author

@kayrus Are we still waiting for any backports?

@kayrus

kayrus commented Jul 16, 2025 •

Copy link
Copy Markdown
Contributor

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

mandre commented Aug 18, 2025

Copy link
Copy Markdown
Contributor

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
@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