Wednesday, 2021-01-13

« Tuesday, 2021-01-12 Index Thursday, 2021-01-14 »
openstackgerritnorman shen proposed openstack/neutron master: Do not report ovs agent state if ovs is dead  https://review.opendev.org/c/openstack/neutron/+/77005800:11
*** macz_ has quit IRC00:15
*** ramishra has quit IRC00:38
*** ramishra has joined #openstack-neutron00:40
*** mlavalle has quit IRC00:47
*** icey has quit IRC01:15
*** icey has joined #openstack-neutron01:16
openstackgerritwdd proposed openstack/networking-sfc master: Update lower-constraints  https://review.opendev.org/c/openstack/networking-sfc/+/76965801:18
*** tkajinam has quit IRC01:41
*** tkajinam has joined #openstack-neutron01:42
*** armax has quit IRC01:47
*** baojg has joined #openstack-neutron01:52
*** armax has joined #openstack-neutron02:08
*** tkajinam has quit IRC02:09
*** tkajinam has joined #openstack-neutron02:10
*** armax has quit IRC02:10
*** macz_ has joined #openstack-neutron02:16
*** macz_ has quit IRC02:21
*** ccstone has quit IRC02:26
*** ccstone has joined #openstack-neutron02:26
*** zzzeek has quit IRC02:30
*** spatel has joined #openstack-neutron02:31
*** zzzeek has joined #openstack-neutron02:31
*** zzzeek has quit IRC02:36
*** zzzeek has joined #openstack-neutron02:40
*** armax has joined #openstack-neutron02:48
*** armax has quit IRC02:54
*** hamalq has quit IRC02:56
*** rcernin has quit IRC02:57
*** sapd1 has joined #openstack-neutron02:58
*** sapd1 has quit IRC03:03
*** rcernin has joined #openstack-neutron03:18
*** rcernin has quit IRC03:21
*** rcernin has joined #openstack-neutron03:21
*** sapd1 has joined #openstack-neutron03:36
*** congnt95 has joined #openstack-neutron03:43
*** sapd1 has quit IRC04:00
*** sapd1 has joined #openstack-neutron04:09
*** rcernin has quit IRC04:35
*** rcernin has joined #openstack-neutron04:35
*** ratailor has joined #openstack-neutron04:52
*** sapd1 has quit IRC05:01
*** rcernin_ has joined #openstack-neutron05:42
*** rcernin has quit IRC05:42
*** sapd1 has joined #openstack-neutron05:55
*** spatel has quit IRC05:58
*** gcheresh has joined #openstack-neutron06:10
*** hemanth_n has joined #openstack-neutron06:47
openstackgerritzhanghao proposed openstack/neutron master: [OVN] Implement router gateway IP QoS  https://review.opendev.org/c/openstack/neutron/+/74901207:13
*** ralonsoh has joined #openstack-neutron07:19
*** zzzeek has quit IRC07:28
*** rcernin_ has quit IRC07:28
*** ccamposr has joined #openstack-neutron07:29
*** tmorin has joined #openstack-neutron07:29
*** zzzeek has joined #openstack-neutron07:31
*** ccamposr__ has quit IRC07:31
*** openstackgerrit has quit IRC07:47
*** jpena|off is now known as jpena07:51
*** slaweq has joined #openstack-neutron07:59
*** slaweq has quit IRC08:04
*** jlibosva has joined #openstack-neutron08:05
*** rcernin_ has joined #openstack-neutron08:06
*** slaweq has joined #openstack-neutron08:10
*** tesseract has joined #openstack-neutron08:17
*** EisNerd has quit IRC08:20
*** EisNerd has joined #openstack-neutron08:21
*** openstackgerrit has joined #openstack-neutron08:22
openstackgerritliuyulong proposed openstack/neutron-specs master: Add spec for elastic snat  https://review.opendev.org/c/openstack/neutron-specs/+/77054008:22
openstackgerritnorman shen proposed openstack/neutron master: Do not report ovs agent state if ovs is dead  https://review.opendev.org/c/openstack/neutron/+/77005808:22
*** rpittau|afk is now known as rpittau08:25
*** rcernin_ has quit IRC08:26
openstackgerritliuyulong proposed openstack/neutron-specs master: Add spec for elastic snat  https://review.opendev.org/c/openstack/neutron-specs/+/77054008:45
openstackgerritRodolfo Alonso proposed openstack/neutron stable/victoria: Randomize port name in "BaseOVSTestCase"  https://review.opendev.org/c/openstack/neutron/+/77054308:47
openstackgerritRodolfo Alonso proposed openstack/neutron stable/ussuri: Randomize port name in "BaseOVSTestCase"  https://review.opendev.org/c/openstack/neutron/+/77054408:48
openstackgerritRodolfo Alonso proposed openstack/neutron stable/victoria: Add WaitForPortCreateEvent in BaseOVSTestCase  https://review.opendev.org/c/openstack/neutron/+/77054508:50
*** lucasagomes has joined #openstack-neutron09:05
openstackgerritRodolfo Alonso proposed openstack/neutron stable/ussuri: Add WaitForPortCreateEvent in BaseOVSTestCase  https://review.opendev.org/c/openstack/neutron/+/77054709:05
*** bengates has joined #openstack-neutron09:05
openstackgerritRodolfo Alonso proposed openstack/neutron stable/train: Randomize port name in "BaseOVSTestCase"  https://review.opendev.org/c/openstack/neutron/+/77054809:07
openstackgerritRodolfo Alonso proposed openstack/neutron stable/train: Add WaitForPortCreateEvent in BaseOVSTestCase  https://review.opendev.org/c/openstack/neutron/+/77054909:07
*** bengates has quit IRC09:09
openstackgerritRodolfo Alonso proposed openstack/neutron master: Add port device profile extension  https://review.opendev.org/c/openstack/neutron/+/76792209:09
*** bengates has joined #openstack-neutron09:10
*** dasp_ has quit IRC09:16
*** dasp has joined #openstack-neutron09:17
openstackgerritEduardo Olivares proposed openstack/neutron-tempest-plugin master: Add first VLAN Transparency tests  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/76500209:28
openstackgerritBernard Cafarelli proposed openstack/networking-sfc stable/victoria: [stable] Drop lower-constraints job  https://review.opendev.org/c/openstack/networking-sfc/+/77036109:42
*** hoonetorg has joined #openstack-neutron09:44
lajoskatonabcafarel: Hi, for networking-odl as I see first the master must be fixed and backported to stable branches.... :-(09:53
lajoskatonabcafarel: I have a wip for that, (https://review.opendev.org/c/openstack/networking-odl/+/769877 ), I try to finalize it today09:54
bcafarellajoskatona: I am pretty sure the folks that wrote "the neverending story" were time travellers that met our CI09:55
openstackgerritBernard Cafarelli proposed openstack/networking-sfc stable/ussuri: [stable] Drop lower-constraints job  https://review.opendev.org/c/openstack/networking-sfc/+/77019009:58
lajoskatonabcafarel: and now it is a builtin feature in pip with the new resolver ( after ~14hours of running tox for taas I got this: pip._vendor.resolvelib.resolvers.ResolutionTooDeep: 2000000 )09:58
bcafarelwow09:59
bcafareltime to add some constraints to help the resolver then :/09:59
bcafarellajoskatona: also if older branches look broken, remember we said we would drop older branches to unmaintained for stadium if needed10:00
bcafarelfor networking-sfc for example stein and older I will send mail for that, these branches require additional work (and probably not worth it unless somebody steps in)10:01
lajoskatonabcafarel: yeah, I keep in mind10:04
*** tesseract has quit IRC10:31
*** tesseract has joined #openstack-neutron10:33
openstackgerritMerged openstack/neutron master: ovn: Support live migration to DPDK nodes  https://review.opendev.org/c/openstack/neutron/+/76930610:35
openstackgerritMerged openstack/ovsdbapp master: Update Python3 wallaby unit tests  https://review.opendev.org/c/openstack/ovsdbapp/+/77022510:36
*** congnt96 has joined #openstack-neutron10:37
*** congnt95 has quit IRC10:38
*** congnt96 is now known as congnt9510:38
*** ociuhandu has joined #openstack-neutron10:40
*** dtantsur|afk is now known as dtantsur10:41
openstackgerritwdd proposed openstack/ovsdbapp master: Drop py37 and add py38 test  https://review.opendev.org/c/openstack/ovsdbapp/+/75799110:45
openstackgerritBernard Cafarelli proposed openstack/networking-sfc stable/train: [stable] Drop lower-constraints job  https://review.opendev.org/c/openstack/networking-sfc/+/77036310:46
openstackgerritBernard Cafarelli proposed openstack/networking-sfc stable/ussuri: Monkey patch original current_thread _active  https://review.opendev.org/c/openstack/networking-sfc/+/72607710:54
openstackgerritRodolfo Alonso proposed openstack/neutron master: [OVS] Fix live-migration connection disruption  https://review.opendev.org/c/openstack/neutron/+/76627711:00
*** abdysn has joined #openstack-neutron11:01
*** hemanth_n has quit IRC11:01
openstackgerritBernard Cafarelli proposed openstack/networking-ovn stable/stein: [stable] Drop lower-constraints job  https://review.opendev.org/c/openstack/networking-ovn/+/77042211:02
openstackgerritBernard Cafarelli proposed openstack/networking-ovn stable/rocky: [stable] Drop lower-constraints job  https://review.opendev.org/c/openstack/networking-ovn/+/77042911:03
*** bengates has quit IRC11:04
*** bengates has joined #openstack-neutron11:04
openstackgerritRodolfo Alonso proposed openstack/neutron master: [OVS] Fix live-migration connection disruption  https://review.opendev.org/c/openstack/neutron/+/76627711:13
ralonsohfolks, just a heads-up11:16
ralonsohwe are having some issues with the mirrors in the CI11:16
ralonsohhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_1e5/770356/2/gate/openstack-tox-py36/1e56e46/job-output.txt11:16
ralonsoh2021-01-13 09:50:18.367963 | controller | W: Failed to fetch https://mirror.regionone.limestone.opendev.org/ubuntu/dists/bionic-security/InRelease  Unable to connect to mirror.regionone.limestone.opendev.org:https:11:16
slaweqralonsoh: can You ask in openstack-infra if they are aware of it?11:19
*** bengates has quit IRC11:21
*** bengates has joined #openstack-neutron11:21
ralonsohslaweq, I'm on it11:31
*** bengates has quit IRC11:33
*** bengates has joined #openstack-neutron11:33
*** ociuhandu has quit IRC11:36
*** bengates has quit IRC11:40
*** bengates has joined #openstack-neutron11:41
openstackgerritMerged openstack/networking-bagpipe stable/ussuri: Dropping lower constraints testing (stable Ussuri)  https://review.opendev.org/c/openstack/networking-bagpipe/+/77044811:42
ralonsohbcafarel, sorry...11:54
*** tmorin has quit IRC11:54
ralonsohstack@dev20:/opt/stack$ /opt/stack/tempest/.tox/tempest/bin/pip install -c /opt/stack/requirements/upper-constraints.txt -r requirements.txt^C11:54
ralonsohERROR: Links are not allowed as constraints11:54
ralonsohI know you are done with this...11:54
ralonsohdid you see that before?11:54
ralonsohthis is during the ./stack.sh process11:54
openstackgerritSlawek Kaplonski proposed openstack/neutron master: DVR: Add ARP responder flows for allowed_address_pair IP  https://review.opendev.org/c/openstack/neutron/+/60133611:57
*** sapd1 has quit IRC12:01
lajoskatonaralonsoh: I've seen when I run stack.sh with enable_plugin tempest, for some reason the tempest venv creation fails with this12:14
ralonsohlajoskatona, I've detected that this is a problem in pip 20.312:14
ralonsohwith pip 20.2.1, the requirements can have links12:14
ralonsohlike12:14
lajoskatonaralonsoh: yeah it is realted to it,12:14
lajoskatonawhat I can't understand why it fails as I have other deps as links in my env12:15
ralonsohI think this is because the .tox/tempest env is already created12:15
ralonsohand it keeps the pip installed12:15
*** raildo has joined #openstack-neutron12:16
lajoskatonathen perhaps installing a newer virtualenv package or tox (perhaps that's pull in pip to venv)12:16
*** ramishra has quit IRC12:19
*** bengates has quit IRC12:22
*** ramishra has joined #openstack-neutron12:23
bcafarelralonsoh: so yay new fun with recent pip? :(12:23
ralonsohyeah...12:23
*** armax has joined #openstack-neutron12:24
*** bengates has joined #openstack-neutron12:24
*** ratailor has quit IRC12:26
bcafarelralonsoh: https://bugs.launchpad.net/devstack/+bug/1906322 maybe you miss recent devstack fixes12:28
openstackLaunchpad bug 1906322 in devstack "pip 20.3 break devstack: ERROR: Links are not allowed as constraints" [High,New]12:28
ralonsohyeah, I saw it12:28
bcafarelwell workaround for now it just caps pip12:28
ralonsohbut this should be done in the tempest installation12:29
*** ociuhandu has joined #openstack-neutron12:45
*** spatel has joined #openstack-neutron12:47
*** jpena is now known as jpena|lunch12:49
*** spatel has quit IRC12:52
*** ociuhandu has quit IRC12:55
*** ociuhandu has joined #openstack-neutron12:56
*** ociuhandu has quit IRC12:56
*** ociuhandu has joined #openstack-neutron13:01
openstackgerritSlawek Kaplonski proposed openstack/neutron-lib master: Add network parameter to some of the ML2 API methods  https://review.opendev.org/c/openstack/neutron-lib/+/77061213:06
*** jpena|lunch is now known as jpena13:51
*** nweinber has joined #openstack-neutron13:55
slaweqlajoskatona: hi, can You take a look at https://review.opendev.org/c/openstack/neutron/+/770345 ? thx in advance13:59
*** tmorin has joined #openstack-neutron13:59
lajoskatonaslaweq: sure14:00
slaweqthx14:00
*** nweinber has quit IRC14:01
*** liuyulong has joined #openstack-neutron14:01
*** nweinber has joined #openstack-neutron14:02
*** liuyulong has quit IRC14:03
openstackgerritSlawek Kaplonski proposed openstack/neutron-lib master: Add description for the "tag-ports-during-bulk-creation" extension  https://review.opendev.org/c/openstack/neutron-lib/+/77010814:03
slaweqralonsoh: lajoskatona: please also take a look at https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/748968 if You will have some time :)14:04
ralonsohsure14:05
ralonsohslaweq, I though the "hostname" problem was sorted out14:06
slaweqralonsoh: hmm, so I probably missed it14:07
ralonsohslaweq, https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/76252714:07
slaweqralonsoh: ok, so I will abandon my patch :)14:07
ralonsohnp!14:08
slaweqthx for that14:08
slaweqralonsoh++14:08
slaweqralonsoh: but please check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/768584 instead :)14:08
ralonsohslaweq, sure14:09
openstackgerritLajos Katona proposed openstack/networking-odl master: WIP: test master of networking-odl  https://review.opendev.org/c/openstack/networking-odl/+/76987714:09
slaweqthx14:09
slaweqralonsoh: thx for +2 in that patch14:12
slaweqralonsoh: but why You didn't approve it?14:12
ralonsohslaweq, waiting for the depends on14:13
slaweqahh14:13
ralonsohanyway14:13
slaweqok14:13
slaweq:)14:13
ralonsohI can do it right now14:13
slaweqsorry for the question14:13
slaweqralonsoh: please also check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/768190 :)14:13
slaweqthis is small one14:13
ralonsohsure14:13
slaweqthx a lot14:14
ralonsohdone14:14
otherwiseguyslaweq: on https://review.opendev.org/c/openstack/neutron/+/752795 the looks good, but add a release note: I added a release note. :D14:14
ralonsohhahahah14:15
ralonsohhe wants a second one!14:15
otherwiseguyralonsoh: lol!14:16
slaweqotherwiseguy: thx :)14:16
slaweqone is enough for me :P14:16
otherwiseguyslaweq: ralonsoh: also, on that fedora periodic issue. it's almost certainly wsgi.14:16
ralonsohyeah, most probably14:16
slaweqotherwiseguy: thx for checking, I saw Your comments in LP today morning14:17
ralonsohwe can change it to non-wsgi for now14:17
ralonsohI think so14:17
otherwiseguystill working on the tempest.14:17
slaweqso I hope lucasagomes will fix it soon :)14:17
slaweqotherwiseguy: and are You sure it's not related to the fact that ovn is installed from packages, not from source?14:17
slaweqor it isn't in that job?14:17
* lucasagomes reads14:18
lucasagomesotherwiseguy, slaweq the uwsgi one trick... yesterday I was able to stack devstack with uwsgi + ovn but, it was very hacky14:19
lucasagomesand otherwiseguy the agent methods such as get_agent etc... that we overwrite, those were giving me an attributeerror, I have to look into that14:20
otherwiseguyslaweq: I ran it with and with the ...wsgi=true setting. worked fine without, failed with.14:20
lucasagomesnot sure the extension hasn't properly enabled etc...14:20
otherwiseguywith and without14:20
lucasagomesyeah OVN doesn't work at all with uwsgi atm :-(14:20
otherwiseguylucasagomes: lol, I was joking when I said no matter what you found i'd have to rewrite agent support.14:20
lucasagomesotherwiseguy, lol IKR, when taht happened I automatically remebered u saying that14:21
slaweq:D14:21
lucasagomesnot sure if it helps or not, but we use the same overwrite mechanism on the AZ methods and those failed the same way too14:21
otherwiseguyif i have to do it again, I'm just going to set up a looping call and write to the sql db and be done with it.14:22
lucasagomeslol14:22
lucasagomesotherwiseguy, my fear is that we may need a substatial refactor on the intialization of our driver to make it work :-(14:23
slaweqotherwiseguy: but getting back to that fedora job, it doesn't look like it is deployed with uwsgi14:23
slaweqso the reason must be different14:24
lucasagomesslaweq, otherwiseguy oh fedora I fixed it on my last patch-set14:24
lucasagomesof the devstack patch14:24
lucasagomes1 sec lemme link it here14:24
slaweqlucasagomes: https://zuul.openstack.org/build/20b15650f5db493dbd343d9806358834/logs14:24
slaweqthose are logs from failing fedora job14:24
slaweqit's periodic job and failing every day since some time14:24
lucasagomesah it's master ok... idk then14:24
otherwiseguylucasagomes: can we sneak sending the process event into the uwsgi starting code?14:25
lucasagomesslaweq, otherwiseguy https://review.opendev.org/c/openstack/devstack/+/735097/34/lib/neutron_plugins/ovn_agent14:25
slaweqlucasagomes: exactly since 31.12 https://zuul.openstack.org/build/ce30a7807f9143189cc432e6f5cab9ee14:25
lucasagomesotherwiseguy, we can try but, there are a few things happening there... we need to call post fork at least twice, once with the maintenance work and another one with the neutron api worker14:25
*** jmlowe has joined #openstack-neutron14:26
lucasagomesbut that wouldn't do it because they would overwrite the attributes holding the connection (_nb_idl, sb_idl)14:26
lucasagomesso I think as a first step, we can refactor the maintenance thread to start it's own connection within that class14:27
otherwiseguyslaweq: hmm. because the processes are definitely started and look configured to listen on those ports...I guess back to the drawing board.14:27
openstackgerritLajos Katona proposed openstack/neutron-specs master: BFD support for Neutron  https://review.opendev.org/c/openstack/neutron-specs/+/76733714:27
lucasagomesanother problem is that, before we only started 1 maintenance task per host (1 main worker and X api workers)14:27
lucasagomeswith this, we probably going to start 1 maintenance per wsgi worker14:28
lucasagomeswhich is not ideal but we can make it work14:28
lucasagomesthere are a bunch of things like that, that we need to take care14:28
*** tmorin has quit IRC14:28
otherwiseguylucasagomes: ugh.14:28
lucasagomesthere's code in post_fork waiting for the pg_drop port group to be created as well14:28
lucasagomesyeah it's tricker than I first thought14:28
lucasagomesotherwiseguy, and the main one is, not knowing whether it's uwsgi or eventlet... there's no neutron.conf or anything14:29
lucasagomesmaybe we can pass a flag in the initialize() method via pecan_wsgi class or something14:29
lucasagomesbut at the mech driver level, I don't see any good way to figure it out atm14:30
otherwiseguylucasagomes: process name is different :D14:35
otherwiseguy[vagrant@localhost devstack]$ cat /proc/105103/cmdline14:35
otherwiseguyneutron-apiuWSGI master14:35
lucasagomesotherwiseguy, hah yeah maybe we can really rely on that :D14:36
otherwiseguyhackity hack :D14:36
lucasagomeslol14:36
bcafarelslaweq: droppping l-c in stable is good occasion to finally check which branches we should move to unmaintained, I think it is better to send separate mails per project, wdyt?14:37
otherwiseguylucasagomes: it looks like there is an /etc/neutron/neutron-api-uwsgi.ini with a uwsgi section.14:37
bcafarelslaweq: netowrking-sfc is good up to train (and patches pass zuul), but stein is a mess, so it looks like first good candidate14:38
otherwiseguylucasagomes: (in devstack, but I'm assuming we could look at something in cfg.CONF.uwsgi)14:38
lucasagomesotherwiseguy, hmm that would be good indeed14:42
lucasagomeslemme check on that14:42
otherwiseguylucasagomes: and since neutron-api starts with the wsgi script (looks PBR generated? in devstack /usr/local/bin/neutron-api and I think we start with neutron/cmd/eventlet/server/__init__.py, seems like we could shove a value in one of those to check?14:45
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Switch tempest jobs to neutron specific ones  https://review.opendev.org/c/openstack/neutron/+/77063014:47
slaweqralonsoh: lajoskatona: if You would have time, ^^ is small improvement (I hope) to our ci jobs :)14:47
ralonsohsure14:47
*** armax has quit IRC14:48
*** armax has joined #openstack-neutron14:50
otherwiseguyralonsoh: slaweq: on https://review.opendev.org/c/openstack/neutron/+/752795 looks like you both +2d at the same time. One of you want to +w? :D14:51
slaweqotherwiseguy: done :)14:52
otherwiseguyslaweq++ ;)14:52
lucasagomesotherwiseguy, yeah that would be a way, kinda the way I am leaning to to be honest14:52
lucasagomesotherwiseguy, also I don't know how far we want to go with the refactor, I was wondering if it would be possible to have a proxy connection started at inialize() and have the workers to reuse it14:53
lucasagomesI mean eventlet workers ^14:53
lucasagomesso when it's wsgi we also start the connection at initialize() the same way14:54
lucasagomesidk :-/14:54
otherwiseguylucasagomes: it would be a little complex. I *think* we'd have to basically use Command objects as the proxy interface, so anything that didn't use them would need to be converted to do so (direct reads wouldn't work) and then things where we actually use a Command object to pull back a large list of objs and assume we can iterate over them quickly locally might need to be converted into command objects to have the14:57
otherwiseguyserver do it quickly?14:57
*** ociuhandu_ has joined #openstack-neutron14:58
otherwiseguylucasagomes: If there was a good way to do it, a shared memory version of the in-memory tables dict would be really nice. (there isn't, really, last time I looked but hints that it could be done).14:58
lucasagomes:-(15:00
otherwiseguymy fears about sending large lists of results multiple times being a performance issue might not end up being one though.15:00
otherwiseguyand it should be fairly easy to find the places we still directly access things w/o using commands.15:01
otherwiseguylucasagomes: git grep "\.tables\["15:02
*** ociuhandu has quit IRC15:02
otherwiseguyalmost entirely in testing code.15:02
lucasagomesoh yeah everytwhere :-/15:07
lucasagomeseverywhere* we need to get rid of those one day15:08
*** abdysn has quit IRC15:08
*** macz_ has joined #openstack-neutron15:13
slaweqlajoskatona: ralonsoh: please also take a look at https://review.opendev.org/c/openstack/neutron/+/686343 if You will have some time, it should close one of our old BPs finally15:14
slaweq(not very important but still)15:14
ralonsohslaweq, let me check15:14
slaweqtjx15:14
slaweq*thx15:14
ralonsohbtw, https://review.opendev.org/c/openstack/neutron/+/77063015:14
ralonsohgood one, for sure15:14
slaweqthx15:15
slaweqI hope it will improve a bit stability of those jobs15:15
slaweqand make them a bit faster too15:15
ralonsohslaweq, https://review.opendev.org/c/openstack/neutron/+/68634315:17
ralonsohI had some comments on this one15:17
ralonsohthe code is ok15:17
ralonsohbut, IMO, we can refactor this a bit15:17
ralonsohIf you don't have time, I can push a patch15:17
*** macz_ has quit IRC15:17
slaweqralonsoh: thx, I will refactor it15:18
ralonsohperfect15:18
openstackgerritSlawek Kaplonski proposed openstack/neutron stable/ussuri: Disable dns-integration API extension if it's not enabled in ML2  https://review.opendev.org/c/openstack/neutron/+/76782215:30
openstackgerritSlawek Kaplonski proposed openstack/neutron stable/victoria: Disable dns-integration API extension if it's not enabled in ML2  https://review.opendev.org/c/openstack/neutron/+/76777515:30
*** ociuhandu_ has quit IRC15:35
*** ociuhandu has joined #openstack-neutron15:35
openstackgerritTerry Wilson proposed openstack/neutron master: Use agent chassis table based on schema  https://review.opendev.org/c/openstack/neutron/+/76096715:39
ralonsohotherwiseguy, ^^ good one15:53
ralonsohI had some problems too15:53
ralonsohand I manually changed the table name15:54
ralonsoh(in the code, of course)15:54
otherwiseguyralonsoh: yeah. that one is jlibosva's patch though. we both had a bunch of patches related to that area of the code all at the same time. it was a mess of dependencies that just now, after several months, is making its way through the gate. :)15:57
jlibosvao/15:58
*** sapd1 has joined #openstack-neutron15:59
jlibosvaoh, I thought this got merged already :( otherwiseguy thanks for rebasing :)16:00
*** macz_ has joined #openstack-neutron16:02
otherwiseguyjlibosva: i'm just excited that things are actually starting to merge. The number of rechecks/rebases on essentially a two-line patch (https://review.opendev.org/c/openstack/neutron/+/753847/28/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovsdb_monitor.py) was insane. I think almost 293 "comments".16:03
otherwiseguys/almost //16:03
jlibosvaotherwiseguy: https://imgflip.com/i/4toaj516:04
otherwiseguyjlibosva: indeed. :D16:05
otherwiseguythe last week has been sooo much better. (thanks everybody!)16:05
*** rnoriega_ is now known as rnoriega12316:06
*** rnoriega123 is now known as rnoriega_16:06
slaweqotherwiseguy: we still have some issues there but IMO it's better than it was before Christmas16:06
otherwiseguynow to shrink the zuul queue so that patches can start running w/o a 6 hour wait :D16:07
otherwiseguyI blame ralonsoh : https://review.opendev.org/q/owner:ralonsoh%2540redhat.com16:08
otherwiseguyHe's clearly working too hard.16:08
*** ccamposr__ has joined #openstack-neutron16:09
*** rnoriega_ is now known as rnoriega16:09
slaweqotherwiseguy: but don't tell it to him :P16:09
slaweqI don't want him to stop ;)16:09
ralonsohnonono, I'm just push random code16:09
otherwiseguyslaweq: Can we get him his on CI machines?16:09
ralonsohpushing*16:09
otherwiseguyown16:09
slaweqI wish that everyone would have own CI machines ;)16:10
slaweqbut how to convince infra folks for that idea? :D16:10
*** ociuhandu_ has joined #openstack-neutron16:11
*** ociuhandu has quit IRC16:11
*** ccamposr has quit IRC16:12
otherwiseguyslaweq: liquor?16:14
slaweqotherwiseguy: it may work :)16:15
slaweqok, I'm done for today, see You guys tomorrow16:15
otherwiseguyslaweq: have a nice night! Thanks!16:15
openstackgerritLajos Katona proposed openstack/networking-odl master: WIP: test master of networking-odl  https://review.opendev.org/c/openstack/networking-odl/+/76987716:17
openstackgerritMerged openstack/neutron-tempest-plugin master: Move fwaas jobs to the experimental queue  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/76819016:20
*** ccamposr has joined #openstack-neutron16:27
*** ccamposr__ has quit IRC16:29
openstackgerritLajos Katona proposed openstack/networking-bgpvpn master: Update lower-constraints  https://review.opendev.org/c/openstack/networking-bgpvpn/+/76965716:30
*** slaweq has quit IRC16:34
*** slaweq has joined #openstack-neutron16:36
*** mchlumsky8 has joined #openstack-neutron16:41
*** adrianc has quit IRC16:41
*** adrianc has joined #openstack-neutron16:42
*** mchlumsky has quit IRC16:42
*** mchlumsky8 is now known as mchlumsky16:42
openstackgerritRodolfo Alonso proposed openstack/neutron master: [fullstack] Be able to delete port when fake VM is destroyed  https://review.opendev.org/c/openstack/neutron/+/77045816:42
*** tmorin has joined #openstack-neutron16:43
*** tmorin has quit IRC16:44
*** jlibosva has quit IRC16:49
*** lucasagomes has quit IRC16:53
*** tesseract has quit IRC16:54
*** jpena is now known as jpena|off17:05
*** ociuhandu_ has quit IRC17:06
*** ociuhandu has joined #openstack-neutron17:09
*** ociuhandu_ has joined #openstack-neutron17:13
*** ociuhandu has quit IRC17:16
*** ociuhandu_ has quit IRC17:17
*** ociuhandu has joined #openstack-neutron17:22
openstackgerritMerged openstack/neutron master: Add support for deleting ml2/ovn agents  https://review.opendev.org/c/openstack/neutron/+/75279517:22
*** ociuhandu has quit IRC17:27
*** bengates has quit IRC17:34
*** baojg has quit IRC17:41
*** baojg has joined #openstack-neutron17:41
*** mlavalle has joined #openstack-neutron17:57
*** hamalq has joined #openstack-neutron18:03
openstackgerritRodolfo Alonso proposed openstack/neutron master: In OVO PortForwarding, retrieve FIP info from db_obj  https://review.opendev.org/c/openstack/neutron/+/77065418:04
*** rpittau is now known as rpittau|afk18:05
*** baojg has quit IRC18:11
*** baojg has joined #openstack-neutron18:11
*** bengates has joined #openstack-neutron18:16
*** baojg has quit IRC18:18
*** bengates has quit IRC18:19
*** baojg has joined #openstack-neutron18:19
*** ralonsoh has quit IRC18:20
*** bengates has joined #openstack-neutron18:20
openstackgerritMerged openstack/ovsdbapp master: Add northbound API support for meters  https://review.opendev.org/c/openstack/ovsdbapp/+/76757418:22
*** bengates has quit IRC18:33
*** dtantsur is now known as dtantsur|afk18:43
*** lajoskatona has quit IRC18:48
*** gcheresh has quit IRC18:56
openstackgerritFlavio Fernandes proposed openstack/neutron stable/train: Don't fail if FIP is not in port forwarding cache during cleaning  https://review.opendev.org/c/openstack/neutron/+/76983319:25
*** hamalq has quit IRC19:29
*** gcheresh has joined #openstack-neutron19:37
*** baojg has quit IRC19:39
*** baojg has joined #openstack-neutron19:39
openstackgerritMerged openstack/neutron-lib master: Add new VNIC types for Cyborg provisioned ports  https://review.opendev.org/c/openstack/neutron-lib/+/76832419:51
*** hoonetorg has quit IRC20:04
*** tmorin has joined #openstack-neutron20:06
openstackgerritMerged openstack/neutron stable/queens: Fix calling of add_tunnel_port method from sanity checks module  https://review.opendev.org/c/openstack/neutron/+/76528820:09
*** ccamposr__ has joined #openstack-neutron20:11
*** ccamposr has quit IRC20:14
*** slaweq has quit IRC20:41
*** tmorin has quit IRC20:42
*** nweinber has quit IRC21:09
*** hoonetorg has joined #openstack-neutron21:18
*** headphoneJames has joined #openstack-neutron21:22
*** elvira has quit IRC21:38
*** hoonetorg has quit IRC21:43
*** baojg has quit IRC21:43
*** baojg has joined #openstack-neutron21:44
*** AlexStaf has quit IRC21:50
*** gcheresh has quit IRC21:57
*** rcernin has joined #openstack-neutron21:59
*** AlexStaf has joined #openstack-neutron22:01
*** baojg has quit IRC22:52
*** baojg has joined #openstack-neutron22:53
openstackgerritFlavio Fernandes proposed openstack/ovsdbapp master: [ovn] AclAddHelper command: use [] instead None for string columns  https://review.opendev.org/c/openstack/ovsdbapp/+/77067623:06
openstackgerritFlavio Fernandes proposed openstack/ovsdbapp master: [ovn] AclAddHelper command: use [] instead None for string columns  https://review.opendev.org/c/openstack/ovsdbapp/+/77067623:17
*** yamamoto has joined #openstack-neutron23:32
*** baojg has quit IRC23:33
*** baojg has joined #openstack-neutron23:34
« Tuesday, 2021-01-12 Index Thursday, 2021-01-14 »

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