Tuesday, 2020-05-12

« Monday, 2020-05-11 Index Wednesday, 2020-05-13 »
*** mlavalle has quit IRC00:15
*** armax has quit IRC00:40
*** gregwork has quit IRC00:42
*** tkajinam has quit IRC01:29
*** tkajinam has joined #openstack-neutron01:29
openstackgerritzhangyuhe proposed openstack/neutron master: [OVN] Support native DHCP service for subnet without gateway IP  https://review.opendev.org/70801102:59
*** hongbin has joined #openstack-neutron03:00
openstackgerritzhangyuhe proposed openstack/neutron master: [OVN]floating IP QoS.  https://review.opendev.org/71223903:27
*** yamamoto has joined #openstack-neutron03:53
openstackgerritnorman shen proposed openstack/neutron master: Keep stale flows if there are failed devices  https://review.opendev.org/72704803:58
*** ratailor has joined #openstack-neutron04:06
*** lajoskatona has joined #openstack-neutron04:07
openstackgerritnorman shen proposed openstack/neutron master: Reraise timeout exception  https://review.opendev.org/72649204:11
*** gcheresh_ has joined #openstack-neutron04:31
*** yamamoto has quit IRC04:43
*** hongbin has quit IRC05:04
openstackgerritLajos Katona proposed openstack/networking-odl master: Use unittest.mock instead of third party mock  https://review.opendev.org/72095605:14
openstackgerritLajos Katona proposed openstack/networking-odl master: Add subnet deleted handler  https://review.opendev.org/72465605:27
*** waleedm has joined #openstack-neutron06:06
*** abdysn has joined #openstack-neutron06:11
*** dayou has quit IRC06:28
*** tkajinam has quit IRC06:35
*** tkajinam has joined #openstack-neutron06:36
openstackgerritzhangyuhe proposed openstack/neutron master: [OVN] Support native DHCP service for subnet without gateway IP  https://review.opendev.org/70801106:44
*** livelace has joined #openstack-neutron06:55
*** maciejjozefczyk has joined #openstack-neutron06:59
*** tmorin has joined #openstack-neutron06:59
*** tesseract has joined #openstack-neutron07:04
maciejjozefczykHey :) Looks like we need +W here: https://review.opendev.org/#/c/726965/107:05
openstackgerritLajos Katona proposed openstack/networking-bgpvpn master: DNM: Check the gate  https://review.opendev.org/72686007:08
*** slaweq has joined #openstack-neutron07:11
openstackgerritzhanghao proposed openstack/neutron-vpnaas master: Add aggressive negotiation mode for ikepolicy  https://review.opendev.org/72707307:19
*** awalende has joined #openstack-neutron07:23
*** ralonsoh has joined #openstack-neutron07:33
*** jlibosva has joined #openstack-neutron07:39
*** dayou has joined #openstack-neutron07:40
*** sapd1_x has joined #openstack-neutron07:45
*** lpetrut has joined #openstack-neutron07:52
*** dtantsur|afk is now known as dtantsur07:54
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Add support for OVN LB selection fields  https://review.opendev.org/72678707:54
*** jpena|off is now known as jpena07:56
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Fix some pep8 errors  https://review.opendev.org/72709307:57
*** rpittau|afk is now known as rpittau07:58
*** tmorin has quit IRC07:59
maciejjozefczykralonsoh, Hey! Brian already proposed the pep8 fix: https://review.opendev.org/#/c/726965/08:01
*** nirmoy has joined #openstack-neutron08:04
ralonsohmaciejjozefczyk, hahaha thanks!!08:05
maciejjozefczykralonsoh, :)08:05
*** tmorin has joined #openstack-neutron08:07
*** lucasagomes has joined #openstack-neutron08:09
*** tmorin has quit IRC08:13
*** yamamoto has joined #openstack-neutron08:22
*** yamamoto has quit IRC08:24
*** tmorin has joined #openstack-neutron08:31
*** awalende_ has joined #openstack-neutron08:32
*** salmankhan has joined #openstack-neutron08:33
*** awalende has quit IRC08:35
*** lpetrut_ has joined #openstack-neutron08:39
*** lpetrut has quit IRC08:42
*** livelace has quit IRC08:49
openstackgerritLajos Katona proposed openstack/networking-bgpvpn stable/ussuri: DNM: Check the gate stable/ussuri  https://review.opendev.org/72711408:50
*** priteau has joined #openstack-neutron09:02
*** yamamoto has joined #openstack-neutron09:04
*** salmankhan has quit IRC09:06
*** salmankhan has joined #openstack-neutron09:07
*** ociuhandu has joined #openstack-neutron09:07
*** ociuhandu has quit IRC09:08
*** ociuhandu has joined #openstack-neutron09:08
*** yamamoto has quit IRC09:09
*** ociuhandu_ has quit IRC09:10
*** livelace has joined #openstack-neutron09:11
*** ociuhandu_ has joined #openstack-neutron09:13
*** ociuhand_ has joined #openstack-neutron09:14
openstackgerritzhanghao proposed openstack/neutron-vpnaas master: Add aggressive negotiation mode for ikepolicy  https://review.opendev.org/72707309:16
*** ociuhandu has quit IRC09:17
*** ociuhandu has joined #openstack-neutron09:18
*** ociuhand_ has quit IRC09:18
*** ociuhandu_ has quit IRC09:18
bcafarelanother new release week, another pep8 update breaking all stable branches sigh09:32
openstackgerritMerged openstack/ovn-octavia-provider master: Fix E741 pep8 errors  https://review.opendev.org/72695309:34
*** ociuhandu has quit IRC09:51
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Use pyroute2 to retrieve the SRIOV VF information  https://review.opendev.org/72691809:56
*** ociuhandu has joined #openstack-neutron09:57
*** tesseract-RH has joined #openstack-neutron10:02
*** tesseract has quit IRC10:04
*** waleedm_ has joined #openstack-neutron10:07
*** livelace has quit IRC10:09
*** waleedm has quit IRC10:09
*** jlibosva has quit IRC10:18
*** jlibosva has joined #openstack-neutron10:19
*** rpittau is now known as rpittau|bbl10:19
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Re-home OvnProviderHelper class  https://review.opendev.org/72442710:19
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Re-home unit tests  https://review.opendev.org/72442810:19
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Re-home ovsdb event classes  https://review.opendev.org/72539810:19
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Re-home functional tests  https://review.opendev.org/72622710:19
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Add support for OVN LB selection fields  https://review.opendev.org/72678710:19
*** nirmoy has quit IRC10:27
openstackgerritDongcan Ye proposed openstack/neutron-vpnaas master: Add Ussuri milestone tag for alembic migration revisions  https://review.opendev.org/72715610:48
*** gcheresh_ has quit IRC10:52
*** lucasagomes is now known as lucas-hungry11:01
openstackgerritLajos Katona proposed openstack/networking-odl master: Update master for stable/ussuri  https://review.opendev.org/72226511:15
openstackgerritLajos Katona proposed openstack/networking-odl master: Add Python3 victoria unit tests  https://review.opendev.org/72226611:15
*** livelace has joined #openstack-neutron11:17
openstackgerritLajos Katona proposed openstack/networking-bgpvpn master: DNM: Check the gate  https://review.opendev.org/72686011:25
*** samueldmq_ has joined #openstack-neutron11:28
*** gcheresh_ has joined #openstack-neutron11:28
*** nirmoy has joined #openstack-neutron11:31
*** jpena is now known as jpena|lunch11:33
*** samueldmq_ has quit IRC11:34
*** samueldmq_ has joined #openstack-neutron11:36
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Re-home OvnProviderHelper class  https://review.opendev.org/72442711:40
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Re-home unit tests  https://review.opendev.org/72442811:40
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Re-home ovsdb event classes  https://review.opendev.org/72539811:40
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Re-home functional tests  https://review.opendev.org/72622711:40
openstackgerritMaciej Józefczyk proposed openstack/ovn-octavia-provider master: Add support for OVN LB selection fields  https://review.opendev.org/72678711:40
*** samueldmq_ is now known as samueldmq11:40
*** nweinber has joined #openstack-neutron11:52
*** lucas-hungry is now known as lucasagomes11:55
*** ratailor has quit IRC12:07
*** sapd1_x has quit IRC12:08
openstackgerritTakashi Natsume proposed openstack/os-vif master: Remove egg_info in setup.cfg  https://review.opendev.org/72717312:10
*** abdysn has quit IRC12:12
*** ratailor has joined #openstack-neutron12:20
*** rpittau|bbl is now known as rpittau12:21
*** waleedm_ has quit IRC12:24
*** adrianc has quit IRC12:24
rubasovnjohnston: hello12:27
rubasovnjohnston: I was reading yesterday's meeting log12:27
rubasovnjohnston: and would like to ask about https://bugs.launchpad.net/neutron/+bug/187803112:28
openstackLaunchpad bug 1878031 in neutron " Unable to delete an instance | Conflict: Port [port-id] is currently a parent port for trunk [trunk-id]" [Undecided,New] - Assigned to Nate Johnston (nate-johnston)12:28
*** ratailor has quit IRC12:33
*** happyhemant has joined #openstack-neutron12:36
*** jpena|lunch is now known as jpena12:40
*** elod_pto is now known as elod12:42
*** adrianc has joined #openstack-neutron12:45
*** adrianc has quit IRC12:50
*** adrianc has joined #openstack-neutron12:53
njohnstonrubasov: Sure!12:55
rubasovhi12:55
rubasovI was wondering how the second interface you write about in the bug report came to be12:56
rubasovstarting with 64f6a222-9e9a-12:56
rubasovI wasn't expecting a 2nd interface at all12:57
rubasovalso do you have the exact workflow of how the trunk and vm was created?12:59
rubasovparticularly 1) do you first create the trunk and then boot the vm, or the other way around?13:05
*** livelace has quit IRC13:05
rubasov2) are subports ever passed to nova in the vm boot?13:05
*** livelace has joined #openstack-neutron13:08
*** spotz has joined #openstack-neutron13:27
openstackgerritnorman shen proposed openstack/neutron master: Reraise timeout exception  https://review.opendev.org/72649213:36
openstackgerritMaciej Józefczyk proposed openstack/neutron master: [WIP] Run functional tests with selected OVN and OVS versions  https://review.opendev.org/72719313:40
openstackgerritMaciej Józefczyk proposed openstack/neutron master: [WIP] Run functional tests with selected OVN and OVS versions  https://review.opendev.org/72719313:42
openstackgerritnorman shen proposed openstack/neutron master: Keep stale flows if there are failed devices  https://review.opendev.org/72704813:45
*** openstackstatus has quit IRC13:53
*** openstackstatus has joined #openstack-neutron13:54
*** ChanServ sets mode: +v openstackstatus13:54
openstackgerritBrian Haley proposed openstack/ovn-octavia-provider stable/ussuri: DNM: test to see if we need to backport pep8 change  https://review.opendev.org/72719914:02
njohnstonrubasov: Apologies was taken away for a meeting.14:07
rubasovnjohnston: np14:07
*** armax has joined #openstack-neutron14:08
njohnstonrubasov: I believe when the VM is boted it is given a network, not a specific port, i.e. `openstack server create --nic net-id=<UUID>`14:08
openstackgerritBrian Haley proposed openstack/ovn-octavia-provider stable/ussuri: Fix E741 pep8 errors  https://review.opendev.org/72720114:09
njohnstonThe order of operations is: 1) parent port created; 2) trunk created and attached to the parent port; 3) indeterminate amount of time elapses; 4) server created on the trunk14:10
rubasovnjohnston: and then the user finds the port which got created by nova and is that used as a parent port in trunk create?14:10
openstackgerritBrian Haley proposed openstack/ovn-octavia-provider stable/ussuri: Fix E741 pep8 errors  https://review.opendev.org/72720114:10
njohnstonthe trunk predates the server14:10
rubasovthat's definitely not how it was originally meant to be used (not meaning to imply it is a wrong way)14:11
rubasovoriginally we never to support the case when a vm is booted with a network14:12
rubasovbut there's a driver dependent part of this14:12
*** sapd1_x has joined #openstack-neutron14:12
rubasovnjohnston: wait, I'm still confused14:13
rubasovnjohnston: how is the order you listed and server create --nic net-id is possible at the same time?14:14
*** adrianc has quit IRC14:15
rubasovif we use --nic net-id then nova creates a port and (unless we are really fast) the vm is booted before we can make the port a parent port of a trunk14:16
*** adrianc has joined #openstack-neutron14:17
njohnstonrubasov: I have not been able to follow the code in nova properly, but my working theory was that the process in nova was 1) create port on trunk, then 2) attach an available subport to the instance.  The key being that the port in 1 is not necessarily the same as the one in 2 if there is another subport that looks unowned hanging around, i.e. the trunk parent port.14:18
njohnstonso that way a previously existing parent port could be attached to an instance14:19
rubasovI feel there's some confusion about the workflow a trunk is meant to be used14:20
rubasovlet me start with the canonical way:14:21
rubasovwhile from an end user perspective (and logically) it would be nice to pass a trunk to a vm boot14:21
rubasovthis would have meant huge changes in nova14:22
rubasovso we tried hard to keep the trunk feature neutron-only14:22
rubasovthis had the consequence of nova never knowing about the trunk14:22
rubasovnova only ever knows about the parent port14:23
njohnstonok14:23
rubasovthe trunk should never be passed to nova, because nova cannot do anything with it14:23
njohnstongot it14:24
rubasovsecond, we considered the combination of booting a vm with a network (as opposed to a port) and making that port part of a trunk unsupported14:24
rubasovthere may be an exception to this, I'll come back to that later14:25
rubasovthe reason was: some drivers (like ovs) requires the trunk to be created before the vm is booted14:25
rubasovbecause a trunked port and a plain old port is wired differently and we did want to do complex live rewiring upgrades14:27
rubasovfor the trunk to exist before the vm, the parent port has to exist before the trunk and the vm14:27
rubasovso we kind of considered that the canonical way14:28
rubasovthat the parent port is always pre-created and passed explicitly to the vm boot14:28
rubasovwith ovs that was actually the only way14:28
rubasovwith other drivers though (e.g. lb or ovn) it is possible to trunk a port which was already used to boot a vm14:29
rubasovbecause these drivers do not need to wire the port differently depending on whether the port is trunked or not14:30
rubasovhere I see a possible bug after I read the bug report14:31
rubasovif we boot a vm with a network then nova creates a plain old port14:31
rubasovwe can find that port and with non-ovs drivers we can make it a parent port of a trunk14:32
rubasovafter that when we delete the vm, nova wants to delete the port (since nova created it in the first place)14:33
rubasovand we may run into the error of 'trunk is in use'14:33
rubasovbut because of the second interface in the bug report I suspect that may not be the problem you have14:34
rubasovcan it be that one or more subports were attached to the vm?14:35
njohnstonrubasov: so in your opinion, am I correct in saying the only way we should ever see a log message like "Failed to delete port 991e4e50-481a-4ca6-9ea6-69f848c4ca9f for instance.: Conflict: Port 991e4e50-481a-4ca6-9ea6-69f848c4ca9f is currently a parent port for trunk 5800ee0f-b558-46cb-bb0b-92799dbe02cf." is if the parent port was specifically requested as instance port on instance creation?14:35
*** tkajinam has quit IRC14:37
rubasovnot sure about the answer yet, but I see three cases14:39
rubasovcase 1) order: create parent port, create trunk, boot vm14:40
*** ralonsoh has quit IRC14:40
rubasovin case 1) we will never see this error because nova does now want to delete the port if it was given to it14:40
*** ralonsoh has joined #openstack-neutron14:40
rubasovcase 2) boot vm with network, never turn the nova-created port into a trunk14:41
rubasovagain no problem here14:41
rubasovcase 3) boot vm with network, find the nova-created port and make it a parent port of a trunk14:42
rubasovthis case was kind of considered unsupported, but still today this is doable and there's no warning that you shouldn't do it14:43
rubasovand here I think it is possible to hit the error you saw14:43
njohnstongiven that the `openstack port show` for the parent port has trunk_details set properly and has device_owner = 'compute:nova' does that indicate scenario 3?14:44
*** ralonsoh has quit IRC14:44
rubasovthat still could be case 1 or case 314:45
*** livelace has quit IRC14:46
rubasovlet me come back to that second interface in the report for a bit14:47
*** rcernin has quit IRC14:48
rubasovwe had to keep nova unchanged while implementing trunks14:48
rubasovthat also means nova should not know about the subports too14:48
rubasova subport should never be passed to server create14:49
rubasovbut if it is passed there's no direct error message, because of how hard it would have been to produce that error message14:50
rubasovso if a subport got directly attached to a vm, that could explain the second interface in the report14:51
rubasovbut that is a case that should not be done14:52
rubasovdoes all this make any sense? not sure if I'm explaining it right14:53
njohnstonIt does make sense, and I am very grateful.  I'm just not sure how to prevent this from happening.14:54
rubasovgreat14:54
rubasovI'm trying to separate the two possible causes I see14:55
rubasov1) subports attached directly to the vm14:55
rubasov2) the previous case 3)14:55
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Don't check if any bridges were recrected when OVS was restarted  https://review.opendev.org/72677014:56
rubasovif 1) is the cause then what needs to be fixed is that we never pass a subport to server create14:57
rubasovif 2) is the cause that we may consider a genuine bug which needs to be fixed (at the moment I'm not sure if this is to be fixed in nova or neutron)14:58
rubasovand the second interface you listed in the report gives me a hunch (not a proof though) that cause 1) is more likely14:59
openstackgerritMerged openstack/networking-midonet stable/ussuri: Update .gitreview for stable/ussuri  https://review.opendev.org/72372215:00
rubasovor more precisely that there's something else beyond cause 2) even if cause 2) is what we are seeing15:00
openstackgerritMerged openstack/networking-midonet stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri  https://review.opendev.org/72372315:00
rubasovI'll have a meeting in 10 minutes, so I'll have to drop off soon15:01
rubasovbut feel free to catch me tomorrow if you'd like to continue the discussion15:01
njohnstonrubasov: If there are investigations I can do in the meantime that would be helpful, please feel free to assign me some homework15:02
njohnstonrubasov: Yes, let's continue it; I also sent an email to the ML if that is a better forum15:02
rubasovit would be helpful to understand what api calls we had and in what other if that's something you can collect from the logs15:03
rubasovwill read the ml too15:04
rubasovthen talk to you tomorrow15:05
rubasovcheers15:05
*** hjensas is now known as hjensas|afk15:06
*** dtantsur is now known as dtantsur|afk15:08
*** dasp_ has quit IRC15:14
openstackgerritBrian Haley proposed openstack/neutron-tempest-plugin master: Add nose to requirements.txt  https://review.opendev.org/72722215:16
*** dasp has joined #openstack-neutron15:16
openstackgerritMaciej Józefczyk proposed openstack/neutron master: [WIP] Run functional tests with selected OVN and OVS versions  https://review.opendev.org/72719315:17
*** tesseract-RH has quit IRC15:22
*** tesseract has joined #openstack-neutron15:23
*** mlavalle has joined #openstack-neutron15:27
openstackgerritTerry Wilson proposed openstack/neutron master: Set class ovsdb_conection to None  https://review.opendev.org/72723215:36
*** dasp has quit IRC15:41
*** dasp has joined #openstack-neutron15:42
*** lpetrut_ has quit IRC15:51
*** maciejjozefczyk has quit IRC15:53
openstackgerritMerged openstack/neutron master: Fix some pep8 errors  https://review.opendev.org/72696516:00
*** lajoskatona has quit IRC16:02
*** livelace has joined #openstack-neutron16:02
openstackgerritNate Johnston proposed openstack/neutron master: Time port fetches  https://review.opendev.org/72478716:02
openstackgerritTerry Wilson proposed openstack/ovsdbapp master: WIP Add indexed column support to ovsdbapp  https://review.opendev.org/67807616:03
*** jlibosva has quit IRC16:05
openstackgerritFlavio Fernandes proposed openstack/neutron master: WIP (DNM): [ovn]: port forwarding functionality  https://review.opendev.org/72386316:10
*** rpittau is now known as rpittau|afk16:10
*** tesseract has quit IRC16:12
openstackgerritTerry Wilson proposed openstack/ovsdbapp master: Don't create new API instance in fixtures  https://review.opendev.org/72623516:14
openstackgerritTerry Wilson proposed openstack/ovsdbapp master: WIP Add indexed column support to ovsdbapp  https://review.opendev.org/67807616:14
*** lucasagomes has quit IRC16:17
*** gcheresh_ has quit IRC16:31
*** gcheresh_ has joined #openstack-neutron16:33
openstackgerritBrian Haley proposed openstack/neutron stable/ussuri: Cap pycodestyle to be < 2.6.0  https://review.opendev.org/72727416:57
openstackgerritMerged openstack/neutron master: Add "qos_policy_id" field to "FloatingIP" OVO  https://review.opendev.org/72620817:05
openstackgerritMerged openstack/neutron master: Open Victoria DB branch  https://review.opendev.org/72120717:05
*** numans has quit IRC17:13
*** ralonsoh has joined #openstack-neutron17:14
*** jpena is now known as jpena|off17:14
*** ralonsoh has quit IRC17:14
*** salmankhan has quit IRC17:17
*** priteau has quit IRC17:17
*** ralonsoh has joined #openstack-neutron17:19
*** iurygregory has quit IRC17:27
*** ralonsoh has quit IRC17:28
*** ralonsoh has joined #openstack-neutron17:31
*** gcheresh_ has quit IRC17:32
*** ralonsoh has quit IRC17:33
*** ociuhandu has quit IRC17:35
*** ociuhandu has joined #openstack-neutron17:37
*** ociuhandu has quit IRC17:37
*** ociuhandu has joined #openstack-neutron17:37
*** ralonsoh has joined #openstack-neutron17:39
*** ralonsoh has quit IRC17:40
*** ralonsoh has joined #openstack-neutron17:48
*** ociuhandu has quit IRC17:48
*** ociuhandu has joined #openstack-neutron17:51
openstackgerritMaciej Józefczyk proposed openstack/neutron master: Skip OVS module compilation for kernels newer than 5.5  https://review.opendev.org/72675917:51
*** ociuhandu has quit IRC17:55
*** sapd1_x has quit IRC18:03
*** JamesBen_ has quit IRC18:06
openstackgerritBrian Haley proposed openstack/neutron stable/ussuri: Cap flake8 to be < 3.8.0  https://review.opendev.org/72727418:06
gmannhaleyb: let's try without flake8 also in requirement ^^ and see if it still pull pycodestyle some somewhere18:12
gmannpycodestyle latest18:12
haleybgmann: i can run it locally, do you mean without either?18:13
gmannyeah, hacking pull flake8 compatible version and flake8 pull pycodestyle compatible18:13
gmanni mean locally you can try first18:14
haleybthis is "hacking>=1.1.0,<1.2.0" since it's stable/ussuri, not hacking 3.0.018:14
*** happyhemant has quit IRC18:15
haleybgmann: so without either it uses flake8==2.6.2 and pycodestyle==2.6.0 :(18:16
*** ralonsoh has quit IRC18:17
*** ociuhandu has joined #openstack-neutron18:18
haleybwith flake8>=3.6.0,<3.8.0 it uses flake8==3.7.9 and pycodestyle==2.5.018:18
haleybi'd have to run with -vv to see why it does that, strange18:19
gmannhaleyb: yeah so it does right things-  " flake8==2.6.2 and pycodestyle==2.6.0 " are correct version for hacking 1.1.0 which you use for stable/ussuri -https://github.com/openstack/hacking/blob/0a43156fab3ca870334098afd1914b5e0bdbc3e6/requirements.txt#L618:22
*** ralonsoh has joined #openstack-neutron18:22
gmannif you pull higher flake8 it might end up asking few code change if something in ussuri code behind some checks18:23
haleybgmann: but the pycodestyle version has the new checks that cause the failure18:23
*** jamesden_ has joined #openstack-neutron18:26
*** jamesdenton has quit IRC18:27
gmannhaleyb: but it will pull pycodestyle==2.5.0 if no flake8 or pycodestyle in test-requirement.txt18:27
*** jamesden_ is now known as jamesdenton18:28
haleybgmann: no, in that case it pulled 2.6.018:28
haleyblet me double-check18:28
haleybUsing cached pycodestyle-2.6.0-py2.py3-none-any.whl18:30
haleybtox -e pep8 -r -vv18:30
*** iurygregory has joined #openstack-neutron18:31
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Remove unneeded DB register retrieval and refresh in network update  https://review.opendev.org/72642418:32
gmannhaleyb: humm and 2.60 does not work for ussuri ?18:33
haleybgmann: no, it has the new E471 checks, and a few others18:33
haleybgmann: i just didn't want to cherry-pick a large change to fix style issues18:33
gmanni tried to find exact version of pycodestyle for https://github.com/PyCQA/flake8/ 2.6.2 but could not find18:33
haleybsince it's broken all the way to rocky18:34
*** livelace has quit IRC18:35
gmannhaleyb: cannot find how flake8 2.6.2 capped pycodestyle or started doing after that18:38
* haleyb shrugs18:38
gmannhaleyb: i think explicit cap as you proposed is way forward to avoid big backports.18:38
haleybit works... for now18:39
gmann? you mean with flake8 2.6.2 ?18:41
haleybgmann: with flake8 3.7.9 it seems to work fine18:41
gmannhaleyb: on ussuri right? it may break on older release? but anything is fine whatever work, either capping pycodestyle or flake818:43
*** ralonsoh has quit IRC18:43
gmannmay be we can check all backports together and then merge ussuri one if it work i all required stable branch18:43
haleybgmann: yes, on ussuri, i haven't tried older than that yet18:43
*** livelace has joined #openstack-neutron18:50
openstackgerritBrian Haley proposed openstack/neutron stable/ussuri: Cap pycodestyle to be < 2.6.0  https://review.opendev.org/72727419:09
*** ociuhandu has quit IRC19:12
*** ociuhandu has joined #openstack-neutron19:12
*** ociuhandu has quit IRC19:13
*** ociuhandu has joined #openstack-neutron19:13
*** nirmoy has quit IRC19:17
*** ociuhandu has quit IRC19:23
*** ociuhandu has joined #openstack-neutron19:24
*** ociuhandu has quit IRC19:29
*** dsneddon has joined #openstack-neutron19:32
*** slaweq has quit IRC19:57
*** slaweq has joined #openstack-neutron20:01
*** slaweq has quit IRC20:06
*** nweinber has quit IRC20:14
openstackgerritBrian Haley proposed openstack/neutron master: Switch to hacking 3.0  https://review.opendev.org/70896620:19
*** tmorin has quit IRC20:24
*** slaweq has joined #openstack-neutron20:35
*** awalende_ has quit IRC20:40
*** ociuhandu has joined #openstack-neutron20:40
*** awalende has joined #openstack-neutron20:42
*** ociuhandu has quit IRC20:53
*** ociuhandu has joined #openstack-neutron20:54
*** ociuhandu has quit IRC20:54
*** ociuhandu has joined #openstack-neutron20:54
openstackgerritBrian Haley proposed openstack/neutron master: Switch to hacking 3.0.1  https://review.opendev.org/70896620:59
*** jamesdenton has quit IRC20:59
*** jamesdenton has joined #openstack-neutron21:03
*** slaweq has quit IRC21:13
*** ccamposr__ has joined #openstack-neutron21:21
*** ccamposr has quit IRC21:23
*** slaweq has joined #openstack-neutron21:25
openstackgerritTerry Wilson proposed openstack/ovsdbapp master: WIP Add indexed column support to ovsdbapp  https://review.opendev.org/67807621:25
openstackgerritBrian Haley proposed openstack/neutron-lib master: Fix errors found by new pycodestyle  https://review.opendev.org/72747021:29
openstackgerritBrian Haley proposed openstack/neutron-lib master: Update to victoria jobs template  https://review.opendev.org/72747121:29
*** slaweq has quit IRC21:30
*** awalende has quit IRC21:36
*** slaweq has joined #openstack-neutron21:40
*** ociuhandu has quit IRC21:41
*** ociuhandu has joined #openstack-neutron21:42
*** ociuhandu has quit IRC21:42
*** ociuhandu has joined #openstack-neutron21:43
*** slaweq has quit IRC21:44
openstackgerritFlavio Fernandes proposed openstack/neutron master: WIP (DNM): [ovn]: port forwarding functionality  https://review.opendev.org/72386321:44
*** ociuhandu has quit IRC21:52
*** ociuhandu has joined #openstack-neutron21:54
*** ociuhandu has quit IRC21:54
*** ociuhandu has joined #openstack-neutron21:54
*** ociuhandu has quit IRC22:05
*** ociuhandu has joined #openstack-neutron22:05
*** ociuhandu has quit IRC22:06
*** ociuhandu has joined #openstack-neutron22:06
*** ociuhandu has quit IRC22:16
*** ociuhandu has joined #openstack-neutron22:17
*** ociuhandu has quit IRC22:17
*** ociuhandu has joined #openstack-neutron22:18
*** ociuhandu has quit IRC22:28
*** ociuhandu has joined #openstack-neutron22:29
*** ociuhandu has quit IRC22:29
*** ociuhandu has joined #openstack-neutron22:29
*** ccamposr has joined #openstack-neutron22:32
*** ccamposr__ has quit IRC22:35
*** ociuhandu has quit IRC22:39
*** ociuhandu has joined #openstack-neutron22:41
*** ociuhandu has quit IRC22:51
*** ociuhandu has joined #openstack-neutron22:52
*** ociuhandu has quit IRC22:52
*** ociuhandu has joined #openstack-neutron22:53
*** tkajinam has joined #openstack-neutron22:54
*** ociuhandu has quit IRC23:03
*** ociuhandu has joined #openstack-neutron23:04
*** ociuhandu has quit IRC23:15
*** ociuhandu has joined #openstack-neutron23:16
*** rcernin has joined #openstack-neutron23:19
*** ociuhandu has quit IRC23:21
*** awalende has joined #openstack-neutron23:36
*** awalende has quit IRC23:41
*** mlavalle has quit IRC23:53
« Monday, 2020-05-11 Index Wednesday, 2020-05-13 »

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