multi-cluster-app-dispatcher icon indicating copy to clipboard operation
multi-cluster-app-dispatcher copied to clipboard

Skipping MCAD CPU Preemption Test

Open Fiona-Waters opened this issue 1 year ago • 5 comments

Skipping the MCAD CPU Preemption Test which is failing intermittently on PRs so that we can get some outstanding PRs merged.

Fiona-Waters avatar Dec 07 '23 13:12 Fiona-Waters

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign anishasthana for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci[bot] avatar Dec 07 '23 13:12 openshift-ci[bot]

For future reference, the root cause analysis of the test's failure has been conducted by @dgrove-oss, and it can be found here: https://github.com/project-codeflare/multi-cluster-app-dispatcher/pull/691#issuecomment-1832070420

ronensc avatar Dec 07 '23 15:12 ronensc

For future reference, the root cause analysis of the test's failure has been conducted by @dgrove-oss, and it can be found here: #691 (comment)

Thanks @ronensc That's good to know!

Fiona-Waters avatar Dec 08 '23 09:12 Fiona-Waters

I don't think it's worth backporting, but I did redo these test cases for mcad v2 to be robust against different cluster sizes in https://github.com/project-codeflare/mcad/pull/83

dgrove-oss avatar Dec 08 '23 14:12 dgrove-oss

More investigation is required as to why these tests are failing. Closing this PR.

Fiona-Waters avatar Dec 11 '23 14:12 Fiona-Waters