go.mod: upgrade to yaml v4 #157800
Merged
go.mod: upgrade to yaml v4 #157800
+62
−48
Conversation
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
Member
|
This change is |
RaduBerinde
force-pushed
the
yaml-v4-a
branch
2 times, most recently
from
87e14a0 to
4537e0c
Compare
RaduBerinde
requested review from
Abhinav1299,
DarrylWong,
aa-joshi,
arjunmahishi,
golgeek,
jasonlmfong and
sumeerbhola
and removed request for
a team
sumeerbhola
approved these changes
Nov 17, 2025
Collaborator
sumeerbhola
left a comment
sumeerbhola
left a comment
There was a problem hiding this comment.
@sumeerbhola reviewed 50 of 50 files at r1, all commit messages.
Reviewable status:complete! 1 of 0 LGTMs obtained (waiting on @aa-joshi, @Abhinav1299, @arjunmahishi, @DarrylWong, @golgeek, and @jasonlmfong)
Member
Author
|
TFTR! |
Member
Author
|
bors r+ |
craig bot
pushed a commit
that referenced
this pull request
Nov 18, 2025
156897: kvserver: remove replica dependency in splitPreApply r=pav-kv a=arulajmani Refactor to remove the need to pass in a replica struct to splitPreApply. This will make testing this in subsequent patches easier. Epic: None Release note: None 157800: go.mod: upgrade to yaml v4 r=RaduBerinde a=RaduBerinde The https://github.com/go-yaml/yaml (import path `gopkg.in/yaml.v3`) repository has been archived. The YAML team took over maintenance in a separate fork at a very confusingly named https://github.com/yaml/go-yaml (import path `go.yaml.in/yaml/v4`). This commit updates uses of v3 to `v4.0.0-rc.3` from the new repository. No code changes are required. Upgrading v2 uses to v4 will be done separately, as that requires code changes. Epic: none Release note: None 158012: kvserver: verbose Raft logging for TestLeaseRequestBumpsEpoch r=miraradeva a=miraradeva See #157128 for details about the test failure. Closes: #157128 Release note: None Co-authored-by: Arul Ajmani <[email protected]> Co-authored-by: Radu Berinde <[email protected]> Co-authored-by: Mira Radeva <[email protected]>
Contributor
|
Build failed (retrying...): |
craig bot
pushed a commit
that referenced
this pull request
Nov 18, 2025
157800: go.mod: upgrade to yaml v4 r=RaduBerinde a=RaduBerinde The https://github.com/go-yaml/yaml (import path `gopkg.in/yaml.v3`) repository has been archived. The YAML team took over maintenance in a separate fork at a very confusingly named https://github.com/yaml/go-yaml (import path `go.yaml.in/yaml/v4`). This commit updates uses of v3 to `v4.0.0-rc.3` from the new repository. No code changes are required. Upgrading v2 uses to v4 will be done separately, as that requires code changes. Epic: none Release note: None 158012: kvserver: verbose Raft logging for TestLeaseRequestBumpsEpoch r=miraradeva a=miraradeva See #157128 for details about the test failure. Closes: #157128 Release note: None Co-authored-by: Radu Berinde <[email protected]> Co-authored-by: Mira Radeva <[email protected]>
Contributor
|
Build failed (retrying...): |
craig bot
pushed a commit
that referenced
this pull request
Nov 18, 2025
157800: go.mod: upgrade to yaml v4 r=RaduBerinde a=RaduBerinde The https://github.com/go-yaml/yaml (import path `gopkg.in/yaml.v3`) repository has been archived. The YAML team took over maintenance in a separate fork at a very confusingly named https://github.com/yaml/go-yaml (import path `go.yaml.in/yaml/v4`). This commit updates uses of v3 to `v4.0.0-rc.3` from the new repository. No code changes are required. Upgrading v2 uses to v4 will be done separately, as that requires code changes. Epic: none Release note: None Co-authored-by: Radu Berinde <[email protected]>
Contributor
|
Build failed: |
RaduBerinde
force-pushed
the
yaml-v4-a
branch
from
4537e0c to
034853b
Compare
Member
Author
|
bors r+ |
Contributor
|
Merge conflict. |
The https://github.com/go-yaml/yaml (import path `gopkg.in/yaml.v3`) repository has been archived. The YAML team took over maintenance in a separate fork at a very confusingly named https://github.com/yaml/go-yaml (import path `go.yaml.in/yaml/v4`). This commit updates uses of v3 to `v4.0.0-rc.3` from the new repository. No code changes are required. Upgrading v2 uses to v4 will be done separately, as that requires code changes. Epic: none Release note: None
RaduBerinde
force-pushed
the
yaml-v4-a
branch
from
034853b to
9d9287f
Compare
Member
Author
|
bors r+ |
Contributor
|
Build succeeded: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 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 https://github.com/go-yaml/yaml (import path
gopkg.in/yaml.v3)repository has been archived. The YAML team took over maintenance in a
separate fork at a very confusingly named
https://github.com/yaml/go-yaml (import path
go.yaml.in/yaml/v4).This commit updates uses of v3 to
v4.0.0-rc.3from the newrepository. No code changes are required.
Upgrading v2 uses to v4 will be done separately, as that requires code
changes.
Epic: none
Release note: None