Skip to content

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 platforms, toolchains, cquery, select(), config transitions 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 fmeum requested a review from Wyverald May 14, 2024 10:03
@fmeum
Copy link
Collaborator Author

fmeum commented May 14, 2024

@bazel-io fork 7.2.0

@fmeum fmeum removed the team-Configurability platforms, toolchains, cquery, select(), config transitions 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 <[email protected]>
Co-authored-by: Xùdōng Yáng <[email protected]>
@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.

2 participants