| pdeore | abhishekk, rosmaita, dansmith, croelandt, mrjoshi glance weekly meeting in 10 minutes at #openstack-meeting | 13:49 |
|---|---|---|
| abhishekk | thanks | 13:52 |
| dansmith | abhishekk: read these comments: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/imagebackend.py#L1182-L1191 | 14:24 |
| abhishekk | ack | 14:25 |
| dansmith | I think that means as long as the fsid is the same, they don't have to be in the same pool | 14:25 |
| dansmith | I don't have a local ceph devstack up at the moment, but I can try to poke around with it later | 14:25 |
| abhishekk | cool, that will not be difficult to test it locally then | 14:26 |
| abhishekk | scenario will be, create image, boot an instance from it and then snapshot it using nova image-create (for time being in add_location I can change it to use new API call like cinder POC) | 14:27 |
| dansmith | yeah | 14:27 |
| dansmith | abhishekk: https://review.opendev.org/c/openstack/nova/+/891036 | 14:28 |
| abhishekk | thanks for the clarification | 14:28 |
| abhishekk | perfect | 14:28 |
| abhishekk | dansmith, Instance is going in error state and log says, | 15:19 |
| abhishekk | Image 41c7c20d-6424-43ba-9f16-a3eaca1bd42d copied to rbd store robust | 15:20 |
| abhishekk | Image locations are: [] | 15:20 |
| abhishekk | Instance failed to spawn: nova.exception.ImageUnacceptable: Image 41c7c20d-6424-43ba-9f16-a3eaca1bd42d is unacceptable: No image locations are accessible | 15:20 |
| dansmith | abhishekk: locally? with current code or with the new glance stuff? | 15:21 |
| abhishekk | locally, with new glance code, but it does not matter because new code is not invoked on instance creation, | 15:22 |
| abhishekk | because I haven't changed anything in nova yet | 15:22 |
| abhishekk | Image 41c7c20d-6424-43ba-9f16-a3eaca1bd42d is not on my ceph and [workarounds]/never_download_image_if_on_rbd=True; refusing to fetch and upload.: nova.exception.ImageUnacceptable: Image 41c7c20d-6424-43ba-9f16-a3eaca1bd42d is unacceptable: No image locations are accessible | 15:22 |
| dansmith | do you have all the show locations things turned on? | 15:22 |
| abhishekk | oops | 15:23 |
| dansmith | this definitely works locally for me | 15:23 |
| abhishekk | you got me | 15:23 |
| abhishekk | dansmith, I am getting not found exception here while creating the snapshot | 16:07 |
| abhishekk | https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/rbd.py#L422 | 16:07 |
| abhishekk | looks like it fails to read image rbd.ImageNotFound: [errno 2] RBD image not found (error opening image b'948d7037-d710-4d11-8919-8d07b4abf299' at snapshot b'snap') | 16:08 |
| abhishekk | from 'images' pool | 16:08 |
| dansmith | abhishekk: is that when it adds the location in glance and glance can't find it? | 16:09 |
| abhishekk | yeah | 16:09 |
| abhishekk | The call comes to new location api, and when glance tries to find it, it fails | 16:10 |
| abhishekk | I will revert the changes and see it is working with old loaction add workflow | 16:11 |
| dansmith | it comes to the new one? | 16:11 |
| abhishekk | yes | 16:11 |
| dansmith | https://zuul.opendev.org/t/openstack/build/4884721dbf764b33995d71c990f65dfe/log/controller/logs/screen-n-cpu.txt#16096 | 16:11 |
| dansmith | it clearly works in the existing stuff | 16:11 |
| abhishekk | ack | 16:12 |
| abhishekk | still, I will verify if something is wrong in my local environment or not | 16:13 |
| dansmith | why is this hitting the new API if glanceclient isn't doing the new..or..old logic? | 16:13 |
| abhishekk | I am directly calling the new api from nova | 16:14 |
| dansmith | oh okay | 16:15 |
| abhishekk | working with old | 16:15 |
| dansmith | this is why we should test the new code with the client services before we merge :D | 16:15 |
| abhishekk | I think I got the problem here | 16:15 |
| abhishekk | if we remove this line here, https://review.opendev.org/c/openstack/glance/+/886749/14/glance/async_/flows/location_import.py#213 it will work | 16:16 |
| dansmith | um, really? | 16:20 |
| abhishekk | i think so,not sure | 16:21 |
| abhishekk | nope | 16:24 |
| abhishekk | with old API it fetches size https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/rbd.py#L422 correctly | 16:35 |
| abhishekk | but for new API it raises not found error at same line :D | 16:35 |
| * abhishekk signing out for the day | 17:56 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!