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

fix integration-cleanup.sh to match prefix only #3887

Closed
wants to merge 1 commit into from
Closed

fix integration-cleanup.sh to match prefix only #3887

wants to merge 1 commit into from

Conversation

stefanwire
Copy link
Contributor

The -f filter is a regex and should match the prefix test-, thus,
the regex should be ^test-. Without ^, the search string is looked
up in the entire release name.

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@stefanwire
fix integration-cleanup.sh to match prefix only (#3885)
b4c6542 
The `-f` filter is a regex and should match the prefix `test-`, thus,
the regex should be `^test-`. Without `^`, the search string is looked
up in the entire release name.
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Feb 13, 2024
@stefanwire
Copy link
Contributor Author

superseded by #3888

@stefanwire stefanwire closed this Feb 13, 2024
@stefanwire stefanwire deleted the sb/integration-cleanup.sh branch February 13, 2024 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stefanwire @zebot