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

Add a toggle for the leading separator for keyed_groups #60882

Open
wants to merge 5 commits into
base: devel
from

Conversation

@s-hertel
Copy link
Contributor

commented Aug 19, 2019

SUMMARY

supersedes #57164
Fixes #54395

keyed_groups:
  - key: tags
  - key: vpc_id
    prefix: ""
    separator: "_"

On devel that creates group names like _tagkey_tagvalue and _vpc_xxxxxxxx.

On this branch with the toggle set to false those groups are tagkey_tagvalue and vpc_xxxxxxxx. Note that the tags host var is a dictionary, so the separator is still used to concatenate the key value pairs.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

constructed

@s-hertel s-hertel force-pushed the s-hertel:keyed_groups_leading_separator branch to d3578f7 Aug 19, 2019

@s-hertel s-hertel force-pushed the s-hertel:keyed_groups_leading_separator branch to 5a7073e Aug 19, 2019

@s-hertel s-hertel force-pushed the s-hertel:keyed_groups_leading_separator branch to 5d0398e Aug 20, 2019

@ansibot ansibot added the stale_ci label Aug 28, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.