| frickler | rosmaita: pranali: please check https://review.opendev.org/c/openstack/glance/+/910460 | 09:41 |
|---|---|---|
| *** tosky_ is now known as tosky | 11:36 | |
| rosmaita | frickler: ack | 12:38 |
| opendevreview | Merged openstack/glance master: Prepare for castellan 4.4.0 https://review.opendev.org/c/openstack/glance/+/910460 | 14:23 |
| croelandt | dansmith: can we +W this one? we both added +2, is there something else we are needing from abhishekk? https://review.opendev.org/c/openstack/glance/+/901649 | 14:46 |
| dansmith | croelandt: it only matters if the grenade patch gets merged, and the devstack, and the tempest one I think | 14:46 |
| abhishekk | this depends on grenade which is yet to merged | 14:46 |
| dansmith | abhishekk: I just commented on your tempest patch | 14:47 |
| dansmith | I'm not sure how that can work reliably (the existing stuff I mean) | 14:47 |
| abhishekk | ack, looking | 14:47 |
| dansmith | maybe we should talk in -qa | 14:48 |
| abhishekk | ack, I am also not sure, but this test is atleast a cycle or two old and never faced race about it | 14:49 |
| dansmith | because it gets lucky a lot? :) | 14:50 |
| dansmith | what's the periodic set to, 1m or more? | 14:50 |
| dansmith | and .. the caching is just by periodic, right? | 14:51 |
| abhishekk | no not periodic any more | 14:54 |
| dansmith | so it's just kicked off from the cache call? (sorry I should remember this) | 14:55 |
| dansmith | if so that makes it even more race-prone | 14:55 |
| abhishekk | https://github.com/openstack/glance/blob/master/glance/api/v2/cached_images.py#L247 | 14:56 |
| abhishekk | yes | 14:56 |
| dansmith | ack, replied | 14:58 |
| abhishekk | making changes, will push new patch in some time, thank you | 15:01 |
| abhishekk | dansmith, should I drop grenade change and glance grenade job from the list? | 17:43 |
| dansmith | abhishekk: it would seem right to me, but you might want to wait for some other comments on that patch | 17:44 |
| abhishekk | ok, tempest passed,now will add recheck on my patch where it will be tested | 17:44 |
| abhishekk | looks like nova-ceph-multistore is broken? | 19:27 |
| dansmith | yes, since this morning | 19:27 |
| dansmith | I've been pushing for the fix including just now in -qa | 19:27 |
| dansmith | basically ceph mirror deleted a file | 19:27 |
| abhishekk | ohh, thanks for looking into it | 19:28 |
| abhishekk | https://zuul.opendev.org/t/openstack/build/f20c31bf143b4714b1d1f343dda9c79d/log/job-output.txt#23369 | 19:50 |
| abhishekk | new job passed! | 19:51 |
| opendevreview | Rajat Dhasmana proposed openstack/glance_store master: Use normal credentials for legacy image update https://review.opendev.org/c/openstack/glance_store/+/911005 | 22:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!