Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Limit e2e test environment age to improve test stability #302

Open
hessjcg opened this issue Apr 12, 2023 · 0 comments
Open

Limit e2e test environment age to improve test stability #302

hessjcg opened this issue Apr 12, 2023 · 0 comments
Assignees
@hessjcg
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: cleanup An internal cleanup or hygiene concern.

Comments

@hessjcg
Copy link
Collaborator

hessjcg commented Apr 12, 2023

Ensure that the CI environments for e2e tests are only valid for 1 week. Then they are deleted and recreated
fresh. This will help ensure that the environments remain stable.

  • Update naming of clusters, dbs, vpcs etc to include the validity date, so that there are no naming conflicts.
  • Create a cleanup job that will delete old environments without using terraform. (Terraform is slow and unreliable for env teardown.)
@hessjcg hessjcg added the type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. label Apr 12, 2023
@hessjcg hessjcg added the priority: p2 Moderately-important priority. Fix may not be included in next release. label Apr 12, 2023
@jackwotherspoon jackwotherspoon mentioned this issue Apr 12, 2023
Merged
hessjcg added a commit that referenced this issue Apr 12, 2023
@hessjcg
ci: change name of ci environment to 'ci-branch-main' to avoid terraf…
cf01c26 
…orm issues (#301)

Terraform has a hard time tearing down and recreating whole environments when they are
too far out of date. Instead, this starts a brand new environment named 'ci-branch-main'.

Changing name as short-term fix, long-term fix we will want to dynamically change the 
env name.  This work will be tracked by #302
@ttosta-google ttosta-google added type: cleanup An internal cleanup or hygiene concern. and removed type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. labels Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hessjcg hessjcg

Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: cleanup An internal cleanup or hygiene concern.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ttosta-google @hessjcg