| dansmith | croelandt: rosmaita: Can you hit this fix for the glance periodic? https://review.opendev.org/c/openstack/glance/+/896124 | 13:44 |
|---|---|---|
| rosmaita | looking | 13:44 |
| dansmith | rosmaita: context is a previous patch tried to disable global venv for this job because centos/fips but wasn't quite complete | 13:52 |
| rosmaita | i remember seeing that patch, and that you were skeptical about it | 13:52 |
| rosmaita | are you going to kill me if i 'check experimental' on your patch? | 13:53 |
| rosmaita | dansmith: left a question for you on https://review.opendev.org/c/openstack/glance/+/896124 | 13:58 |
| dansmith | rosmaita: replied | 14:01 |
| dansmith | rosmaita: no, but does anything in experimental inherit from this? | 14:02 |
| dansmith | rosmaita: you can see in PS2 that it ran the job in check and passed, I just did that to avoid needing to run two sets of jobs against just a job change while I got the devstack patch this depends-on working | 14:03 |
| dansmith | but if it'll make you feel better, go for it | 14:03 |
| rosmaita | i didn't see it in periodic (didn't look carefully enough), it's listed in experimental | 14:03 |
| dansmith | not periodic, check | 14:04 |
| dansmith | it's back in periodic *now* | 14:04 |
| rosmaita | what i meant was, i did not see this job running anywhere, but it is listed in the experimental pipeline in .zuul.yaml | 14:04 |
| dansmith | rosmaita: https://review.opendev.org/c/openstack/glance/+/896124/3/.zuul.yaml#440 | 14:05 |
| dansmith | in PS2 I ran it in check to verify that it was working: https://review.opendev.org/c/openstack/glance/+/896124/2/.zuul.yaml#355 | 14:06 |
| rosmaita | yeah, i see it there *now* but didnt' see any evidence of it running ... forgot about the "show all entries" in the gerrit ui | 14:07 |
| dansmith | then removed it from check so it's back to periodic only between 2-3: https://review.opendev.org/c/openstack/glance/+/896124/2..3/.zuul.yaml | 14:07 |
| rosmaita | yep, that's what i was looking for | 14:07 |
| dansmith | okay sorry if I made that too complicated | 14:12 |
| rosmaita | dansmith: not at all, i was in the middle of something else, and a context-switch to zuul on a monday morning ... just was not sufficiently caffeinated to handle it | 14:16 |
| opendevreview | Merged openstack/glance master: db: Re-use a single connection https://review.opendev.org/c/openstack/glance/+/889380 | 14:21 |
| opendevreview | Merged openstack/python-glanceclient master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/python-glanceclient/+/895483 | 14:56 |
| opendevreview | Merged openstack/glance_store master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/glance_store/+/895480 | 14:57 |
| opendevreview | Merged openstack/glance master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/glance/+/895895 | 15:02 |
| opendevreview | Lajos Katona proposed openstack/glance master: Round up to an integer megabyte when calculating usage https://review.opendev.org/c/openstack/glance/+/866584 | 15:52 |
| lajoskatona | rosmaita, dansmith: Hi, there's a long hanging patch related to image size calculation | 15:54 |
| lajoskatona | rosmaita, dansmith: the lp bug: https://bugs.launchpad.net/glance/+bug/1998820 and the patch https://review.opendev.org/c/openstack/glance/+/866584 | 15:55 |
| rosmaita | lajoskatona: thanks, will take a look a bit later | 15:55 |
| lajoskatona | rosmaita, dansmith: thanks, if anything else is needed please comment it, and me or my colleague come back with it :-) | 15:56 |
| dansmith | lajoskatona: quota enforcement is more stop-the-bleeding, because you can run over your quota with a single operation if we don't know the size ahead of time | 16:07 |
| dansmith | so I mean, I guess we can do that, but it seems a little pointless given that we don't stop you at exactly the number of MiB you have quota for | 16:07 |
| opendevreview | Cyril Roelandt proposed openstack/glance_store stable/2023.1: RBD: Trash image when snapshots prevent deletion https://review.opendev.org/c/openstack/glance_store/+/896447 | 17:23 |
| opendevreview | Merged openstack/glance master: Remove 'egg_info' configuration https://review.opendev.org/c/openstack/glance/+/889069 | 18:55 |
| opendevreview | Merged openstack/glance master: Unset VENV python_interpreter in fips job https://review.opendev.org/c/openstack/glance/+/896124 | 20:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!