Skip to content

feat: multiple cleanup_name values#233

Merged
DerekRoberts merged 4 commits intomainfrom
feat/cleanup-name-multi-value
Mar 4, 2026
Merged

feat: multiple cleanup_name values#233
DerekRoberts merged 4 commits intomainfrom
feat/cleanup-name-multi-value

Conversation

@DerekRoberts
Copy link
Member

@DerekRoberts DerekRoberts commented Mar 3, 2026

Summary

Allow cleanup_name to accept newline-separated strings so multiple label-based cleanup passes can run in a single PR close workflow.

Changes

  • Updated cleanup_name description to document newline-separated multi-value support
  • OC Template (label) Cleanup step now loops over each name, matching the existing remove_pvc pattern

Backwards Compatible

Single-value callers are unaffected — the loop runs once, same behaviour as before.


Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

Copilot AI review requested due to automatic review settings March 3, 2026 21:28
@DerekRoberts DerekRoberts requested a review from mishraomp as a code owner March 3, 2026 21:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the PR-close GitHub Actions workflow to allow cleanup_name to be specified as multiple newline-separated values, enabling multiple label-based OpenShift cleanup passes in a single run.

Changes:

  • Documented that cleanup_name supports newline-separated multi-value input.
  • Updated the OpenShift label cleanup step to iterate over each cleanup_name line and run oc delete per name.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@DerekRoberts DerekRoberts force-pushed the feat/cleanup-name-multi-value branch from 3905769 to af58392 Compare March 3, 2026 21:35
@DerekRoberts DerekRoberts self-assigned this Mar 3, 2026
@DerekRoberts DerekRoberts force-pushed the feat/cleanup-name-multi-value branch 4 times, most recently from 0575533 to d51dd4b Compare March 3, 2026 21:55
Allow cleanup_name to accept space-separated strings so multiple
label-based cleanup passes can run in a single PR close workflow.

When cleanup_name is not provided (empty default), falls back to
the original behavior using the repository name.
@DerekRoberts DerekRoberts force-pushed the feat/cleanup-name-multi-value branch from d51dd4b to 10a32c3 Compare March 3, 2026 21:57
@DerekRoberts DerekRoberts changed the title feat(pr-close): support multiple cleanup_name values feat: multiple cleanup_name values Mar 3, 2026
@DerekRoberts DerekRoberts moved this from New to Active in DevOps (NR) Mar 3, 2026
@DerekRoberts DerekRoberts merged commit ca6579b into main Mar 4, 2026
22 checks passed
@DerekRoberts DerekRoberts deleted the feat/cleanup-name-multi-value branch March 4, 2026 01:02
@github-project-automation github-project-automation bot moved this from Active to Done in DevOps (NR) Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants