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

Build test infrastructure #8

Closed
timfpark opened this issue Nov 22, 2018 · 5 comments · Fixed by #131
Closed

Build test infrastructure #8

timfpark opened this issue Nov 22, 2018 · 5 comments · Fixed by #131
Assignees
Labels
effort - M less than one week effort infra-team

Comments

@timfpark
Copy link
Contributor

timfpark commented Nov 22, 2018

We should be able to validate that the project works end to end before and after each commit with a CI system.

Some ideas on how to test a Terraform deployment here: https://devops.stackexchange.com/questions/863/how-to-test-a-terraform-configuration

@jmspring jmspring added this to the Simplify Bedrock repository to support infrastructure deployment only milestone Jan 14, 2019
@NathanielRose NathanielRose self-assigned this Jan 18, 2019
@NathanielRose NathanielRose added the effort - M less than one week effort label Jan 22, 2019
@NathanielRose
Copy link
Contributor

NathanielRose commented Jan 23, 2019

Investigate Terratest

  • Create Simple test with terratest + Azure
  • Create AKS integration test with terratest
  • Init integration with Bedrock
  • Azure Helpers #89
  • Templated infrastructure tests (AKS Get Credentials | Ensure Flux Pods)
  • Embedding Bedrock Integration Terratests into a CI [Travis CI | Jenkins | GitLab ]
  • Documentation

@jmspring
Copy link
Contributor

jmspring commented Jan 28, 2019

Request from Nate -

We would like to determine a minimal set of initial integration tests for the testing of infrastructure deployment. Two identified tests are:

  • AKS get credentials
  • Ensure Flux pods running (kubectl get po --namespace=flux, make sure pods running)

@andrebriggs, @evanlouie, @bnookala - any thoughts on additional tests

@timfpark asked that the above integrated into TravisCI

@NathanielRose
Copy link
Contributor

NathanielRose commented Jan 30, 2019

Confirmed Integration Test

  • AKS get credentials
  • Ensure Flux pods running (kubectl get po --namespace=flux, make sure pods running)
  • Log Analytics add on running

@andrebriggs
Copy link
Member

@NathanielRose I could see some intersection. We want to make sure our code is tested before any merges to master. I'll schedule time to chat about this

@drmmarsunited
Copy link

@NathanielRose - way late to anything in this issue, but I noticed it was linking to a Terratest issue that I'd worked on. Foundational authentication mechanisms for an Azure module were rolled in a little while ago via gruntwork-io/terratest#333.

More helpers to come in the Azure module, but the groundwork is there moving forward.

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 infra-team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants