-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[Build] Add the missing debian source bullseye-updates/buster-updates #12522
Conversation
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.
Some files already contain required lines. You duplicate them.
BTW, should we also fix sonic-slave-bullseye/Dockerfile.j2 and sonic-slave-buster/Dockerfile.j2?
deb [arch=arm64] http://deb.debian.org/debian bullseye-updates main contrib non-free | ||
deb [arch=arm64] http://security.debian.org bullseye-security main contrib non-free | ||
deb [arch=arm64] http://deb.debian.org/debian bullseye-updates main contrib non-free |
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.
Line duplication (bullseye-updates).
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.
@kv-y , it is different mirror with the same name bullseye-update, it is not a duplicate. Could you please double check?
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.
@xumia,
line 5: deb [arch=arm64] http://deb.debian.org/debian bullseye-updates main contrib non-free
line 7: deb [arch=arm64] http://deb.debian.org/debian bullseye-updates main contrib non-free
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.
@kv-y , thanks a lot, fixed.
$ grep "updates main" dockers/ -r | uniq -cd
2 dockers/docker-base-bullseye/sources.list.arm64:deb [arch=arm64] http://deb.debian.org/debian bullseye-updates main contrib non-free
2 dockers/docker-base-bullseye/sources.list.armhf:deb [arch=armhf] http://deb.debian.org/debian bullseye-updates main contrib non-free
deb [arch=armhf] http://deb.debian.org/debian bullseye-updates main contrib non-free | ||
deb [arch=armhf] http://security.debian.org bullseye-security main contrib non-free | ||
deb [arch=armhf] http://deb.debian.org/debian bullseye-updates main contrib non-free |
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.
Line duplication (bullseye-updates).
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.
@kv-y , it is different mirror with the same name bullseye-update, it is not a duplicate. Could you please double check?
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.
@xumia
line 5: deb [arch=armhf] http://deb.debian.org/debian bullseye-updates main contrib non-free
line 7: deb [arch=armhf] http://deb.debian.org/debian bullseye-updates main contrib non-free
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.
@kv-y , fixed, thanks.
By default sonic-slave docker images for amd64 have this mirror in /etc/apt/sources.list: And you append (but not overwrite) new mirrors (trafficmanager.net) in sonic-slave-xxx/Dockerfile.j2. Also same question about bullseye-updates mirrors in files/apt/sources.list.xxx |
|
…sonic-net#12522) Why I did it Add the missing debian source bullseye-updates/buster-updates The build failure as below, it is caused by the docker image debian:bullseye used the version 2.31-13+deb11u5, but the version only available in bullseye-update.
Cherry-pick PR to 202205: #12555 |
…#12522) Why I did it Add the missing debian source bullseye-updates/buster-updates The build failure as below, it is caused by the docker image debian:bullseye used the version 2.31-13+deb11u5, but the version only available in bullseye-update.
Why I did it
Add the missing debian source bullseye-updates/buster-updates
The build failure as below, it is caused by the docker image debian:bullseye used the version 2.31-13+deb11u5, but the version only available in bullseye-update.
https://dev.azure.com/mssonic/build/_build/results?buildId=165930&view=logs&j=e2a1d004-6151-514d-dd3b-ba225f574bd9&t=f0777b63-8377-5b65-c700-6568122e76b9
Skip the stretch-update.
How I did it
How to verify it
Which release branch to backport (provide reason below if selected)
Description for the changelog
Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)