-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Upgrade docker-sonic-vs to bookworm #20697
Merged
qiluo-msft
merged 9 commits into
sonic-net:master
from
saiarcot895:docker-sonic-vs-to-bookworm
Dec 3, 2024
Merged
Upgrade docker-sonic-vs to bookworm #20697
qiluo-msft
merged 9 commits into
sonic-net:master
from
saiarcot895:docker-sonic-vs-to-bookworm
Dec 3, 2024
Conversation
This file contains 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
This updates libnl3 to the version that is in Bookworm. There is one patch still pending that is not present in 3.7.0. The functionality provided by the patch has been upstreamed and is present in newer versions of libnl3, but the implementation is slightly different. When the update to newer versions of libnl3 happens and this functionality is present, some code changes may be needed. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
fb4eada
to
6b0ca63
Compare
Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
Looks like there may have been a no-change rebuild in Debian for this package that's not visible from tracker.debian.org Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
The explicit libnl3 package installation should've been added to docker-syncd-vs instead of docker-sonic-vs. Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
6b0ca63
to
0c45df5
Compare
Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
/azpw run |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
9 tasks
prsunny
approved these changes
Dec 3, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, request signoff from other reviewers.
kperumalbfn
approved these changes
Dec 3, 2024
@saiarcot895 If any vs setup doc to be updated, please update. |
This was referenced Dec 3, 2024
Merged
AkeelAli
added a commit
to AkeelAli/sonic-platform-vpp
that referenced
this pull request
Dec 5, 2024
Version and name were changed in sonic-net/sonic-buildimage#20697
prsunny
pushed a commit
to sonic-net/sonic-swss
that referenced
this pull request
Dec 6, 2024
What I did Update the Azure Pipeline build to build for Bookworm. This is dependent on sonic-net/sonic-buildimage#20697.
yue-fred-gao
pushed a commit
to sonic-net/sonic-platform-vpp
that referenced
this pull request
Dec 8, 2024
Version and name were changed in sonic-net/sonic-buildimage#20697
divyachandralekha
pushed a commit
to divyachandralekha/sonic-swss
that referenced
this pull request
Dec 12, 2024
What I did Update the Azure Pipeline build to build for Bookworm. This is dependent on sonic-net/sonic-buildimage#20697.
divyachandralekha
pushed a commit
to divyachandralekha/sonic-swss
that referenced
this pull request
Dec 12, 2024
What I did Update the Azure Pipeline build to build for Bookworm. This is dependent on sonic-net/sonic-buildimage#20697.
VladimirKuk
pushed a commit
to Marvell-switching/sonic-buildimage
that referenced
this pull request
Jan 21, 2025
Why I did it This updates the docker-sonic-vs container to be based on Bookworm.
stepanblyschak
pushed a commit
to stepanblyschak/sonic-swss
that referenced
this pull request
Jan 27, 2025
What I did Update the Azure Pipeline build to build for Bookworm. This is dependent on sonic-net/sonic-buildimage#20697.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Why I did it
This updates the docker-sonic-vs container to be based on Bookworm.
Work item tracking
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)