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

Add --lockfile_mode=refresh #22333

Closed
wants to merge 3 commits into from
Closed

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented May 11, 2024

RELNOTES: The new refresh value for --lockfile_mode behaves like the update mode, but additionally forces a refresh of mutable registry content (yanked versions and missing module versions) when switched to or from time to time while enabled.

@fmeum fmeum requested review from a team, Wyverald and meteorcloudy as code owners May 11, 2024 19:36
@fmeum fmeum requested review from katre and removed request for a team May 11, 2024 19:36
@github-actions github-actions bot added team-Configurability Issues for Configurability team team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels May 11, 2024
@fmeum fmeum removed the request for review from katre May 11, 2024 19:39
@fmeum
Copy link
Collaborator Author

fmeum commented May 13, 2024

I added the refresh logic. I didn't find a way to test it that would require making the timeout configurable, but I verified locally that it works with shorter durations.

@fmeum fmeum force-pushed the 20369-refresh branch 2 times, most recently from 9eb0064 to d9bca3c Compare May 13, 2024 20:00
Copy link
Member

@Wyverald Wyverald left a comment

Choose a reason for hiding this comment

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

LGTM. just need some documentation updates etc

@fmeum
Copy link
Collaborator Author

fmeum commented May 14, 2024

@bazel-io fork 7.2.0

@fmeum fmeum removed the team-Configurability Issues for Configurability team label May 14, 2024
@Wyverald Wyverald added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels May 14, 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 May 14, 2024
bazel-io pushed a commit to bazel-io/bazel that referenced this pull request May 14, 2024
RELNOTES: The new `refresh` value for `--lockfile_mode` behaves like the `update` mode, but additionally forces a refresh of mutable registry content (yanked versions and missing module versions) when switched to or from time to time while enabled.

Closes bazelbuild#22333.

PiperOrigin-RevId: 633737372
Change-Id: I2cae552f6adfcd07a5b3263b5e4997e21653f106
github-merge-queue bot pushed a commit that referenced this pull request May 15, 2024
RELNOTES: The new `refresh` value for `--lockfile_mode` behaves like the
`update` mode, but additionally forces a refresh of mutable registry
content (yanked versions and missing module versions) when switched to
or from time to time while enabled.

Closes #22333.

PiperOrigin-RevId: 633737372
Change-Id: I2cae552f6adfcd07a5b3263b5e4997e21653f106

Commit
88a230f

---------

Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
Co-authored-by: Xùdōng Yáng <wyverald@gmail.com>
@fmeum fmeum deleted the 20369-refresh branch May 15, 2024 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants