Skip to content

Leverage RandomUUID() in Placement resourceproviders tests - #3729

Merged
mandre merged 1 commit into
gophercloud:mainfrom
Etua:backport-random-uuid-to-resource-providers
May 5, 2026
Merged

mandre merged 1 commit into
gophercloud:mainfrom
Etua:backport-random-uuid-to-resource-providers

Conversation

@Etua

@Etua Etua commented May 4, 2026 •

Copy link
Copy Markdown
Contributor

In response to the following addition.

Related #526

@Etua
Etua force-pushed the backport-random-uuid-to-resource-providers branch from 3002343 to 5cbbb20 Compare May 4, 2026 10:23
@Etua Etua changed the title Backport RandomUUID to older Placement resourceproviders's code Backport RandomUUID to older Placement resourceproviders code May 4, 2026
@github-actions github-actions Bot added edit:placement This PR updates placement code semver:patch No API change backport-v2 This PR will be backported to v2 labels May 4, 2026
@coveralls

coveralls commented May 4, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 64.013%. remained the same — Etua:backport-random-uuid-to-resource-providers into gophercloud:main

@mandre

mandre commented May 5, 2026

Copy link
Copy Markdown
Contributor

Hey @Etua, it should be included in #3728 already, could you double check?

@Etua

Etua commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

@mandre I modified the tests in that PR, as well as the usage one that you've just accepted (#3715). Here I make the changes to the code introduced in the older PRs (inventories, aggregates); I figured it would scope better if I didn't to it in #3713

@mandre

mandre commented May 5, 2026

Copy link
Copy Markdown
Contributor

Gotcha, thanks for the clarification. Would you mind then updating the commit's title to something like Leverage RandomUUID() in Placement resourceproviders tests or equivalent?

@Etua Etua changed the title Backport RandomUUID to older Placement resourceproviders code Leverage RandomUUID() in Placement resourceproviders tests May 5, 2026
@Etua
Etua force-pushed the backport-random-uuid-to-resource-providers branch from 5cbbb20 to 9769596 Compare May 5, 2026 07:52
@Etua

Etua commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

Done

@Etua

Etua commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

@mandre Pipeline failed on the name change at the Devstack deployment stage.

@mandre

mandre commented May 5, 2026

Copy link
Copy Markdown
Contributor

Seems like a github isue:

Resolving github.com (github.com)... 140.82.112.4
Connecting to github.com (github.com)|140.82.112.4|:443... connected.
HTTP request sent, awaiting response... 502 Bad Gateway or Proxy Error
2026-05-05 07:57:44 ERROR 502: Bad Gateway or Proxy Error.

Re-running.

@mandre
mandre enabled auto-merge May 5, 2026 09:21
@mandre
mandre added this pull request to the merge queue May 5, 2026
Merged via the queue into gophercloud:main with commit f4ea62f May 5, 2026
66 of 67 checks passed
@Etua
Etua deleted the backport-random-uuid-to-resource-providers branch May 5, 2026 09:56
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:placement This PR updates placement code semver:patch No API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants