ci-operator: Introduce a dedicated prowjob for running OLM's psm unit tests#27305
Conversation
|
Looks like the rehearsals worked: |
exdx
left a comment
There was a problem hiding this comment.
Do these changes also incorporate changes to OLM to not run the PSM unit tests as part of it's own unit suite run? Just making sure we aren't duplicating running the same tests twice in CI
|
@exdx Nope - that's needs to be done as a follow-up (I'm tracking this locally) once the upstream sync lands and CI is a bit more stable. |
|
Ok cool -- let's do it as a followup |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: exdx, timflannagan The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
… tests Signed-off-by: timflannagan <timflannagan@gmail.com>
d4d7952 to
63e4078
Compare
|
/lgtm |
|
@timflannagan: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
The PSM is a sub-component of OLM that was introduced in 4.9. These unit tests are currently run by through the unit/olm prowjob as an original hack to get CI coverage without having to introduce a new prowjob, but we never fixed that hack after the PSM work was merged.
Signed-off-by: timflannagan timflannagan@gmail.com