Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

acceptance: Prevent 409 when bulk-creating secgroup rules #3392

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 22, 2025
Merged

acceptance: Prevent 409 when bulk-creating secgroup rules #3392

merged 1 commit into from
May 22, 2025
+5 −7

Conversation

pierreprinetti
Copy link
Member

Reduce the randomness of port creation.

@github-actions github-actions bot added the edit:networking This PR updates networking code label May 22, 2025
@pierreprinetti pierreprinetti added backport-v1 This PR will be backported to v1 backport-v2 This PR will be backported to v2 labels May 22, 2025
@github-actions github-actions bot added the semver:patch No API change label May 22, 2025
@pierreprinetti pierreprinetti mentioned this pull request May 22, 2025
Merged
@coveralls
Copy link

coveralls commented May 22, 2025 •
edited
Loading

Coverage Status

coverage: 63.83% (+0.002%) from 63.828%
when pulling 0cc4e3f on shiftstack:less_random_fewer_errors
into dbed7c1 on gophercloud:main.

@pierreprinetti pierreprinetti force-pushed the less_random_fewer_errors branch from 35cb01b to c868006 Compare May 22, 2025 14:25
mandre
mandre reviewed May 22, 2025
View reviewed changes
internal/acceptance/openstack/networking/v2/extensions/extensions.go Outdated Show resolved Hide resolved
internal/acceptance/openstack/networking/v2/extensions/extensions.go Outdated Show resolved Hide resolved
@pierreprinetti pierreprinetti force-pushed the less_random_fewer_errors branch from c868006 to bfceff6 Compare May 22, 2025 15:50
@github-actions github-actions bot added the edit:testinfra This PR updates testing infrastructure code label May 22, 2025
@pierreprinetti pierreprinetti force-pushed the less_random_fewer_errors branch from bfceff6 to 290ae22 Compare May 22, 2025 16:00
@pierreprinetti
acceptance: Prevent 409 when bulk-creating secgroup rules
0cc4e3f 
Reduce the randomness of port creation.
@pierreprinetti pierreprinetti force-pushed the less_random_fewer_errors branch from 290ae22 to 0cc4e3f Compare May 22, 2025 16:02
@mandre mandre merged commit ad0459b into gophercloud:main May 22, 2025
18 checks passed
@mandre mandre deleted the less_random_fewer_errors branch May 22, 2025 16:23
@github-actions GitHub Actions
Copy link

Failed to backport PR to v1 branch. See logs for details.

@gophercloud-backport-bot gophercloud-backport-bot bot mentioned this pull request May 22, 2025
Merged
@pierreprinetti pierreprinetti removed the backport-v1 This PR will be backported to v1 label May 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mandre mandre mandre approved these changes

@stephenfin stephenfin Awaiting requested review from stephenfin

Assignees
No one assigned
Labels
backport-v2 This PR will be backported to v2 edit:networking This PR updates networking code edit:testinfra This PR updates testing infrastructure code semver:patch No API change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pierreprinetti @coveralls @mandre
Morty Proxy This is a proxified and sanitized view of the page, visit original site.