Skip to content
This repository has been archived by the owner on Feb 15, 2022. It is now read-only.

Provide support for passing backend remote state using spk infra generate #728

Closed
2 tasks done
NathanielRose opened this issue Oct 25, 2019 · 7 comments · Fixed by CatalystCode/spk#159
Closed
2 tasks done
Assignees
Labels
effort - M less than one week effort enhancement New feature or request

Comments

@NathanielRose
Copy link
Contributor

NathanielRose commented Oct 25, 2019

As a:
Developer

I want:
spk infra generate to recursively apply remote state configurations for tfstate files from parent directory to sub-directories for terraform deployment

So that:
at the final sub-directory, it creates a generated directory and fills the backend tfvars using the backend state variables accumulated from parent and child levels.

Describe the solution you'd like:
Upon generation, spk infra generate should:

  • Check in (parent) current working directory and verifies if it contains a definition.json:
  • If so, check the definition for any specification of a backend state. Then, iterate through child directory and create a backend.tfvars file

Acceptance Criteria:

When a user runs spk infra generate east for example, this should read in the backend configuration in the definition.json of the (parent) current working directory, and then apply the backend configuration of the definition.json of the east cluster. It creates a generated directory, with a backend.tfvars with the accumulated variables.

Does this require updates to documentation?:
Yes.

@NathanielRose NathanielRose added enhancement New feature or request triage labels Oct 25, 2019
@NathanielRose NathanielRose added the effort - M less than one week effort label Oct 25, 2019
@andrebriggs
Copy link
Member

@yradsmikham what the situation with this?

@yradsmikham
Copy link
Contributor

yradsmikham commented Nov 13, 2019

@andrebriggs, I plan on working on this later this week. The description for this issue is accurate.

@yradsmikham yradsmikham self-assigned this Nov 13, 2019
@andrebriggs
Copy link
Member

@NathanielRose do you agree with the description? cc @yradsmikham

@NathanielRose
Copy link
Contributor Author

@andrebriggs Yes I agree with the description, I was initially unsure whether or not this functionality was in the new Iteration II PR.

@yradsmikham
Copy link
Contributor

@andrebriggs @NathanielRose

If we are all in agreement here, I can start working on this issue at some point today or tomorrow. Thanks y'all! :)

@NathanielRose
Copy link
Contributor Author

Hey, I would like to take this on, But I am realizing due to issues I may be experiencing with the new addition of spk infra generate that I mentioned in (CatalystCode/spk#132), that this may be blocked. Since we have a day left, this may need to be pushed to another sprint. Thoughts?

@yradsmikham
Copy link
Contributor

@NathanielRose, you're welcome to work on this! I also agree that this item could be pushed to the next sprint given the amount of the time left in our current sprint.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
effort - M less than one week effort enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants