Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

temporary disable undefined-symbol test #1007

Merged

Conversation

cperkinsintel
Copy link

Because of problems getting the GPU driver uplifted on GitHub Actions, this test both fails on GHA and passes on Jenkins, which is just confusing for everyone. Disabling temporarily.

Signed-off-by: Chris Perkins [email protected]

…the GPU driver uplifted on GitHub Actions, this test both fails on GHA and passes on Jenkins, which is just confusing for everyone. Disabling temporarily.

Signed-off-by: Chris Perkins <[email protected]>
@cperkinsintel cperkinsintel requested a review from a team as a code owner April 22, 2022 23:43
@cperkinsintel cperkinsintel requested a review from againull April 22, 2022 23:43
// because of problems getting GPU driver uplifted on GitHub Actions
// this test both fails on GHA and passes on Jenkins.
// Disabling it until this is sorted
// UNSUPPORTED: windows || linux

Choose a reason for hiding this comment

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

Probably it would be better to use // REQUIRES: TEMPORARILY_DISABLED, so that it is more clear that this test is temporarily disabled.

Copy link
Author

Choose a reason for hiding this comment

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

Is that how we normally do it? That's a good BKM.

Choose a reason for hiding this comment

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

Yes. It also allows to grep for TEMPORARILY_DISABLED and see our debts.

@againull againull merged commit 0cedbae into intel:intel Apr 24, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Jun 17, 2022
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants