| opendevreview | Ian Wienand proposed openstack/devstack master: Update to Fedora 36 https://review.opendev.org/c/openstack/devstack/+/854334 | 04:43 |
|---|---|---|
| *** soniya29 is now known as soniya29|ruck | 05:50 | |
| opendevreview | Rajat Dhasmana proposed openstack/tempest master: Add test to rebuild volume backed instance https://review.opendev.org/c/openstack/tempest/+/831018 | 06:18 |
| *** jpena|off is now known as jpena | 07:34 | |
| *** soniya29|ruck is now known as soniya29|ruck|lunch | 08:09 | |
| *** soniya29|ruck|lunch is now known as soniya29|ruck | 09:37 | |
| *** maaritamm_ is now known as maaritamm | 11:06 | |
| *** simondodsley_ is now known as simondodsley | 11:06 | |
| *** rpittau_ is now known as rpittau | 11:06 | |
| *** vkmc_ is now known as vkmc | 11:06 | |
| *** TheJulia_ is now known as TheJulia | 11:06 | |
| *** clarkb is now known as Guest1034 | 11:14 | |
| *** pojadhav is now known as pojadhav|afk | 11:14 | |
| opendevreview | Abhishek Kekane proposed openstack/tempest master: Add test to rebuild volume backed instance https://review.opendev.org/c/openstack/tempest/+/831018 | 12:10 |
| opendevreview | Rajat Dhasmana proposed openstack/tempest master: Add test to rebuild volume backed instance https://review.opendev.org/c/openstack/tempest/+/831018 | 12:13 |
| opendevreview | Fernando Royo proposed openstack/devstack master: Run northd services as root user https://review.opendev.org/c/openstack/devstack/+/854371 | 12:14 |
| *** pojadhav|afk is now known as pojadhav | 12:35 | |
| *** kopecmartin_ is now known as kopecmartin | 13:09 | |
| *** soniya29|ruck is now known as soniya29|ruck|afk | 13:27 | |
| *** soniya29|ruck|afk is now known as soniya29|ruck | 13:40 | |
| *** mnasiadka_ is now known as mnasiadka | 14:25 | |
| *** dansmith_ is now known as dansmith | 14:29 | |
| dansmith | gmann: kopecmartin: The glance patch this is testing is now merged, so if we could get this test in, that'd be great: https://review.opendev.org/c/openstack/tempest/+/852608 | 14:45 |
| gmann | dansmith: ack, will do today | 14:47 |
| dansmith | thanks, it should be an easy one | 14:48 |
| gmann | dansmith: 1 comment on it - https://review.opendev.org/c/openstack/tempest/+/852608/6/tempest/api/image/v2/test_images.py#143 | 15:08 |
| dansmith | gmann: replied | 15:11 |
| gmann | dansmith: there you checked 'glance-download' but you are doing 'glance-direct' import at L143 | 15:14 |
| dansmith | oh, sorry you mean because I use that for the initial import, gotcha | 15:14 |
| gmann | yeah | 15:15 |
| opendevreview | Slawek Kaplonski proposed openstack/devstack stable/stein: Fix is_neutron_legacy_enabled function https://review.opendev.org/c/openstack/devstack/+/854144 | 15:16 |
| *** pojadhav is now known as pojadhav|out | 15:18 | |
| opendevreview | Dan Smith proposed openstack/tempest master: Add testing for glance-download import method https://review.opendev.org/c/openstack/tempest/+/852608 | 15:19 |
| dansmith | gmann: gah sorry | 16:12 |
| dansmith | I'm not actually sure why I used glance-direct in the first test anyway | 16:13 |
| opendevreview | Dan Smith proposed openstack/tempest master: Add testing for glance-download import method https://review.opendev.org/c/openstack/tempest/+/852608 | 16:15 |
| gmann | dansmith: ohk but as you are using glance-direct. in this test we need to check and rest other test are ok as there is no glance-direct used https://review.opendev.org/c/openstack/tempest/+/852608/7..8/tempest/api/image/v2/test_images.py#b140 | 16:22 |
| gmann | dansmith: it seems in PS8 you removed from there too | 16:22 |
| dansmith | gmann: I just pushed a rev to remove glance-direct from the first test | 16:22 |
| dansmith | yeah | 16:22 |
| gmann | dansmith: but that is still there https://review.opendev.org/c/openstack/tempest/+/852608/8/tempest/api/image/v2/test_images.py#143 | 16:23 |
| dansmith | crap, didn't snip enough lines | 16:24 |
| opendevreview | Dan Smith proposed openstack/tempest master: Add testing for glance-download import method https://review.opendev.org/c/openstack/tempest/+/852608 | 16:24 |
| dansmith | sorry, frantically doing two things at once | 16:24 |
| dansmith | wait | 16:26 |
| dansmith | that's not going to work | 16:26 |
| dansmith | gmann: ignore me, I'll circle back when I can actually pay attention to this, sorry | 16:26 |
| gmann | ohk, i did recheck on testing patch https://review.opendev.org/c/openstack/glance/+/852609/5 | 16:26 |
| gmann | sure | 16:27 |
| *** jpena is now known as jpena|off | 16:42 | |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Use a standard port for NFS https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/854436 | 16:42 |
| opendevreview | Dan Smith proposed openstack/tempest master: Add testing for glance-download import method https://review.opendev.org/c/openstack/tempest/+/852608 | 17:20 |
| dansmith | gmann: I have to run, just let this ^ run through the gate before you look | 17:20 |
| gmann | dansmith: ack, will check in evening | 17:21 |
| opendevreview | Merged openstack/tempest master: Allow free-form image import parameters https://review.opendev.org/c/openstack/tempest/+/852607 | 17:32 |
| opendevreview | Merged openstack/devstack master: Clean up n-net remnants https://review.opendev.org/c/openstack/devstack/+/853870 | 17:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!