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

[PR #1216/01f32747 backport][stable-3] wafv2_web_acl - fix return values #1227

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jun 7, 2022

This is a backport of PR #1216 as merged into main (01f3274).

SUMMARY
  • split integration tests from full wafv2 tests
  • relax botocore requirement to bare minimum required
  • return web acl info on update
  • consistently return web acl info as described in documentation (create would nest it under "web_acl")
  • fix "changed" value when description not specified
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

wafv2_web_acl

ADDITIONAL INFORMATION

wafv2_web_acl - fix return values

SUMMARY

split integration tests from full wafv2 tests
relax botocore requirement to bare minimum required
return web acl info on update
consistently return web acl info as described in documentation (create would nest it under "web_acl")
fix "changed" value when description not specified

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
wafv2_web_acl
ADDITIONAL INFORMATION

Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
(cherry picked from commit 01f3274)
@ansibullbot
Copy link

@ansibullbot ansibullbot added bug This issue/PR relates to a bug community_review integration tests/integration module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Jun 7, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

ansible-galaxy-importer FAILURE in 4m 14s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 51s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 33s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 23s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 13m 11s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 9m 40s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 04s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 46s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 5m 50s
✔️ ansible-test-splitter SUCCESS in 2m 56s
✔️ integration-community.aws-1 SUCCESS in 7m 23s
⚠️ 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

@markuman markuman added the mergeit Merge the PR (SoftwareFactory) label Jun 8, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

ansible-galaxy-importer FAILURE in 4m 44s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 41s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 41s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 12m 22s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 11m 14s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 10m 52s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 45s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 52s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 15s
✔️ ansible-test-splitter SUCCESS in 2m 24s
✔️ integration-community.aws-1 SUCCESS in 7m 29s
⚠️ 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 9e84535 into stable-3 Jun 8, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-3/01f3274762d475922db6b6c8ccca468cbae92905/pr-1216 branch June 8, 2022 04:39
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…e reuse (ansible-collections#1227)

Move botocore version validation into module_utils.botocore for future reuse

Depends-On: ansible-collections#1246
SUMMARY
In preparation for ansible-collections#1225 move the SDK version validation pieces over into module_utils.botocore
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/module_utils/botocore.py
plugins/module_utils/core.py
plugins/module_utils/exceptions.py
plugins/module_utils/modules.py
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Mark Chappell <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug community_review integration tests/integration mergeit Merge the PR (SoftwareFactory) module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants