Skip to content

Conversation

@patchback
Copy link

@patchback patchback bot commented May 24, 2022

This is a backport of PR #826 as merged into main (944bb1f).

SUMMARY

Allow for literal strings in hostnames rather than throwing an AnsibleError

Fixes #814

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

aws_ec2

ADDITIONAL INFORMATION

Is there a better way of allowing for literal strings here?

inventory aws_ec2 - allow for literal strings in hostnames

SUMMARY
Allow for literal strings in hostnames rather than throwing an AnsibleError
Fixes #814
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
aws_ec2
ADDITIONAL INFORMATION
Is there a better way of allowing for literal strings here?

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Jill R <None>
(cherry picked from commit 944bb1f)
@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration inventory inventory plugin needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels May 24, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 3m 55s
✔️ build-ansible-collection SUCCESS in 5m 03s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 09s
✔️ ansible-test-sanity-aws-ansible-2.9-python38 SUCCESS in 10m 54s
✔️ ansible-test-sanity-aws-ansible-2.11-python38 SUCCESS in 9m 02s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 5m 27s
✔️ ansible-test-splitter SUCCESS in 2m 27s
✔️ integration-amazon.aws-1 SUCCESS in 46m 32s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
⚠️ integration-community.aws-1 SKIPPED
⚠️ 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

@jillr jillr added the mergeit Merge the PR (SoftwareFactory) label May 24, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-galaxy-importer SUCCESS in 4m 29s
✔️ build-ansible-collection SUCCESS in 5m 09s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 8m 43s
✔️ ansible-test-sanity-aws-ansible-2.9-python38 SUCCESS in 11m 28s
✔️ ansible-test-sanity-aws-ansible-2.11-python38 SUCCESS in 10m 27s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 10s
✔️ ansible-test-splitter SUCCESS in 2m 22s
✔️ integration-amazon.aws-1 SUCCESS in 41m 52s
⚠️ integration-amazon.aws-2 SKIPPED
⚠️ integration-amazon.aws-3 SKIPPED
⚠️ integration-amazon.aws-4 SKIPPED
⚠️ integration-amazon.aws-5 SKIPPED
⚠️ integration-amazon.aws-6 SKIPPED
⚠️ integration-amazon.aws-7 SKIPPED
⚠️ integration-amazon.aws-8 SKIPPED
⚠️ integration-amazon.aws-9 SKIPPED
⚠️ integration-amazon.aws-10 SKIPPED
⚠️ integration-amazon.aws-11 SKIPPED
⚠️ integration-amazon.aws-12 SKIPPED
⚠️ integration-amazon.aws-13 SKIPPED
⚠️ integration-community.aws-1 SKIPPED
⚠️ 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 7439128 into stable-3 May 25, 2022
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-3/944bb1feed6bc6ec8203a7ec9532607e7d51d197/pr-826 branch May 25, 2022 00:19
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…tions#1195)

SUMMARY

Add region detection to skip usage of FIFO topics when using GovCloud regions
Fixes ansible-collections#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
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…tions#1195)

SUMMARY

Add region detection to skip usage of FIFO topics when using GovCloud regions
Fixes ansible-collections#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
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…tions#1195)

SUMMARY

Add region detection to skip usage of FIFO topics when using GovCloud regions
Fixes ansible-collections#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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community_review feature This issue/PR relates to a feature request integration tests/integration inventory inventory plugin mergeit Merge the PR (SoftwareFactory) needs_triage 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.

4 participants