| opendevreview | melanie witt proposed openstack/devstack master: DNM Create flavors with disk encryption when enabled https://review.opendev.org/c/openstack/devstack/+/864160 | 03:03 |
|---|---|---|
| opendevreview | Rodolfo Alonso proposed openstack/devstack master: [OVN] Add support for the Neutron OVN agent service https://review.opendev.org/c/openstack/devstack/+/904171 | 10:15 |
| opendevreview | Rodolfo Alonso proposed openstack/devstack master: [OVN] Add support for the Neutron OVN agent service https://review.opendev.org/c/openstack/devstack/+/904171 | 10:34 |
| opendevreview | Rodolfo Alonso proposed openstack/devstack master: [OVN] Add support for the Neutron OVN agent service https://review.opendev.org/c/openstack/devstack/+/904171 | 10:51 |
| ralonsoh | ^^ folks, do you know why this nested conditions check is failing? | 11:20 |
| ralonsoh | 2024-01-10 10:57:20.786079 | controller | /opt/stack/devstack/lib/neutron_plugins/ovn_agent: line 521: conditional binary operator expected | 11:20 |
| ralonsoh | --> https://review.opendev.org/c/openstack/devstack/+/904171/10/lib/neutron_plugins/ovn_agent#521 | 11:21 |
| opendevreview | Rodolfo Alonso proposed openstack/devstack master: [OVN] Add support for the Neutron OVN agent service https://review.opendev.org/c/openstack/devstack/+/904171 | 11:55 |
| frickler | ralonsoh: I think this would have been the solution: if [[ $( is_service_enabled q-ovn-metadata-agent neutron-ovn-metadata-agent ) || ... but your split is fine for me, too | 12:41 |
| ralonsoh | frickler, thanks! | 12:43 |
| frickler | or even simpler: if is_service_enabled q-ovn-metadata-agent neutron-ovn-metadata-agent || [[ ... | 12:45 |
| ralonsoh | I tried this one before but didn't work | 12:45 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: [WIP] Update nodes yaml to use cpu architecture https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/902168 | 15:20 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: [WIP] Update nodes yaml to use cpu architecture https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/902168 | 15:21 |
| opendevreview | James Parker proposed openstack/whitebox-tempest-plugin master: Update nodes yaml to use cpu architecture https://review.opendev.org/c/openstack/whitebox-tempest-plugin/+/902168 | 17:54 |
| gmann | kopecmartin: I agree with the new condition check in this, but we should check admin creds in different way not just value present in conf. comemented, please check https://review.opendev.org/c/openstack/tempest/+/903274 | 19:23 |
| *** jph4 is now known as jph | 23:48 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!