Thursday, 2018-02-22

*** yamamoto has joined #openstack-dragonflow00:08
*** yamamoto has quit IRC00:14
*** yamamoto has joined #openstack-dragonflow01:10
*** yamamoto has quit IRC01:16
*** yamamoto has joined #openstack-dragonflow02:12
*** yamamoto has quit IRC02:18
*** yamamoto has joined #openstack-dragonflow03:14
*** yamamoto has quit IRC03:20
*** openstackgerrit has joined #openstack-dragonflow03:32
openstackgerritpangliye proposed openstack/dragonflow master: fix misspelling of 'neutron'  https://review.openstack.org/54685403:32
*** yamamoto has joined #openstack-dragonflow03:37
leyalgood morning06:29
irenabmorning06:41
oansonMorning06:43
*** snapiri1 has joined #openstack-dragonflow07:28
*** snapiri1 is now known as snapiri-07:35
*** Natanbro has joined #openstack-dragonflow08:24
openstackgerritMerged openstack/dragonflow master: fix misspelling of 'neutron'  https://review.openstack.org/54685409:15
oansonlihi, ping09:31
lihioanson, pong09:32
oansonI'm looking at https://review.openstack.org/#/c/539546/09:32
oansonAnd I'm not sure what sort of tests you're thinking09:33
oansonThe only thing I can think of is a flow-based test, and these turn out badly09:33
oansonAnd in the end the related bug will be verified by tempest09:33
lihiI was thinking for a fullstack test with dnat & floating ip09:34
lihiBut if there's a tempest test testing it, I guess that's good enough09:35
lihiWhat the does the tempest test checks?09:36
oansonThe tempest verifies the bug (more coming)09:36
oansonTopology is: network1 (tenant1) + FIP connected to external. network2 (tenant2) + FIP connected to external. FIPs should be able to connect. lports behind the FIPs are on the same security group.09:37
oansondnat provides reg6, and the sg uses reg6 to verify ports can communicate.09:37
oansonreg6 - source port unique key09:38
lihiHmm ok then. that sort-of test it09:40
oansonIn theory, the application decoupling you and dimak are doing should allow isolated application tests, right?09:43
oansonWhen we have that, we can verify the packet is tagged with the correct metadata in isolation.09:43
lihioanson, yes.09:43
oansonSo we have a plan?09:44
lihiyes09:44
lihiI've already removed the -109:44
oansonThanks!09:45
lihiCould you open a bug for the future test?09:45
oansonWorking on it now :)09:46
oansonlihi, done: https://bugs.launchpad.net/dragonflow/+bug/175101309:48
openstackLaunchpad bug 1751013 in DragonFlow "dnat fullstack missing test: Incoming packet is tagged" [Low,New]09:48
openstackgerritOmer Anson proposed openstack/dragonflow master: BGP service - take logical port info from Neutron DB  https://review.openstack.org/54085310:06
*** yamamoto has quit IRC10:06
openstackgerritOmer Anson proposed openstack/dragonflow master: BGP service - take logical port info from Neutron DB  https://review.openstack.org/54085310:07
openstackgerritOmer Anson proposed openstack/dragonflow master: BGP service - take logical port info from Neutron DB  https://review.openstack.org/54085310:10
*** yamamoto has joined #openstack-dragonflow11:06
*** yamamoto has quit IRC11:13
*** snapiri- has quit IRC11:47
*** snapiri1 has joined #openstack-dragonflow11:48
openstackgerritShachar Snapiri proposed openstack/dragonflow master: [WIP] [DNR] SkyDive agent app  https://review.openstack.org/51867011:58
*** snapiri1 has quit IRC11:58
*** yamamoto has joined #openstack-dragonflow12:09
*** yamamoto has quit IRC12:14
*** yamamoto has joined #openstack-dragonflow12:27
oansonsnapiri, are you in?12:33
*** CrayZee has joined #openstack-dragonflow13:03
oansonCrayZee, hi13:04
CrayZeeyo13:04
oansonI have a zuul question for you13:04
oansonI'm getting this error on a py27/py35 job: http://logs.openstack.org/53/540853/8/check/openstack-tox-py27/09f6c74/job-output.txt.gz#_2018-02-22_10_24_49_98103113:04
CrayZeehope I can answer :)13:04
CrayZeeIt is what it says...13:05
oansonSomehow networking-sfc has made the cut, but I don't know how to add neutron-dynamic-routing (needed for: https://review.openstack.org/#/c/540853/ )13:05
oansonAny thoughts on how to add it other than replicating the job definition in our repo?13:06
CrayZeeI am not sure13:07
CrayZeeI believe we can create jobs that inherit from the py27/py35 and extend them13:07
CrayZeeDid you look at networking-sfc project to see what they did?13:09
CrayZeewhy did you add it to tox.sh?13:10
CrayZeeor maybe change the order of the lines in tox.sh?13:11
CrayZeeas networking-sfc is dependent on neutron-dynamic-routing....13:11
CrayZeeno, my bad13:12
oansonAdded to tox_install.sh to get it to work on my local environment13:15
oansonI see in project-config (zuul.d/projects.yaml, line 5058) that networking-sfc was added for us13:16
oansonCrayZee, let me try and update our project-config file, and take it from there.13:17
oansonThanks for your help!13:17
oansonpy27 also appears twice in the 'gate' section...13:19
openstackgerritOmer Anson proposed openstack/dragonflow master: BGP service - take logical port info from Neutron DB  https://review.openstack.org/54085313:22
CrayZeeoanson: https://github.com/openstack-infra/project-config/blob/master/zuul.d/projects.yaml13:26
oansonCrayZee, https://review.openstack.org/#/c/546978/ ?13:26
CrayZeeOh... I am a bit lsow :)13:26
CrayZee*slow13:27
*** yamamoto has quit IRC13:34
CrayZeeoanson: Is the title of the patch correct?13:37
oansonI think so. Why?13:38
CrayZeeit says "add to tests"13:39
CrayZeeshouldn't it b to gates?13:39
CrayZeejust being picky :P13:39
*** CrayZee has quit IRC13:41
*** yamamoto has joined #openstack-dragonflow13:44
*** yamamoto has quit IRC13:44
*** CrayZee has joined #openstack-dragonflow13:59
openstackgerritOmer Anson proposed openstack/dragonflow master: TrivialFix: '_lport_index' is used prior to global declaration  https://review.openstack.org/54699014:03
*** yamamoto has joined #openstack-dragonflow14:45
CrayZeeoanson: why do we use legacy-install-dsvm-dragonflow-kuryr-kubernetes and not the new native zuul 3.0 one?14:49
*** yamamoto has quit IRC14:51
*** Natanbro has quit IRC14:56
*** mlavalle has joined #openstack-dragonflow15:18
*** yamamoto has joined #openstack-dragonflow15:47
*** yamamoto has quit IRC15:51
*** CrayZee has quit IRC16:29
*** yamamoto has joined #openstack-dragonflow16:48
*** yamamoto has quit IRC16:52
oansonirenab, snapiri, your reviews conflict on patch https://review.openstack.org/#/c/540853/ . Could you please let me know if I should split the change into two (fix and tests) or not?17:15
*** yamamoto has joined #openstack-dragonflow17:48
*** yamamoto has quit IRC17:54
irenaboanson: I truct you to add follow up tests18:12
oansonirenab, sure :) The tests were added, they are just blocked by other stuff. I'll break up the patch.18:12
irenab+118:13
irenabthen maybe it will allow to pass kuryr gate18:13
*** openstackgerrit has quit IRC18:33
*** yamamoto has joined #openstack-dragonflow18:50
*** yamamoto has quit IRC18:56
*** yamamoto has joined #openstack-dragonflow18:59
*** yamamoto has quit IRC19:04
*** yamamoto has joined #openstack-dragonflow19:14
*** yamamoto has quit IRC19:19
*** yamamoto has joined #openstack-dragonflow19:29
*** yamamoto has quit IRC19:34
*** yamamoto has joined #openstack-dragonflow19:44
*** yamamoto has quit IRC19:52
*** yamamoto has joined #openstack-dragonflow20:48
*** yamamoto has quit IRC20:55
*** yamamoto has joined #openstack-dragonflow21:53
*** yamamoto has quit IRC21:57
*** yamamoto has joined #openstack-dragonflow22:01
*** yamamoto has quit IRC23:02

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!