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

aws2_ec2 inventory: populate ec2_tag_* hostvars #331

Conversation

goneri
Copy link
Member

@goneri goneri commented Apr 16, 2021

This patch exposes a new use_contrib_script_compatible_ec2_tag_keys
that can be used to reproduce a behior of the old ec2.py inventory
script from contrib.

Closes: #183

@ansibullbot
Copy link

@goneri: Greetings! Thanks for taking the time to open this pullrequest. In order for the community to handle your pullrequest effectively, we need a bit more information.

Here are the items we could not find in your description:

  • issue type

Please set the description of this pullrequest with this template:
https://raw.githubusercontent.com/ansible/ansible/devel/.github/PULL_REQUEST_TEMPLATE.md

click here for bot help

@ansibullbot ansibullbot added inventory inventory plugin needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly labels Apr 16, 2021
@goneri goneri force-pushed the aws2_ec2-inventory-populate-ec2_tag_-hostvars_30441 branch from 712fd84 to a13e788 Compare April 16, 2021 18:16
@ansibullbot ansibullbot added needs_triage plugins plugin (any type) labels Apr 16, 2021
Copy link
Contributor

@tremble tremble left a comment

Choose a reason for hiding this comment

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

Hi @goneri,

Integration tests please :)

@goneri
Copy link
Member Author

goneri commented Apr 16, 2021

Hi @goneri,

Integration tests please :)

I'm on it!

@goneri goneri force-pushed the aws2_ec2-inventory-populate-ec2_tag_-hostvars_30441 branch from a13e788 to f002d94 Compare April 17, 2021 03:20
@ansibullbot ansibullbot added integration tests/integration tests tests labels Apr 17, 2021
@goneri goneri force-pushed the aws2_ec2-inventory-populate-ec2_tag_-hostvars_30441 branch from f002d94 to 758d4b2 Compare April 17, 2021 13:36
This patch exposes a new `use_contrib_script_compatible_ec2_tag_keys`
that can be used to reproduce a behior of the old `ec2.py` inventory
script from contrib.

Closes: ansible-collections#183
@goneri goneri force-pushed the aws2_ec2-inventory-populate-ec2_tag_-hostvars_30441 branch from 758d4b2 to e827363 Compare April 19, 2021 12:31
@goneri goneri requested review from tremble and jillr April 19, 2021 13:52
@goneri goneri added the gate label Apr 22, 2021
@ansible-zuul ansible-zuul bot merged commit 1b0b05f into ansible-collections:main Apr 22, 2021
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jun 7, 2022
…_contrib_script_compatible_ec2_tag_keys (#858)

Add version_added: 1.5.0 for exclude_filters, include_filters and use_contrib_script_compatible_ec2_tag_keys

SUMMARY

Add version_added: 1.5.0 for exclude_filters, include_filters and use_contrib_script_compatible_ec2_tag_keys
include_filters and exclude_filters have been added #328 and released with amazon.aws 1.5.0
use_contrib_script_compatible_ec2_tag_keys has been added #331 and released with  amazon.aws 1.5.0
Let's update the aws_ec2 inventory plugin documentation with this information.
This should be a step towards closing this one #676 and #675
aws_ec2 documentation will be enriched with exhaustive examples in an upcoming PR.

ISSUE TYPE


Docs Pull Request

COMPONENT NAME

aws_ec2
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
patchback bot pushed a commit that referenced this pull request Jun 7, 2022
…_contrib_script_compatible_ec2_tag_keys (#858)

Add version_added: 1.5.0 for exclude_filters, include_filters and use_contrib_script_compatible_ec2_tag_keys

SUMMARY

Add version_added: 1.5.0 for exclude_filters, include_filters and use_contrib_script_compatible_ec2_tag_keys
include_filters and exclude_filters have been added #328 and released with amazon.aws 1.5.0
use_contrib_script_compatible_ec2_tag_keys has been added #331 and released with  amazon.aws 1.5.0
Let's update the aws_ec2 inventory plugin documentation with this information.
This should be a step towards closing this one #676 and #675
aws_ec2 documentation will be enriched with exhaustive examples in an upcoming PR.

ISSUE TYPE

Docs Pull Request

COMPONENT NAME

aws_ec2
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
(cherry picked from commit 5f05853)
softwarefactory-project-zuul bot pushed a commit that referenced this pull request Jun 7, 2022
…_contrib_script_compatible_ec2_tag_keys (#858) (#868)

[PR #858/5f058532 backport][stable-3] Add version_added: 1.5.0 for exclude_filters, include_filters and use_contrib_script_compatible_ec2_tag_keys

This is a backport of PR #858 as merged into main (5f05853).
SUMMARY

Add version_added: 1.5.0 for exclude_filters, include_filters and use_contrib_script_compatible_ec2_tag_keys
include_filters and exclude_filters have been added #328 and released with amazon.aws 1.5.0
use_contrib_script_compatible_ec2_tag_keys has been added #331 and released with  amazon.aws 1.5.0
Let's update the aws_ec2 inventory plugin documentation with this information.
This should be a step towards closing this one #676 and #675
aws_ec2 documentation will be enriched with exhaustive examples in an upcoming PR.

ISSUE TYPE


Docs Pull Request

COMPONENT NAME

aws_ec2
ADDITIONAL INFORMATION
jatorcasso pushed a commit to jatorcasso/amazon.aws that referenced this pull request Jun 27, 2022
…_contrib_script_compatible_ec2_tag_keys (ansible-collections#858)

Add version_added: 1.5.0 for exclude_filters, include_filters and use_contrib_script_compatible_ec2_tag_keys

SUMMARY

Add version_added: 1.5.0 for exclude_filters, include_filters and use_contrib_script_compatible_ec2_tag_keys
include_filters and exclude_filters have been added ansible-collections#328 and released with amazon.aws 1.5.0
use_contrib_script_compatible_ec2_tag_keys has been added ansible-collections#331 and released with  amazon.aws 1.5.0
Let's update the aws_ec2 inventory plugin documentation with this information.
This should be a step towards closing this one ansible-collections#676 and ansible-collections#675
aws_ec2 documentation will be enriched with exhaustive examples in an upcoming PR.

ISSUE TYPE


Docs Pull Request

COMPONENT NAME

aws_ec2
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
alinabuzachis pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 16, 2022
ec2_eip tagging support

SUMMARY
Add support for tagging EIPs on creation.
Todo:

 EIP Tagging
 Tests
 Retry decorator

Note: While it's now possible to pass tagging information into the association call this was only added Dec 2020 so won't work for most folks.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_eip
ec2_eip_info
ADDITIONAL INFORMATION
fixes: ansible-collections#331

Reviewed-by: Rick Mendes <None>
Reviewed-by: None <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@02836f3
goneri pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Sep 21, 2022
ec2_eip tagging support

SUMMARY
Add support for tagging EIPs on creation.
Todo:

 EIP Tagging
 Tests
 Retry decorator

Note: While it's now possible to pass tagging information into the association call this was only added Dec 2020 so won't work for most folks.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_eip
ec2_eip_info
ADDITIONAL INFORMATION
fixes: ansible-collections#331

Reviewed-by: Rick Mendes <None>
Reviewed-by: None <None>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@02836f3
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
ec2_eip tagging support

SUMMARY
Add support for tagging EIPs on creation.
Todo:

 EIP Tagging
 Tests
 Retry decorator

Note: While it's now possible to pass tagging information into the association call this was only added Dec 2020 so won't work for most folks.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_eip
ec2_eip_info
ADDITIONAL INFORMATION
fixes: ansible-collections#331

Reviewed-by: Rick Mendes <None>
Reviewed-by: None <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
ec2_eip tagging support

SUMMARY
Add support for tagging EIPs on creation.
Todo:

 EIP Tagging
 Tests
 Retry decorator

Note: While it's now possible to pass tagging information into the association call this was only added Dec 2020 so won't work for most folks.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_eip
ec2_eip_info
ADDITIONAL INFORMATION
fixes: ansible-collections#331

Reviewed-by: Rick Mendes <None>
Reviewed-by: None <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
ec2_eip tagging support

SUMMARY
Add support for tagging EIPs on creation.
Todo:

 EIP Tagging
 Tests
 Retry decorator

Note: While it's now possible to pass tagging information into the association call this was only added Dec 2020 so won't work for most folks.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_eip
ec2_eip_info
ADDITIONAL INFORMATION
fixes: ansible-collections#331

Reviewed-by: Rick Mendes <None>
Reviewed-by: None <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has_issue integration tests/integration inventory inventory plugin needs_info This issue requires further information. Please answer any outstanding questions needs_template This issue/PR has an incomplete description. Please fill in the proposed template correctly plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EC2 instance tags do not populate ec2_tag_* hostvars
5 participants