| ykarel | kopecmartin, dansmith can you also check pending ones https://review.opendev.org/q/I50951e077e73297d10b075677a440992d1e2fa91 | 11:36 |
|---|---|---|
| opendevreview | Markus Hentsch proposed openstack/tempest master: Add credential generation for manager personas https://review.opendev.org/c/openstack/tempest/+/924133 | 12:52 |
| tobias-urdin | nova and cinder ci is having issues with tempest failing due to instance quota being full | 12:57 |
| tobias-urdin | i assume this has something to do with https://opendev.org/openstack/tempest/commit/eccd82f3113c9f045a174572ea7c191732b267f1 or similar | 12:57 |
| tobias-urdin | that it doesn't cleanup the instances after testing the build failure? | 12:57 |
| tobias-urdin | Details: {'code': 403, 'message': 'Quota exceeded for instances: Requested 1, but already used 10 of 10 instances'} | 12:58 |
| tobias-urdin | tempest.api.image.v2.test_images_formats.ImagesFormatTest.test_compute_rejects_invalid | 12:58 |
| opendevreview | Markus Hentsch proposed openstack/tempest master: Add credential generation for manager personas https://review.opendev.org/c/openstack/tempest/+/924133 | 12:59 |
| tobias-urdin | dansmith: ^ | 13:00 |
| dansmith | tobias-urdin: shouldn't be related unless other things are failing | 13:28 |
| dansmith | tobias-urdin: got a link to a failure? | 13:28 |
| dansmith | the ceph job was failing yesterday because we weren't skipping those, but unless that skip didn't land.. | 13:29 |
| dansmith | tobias-urdin: https://review.opendev.org/c/openstack/nova/+/926215 | 13:29 |
| dansmith | ykarel: I'm not core on unmaintained | 13:32 |
| ykarel | dansmith, kopecmartin thx will look for someone else for unmaintained | 13:33 |
| kopecmartin | np .. sorry, i don't have +2 there :/ | 13:33 |
| opendevreview | Dan Smith proposed openstack/tempest master: Handle glance with fixed ISO handling https://review.opendev.org/c/openstack/tempest/+/926238 | 13:34 |
| tobias-urdin | dansmith: ack thanks, should be ok when that lands | 14:18 |
| opendevreview | Merged openstack/devstack stable/2023.1: Replacing usage of rdo-release rpm https://review.opendev.org/c/openstack/devstack/+/925748 | 15:15 |
| *** ykarel is now known as ykarel|away | 15:16 | |
| *** gthiemon1e is now known as gthiemonge | 16:00 | |
| opendevreview | Dan Smith proposed openstack/tempest master: Handle glance with fixed ISO handling https://review.opendev.org/c/openstack/tempest/+/926238 | 18:35 |
| dansmith | gmann: thanks for catching, I think this ^ is what we want, but my devstack machine is otherwise committed, so I'll review when the test report comes in | 18:36 |
| dansmith | actually, let me add needed-by | 18:36 |
| opendevreview | Dan Smith proposed openstack/tempest master: Handle glance with fixed ISO handling https://review.opendev.org/c/openstack/tempest/+/926238 | 18:36 |
| gmann | dansmith: but this will make status check as 'queued' for all image other than iso. you wanted to make 'active or queued' right | 18:39 |
| dansmith | oh dangit, for the first one yeah | 18:40 |
| dansmith | guh | 18:40 |
| gmann | here in L128 we can make active or queued https://review.opendev.org/c/openstack/tempest/+/926238/2/tempest/api/image/v2/test_images_formats.py#128 | 18:40 |
| gmann | and add a todo to make 'active' only and rest all same as PS2 | 18:41 |
| dansmith | gmann: we only want active/queued for iso though | 18:41 |
| dansmith | any other image type being queued there is wrong | 18:41 |
| gmann | dansmith: ah right, just if condition then. | 18:42 |
| opendevreview | Dan Smith proposed openstack/tempest master: Handle glance with fixed ISO handling https://review.opendev.org/c/openstack/tempest/+/926238 | 18:47 |
| dansmith | gmann: what about that ^? | 18:47 |
| gmann | dansmith: ++, looks good. waiting for test result | 18:48 |
| dansmith | ack | 18:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!