[Bug 1826870] [NEW] cache.commit() doesn't release the archives lock

[Julian Andres Klode]

Public bug reported:

[Impact]
cache.commit() does not release all the locks it acquires as part of committing. This is a regression of locking fixes in bug 1795407.

[Test case]
This script should work:

#!/usr/bin/env python
import apt
import subprocess

cache = apt.Cache()
pkg = cache["hello"]
pkg.mark_install()
cache.commit()
subprocess.check_call(["apt", "remove", "--yes", "hello"])

[Regression potential]
Other new locking bugs could pop up

** Affects: python-apt (Ubuntu)
     Importance: Undecided
         Status: Fix Released

** Affects: python-apt (Ubuntu Xenial)
     Importance: Undecided
         Status: Triaged

** Affects: python-apt (Ubuntu Bionic)
     Importance: Undecided
         Status: Triaged

** Affects: python-apt (Ubuntu Cosmic)
     Importance: Undecided
         Status: Triaged

** Affects: python-apt (Debian)
     Importance: Unknown
         Status: Unknown

** Bug watch added: Debian Bug tracker #922416
   https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=922416

** Also affects: python-apt (Debian) via
   https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=922416
   Importance: Unknown
       Status: Unknown

** Changed in: python-apt (Ubuntu)
       Status: New => Fix Released

** Changed in: python-apt (Ubuntu)
       Status: Fix Released => New

** Also affects: python-apt (Ubuntu Cosmic)
   Importance: Undecided
       Status: New

** Also affects: python-apt (Ubuntu Bionic)
   Importance: Undecided
       Status: New

** Also affects: python-apt (Ubuntu Xenial)
   Importance: Undecided
       Status: New

** Changed in: python-apt (Ubuntu)
       Status: New => Fix Released

** Changed in: python-apt (Ubuntu Xenial)
       Status: New => Triaged

** Changed in: python-apt (Ubuntu Bionic)
       Status: New => Triaged

** Changed in: python-apt (Ubuntu Cosmic)
       Status: New => Triaged

** Description changed:

  [Impact]
- cache.commit() does not release all the locks it acquires as part of committing.
+ cache.commit() does not release all the locks it acquires as part of committing. This is a regression of locking fixes in bug 1795407.
  
  [Test case]
  This script should work:
  
  #!/usr/bin/env python
  import apt
  import subprocess
  
  cache = apt.Cache()
  pkg = cache["hello"]
  pkg.mark_install()
  cache.commit()
  subprocess.check_call(["apt", "remove", "--yes", "hello"])
  
  [Regression potential]
  Other new locking bugs could pop up

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to python-apt in Ubuntu.
https://bugs.launchpad.net/bugs/1826870

Title:
  cache.commit() doesn't release the archives lock

Status in python-apt package in Ubuntu:
  Fix Released
Status in python-apt source package in Xenial:
  Triaged
Status in python-apt source package in Bionic:
  Triaged
Status in python-apt source package in Cosmic:
  Triaged
Status in python-apt package in Debian:
  Unknown

Bug description:
  [Impact]
  cache.commit() does not release all the locks it acquires as part of committing. This is a regression of locking fixes in bug 1795407.

  [Test case]
  This script should work:

  #!/usr/bin/env python
  import apt
  import subprocess

  cache = apt.Cache()
  pkg = cache["hello"]
  pkg.mark_install()
  cache.commit()
  subprocess.check_call(["apt", "remove", "--yes", "hello"])

  [Regression potential]
  Other new locking bugs could pop up

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/python-apt/+bug/1826870/+subscriptions