Skip to content

[AIX] Disable flaky ClangScanDeps tests #72304

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.

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

Merged
merged 2 commits into from
Nov 14, 2023
Merged

Conversation

jakeegan
Copy link
Member

These tests are failing intermittently on the AIX bot, possibly due to some issue with getpwuid_r. Disable them for now while they are investigated.

@jakeegan jakeegan self-assigned this Nov 14, 2023
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Nov 14, 2023
@llvmbot
Copy link
Member

llvmbot commented Nov 14, 2023

@llvm/pr-subscribers-clang

Author: Jake Egan (jakeegan)

Changes

These tests are failing intermittently on the AIX bot, possibly due to some issue with getpwuid_r. Disable them for now while they are investigated.


Full diff: https://github.com/llvm/llvm-project/pull/72304.diff

2 Files Affected:

  • (modified) clang/test/ClangScanDeps/regular_cdb.cpp (+2)
  • (modified) clang/test/ClangScanDeps/relative_directory.cpp (+2)
diff --git a/clang/test/ClangScanDeps/regular_cdb.cpp b/clang/test/ClangScanDeps/regular_cdb.cpp
index 2d7edc8c34d5a07..1972c6700d37f55 100644
--- a/clang/test/ClangScanDeps/regular_cdb.cpp
+++ b/clang/test/ClangScanDeps/regular_cdb.cpp
@@ -1,3 +1,5 @@
+// UNSUPPORTED:  target={{.*}}-aix{{.*}}
+
 // RUN: rm -rf %t.dir
 // RUN: rm -rf %t.cdb
 // RUN: rm -rf %t_clangcl.cdb
diff --git a/clang/test/ClangScanDeps/relative_directory.cpp b/clang/test/ClangScanDeps/relative_directory.cpp
index bf5cf91a2f7b6be..495867467f6e114 100644
--- a/clang/test/ClangScanDeps/relative_directory.cpp
+++ b/clang/test/ClangScanDeps/relative_directory.cpp
@@ -1,3 +1,5 @@
+// UNSUPPORTED:  target={{.*}}-aix{{.*}}
+
 // RUN: rm -rf %t.dir
 // RUN: rm -rf %t.cdb
 // RUN: mkdir -p %t.dir

Copy link
Contributor

@qiongsiwu qiongsiwu left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks Jake!

@jakeegan jakeegan merged commit 2c2f785 into llvm:main Nov 14, 2023
@jakeegan jakeegan deleted the flakey_tests branch November 14, 2023 22:40
jakeegan added a commit that referenced this pull request Nov 16, 2023
sr-tream pushed a commit to sr-tream/llvm-project that referenced this pull request Nov 20, 2023
zahiraam pushed a commit to zahiraam/llvm-project that referenced this pull request Nov 20, 2023
These tests are failing intermittently on the AIX bot, possibly due to
some issue with `getpwuid_r`. Disable them for now while they are
investigated.
zahiraam pushed a commit to zahiraam/llvm-project that referenced this pull request Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants