Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Bazel CI] Build failure on macOS with error: symbolic_macro_inherit_attrs_bazel_8_test #280

Closed
sgowroji opened this issue Feb 10, 2025 · 2 comments · Fixed by #281
Closed

[Bazel CI] Build failure on macOS with error: symbolic_macro_inherit_attrs_bazel_8_test #280

sgowroji opened this issue Feb 10, 2025 · 2 comments · Fixed by #281
Assignees
@tetromino
Labels
P2 type: bug

Comments

@sgowroji
Copy link
Member

CI: bazelbuild/intellij#7232

Platform: MacOS

Logs:


FAILED: �[0m//test:symbolic_macro_inherit_attrs_bazel_8_test (Summary)

Culprit:

Steps:

git clone https://github.com/bazelbuild/stardoc
git reset a9fba1abf94a116514d1517caadcd1b469400328  --hard
export USE_BAZEL_VERSION=f6cb2fb25c1deee9776eda976be151c598597b35
bazel test -- //... //test:symbolic_macro_inherit_attrs_bazel_8_test

CC Greenteam @comius
@tetromino
Copy link
Collaborator

Why are you running //test:symbolic_macro_inherit_attrs_bazel_8_test with Bazel 9 pre-release?

The test is explicitly tagged as manual (because it produces different results in Bazel 8 and 9) and even has "bazel_8_test" in its name :)

@tetromino
Copy link
Collaborator

Ah, I see the problem: ffbada6 added "//test:symbolic_macro_inherit_attrs_bazel_8_test" to common_task_config, but didn't add a corresponding skip_in_bazel_downstream_pipeline key to affected configs.

@tetromino tetromino self-assigned this Feb 10, 2025
tetromino added a commit to tetromino/stardoc that referenced this issue Feb 10, 2025
@tetromino
Ensure test configs with symbolic_macro_inherit_attrs_bazel_8_test ar…
7d8c8b3 
…e marked skip_in_bazel_downstream_pipeline

As the name says, symbolic_macro_inherit_attrs_bazel_8_test requires stable Bazel.

Fixes bazelbuild#280
@tetromino tetromino mentioned this issue Feb 10, 2025
Merged
tetromino added a commit that referenced this issue Feb 11, 2025
@tetromino
Ensure test configs with symbolic_macro_inherit_attrs_bazel_8_test ar…
d45a141 
…e marked skip_in_bazel_downstream_pipeline (#281)

As the name says, symbolic_macro_inherit_attrs_bazel_8_test requires stable Bazel.

Fixes #280
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tetromino tetromino

Labels
P2 type: bug
Projects
None yet
Milestone
No milestone
2 participants
@tetromino @sgowroji