Tuesday, 2016-12-13

openstackgerritHongbin Lu proposed openstack/fuxi: Revert "Move from "tenant" to "project""  https://review.openstack.org/40998200:10
*** hongbin has quit IRC00:14
*** dougbtv has joined #openstack-kuryr00:15
*** diogogmt has joined #openstack-kuryr01:01
*** limao has joined #openstack-kuryr01:06
*** yuanying has quit IRC01:11
*** yuanying has joined #openstack-kuryr01:11
*** tonanhngo_ has quit IRC01:36
*** tonanhngo has joined #openstack-kuryr01:38
*** tonanhngo has quit IRC01:42
*** tonanhngo has joined #openstack-kuryr01:42
*** tonanhngo has quit IRC01:47
*** shashank_hegde has quit IRC01:58
*** limao has quit IRC02:00
*** limao has joined #openstack-kuryr02:00
*** hongbin has joined #openstack-kuryr02:04
*** yuanying has quit IRC02:11
*** yedongcan has joined #openstack-kuryr02:21
*** hongbin has quit IRC02:46
*** hongbin has joined #openstack-kuryr02:46
openstackgerritJaivish Kothari(janonymous) proposed openstack/kuryr-libnetwork: Unittest with mock  https://review.openstack.org/40520303:05
*** pmannidi_ has joined #openstack-kuryr03:18
*** pmannidi has quit IRC03:19
*** shashank_hegde has joined #openstack-kuryr03:54
*** yedongcan has quit IRC03:56
*** yedongcan has joined #openstack-kuryr03:57
*** diogogmt has quit IRC04:00
*** yedongcan has quit IRC04:05
*** hongbin has quit IRC04:06
*** yuanying has joined #openstack-kuryr04:33
*** yedongcan has joined #openstack-kuryr04:57
*** shashank_hegde has quit IRC05:06
*** reedip has quit IRC05:07
*** reedip has joined #openstack-kuryr05:20
*** yedongcan has quit IRC05:25
*** reedip has quit IRC05:28
*** shashank_hegde has joined #openstack-kuryr05:32
*** janki has joined #openstack-kuryr05:34
*** irenab_ has joined #openstack-kuryr05:38
*** reedip has joined #openstack-kuryr05:41
*** reedip has quit IRC05:50
*** yedongcan has joined #openstack-kuryr05:55
*** shashank_hegde has quit IRC05:55
*** yedongcan1 has joined #openstack-kuryr06:01
*** yedongcan has quit IRC06:01
*** reedip has joined #openstack-kuryr06:02
openstackgerritMerged openstack/kuryr-libnetwork: Fix the file permissions of README.rst  https://review.openstack.org/39651906:03
openstackgerritMerged openstack/kuryr-libnetwork: fix tox -e genconfig  https://review.openstack.org/40850106:04
*** yuanying has quit IRC06:12
*** yuanying has joined #openstack-kuryr06:12
*** yuanying has quit IRC06:12
*** yuanying has joined #openstack-kuryr06:13
*** yuanying has quit IRC06:17
*** vikasc has quit IRC06:25
*** shashank_hegde has joined #openstack-kuryr06:25
*** janki has quit IRC06:33
*** janki has joined #openstack-kuryr06:33
*** shashank_hegde has quit IRC06:35
*** vikasc has joined #openstack-kuryr06:37
*** janki has quit IRC06:38
*** roeyc has joined #openstack-kuryr06:39
*** janki has joined #openstack-kuryr06:54
*** saneax-_-|AFK is now known as saneax07:02
*** shashank_hegde has joined #openstack-kuryr07:20
*** vikasc has quit IRC07:34
openstackgerritBerezovsky Irena proposed openstack/kuryr-kubernetes: kuryr kubernetes design reference document  https://review.openstack.org/40807807:45
*** yuanying has joined #openstack-kuryr07:51
*** vikasc has joined #openstack-kuryr07:52
*** yamamoto has quit IRC07:55
*** roeyc has quit IRC08:06
apuimedoirenab: thanks for pushing the update08:07
*** pcaruana has joined #openstack-kuryr08:10
*** oanson has joined #openstack-kuryr08:11
*** shashank_hegde has quit IRC08:14
*** yedongcan1 has quit IRC08:22
*** dimak has joined #openstack-kuryr08:23
*** yedongcan has joined #openstack-kuryr08:23
*** yamamoto has joined #openstack-kuryr08:32
*** yamamoto_ has joined #openstack-kuryr08:33
*** yamamoto has quit IRC08:37
*** wanghua has joined #openstack-kuryr08:49
*** roeyc has joined #openstack-kuryr08:58
vikascivc_, ping09:11
vikascapuimedo, ping09:12
apuimedopong09:13
vikascapuimedo, when i am trying to launch a pod with current kuryr-k8s, binding is failing09:15
vikascmechanism_drivers is both ovs and linuxbridge in ml2_conf.ini09:16
apuimedovikasc: with which patch?09:17
vikascapuimedo, kuryr_kubernetes.controller.handlers.vif [-] Got VIF from annotation: VIFBridge(active=False,address=fa:16:3e:86:65:0b,bridge_name='qbr23747a20-1c'09:17
vikascmaster09:17
vikascno patch09:17
vikasccontroller never able to see port as active09:17
vikascapuimedo, on CNI logs:09:18
vikascFile "/usr/lib/python2.7/site-packages/kuryr_kubernetes/cni/api.py", line 84, in run09:18
vikasc    vif = self._plugin.add(params)09:18
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/cni/main.py", line 39, in add09:18
vikasc    self._watcher.start()09:18
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/watcher.py", line 111, in start09:18
vikasc    self._start_watch(path)09:18
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/watcher.py", line 129, in _start_watch09:18
vikasc    self._watch(path)09:18
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/watcher.py", line 141, in _watch09:18
vikasc    self._handler(event)09:18
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/handlers/dispatch.py", line 110, in __call__09:18
apuimedowas the os-vif plug called by CNI?09:19
vikasc    self._handler(event)09:19
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/handlers/dispatch.py", line 64, in __call__09:19
vikasc    handler(event)09:19
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/handlers/k8s_base.py", line 63, in __call__09:19
vikasc    self.on_present(obj)09:19
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/cni/handlers.py", line 44, in on_present09:19
vikasc    self.on_vif(pod, vif)09:19
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/cni/handlers.py", line 78, in on_vif09:19
vikasc    self._cni.CNI_IFNAME, self._cni.CNI_NETNS)09:19
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/cni/binding/base.py", line 65, in connect09:19
vikasc    driver.connect(vif, ifname, netns)09:19
vikasc  File "/usr/lib/python2.7/site-packages/kuryr_kubernetes/cni/binding/bridge.py", line 37, in connect09:19
vikasc    h_iface.net_ns_pid = os.getpid()09:19
vikasc  File "/usr/lib/python2.7/site-packages/pyroute2/ipdb/transactional.py", line 194, in __exit__09:19
vikasc    self.commit()09:19
vikasc  File "/usr/lib/python2.7/site-packages/pyroute2/ipdb/interface.py", line 905, in commit09:19
vikasc    raise error09:19
vikasc..09:19
vikasc..09:19
vikascbinding failing09:19
* vikasc checking09:19
apuimedothanks09:21
vikascthanks apuimedo, will get back with finding09:21
apuimedo:-)09:22
*** lezbar has quit IRC09:23
*** lezbar has joined #openstack-kuryr09:23
vikascapuimedo, os-vif plug is called, it is failing on "connect" which after os-vif.plug09:26
apuimedodo you have the connect stacktrace on paste.openstack.org?09:27
vikaschttps://github.com/openstack/kuryr-kubernetes/blob/master/kuryr_kubernetes/cni/binding/base.py#L6509:27
vikascapuimedo, http://paste.openstack.org/show/592183/09:29
apuimedovikasc: maybe you can hook winpdb with the remote debugger and step through to see what makes ipdb fail09:32
vikascapuimedo, yeah i can do that. wondering if i am missing something too obvious and could save the efoort :)09:32
vikascs/effort09:32
apuimedocan't see it from the stacktrace09:33
vikascwill wait for ivc_ and meanwhile will investigate further09:33
vikascthanks for the time apuimedo09:33
apuimedovikasc: which pyroute2 version are you running?09:34
*** reedip has quit IRC09:34
vikascapuimedo, pyroute2==0.4.1009:35
*** limao_ has joined #openstack-kuryr09:39
apuimedovikasc: maybe you can try with 0.4.1109:40
*** limao has quit IRC09:41
vikascapuimedo, sure09:41
*** limao_ has quit IRC09:45
*** limao has joined #openstack-kuryr09:45
*** reedip has joined #openstack-kuryr09:46
*** yamamoto_ has quit IRC09:46
*** yuanying has quit IRC09:47
*** yamamoto has joined #openstack-kuryr09:47
*** yedongcan has left #openstack-kuryr09:49
irenab_apuimedo, ping09:49
apuimedoirenab_: pong09:50
irenab_apuimedo, regardng the call tomorrow, maybe worth you host it on bluejeans, so it can be recorded09:51
vikasc+1 for recording09:52
apuimedoirenab_: good idea09:52
apuimedoI'll send an email with my meeting link09:52
*** lmdaly has joined #openstack-kuryr09:53
irenab_apuimedo, thanks09:54
irenab_apuimedo, vikasc I ran yesterday tyhe last WIP ivc_ patch and it worked for me (Ubuntu16.04)09:54
*** reedip has quit IRC09:55
vikascirenab, yeah, i read logs.. so i am expecting some configuration issue my side09:55
irenab_vikasc, I used the local.conf that ivc_ posted on that patch09:56
apuimedoirenab_: I still have neutron devstack issues...09:56
irenab_apuimedo, did you use the local.conf as ivc_ posted?09:57
irenab_http://paste.openstack.org/show/592053/09:57
apuimedoyes09:57
vikascirenab, primafacia problem does not seem on neutron side for me. Binding is failing, will check with remote pdb and update you guys09:58
apuimedoI diffed it with mine and I didn't see any significant different09:58
apuimedo*difference09:58
irenab_vikasc, waht distro you use?09:58
vikascirenab, fedora 2409:58
apuimedoirenab_: https://bluejeans.com/5508709975/10:02
apuimedoirenab_: I just updated the mailing list with the link10:04
irenab_great10:05
apuimedoirenab_: did you check the etherpad?10:07
*** reedip has joined #openstack-kuryr10:08
irenab_yes, need to add more10:08
apuimedogreat10:11
vikascapuimedo, irenab , worked with pyroute2 0.4.1110:13
*** reedip has quit IRC10:14
*** yamamoto has quit IRC10:14
*** yamamoto has joined #openstack-kuryr10:15
*** reedip has joined #openstack-kuryr10:25
apuimedovikasc: got it10:26
*** yamamoto has quit IRC10:26
apuimedovikasc: it is already in upper-constraints.txt, so new devstack deployments should get it automatically10:26
apuimedoI wonder how you got 0.4.1010:27
*** limao has quit IRC10:27
*** yamamoto has joined #openstack-kuryr10:27
*** reedip has quit IRC10:31
*** dimak has quit IRC10:32
vikascapuimedo, i have very old devstack :)10:32
apuimedoah10:33
vikascapuimedo, i just checkout patches on kuryr-libnetwork and restack10:33
vikascsorry kuryr-k8s10:33
vikascwith reclone=no and offline=yes :)10:34
apuimedoaha10:36
*** yamamoto has quit IRC10:41
*** reedip has joined #openstack-kuryr10:42
*** dimak has joined #openstack-kuryr10:45
*** yamamoto has joined #openstack-kuryr10:46
*** reedip has quit IRC11:06
*** yamamoto has quit IRC11:08
*** reedip has joined #openstack-kuryr11:18
*** yamamoto has joined #openstack-kuryr11:21
*** yamamoto has quit IRC11:22
*** reedip has quit IRC11:24
*** yamamoto has joined #openstack-kuryr11:26
openstackgerritMerged openstack/kuryr-kubernetes: devstack: LBaaSv2 and K8s service subnet  https://review.openstack.org/40724411:28
ivc_apuimedo, i've restacked from scratch several times on both 14.04 and 16.04 and it's working for me11:29
*** dimak has quit IRC11:37
apuimedoivc_: I also restacked from scratch 10 minutes ago and it worked11:38
apuimedoI really wonder what got borked in the other f24 install11:38
apuimedocause I removed /var/lib/docker, /etc/neutron, recloned, etc11:38
ivc_apuimedo also note that i've updated to ps12 - there was one line that had to be uncommented. without it you might see multiple lbaas pools for the same lb and it will fail on removing lb11:41
ivc_apuimedo its due to lbaasv2 bug/feature where they raise on 'create' but still 'create' in background11:42
mchiapperohi folks11:43
mchiapperoI'm struggling with my lack of python/mock knowledge11:43
mchiapperohow can I mock "mac_address = ip.interfaces.get(ifname, {}).get('address', None)"?11:44
mchiapperoI tried a few approaches without success11:44
*** dimak has joined #openstack-kuryr11:50
apuimedoivc_: I was only verifying the devstack patch11:52
apuimedonot the services one11:52
apuimedomchiappero: ip as in ipdb?11:53
apuimedobrb11:53
apuimedogotta pick son from school11:53
ivc_mchiappero you mock (m1) "ip.interfaces.get" and set its ".return_value" to another mock (m2) then you have access to the second 'get' through m2.get12:02
mchiapperoyes, but I can't get the first one to work :(12:16
mchiapperomaybe I can copy and paste the code somewhere12:16
mchiapperobut12:16
mchiapperodo you mean by MagicMock or simply patching "ip.interfaces.get"?12:17
openstackgerritBerezovsky Irena proposed openstack/kuryr-kubernetes: kuryr kubernetes design reference document  https://review.openstack.org/40807812:18
openstackgerritMarco Chiappero proposed openstack/kuryr-libnetwork: [WIP] Driver based model for kuryr-libnetwork  https://review.openstack.org/40036512:20
ivc_mchiappero what is 'ip' in your case is it IPDB?12:25
ivc_mchiappero yeah, post the code somewhere, that might help12:25
mchiapperohttp://paste.openstack.org/show/592212/12:28
mchiapperothe test is for L31 in https://review.openstack.org/#/c/400365/22/kuryr_libnetwork/port_driver/base.py12:29
*** yamamoto has quit IRC12:33
*** lmdaly has quit IRC12:35
mchiapperowell, L45 to be more accurate12:38
*** yamamoto has joined #openstack-kuryr12:54
*** lmdaly has joined #openstack-kuryr13:01
openstackgerritMerged openstack/fuxi: Add test hooks for Fuxi gate job  https://review.openstack.org/40832913:04
apuimedovikasc: please, review https://review.openstack.org/#/c/408078/9 :P13:04
vikascapuimedo, had already read gone through it for my understanding :)13:06
apuimedovikasc: great. Now on to the formal review :-)13:07
vikascapuimedo, LGTM13:07
apuimedovikasc: but feel free to withold your vote until after the syncup meeting13:07
vikascapuimedo, done13:07
vikascapuimedo, we acn always reiterate13:07
vikascs/can13:07
vikascapuimedo, i feel it good enough as the base13:08
irenab_apuimedo, vikasc : I will fix the nit comment and upload new version in few mins13:09
vikascack13:09
apuimedovikasc: agreed13:10
*** yamamoto has quit IRC13:10
*** yamamoto has joined #openstack-kuryr13:11
openstackgerritBerezovsky Irena proposed openstack/kuryr-kubernetes: kuryr kubernetes design reference document  https://review.openstack.org/40807813:13
apuimedothanks irenab_13:14
apuimedovikasc: ^^13:14
*** yamamoto has quit IRC13:16
vikascapuimedo, done13:19
*** janki has quit IRC13:32
ivc_mchiappero i'd suggest that you mock 'utils.get_ipdb()' - then you get access to 'ip' as a mock and can verify first 'get', make it return another mock13:33
ivc_mchiappero by "mock 'utils.get_ipdb()'" i mean @mock.patch('...utils.get_ipdb') decorator13:33
mchiapperois getting even worse :D13:36
mchiapperoivc_: thank you anyway, I'll try to make some progress13:37
ivc_mchiappero there are few things you need to be aware of about 'mock' that it 'MagicMock's anything you touch and you need to be careful about it :)13:38
*** limao has joined #openstack-kuryr13:42
*** limao_ has joined #openstack-kuryr13:44
*** gsagie has joined #openstack-kuryr13:45
*** limao has quit IRC13:46
ivc_mchiappero http://paste.openstack.org/show/592221/ here's a draft for your unit test13:51
*** yamamoto has joined #openstack-kuryr14:14
*** yamamoto has quit IRC14:26
*** saneax is now known as saneax-_-|AFK14:29
*** irenab_ has quit IRC14:35
*** tonanhngo has joined #openstack-kuryr14:55
*** wanghua has quit IRC14:56
*** gsagie has quit IRC15:03
*** garyloug has joined #openstack-kuryr15:35
*** dimak has quit IRC15:39
*** saneax-_-|AFK is now known as saneax15:48
*** hongbin has joined #openstack-kuryr15:53
*** pcaruana has quit IRC16:09
*** roeyc has quit IRC16:16
*** limao_ has quit IRC16:21
*** limao has joined #openstack-kuryr16:21
mchiapperoivc_: thank you so much for the draft16:35
*** limao has quit IRC16:39
*** limao has joined #openstack-kuryr16:40
mchiapperoivc_: one thing I don't understand: what is the benefit of having port as sentinel?16:42
*** lmdaly has quit IRC17:06
*** diogogmt has joined #openstack-kuryr17:23
*** shashank_hegde has joined #openstack-kuryr17:29
*** shashank_hegde has quit IRC17:41
*** tonanhngo has quit IRC18:04
*** garyloug has quit IRC18:13
*** diogogmt has quit IRC18:22
*** tonanhngo has joined #openstack-kuryr18:53
*** shashank_hegde has joined #openstack-kuryr18:53
*** tonanhngo has quit IRC18:58
*** diogogmt has joined #openstack-kuryr19:03
*** tonanhngo has joined #openstack-kuryr19:03
*** diogogmt has quit IRC19:04
*** tonanhngo has quit IRC19:05
*** diogogmt has joined #openstack-kuryr19:05
*** tonanhngo has joined #openstack-kuryr19:06
ivc_mchiappero sentinel is the way to guarantee that values are identical without taking care of their type. it's just more idiomatic than using real values19:32
*** jgriffith is now known as jgriffith_AutoAw19:58
*** jgriffith_AutoAw is now known as jgriffith20:01
*** limao has quit IRC20:13
*** limao has joined #openstack-kuryr20:13
*** jgriffith is now known as jgriffith_AutoAw20:17
*** jgriffith_AutoAw is now known as jgriffith20:34
openstackgerritHongbin Lu proposed openstack/fuxi: [WIP] Use keystoneauth1 for cinder access  https://review.openstack.org/41040321:28
openstackgerritOpenStack Proposal Bot proposed openstack/fuxi: Updated from global requirements  https://review.openstack.org/37374521:41
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-kubernetes: Updated from global requirements  https://review.openstack.org/40979721:43
openstackgerritHongbin Lu proposed openstack/fuxi: [WIP] Use keystoneauth1 for cinder access  https://review.openstack.org/41040321:43
openstackgerritHongbin Lu proposed openstack/fuxi: Use keystoneauth1 for cinder access  https://review.openstack.org/41040322:22
openstackgerritHongbin Lu proposed openstack/fuxi: Use keystoneauth1 for cinder access  https://review.openstack.org/41040322:23
*** yamamoto has joined #openstack-kuryr22:28
*** tonanhngo_ has joined #openstack-kuryr22:34
*** tonanhngo has quit IRC22:37
openstackgerritMerged openstack/kuryr-kubernetes: kuryr kubernetes design reference document  https://review.openstack.org/40807822:48
*** limao has quit IRC23:18
*** tonanhngo_ has quit IRC23:28
*** tonanhngo has joined #openstack-kuryr23:29
openstackgerritOpenStack Proposal Bot proposed openstack/kuryr-kubernetes: Updated from global requirements  https://review.openstack.org/40979723:45

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