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

sns_topic - Fix bug when used in GovCloud - issue 836 #1195

Merged
merged 5 commits into from
Jun 29, 2022

Conversation

Razique
Copy link
Contributor

@Razique Razique commented Jun 2, 2022

SUMMARY

Add region detection to skip usage of FIFO topics when using GovCloud regions
Fixes #836

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

community.aws.sns_topic

plugins/modules/sns_topic.py Outdated Show resolved Hide resolved
@ansibullbot
Copy link

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

Build succeeded.

ansible-galaxy-importer FAILURE in 12m 51s (non-voting)
✔️ build-ansible-collection SUCCESS in 10m 48s
✔️ ansible-test-sanity-docker-devel SUCCESS in 24m 05s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 24m 25s
✔️ ansible-test-sanity-docker-stable-2.9 SUCCESS in 25m 33s
✔️ ansible-test-sanity-docker-stable-2.11 SUCCESS in 22m 42s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 20m 39s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 18m 09s
✔️ ansible-test-splitter SUCCESS in 5m 52s
✔️ integration-community.aws-1 SUCCESS in 6m 48s
⚠️ 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

plugins/modules/sns_topic.py Outdated Show resolved Hide resolved
@ansibullbot ansibullbot added needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR and removed community_review labels Jun 13, 2022
@github-actions
Copy link

github-actions bot commented Jun 28, 2022

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@ansibullbot ansibullbot added community_review and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR small_patch Hopefully easy to review labels Jun 28, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

ansible-galaxy-importer FAILURE in 4m 19s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 52s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 55s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 59s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 10s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 38s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 6m 41s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 09s
✔️ ansible-test-splitter SUCCESS in 2m 56s
integration-community.aws-1 FAILURE in 6m 25s
⚠️ 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

@azure-pipelines
Copy link

There was an error handling pipeline event 2339f52e-ad70-4330-95fd-93d2faff75c3.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

ansible-galaxy-importer FAILURE in 5m 05s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 18s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 53s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 11m 28s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 11m 29s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 11m 05s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 12m 15s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 23s
✔️ ansible-test-splitter SUCCESS in 2m 28s
✔️ integration-community.aws-1 SUCCESS in 8m 21s
⚠️ 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 Add endpoint detection to skip usage of FIFO topics for GovCloud regions - issue 836 sns_topic - Fix bug when used in GovCloud - issue 836 Jun 29, 2022
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Jun 29, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 27s (non-voting)
✔️ build-ansible-collection SUCCESS in 5m 44s
✔️ ansible-test-sanity-docker-devel SUCCESS in 9m 27s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 9m 14s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 9m 01s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 09s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 06s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 15s
✔️ ansible-test-splitter SUCCESS in 2m 28s
✔️ integration-community.aws-1 SUCCESS in 6m 24s
⚠️ 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
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 3m 58s (non-voting)
✔️ build-ansible-collection SUCCESS in 4m 44s
✔️ ansible-test-sanity-docker-devel SUCCESS in 10m 19s (non-voting)
✔️ ansible-test-sanity-docker-milestone SUCCESS in 10m 37s
✔️ ansible-test-sanity-docker-stable-2.12 SUCCESS in 10m 00s
✔️ ansible-test-sanity-docker-stable-2.13 SUCCESS in 10m 15s
✔️ ansible-test-units-community-aws-python38 SUCCESS in 5m 54s
✔️ ansible-test-units-community-aws-python39 SUCCESS in 6m 59s
✔️ ansible-test-splitter SUCCESS in 2m 31s
✔️ integration-community.aws-1 SUCCESS in 6m 10s
⚠️ 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
Copy link
Contributor

Pull request merge failed: Resource not accessible by integration, You may need to manually rebase your PR and retry.

@tremble tremble merged commit 2d74eda into ansible-collections:main Jun 29, 2022
@tremble tremble added the backport-4 PR should be backported to the stable-4 branch label Jun 29, 2022
@patchback
Copy link

patchback bot commented Jun 29, 2022

Backport to stable-4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4/2d74eda72b97cb96a53d93c71e9f912ac243ed40/pr-1195

Backported as #1275

🤖 @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 29, 2022
SUMMARY

Add region detection to skip usage of FIFO topics when using GovCloud regions
Fixes #836
ISSUE TYPE

    Bugfix Pull Request

COMPONENT NAME

community.aws.sns_topic

* * Add endpoint detection to skip usage of FIFO topics for GovCloud regions
* * Remove GovCloud hack for FIFO topics
* Update plugins/modules/sns_topic.py
* Update plugins/modules/sns_topic.py
* minor docs tweak

(cherry picked from commit 2d74eda)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jun 29, 2022
Add missing changelog fragment for 1195

SUMMARY
Forgot to add a changelog fragment for #1195
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
sns_topic
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis <None>
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jun 29, 2022
…in GovCloud - issue 836 (#1275)

[PR #1195/2d74eda7 backport][stable-4] sns_topic - Fix bug when used in GovCloud - issue 836

This is a backport of PR #1195 as merged into main (2d74eda).
SUMMARY
Add region detection to skip usage of FIFO topics when using GovCloud regions
Fixes #836
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
community.aws.sns_topic

Reviewed-by: Mark Chappell <None>
@Razique Razique deleted the issue-836 branch June 29, 2022 12:03
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…1195)

ec2_vpc_endpoint - Remove policy_file parameter

SUMMARY
fixes: ansible-collections#1178
Drops the previously deprecated policy_file parameter.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_vpc_endpoint
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-4 PR should be backported to the stable-4 branch bug This issue/PR relates to a bug community_review mergeit Merge the PR (SoftwareFactory) module module new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

community.aws.sns_topic does not work in GovCloud
3 participants