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

Testing updates #5

Merged
merged 1 commit into from
May 31, 2022
Merged

Testing updates #5

merged 1 commit into from
May 31, 2022

Conversation

troyready
Copy link
Contributor

A few testing updates:

  • Most notably, there's an AWS cleanup bug that can sometimes cause the terraform apply/destroy cycle to fail (this isn't specific to the automated setup or running on CI). Now the tests will watch for those lingering ENIs & Security Groups and purge them during the destroy process.
  • When running locally (not in CI), the test script will now ensure that the Terraform workspace name is persisted to disk. This should be useful in debugging sessions -- now one can just switch to the relevant module directory and run terraform commands without selecting the workspace again.
  • Finally, dropped legacy terraform-credentials-env plugin use now that tests only use TF 1.2+

@troyready troyready requested a review from a team May 26, 2022 23:08
@troyready troyready merged commit 9ab6549 into main May 31, 2022
@troyready troyready deleted the eni_deletion_workaround branch May 31, 2022 18:28
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