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

[credentialhelper] Respect expires field from helper #21249

Closed
wants to merge 4 commits into from

Conversation

Yannic
Copy link
Contributor

@Yannic Yannic commented Feb 8, 2024

This was recently specified in EngFlow/credential-helper-spec#2.

RELNOTES[NEW]: Bazel now respects expires from Credential Helpers.

This was recently specified in EngFlow/credential-helper-spec#2.

RELNOTES[NEW]: Bazel now repects `expires` from Credential Helpers.
@Yannic Yannic requested a review from a team as a code owner February 8, 2024 00:14
@github-actions github-actions bot added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Feb 8, 2024
@Yannic
Copy link
Contributor Author

Yannic commented Feb 8, 2024

@tjgq PTAL

@tjgq
Copy link
Contributor

tjgq commented Feb 9, 2024

@bazel-io fork 7.1.0

@tjgq tjgq added awaiting-user-response Awaiting a response from the author and removed awaiting-review PR is awaiting review from an assigned reviewer labels Feb 10, 2024
@Yannic
Copy link
Contributor Author

Yannic commented Feb 14, 2024

PTAL

Copy link
Contributor

@tjgq tjgq left a comment

Choose a reason for hiding this comment

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

Thanks!

@tjgq tjgq added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-user-response Awaiting a response from the author labels Feb 15, 2024
@github-actions github-actions bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Feb 19, 2024
bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Feb 19, 2024
This was recently specified in EngFlow/credential-helper-spec#2.

RELNOTES[NEW]: Bazel now respects `expires` from Credential Helpers.

Closes bazelbuild#21249.

PiperOrigin-RevId: 608208538
Change-Id: Id168f654093c7491a40364e3988af66ad1767443
tjgq pushed a commit to tjgq/bazel that referenced this pull request Feb 20, 2024
This was recently specified in EngFlow/credential-helper-spec#2.

RELNOTES[NEW]: Bazel now respects `expires` from Credential Helpers.

Closes bazelbuild#21249.

PiperOrigin-RevId: 608208538
Change-Id: Id168f654093c7491a40364e3988af66ad1767443
github-merge-queue bot pushed a commit that referenced this pull request Feb 20, 2024
This was recently specified in
EngFlow/credential-helper-spec#2.

RELNOTES[NEW]: Bazel now respects `expires` from Credential Helpers.

Closes #21249.

PiperOrigin-RevId: 608208538
Change-Id: Id168f654093c7491a40364e3988af66ad1767443

Co-authored-by: Yannic Bonenberger <[email protected]>
@iancha1992
Copy link
Member

The changes in this PR have been included in Bazel 7.1.0 RC1. Please test out the release candidate and report any issues as soon as possible. If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=last_rc.
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Remote-Exec Issues and PRs for the Execution (Remote) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants