Ubuntu
ubuntu-fan package

ubuntu-fan fails to set https_proxy when running the "docker" dep8 test on autopkgtest.u.c

Bug #1968387 reported by Sergio Durigan Junior
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-fan (Ubuntu)
Fix Released
High
Sergio Durigan Junior
Focal
Fix Released
High
Sergio Durigan Junior
Impish
Fix Released
Medium
Sergio Durigan Junior
Jammy
Fix Released
High
Sergio Durigan Junior

Bug Description

[Impact]

ubuntu-fan currently fails to properly set the $https_proxy variable when running the "docker" dep8 test on autopkgtest.u.c.

At the time of this writing this is a major problem on Focal because ubuntu-fan is blocking docker.io from migrating:

Error response from daemon: Get "https://registry-1.docker.io/v2/": net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)
FAIL: error on docker test
local docker test: docker pull ubuntu failure!
local docker test: skipping test.
autopkgtest [03:44:06]: test docker: -----------------------]
autopkgtest [03:44:06]: test docker: - - - - - - - - - - results - - - - - - - - - -
docker FAIL non-zero exit status 1

[Test Plan]

The dep8 test (named "docker") shall pass when performed in the autopkgtest.u.c. infra.

[Where problems could occur]

This is a very specific issue on how a dep8 test from the package interacts with autopkgtest.u.c, so there is really no way that a regression can be caused by fixing this problem.

See original description

Related branches

~sergiodj/ubuntu/+source/ubuntu-fan:fix-https-proxy-impish
Lucas Kanashiro: Pending requested
Andy Whitcroft: Pending requested
Canonical Server: Pending requested
Diff: 53 lines (+16/-5)
2 files modified
debian/changelog (+7/-0)
debian/tests/docker (+9/-5)
~sergiodj/ubuntu/+source/ubuntu-fan:fix-https-proxy-jammy
Andy Whitcroft (community): Approve
Bryce Harrington (community): Approve
Lucas Kanashiro: Pending requested
Diff: 55 lines (+18/-5)
2 files modified
debian/changelog (+7/-0)
debian/tests/docker (+11/-5)
~sergiodj/ubuntu/+source/ubuntu-fan:fix-https-proxy-focal
Andy Whitcroft (community): Approve
Lucas Kanashiro (community): Approve
Canonical Server: Pending requested
Diff: 53 lines (+16/-5)
2 files modified
debian/changelog (+7/-0)
debian/tests/docker (+9/-5)
Sergio Durigan Junior (sergiodj)
Changed in ubuntu-fan (Ubuntu Focal):
status: New → In Progress
importance: Undecided → High
assignee: nobody → Sergio Durigan Junior (sergiodj)
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

The Focal package has been uploaded.

I'm waiting on apw's reply to upload the Jammy package.

description: updated
Sergio Durigan Junior (sergiodj)
Changed in ubuntu-fan (Ubuntu Impish):
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Sergio Durigan Junior (sergiodj)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-fan - 0.12.16

---------------
ubuntu-fan (0.12.16) jammy; urgency=medium

  * d/t/docker: Don't use "https://" for $https_proxy. Set
    HTTPS_PROXY for docker.service as well. (LP: #1968387)

 -- Sergio Durigan Junior <email address hidden> Fri, 08 Apr 2022 21:02:22 -0400

Changed in ubuntu-fan (Ubuntu Jammy):
status: In Progress → Fix Released
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Please test proposed package

Hello Sergio, or anyone else affected,

Accepted ubuntu-fan into impish-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubuntu-fan/0.12.15ubuntu0.21.10.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-impish to verification-done-impish. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-impish. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in ubuntu-fan (Ubuntu Impish):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-impish
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Hello Sergio, or anyone else affected,

Accepted ubuntu-fan into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubuntu-fan/0.12.13ubuntu0.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in ubuntu-fan (Ubuntu Focal):
status: In Progress → Fix Committed
tags: added: verification-needed-focal
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

So normally I would be a bit grumpy about the version number, as it seems to be not quite 'consistent' with how the versioning for ubuntu-fan is being done (we never use ubuntuX from what I can see) - but since I saw Andy going +1 on the MPs, I guess he's fine with it!

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

Thanks, Łukasz. FWIW it was a bit confusing to choose a proper version number and there was some discussion about it in the MPs.

Either way, the verification for both uploads, as explained in the SRU template, is to make sure that the autopkgtests are passing. I have now verified that everything passes on Focal and Impish, so I consider these verified.

tags: added: verification-done-focal verification-done-impish
removed: server-todo verification-needed verification-needed-focal verification-needed-impish
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-fan - 0.12.15ubuntu0.21.10.1

---------------
ubuntu-fan (0.12.15ubuntu0.21.10.1) impish; urgency=medium

  * d/t/docker: Don't use "https://" for $https_proxy. Set
    HTTPS_PROXY for docker.service as well. (LP: #1968387)

 -- Sergio Durigan Junior <email address hidden> Mon, 11 Apr 2022 15:26:05 -0400

Changed in ubuntu-fan (Ubuntu Impish):
status: Fix Committed → Fix Released
Revision history for this message
Brian Murray (brian-murray) wrote : Update Released

The verification of the Stable Release Update for ubuntu-fan has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-fan - 0.12.13ubuntu0.1

---------------
ubuntu-fan (0.12.13ubuntu0.1) focal; urgency=medium

  * d/t/docker: Don't use "https://" for $https_proxy. Set
    HTTPS_PROXY for docker.service as well. (LP: #1968387)

 -- Sergio Durigan Junior <email address hidden> Fri, 08 Apr 2022 21:11:32 -0400

Changed in ubuntu-fan (Ubuntu Focal):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.