| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Replace deprecated ceilometer::agent::auth https://review.opendev.org/c/openstack/puppet-openstack-integration/+/803658 | 01:09 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Replace deprecated aodh::auth https://review.opendev.org/c/openstack/puppet-openstack-integration/+/803659 | 01:11 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tempest master: Fix the wrong default of tempest::panko_available https://review.opendev.org/c/openstack/puppet-tempest/+/803660 | 01:13 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Replace deprecated notification_worker https://review.opendev.org/c/openstack/puppet-openstack-integration/+/803661 | 01:15 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Replace deprecated notification_workers https://review.opendev.org/c/openstack/puppet-openstack-integration/+/803661 | 01:17 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Use independent key_manager classes for castellan options https://review.opendev.org/c/openstack/puppet-openstack-integration/+/803662 | 01:23 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone master: Replace deprecated is_string method https://review.opendev.org/c/openstack/puppet-keystone/+/803663 | 01:36 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-oslo master: Replace deprecated is_string method https://review.opendev.org/c/openstack/puppet-oslo/+/803664 | 01:39 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Replace deprecated is_string/is_array method https://review.opendev.org/c/openstack/puppet-nova/+/803666 | 01:47 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-swift master: Replace deprecated is_array method https://review.opendev.org/c/openstack/puppet-swift/+/803667 | 01:50 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone master: Replace deprecated is_string method https://review.opendev.org/c/openstack/puppet-keystone/+/803663 | 03:32 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-oslo master: Replace deprecated is_string method https://review.opendev.org/c/openstack/puppet-oslo/+/803664 | 03:36 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-nova master: Replace deprecated is_string/is_array method https://review.opendev.org/c/openstack/puppet-nova/+/803666 | 03:37 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-swift master: Replace deprecated is_array method https://review.opendev.org/c/openstack/puppet-swift/+/803667 | 03:39 |
| opendevreview | Michael Johnson proposed openstack/puppet-octavia master: TripleO: Reduce tempest test scope https://review.opendev.org/c/openstack/puppet-octavia/+/801400 | 05:03 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-swift master: Replace deprecated is_array method https://review.opendev.org/c/openstack/puppet-swift/+/803667 | 05:13 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Replace deprecated ceilometer::agent::auth https://review.opendev.org/c/openstack/puppet-openstack-integration/+/803658 | 05:17 |
| *** ykarel|away is now known as ykarel | 05:17 | |
| ykarel | tkajinam, hi | 05:17 |
| tkajinam | ykarel, hey | 05:18 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Use independent key_manager classes for castellan options https://review.opendev.org/c/openstack/puppet-openstack-integration/+/803662 | 05:18 |
| ykarel | tkajinam, Thanks for rechecking test patch | 05:18 |
| ykarel | i see it's failed and looks same i got while testing locally | 05:18 |
| ykarel | making connection up at https://review.opendev.org/c/openstack/puppet-openstack-integration/+/801632/3/manifests/neutron.pp#69 i think should resolve it | 05:19 |
| ykarel | && nmcli c up br-ex-ovs-interface | 05:19 |
| tkajinam | ykarel, in which while did you find error ? | 05:21 |
| ykarel | tkajinam, tempest tests failed | 05:21 |
| ykarel | while trying to reach vms via fip | 05:21 |
| tkajinam | ahh ok | 05:21 |
| tkajinam | I was just looking at puppet logs and didn't see anything there | 05:21 |
| ykarel | yeap puppet went fine | 05:22 |
| tkajinam | didn't see any failures I mean | 05:22 |
| tkajinam | ack | 05:22 |
| tkajinam | I think there are still some issue with address management | 05:23 |
| tkajinam | my last update was only around missing configuration to enable ovs plugin | 05:23 |
| tkajinam | so the logic in provider is not yet updated, which I should check next | 05:24 |
| ykarel | tkajinam, is enabling ovs plugin really needed, atleast in my tests i didn't faced any issue without it | 05:25 |
| ykarel | current failure atleast should be resoved if connection is made up | 05:25 |
| ykarel | i meant adding ovs in NetworkManager.conf needed | 05:26 |
| tkajinam | ykarel, when I initially tested the patch in my env installing NetworkManager-ovs was not enough and I needed that update in cof | 05:26 |
| tkajinam | conf | 05:26 |
| ykarel | what issue you faced without that setting? | 05:27 |
| ykarel | may be i not tested that case | 05:27 |
| tkajinam | NetworkManager rejected enabling ovs interface | 05:27 |
| tkajinam | saying ovs-bridge is not supported | 05:27 |
| tkajinam | but it might be because I didn't restart NetworkManager after I installed NetworkManager-ovs | 05:27 |
| tkajinam | I don't remember precise steps I did because I did most of work after 10pm :-p | 05:28 |
| tkajinam | let me check it again... though adding that to .conf doesn't harm at this moment, IMO | 05:28 |
| ykarel | :) ok np, strange i didn't faced it | 05:28 |
| ykarel | yeap but i see you removed default that's why thought if that needed or not | 05:28 |
| tkajinam | ykarel, yeah that's one concerning point with current implementation | 05:30 |
| tkajinam | which I mentioned in my TODO comment | 05:30 |
| ykarel | tkajinam, i think NetworkManager-ovs installation triggers restart for NetworkManager | 05:30 |
| tkajinam | ykarel, in puppet, right ? | 05:31 |
| ykarel | rpm installation | 05:31 |
| tkajinam | hm | 05:31 |
| ykarel | so with puppet should be same | 05:31 |
| tkajinam | ook | 05:31 |
| ykarel | yes, i see systemctl --no-reload preset NetworkManager.service NetworkManager-wait-online.service NetworkManager-dispatcher.service in install script of NetworkManager-ovs | 05:32 |
| tkajinam | hm | 05:32 |
| tkajinam | I'll make some more trial so test without updating NetworkManager.conf | 05:32 |
| ykarel | ok, may be first can do connection up and see if all goes green with that | 05:33 |
| tkajinam | yup | 05:33 |
| ykarel | improvements can always be done | 05:34 |
| tkajinam | my current goal is to let poi job green. we have some todo items like refactoring, update path and etc but at the moment we can focus on making poi happy | 05:34 |
| tkajinam | yeah. I totally agree | 05:34 |
| *** jpena|off is now known as jpena | 07:36 | |
| *** ykarel is now known as ykarel|lunch | 09:06 | |
| opendevreview | Merged openstack/puppet-ironic master: Fix ppc64le support to coexist with PXE preference https://review.opendev.org/c/openstack/puppet-ironic/+/803537 | 09:59 |
| opendevreview | Merged openstack/puppet-ironic master: Add support for policy settings of ironic-inspector https://review.opendev.org/c/openstack/puppet-ironic/+/803535 | 09:59 |
| opendevreview | Martin Schuppert proposed openstack/puppet-nova master: Delete policy file if policies hash is empty https://review.opendev.org/c/openstack/puppet-nova/+/802251 | 10:11 |
| opendevreview | Merged openstack/puppet-ironic stable/wallaby: Clean up ipxe support https://review.opendev.org/c/openstack/puppet-ironic/+/801484 | 10:16 |
| *** ykarel|lunch is now known as ykarel | 10:23 | |
| opendevreview | Merged openstack/puppet-ironic stable/victoria: We shouldn't set pxe/ipxe_enabled in ironic config https://review.opendev.org/c/openstack/puppet-ironic/+/802346 | 10:39 |
| opendevreview | Merged openstack/puppet-ironic stable/victoria: Clean up ipxe support https://review.opendev.org/c/openstack/puppet-ironic/+/802347 | 11:11 |
| *** jpena is now known as jpena|lunch | 11:26 | |
| *** ykarel_ is now known as ykarel | 11:34 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Replace deprecated memcached::listen_ip https://review.opendev.org/c/openstack/puppet-openstack-integration/+/803735 | 11:40 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: qdr: replace yes/no by true/false https://review.opendev.org/c/openstack/puppet-openstack-integration/+/803739 | 12:12 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Gnocchi: Use rados library in CentOS https://review.opendev.org/c/openstack/puppet-openstack-integration/+/803740 | 12:17 |
| *** jpena|lunch is now known as jpena | 12:31 | |
| *** beagles is now known as brb | 13:18 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-heat master: Migrate max_stacks_per_tenant to heat::engine https://review.opendev.org/c/openstack/puppet-heat/+/803751 | 13:56 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: TripleO: Reduce tempest test scope https://review.opendev.org/c/openstack/puppet-octavia/+/801400 | 14:32 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: TripleO: Use nested virt https://review.opendev.org/c/openstack/puppet-octavia/+/803769 | 14:32 |
| *** rpittau is now known as rpittau|afk | 14:43 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-heat master: Migrate max_stacks_per_tenant to heat::engine https://review.opendev.org/c/openstack/puppet-heat/+/803751 | 15:20 |
| *** jpena is now known as jpena|off | 15:34 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!