Thursday, 2021-08-12

opendevreviewIan Wienand proposed openstack/puppet-openstack-integration stable/ussuri: Remove Debian Stable jobs  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/80431000:35
opendevreviewMerged openstack/puppet-openstack-integration stable/ussuri: Remove Debian Stable jobs  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/80431004:01
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Add support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-keystone/+/80432504:12
*** ykarel|away is now known as ykarel04:57
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Add support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-keystone/+/80432505:47
opendevreviewTakashi Kajinami proposed openstack/puppet-aodh master: Add support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-aodh/+/80435005:59
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Add support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-keystone/+/80432506:01
opendevreviewTakashi Kajinami proposed openstack/puppet-aodh master: Add support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-aodh/+/80435006:03
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Add support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-keystone/+/80432506:04
opendevreviewTakashi Kajinami proposed openstack/puppet-aodh master: Add support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-aodh/+/80435006:04
*** rpittau|afk is now known as rpittau06:22
ykareltkajinam, wrt your comment in https://review.rdoproject.org/r/c/testproject/+/34808, even after connection up you don't see the ip with ip a ?06:31
tkajinamykarel, yeah06:31
tkajinamykarel, but the ip is set on a connection according to nmcli conn show06:32
tkajinamwhen I set an ip on a ovs bridge06:32
tkajinamwhen I set an ip on a ovs port then it is not shown even in nmcli conn show which I'm looking at now06:32
ykareltkajinam, you ran nmcli connection up br-ex-ovs-interface and still no ip?06:32
tkajinamI mean ip is not shown06:33
tkajinamykarel, connections are all up06:33
ykarelin my tests ip was visible after connection up06:33
tkajinamykarel, I restarted connections but it didn't solve the problem06:33
tkajinamhmm06:33
ykarelokk will check again06:33
ykarelwill hold the node and check06:34
*** dpawlik6 is now known as dpawlik07:07
*** jpena|off is now known as jpena07:33
opendevreviewTakashi Kajinami proposed openstack/puppet-aodh master: Add support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-aodh/+/80435007:53
opendevreviewTakashi Kajinami proposed openstack/puppet-keystone master: Add support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-keystone/+/80432508:08
*** ykarel is now known as ykarel|lunch08:27
opendevreviewTakashi Kajinami proposed openstack/puppet-aodh master: Add support for the system_scope parameter  https://review.opendev.org/c/openstack/puppet-aodh/+/80435008:38
opendevreviewTakashi Kajinami proposed openstack/puppet-openstack-integration master: Ubuntu: Enforce utf8mb3 charset in MySQL  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/80436508:49
opendevreviewTakashi Kajinami proposed openstack/puppet-openstack-integration master: Ubuntu: Use utf8mb3 charset in MySQL  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/80436508:51
tkajinamtobias-urdin, hi. are you around ?09:31
*** ykarel|lunch is now known as ykarel09:46
tkajinamtobias-urdin, ignore it. I was wondering whether you want to keep puppet-monasca. see my mail in community ml to find some details10:21
opendevreviewTakashi Kajinami proposed openstack/puppet-openstack-integration master: Ubuntu: Use utf8mb3 charset in MySQL  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/80436510:29
*** jpena is now known as jpena|lunch11:33
*** beagles is now known as eagles11:35
ykareltkajinam, yes correct, connection was up but no ip on br-ex11:37
ykarelonly after restart of NetworkManager i can see the ip11:37
tkajinamthat's quite interesting > only after restart of NetworkManager12:01
tkajinamlet me check that in my env as well12:02
tkajinamhmm... I restarted NetworkManager but it doesn't work12:10
ykarelstrange12:10
tkajinamthat's what I always feel with NetworkManager :-)12:11
* tkajinam sighs12:12
tkajinamykarel, do you mind sharing output of ip a, and nmcli conn, nmcli conn show <connection>12:12
tkajinamafter ip is "activated" by restarting NetworkManager12:12
ykareltkajinam, sure, if you want access to vm, can add your keys12:13
tkajinamthat's awesome12:14
tkajinamgive me a min12:14
ykarelack12:15
tkajinamykarel, https://github.com/kajinamit.keys12:16
* tkajinam was looking for the smartest way to share his public key :-P12:16
ykareltkajinam, ssh zuul-worker@38.102.83.8912:17
tkajinamthx !12:17
*** jpena|lunch is now known as jpena12:24
tkajinamykarel, it seems the current ips is not one assigned to br-ex but the one assigned to br-ex-ovs-interface12:28
tkajinamI mean the ip shown in ip a12:28
ykareltkajinam, shouldn't that be the case?12:29
tkajinaminterestingly there are wto br-ex-ovs-interface connections... maybe because of broken idepotency12:29
ykarelthe interface ip get's visible in br-ex itself12:29
ykareltkajinam, two connections i think caused with my nmcli c up and down12:30
ykarelinitially there was only 1 br-ex-ovs-interface connection12:30
ykareland was up12:30
ykarelso i tried nmcli c up again, but that timedout, then down that too timedout,12:30
tkajinamso if the implementation works as intendend, br-ex should have two ips12:30
tkajinamone migrated from loop1, which is 127.2.0.1/2412:31
tkajinamand the other one created after br-ex bridge is created, 172.24.5.112:31
ykarelwithout NM it has only 1 ip https://logserver.rdoproject.org/78/33878/4/check/periodic-xena-puppet-scenario001-centos-stream-9/29e71b4/logs/ip.txt.gz12:32
ykarelnot sure why with NM should have 2 ips12:32
tkajinamhmmm maybe I misunderstand something12:33
*** jpena is now known as jpena|off12:33
tkajinamlet me test the same in my env. I didn't try adding ip on br-ex12:33
tkajinamI mean I didn't have that create_br-ex_vif resource in my test manifest...12:34
ykarelahh12:34
ykarelthat seems related12:34
tkajinamohk12:35
tkajinamgood news. I reproduced what you mentioned12:35
tkajinambad news. so there is something wrong12:35
tkajinam:-P12:35
ykarelthat's good progress :)12:35
tkajinamyeah we are on the same page12:35
tkajinamI still don't understand why we need to add that ip on br-ex. if ip is properly migrated from loop1 to br-ex then we don't need additional ip there12:36
tkajinamI might misunderstand expected behavior of vs_port without NetworkManager12:36
tkajinambut anyway we can focus on the issue with adding ip on br-ex now12:37
opendevreviewTakashi Kajinami proposed openstack/puppet-openstack-integration master: Add network manager support  https://review.opendev.org/c/openstack/puppet-openstack-integration/+/80163212:44
tkajinamykarel, ^^^12:44
tkajinamseems conn modify is not applied immediately12:44
tkajinamand we need to restart the connection12:44
ykareltkajinam, okk let's try12:45
ykarelme rechecks test patch12:45
tkajinamhope this works better12:54
ykareltkajinam, hmm i see the ip now on br-ex13:25
ykareljob should move further this time13:25
tkajinamykarel, :-D13:25
tkajinamykarel, so restarting the interface was a missing piece13:25
ykarelyes13:25
tkajinamI don't see any available option to update ip immediately by NetworkManager13:26
ykarelme neither13:26
*** ysandeep|PTO is now known as ysandeep13:26
tkajinamone option is to set ip immediately by ip command while we persist that using nmcli command13:30
tkajinamwhich sounds like very strange... tbh 13:32
tkajinamso scenario001 job passed13:34
ykarelyeap13:39
opendevreviewMerged openstack/puppet-manila stable/wallaby: Drop tenant_id templating from v2 endpoint  https://review.opendev.org/c/openstack/puppet-manila/+/80294014:25
*** jpena|off is now known as jpena14:43
opendevreviewTakashi Kajinami proposed openstack/puppet-ironic master: Fix the wrong method name  https://review.opendev.org/c/openstack/puppet-ironic/+/80440114:55
opendevreviewTakashi Kajinami proposed openstack/puppet-ironic master: Fix the wrong function used for warning message  https://review.opendev.org/c/openstack/puppet-ironic/+/80440114:56
opendevreviewTakashi Kajinami proposed openstack/puppet-ironic stable/victoria: Fix the wrong function used for warning message  https://review.opendev.org/c/openstack/puppet-ironic/+/80440214:57
opendevreviewTakashi Kajinami proposed openstack/puppet-ironic stable/wallaby: Fix the wrong function used for warning message  https://review.opendev.org/c/openstack/puppet-ironic/+/80440314:57
opendevreviewTakashi Kajinami proposed openstack/puppet-ironic stable/victoria: Fix the wrong function used for warning message  https://review.opendev.org/c/openstack/puppet-ironic/+/80440214:58
opendevreviewTakashi Kajinami proposed openstack/puppet-ironic master: Fix the wrong function name  https://review.opendev.org/c/openstack/puppet-ironic/+/80440114:59
opendevreviewTakashi Kajinami proposed openstack/puppet-ironic stable/wallaby: Fix the wrong function name  https://review.opendev.org/c/openstack/puppet-ironic/+/80440315:00
opendevreviewTakashi Kajinami proposed openstack/puppet-ironic stable/victoria: Fix the wrong function name  https://review.opendev.org/c/openstack/puppet-ironic/+/80440215:01
opendevreviewTakashi Kajinami proposed openstack/puppet-ironic stable/ussuri: Fix the wrong function name  https://review.opendev.org/c/openstack/puppet-ironic/+/80440415:02
*** mschuppert is now known as mschuppert|pto15:21
*** rpittau is now known as rpittau|afk15:31
*** jpena is now known as jpena|off15:34
*** ykarel is now known as ykarel|away16:31
opendevreviewJulia Kreger proposed openstack/puppet-ironic stable/victoria: Fix ppc64le support to coexist with PXE preference  https://review.opendev.org/c/openstack/puppet-ironic/+/80434816:46
opendevreviewJulia Kreger proposed openstack/puppet-ironic stable/ussuri: Fix ppc64le support to coexist with PXE preference  https://review.opendev.org/c/openstack/puppet-ironic/+/80434916:46
opendevreviewJulia Kreger proposed openstack/puppet-ironic stable/train: Fix ppc64le support to coexist with PXE preference  https://review.opendev.org/c/openstack/puppet-ironic/+/80443016:47
opendevreviewMerged openstack/puppet-octavia master: Drop quota parameters without default_ prefix  https://review.opendev.org/c/openstack/puppet-octavia/+/80417318:45

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!