Wednesday, 2016-12-21

*** hongbin has quit IRC00:18
*** salv-orlando has joined #openstack-kuryr00:41
*** salv-orlando has quit IRC00:46
openstackgerritTony Breeds proposed openstack/kuryr-kubernetes: Add Constraints support  https://review.openstack.org/41329700:49
*** roeyc has quit IRC00:55
*** limao has joined #openstack-kuryr01:02
*** yedongcan has joined #openstack-kuryr01:19
openstackgerritCao Xuan Hoang proposed openstack/kuryr: Removes unnecessary utf-8 encoding  https://review.openstack.org/41226901:29
*** salv-orlando has joined #openstack-kuryr01:42
*** salv-orlando has quit IRC01:46
*** yamamot__ has joined #openstack-kuryr03:40
*** tonanhngo has quit IRC03:48
*** dougbtv_ has joined #openstack-kuryr03:56
*** limao has quit IRC04:02
*** limao has joined #openstack-kuryr04:03
*** vikasc has quit IRC04:07
*** vikasc has joined #openstack-kuryr04:20
*** salv-orlando has joined #openstack-kuryr04:44
*** salv-orlando has quit IRC04:49
*** tonanhngo has joined #openstack-kuryr04:59
*** tonanhngo has quit IRC05:01
openstackgerritZhang Ni proposed openstack/fuxi: Enable Fuxi to use Manila share  https://review.openstack.org/37545205:07
*** janki has joined #openstack-kuryr05:20
*** salv-orlando has joined #openstack-kuryr05:45
openstackgerritDongcan Ye proposed openstack/kuryr-libnetwork: Delete port created by kuryr  https://review.openstack.org/41277205:47
*** salv-orlando has quit IRC05:49
*** yedongcan1 has joined #openstack-kuryr06:09
*** yedongcan has quit IRC06:11
*** pmannidi_ has quit IRC06:13
*** pmannidi has joined #openstack-kuryr06:14
openstackgerritZhang Ni proposed openstack/fuxi: Enable Fuxi to use Manila share  https://review.openstack.org/37545206:17
*** irenab_ has joined #openstack-kuryr06:21
*** irenab has quit IRC06:23
*** irenab_ is now known as irenab06:23
*** tonanhngo has joined #openstack-kuryr06:35
*** tonanhngo has quit IRC06:36
*** salv-orlando has joined #openstack-kuryr06:46
*** oanson has joined #openstack-kuryr06:57
*** limao has quit IRC07:05
*** limao has joined #openstack-kuryr07:06
*** limao has quit IRC07:09
*** limao_ has joined #openstack-kuryr07:09
*** limao_ has quit IRC07:19
*** bailin has joined #openstack-kuryr07:30
openstackgerritHong Hui Xiao proposed openstack/kuryr-libnetwork: Fix the typo in comment  https://review.openstack.org/41346107:34
*** pcaruana has joined #openstack-kuryr07:34
openstackgerritDongcan Ye proposed openstack/kuryr-libnetwork: Delete port created by kuryr  https://review.openstack.org/41277207:41
*** openstackgerrit has quit IRC07:48
*** limao has joined #openstack-kuryr08:02
*** tonanhngo has joined #openstack-kuryr08:03
*** limao has quit IRC08:03
*** limao has joined #openstack-kuryr08:03
*** tonanhngo has quit IRC08:04
*** salv-orl_ has joined #openstack-kuryr08:04
*** salv-orlando has quit IRC08:07
*** dimak has joined #openstack-kuryr08:23
*** pmannidi has quit IRC08:27
*** dimak has quit IRC08:39
*** dimak has joined #openstack-kuryr08:41
*** dimak has quit IRC08:41
*** dimak has joined #openstack-kuryr08:42
*** limao has quit IRC08:43
*** salv-orl_ has quit IRC08:44
*** salv-orlando has joined #openstack-kuryr08:44
*** limao has joined #openstack-kuryr08:45
*** dimak has quit IRC08:45
*** dimak has joined #openstack-kuryr08:46
*** dimak has quit IRC08:46
*** limao_ has joined #openstack-kuryr08:48
*** limao has quit IRC08:49
*** limao_ has quit IRC08:50
*** garyloug has joined #openstack-kuryr08:59
*** oanson_ has joined #openstack-kuryr09:04
*** oanson has quit IRC09:05
*** oanson_ is now known as oanson09:05
*** garyloug has quit IRC09:23
*** garyloug has joined #openstack-kuryr09:23
*** limao has joined #openstack-kuryr09:26
*** roeyc has joined #openstack-kuryr09:34
apuimedovikasc: ivc_: irenab: https://review.openstack.org/#/c/413247/09:45
*** bailin has quit IRC09:48
apuimedomchiappero: https://review.openstack.org/#/c/412772/09:49
irenabapuimedo: checking09:50
apuimedothanks irenab09:50
*** jchhatbar has joined #openstack-kuryr09:50
*** janki has quit IRC09:52
*** limao has quit IRC09:59
*** limao_ has joined #openstack-kuryr09:59
apuimedoirenab: :-)10:03
apuimedoplease, review https://review.openstack.org/#/c/412772/5/kuryr_libnetwork/controllers.py also10:04
apuimedoit's for the fix when releasing IPs of pre-existing networks10:04
*** openstackgerrit has joined #openstack-kuryr10:16
openstackgerritLiping Mao proposed openstack/kuryr-libnetwork: Check docker subnet is same with neutron subnet when use existed network  https://review.openstack.org/41354210:16
apuimedolimao_: yedongcan1: Do I read that code wrong? It looks to me like we are checking all kuryr ports to see if we have a matching IP address (of subnets with the same cidr) and then delete all those ports10:21
apuimedowon't that give us false positives?10:21
apuimedolike for example, let's say that we reused to different Neutron networks10:23
apuimedoand both have a subnet with range 192.168.11.0/2410:24
apuimedoand a port on each of those subnets 192.168.11.4210:24
apuimedoI think this patch would unintentionally delete the port of both networks10:25
limao_apuimedo: do we support kuryr to use two different neutron network with same subnet now?10:28
apuimedoiirc, not with kuryr created networks, but we should with reused neus10:29
apuimedo*nets10:29
limao_# neutron net-create 1221_net010:30
limao_# docker network create -d kuryr --ipam-driver=kuryr --subnet=12.21.0.0/24 --gateway=12.21.0.1 -o neutron.net.name=1221_net0 1221_docker_net010:30
limao_# neutron net-create 1221_net110:30
limao_# docker network create -d kuryr --ipam-driver=kuryr --subnet=12.21.0.0/24 --gateway=12.21.0.1 -o neutron.net.name=1221_net1 1221_docker_net110:30
limao_Error response from daemon: IpamDriver.RequestPool: Another pool with same cidr exist. ipam and network options not used to pass pool name10:30
apuimedolimao_: what I meant is if there was a neutron cli created matching subnet10:31
apuimedoso on net0 you'd create 12.21.0.0/24 with kuryr10:31
apuimedoand you'd have in net1 12.22.0.0/24 with kuryr, but 12.21.0.0/24 created with neutron for VMs10:31
limao_all_ports = app.neutron.list_ports(device_owner=lib_const.DEVICE_OWNER)10:32
apuimedotrue10:32
apuimedo:P10:32
limao_the device owner would not be kuryr10:32
apuimedoGood :-)10:32
apuimedolimao_: ok, so the only thing missing in the patch, is to have a fullstack test that tests that the ports are deleted, so we don't regress10:33
limao_It looks not good, but... we do not have better way in currently our mind10:33
apuimedolimao_: I agree10:34
limao_apuimedo: sure, will add fullstack test to cover10:34
apuimedoI was thinking about annotating the network with the poolid10:34
apuimedowhen requesting the addresses10:34
apuimedos/annotating/tagging/10:34
limao_apuimedo: please help to add comments there, have too go now, sorry10:36
*** limao_ has quit IRC10:36
yedongcan1apuimedo, limao_ : Just have dinner, now come back.10:38
*** oanson has quit IRC10:38
yedongcan1will add the fullstack test.10:39
apuimedothanks yedongcan1!10:40
yedongcan1apuimedo: you're welcome:-D10:43
*** focusorange has joined #openstack-kuryr10:43
*** oanson has joined #openstack-kuryr10:48
*** yedongcan1 has left #openstack-kuryr10:52
*** jchhatbar is now known as janki10:58
apuimedovikasc: ping11:01
*** irenab_ has joined #openstack-kuryr11:01
vikascapuimedo, pong11:01
vikasci will try our devstack patch11:01
apuimedovikasc: please do11:01
apuimedo:-)11:01
vikasc:)11:02
vikascok11:02
vikasc10 -15 mins pls11:02
apuimedothanks11:02
*** irenab has quit IRC11:21
*** irenab_ is now known as irenab11:21
*** irenab_ has joined #openstack-kuryr11:23
*** irenab has quit IRC11:26
*** irenab has joined #openstack-kuryr11:26
*** irenab__ has joined #openstack-kuryr11:27
openstackgerritMerged openstack/kuryr-kubernetes: devstack: Have controller wait for K8s API  https://review.openstack.org/41324711:28
*** irenab_ has quit IRC11:28
*** garyloug_ has joined #openstack-kuryr11:49
*** garyloug has quit IRC11:53
*** tonanhngo has joined #openstack-kuryr12:05
*** tonanhngo has quit IRC12:05
*** focusorange has left #openstack-kuryr12:16
*** dougbtv_ has quit IRC12:36
openstackgerritBerezovsky Irena proposed openstack/kuryr-kubernetes: Add DragonFlow Kuryr-Kubernetes local.conf sample  https://review.openstack.org/41360012:38
*** pcaruana has quit IRC12:49
apuimedoirenab: mind if I sort that local.conf differently, so it is more similar to the local.conf.sample?12:55
apuimedo(nice diff)12:56
apuimedo*nicer diff12:56
apuimedoirenab: why did you put 'disable_service n-net'12:57
apuimedoit already gets disabled when you do ENABLED_SERVICES="", doesn't it?12:58
*** salv-orlando has quit IRC13:05
*** garyloug_ has quit IRC13:13
*** pcaruana has joined #openstack-kuryr13:32
*** limao has joined #openstack-kuryr13:43
*** salv-orlando has joined #openstack-kuryr13:46
*** limao_ has joined #openstack-kuryr13:47
*** limao has quit IRC13:48
ivc_apuimedo irenab iirc 'disable_service n-net' is not needed anymore (since newton i think) as neutron is the default for devstack13:49
*** limao_ has quit IRC14:00
*** limao has joined #openstack-kuryr14:01
*** salv-orl_ has joined #openstack-kuryr14:04
*** limao has quit IRC14:05
*** limao has joined #openstack-kuryr14:07
*** salv-orlando has quit IRC14:07
irenab__apuimedo: will remove it, previously had some issues, but probably not relate to it14:07
apuimedoirenab__: please, check this http://paste.openstack.org/show/593020/14:09
apuimedoI reordered it and dropped some disables14:09
apuimedoit makes for a nicer diff14:10
irenab__apuimedo: I think ‘enable_service neutron14:10
irenab__‘ imposed some issues with DF conf file location14:10
apuimedooh, so then drop it14:10
irenab__why do we need enable neutron?14:10
apuimedoIIRC it uses newer devstack plugin in openstack/neutron14:11
apuimedorather than the builting devstack14:11
apuimedobut I may be wrong14:11
irenab__apuimedo: this may explain the issue14:11
irenab__I will try to run without it and check if it deploys properly14:12
apuimedovery well14:12
irenab__ivc_: locally, I have some issues with few unit tests: http://paste.openstack.org/show/593023/14:14
irenab__ivc_:  not related to the change I do14:14
apuimedoirenab__: did you try removing .tox ?14:16
ivc_irenab__ are you running those on linux or mac/win?14:16
irenab__ivc_: mac14:17
irenab__ivc_: but I wonder why unit tests areaffected14:17
ivc_irenab__ it's pyroute2, it's not available on mac i think14:17
ivc_irenab__ but we import it14:17
irenab__ivc_: ok then, will challenge the gate14:18
openstackgerritBerezovsky Irena proposed openstack/kuryr-kubernetes: Add support for native OVS binding  https://review.openstack.org/41221514:19
ivc_irenab__ it'd be nice if we could allow our ut to run anywhere (with mock). but it seems the problem happens during import in "kuryr/lib/binding/drivers/utils.py"14:22
apuimedoivc_: agreed14:22
irenab__ivc_: I agree, will fill a bug on this. I think technically its possible to fix14:23
ivc_irenab__ bug for kuryr.lib?14:24
apuimedoyup14:24
irenab__yea14:24
*** limao has quit IRC14:24
*** salv-orl_ has quit IRC14:24
irenab__we have all kuryr projects on same launchpad, right?14:24
apuimedonope14:25
*** limao has joined #openstack-kuryr14:25
*** pcaruana has quit IRC14:38
*** limao has quit IRC14:43
openstackgerritBerezovsky Irena proposed openstack/kuryr-kubernetes: Add DragonFlow Kuryr-Kubernetes local.conf sample  https://review.openstack.org/41360014:45
irenab__apuimedo: looks like ‘nicer’ confing worked for me, please verify as well14:46
*** irenab__ has quit IRC14:48
*** pcaruana has joined #openstack-kuryr14:51
apuimedoivc_: without python, the images with and without python14:56
apuimedogliderlabs/alpine                     3.4                 ec21b98e6f3f        2 seconds ago       4.799 MB14:56
apuimedokuryr/alpine                          3.4                 e54de033e15d        38 seconds ago      43.45 MB14:56
apuimedothe rootfs that we commit to the repo goes from 5MiB to 15MiB14:57
apuimedoit's not too bad14:57
ivc_apuimedo i think we can even skip alpine if we use static go14:57
apuimedoivc_: well, using Go we could use busybox most likely14:58
apuimedolet me check14:58
ivc_apuimedo ah right, we still need ssh/curl14:58
apuimedoyu[15:00
apuimedoyup15:00
apuimedoI was using alpine in case we want to add things like perf later15:00
ivc_apuimedo remind me why do you want to keep images in-tree?15:00
apuimedoor use python for packet generation15:00
apuimedonot to have false negatives from pulling images15:01
apuimedoas we had in kuryr-libnetwork15:01
ivc_hmm. you mean in case dockerhub is down?15:02
apuimedoor just inaccessible from ci15:02
apuimedoI like to be able to run tests without internet in general15:02
ivc_i wonder if kolla relies on dockerhub too15:02
apuimedonope15:02
apuimedothey build15:02
apuimedoiirc15:02
*** tonanhngo has joined #openstack-kuryr15:03
ivc_apuimedo for no-internet ci, how does it deal with pip and github-based remos in requirements.txt?15:04
ivc_*repos15:04
*** tonanhngo has quit IRC15:04
* ivc_ is not a big fan of keeping tarballs in git15:05
apuimedogithub we should drop from local.conf, as it could use openstack authoritative mirrors15:05
apuimedopip is a problem with devstack, but I suspect they have a mirror in CI15:05
ivc_aye i've used git://git.openstack.org instead of https://github.com for local.conf in demo15:06
ivc_apuimedo wdyt if we make a new kuryr-artifacts repo maybe? to keep things like docker tarballs there15:08
apuimedoivc_: in the long term maybe15:09
apuimedonot in the short one15:09
ivc_apuimedo adding those to main code repos only cripple git clone15:09
apuimedoI don't want syncing issues15:09
*** garyloug_ has joined #openstack-kuryr15:09
ivc_apuimedo i just don't like that we'd have to pull those blobs when we don't need them15:10
ivc_apuimedo and if you want to get rid of them later you'd have to modify git history15:10
apuimedoI know15:10
apuimedoivc_: let me ask infra if they already have a registry15:11
*** irenab_ has joined #openstack-kuryr15:11
ivc_apuimedo cool. that would be the best solution imo15:12
*** hongbin has joined #openstack-kuryr15:14
*** yamamot__ has quit IRC15:23
*** salv-orlando has joined #openstack-kuryr15:25
*** janki has quit IRC15:28
*** salv-orlando has quit IRC15:30
*** yamamoto has joined #openstack-kuryr15:41
apuimedoivc_: ok. I was talking to kolla folks15:43
apuimedothe approach is15:43
apuimedowe have a gate that builds those containers15:43
apuimedoand pushes the images to tarballs.openstack.org15:43
*** openstackgerrit_ has joined #openstack-kuryr15:47
*** openstackgerrit_ has quit IRC15:49
*** pcaruana has quit IRC16:13
*** salv-orlando has joined #openstack-kuryr16:26
*** salv-orlando has quit IRC16:31
*** yamamoto has quit IRC16:41
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes: gating: devstackrc for CI  https://review.openstack.org/41371516:43
*** diogogmt has joined #openstack-kuryr16:43
apuimedoivc_: irenab: vikasc: please, review ^^16:46
apuimedoSo that we can have install gate asap16:47
*** openstack has quit IRC17:02
*** openstack has joined #openstack-kuryr17:03
*** neiljerram has quit IRC17:12
*** yamamoto has joined #openstack-kuryr17:12
*** yamamoto has quit IRC17:21
*** garyloug_ has quit IRC17:53
hongbinapuimedo: hey, a question for you, does kuryr have plan to solve the multi-tenancy problem (right now, it seems the libnetwork and k8s hard-code a tenant in the config file)18:06
*** salv-orlando has joined #openstack-kuryr18:27
*** salv-orlando has quit IRC18:32
*** pcaruana has joined #openstack-kuryr18:34
*** roeyc has quit IRC18:49
*** neiljerram has joined #openstack-kuryr18:53
apuimedohongbin: for libnetwork, since docker decided not to tackle it, we decided to drop plans, for k8s we are planning19:07
hongbinapuimedo: i see, what is the k8s plan?19:08
*** neiljerram has quit IRC19:13
apuimedohongbin: I've got to check the 1.5 multitenancy still19:55
apuimedoI'll probably look at it over the holidays19:55
hongbinapuimedo: ok, we can discuss the details later. i am thinking the multi-tenancy on fuxi, just want to know how kuryr is going to do it so fuxi can follow19:56
*** yamamoto has joined #openstack-kuryr19:59
hongbinapuimedo: for docker, i am thinking if it is possible to let users to pre-create network/volume on their tenant, then pass the network/volume id/name to kuryr-libnetwork/fuxi20:01
*** yamamoto has quit IRC20:04
apuimedohongbin: for volumes I'm not sure20:07
apuimedofor networks it is not possible20:07
apuimedobecause on docker run you can't pass auth info20:07
*** pcaruana has quit IRC20:11
hongbinok, get it20:13
*** salv-orlando has joined #openstack-kuryr20:15
irenab_apuimedo: vikasc ivc_ : please take a loom once you have time https://review.openstack.org/#/c/412215/20:15
*** salv-orlando has quit IRC20:19
ivc_irenab_ linux_net_utils is just a copy from nova, right?20:25
irenab_ivc_: not exactly 1:1, but mostly20:25
irenab_I think I added it in the header of this file20:25
ivc_irenab_ yup20:26
ivc_irenab_ the code looks good. but can't test it now20:27
ivc_irenab_ you tested it with both df and ovs-native, right?20:28
ivc_irenab_ can you link your local.conf in comments?20:30
irenab_ivc_: yes, for checking on both.20:32
irenab_ivc_: for DF, there is local.conf in https://review.openstack.org/#/c/413600/120:33
irenab_ivc_:  for ovs_native, I actually just hacked the conf settings to make this condition to https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/mech_driver/mech_openvswitch.py#L51 to be false. Will check what option to set in devstack20:34
ivc_irenab_ ok. i'll also ask my team about devstack with ovs-native. i'll +1 as soon as i can verify it on my env.20:40
irenab_ivc_: thanks!20:40
ivc_irenab_ thanks to you :)20:41
openstackgerritAntoni Segura Puimedon proposed openstack/kuryr-kubernetes: gating: devstackrc for CI  https://review.openstack.org/41371520:49
*** mattmceuen_away is now known as mattmceuen21:01
*** yamamoto has joined #openstack-kuryr21:04
*** irenab__ has joined #openstack-kuryr21:11
*** irenab_ has quit IRC21:13
*** yamamoto has quit IRC21:13
mattmceuenHey apuimedo, I'd tried running kuryr-libnetwork (veth) from the docker hub container a couple days ago, and ran into an error.  Now I'm running nested (macvlan) from source, and am hitting the same error!21:21
mattmceuenHandler for POST /v1.24/networks/create returned error: failed to allocate gateway (): No address returned21:21
mattmceuenI can dig into the code and try to see what would cause that, but wanted to make sure it didn't ring any bells for you first.21:21
*** pmannidi has joined #openstack-kuryr21:59
*** salv-orlando has joined #openstack-kuryr22:15
*** salv-orl_ has joined #openstack-kuryr22:19
*** salv-orlando has quit IRC22:23
*** diogogmt has quit IRC22:44
*** lezbar__ has quit IRC22:44
apuimedomattmceuen: can you try veth with source?22:52
apuimedoIIRC correctly we are missing one patch for kuryr-libnetwork macvlan container-in-vm binding22:52
mattmceuengotcha - sure, I will try veth from source, non-nested.  Thanks!22:53
*** saneax-_-|AFK is now known as saneax23:09
*** salv-orl_ has quit IRC23:21
*** mattmceuen is now known as mattmceuen_away23:22
*** roeyc has joined #openstack-kuryr23:36
* janonymous apuimedo: https://bugs.launchpad.net/kolla/+bug/163470023:37
openstackLaunchpad bug 1634700 in kolla "kuryr deploy fails" [High,Confirmed]23:37
apuimedojanonymous: could you try the deployment?23:37
janonymousapuimedo: kuryr-kubernetes?23:38
apuimedojanonymous: the kolla ansible you linked23:38
janonymousapuimedo: ahh, ya i can i ask my colleague working in kolla23:39
janonymousapuimedo: but yeah devstack one i will try on my env23:40
janonymousapuimedo: one question, why is init/kuryr.conf ?23:43
* apuimedo checking23:48
apuimedojanonymous: that was for upstart23:49
apuimedoit can be sent to hell already23:49
*** yuanying has quit IRC23:50
apuimedoI'm going to check the kolla-ansible code23:51
janonymous:P23:51
janonymousapuimedo:https://github.com/openstack/kolla/blob/b4e3ab5e63c502ef71de2702a5531850cc41e3c7/docker/base/set_configs.py#L26823:53
apuimedo?23:53
janonymousbug report generated error : http://paste.openstack.org/show/586295/23:54
apuimedojanonymous: I don't know why it doesn't find it...23:57
apuimedopip install . definitely creates it23:57
apuimedokuryr-server, I mean23:58
janonymousyes i checked on my local env, it installs kuryr-server23:58
janonymousbut my only doubt was the place from where kuryr-server was called was not absolute..23:58

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