*** abhiraut has joined #openstack-kuryr | 00:31 | |
*** limao_ has joined #openstack-kuryr | 00:43 | |
*** diogogmt has joined #openstack-kuryr | 00:52 | |
*** abhiraut has quit IRC | 01:00 | |
*** diogogmt has quit IRC | 01:06 | |
*** hongbin has joined #openstack-kuryr | 01:30 | |
*** hongbin has quit IRC | 01:45 | |
*** banix has joined #openstack-kuryr | 01:56 | |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Allowing ip operations in fullstack test https://review.openstack.org/329729 | 01:59 |
---|---|---|
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Allowing ip operations in fullstack test https://review.openstack.org/329729 | 02:07 |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Allowing ip operations in fullstack test https://review.openstack.org/329729 | 02:33 |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Allowing ip operations in fullstack test https://review.openstack.org/329729 | 02:41 |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Allowing ip operations in fullstack test https://review.openstack.org/329729 | 02:47 |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Allowing ip operations in fullstack test https://review.openstack.org/329729 | 02:51 |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Allowing ip operations in fullstack test https://review.openstack.org/329729 | 03:02 |
*** huikang has quit IRC | 03:22 | |
*** tfukushima has joined #openstack-kuryr | 03:28 | |
*** vikasc has joined #openstack-kuryr | 03:32 | |
vikasc | banix, ping | 03:32 |
banix | hi | 03:32 |
vikasc | on https://review.openstack.org/#/c/328236/3, | 03:33 |
vikasc | functional test case, please guide me on jenkins check failure | 03:33 |
vikasc | i am not able to even figure out the problem, solution comes later :) | 03:33 |
vikasc | banix, | 03:34 |
banix | VikasC: i am trying to fix that | 03:34 |
vikasc | banix, is it with that test patch? | 03:34 |
banix | yes | 03:35 |
banix | i said n the review that if the test fails (as is the case now) it shouldnt get merged | 03:35 |
vikasc | ok,,, just to stop this patch until you fix issue | 03:36 |
vikasc | banix, other than this, on overlapping cidr alternative approach which you suggested, I have added that it devref under "alternatives" section | 03:37 |
vikasc | banix, https://review.openstack.org/#/c/326894/ | 03:38 |
banix | VikasC: thx | 03:38 |
banix | have you tried to implement it? | 03:38 |
vikasc | banix, thats what i wanted to take your suggestion on. | 03:38 |
vikasc | SHould we wait for others opinion on devref or push a patch | 03:39 |
vikasc | banix, | 03:39 |
banix | VikasC: i think it is worth having the code up for review, | 03:39 |
banix | because the devref have a lot more and may be a while before we get to doing it | 03:40 |
vikasc | banix, ok then... soon I will be pushing code for review | 03:40 |
banix | VikasC: that would be great. thanks | 03:40 |
vikasc | banix, yes... also I am planning, as you suggested on router related bug, to push a patch in neutron for tag support on router resources | 03:40 |
banix | VikasC: awsome | 03:41 |
vikasc | banix, will take up overllaping first and then tags | 03:41 |
banix | sounds good | 03:41 |
vikasc | banix, thanks | 03:41 |
banix | thank you | 03:42 |
openstackgerrit | vikas choudhary proposed openstack/kuryr: Add meaningful default address-space names https://review.openstack.org/326933 | 03:52 |
openstackgerrit | vikas choudhary proposed openstack/kuryr: Delay neutron extension checks to first neutron interaction https://review.openstack.org/326303 | 04:04 |
openstackgerrit | vikas choudhary proposed openstack/kuryr: Add default subnetpools to configuration https://review.openstack.org/330471 | 04:16 |
*** yuanying has quit IRC | 04:41 | |
*** yamamoto has quit IRC | 04:46 | |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Allowing ip operations in fullstack test https://review.openstack.org/329729 | 04:52 |
*** salv-orl_ has joined #openstack-kuryr | 05:09 | |
*** salv-orlando has quit IRC | 05:12 | |
*** banix has quit IRC | 05:19 | |
*** yamamoto has joined #openstack-kuryr | 05:26 | |
*** yuanying has joined #openstack-kuryr | 06:02 | |
*** tfukushima has quit IRC | 06:05 | |
*** tfukushima has joined #openstack-kuryr | 06:45 | |
*** yuanying has quit IRC | 07:02 | |
*** huikang has joined #openstack-kuryr | 07:03 | |
*** salv-orl_ has quit IRC | 07:09 | |
*** salv-orlando has joined #openstack-kuryr | 07:09 | |
*** salv-orlando has quit IRC | 07:35 | |
*** huikang has quit IRC | 07:40 | |
*** huikang has joined #openstack-kuryr | 07:40 | |
*** huikang has quit IRC | 07:44 | |
vikasc | apuimedo, ping | 07:58 |
*** yamamoto has quit IRC | 08:01 | |
vikasc | gsagie, ping | 08:04 |
*** diga has joined #openstack-kuryr | 08:05 | |
*** yuanying has joined #openstack-kuryr | 08:05 | |
apuimedo | vikasc: pong | 08:14 |
vikasc | apuimedo, https://review.openstack.org/#/c/319524/ | 08:15 |
vikasc | apuimedo, this can be merged i guess | 08:15 |
apuimedo | vikasc: not without another +2 | 08:16 |
apuimedo | I already gave mine | 08:16 |
apuimedo | vikasc: also, it would be nice if you replied to Szilard's comments | 08:16 |
vikasc | apuimedo, patch is liping mao's. I will ask him | 08:17 |
vikasc | apuimedo, I was doing changes to same part of code , so this patch came to my mind | 08:17 |
apuimedo | oh, sorry | 08:17 |
apuimedo | :P | 08:17 |
apuimedo | vikasc: in any case, we need either banix, irenab, gsagie or tfukushima to approve before merging | 08:18 |
vikasc | apuimedo, thanks Toni. Will try reaching limao as well | 08:19 |
apuimedo | vikasc: you are welcome. Thanks to you both for all the patches you've been putting on lately ;-) | 08:21 |
vikasc | apuimedo, :) | 08:22 |
*** mestery_ has joined #openstack-kuryr | 08:36 | |
diga | apuimedo: Hi | 08:38 |
diga | Good Morning | 08:38 |
apuimedo | good morning | 08:38 |
diga | apuimedo: did you get a time to go through this - https://blueprints.launchpad.net/kuryr/+spec/autogen-kuryr-conf-file | 08:39 |
*** yamamoto has joined #openstack-kuryr | 08:39 | |
diga | do I need to submit spec for this ? | 08:39 |
apuimedo | diga: I see you did not notice I approved the blueprint :P | 08:39 |
diga | apuimedo: :) | 08:40 |
diga | Thank you | 08:40 |
apuimedo | diga: I do not think a spec is needed. Work can start on patching the config mechanism | 08:41 |
diga | I will go ahead, sorry its cache problem, just noticed | 08:41 |
apuimedo | ;-) | 08:41 |
diga | sure | 08:41 |
*** huats has quit IRC | 08:42 | |
*** mestery has quit IRC | 08:43 | |
*** mestery_ is now known as mestery | 08:43 | |
*** huats has joined #openstack-kuryr | 08:44 | |
*** vikasc has quit IRC | 09:07 | |
*** tfukushima has quit IRC | 09:15 | |
*** tfukushima has joined #openstack-kuryr | 09:15 | |
*** openstackgerrit has quit IRC | 09:18 | |
*** openstackgerrit has joined #openstack-kuryr | 09:18 | |
*** limao_ has quit IRC | 09:18 | |
*** vikasc has joined #openstack-kuryr | 09:20 | |
apuimedo | vikasc: thanks for the review to the new devref | 09:34 |
apuimedo | very thoughtful | 09:34 |
vikasc | apuimedo, thank you !!! | 09:34 |
*** openstackgerrit has quit IRC | 10:03 | |
*** openstackgerrit has joined #openstack-kuryr | 10:03 | |
*** tfukushima has quit IRC | 10:10 | |
openstackgerrit | vikas choudhary proposed openstack/kuryr: Short-Term fix for overlapping cidrs using docker options https://review.openstack.org/331050 | 10:59 |
*** diga has quit IRC | 11:11 | |
*** irenab has joined #openstack-kuryr | 11:42 | |
*** irenab has quit IRC | 11:47 | |
*** vikasc has quit IRC | 12:11 | |
*** limao_ has joined #openstack-kuryr | 12:42 | |
*** limao__ has joined #openstack-kuryr | 12:45 | |
*** limao_ has quit IRC | 12:49 | |
*** huikang has joined #openstack-kuryr | 13:21 | |
*** huikang has quit IRC | 13:27 | |
*** limao_ has joined #openstack-kuryr | 13:40 | |
*** limao_ has quit IRC | 13:40 | |
*** limao_ has joined #openstack-kuryr | 13:41 | |
*** limao__ has quit IRC | 13:43 | |
*** hongbin has joined #openstack-kuryr | 13:47 | |
limao_ | https://review.openstack.org/#/c/323895/12 | 13:52 |
limao_ | anyone know how to let OpenStack Proposal Bot update this to include Flask 0.11? | 13:54 |
*** huikang has joined #openstack-kuryr | 14:23 | |
*** huikang has quit IRC | 14:28 | |
*** diogogmt has joined #openstack-kuryr | 15:02 | |
*** diogogmt has quit IRC | 15:04 | |
*** huikang has joined #openstack-kuryr | 15:08 | |
*** huikang has quit IRC | 15:25 | |
*** huikang has joined #openstack-kuryr | 15:25 | |
*** huikang has quit IRC | 15:29 | |
openstackgerrit | Liping Mao proposed openstack/kuryr: Add existed network failed if the gateway already created https://review.openstack.org/319524 | 15:42 |
*** limao_ has quit IRC | 15:59 | |
*** vikasc has joined #openstack-kuryr | 16:08 | |
*** huikang has joined #openstack-kuryr | 16:26 | |
*** huikang has quit IRC | 16:32 | |
*** vikasc has quit IRC | 16:43 | |
*** diga has joined #openstack-kuryr | 17:05 | |
*** irenab has joined #openstack-kuryr | 17:06 | |
*** yamamoto has quit IRC | 17:10 | |
*** irenab has quit IRC | 17:11 | |
*** huikang has joined #openstack-kuryr | 17:14 | |
*** abhiraut has joined #openstack-kuryr | 17:29 | |
*** diga has quit IRC | 17:31 | |
*** abhiraut has quit IRC | 17:35 | |
*** irenab has joined #openstack-kuryr | 17:36 | |
*** huikang has quit IRC | 17:39 | |
*** banix has joined #openstack-kuryr | 17:39 | |
*** abhiraut has joined #openstack-kuryr | 17:39 | |
*** irenab has quit IRC | 17:41 | |
*** abhiraut has quit IRC | 17:41 | |
*** tango has joined #openstack-kuryr | 17:48 | |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Allowing ip operations in fullstack test https://review.openstack.org/329729 | 18:05 |
*** irenab has joined #openstack-kuryr | 18:09 | |
*** yamamoto has joined #openstack-kuryr | 18:10 | |
*** irenab has quit IRC | 18:13 | |
*** yamamoto has quit IRC | 18:24 | |
*** irenab has joined #openstack-kuryr | 18:29 | |
*** irenab has quit IRC | 18:34 | |
*** huikang has joined #openstack-kuryr | 18:40 | |
*** huikang has quit IRC | 18:45 | |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Allowing ip operations in fullstack test https://review.openstack.org/329729 | 19:13 |
apuimedo | banix: does it seem to work with the new fix in pyroute2? | 19:15 |
banix | apuimedo: yes it does | 19:15 |
apuimedo | :-) | 19:15 |
apuimedo | banix: did I introduce you to svinota? | 19:16 |
banix | should we update the requirement to the latest version of pyroute | 19:16 |
banix | apuimedo: no | 19:16 |
apuimedo | banix: we definitely should | 19:17 |
apuimedo | give me a second | 19:17 |
apuimedo | I'll try to get somebody to send the patch | 19:17 |
apuimedo | so we can merge it | 19:17 |
apuimedo | right now :P | 19:17 |
banix | apuimedo: ok i will clean up the patch and add the requirement part to it if you dont get to doing it | 19:17 |
apuimedo | banix: I'll do it in 10min | 19:18 |
banix | great; even if you submit yourself we can merge with one +2 I would say ;) | 19:18 |
apuimedo | it's against the rules, but we could | 19:19 |
banix | each rule should be justified…. i thought you were an anarchist | 19:19 |
banix | so looks like we will have the testing in decent shape. the problem turned out to be outside Kuryr. I suppose we will get to Kuryr issues next :) | 19:20 |
banix | 3 patches, each one line long. one in Rally, one in using Flask, one in pyroute2. wow | 19:21 |
apuimedo | limao: no idea | 19:23 |
*** huikang has joined #openstack-kuryr | 19:40 | |
*** huikang has quit IRC | 19:47 | |
openstackgerrit | Lenka Segura Puimedon proposed openstack/kuryr: requirements: upgrade to latest pyroute2 https://review.openstack.org/331320 | 19:47 |
apuimedo | banix: ^^ | 19:48 |
banix | +2’ed | 19:49 |
apuimedo | good | 19:52 |
banix | apuimedo: http://logs.openstack.org/20/331320/1/check/gate-kuryr-requirements/97c75f4/console.html | 19:59 |
apuimedo | banix: true | 20:08 |
apuimedo | now they are managed by the fscking bot | 20:08 |
apuimedo | ok, let's see if we can send it to the other repo | 20:08 |
apuimedo | banix: https://review.openstack.org/#/c/331333/ | 20:13 |
banix | +1’ed | 20:14 |
apuimedo | thanks banix | 20:15 |
apuimedo | I'm trying to get the infra people to merge | 20:15 |
banix | thank you | 20:15 |
apuimedo | banix: thank you for finding the original source of the issue | 20:15 |
apuimedo | ! | 20:15 |
banix | our tests are passing now so the latest seems to be picked up | 20:15 |
apuimedo | banix: let's merge https://review.openstack.org/#/c/323895/12 | 20:16 |
apuimedo | we can raise the openstack/requirements flask version later | 20:16 |
banix | done | 20:17 |
banix | have to run. catch you later | 20:22 |
*** irenab has joined #openstack-kuryr | 20:22 | |
*** banix has quit IRC | 20:22 | |
*** irenab has quit IRC | 20:27 | |
openstackgerrit | Merged openstack/kuryr: Updated from global requirements https://review.openstack.org/323895 | 20:31 |
*** huikang has joined #openstack-kuryr | 20:43 | |
*** banix has joined #openstack-kuryr | 20:48 | |
*** huikang has quit IRC | 20:48 | |
*** huikang has joined #openstack-kuryr | 20:53 | |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Add container connect/disconnect tests to fullstack gate job https://review.openstack.org/265105 | 20:56 |
*** huikang has quit IRC | 21:15 | |
openstackgerrit | Mohammad Banikazemi proposed openstack/kuryr: Add container connect/disconnect tests to fullstack gate job https://review.openstack.org/265105 | 21:24 |
*** irenab has joined #openstack-kuryr | 21:24 | |
*** irenab has quit IRC | 21:28 | |
*** banix has quit IRC | 22:13 | |
*** banix has joined #openstack-kuryr | 22:32 | |
*** hongbin has quit IRC | 23:27 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!