| zaitcev | I have a small problem. I run: | 03:12 |
|---|---|---|
| zaitcev | cinder create --image-id 6413a4d1-c74f-41d1-b561-f2c936c0d979 4 | 03:12 |
| zaitcev | Dec 22 04:49:11 zaitcev-cinder-5 cinder-volume[132138]: ERROR cinder.volume.flows.manager.create_volume [None req-11a0f805-da37-4b1c-9f0c-60a6b9c4258f demo None] Failed updating volume 1972b945-c120-4926-a537-132fd7945465 metadata using the provided image 6413a4d1-c74f-41d1-b561-f2c936c0d979 metadata: cinder.exception.GlanceMetadataExists: Glance metadata cannot be updated, key signature_verified exists for volume i | 03:13 |
| zaitcev | 20-4926-a537-132fd794546 | 03:13 |
| zaitcev | What in the world does this mean? | 03:14 |
| whoami-rajat | zaitcev, does the logs show that the initial create volume request failed and it was rescheduled? | 03:41 |
| zaitcev | That error is the first relevant error. | 03:42 |
| zaitcev | I don't think it's re-scheduled, but I don't know. | 03:42 |
| whoami-rajat | zaitcev, i had a patch related to this https://review.opendev.org/c/openstack/cinder/+/800768 | 03:43 |
| whoami-rajat | i saw it when creating instance from BFV | 03:43 |
| zaitcev | oh, maybe it is similar. What happened here was, I created that image from an existing volume, by using "openstack image create --volume 801478a6-596f-486e-a3f7-f3c16cde65c2 test_1G2". | 03:46 |
| zaitcev | Then, I deleted the original volume, in cinder. That caused the underlying LVM volume get deleted, although not the cache. | 03:47 |
| zaitcev | BTW, when creating volume fails, cinder leaves two LVM volumes just laying around: the volume in question (in state "error") and an image cache volume. | 03:48 |
| whoami-rajat | but the glance metadata is filtered out with volume_id so even if we are uploading volume as an image and again creating volume from it, the ID will be different and this error shouldn't appear | 03:52 |
| whoami-rajat | yeah the cleanup is a bit manual, you will need to reset-state for the volume and then delete it | 03:53 |
| whoami-rajat | reset-state to available | 03:53 |
| zaitcev | I thought so too. The "cinder create" command was exactly and completely as I quoted above, and I didn't provide volume_id. | 03:53 |
| zaitcev | oh, thanks | 03:53 |
| whoami-rajat | the volume id is a random uuid generated so it should always be unique | 04:00 |
| whoami-rajat | the case you mentioned can only error out if we're rescheduling the same volume for create and the first attempt was not properly cleaned up | 04:02 |
| whoami-rajat | https://github.com/openstack/cinder/blob/3f0f49bd8e9670da3588527329f35bf5dc40948f/cinder/volume/flows/manager/create_volume.py#L73 | 04:02 |
| *** bhagyashris is now known as bhagyashris|ruck | 07:06 | |
| *** akekane is now known as abhishekk | 07:38 | |
| *** dviroel|out is now known as dviroel | 11:02 | |
| opendevreview | Jorge Merlino proposed openstack/cinder master: Increase size of volume image metadata values https://review.opendev.org/c/openstack/cinder/+/868485 | 12:53 |
| whoami-rajat | hi rosmaita jungleboyj, can you take a look at this spec? https://review.opendev.org/c/openstack/cinder-specs/+/866718 | 13:16 |
| harsh | Hello reviewers, i got two +2's on the review: https://review.opendev.org/c/openstack/cinder/+/853126 but after zuul is failing with tox errors. | 13:47 |
| harsh | anyone has any idea on this? | 13:47 |
| harsh | Before the +2's zuul and CI passed successfully. | 13:49 |
| whoami-rajat | harsh, looks like a tox4 issue | 14:19 |
| harsh | yes. in the last wednesday cinder meeting we were discussing on this that how the tox4 change will result. | 14:23 |
| harsh | not sure how to resolve this though | 14:23 |
| whoami-rajat | there is a patch up which I'm reviewing which should fix this | 14:24 |
| harsh | ok. | 14:24 |
| harsh | can you share the patch link? i will also go through it | 14:24 |
| whoami-rajat | https://review.opendev.org/c/openstack/cinder/+/868253 | 14:25 |
| harsh | thanks | 14:26 |
| whoami-rajat | np | 14:27 |
| whoami-rajat | the patch is merging now so gate should be fixed in some time (hopefully) | 14:27 |
| harsh | yea this looks like might solve the issue because in the logs i can see the error: whitespace in passenv values | 14:27 |
| harsh | sure. thanks for the support whoami-rajat | 14:28 |
| whoami-rajat | no worries | 14:28 |
| sfv880_ | Hello reviewers, I addressed all the comments on https://review.opendev.org/c/openstack/cinder/+/852369 and https://review.opendev.org/c/openstack/cinder/+/864287 - our customers are waiting for these fixes to be merged. Also, these changes are required to pass the RHOSP certification. Could you please review it ? Thank you so much! | 15:10 |
| *** dviroel is now known as dviroel|lunch | 15:28 | |
| opendevreview | Merged openstack/cinder master: Get ready for tox 4 https://review.opendev.org/c/openstack/cinder/+/868253 | 16:04 |
| opendevreview | Merged openstack/rbd-iscsi-client master: Get ready for tox 4 https://review.opendev.org/c/openstack/rbd-iscsi-client/+/868324 | 16:27 |
| *** dviroel|lunch is now known as dviroel | 16:41 | |
| opendevreview | Merged openstack/os-brick master: Get ready for tox 4 https://review.opendev.org/c/openstack/os-brick/+/868315 | 17:25 |
| opendevreview | Jorge Merlino proposed openstack/cinder master: Increase size of volume image metadata values https://review.opendev.org/c/openstack/cinder/+/868485 | 18:34 |
| *** dviroel is now known as dviroel|out | 19:28 | |
| opendevreview | Jorge Merlino proposed openstack/cinder master: Increase size of volume image metadata values https://review.opendev.org/c/openstack/cinder/+/868485 | 22:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!