Merge ~lucaskanashiro/ubuntu/+source/docker.io:fix-autopkgtest-xenial into ubuntu/+source/docker.io:ubuntu/xenial-devel

Proposed by Lucas Kanashiro
Status: Approved
Approved by: Lucas Kanashiro
Approved revision: ad8a0db48301090eaa5c4d3d433735de92e5520f
Proposed branch: ~lucaskanashiro/ubuntu/+source/docker.io:fix-autopkgtest-xenial
Merge into: ubuntu/+source/docker.io:ubuntu/xenial-devel
Diff against target: 40 lines (+15/-5)
2 files modified
debian/changelog (+10/-0)
debian/tests/basic-smoke (+5/-5)
Reviewer Review Type Date Requested Status
Richard Harding (community) Approve
git-ubuntu developers Pending
Review via email: mp+395728@code.launchpad.net

Description of the change

Fix autopkgtest failure in Xenial. The basic-smoke test relies on an image built based on Debian Stable but the GPG key used to sign the repository is not recognized by default in Ubuntu Xenial. The proposed change replace Debian Stable to Ubuntu Focal which works fine in a clean Ubuntu Xenial VM. Here is the log of the test failure:

https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac/autopkgtest-xenial/xenial/amd64/d/docker.io/20201218_135506_6bcc0@/log.gz

Here is a PPA with the proposed package:

https://launchpad.net/~lucaskanashiro/+archive/ubuntu/containerd-upgrade-issue/+packages

autopkgtest is now happy with the proposed change:

autopkgtest [10:58:47]: @@@@@@@@@@@@@@@@@@@@ summary
basic-smoke PASS
docker-in-lxd PASS

To post a comment you must log in.
Revision history for this message
Richard Harding (rharding) wrote :

I'm not sure why it was debian stable to start with but makes sense to use focal.

review: Approve
Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

This is the default in Debian and I think none wanted to introduce a delta since it was working fine, but now it is causing a "regression".

Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

LGTM as well.

Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

Thanks! Uploaded:

$ git push pkg upload/18.09.7-0ubuntu1_16.04.8
Enumerating objects: 15, done.
Counting objects: 100% (15/15), done.
Delta compression using up to 32 threads
Compressing objects: 100% (10/10), done.
Writing objects: 100% (10/10), 1.45 KiB | 741.00 KiB/s, done.
Total 10 (delta 7), reused 0 (delta 0)
To ssh://git.launchpad.net/ubuntu/+source/docker.io
 * [new tag] upload/18.09.7-0ubuntu1_16.04.8 -> upload/18.09.7-0ubuntu1_16.04.8
$ dput ubuntu ../docker.io_18.09.7-0ubuntu1~16.04.8_source.changes
Checking signature on .changes
gpg: ../docker.io_18.09.7-0ubuntu1~16.04.8_source.changes: Valid signature from F823A2729883C97C
Checking signature on .dsc
gpg: ../docker.io_18.09.7-0ubuntu1~16.04.8.dsc: Valid signature from F823A2729883C97C
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading docker.io_18.09.7-0ubuntu1~16.04.8.dsc: done.
  Uploading docker.io_18.09.7-0ubuntu1~16.04.8.debian.tar.xz: done.
  Uploading docker.io_18.09.7-0ubuntu1~16.04.8_source.changes: done.
Successfully uploaded packages.

Unmerged commits

ad8a0db... by Lucas Kanashiro

changelog

1b9560f... by Lucas Kanashiro

d/t/basic-smoke: bootstrap Ubuntu Focal instead of Debian Stable

The GPG key used to sign the Debian Stable repository is not
recognized by default in Ubuntu Xenial which leads the test to a
failure. In order to fix that, bootstrap Ubuntu Focal which works
out-of-the-box.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index a5be997..9bdf0b0 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,13 @@
6+docker.io (18.09.7-0ubuntu1~16.04.8) xenial; urgency=medium
7+
8+ * d/t/basic-smoke: bootstrap Ubuntu Focal instead of Debian Stable.
9+ The GPG key used to sign the Debian Stable repository is not
10+ recognized by default in Ubuntu Xenial which leads the test to a
11+ failure. In order to fix that, bootstrap Ubuntu Focal which works
12+ out-of-the-box.
13+
14+ -- Lucas Kanashiro <kanashiro@ubuntu.com> Mon, 04 Jan 2021 10:36:11 -0300
15+
16 docker.io (18.09.7-0ubuntu1~16.04.7) xenial; urgency=medium
17
18 * d/p/do_not_bind_docker_to_containerd.patch: Update docker.io to not
19diff --git a/debian/tests/basic-smoke b/debian/tests/basic-smoke
20index ccf8825..ad82a88 100755
21--- a/debian/tests/basic-smoke
22+++ b/debian/tests/basic-smoke
23@@ -11,12 +11,12 @@ docker version
24
25 debootstrap \
26 --variant=minbase \
27- stable \
28+ focal \
29 "$tempDir" \
30- http://httpredir.debian.org/debian
31+ http://archive.ubuntu.com/ubuntu
32
33-tar -cC "$tempDir" . | docker import - debian
34-defer 'docker rmi debian'
35+tar -cC "$tempDir" . | docker import - focal
36+defer 'docker rmi focal'
37
38-docker run --name test debian true
39+docker run --name test focal true
40 defer 'docker rm -f test'

Subscribers

People subscribed via source and target branches