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

[vs]: docker-syncd-vs.gz build constantly fails #6711

Closed
nazariig opened this issue Feb 7, 2021 · 2 comments · Fixed by #6713
Closed

[vs]: docker-syncd-vs.gz build constantly fails #6711

nazariig opened this issue Feb 7, 2021 · 2 comments · Fixed by #6713

Comments

@nazariig
Copy link
Collaborator

nazariig commented Feb 7, 2021

Description

docker-syncd-vs.gz build constantly fails because of unmet dependencies:
https://github.com/Azure/sonic-buildimage/blob/master/platform/vs/docker-syncd-vs/Dockerfile.j2#L11

RUN apt-get install -f -y iproute2=4.20.0-2 libcap2-bin=1:2.25-2

Steps to reproduce the issue:

  1. git clone https://github.com/Azure/sonic-buildimage.git
  2. git submodule update --recursive --init --progress
  3. make target/docker-syncd-vs.gz

Describe the results you received:
Build log:

13:04:28  Step 9/20 : RUN apt-get install -f -y iproute2=4.20.0-2 libcap2-bin=1:2.25-2
13:04:28   ---> Running in 2c2817a4abd7
13:04:28  Reading package lists...
13:04:28  Building dependency tree...
13:04:28  Reading state information...
13:04:28  libcap2-bin is already the newest version (1:2.25-2).
13:04:28  libcap2-bin set to manually installed.
13:04:28  Suggested packages:
13:04:28    iproute2-doc
13:04:28  Recommended packages:
13:04:28    libatm1
13:04:28  The following packages will be DOWNGRADED:
13:04:28    iproute2
13:04:28  �[91mE: Packages were downgraded and -y was used without --allow-downgrades.
13:04:28  �[0m0 upgraded, 0 newly installed, 1 downgraded, 0 to remove and 8 not upgraded.
13:04:28  The command '/bin/sh -c apt-get install -f -y iproute2=4.20.0-2 libcap2-bin=1:2.25-2' returned a non-zero code: 100
13:04:28  [  FAIL LOG END  ] [ target/docker-syncd-vs.gz ]
13:04:28  make: *** [slave.mk:717: target/docker-syncd-vs.gz] Error 1
13:04:29  DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. pip 21.0 will drop support for Python 2.7 in January 2021. More details about Python 2 support in pip can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support pip 21.0 will remove support for this functionality.
13:23:48  make[1]: *** [Makefile.work:264: target/sonic-vs.bin] Error 2
13:23:48  make[1]: Leaving directory '/data/johnar/workspace/vs/buildimage-vs-image/sonic-buildimage'
13:23:48  make: *** [Makefile:9: target/sonic-vs.bin] Error 2

Describe the results you expected:
Build should pass

Additional information you deem important (e.g. issue happens only occasionally):

user@build-server: sonic-buildimage$ docker run --rm -ti --name cfg-test docker-config-engine-buster bash

root@31c6dc1ea373:/# apt-get update
Hit:1 http://packages.trafficmanager.net/debian/debian buster InRelease
Hit:2 http://debian-archive.trafficmanager.net/debian buster InRelease
Hit:3 http://packages.trafficmanager.net/debian/debian buster-updates InRelease
Hit:4 http://packages.trafficmanager.net/debian/debian buster-backports InRelease
Hit:5 http://debian-archive.trafficmanager.net/debian-security buster/updates InRelease
Hit:6 http://debian-archive.trafficmanager.net/debian buster-backports InRelease
Reading package lists... Done

root@31c6dc1ea373:/# apt-get install -f -y iproute2=4.20.0-2 libcap2-bin=1:2.25-2
Reading package lists... Done
Building dependency tree
Reading state information... Done
libcap2-bin is already the newest version (1:2.25-2).
libcap2-bin set to manually installed.
Suggested packages:
  iproute2-doc
Recommended packages:
  libatm1
The following packages will be DOWNGRADED:
  iproute2
0 upgraded, 0 newly installed, 1 downgraded, 0 to remove and 8 not upgraded.
E: Packages were downgraded and -y was used without --allow-downgrades.

root@31c6dc1ea373:/# echo $?
100

root@31c6dc1ea373:/# apt-cache policy iproute2
iproute2:
  Installed: 4.20.0-2+deb10u1
  Candidate: 4.20.0-2+deb10u1
  Version table:
     5.9.0-1~bpo10+1 100
        100 http://debian-archive.trafficmanager.net/debian buster-backports/main amd64 Packages
 *** 4.20.0-2+deb10u1 500
        500 http://debian-archive.trafficmanager.net/debian buster/main amd64 Packages
        100 /var/lib/dpkg/status
     4.20.0-2 500
        500 http://packages.trafficmanager.net/debian/debian buster/main amd64 Packages

root@31c6dc1ea373:/# apt-cache policy libcap2-bin
libcap2-bin:
  Installed: 1:2.25-2
  Candidate: 1:2.25-2
  Version table:
 *** 1:2.25-2 500
        500 http://debian-archive.trafficmanager.net/debian buster/main amd64 Packages
        500 http://packages.trafficmanager.net/debian/debian buster/main amd64 Packages
        100 /var/lib/dpkg/status

Output of show version:

(paste your output here)

Attach debug file sudo generate_dump:

(paste your output here)
@nazariig
Copy link
Collaborator Author

nazariig commented Feb 7, 2021

The potential fix is to add --allow-downgrades, still need to revisit the logic behind of overriding the packages from docker-config-engine-buster

@nazariig
Copy link
Collaborator Author

nazariig commented Feb 7, 2021

lguohan added a commit that referenced this issue Feb 8, 2021
Fix #6711 

the requirement was introduced in commit 75104bb
to support sflow in stretch build. in buster build, the requirement
is met, no need to pin down the version.

Signed-off-by: Guohan Lu <lguohan@gmail.com>
daall pushed a commit that referenced this issue Feb 17, 2021
Fix #6711 

the requirement was introduced in commit 75104bb
to support sflow in stretch build. in buster build, the requirement
is met, no need to pin down the version.

Signed-off-by: Guohan Lu <lguohan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant