openstackgerrit | Merged openstack/neutron master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/729750 | 00:01 |
---|---|---|
*** rcernin has quit IRC | 00:23 | |
*** rcernin has joined #openstack-neutron | 00:23 | |
*** iurygregory has quit IRC | 00:49 | |
*** cp- has quit IRC | 00:56 | |
*** cp- has joined #openstack-neutron | 00:57 | |
*** rcernin has quit IRC | 01:04 | |
*** rcernin has joined #openstack-neutron | 01:05 | |
*** rcernin has quit IRC | 01:05 | |
*** cp- has quit IRC | 01:08 | |
*** sapd1 has joined #openstack-neutron | 02:05 | |
openstackgerrit | yangkai proposed openstack/neutron-lib master: Update agent's attrs can be sorted https://review.opendev.org/726497 | 02:21 |
*** hongbin has joined #openstack-neutron | 03:08 | |
*** rcernin has joined #openstack-neutron | 03:19 | |
*** ramishra has joined #openstack-neutron | 03:23 | |
*** hongbin has quit IRC | 03:28 | |
openstackgerrit | yangkai proposed openstack/neutron-lib master: Update agent's attrs can be sorted https://review.opendev.org/726497 | 03:50 |
*** xiaolin has quit IRC | 03:55 | |
*** lajoskatona has joined #openstack-neutron | 03:59 | |
*** ramishra has quit IRC | 04:13 | |
*** rcernin has quit IRC | 04:20 | |
*** rcernin has joined #openstack-neutron | 04:22 | |
*** ratailor has joined #openstack-neutron | 04:28 | |
*** ratailor has quit IRC | 04:30 | |
*** rcernin has quit IRC | 04:38 | |
*** threestrands has joined #openstack-neutron | 04:40 | |
*** rcernin has joined #openstack-neutron | 04:43 | |
*** jamesdenton has quit IRC | 04:57 | |
*** jamesden_ has joined #openstack-neutron | 04:57 | |
*** ratailor has joined #openstack-neutron | 05:02 | |
*** cp- has joined #openstack-neutron | 05:13 | |
*** cp- has quit IRC | 05:14 | |
*** cp- has joined #openstack-neutron | 05:15 | |
*** cp- has quit IRC | 05:16 | |
*** gcheresh has joined #openstack-neutron | 05:16 | |
*** cp- has joined #openstack-neutron | 05:17 | |
*** ramishra has joined #openstack-neutron | 05:24 | |
frickler | sean-k-mooney: just for reference this is the gerritbot config https://opendev.org/openstack/project-config/src/branch/master/gerritbot/channels.yaml#L1041 | 05:49 |
*** hjensas__ has quit IRC | 06:14 | |
*** mvkr has quit IRC | 06:21 | |
*** ramishra has quit IRC | 06:25 | |
openstackgerrit | yangkai proposed openstack/neutron master: Support pagination for agent list https://review.opendev.org/726724 | 06:28 |
*** ramishra has joined #openstack-neutron | 06:28 | |
*** mvkr has joined #openstack-neutron | 06:33 | |
*** maciejjozefczyk has joined #openstack-neutron | 06:48 | |
*** slaweq has joined #openstack-neutron | 06:52 | |
*** tesseract has joined #openstack-neutron | 07:05 | |
*** hjensas has joined #openstack-neutron | 07:12 | |
*** S4ren has joined #openstack-neutron | 07:14 | |
openstackgerrit | inspurericzhang proposed openstack/networking-bgpvpn master: Remove the unused coding style modules https://review.opendev.org/730719 | 07:19 |
*** mvkr has quit IRC | 07:19 | |
*** ratailor has quit IRC | 07:20 | |
*** ratailor has joined #openstack-neutron | 07:20 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Remove unnecessary check if agent object from db is not None https://review.opendev.org/713432 | 07:21 |
*** iurygregory has joined #openstack-neutron | 07:22 | |
*** ralonsoh has joined #openstack-neutron | 07:24 | |
*** amoralej|off is now known as amoralej | 07:27 | |
*** jpena|off is now known as jpena | 07:27 | |
slaweq | lajoskatona: hi | 07:28 |
slaweq | lajoskatona: can You take a look at https://review.opendev.org/#/c/729600/2 ? | 07:28 |
slaweq | there is Your comment related to the changed code so I would like You to take a look into this | 07:28 |
*** ociuhandu has quit IRC | 07:29 | |
*** ociuhandu has joined #openstack-neutron | 07:29 | |
*** mvkr has joined #openstack-neutron | 07:32 | |
*** rcernin has quit IRC | 07:36 | |
*** nirmoy has joined #openstack-neutron | 07:38 | |
*** openstackstatus has quit IRC | 07:39 | |
*** openstackstatus has joined #openstack-neutron | 07:40 | |
*** ChanServ sets mode: +v openstackstatus | 07:40 | |
*** ociuhandu has quit IRC | 07:42 | |
*** ociuhandu has joined #openstack-neutron | 07:43 | |
*** rpittau|afk is now known as rpittau | 07:46 | |
*** ociuhandu has quit IRC | 07:48 | |
lajoskatona | slaweq: Hi, I check it | 07:55 |
slaweq | lajoskatona: thx | 07:55 |
*** lucasagomes has joined #openstack-neutron | 07:57 | |
lajoskatona | slaweq: done, thanks for the notification | 08:07 |
bcafarel | gmann: lajoskatona: nice to see these stein fixes for bgpvpn/bagpipe thanks! W+1 | 08:08 |
lajoskatona | bcafarel: and elod :-) | 08:10 |
bcafarel | right elod++ too :) | 08:10 |
*** nirmoy has quit IRC | 08:12 | |
*** nirmoy has joined #openstack-neutron | 08:13 | |
*** jlibosva has joined #openstack-neutron | 08:17 | |
*** ramishra has quit IRC | 08:20 | |
*** ramishra has joined #openstack-neutron | 08:26 | |
*** xiaolin has joined #openstack-neutron | 08:33 | |
*** rcernin has joined #openstack-neutron | 08:39 | |
*** rcernin has quit IRC | 08:40 | |
*** nirmoy has quit IRC | 08:53 | |
*** nirmoy has joined #openstack-neutron | 08:54 | |
*** priteau has joined #openstack-neutron | 08:54 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Set OVS manager command timeout and inactiviy probe https://review.opendev.org/720711 | 08:55 |
*** nirmoy_ has joined #openstack-neutron | 08:58 | |
*** nirmoy_ has quit IRC | 09:00 | |
rm_work | wow, neutron unit tests require sudo privs? O_o | 09:01 |
*** nirmoy has quit IRC | 09:01 | |
*** nirmoy has joined #openstack-neutron | 09:01 | |
rm_work | because of https://github.com/openstack/neutron/blob/master/neutron/agent/linux/iptables_firewall.py#L103-L105 | 09:01 |
rm_work | called from https://github.com/openstack/neutron/blob/master/neutron/tests/unit/agent/linux/openvswitch_firewall/test_iptables.py#L73 | 09:02 |
rm_work | that seems... not unit-test-ey | 09:02 |
rm_work | IMO a unit test should not run `sysctl` on my system as root <_< | 09:03 |
*** dtantsur|afk is now known as dtantsur | 09:05 | |
*** tkajinam has quit IRC | 09:05 | |
openstackgerrit | Adam Harwell proposed openstack/neutron-lib master: WIP: Add `access_as_readonly` sharing action https://review.opendev.org/730736 | 09:09 |
openstackgerrit | Adam Harwell proposed openstack/neutron master: WIP: Add `access_as_readonly` sharing action https://review.opendev.org/730737 | 09:09 |
openstackgerrit | Adam Harwell proposed openstack/neutron master: WIP: Add `access_as_readonly` sharing action https://review.opendev.org/730737 | 09:10 |
openstackgerrit | zhanghao proposed openstack/networking-bgpvpn master: Remove usage of six https://review.opendev.org/730738 | 09:10 |
*** ccamposr has joined #openstack-neutron | 09:13 | |
*** ccamposr__ has quit IRC | 09:16 | |
*** iurygregory has quit IRC | 09:17 | |
*** iurygregory has joined #openstack-neutron | 09:18 | |
rm_work | uhh, do neutron's gates not work with Depends-On for neutron-lib changes? | 09:19 |
rm_work | https://review.opendev.org/#/c/730737/ -> https://02c2159d30aff3e28da4-bbce49889c764923168fb20665b2170c.ssl.cf2.rackcdn.com/730737/2/check/openstack-tox-py36/e40ce04/job-output.txt | 09:19 |
rm_work | 2020-05-26 09:15:58.698431 | ubuntu-bionic | AttributeError: module 'neutron_lib.constants' has no attribute 'ACCESS_SHARED' | 09:19 |
rm_work | or 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-neutron | 09:25 | |
openstackgerrit | Merged openstack/networking-bgpvpn stable/stein: Use tox 'all' env instead of 'all-plugins' for networking bgpvpn tempest plugin https://review.opendev.org/730379 | 09:29 |
openstackgerrit | Merged openstack/neutron-tempest-plugin master: Use default network type during PortTestCasesResourceRequest setup https://review.opendev.org/729600 | 09:33 |
*** tesseract has quit IRC | 09:34 | |
openstackgerrit | Noy fridman proposed openstack/neutron-tempest-plugin master: Add "test_port_forwardings_negative" module https://review.opendev.org/730741 | 09:35 |
ralonsoh | rm_work, do you have a log for the firewall problem? | 09:37 |
*** tesseract has joined #openstack-neutron | 09:39 | |
openstackgerrit | zhanghao proposed openstack/networking-bgpvpn master: Remove usage of six https://review.opendev.org/730738 | 09:41 |
openstackgerrit | Flavio Fernandes proposed openstack/neutron master: DNM [ovn]: port forwarding functionality (1 of 2) https://review.opendev.org/723863 | 09:41 |
slaweq | ralonsoh: 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 |
slaweq | thx in advance | 09:44 |
ralonsoh | slaweq, give me 20 mins | 09:44 |
slaweq | ralonsoh: thx a lot | 09:45 |
*** nirmoy has quit IRC | 09:55 | |
*** nirmoy has joined #openstack-neutron | 09:55 | |
*** tmorin has joined #openstack-neutron | 10:02 | |
*** threestrands has quit IRC | 10:05 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Use pyroute2 for SRIOV VF commands https://review.opendev.org/726918 | 10:06 |
frickler | rm_work: in my experience neutron-lib needs a release before it can be tested on neutron | 10:09 |
slaweq | rm_work: sorry for late reply | 10:10 |
rm_work | hmm ok | 10:10 |
slaweq | rm_work: according to the tests issue, it's probably bug and should be fixed, UT shouldn't require any sudo | 10:10 |
rm_work | I can get a traceback log sure, sec | 10:10 |
slaweq | rm_work: and according to neutron-lib - frickler is correct here, You need new neutron-lib release to use it in neutron patch | 10:10 |
rm_work | hmm, so ... it's very hard to actually test them together then? | 10:12 |
rm_work | that'll be difficult :( | 10:13 |
frickler | slaweq: 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 features | 10:13 |
rm_work | yeah, in octavia we made sure octavia-lib dependencies will work | 10:13 |
rm_work | though there is one case where they are problematic | 10:13 |
rm_work | ... or is it that we have at least one specific job where they work ... crap, I forgot. I'll have to check | 10:13 |
rm_work | http://paste.openstack.org/show/793980/ | 10:14 |
rm_work | ^^ that's the firewall test requiring sudo | 10:14 |
ralonsoh | ok, I'll open a bug for this one | 10:15 |
rm_work | I wonder if possibly it's only being *triggered* because I'm on OSX? | 10:15 |
rm_work | but it's clear when I follow it that it would be doing a sudo there if that driver was ever loaded | 10:15 |
slaweq | rm_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 point | 10:16 |
slaweq | idk | 10:16 |
rm_work | the 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 not | 10:16 |
rm_work | yeah it may be an opinion thing -- IMO a "unit test" should not rely on any system binary at all | 10:17 |
rm_work | other than python | 10:17 |
slaweq | rm_work: that's for sure, please open bug and we can check that - it should be fixed | 10:17 |
rm_work | but I am also in a very very small minority of people trying to keep their testing running on native OSX :/ | 10:17 |
rm_work | I'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 |
ralonsoh | rm_work, indeed it's calling sysctl | 10:18 |
ralonsoh | I'll open a bug for this now | 10:18 |
rm_work | thanks ralonsoh | 10:18 |
slaweq | rm_work: thx for reporting this | 10:19 |
*** rpittau is now known as rpittau|bbl | 10:20 | |
rm_work | np, it's a very selfish report anyway ;) | 10:20 |
rm_work | but with no neutron-lib dependency support in the gate, this means I have to test everything locally I guess, ugh | 10:21 |
rm_work | maybe I'll see what I can do about making it pull the dep ;) | 10:21 |
slaweq | rm_work: if you "just" need api extension to be in neutron-lib, You can first introduce it in neutron and later move to neutron-lib | 10:22 |
slaweq | rm_work: or if You propose patch, we can do new release as soon as we will merge this patch | 10:22 |
slaweq | to unblock Your further work :) | 10:22 |
rm_work | well 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 import | 10:22 |
rm_work | but yeah i can just copy/paste them around for now | 10:22 |
rm_work | and fix later | 10:22 |
rm_work | that's probably easiest | 10:23 |
openstackgerrit | Adam Harwell proposed openstack/neutron-lib master: WIP: Add `access_as_readonly` sharing action https://review.opendev.org/730736 | 10:28 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Mock command execution in "test_iptables.TestHelper" UTs https://review.opendev.org/730752 | 10:29 |
lajoskatona | slaweq: I checked https://review.opendev.org/721554 as far as my limited DVR knowledge let me check | 10:31 |
*** gcheresh has quit IRC | 10:32 | |
*** priteau has quit IRC | 10:35 | |
*** ramishra has quit IRC | 10:39 | |
openstackgerrit | Adam Harwell proposed openstack/neutron master: WIP: Add `access_as_readonly` sharing action https://review.opendev.org/730737 | 10:41 |
rm_work | will try that mock patch and see if I can get a full unit test run now :) thanks | 10:41 |
rm_work | oh rofl i forgot I have core in neutron-lib, thought gerrit was broken for a sec, showing me +/-2 options :P | 10:45 |
sean-k-mooney | hehe 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 chain | 10:52 |
*** gcheresh has joined #openstack-neutron | 10:56 | |
*** also_stingrayza has joined #openstack-neutron | 10:56 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: [DVR] Reconfigure re-created physical bridges for dvr routers https://review.opendev.org/721554 | 10:59 |
*** stingrayza has quit IRC | 10:59 | |
ralonsoh | slaweq, +1 only for the question | 11:02 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: [DVR] Recofigure re-created physical bridges for dvr routers https://review.opendev.org/721554 | 11:02 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Don't check if any bridges were recrected when OVS was restarted https://review.opendev.org/726770 | 11:02 |
*** ramishra has joined #openstack-neutron | 11:10 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: [DVR] Reconfigure re-created physical bridges for dvr routers https://review.opendev.org/721554 | 11:16 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Don't check if any bridges were recrected when OVS was restarted https://review.opendev.org/726770 | 11:16 |
*** ociuhandu has quit IRC | 11:18 | |
*** ociuhandu has joined #openstack-neutron | 11:18 | |
*** sapd1 has quit IRC | 11:23 | |
*** ociuhandu has quit IRC | 11:23 | |
slaweq | ralonsoh: I replied You | 11:28 |
rm_work | ralonsoh: another one :/ http://paste.openstack.org/show/793984/ | 11:29 |
rm_work | in neutron/tests/unit/plugins/ml2/drivers/openvswitch/agent/test_ovs_neutron_agent.py | 11:30 |
*** jpena is now known as jpena|lunch | 11:30 | |
rm_work | same eventual `_check_netfilter_for_bridges` call | 11:30 |
*** ociuhandu has joined #openstack-neutron | 11:56 | |
*** nirmoy has quit IRC | 11:59 | |
*** lajoskatona has quit IRC | 12:03 | |
*** ociuhandu has quit IRC | 12:03 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Ping external resource in the neutron tests https://review.opendev.org/730766 | 12:06 |
*** nirmoy has joined #openstack-neutron | 12:13 | |
rm_work | slaweq: 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_work | if 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_T | 12:17 |
rm_work | unless the only option is in Tempest | 12:17 |
*** lajoskatona has joined #openstack-neutron | 12:18 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Enable bridge debugging services on the tempest and grenade jobs https://review.opendev.org/730772 | 12:19 |
slaweq | rm_work: looking, give me a sec | 12:21 |
rm_work | oh no rush | 12:22 |
rm_work | i 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 here | 12:22 |
*** rpittau|bbl is now known as rpittau | 12:22 | |
slaweq | rm_work: did You check in https://github.com/openstack/neutron/blob/master/neutron/tests/unit/db/test_rbac_db_mixin.py ? | 12:22 |
rm_work | yes | 12:23 |
rm_work | at least, I think so... | 12:23 |
rm_work | let me review that, so many of these files have similar names | 12:23 |
slaweq | sure | 12:23 |
rm_work | ah yeah, so that does have networks, I will look into that, though nothing for the other object types | 12:24 |
slaweq | ralonsoh: 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_work | other than a small piece I already did edit | 12:24 |
slaweq | rm_work: at the bottom there are also tests for QoS policies there | 12:24 |
rm_work | yeah i added SG to that but it isn't REALLY testing anything | 12:25 |
rm_work | https://review.opendev.org/#/c/730737/3/neutron/tests/unit/db/test_rbac_db_mixin.py@303 | 12:25 |
rm_work | but i'll try to add a test for Network with readonly and see if it helps me | 12:26 |
slaweq | rm_work: You can also look at dougwig's patch https://review.opendev.org/#/c/635311/22 which introduced rbac for SG | 12:26 |
rm_work | yeah, I have been perusing that | 12:26 |
slaweq | :) | 12:26 |
rm_work | the bits he added are... confusing | 12:26 |
lajoskatona | slaweq: I check it | 12:27 |
rm_work | like in neutron/tests/unit/objects/test_rbac.py | 12:27 |
rm_work | or neutron/tests/unit/objects/test_securitygroup.py | 12:27 |
*** ociuhandu has joined #openstack-neutron | 12:28 | |
*** amoralej is now known as amoralej|lunch | 12:28 | |
rm_work | reading them I have no idea how they function at all, but I will try again | 12:28 |
openstackgerrit | LIU Yulong proposed openstack/neutron master: Speedup: do not use root_helper_daemon by default https://review.opendev.org/653378 | 12:28 |
*** ociuhandu has quit IRC | 12:29 | |
*** nirmoy has quit IRC | 12:29 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Ping external resource in the neutron tests https://review.opendev.org/730766 | 12:30 |
ralonsoh | slaweq, let me check | 12:30 |
rm_work | i 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_work | it seems like this should be possible in units, since it's all just DB+API stuff, but I didn't see anything similar | 12:30 |
rm_work | these all just do like "create policy and verify the create returned successfully" | 12:31 |
slaweq | rm_work: I think that those are just "lower level: tests which are testing if OVO works fine, | 12:32 |
slaweq | rm_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 that | 12:32 |
slaweq | so maybe we simply don't have UT which You are looking for | 12:32 |
rm_work | yeah, so maybe tempest is the only option unless I figure out how to do it all from scratch | 12:33 |
rm_work | though 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_work | I will try and see | 12:33 |
*** jpena|lunch is now known as jpena | 12:34 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Enable bridge debugging services on the ci jobs https://review.opendev.org/730775 | 12:35 |
*** nweinber has joined #openstack-neutron | 12:39 | |
ralonsoh | slaweq, https://review.opendev.org/#/c/729567 | 12:39 |
ralonsoh | very good patch! | 12:39 |
ralonsoh | let's clean the zuul definition | 12:39 |
*** jawad_axd has quit IRC | 12:39 | |
*** jawad_axd has joined #openstack-neutron | 12:40 | |
*** nirmoy has joined #openstack-neutron | 12:42 | |
*** jawad_axd has quit IRC | 12:45 | |
*** jawad_ax_ has joined #openstack-neutron | 12:45 | |
*** dave-mccowan has joined #openstack-neutron | 12:46 | |
openstackgerrit | Adam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action https://review.opendev.org/730784 | 12:53 |
*** ysastri has joined #openstack-neutron | 12:53 | |
rm_work | we'll see what happens in that run (if it pulls in neutron-lib correctly, etc) | 12:54 |
rm_work | oh, best way to check that is to remove the temporary things :D | 12:54 |
openstackgerrit | Adam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action https://review.opendev.org/730784 | 12:56 |
rm_work | will see what happened with that when I wake up | 12:56 |
ysastri | https://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 |
openstack | Launchpad bug 1873761 in neutron "Internal IP leak to physical interface from qrouter in DVR mode" [Undecided,New] | 13:02 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Fix iptables rules comments https://review.opendev.org/730793 | 13:02 |
*** raildo has joined #openstack-neutron | 13:09 | |
*** S4ren has quit IRC | 13:09 | |
openstackgerrit | Merged openstack/neutron-dynamic-routing master: Use unittest.mock instead of third party mock https://review.opendev.org/720962 | 13:13 |
*** amoralej|lunch is now known as amoralej | 13:18 | |
*** nirmoy has quit IRC | 13:18 | |
*** ociuhandu has joined #openstack-neutron | 13:19 | |
*** nirmoy has joined #openstack-neutron | 13:19 | |
*** nirmoy has quit IRC | 13:19 | |
*** nirmoy has joined #openstack-neutron | 13:19 | |
*** nirmoy has quit IRC | 13:21 | |
*** nirmoy has joined #openstack-neutron | 13:21 | |
*** nirmoy has quit IRC | 13:23 | |
*** nirmoy has joined #openstack-neutron | 13:23 | |
*** ociuhandu has quit IRC | 13:24 | |
*** nirmoy has quit IRC | 13:25 | |
*** nirmoy has joined #openstack-neutron | 13:25 | |
*** nirmoy has quit IRC | 13:26 | |
*** nirmoy has joined #openstack-neutron | 13:26 | |
*** raildo has quit IRC | 13:30 | |
*** ratailor has quit IRC | 13:33 | |
*** aedc_ has joined #openstack-neutron | 13:35 | |
*** tkajinam has joined #openstack-neutron | 13:37 | |
*** S4ren has joined #openstack-neutron | 13:37 | |
*** aedc has quit IRC | 13:38 | |
*** jawad_ax_ has quit IRC | 13:39 | |
*** JamesBenson has joined #openstack-neutron | 13:42 | |
*** aedc_ has quit IRC | 13:44 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron stable/ussuri: Ensure that stale flows are cleaned from phys_bridges https://review.opendev.org/730820 | 13:45 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron stable/train: Ensure that stale flows are cleaned from phys_bridges https://review.opendev.org/730822 | 13:46 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron stable/stein: Ensure that stale flows are cleaned from phys_bridges https://review.opendev.org/730823 | 13:46 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron stable/rocky: Ensure iptables rule was added only once https://review.opendev.org/730824 | 13:48 |
*** nirmoy_ has joined #openstack-neutron | 13:48 | |
*** nirmoy has quit IRC | 13:48 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron stable/queens: Ensure iptables rule was added only once https://review.opendev.org/730825 | 13:48 |
*** nirmoy_ has quit IRC | 13:52 | |
*** nirmoy has joined #openstack-neutron | 13:52 | |
*** dtantsur is now known as dtantsur|brb | 13:53 | |
*** rpittau is now known as rpittau|brb | 14:10 | |
*** raildo has joined #openstack-neutron | 14:17 | |
*** amoralej is now known as amoralej|brb | 14:17 | |
*** aedc has joined #openstack-neutron | 14:23 | |
*** nirmoy has quit IRC | 14:32 | |
*** nirmoy has joined #openstack-neutron | 14:32 | |
openstackgerrit | zhanghao proposed openstack/neutron-dynamic-routing master: Remove usage of six https://review.opendev.org/730590 | 14:33 |
*** nirmoy has quit IRC | 14:33 | |
*** nirmoy has joined #openstack-neutron | 14:33 | |
*** tkajinam has quit IRC | 14:36 | |
*** rpittau|brb is now known as rpittau | 14:39 | |
*** iurygregory has quit IRC | 14:45 | |
openstackgerrit | Merged openstack/networking-ovn stable/queens: Fix ovn-db-sync-util after removing l3 ovsdb connection https://review.opendev.org/728937 | 14:48 |
openstackgerrit | Merged openstack/networking-ovn stable/rocky: Fix ovn-db-sync-util after removing l3 ovsdb connection https://review.opendev.org/728936 | 14:48 |
*** dtantsur|brb is now known as dtantsur | 14:50 | |
*** amoralej|brb is now known as amoralej | 14:50 | |
*** stingrayza has joined #openstack-neutron | 14:52 | |
*** dayou has quit IRC | 14:52 | |
*** iurygregory has joined #openstack-neutron | 14:53 | |
*** dayou has joined #openstack-neutron | 14:55 | |
*** also_stingrayza has quit IRC | 14:55 | |
*** S4ren has quit IRC | 15:01 | |
*** armax has joined #openstack-neutron | 15:03 | |
*** mlavalle has joined #openstack-neutron | 15:09 | |
openstackgerrit | Merged openstack/neutron-tempest-plugin master: Reorganize zuul jobs definitions https://review.opendev.org/729567 | 15:13 |
*** ociuhandu has joined #openstack-neutron | 15:19 | |
*** ociuhandu has quit IRC | 15:19 | |
*** ociuhandu has joined #openstack-neutron | 15:19 | |
openstackgerrit | Lajos Katona proposed openstack/networking-bagpipe master: Remove usage of six library https://review.opendev.org/730853 | 15:24 |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron stable/rocky: Dynamically increase l3 router process queue green pool size https://review.opendev.org/728287 | 15:30 |
*** xiaolin has quit IRC | 15:31 | |
*** priteau has joined #openstack-neutron | 15:34 | |
*** dtantsur is now known as dtantsur|afk | 15:34 | |
*** gcheresh has quit IRC | 15:35 | |
*** S4ren has joined #openstack-neutron | 15:36 | |
*** tmorin has quit IRC | 15:40 | |
*** rpittau is now known as rpittau|afk | 15:51 | |
*** tmorin has joined #openstack-neutron | 15:56 | |
*** jlibosva has quit IRC | 15:56 | |
*** jawad_axd has joined #openstack-neutron | 15:59 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Mock command execution in "test_hybrid_plug_flag_based_on_firewall" UT https://review.opendev.org/730869 | 16:00 |
*** hamalq has joined #openstack-neutron | 16:02 | |
*** jawad_axd has quit IRC | 16:04 | |
*** ociuhandu has quit IRC | 16:19 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron master: Use pyroute2 for SRIOV VF commands https://review.opendev.org/726918 | 16:19 |
*** ociuhandu has joined #openstack-neutron | 16:20 | |
openstackgerrit | Merged openstack/neutron master: Remove usage of six.PY2 https://review.opendev.org/730385 | 16:21 |
openstackgerrit | Merged openstack/neutron master: Delete segment RPs when network is deleted https://review.opendev.org/728507 | 16:21 |
ildikov | mlavalle: slaweq: hi | 16:22 |
ildikov | mlavalle: 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 projects | 16:22 |
openstackgerrit | Merged openstack/neutron master: ovn: Fix use_new_ovn_repository https://review.opendev.org/729318 | 16:23 |
openstackgerrit | Merged openstack/neutron master: Remove unneeded DB register retrieval and refresh in network update https://review.opendev.org/726424 | 16:23 |
ildikov | mlavalle: 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 item | 16:24 |
openstack | Launchpad bug 1832526 in neutron "[RFE] Network segmentation support for edge deployments" [Wishlist,Confirmed] - Assigned to Miguel Lavalle (minsel) | 16:24 |
*** ociuhandu has quit IRC | 16:24 | |
ildikov | mlavalle: slaweq: Would this slot work for you to join and discuss the above topics? | 16:25 |
*** ysastri has quit IRC | 16:28 | |
*** maciejjozefczyk has quit IRC | 16:29 | |
openstackgerrit | Nate Johnston proposed openstack/neutron-lib master: Migrate timecost decorator to neutron-lib https://review.opendev.org/730876 | 16:30 |
ralonsoh | ildikov, slaweq is AFK right now | 16:32 |
ildikov | ralonsoh: thanks for the letting me know | 16:33 |
ildikov | ralonsoh: 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 |
openstackgerrit | Nate Johnston proposed openstack/neutron-lib master: Migrate timecost decorator to neutron-lib https://review.opendev.org/730876 | 16:40 |
openstackgerrit | Nate Johnston proposed openstack/neutron-lib master: Check time to apply resource helper functions https://review.opendev.org/730879 | 16:40 |
*** jpena is now known as jpena|off | 16:42 | |
openstackgerrit | Brian Haley proposed openstack/neutron master: Remove usage of six.next and iterators https://review.opendev.org/730387 | 16:48 |
*** mnasiadka has quit IRC | 16:49 | |
mlavalle | ildikov: 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 focus | 16:50 |
mlavalle | joint session ^^^^ | 16:50 |
ildikov | mlavalle: noted | 16:51 |
*** coreycb has quit IRC | 16:51 | |
ildikov | mlavalle: 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 work | 16:51 |
ildikov | mlavalle: let me know how I can help to make that happen :) | 16:52 |
*** lajoskatona has quit IRC | 16:52 | |
mlavalle | ildikov: 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 |
mlavalle | just kidding :-) | 16:54 |
*** coreycb has joined #openstack-neutron | 16:58 | |
*** mnasiadka has joined #openstack-neutron | 16:58 | |
ildikov | mlavalle: lol, noted :) | 16:59 |
*** beekneemech is now known as bnemec | 17:01 | |
*** lucasagomes has quit IRC | 17:03 | |
*** tesseract has quit IRC | 17:03 | |
*** gcheresh has joined #openstack-neutron | 17:04 | |
*** mnasiadka has quit IRC | 17:07 | |
ralonsoh | hahahaha | 17:07 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron stable/ussuri: Delete segment RPs when network is deleted https://review.opendev.org/728443 | 17:08 |
openstackgerrit | Merged openstack/networking-ovn stable/stein: [OVN][metadata] Adding ERROR trace upon unexpected data https://review.opendev.org/727984 | 17:08 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/neutron stable/train: Delete segment RPs when network is deleted https://review.opendev.org/730888 | 17:12 |
openstackgerrit | Merged openstack/neutron master: [Fullstack] Mark security groups test as unstable https://review.opendev.org/730580 | 17:21 |
openstackgerrit | Merged openstack/neutron master: Mock command execution in "test_iptables.TestHelper" UTs https://review.opendev.org/730752 | 17:21 |
openstackgerrit | Merged openstack/neutron master: ovn: Enable debugging services on subnodes https://review.opendev.org/729879 | 17:21 |
openstackgerrit | Merged openstack/networking-bgpvpn master: Remove usage of six https://review.opendev.org/730738 | 17:21 |
*** mnasiadka has joined #openstack-neutron | 17:22 | |
*** gcheresh has quit IRC | 17:23 | |
openstackgerrit | Merged openstack/neutron master: Remove usage of six.text_type and six.string_type https://review.opendev.org/730386 | 17:29 |
*** ccamposr__ has joined #openstack-neutron | 17:35 | |
*** ccamposr has quit IRC | 17:38 | |
*** maciejjozefczyk has joined #openstack-neutron | 17:38 | |
openstackgerrit | Merged openstack/networking-ovn stable/train: ovn: Override clean_hash_ring for sync util https://review.opendev.org/730309 | 17:40 |
*** gcheresh has joined #openstack-neutron | 17:41 | |
*** KeithMnemonic has joined #openstack-neutron | 17:42 | |
openstackgerrit | Merged openstack/networking-bagpipe stable/stein: [stein-only] Fix networking-bgpvpn and neutron requirement constraints https://review.opendev.org/729936 | 17:47 |
*** gcheresh has quit IRC | 17:48 | |
*** mlavalle has quit IRC | 17:54 | |
openstackgerrit | Merged openstack/networking-ovn stable/train: test_router: Verify the creation and removal of a lrp https://review.opendev.org/728559 | 18:03 |
openstackgerrit | Merged openstack/neutron master: Install "wmi" library in "win32" systems https://review.opendev.org/719960 | 18:04 |
*** dave-mccowan has quit IRC | 18:06 | |
*** dave-mccowan has joined #openstack-neutron | 18:08 | |
*** mlavalle has joined #openstack-neutron | 18:13 | |
*** irclogbot_1 has quit IRC | 18:26 | |
*** irclogbot_1 has joined #openstack-neutron | 18:30 | |
*** ociuhandu has joined #openstack-neutron | 18:33 | |
*** ociuhandu has quit IRC | 18:43 | |
*** ociuhandu has joined #openstack-neutron | 18:44 | |
*** nirmoy has quit IRC | 18:48 | |
*** ociuhandu has quit IRC | 18:50 | |
*** nirmoy has joined #openstack-neutron | 18:53 | |
*** jawad_axd has joined #openstack-neutron | 18:56 | |
*** nirmoy has quit IRC | 19:12 | |
*** amoralej is now known as amoralej|off | 19:13 | |
*** nirmoy has joined #openstack-neutron | 19:14 | |
*** nirmoy has quit IRC | 19:16 | |
*** jawad_axd has quit IRC | 19:24 | |
*** jawad_axd has joined #openstack-neutron | 19:25 | |
*** ociuhandu has joined #openstack-neutron | 19:37 | |
openstackgerrit | Brian Haley proposed openstack/neutron master: Remove usage of six.next and iterators https://review.opendev.org/730387 | 19:41 |
*** ociuhandu has quit IRC | 19:44 | |
*** dsneddon has joined #openstack-neutron | 20:06 | |
*** nirmoy has joined #openstack-neutron | 20:19 | |
*** irclogbot_1 has quit IRC | 20:22 | |
*** nweinber has quit IRC | 20:23 | |
*** irclogbot_2 has joined #openstack-neutron | 20:25 | |
openstackgerrit | Merged openstack/neutron stable/rocky: Dynamically increase l3 router process queue green pool size https://review.opendev.org/728287 | 20:35 |
*** priteau has quit IRC | 20:36 | |
*** ociuhandu has joined #openstack-neutron | 20:54 | |
*** ociuhandu has quit IRC | 21:00 | |
*** ociuhandu has joined #openstack-neutron | 21:01 | |
*** nirmoy has quit IRC | 21:07 | |
*** ociuhandu has quit IRC | 21:08 | |
*** raildo has quit IRC | 21:19 | |
*** tmorin has quit IRC | 21:24 | |
*** jmlowe has quit IRC | 21:28 | |
slaweq | njohnston: mlavalle: hi, please take a look at https://review.opendev.org/#/c/721554/ if You will have few minutes | 21:29 |
slaweq | thx in advance :) | 21:29 |
slaweq | and Good night :) | 21:29 |
*** jmlowe has joined #openstack-neutron | 21:32 | |
*** slaweq has quit IRC | 21:34 | |
*** slaweq has joined #openstack-neutron | 21:35 | |
*** slaweq has quit IRC | 21:39 | |
*** ralonsoh has quit IRC | 22:00 | |
*** dsneddon has quit IRC | 22:05 | |
*** tbachman has quit IRC | 22:08 | |
rm_work | ok, I confirmed that the tempest jobs also don't pull in neutron-lib versions from Depends-On T_T | 22:23 |
rm_work | this is very problematic | 22:23 |
rm_work | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_570/730784/2/check/neutron-tempest-plugin-api/5701d1d/testr_results.html | 22:23 |
rm_work | AttributeError: module 'neutron_lib.constants' has no attribute 'ACCESS_SHARED' | 22:23 |
rm_work | proves it | 22:23 |
rm_work | Technically 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_work | mlavalle: ^^ this is going to be somewhat of a blocker for me | 22:25 |
*** dsneddon has joined #openstack-neutron | 22:27 | |
rm_work | I can't actually find where the "neutron-tempest-plugin-api" job is defined <_< | 22:43 |
openstackgerrit | Adam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action https://review.opendev.org/730784 | 22:53 |
openstackgerrit | Adam Harwell proposed openstack/neutron master: WIP: Add `access_as_readonly` sharing action https://review.opendev.org/730737 | 22:56 |
*** tkajinam has joined #openstack-neutron | 22:57 | |
openstackgerrit | Adam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action https://review.opendev.org/730784 | 22:58 |
openstackgerrit | Adam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action https://review.opendev.org/730784 | 23:02 |
openstackgerrit | Adam Harwell proposed openstack/neutron-tempest-plugin master: Require neutron-lib for base test template https://review.opendev.org/731002 | 23:03 |
openstackgerrit | Adam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action https://review.opendev.org/730784 | 23:03 |
openstackgerrit | Adam Harwell proposed openstack/neutron master: WIP: Add `access_as_readonly` sharing action https://review.opendev.org/730737 | 23:03 |
openstackgerrit | Adam Harwell proposed openstack/neutron-tempest-plugin master: WIP: Test `access_as_readonly` sharing action https://review.opendev.org/730784 | 23:04 |
*** rcernin has joined #openstack-neutron | 23:05 | |
rm_work | AUGH DEPENDENCIES @*%)*O#&QHECQGFI#RT&N | 23:05 |
*** threestrands has joined #openstack-neutron | 23:17 | |
*** armax has quit IRC | 23:32 | |
*** armax has joined #openstack-neutron | 23:33 | |
*** ociuhandu has joined #openstack-neutron | 23:40 | |
*** ociuhandu has quit IRC | 23:47 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!