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

[Bedrock] Update Bedrock Integration tests to have terraform commands and terratest script separate #985

Open
3 tasks
Tracked by #964
NathanielRose opened this issue Feb 12, 2020 · 2 comments
Labels
effort - M less than one week effort enhancement New feature or request

Comments

@NathanielRose
Copy link
Contributor

As a: Developer

I want: Decoupling of Terraform commands from the Bedrock terratest integration scripts

So that: I can know when an integration test fails during deployment of resources, interoperability of resources, and deletion of resources.

Describe the solution you'd like: Terratest scripts should no longer directly call terraform commands, this should be handled by the pipeline yaml in separate steps. In addition, variable configurations should be passed between steps from successful terraform apply to the terratest validation.

Acceptance Criteria:

  • Decouple terraform init, plan, apply and destroy to separate steps in a template pipeline
  • Pass terraform configuration once successfully applied to terratest script
  • Build logic that surfaces cloud shell failures vs script failures

Additional context:
Task of #964

Does this require updates to documentation?:
Yes

@NathanielRose
Copy link
Contributor Author

Steps:

  • Export Env Vars to map
  • Create a Resource Group
  • Copy to temporary infra folder
  • Export map with TF config
  • Terraform init
  • Terraform plan
  • Terraform Apply
  • Run terratest integration tests
  • Delete resources

@bnookala bnookala removed the triage label Mar 11, 2020
@NathanielRose NathanielRose removed their assignment May 27, 2020
@NathanielRose
Copy link
Contributor Author

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

No branches or pull requests

2 participants