muranoclient doesn't compare hashes before uploading image to glance

[Dmytro Dovbii]

If user wants to update his package in murano and download newer version from apps.openstack.org he will not get the image required by package if it also was updated previously, but has the same name as previous one.
This occurs because muranoclient does not compare checksum of already uploaded to glance image and image specified in dependencies of package:

https://github.com/openstack/python-muranoclient/blob/master/muranoclient/common/utils.py#L578-L584

[Kirill Zaitsev (kzaitsev)]

That;s what versions are for

[Kirill Zaitsev (kzaitsev)]

Ah, I see it's about images. Then the bug is actually invalid. Since this would cause muranoclient to download the same image multiple times. hash sum is only available after the image is fully downloaded to glance — then if you download 2 packages and both of them need the same image — by your logic it would be downloaded twice. We had a bug back in L for fixing this I believe

[Dmytro Dovbii (ddovbii)]

well, we decided to resolve this bug by following logic:

* if image with the same name is already registered in glance, but does not have checksum (because it is downloading at this time, for example) we skip uploading

* if image with the same name is already registered and has different checksum then image in package's dependencies, we update metadata of old image by adding `-old` prefix to it's title and upload new one