| *** pojadhav|out is now known as pojadhav | 01:35 | |
| *** soniya29|ruck is now known as soniya29 | 04:56 | |
| *** pojadhav is now known as pojadhav|ruck | 06:47 | |
| opendevreview | Slawek Kaplonski proposed openstack/devstack master: Install libssl dev package if tls service is enabled https://review.opendev.org/c/openstack/devstack/+/854745 | 07:21 |
|---|---|---|
| *** jpena|off is now known as jpena | 07:52 | |
| *** pojadhav is now known as pojadhav|ruck | 08:57 | |
| *** pojadhav is now known as pojadhav|ruck | 09:03 | |
| slaweq | kopecmartin frickler hi, can You check patch https://review.opendev.org/c/openstack/devstack/+/854745/ and patches which are below it? We need those 3 patches to make neutron-ovn-tempest-ovs-master-centos-9-stream green | 09:03 |
| slaweq | that's tested in DNM https://review.opendev.org/c/openstack/neutron/+/854191 | 09:03 |
| *** pojadhav|ruck is now known as pojadhav|lunch | 09:22 | |
| frickler | slaweq: do you intend to backport those fixes, too, or are they only for master? in the former case, referencing a bug report would be useful | 09:43 |
| frickler | slaweq: I also think it would be easier then to squash those three patches into one | 09:44 |
| slaweq | frickler yes, I think it would be good to backport them | 10:17 |
| slaweq | I will squash all those patches into one and will open LP for it | 10:17 |
| *** pojadhav- is now known as pojadhav|ruck | 10:23 | |
| opendevreview | Slawek Kaplonski proposed openstack/devstack master: Fix installation of OVS/OVN from sources https://review.opendev.org/c/openstack/devstack/+/854770 | 11:00 |
| slaweq | frickler ^^ I squashed those 3 small patches into one | 11:03 |
| slaweq | and it's linked to the LP bug now | 11:03 |
| slaweq | please check when You will have time | 11:04 |
| frickler | slaweq: ack, thx | 11:37 |
| *** sean-k-mooney1 is now known as sean-k-mooney | 11:47 | |
| abishop | hi cores, https://review.opendev.org/c/openstack/tempest/+/852470 needs reviews (it has one +2 already) | 13:37 |
| slaweq | gmann kopecmartin if You will have a minute, please look at https://review.opendev.org/c/openstack/devstack/+/854770 | 14:17 |
| slaweq | thx in advance | 14:17 |
| *** jpena is now known as jpena|off | 14:29 | |
| gmann | slaweq: +W, i missed yesterday while waiting for result in DNM patch. | 14:32 |
| gmann | and +1 on backport | 14:32 |
| slaweq | gmann thx a lot | 14:32 |
| gmann | abishop: done, 1 comment, please let me know if you want to address that in same patch or folllowup, both work for me https://review.opendev.org/c/openstack/tempest/+/852470/4/tempest/lib/services/volume/v3/transfers_client.py#86 | 14:39 |
| gmann | abishop: if in followup then I can +w the current one | 14:39 |
| abishop | gmann: thx, I'll take a look. my preference is usually to fix the patch now and not carry tech debt that requires a follow up | 14:41 |
| gmann | abishop: ok, if you fix in same patch then also I can +W as we have masayukig +2 there in current version. so will not leave it for 2nd review | 14:42 |
| abishop | gmann: ack, thx! | 14:43 |
| masayukig | thx! | 14:46 |
| *** pojadhav|ruck is now known as pojadhav|out | 14:55 | |
| opendevreview | Alan Bishop proposed openstack/tempest master: Update volume and volume transfer schema https://review.opendev.org/c/openstack/tempest/+/852470 | 15:44 |
| opendevreview | Sofia Enriquez proposed openstack/devstack-plugin-nfs master: Runs all encrypted scenario tests https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/742905 | 15:54 |
| opendevreview | Merged openstack/devstack master: Fix installation of OVS/OVN from sources https://review.opendev.org/c/openstack/devstack/+/854770 | 16:17 |
| opendevreview | Merged openstack/whitebox-tempest-plugin master: Add vGPU inventory check of Resource Providers https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/852469 | 18:19 |
| opendevreview | Tony Breeds proposed openstack/devstack stable/yoga: Respect constraints on tempest venv consistently https://review.opendev.org/c/openstack/devstack/+/854807 | 19:59 |
| opendevreview | Merged openstack/whitebox-tempest-plugin master: Attach and detach an SR-IOV port https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/845382 | 20:48 |
| opendevreview | Merged openstack/tempest master: Update volume and volume transfer schema https://review.opendev.org/c/openstack/tempest/+/852470 | 20:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!