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.

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

[8.0.0] Match --worker_extra_flag against the worker key mnemonic, not the action mnemonic, as is the case for other worker flags taking a mnemonic. #24251

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

bazel-io
Copy link
Member

@bazel-io bazel-io commented Nov 8, 2024

Fixes #24237.

RELNOTES[INC]: The mnemonic passed to --worker_extra_flag is now matched against the worker key mnemonic when one is available, instead of the action mnemonic. This makes it consistent with other worker flags taking a mnemonic.

PiperOrigin-RevId: 694436563
Change-Id: I7c6e6ede5e3ef2f53aeb847bbfe173042c528d68

Commit 8357fba

…tion mnemonic, as is the case for other worker flags taking a mnemonic.

Fixes bazelbuild#24237.

RELNOTES[INC]: The mnemonic passed to --worker_extra_flag is now matched against the worker key mnemonic when one is available, instead of the action mnemonic. This makes it consistent with other worker flags taking a mnemonic.

PiperOrigin-RevId: 694436563
Change-Id: I7c6e6ede5e3ef2f53aeb847bbfe173042c528d68
@bazel-io bazel-io requested a review from a team as a code owner November 8, 2024 11:49
@bazel-io bazel-io added team-Local-Exec Issues and PRs for the Execution (Local) team awaiting-review PR is awaiting review from an assigned reviewer labels Nov 8, 2024
@iancha1992 iancha1992 added this pull request to the merge queue Nov 8, 2024
Merged via the queue into bazelbuild:release-8.0.0 with commit 0a7fd4b Nov 8, 2024
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Local-Exec Issues and PRs for the Execution (Local) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants