openshift / openshift/oadp-operator
Rethink CI regex
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 92
- Forks
- 93
- Avg merge
- 1d 23h
- Merged PRs (30d)
- 43
Description
We need to rethink the trigger regex for E2E tests in our CI. This simple PR https://github.com/openshift/oadp-operator/pull/1239 is taking to long to merge because of it.
Today, our regex is
https://github.com/openshift/release/blob/master/ci-operator/config/openshift/oadp-operator/openshift-oadp-operator-master__4.14.yaml#L34
Instead of triggering based of file types, we should think what folders are important for the operator.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the trigger regex at ci-operator/config/openshift/oadp-operator/openshift-oadp-operator-master__4.14.yaml#L34 and review PR 1239 to understand the current CI impact. Identify which operator folders should trigger E2E tests instead of relying on file types, and verify that the revised scope resolves the reported PR delay.
Written by the indexing model from the issue text.
Assessment
- Domain
- build-system, ci-cd
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100