Tuesday, 2020-05-26

openstackgerritMerged openstack/neutron master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72975000:01
*** rcernin has quit IRC00:23
*** rcernin has joined #openstack-neutron00:23
*** iurygregory has quit IRC00:49
*** cp- has quit IRC00:56
*** cp- has joined #openstack-neutron00:57
*** rcernin has quit IRC01:04
*** rcernin has joined #openstack-neutron01:05
*** rcernin has quit IRC01:05
*** cp- has quit IRC01:08
*** sapd1 has joined #openstack-neutron02:05
openstackgerrityangkai proposed openstack/neutron-lib master: Update agent's attrs can be sorted  https://review.opendev.org/72649702:21
*** hongbin has joined #openstack-neutron03:08
*** rcernin has joined #openstack-neutron03:19
*** ramishra has joined #openstack-neutron03:23
*** hongbin has quit IRC03:28
openstackgerrityangkai proposed openstack/neutron-lib master: Update agent's attrs can be sorted  https://review.opendev.org/72649703:50
*** xiaolin has quit IRC03:55
*** lajoskatona has joined #openstack-neutron03:59
*** ramishra has quit IRC04:13
*** rcernin has quit IRC04:20
*** rcernin has joined #openstack-neutron04:22
*** ratailor has joined #openstack-neutron04:28
*** ratailor has quit IRC04:30
*** rcernin has quit IRC04:38
*** threestrands has joined #openstack-neutron04:40
*** rcernin has joined #openstack-neutron04:43
*** jamesdenton has quit IRC04:57
*** jamesden_ has joined #openstack-neutron04:57
*** ratailor has joined #openstack-neutron05:02
*** cp- has joined #openstack-neutron05:13
*** cp- has quit IRC05:14
*** cp- has joined #openstack-neutron05:15
*** cp- has quit IRC05:16
*** gcheresh has joined #openstack-neutron05:16
*** cp- has joined #openstack-neutron05:17
*** ramishra has joined #openstack-neutron05:24
fricklersean-k-mooney: just for reference this is the gerritbot config https://opendev.org/openstack/project-config/src/branch/master/gerritbot/channels.yaml#L104105:49
*** hjensas__ has quit IRC06:14
*** mvkr has quit IRC06:21
*** ramishra has quit IRC06:25
openstackgerrityangkai proposed openstack/neutron master: Support pagination for agent list  https://review.opendev.org/72672406:28
*** ramishra has joined #openstack-neutron06:28
*** mvkr has joined #openstack-neutron06:33
*** maciejjozefczyk has joined #openstack-neutron06:48
*** slaweq has joined #openstack-neutron06:52
*** tesseract has joined #openstack-neutron07:05
*** hjensas has joined #openstack-neutron07:12
*** S4ren has joined #openstack-neutron07:14
openstackgerritinspurericzhang proposed openstack/networking-bgpvpn master: Remove the unused coding style modules  https://review.opendev.org/73071907:19
*** mvkr has quit IRC07:19
*** ratailor has quit IRC07:20
*** ratailor has joined #openstack-neutron07:20
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Remove unnecessary check if agent object from db is not None  https://review.opendev.org/71343207:21
*** iurygregory has joined #openstack-neutron07:22
*** ralonsoh has joined #openstack-neutron07:24
*** amoralej|off is now known as amoralej07:27
*** jpena|off is now known as jpena07:27
slaweqlajoskatona: hi07:28
slaweqlajoskatona: can You take a look at https://review.opendev.org/#/c/729600/2 ?07:28
slaweqthere is Your comment related to the changed code so I would like You to take a look into this07:28
*** ociuhandu has quit IRC07:29
*** ociuhandu has joined #openstack-neutron07:29
*** mvkr has joined #openstack-neutron07:32
*** rcernin has quit IRC07:36
*** nirmoy has joined #openstack-neutron07:38
*** openstackstatus has quit IRC07:39
*** openstackstatus has joined #openstack-neutron07:40
*** ChanServ sets mode: +v openstackstatus07:40
*** ociuhandu has quit IRC07:42
*** ociuhandu has joined #openstack-neutron07:43
*** rpittau|afk is now known as rpittau07:46
*** ociuhandu has quit IRC07:48
lajoskatonaslaweq: Hi, I check it07:55
slaweqlajoskatona: thx07:55
*** lucasagomes has joined #openstack-neutron07:57
lajoskatonaslaweq: done, thanks for the notification08:07
bcafarelgmann: lajoskatona: nice to see these stein fixes for bgpvpn/bagpipe thanks! W+108:08
lajoskatonabcafarel: and elod :-)08:10
bcafarelright elod++ too :)08:10
*** nirmoy has quit IRC08:12
*** nirmoy has joined #openstack-neutron08:13
*** jlibosva has joined #openstack-neutron08:17
*** ramishra has quit IRC08:20
*** ramishra has joined #openstack-neutron08:26
*** xiaolin has joined #openstack-neutron08:33
*** rcernin has joined #openstack-neutron08:39
*** rcernin has quit IRC08:40
*** nirmoy has quit IRC08:53
*** nirmoy has joined #openstack-neutron08:54
*** priteau has joined #openstack-neutron08:54
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Set OVS manager command timeout and inactiviy probe  https://review.opendev.org/72071108:55
*** nirmoy_ has joined #openstack-neutron08:58
*** nirmoy_ has quit IRC09:00
rm_workwow, neutron unit tests require sudo privs? O_o09:01
*** nirmoy has quit IRC09:01
*** nirmoy has joined #openstack-neutron09:01
rm_workbecause of https://github.com/openstack/neutron/blob/master/neutron/agent/linux/iptables_firewall.py#L103-L10509:01
rm_workcalled from https://github.com/openstack/neutron/blob/master/neutron/tests/unit/agent/linux/openvswitch_firewall/test_iptables.py#L7309:02
rm_workthat seems... not unit-test-ey09:02
rm_workIMO a unit test should not run `sysctl` on my system as root <_<09:03
*** dtantsur|afk is now known as dtantsur09:05
*** tkajinam has quit IRC09:05
openstackgerritAdam Harwell proposed openstack/neutron-lib master: WIP: Add `access_as_readonly` sharing action  https://review.opendev.org/73073609:09
openstackgerritAdam Harwell proposed openstack/neutron master: WIP: Add `access_as_readonly` sharing action  https://review.opendev.org/73073709:09
openstackgerritAdam Harwell proposed openstack/neutron master: WIP: Add `access_as_readonly` sharing action  https://review.opendev.org/73073709:10
openstackgerritzhanghao proposed openstack/networking-bgpvpn master: Remove usage of six  https://review.opendev.org/73073809:10
*** ccamposr has joined #openstack-neutron09:13
*** ccamposr__ has quit IRC09:16
*** iurygregory has quit IRC09:17
*** iurygregory has joined #openstack-neutron09:18
rm_workuhh, do neutron's gates not work with Depends-On for neutron-lib changes?09:19
rm_workhttps://review.opendev.org/#/c/730737/ -> https://02c2159d30aff3e28da4-bbce49889c764923168fb20665b2170c.ssl.cf2.rackcdn.com/730737/2/check/openstack-tox-py36/e40ce04/job-output.txt09:19
rm_work2020-05-26 09:15:58.698431 | ubuntu-bionic | AttributeError: module 'neutron_lib.constants' has no attribute 'ACCESS_SHARED'09:19
rm_workor am I missing something I need to do in the commit to get it to pull the newer neutron-lib?09:20
*** ociuhandu has joined #openstack-neutron09:25
openstackgerritMerged openstack/networking-bgpvpn stable/stein: Use tox 'all' env instead of 'all-plugins' for networking bgpvpn tempest plugin  https://review.opendev.org/73037909:29
openstackgerritMerged openstack/neutron-tempest-plugin master: Use default network type during PortTestCasesResourceRequest setup  https://review.opendev.org/72960009:33
*** tesseract has quit IRC09:34
openstackgerritNoy fridman proposed openstack/neutron-tempest-plugin master: Add "test_port_forwardings_negative" module  https://review.opendev.org/73074109:35
ralonsohrm_work, do you have a log for the firewall problem?09:37
*** tesseract has joined #openstack-neutron09:39
openstackgerritzhanghao proposed openstack/networking-bgpvpn master: Remove usage of six  https://review.opendev.org/73073809:41
openstackgerritFlavio Fernandes proposed openstack/neutron master: DNM [ovn]: port forwarding functionality (1 of 2)  https://review.opendev.org/72386309:41
slaweqralonsoh: lajoskatona: hi, can You maybe take a look at https://review.opendev.org/#/c/721554/ and https://review.opendev.org/#/c/726770/5 ?09:44
slaweqthx in advance09:44
ralonsohslaweq, give me 20 mins09:44
slaweqralonsoh: thx a lot09:45
*** nirmoy has quit IRC09:55
*** nirmoy has joined #openstack-neutron09:55
*** tmorin has joined #openstack-neutron10:02
*** threestrands has quit IRC10:05
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Use pyroute2 for SRIOV VF commands  https://review.opendev.org/72691810:06
fricklerrm_work: in my experience neutron-lib needs a release before it can be tested on neutron10:09
slaweqrm_work: sorry for late reply10:10
rm_workhmm ok10:10
slaweqrm_work: according to the tests issue, it's probably bug and should be fixed, UT shouldn't require any sudo10:10
rm_workI can get a traceback log sure, sec10:10
slaweqrm_work: and according to neutron-lib - frickler is correct here, You need new neutron-lib release to use it in neutron patch10:10
rm_workhmm, so ... it's very hard to actually test them together then?10:12
rm_workthat'll be difficult :(10:13
fricklerslaweq: though I'm not sure whether this is the intended behaviour on purpose or whether just nobody figured out how to make the dependencies work. because it certainly makes it difficult to test new features10:13
rm_workyeah, in octavia we made sure octavia-lib dependencies will work10:13
rm_workthough there is one case where they are problematic10:13
rm_work... or is it that we have at least one specific job where they work ... crap, I forgot. I'll have to check10:13
rm_workhttp://paste.openstack.org/show/793980/10:14
rm_work^^ that's the firewall test requiring sudo10:14
ralonsohok, I'll open a bug for this one10:15
rm_workI wonder if possibly it's only being *triggered* because I'm on OSX?10:15
rm_workbut it's clear when I follow it that it would be doing a sudo there if that driver was ever loaded10:15
slaweqrm_work: frickler: last time when we discussed that, it was in Denver IIRC we agreed that this is "intended" behaviour - but maybe we should change it at some point10:16
slaweqidk10:16
rm_workthe fact that I don't even have sysctl is also worth mentioning, as that's also not unit-test-ey, whether it requires sudo or not10:16
rm_workyeah it may be an opinion thing -- IMO a "unit test" should not rely on any system binary at all10:17
rm_workother than python10:17
slaweqrm_work: that's for sure, please open bug and we can check that - it should be fixed10:17
rm_workbut I am also in a very very small minority of people trying to keep their testing running on native OSX :/10:17
rm_workI'd cave in and just run it all in docker or a VM, but my RAM is very small for what I'm running so I am already at like 2x RAM in swap >_<10:18
ralonsohrm_work, indeed it's calling sysctl10:18
ralonsohI'll open a bug for this now10:18
rm_workthanks ralonsoh10:18
slaweqrm_work: thx for reporting this10:19
*** rpittau is now known as rpittau|bbl10:20
rm_worknp, it's a very selfish report anyway ;)10:20
rm_workbut with no neutron-lib dependency support in the gate, this means I have to test everything locally I guess, ugh10:21
rm_workmaybe I'll see what I can do about making it pull the dep ;)10:21
slaweqrm_work: if you "just" need api extension to be in neutron-lib, You can first introduce it in neutron and later move to neutron-lib10:22
slaweqrm_work: or if You propose patch, we can do new release as soon as we will merge this patch10:22
slaweqto unblock Your further work :)10:22
rm_workwell part of the issue is i need it in a few places and there's no one place i can put those constants to have them not be a circular import10:22
rm_workbut yeah i can just copy/paste them around for now10:22
rm_workand fix later10:22
rm_workthat's probably easiest10:23
openstackgerritAdam Harwell proposed openstack/neutron-lib master: WIP: Add `access_as_readonly` sharing action  https://review.opendev.org/73073610:28
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Mock command execution in "test_iptables.TestHelper" UTs  https://review.opendev.org/73075210:29
lajoskatonaslaweq: I checked https://review.opendev.org/721554 as far as my limited DVR knowledge let me check10:31
*** gcheresh has quit IRC10:32
*** priteau has quit IRC10:35
*** ramishra has quit IRC10:39
openstackgerritAdam Harwell proposed openstack/neutron master: WIP: Add `access_as_readonly` sharing action  https://review.opendev.org/73073710:41
rm_workwill try that mock patch and see if I can get a full unit test run now :) thanks10:41
rm_workoh rofl i forgot I have core in neutron-lib, thought gerrit was broken for a sec, showing me +/-2 options :P10:45
sean-k-mooneyhehe i have dont the same when i taught i was looking at a patch against nova and realised it was against os-vif and i misclicked when i was going through the patch chain10:52
*** gcheresh has joined #openstack-neutron10:56
*** also_stingrayza has joined #openstack-neutron10:56
openstackgerritSlawek Kaplonski proposed openstack/neutron master: [DVR] Reconfigure re-created physical bridges for dvr routers  https://review.opendev.org/72155410:59
*** stingrayza has quit IRC10:59
ralonsohslaweq, +1 only for the question11:02
openstackgerritSlawek Kaplonski proposed openstack/neutron master: [DVR] Recofigure re-created physical bridges for dvr routers  https://review.opendev.org/72155411:02
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Don't check if any bridges were recrected when OVS was restarted  https://review.opendev.org/72677011:02
*** ramishra has joined #openstack-neutron11:10
openstackgerritSlawek Kaplonski proposed openstack/neutron master: [DVR] Reconfigure re-created physical bridges for dvr routers  https://review.opendev.org/72155411:16
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Don't check if any bridges were recrected when OVS was restarted  https://review.opendev.org/72677011:16
*** ociuhandu has quit IRC11:18
*** ociuhandu has joined #openstack-neutron11:18
*** sapd1 has quit IRC11:23
*** ociuhandu has quit IRC11:23
slaweqralonsoh: I replied You11:28
rm_workralonsoh: another one :/ http://paste.openstack.org/show/793984/11:29
rm_workin neutron/tests/unit/plugins/ml2/drivers/openvswitch/agent/test_ovs_neutron_agent.py11:30
*** jpena is now known as jpena|lunch11:30
rm_worksame eventual `_check_netfilter_for_bridges` call11:30
*** ociuhandu has joined #openstack-neutron11:56
*** nirmoy has quit IRC11:59
*** lajoskatona has quit IRC12:03
*** ociuhandu has quit IRC12:03
openstackgerritSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: Ping external resource in the neutron tests  https://review.opendev.org/73076612:06
*** nirmoy has joined #openstack-neutron12:13
rm_workslaweq: so, I intended to do this TDD style, but... the thing that's stumping me right now is how to add a test for doing a shared SG (or network, or anything). I can't find anything in the unit framework, or the functional framework that seems like an example of a test for even the current sharing actions? or maybe I just don't understand how the current tests work...12:16
rm_workif you have a moment, just pointing out where I can put such a test would be very helpful. I was hoping there was an existing test that did this and I could just copy/paste it and swap around some variables, but no such luck I guess? T_T12:17
rm_workunless the only option is in Tempest12:17
*** lajoskatona has joined #openstack-neutron12:18
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Enable bridge debugging services on the tempest and grenade jobs  https://review.opendev.org/73077212:19
slaweqrm_work: looking, give me a sec12:21
rm_workoh no rush12:22
rm_worki am about to sleep for a bit and then start my day again in like 8 hours :D just wanted to catch you while you are here12:22
*** rpittau|bbl is now known as rpittau12:22
slaweqrm_work: did You check in https://github.com/openstack/neutron/blob/master/neutron/tests/unit/db/test_rbac_db_mixin.py ?12:22
rm_workyes12:23
rm_workat least, I think so...12:23
rm_worklet me review that, so many of these files have similar names12:23
slaweqsure12:23
rm_workah yeah, so that does have networks, I will look into that, though nothing for the other object types12:24
slaweqralonsoh: njohnston lajoskatona: if You will have few minutes, please check https://review.opendev.org/#/c/729567/ - it should make our jobs definitions a bit better organized IMO :)12:24
rm_workother than a small piece I already did edit12:24
slaweqrm_work: at the bottom there are also tests for QoS policies there12:24
rm_workyeah i added SG to that but it isn't REALLY testing anything12:25
rm_workhttps://review.opendev.org/#/c/730737/3/neutron/tests/unit/db/test_rbac_db_mixin.py@30312:25
rm_workbut i'll try to add a test for Network with readonly and see if it helps me12:26
slaweqrm_work: You can also look at dougwig's patch https://review.opendev.org/#/c/635311/22 which introduced rbac for SG12:26
rm_workyeah, I have been perusing that12:26
slaweq:)12:26
rm_workthe bits he added are... confusing12:26
lajoskatonaslaweq: I check it12:27
rm_worklike in neutron/tests/unit/objects/test_rbac.py12:27
rm_workor neutron/tests/unit/objects/test_securitygroup.py12:27
*** ociuhandu has joined #openstack-neutron12:28
*** amoralej is now known as amoralej|lunch12:28
rm_workreading them I have no idea how they function at all, but I will try again12:28
openstackgerritLIU Yulong proposed openstack/neutron master: Speedup: do not use root_helper_daemon by default  https://review.opendev.org/65337812:28
*** ociuhandu has quit IRC12:29
*** nirmoy has quit IRC12:29
openstackgerritSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: Ping external resource in the neutron tests  https://review.opendev.org/73076612:30
ralonsohslaweq, let me check12:30
rm_worki was hoping to make a test that like... sets up a SG, makes the RBAC policy for access_as_readonly, then tries to GET it as another tenant (positive), and update or delete it (negative)12:30
rm_workit seems like this should be possible in units, since it's all just DB+API stuff, but I didn't see anything similar12:30
rm_workthese all just do like "create policy and verify the create returned successfully"12:31
slaweqrm_work: I think that those are just "lower level: tests which are testing if OVO works fine,12:32
slaweqrm_work: we also have some api tests in https://review.opendev.org/#/c/635312/17/neutron_tempest_plugin/api/test_security_groups.py to cover that12:32
slaweqso maybe we simply don't have UT which You are looking for12:32
rm_workyeah, so maybe tempest is the only option unless I figure out how to do it all from scratch12:33
rm_workthough I wonder if tempest plugin will correctly pull in neutron-lib dependency? I hope so? otherwise it will literally be impossible to test, as I am making a change to the core DB filter function in neutron-lib >_>12:33
rm_workI will try and see12:33
*** jpena|lunch is now known as jpena12:34
openstackgerritSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: Enable bridge debugging services on the ci jobs  https://review.opendev.org/73077512:35
*** nweinber has joined #openstack-neutron12:39
ralonsohslaweq, https://review.opendev.org/#/c/72956712:39
ralonsohvery good patch!12:39
ralonsohlet's clean the zuul definition12:39
*** jawad_axd has quit IRC12:39
*** jawad_axd has joined #openstack-neutron12:40
*** nirmoy has joined #openstack-neutron12:42
*** jawad_axd has quit IRC12:45
*** jawad_ax_ has joined #openstack-neutron12:45
*** dave-mccowan has joined #openstack-neutron12:46
openstackgerritAdam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action  https://review.opendev.org/73078412:53
*** ysastri has joined #openstack-neutron12:53
rm_workwe'll see what happens in that run (if it pulls in neutron-lib correctly, etc)12:54
rm_workoh, best way to check that is to remove the temporary things :D12:54
openstackgerritAdam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action  https://review.opendev.org/73078412:56
rm_workwill see what happened with that when I wake up12:56
ysastrihttps://bugs.launchpad.net/neutron/+bug/1873761 Hello we have this issue continuously in our production environment. It would be nice if someone can look into this issue or point me where I need to start to solve this issue.13:02
openstackLaunchpad bug 1873761 in neutron "Internal IP leak to physical interface from qrouter in DVR mode" [Undecided,New]13:02
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Fix iptables rules comments  https://review.opendev.org/73079313:02
*** raildo has joined #openstack-neutron13:09
*** S4ren has quit IRC13:09
openstackgerritMerged openstack/neutron-dynamic-routing master: Use unittest.mock instead of third party mock  https://review.opendev.org/72096213:13
*** amoralej|lunch is now known as amoralej13:18
*** nirmoy has quit IRC13:18
*** ociuhandu has joined #openstack-neutron13:19
*** nirmoy has joined #openstack-neutron13:19
*** nirmoy has quit IRC13:19
*** nirmoy has joined #openstack-neutron13:19
*** nirmoy has quit IRC13:21
*** nirmoy has joined #openstack-neutron13:21
*** nirmoy has quit IRC13:23
*** nirmoy has joined #openstack-neutron13:23
*** ociuhandu has quit IRC13:24
*** nirmoy has quit IRC13:25
*** nirmoy has joined #openstack-neutron13:25
*** nirmoy has quit IRC13:26
*** nirmoy has joined #openstack-neutron13:26
*** raildo has quit IRC13:30
*** ratailor has quit IRC13:33
*** aedc_ has joined #openstack-neutron13:35
*** tkajinam has joined #openstack-neutron13:37
*** S4ren has joined #openstack-neutron13:37
*** aedc has quit IRC13:38
*** jawad_ax_ has quit IRC13:39
*** JamesBenson has joined #openstack-neutron13:42
*** aedc_ has quit IRC13:44
openstackgerritSlawek Kaplonski proposed openstack/neutron stable/ussuri: Ensure that stale flows are cleaned from phys_bridges  https://review.opendev.org/73082013:45
openstackgerritSlawek Kaplonski proposed openstack/neutron stable/train: Ensure that stale flows are cleaned from phys_bridges  https://review.opendev.org/73082213:46
openstackgerritSlawek Kaplonski proposed openstack/neutron stable/stein: Ensure that stale flows are cleaned from phys_bridges  https://review.opendev.org/73082313:46
openstackgerritSlawek Kaplonski proposed openstack/neutron stable/rocky: Ensure iptables rule was added only once  https://review.opendev.org/73082413:48
*** nirmoy_ has joined #openstack-neutron13:48
*** nirmoy has quit IRC13:48
openstackgerritSlawek Kaplonski proposed openstack/neutron stable/queens: Ensure iptables rule was added only once  https://review.opendev.org/73082513:48
*** nirmoy_ has quit IRC13:52
*** nirmoy has joined #openstack-neutron13:52
*** dtantsur is now known as dtantsur|brb13:53
*** rpittau is now known as rpittau|brb14:10
*** raildo has joined #openstack-neutron14:17
*** amoralej is now known as amoralej|brb14:17
*** aedc has joined #openstack-neutron14:23
*** nirmoy has quit IRC14:32
*** nirmoy has joined #openstack-neutron14:32
openstackgerritzhanghao proposed openstack/neutron-dynamic-routing master: Remove usage of six  https://review.opendev.org/73059014:33
*** nirmoy has quit IRC14:33
*** nirmoy has joined #openstack-neutron14:33
*** tkajinam has quit IRC14:36
*** rpittau|brb is now known as rpittau14:39
*** iurygregory has quit IRC14:45
openstackgerritMerged openstack/networking-ovn stable/queens: Fix ovn-db-sync-util after removing l3 ovsdb connection  https://review.opendev.org/72893714:48
openstackgerritMerged openstack/networking-ovn stable/rocky: Fix ovn-db-sync-util after removing l3 ovsdb connection  https://review.opendev.org/72893614:48
*** dtantsur|brb is now known as dtantsur14:50
*** amoralej|brb is now known as amoralej14:50
*** stingrayza has joined #openstack-neutron14:52
*** dayou has quit IRC14:52
*** iurygregory has joined #openstack-neutron14:53
*** dayou has joined #openstack-neutron14:55
*** also_stingrayza has quit IRC14:55
*** S4ren has quit IRC15:01
*** armax has joined #openstack-neutron15:03
*** mlavalle has joined #openstack-neutron15:09
openstackgerritMerged openstack/neutron-tempest-plugin master: Reorganize zuul jobs definitions  https://review.opendev.org/72956715:13
*** ociuhandu has joined #openstack-neutron15:19
*** ociuhandu has quit IRC15:19
*** ociuhandu has joined #openstack-neutron15:19
openstackgerritLajos Katona proposed openstack/networking-bagpipe master: Remove usage of six library  https://review.opendev.org/73085315:24
openstackgerritBernard Cafarelli proposed openstack/neutron stable/rocky: Dynamically increase l3 router process queue green pool size  https://review.opendev.org/72828715:30
*** xiaolin has quit IRC15:31
*** priteau has joined #openstack-neutron15:34
*** dtantsur is now known as dtantsur|afk15:34
*** gcheresh has quit IRC15:35
*** S4ren has joined #openstack-neutron15:36
*** tmorin has quit IRC15:40
*** rpittau is now known as rpittau|afk15:51
*** tmorin has joined #openstack-neutron15:56
*** jlibosva has quit IRC15:56
*** jawad_axd has joined #openstack-neutron15:59
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Mock command execution in "test_hybrid_plug_flag_based_on_firewall" UT  https://review.opendev.org/73086916:00
*** hamalq has joined #openstack-neutron16:02
*** jawad_axd has quit IRC16:04
*** ociuhandu has quit IRC16:19
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Use pyroute2 for SRIOV VF commands  https://review.opendev.org/72691816:19
*** ociuhandu has joined #openstack-neutron16:20
openstackgerritMerged openstack/neutron master: Remove usage of six.PY2  https://review.opendev.org/73038516:21
openstackgerritMerged openstack/neutron master: Delete segment RPs when network is deleted  https://review.opendev.org/72850716:21
ildikovmlavalle: slaweq: hi16:22
ildikovmlavalle: slaweq: I'm reaching out about the edge sessions at the PTG next week. The edge working group has a few slots reserved including an hour on Wednesday at 1300 UTC - 1400 UTC to sync up with OpenStack projects16:22
openstackgerritMerged openstack/neutron master: ovn: Fix use_new_ovn_repository  https://review.opendev.org/72931816:23
openstackgerritMerged openstack/neutron master: Remove unneeded DB register retrieval and refresh in network update  https://review.opendev.org/72642416:23
ildikovmlavalle: slaweq: It would be great if we could sync up with the Neutron team as well to talk about this RFE: https://bugs.launchpad.net/neutron/+bug/1832526 and any other relevant item16:24
openstackLaunchpad bug 1832526 in neutron "[RFE] Network segmentation support for edge deployments" [Wishlist,Confirmed] - Assigned to Miguel Lavalle (minsel)16:24
*** ociuhandu has quit IRC16:24
ildikovmlavalle: slaweq: Would this slot work for you to join and discuss the above topics?16:25
*** ysastri has quit IRC16:28
*** maciejjozefczyk has quit IRC16:29
openstackgerritNate Johnston proposed openstack/neutron-lib master: Migrate timecost decorator to neutron-lib  https://review.opendev.org/73087616:30
ralonsohildikov, slaweq is AFK right now16:32
ildikovralonsoh: thanks for the letting me know16:33
ildikovralonsoh: the invitation is for everyone in the team who's interested. I thought to ping a handful of people as it's easier to coordinate that way. :)16:33
openstackgerritNate Johnston proposed openstack/neutron-lib master: Migrate timecost decorator to neutron-lib  https://review.opendev.org/73087616:40
openstackgerritNate Johnston proposed openstack/neutron-lib master: Check time to apply resource helper functions  https://review.opendev.org/73087916:40
*** jpena is now known as jpena|off16:42
openstackgerritBrian Haley proposed openstack/neutron master: Remove usage of six.next and iterators  https://review.opendev.org/73038716:48
*** mnasiadka has quit IRC16:49
mlavalleildikov: I didn't make progress on that during Ussuri. I'll be glad to work on it during Victoria. I need, though, guidance from El Comandante (slaweq) as to where he wants me to devote my upstream bandwidth. Same applies to attending the joint. I'll him prioritize where I should focus16:50
mlavallejoint session ^^^^16:50
ildikovmlavalle: noted16:51
*** coreycb has quit IRC16:51
ildikovmlavalle: it would be great if you could join or anyone from the Neutron team who's interested in getting involved in this work item or any other edge related work16:51
ildikovmlavalle: let me know how I can help to make that happen :)16:52
*** lajoskatona has quit IRC16:52
mlavalleildikov: yeah, I'll discuss with El Comandante. I need tread carefully, slaweq is also know as the Hulk, so I don't want to upset him ;-)16:53
mlavallejust kidding :-)16:54
*** coreycb has joined #openstack-neutron16:58
*** mnasiadka has joined #openstack-neutron16:58
ildikovmlavalle: lol, noted :)16:59
*** beekneemech is now known as bnemec17:01
*** lucasagomes has quit IRC17:03
*** tesseract has quit IRC17:03
*** gcheresh has joined #openstack-neutron17:04
*** mnasiadka has quit IRC17:07
ralonsohhahahaha17:07
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron stable/ussuri: Delete segment RPs when network is deleted  https://review.opendev.org/72844317:08
openstackgerritMerged openstack/networking-ovn stable/stein: [OVN][metadata] Adding ERROR trace upon unexpected data  https://review.opendev.org/72798417:08
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron stable/train: Delete segment RPs when network is deleted  https://review.opendev.org/73088817:12
openstackgerritMerged openstack/neutron master: [Fullstack] Mark security groups test as unstable  https://review.opendev.org/73058017:21
openstackgerritMerged openstack/neutron master: Mock command execution in "test_iptables.TestHelper" UTs  https://review.opendev.org/73075217:21
openstackgerritMerged openstack/neutron master: ovn: Enable debugging services on subnodes  https://review.opendev.org/72987917:21
openstackgerritMerged openstack/networking-bgpvpn master: Remove usage of six  https://review.opendev.org/73073817:21
*** mnasiadka has joined #openstack-neutron17:22
*** gcheresh has quit IRC17:23
openstackgerritMerged openstack/neutron master: Remove usage of six.text_type and six.string_type  https://review.opendev.org/73038617:29
*** ccamposr__ has joined #openstack-neutron17:35
*** ccamposr has quit IRC17:38
*** maciejjozefczyk has joined #openstack-neutron17:38
openstackgerritMerged openstack/networking-ovn stable/train: ovn: Override clean_hash_ring for sync util  https://review.opendev.org/73030917:40
*** gcheresh has joined #openstack-neutron17:41
*** KeithMnemonic has joined #openstack-neutron17:42
openstackgerritMerged openstack/networking-bagpipe stable/stein: [stein-only] Fix networking-bgpvpn and neutron requirement constraints  https://review.opendev.org/72993617:47
*** gcheresh has quit IRC17:48
*** mlavalle has quit IRC17:54
openstackgerritMerged openstack/networking-ovn stable/train: test_router: Verify the creation and removal of a lrp  https://review.opendev.org/72855918:03
openstackgerritMerged openstack/neutron master: Install "wmi" library in "win32" systems  https://review.opendev.org/71996018:04
*** dave-mccowan has quit IRC18:06
*** dave-mccowan has joined #openstack-neutron18:08
*** mlavalle has joined #openstack-neutron18:13
*** irclogbot_1 has quit IRC18:26
*** irclogbot_1 has joined #openstack-neutron18:30
*** ociuhandu has joined #openstack-neutron18:33
*** ociuhandu has quit IRC18:43
*** ociuhandu has joined #openstack-neutron18:44
*** nirmoy has quit IRC18:48
*** ociuhandu has quit IRC18:50
*** nirmoy has joined #openstack-neutron18:53
*** jawad_axd has joined #openstack-neutron18:56
*** nirmoy has quit IRC19:12
*** amoralej is now known as amoralej|off19:13
*** nirmoy has joined #openstack-neutron19:14
*** nirmoy has quit IRC19:16
*** jawad_axd has quit IRC19:24
*** jawad_axd has joined #openstack-neutron19:25
*** ociuhandu has joined #openstack-neutron19:37
openstackgerritBrian Haley proposed openstack/neutron master: Remove usage of six.next and iterators  https://review.opendev.org/73038719:41
*** ociuhandu has quit IRC19:44
*** dsneddon has joined #openstack-neutron20:06
*** nirmoy has joined #openstack-neutron20:19
*** irclogbot_1 has quit IRC20:22
*** nweinber has quit IRC20:23
*** irclogbot_2 has joined #openstack-neutron20:25
openstackgerritMerged openstack/neutron stable/rocky: Dynamically increase l3 router process queue green pool size  https://review.opendev.org/72828720:35
*** priteau has quit IRC20:36
*** ociuhandu has joined #openstack-neutron20:54
*** ociuhandu has quit IRC21:00
*** ociuhandu has joined #openstack-neutron21:01
*** nirmoy has quit IRC21:07
*** ociuhandu has quit IRC21:08
*** raildo has quit IRC21:19
*** tmorin has quit IRC21:24
*** jmlowe has quit IRC21:28
slaweqnjohnston: mlavalle: hi, please take a look at https://review.opendev.org/#/c/721554/ if You will have few minutes21:29
slaweqthx in advance :)21:29
slaweqand Good night :)21:29
*** jmlowe has joined #openstack-neutron21:32
*** slaweq has quit IRC21:34
*** slaweq has joined #openstack-neutron21:35
*** slaweq has quit IRC21:39
*** ralonsoh has quit IRC22:00
*** dsneddon has quit IRC22:05
*** tbachman has quit IRC22:08
rm_workok, I confirmed that the tempest jobs also don't pull in neutron-lib versions from Depends-On T_T22:23
rm_workthis is very problematic22:23
rm_workhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_570/730784/2/check/neutron-tempest-plugin-api/5701d1d/testr_results.html22:23
rm_workAttributeError: module 'neutron_lib.constants' has no attribute 'ACCESS_SHARED'22:23
rm_workproves it22:23
rm_workTechnically I can work around the variables being missing, that was a minor issue, the REAL issue is that it's impossible for me to test the neutron-lib DB Object Filter function that I modified, which is a very core piece of all the object workflows...22:25
rm_workmlavalle: ^^ this is going to be somewhat of a blocker for me22:25
*** dsneddon has joined #openstack-neutron22:27
rm_workI can't actually find where the "neutron-tempest-plugin-api" job is defined <_<22:43
openstackgerritAdam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action  https://review.opendev.org/73078422:53
openstackgerritAdam Harwell proposed openstack/neutron master: WIP: Add `access_as_readonly` sharing action  https://review.opendev.org/73073722:56
*** tkajinam has joined #openstack-neutron22:57
openstackgerritAdam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action  https://review.opendev.org/73078422:58
openstackgerritAdam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action  https://review.opendev.org/73078423:02
openstackgerritAdam Harwell proposed openstack/neutron-tempest-plugin master: Require neutron-lib for base test template  https://review.opendev.org/73100223:03
openstackgerritAdam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action  https://review.opendev.org/73078423:03
openstackgerritAdam Harwell proposed openstack/neutron master: WIP: Add `access_as_readonly` sharing action  https://review.opendev.org/73073723:03
openstackgerritAdam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action  https://review.opendev.org/73078423:04
*** rcernin has joined #openstack-neutron23:05
rm_workAUGH DEPENDENCIES @*%)*O#&QHECQGFI#RT&N23:05
*** threestrands has joined #openstack-neutron23:17
*** armax has quit IRC23:32
*** armax has joined #openstack-neutron23:33
*** ociuhandu has joined #openstack-neutron23:40
*** ociuhandu has quit IRC23:47

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