[Bug 1805332] Re: [Swift backend] Upload image hit error: Unicode-objects must be encoded before hashing

[Liam Young]

The disco package worked for me to. Version info below:

# apt-cache policy python3-glance-store
python3-glance-store:
  Installed: 0.28.0-0ubuntu1.1
  Candidate: 0.28.0-0ubuntu1.1
  Version table:
 *** 0.28.0-0ubuntu1.1 500
        500 http://archive.ubuntu.com/ubuntu disco-proposed/main amd64 Packages
        100 /var/lib/dpkg/status
     0.28.0-0ubuntu1 500
        500 http://nova.clouds.archive.ubuntu.com/ubuntu disco/main amd64 Packages


** Tags removed: verification-needed-disco
** Tags added: verification-done-disco

-- 
You received this bug notification because you are a member of Ubuntu
OpenStack, which is subscribed to Ubuntu Cloud Archive.
https://bugs.launchpad.net/bugs/1805332

Title:
  [Swift backend] Upload image hit error: Unicode-objects must be
  encoded before hashing

Status in Ubuntu Cloud Archive:
  Fix Committed
Status in Ubuntu Cloud Archive stein series:
  Fix Committed
Status in glance_store:
  Triaged
Status in python-glance-store package in Ubuntu:
  Fix Released
Status in python-glance-store source package in Disco:
  Fix Committed
Status in python-glance-store source package in Eoan:
  Fix Released

Bug description:
  [Impact]
  If we upload a large image (larger than 1G), the glance_store will hit a Unicode error. To fix this a patch has been merged in upstream master and backported to stable rocky.

  [Test Case]
  Deploy glance related to swift-proxy using the object-store relation. Then attempt to upload a large image (not cirros)

  $ openstack image create --public --file /home/ubuntu/images/bionic-server-cloudimg-amd64.img bionic-test
  500 Internal Server Error: The server has either erred or is incapable of performing the requested operation. (HTTP 500)

  If the patch is manually applied and glance-api restarted then the
  above command succeeds.

  In order to avoid regression of existing consumers, the OpenStack team will
  run their continuous integration test against the packages that are in
  -proposed. A successful run of all available tests will be required before the
  proposed packages can be let into -updates.

  The OpenStack team will be in charge of attaching the output summary of the
  executed tests. The OpenStack team members will not mark ‘verification-done’ until
  this has happened.

  [Regression Potential]
  In order to mitigate the regression potential, the results of the
  aforementioned tests are attached to this bug.

  [Discussion]
  n/a

  [Original Description]

  env: master branch, Glance using swift backend.

  We hit a strange error， if we upload a large image (larger than 1G),
  the glance_store will hit a error:Unicode-objects must be encoded
  before hashing. But if the image is small enough, the error won't
  happen.

  error log:
  https://www.irccloud.com/pastebin/jP3DapNy/

  After dig into the code, it appears that when chunk reading the image
  data, the date piece may be non-byte, so the checksum.updating will
  raise the error.

  encoding the date piece to ensure it's byte can solve the problem.

To manage notifications about this bug go to:
https://bugs.launchpad.net/cloud-archive/+bug/1805332/+subscriptions