Skip to content

bulk security group rule creation: Allow using CreateOptsBuilder#3276

Merged
EmilienM merged 1 commit intogophercloud:masterfrom
shiftstack:sg_bulk_builder
Dec 22, 2024
Merged

bulk security group rule creation: Allow using CreateOptsBuilder#3276
EmilienM merged 1 commit intogophercloud:masterfrom
shiftstack:sg_bulk_builder

Conversation

@pierreprinetti
Copy link
Copy Markdown
Member

Before this patch, security group rules could only be created in a batch when using slices of plain rules.CreateOpts. With this change, a slice of any type implementing rules.CreateOptsBuilder can be used to create security group rules in a batch.

Before this patch, security group rules could only be created in a batch
when using slices of plain `rules.CreateOpts`. With this change, a slice
of any type implementing `rules.CreateOptsBuilder` can be used to create
security group rules in a batch.
@pierreprinetti pierreprinetti requested a review from a team December 20, 2024 11:31
@github-actions github-actions bot added edit:networking This PR updates networking code semver:major Breaking change labels Dec 20, 2024
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 20, 2024

Coverage Status

coverage: 78.648%. remained the same
when pulling c3046b1 on shiftstack:sg_bulk_builder
into b747927 on gophercloud:master.

mandre
mandre previously approved these changes Dec 20, 2024
@pierreprinetti pierreprinetti added the hold Do not merge label Dec 20, 2024
@pierreprinetti
Copy link
Copy Markdown
Member Author

There may be a race condition in the tests

@EmilienM EmilienM merged commit f7b156c into gophercloud:master Dec 22, 2024
@EmilienM EmilienM deleted the sg_bulk_builder branch December 22, 2024 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

edit:networking This PR updates networking code semver:major Breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants