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

inventory aws_ec2 - allow for literal strings in hostnames #826

Conversation

jatorcasso
Copy link
Contributor

@jatorcasso jatorcasso commented May 23, 2022

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?

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request has_issue integration tests/integration inventory inventory plugin needs_triage plugins plugin (any type) tests tests labels May 23, 2022
@jatorcasso jatorcasso changed the title aws_ec2 - allow for literal strings in hostnames inventory aws_ec2 - allow for literal strings in hostnames May 23, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 49s
✔️ build-ansible-collection SUCCESS in 4m 43s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 8m 55s
✔️ ansible-test-sanity-aws-ansible-2.9-python38 SUCCESS in 12m 57s
✔️ ansible-test-sanity-aws-ansible-2.11-python38 SUCCESS in 9m 40s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 33s
✔️ ansible-test-splitter SUCCESS in 2m 30s
✔️ integration-amazon.aws-1 SUCCESS in 39m 27s
⚠️ 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

@jatorcasso jatorcasso marked this pull request as draft May 24, 2022 10:55
@jatorcasso jatorcasso closed this May 24, 2022
@jatorcasso jatorcasso reopened this May 24, 2022
@jatorcasso jatorcasso marked this pull request as ready for review May 24, 2022 12:05
@jatorcasso jatorcasso requested a review from markuman May 24, 2022 12:37
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 29s
✔️ build-ansible-collection SUCCESS in 4m 39s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 8m 51s
✔️ ansible-test-sanity-aws-ansible-2.9-python38 SUCCESS in 11m 47s
✔️ ansible-test-sanity-aws-ansible-2.11-python38 SUCCESS in 10m 29s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 7m 48s
✔️ ansible-test-splitter SUCCESS in 2m 24s
✔️ integration-amazon.aws-1 SUCCESS in 40m 15s
⚠️ 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 SUCCESS in 48m 45s
✔️ integration-community.aws-2 SUCCESS in 13m 04s
⚠️ 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

Copy link
Member

@markuman markuman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be great if the DOCUMENTATION and EXAMPLE section covers this feature.
Are we planning to release it with 3.3.0? Or Do we wait 2 weeks for 4.0.0?

@jatorcasso
Copy link
Contributor Author

It would be great if the DOCUMENTATION and EXAMPLE section covers this feature.

Are we planning to release it with 3.3.0? Or Do we wait 2 weeks for 4.0.0?

Yeah, that's a good point. I'll add examples and sub options for the hostnames param.

And who/what does the release depend on?

@markuman
Copy link
Member

And who/what does the release depend on?

I think it's a matter of speed :) if it's get ready before next 3.x.x release happen.

@alinabuzachis
Copy link
Collaborator

@markuman @jatorcasso I guess we can include this if we merge by tomorrow.

Copy link
Collaborator

@alinabuzachis alinabuzachis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only thing as @markuman suggested, please add an example. Other than that, LGTM!

@alinabuzachis alinabuzachis added the backport-3 PR should be backported to the stable-3 branch label May 24, 2022
@jatorcasso
Copy link
Contributor Author

jatorcasso commented May 24, 2022

It would be great if the DOCUMENTATION and EXAMPLE section covers this feature. Are we planning to release it with 3.3.0? Or Do we wait 2 weeks for 4.0.0?

Updated docs with suboptions and example - hostnames is a list, but its elements can be either a dict or a string.. wasnt sure how to properly convey that in the docs

@github-actions
Copy link

github-actions bot commented May 24, 2022

Docs Build 📝

Thank you for contribution!✨

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

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-galaxy-importer SUCCESS in 4m 15s
✔️ build-ansible-collection SUCCESS in 4m 55s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 9m 59s
✔️ ansible-test-sanity-aws-ansible-2.9-python38 SUCCESS in 13m 42s
✔️ ansible-test-sanity-aws-ansible-2.11-python38 SUCCESS in 9m 54s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 00s
✔️ ansible-test-splitter SUCCESS in 2m 40s
✔️ integration-amazon.aws-1 SUCCESS in 43m 01s
⚠️ 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 SUCCESS in 50m 26s
✔️ integration-community.aws-2 SUCCESS in 12m 31s
⚠️ 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 3m 50s
✔️ build-ansible-collection SUCCESS in 4m 52s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 10m 49s
✔️ ansible-test-sanity-aws-ansible-2.9-python38 SUCCESS in 12m 55s
✔️ ansible-test-sanity-aws-ansible-2.11-python38 SUCCESS in 9m 59s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 6m 14s
✔️ ansible-test-splitter SUCCESS in 2m 42s
✔️ integration-amazon.aws-1 SUCCESS in 39m 51s
⚠️ 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 944bb1f into ansible-collections:main May 24, 2022
@patchback
Copy link

patchback bot commented May 24, 2022

Backport to stable-3: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-3/944bb1feed6bc6ec8203a7ec9532607e7d51d197/pr-826

Backported as #836

🤖 @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 May 24, 2022
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)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request May 25, 2022
[PR #826/944bb1fe backport][stable-3] inventory aws_ec2 - allow for literal strings in hostnames

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?

Reviewed-by: Joseph Torcasso <None>
Reviewed-by: Jill R <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…ansible-collections#826)

Respect wait parameter in elb_instance when adding/removing instances

SUMMARY
The wait parameter is currently ignored when registering or
deregistering an instance with an ELB. Looks like this was lost in the
boto3 migration: 96f1518
Related: ansible-collections#825
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
elb_instance
ADDITIONAL INFORMATION
See ansible-collections#825

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <[email protected]>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
…ansible-collections#826)

Respect wait parameter in elb_instance when adding/removing instances

SUMMARY
The wait parameter is currently ignored when registering or
deregistering an instance with an ELB. Looks like this was lost in the
boto3 migration: 96f1518
Related: ansible-collections#825
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
elb_instance
ADDITIONAL INFORMATION
See ansible-collections#825

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <[email protected]>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
…ansible-collections#826)

Respect wait parameter in elb_instance when adding/removing instances

SUMMARY
The wait parameter is currently ignored when registering or
deregistering an instance with an ELB. Looks like this was lost in the
boto3 migration: 96f1518
Related: ansible-collections#825
ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
elb_instance
ADDITIONAL INFORMATION
See ansible-collections#825

Reviewed-by: Alina Buzachis <None>
Reviewed-by: Markus Bergholz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-3 PR should be backported to the stable-3 branch community_review feature This issue/PR relates to a feature request has_issue integration tests/integration inventory inventory plugin mergeit Merge the PR (SoftwareFactory) needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Composing inventory_hostname manually
5 participants