Skip to content
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

docker-sonic-vs.gz on master branch is on buster #11706

Closed
Ndancejic opened this issue Aug 11, 2022 · 2 comments · Fixed by #13294
Closed

docker-sonic-vs.gz on master branch is on buster #11706

Ndancejic opened this issue Aug 11, 2022 · 2 comments · Fixed by #13294
Assignees
Labels
Triaged this issue has been triaged

Comments

@Ndancejic
Copy link
Contributor

Description

docker-sonic-vs.gz is buster on the master branch build

Steps to reproduce the issue:

Build sonic-buildimage from master branch

Describe the results you received:

cannot test swss changes with pytest because swss master is on bullseye and docker-sonic-vs.gz provides a buster image.

Describe the results you expected:

docker-sonic-vs.gz should be on bullseye\

@yxieca
Copy link
Contributor

yxieca commented Aug 17, 2022

@saiarcot895 can you assess the change with @Ndancejic , my concerns are:

  • Do we have test depends on buster vs image that will fail after the move?
  • The effort of moving to bullseye. Do we need help from the community?

@yxieca yxieca added the Triaged this issue has been triaged label Aug 17, 2022
@saiarcot895
Copy link
Contributor

sairedis, swss, and swss-common all use the docker-sonic-vs image in pipelines and/or README instructions. The upgrade to Bullseye may need pipeline updates in these submodules as well to build Bullseye artifacts.

As for moving the container itself to Bullseye, it doesn't look to be difficult, just need to remove Python 2 packages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Triaged this issue has been triaged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants