Ubuntu
ubuntu-dev-tools package

can't run anymore ubuntu-build --batch --retry from cmdline due to wrong login() call in Python

Bug #1984113 reported by Dan Streetman
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Debian)
Fix Released
Unknown
ubuntu-dev-tools (Ubuntu)
Fix Released
Undecided
Unassigned
Jammy
New
Undecided
Unassigned
Kinetic
Fix Released
Undecided
Unassigned

Bug Description

[impact]

ubuntu-build assumes the lpapicache Launchpad will default to login(), but it now defaults to anonymous login, which breaks ubuntu-build operation.

[test case]

ubuntu-build --batch --retry openimageio
Traceback (most recent call last):
    File "/usr/bin/ubuntu-build", line 282, in <module>
      main()
    File "/usr/bin/ubuntu-build", line 260, in main
     can_retry = options.retry and me.canUploadPackage(ubuntu_archive,
AttributeError: 'NoneType' object has no attribute 'canUploadPackage'

[regression potential]

failure and/or errors running ubuntu-build

[scope]

This is needed only for jammy and kinetic.

The default was changed from login() to login_anonymously() with commit cb865e3b8, which was first included in 0.179, so this bug does not exist in focal or earlier.

See original description

Related branches

~ddstreet/ubuntu-dev-tools:lp1984113
Mattia Rizzolo: Pending requested
Diff: 55 lines (+15/-10)
2 files modified
ubuntu-build (+7/-4)
ubuntutools/lp/lpapicache.py (+8/-6)
Dan Streetman (ddstreet)
description: updated
Revision history for this message
Dan Streetman (ddstreet) wrote :

Can you check this test build to see if the patches from the MR fix this?

https://launchpad.net/~ddstreet/+archive/ubuntu/ubuntu-dev-tools

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

This bug was fixed in the package ubuntu-dev-tools - 0.191

---------------
ubuntu-dev-tools (0.191) unstable; urgency=medium

  [ Dan Streetman ]
  * lpapicache:
    + Make sure that login() actually logins and doesn't use cached credentials.
  * ubuntu-build:
    + Fix crash caused by a change in lpapicache that changed the default
      operation mode from authenticated to anonymous. LP: #1984113

  [ Stefano Rivera ]
  * backportpackage:
    + Add support for lsb-release-minimal, which doesn't have a Python module.
      Thanks to Gioele Barabucci for the patch. Closes: #1020901

  [ Mattia Rizzolo ]
  * ubuntutools/archive.py:
    + Fix operation of SourcePackage._source_urls() (as used, for example, in
      SourcePackage.pull() called by backportpackage) to also work when the
      class is instantiated with a URL as .dsc. Fixes regression from v0.184.
      Thanks to Unit 193 for the initial patch.

 -- Mattia Rizzolo <email address hidden> Tue, 11 Oct 2022 13:56:03 +0200

Changed in ubuntu-dev-tools (Ubuntu Kinetic):
status: New → Fix Released
Bug Watch Updater (bug-watch-updater)
Changed in ubuntu-dev-tools (Debian):
status: Unknown → 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.