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

Ability to toggle off leading_separator for keyed_groups #402

Closed
1 task done
hs-amartin opened this issue Jul 12, 2021 · 3 comments
Closed
1 task done

Ability to toggle off leading_separator for keyed_groups #402

hs-amartin opened this issue Jul 12, 2021 · 3 comments
Labels
feature This issue/PR relates to a feature request inventory inventory plugin needs_triage plugins plugin (any type)

Comments

@hs-amartin
Copy link

Summary

I am in the middle of migrating from a manually managed inventory file to a generated one, and the fact that I can't omit the _ even if I don't specify a prefix means I cannot exactly replicate my existing group structure. Also leading underscores are kind of weird.

Issue Type

Feature Idea

Component Name

keyed_groups

Additional Information

plugin: aws_ec2
boto_profile: account7-profile
regions:
  - us-east-1
keyed_groups:
  - key: tags['Name']|regex_replace("-account7-.*","")

Code of Conduct

  • I agree to follow the Ansible Code of Conduct
@ansibullbot
Copy link

Files identified in the description:

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

@ansibullbot ansibullbot added feature This issue/PR relates to a feature request inventory inventory plugin needs_triage plugins plugin (any type) labels Jul 12, 2021
@s-hertel
Copy link
Collaborator

s-hertel commented Jul 12, 2021

There is a leading_separator toggle available in ansible-core 2.11.

plugin: aws_ec2
boto_profile: account7-profile
regions:
  - us-east-1
leading_separator: False
keyed_groups:
  - key: tags['Name']|regex_replace("-account7-.*","")

@hs-amartin
Copy link
Author

Ah, I'm on 2.9, didn't realize this was added

abikouo pushed a commit to abikouo/amazon.aws that referenced this issue Sep 18, 2023
* Fix parameter validation in ecs_task
* Require cluster parameter in ecs_task module
* Move parameter validation to AnsibleAWSModule
* Fix pep8 formatting line too long
* changelog
abikouo pushed a commit to abikouo/amazon.aws that referenced this issue Sep 18, 2023
* Fix parameter validation in ecs_task
* Require cluster parameter in ecs_task module
* Move parameter validation to AnsibleAWSModule
* Fix pep8 formatting line too long
* changelog
abikouo pushed a commit to abikouo/amazon.aws that referenced this issue Oct 24, 2023
* Fix parameter validation in ecs_task
* Require cluster parameter in ecs_task module
* Move parameter validation to AnsibleAWSModule
* Fix pep8 formatting line too long
* changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue/PR relates to a feature request inventory inventory plugin needs_triage plugins plugin (any type)
Projects
None yet
Development

No branches or pull requests

3 participants