Skip to content

Pick a non-conflicting env name for fips bats#2693

Merged
selzoc merged 1 commit intomainfrom
better-namespace-bats-fips-resources
Mar 13, 2026
Merged

Pick a non-conflicting env name for fips bats#2693
selzoc merged 1 commit intomainfrom
better-namespace-bats-fips-resources

Conversation

@selzoc
Copy link
Copy Markdown
Member

@selzoc selzoc commented Mar 13, 2026

Leftovers uses substring matching to decide which resources to destroy, so we CANNOT share any substrings involved in cleanup with ANY other resources.

Using just "fips" lead to the unintentional deletion of several buckets not related to the fips bats tests:
https://bosh.ci.cloudfoundry.org/teams/main/pipelines/bosh-director/jobs/bats-fips-cleanup-leftovers/builds/468#L69a9299f:936:937

Leftovers uses substring matching to decide which resources to destroy,
so we CANNOT share any substrings involved in cleanup with ANY other
resources.

Using just "fips" lead to the unintentional deletion of several buckets
not related to the fips bats tests:
https://bosh.ci.cloudfoundry.org/teams/main/pipelines/bosh-director/jobs/bats-fips-cleanup-leftovers/builds/468#L69a9299f:936:937

ai-assisted=yes
@github-project-automation github-project-automation bot moved this from Inbox to Pending Merge | Prioritized in Foundational Infrastructure Working Group Mar 13, 2026
@selzoc selzoc merged commit 818dba3 into main Mar 13, 2026
19 checks passed
@github-project-automation github-project-automation bot moved this from Pending Merge | Prioritized to Done in Foundational Infrastructure Working Group Mar 13, 2026
@selzoc selzoc deleted the better-namespace-bats-fips-resources branch March 13, 2026 16:28
@selzoc
Copy link
Copy Markdown
Member Author

selzoc commented Mar 13, 2026

Flew the pipeline as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

4 participants