yahoo-eng-team team mailing list archive

[Dan Smith <1932337@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

During an upgrade to Xena, cinder-backed image locations are migrated to
include the store name in the URL field. This is lazily done on the
first GET of the image. The problem is that the first user to GET an
image after the migration may not be an admin or the owner of the image,
as would be the case for a public or shared image. If that happens, the
user gets a 404 for a valid image because the DB layer refuses the
modify operation. This is logged:

2021-06-17 08:50:06,559 WARNING [glance.db.sqlalchemy.api] Attempted to
modify image user did not own.

The lazy migration code needs to tolerate this and allow someone else to
perform the migration without breaking non-owner GET operations until
the migration is complete.

** Affects: glance
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Glance.
https://bugs.launchpad.net/bugs/1932337

Title:
  Cinder store migration will fail if first GET'er is not the owner

Status in Glance:
  New

Bug description:
  During an upgrade to Xena, cinder-backed image locations are migrated
  to include the store name in the URL field. This is lazily done on the
  first GET of the image. The problem is that the first user to GET an
  image after the migration may not be an admin or the owner of the
  image, as would be the case for a public or shared image. If that
  happens, the user gets a 404 for a valid image because the DB layer
  refuses the modify operation. This is logged:

  2021-06-17 08:50:06,559 WARNING [glance.db.sqlalchemy.api] Attempted
  to modify image user did not own.

  The lazy migration code needs to tolerate this and allow someone else
  to perform the migration without breaking non-owner GET operations
  until the migration is complete.

To manage notifications about this bug go to:
https://bugs.launchpad.net/glance/+bug/1932337/+subscriptions