*** yamamoto has joined #openstack-dragonflow | 00:08 | |
*** yamamoto has quit IRC | 00:14 | |
*** yamamoto has joined #openstack-dragonflow | 01:10 | |
*** yamamoto has quit IRC | 01:16 | |
*** yamamoto has joined #openstack-dragonflow | 02:12 | |
*** yamamoto has quit IRC | 02:18 | |
*** yamamoto has joined #openstack-dragonflow | 03:14 | |
*** yamamoto has quit IRC | 03:20 | |
*** openstackgerrit has joined #openstack-dragonflow | 03:32 | |
openstackgerrit | pangliye proposed openstack/dragonflow master: fix misspelling of 'neutron' https://review.openstack.org/546854 | 03:32 |
---|---|---|
*** yamamoto has joined #openstack-dragonflow | 03:37 | |
leyal | good morning | 06:29 |
irenab | morning | 06:41 |
oanson | Morning | 06:43 |
*** snapiri1 has joined #openstack-dragonflow | 07:28 | |
*** snapiri1 is now known as snapiri- | 07:35 | |
*** Natanbro has joined #openstack-dragonflow | 08:24 | |
openstackgerrit | Merged openstack/dragonflow master: fix misspelling of 'neutron' https://review.openstack.org/546854 | 09:15 |
oanson | lihi, ping | 09:31 |
lihi | oanson, pong | 09:32 |
oanson | I'm looking at https://review.openstack.org/#/c/539546/ | 09:32 |
oanson | And I'm not sure what sort of tests you're thinking | 09:33 |
oanson | The only thing I can think of is a flow-based test, and these turn out badly | 09:33 |
oanson | And in the end the related bug will be verified by tempest | 09:33 |
lihi | I was thinking for a fullstack test with dnat & floating ip | 09:34 |
lihi | But if there's a tempest test testing it, I guess that's good enough | 09:35 |
lihi | What the does the tempest test checks? | 09:36 |
oanson | The tempest verifies the bug (more coming) | 09:36 |
oanson | Topology 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 |
oanson | dnat provides reg6, and the sg uses reg6 to verify ports can communicate. | 09:37 |
oanson | reg6 - source port unique key | 09:38 |
lihi | Hmm ok then. that sort-of test it | 09:40 |
oanson | In theory, the application decoupling you and dimak are doing should allow isolated application tests, right? | 09:43 |
oanson | When we have that, we can verify the packet is tagged with the correct metadata in isolation. | 09:43 |
lihi | oanson, yes. | 09:43 |
oanson | So we have a plan? | 09:44 |
lihi | yes | 09:44 |
lihi | I've already removed the -1 | 09:44 |
oanson | Thanks! | 09:45 |
lihi | Could you open a bug for the future test? | 09:45 |
oanson | Working on it now :) | 09:46 |
oanson | lihi, done: https://bugs.launchpad.net/dragonflow/+bug/1751013 | 09:48 |
openstack | Launchpad bug 1751013 in DragonFlow "dnat fullstack missing test: Incoming packet is tagged" [Low,New] | 09:48 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: BGP service - take logical port info from Neutron DB https://review.openstack.org/540853 | 10:06 |
*** yamamoto has quit IRC | 10:06 | |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: BGP service - take logical port info from Neutron DB https://review.openstack.org/540853 | 10:07 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: BGP service - take logical port info from Neutron DB https://review.openstack.org/540853 | 10:10 |
*** yamamoto has joined #openstack-dragonflow | 11:06 | |
*** yamamoto has quit IRC | 11:13 | |
*** snapiri- has quit IRC | 11:47 | |
*** snapiri1 has joined #openstack-dragonflow | 11:48 | |
openstackgerrit | Shachar Snapiri proposed openstack/dragonflow master: [WIP] [DNR] SkyDive agent app https://review.openstack.org/518670 | 11:58 |
*** snapiri1 has quit IRC | 11:58 | |
*** yamamoto has joined #openstack-dragonflow | 12:09 | |
*** yamamoto has quit IRC | 12:14 | |
*** yamamoto has joined #openstack-dragonflow | 12:27 | |
oanson | snapiri, are you in? | 12:33 |
*** CrayZee has joined #openstack-dragonflow | 13:03 | |
oanson | CrayZee, hi | 13:04 |
CrayZee | yo | 13:04 |
oanson | I have a zuul question for you | 13:04 |
oanson | I'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_981031 | 13:04 |
CrayZee | hope I can answer :) | 13:04 |
CrayZee | It is what it says... | 13:05 |
oanson | Somehow 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 |
oanson | Any thoughts on how to add it other than replicating the job definition in our repo? | 13:06 |
CrayZee | I am not sure | 13:07 |
CrayZee | I believe we can create jobs that inherit from the py27/py35 and extend them | 13:07 |
CrayZee | Did you look at networking-sfc project to see what they did? | 13:09 |
CrayZee | why did you add it to tox.sh? | 13:10 |
CrayZee | or maybe change the order of the lines in tox.sh? | 13:11 |
CrayZee | as networking-sfc is dependent on neutron-dynamic-routing.... | 13:11 |
CrayZee | no, my bad | 13:12 |
oanson | Added to tox_install.sh to get it to work on my local environment | 13:15 |
oanson | I see in project-config (zuul.d/projects.yaml, line 5058) that networking-sfc was added for us | 13:16 |
oanson | CrayZee, let me try and update our project-config file, and take it from there. | 13:17 |
oanson | Thanks for your help! | 13:17 |
oanson | py27 also appears twice in the 'gate' section... | 13:19 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: BGP service - take logical port info from Neutron DB https://review.openstack.org/540853 | 13:22 |
CrayZee | oanson: https://github.com/openstack-infra/project-config/blob/master/zuul.d/projects.yaml | 13:26 |
oanson | CrayZee, https://review.openstack.org/#/c/546978/ ? | 13:26 |
CrayZee | Oh... I am a bit lsow :) | 13:26 |
CrayZee | *slow | 13:27 |
*** yamamoto has quit IRC | 13:34 | |
CrayZee | oanson: Is the title of the patch correct? | 13:37 |
oanson | I think so. Why? | 13:38 |
CrayZee | it says "add to tests" | 13:39 |
CrayZee | shouldn't it b to gates? | 13:39 |
CrayZee | just being picky :P | 13:39 |
*** CrayZee has quit IRC | 13:41 | |
*** yamamoto has joined #openstack-dragonflow | 13:44 | |
*** yamamoto has quit IRC | 13:44 | |
*** CrayZee has joined #openstack-dragonflow | 13:59 | |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: TrivialFix: '_lport_index' is used prior to global declaration https://review.openstack.org/546990 | 14:03 |
*** yamamoto has joined #openstack-dragonflow | 14:45 | |
CrayZee | oanson: why do we use legacy-install-dsvm-dragonflow-kuryr-kubernetes and not the new native zuul 3.0 one? | 14:49 |
*** yamamoto has quit IRC | 14:51 | |
*** Natanbro has quit IRC | 14:56 | |
*** mlavalle has joined #openstack-dragonflow | 15:18 | |
*** yamamoto has joined #openstack-dragonflow | 15:47 | |
*** yamamoto has quit IRC | 15:51 | |
*** CrayZee has quit IRC | 16:29 | |
*** yamamoto has joined #openstack-dragonflow | 16:48 | |
*** yamamoto has quit IRC | 16:52 | |
oanson | irenab, 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-dragonflow | 17:48 | |
*** yamamoto has quit IRC | 17:54 | |
irenab | oanson: I truct you to add follow up tests | 18:12 |
oanson | irenab, sure :) The tests were added, they are just blocked by other stuff. I'll break up the patch. | 18:12 |
irenab | +1 | 18:13 |
irenab | then maybe it will allow to pass kuryr gate | 18:13 |
*** openstackgerrit has quit IRC | 18:33 | |
*** yamamoto has joined #openstack-dragonflow | 18:50 | |
*** yamamoto has quit IRC | 18:56 | |
*** yamamoto has joined #openstack-dragonflow | 18:59 | |
*** yamamoto has quit IRC | 19:04 | |
*** yamamoto has joined #openstack-dragonflow | 19:14 | |
*** yamamoto has quit IRC | 19:19 | |
*** yamamoto has joined #openstack-dragonflow | 19:29 | |
*** yamamoto has quit IRC | 19:34 | |
*** yamamoto has joined #openstack-dragonflow | 19:44 | |
*** yamamoto has quit IRC | 19:52 | |
*** yamamoto has joined #openstack-dragonflow | 20:48 | |
*** yamamoto has quit IRC | 20:55 | |
*** yamamoto has joined #openstack-dragonflow | 21:53 | |
*** yamamoto has quit IRC | 21:57 | |
*** yamamoto has joined #openstack-dragonflow | 22:01 | |
*** yamamoto has quit IRC | 23:02 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!