Ensure lockfile is updated after reset to pre-build state#19343
Closed
fmeum wants to merge 2 commits intobazelbuild:masterfrom
Closed
Ensure lockfile is updated after reset to pre-build state#19343fmeum wants to merge 2 commits intobazelbuild:masterfrom
fmeum wants to merge 2 commits intobazelbuild:masterfrom
Conversation
The events that update the lockfile are important for incremental correctness and thus need to return `true` from `storeForReplay`. Before this change, if a build creates the lockfile because it didn't exist and the lockfile is then deleted, subsequent builds did not regenerate it as the relevant SkyFunctions wouldn't rerun and the events were not replayed.
251cb5a to
72d869f
Compare
Collaborator
Author
|
The new test may be flaky without #19339. |
SalmaSamy
reviewed
Aug 28, 2023
| ], | ||
| ) | ||
|
|
||
| _, _, _ = self.RunBazel(['build', '@hello//:all']) |
Member
There was a problem hiding this comment.
No need for this part: _, _, _ =
Collaborator
Author
There was a problem hiding this comment.
Wasn't sure whether this would trigger a linter. Feel free to remove on import (or let me know that I should drop it here).
SalmaSamy
approved these changes
Aug 28, 2023
Collaborator
Author
|
@bazel-io flag |
Member
|
@bazel-io fork 6.4.0 |
iancha1992
pushed a commit
to iancha1992/bazel
that referenced
this pull request
Aug 30, 2023
The events that update the lockfile are important for incremental correctness and thus need to return `true` from `storeForReplay`. Before this change, if a build creates the lockfile because it didn't exist and the lockfile is then deleted, subsequent builds did not regenerate it as the relevant SkyFunctions wouldn't rerun and the events were not replayed. Closes bazelbuild#19343. PiperOrigin-RevId: 561287438 Change-Id: I549f99b896a0095e8ffc35b7bacc8a841a44219a
iancha1992
pushed a commit
to iancha1992/bazel
that referenced
this pull request
Aug 30, 2023
The events that update the lockfile are important for incremental correctness and thus need to return `true` from `storeForReplay`. Before this change, if a build creates the lockfile because it didn't exist and the lockfile is then deleted, subsequent builds did not regenerate it as the relevant SkyFunctions wouldn't rerun and the events were not replayed. Closes bazelbuild#19343. PiperOrigin-RevId: 561287438 Change-Id: I549f99b896a0095e8ffc35b7bacc8a841a44219a
iancha1992
added a commit
that referenced
this pull request
Aug 31, 2023
…9371) The events that update the lockfile are important for incremental correctness and thus need to return `true` from `storeForReplay`. Before this change, if a build creates the lockfile because it didn't exist and the lockfile is then deleted, subsequent builds did not regenerate it as the relevant SkyFunctions wouldn't rerun and the events were not replayed. Closes #19343. Commit 19c0c80 PiperOrigin-RevId: 561287438 Change-Id: I549f99b896a0095e8ffc35b7bacc8a841a44219a Co-authored-by: Salma Samy <salmasamy@google.com>
Member
|
The changes in this PR have been included in Bazel 6.4.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. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The events that update the lockfile are important for incremental correctness and thus need to return
truefromstoreForReplay.Before this change, if a build creates the lockfile because it didn't exist and the lockfile is then deleted, subsequent builds did not regenerate it as the relevant SkyFunctions wouldn't rerun and the events were not replayed.