Skip to content
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

[stable-3] wafv2_ip_set - fix bugs with changing description #1214

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jun 4, 2022

Manual backport of #1211

SUMMARY

Updating just the description didn't update the changed state

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

wafv2_ip_set

ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz

…s#1211)

wafv2_ip_set - fix bugs with changing description

SUMMARY

updating just the description didn't update the changed state

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
wafv2_ip_set
ADDITIONAL INFORMATION

Reviewed-by: Markus Bergholz <[email protected]>
@tremble tremble requested a review from markuman June 4, 2022 12:19
@tremble tremble added the backport-2 PR should be backported to the stable-2 branch label Jun 4, 2022
@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration module module plugins plugin (any type) tests tests labels Jun 4, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

ansible-galaxy-importer FAILURE in 4m 33s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 52s
✔️ ansible-test-sanity-docker-devel SUCCESS in 11m 20s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 20s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 12m 11s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 9m 54s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 04s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 46s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 41s
✔️ ansible-test-splitter SUCCESS in 3m 31s
✔️ integration-community.aws-1 SUCCESS in 5m 49s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@tremble tremble changed the title wafv2_ip_set - fix bugs with changing description [stable-3] wafv2_ip_set - fix bugs with changing description Jun 4, 2022
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jun 7, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

ansible-galaxy-importer FAILURE in 4m 58s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 21s
✔️ ansible-test-sanity-docker-devel SUCCESS in 11m 55s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 07s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 16m 39s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 11m 24s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 42s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 31s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 28s
✔️ ansible-test-splitter SUCCESS in 2m 41s
✔️ integration-community.aws-1 SUCCESS in 6m 07s
⚠️ integration-community.aws-2 SKIPPED
⚠️ integration-community.aws-3 SKIPPED
⚠️ integration-community.aws-4 SKIPPED
⚠️ integration-community.aws-5 SKIPPED
⚠️ integration-community.aws-6 SKIPPED
⚠️ integration-community.aws-7 SKIPPED
⚠️ integration-community.aws-8 SKIPPED
⚠️ integration-community.aws-9 SKIPPED
⚠️ integration-community.aws-10 SKIPPED
⚠️ integration-community.aws-11 SKIPPED
⚠️ integration-community.aws-12 SKIPPED
⚠️ integration-community.aws-13 SKIPPED

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 7a5ece4 into ansible-collections:stable-3 Jun 7, 2022
@patchback
Copy link

patchback bot commented Jun 7, 2022

Backport to stable-2: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-2/7a5ece48d95b009a5de9ee01b87bb77b8d17915f/pr-1214

Backported as #1228

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Jun 7, 2022
[stable-3] wafv2_ip_set - fix bugs with changing description

Manual backport of #1211
SUMMARY
Updating just the description didn't update the changed state
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
wafv2_ip_set
ADDITIONAL INFORMATION
Reviewed-by: Markus Bergholz

Reviewed-by: Alina Buzachis <None>
(cherry picked from commit 7a5ece4)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jun 8, 2022
)

[PR #1214/7a5ece48 backport][stable-2] wafv2_ip_set - fix bugs with changing description

This is a backport of PR #1214 as merged into stable-3 (7a5ece4).
Manual backport of #1211
SUMMARY
Updating just the description didn't update the changed state
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
wafv2_ip_set
ADDITIONAL INFORMATION
Reviewed-by: Markus Bergholz

Reviewed-by: Markus Bergholz <[email protected]>
@tremble tremble deleted the backport/stable-3/1211 branch July 7, 2022 19:23
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…ible-collections#1214)

ec2_security_group - Move validation for rules into the arg_spec

SUMMARY
The bulk of the current rule validation can be better handled by arg_spec parameter validation.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_security_group
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
ec2_security_group - refacter get_target_from_rule()

SUMMARY
refacter get_target_from_rule to bring down the complexity score
Builds on top of ansible-collections#1214
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_security_group
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
Fix ec2_metadata_facts integration test

SUMMARY
With ansible-collections#1214 security_group now validates the rules its passed, requiring a "source" be passed (SG/CIDR).  Previously it just dropped the rule on the floor.  This has however broken the ec2_metadata_facts integration test
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
ec2_metadata_facts
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-2 PR should be backported to the stable-2 branch bug This issue/PR relates to a bug community_review integration tests/integration mergeit Merge the PR (SoftwareFactory) module module plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants