Skip to content
This repository has been archived by the owner on May 18, 2020. It is now read-only.

Added the TAGS_TO_COPY configuration parameter. #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

rastandy
Copy link

Added the possibility to specify a list of tag keys that must be copied
from the instance to the snapshots created.

Added the possibility to specify a list of tag keys that must be copied
from the instance to the snapshots created.
@Mistawes
Copy link

Mistawes commented Feb 26, 2018

Hey @rastandy - Nice addition you've created here.

I'm just curious, has it been tested and confirmed as working? I keep getting errors such as the following:

  • module.ebs_bckup.var.TAGS_TO_COPY: variable TAGS_TO_COPY in module ebs_bckup should be type list, got string

I've tried adding the following to variable definition, but no luck: (type = "list")

Thanks!

--

EDIT: Knew it, as soon as I asked, I found the fix! I simply set TAGS_TO_COPY to a list of strings, instead of pointing to variables.tf.

Original:
TAGS_TO_COPY = "${join(",", var.TAGS_TO_COPY)}"

Became:
TAGS_TO_COPY = ["Name", "CostCenter", "JIRA"]

@rastandy
Copy link
Author

Hi @Mistawes,

Yes, exactly. The TAGS_TO_COPY variable is supposed to be of type "list".
Guess I should make it explicit in the variable description.

Thanks for having tried it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants