Friday, 2017-06-16

*** zhurong has joined #openstack-zun00:34
openstackgerritHongbin Lu proposed openstack/zun master: [WIP] Make sandbox container optional  https://review.openstack.org/47163400:37
*** hongbin has quit IRC00:38
*** caowei_ has joined #openstack-zun00:49
*** caowei has quit IRC00:52
*** caowei_ is now known as caowei00:52
*** zhurong has quit IRC01:30
*** Shunli has joined #openstack-zun01:30
openstackgerritMerged openstack/zun master: Updated from global requirements  https://review.openstack.org/47470001:43
openstackgerritMerged openstack/zun master: Add label filter  https://review.openstack.org/46660301:47
*** zhurong has joined #openstack-zun01:50
*** jwcroppe has joined #openstack-zun02:11
*** rbudden has quit IRC02:29
*** hongbin has joined #openstack-zun03:17
openstackgerritHongbin Lu proposed openstack/zun master: [WIP] Make sandbox container optional  https://review.openstack.org/47163403:19
*** hongbin has quit IRC03:52
*** lakerzhou2 has joined #openstack-zun03:53
*** zhurong has quit IRC04:25
*** zhurong has joined #openstack-zun04:31
*** adisky__ has joined #openstack-zun04:31
*** lakerzhou2 has quit IRC05:36
*** janki has joined #openstack-zun05:41
*** arund has joined #openstack-zun07:08
*** egonzalez has joined #openstack-zun07:48
*** zhurong has quit IRC08:37
*** zhurong has joined #openstack-zun08:45
*** Shunli has quit IRC09:15
*** sudipto has joined #openstack-zun09:33
*** sudipto_ has joined #openstack-zun09:34
openstackgerritfeng.shengqin proposed openstack/zun master: Remove Z323 hacking rule  https://review.openstack.org/47492509:35
*** zhurong has quit IRC09:59
*** vikasc has joined #openstack-zun10:16
*** caowei has quit IRC10:31
*** nicolasbock has joined #openstack-zun11:02
*** sudipto_ has quit IRC11:24
*** sudipto has quit IRC11:24
*** rbudden has joined #openstack-zun12:15
*** catintheroof has joined #openstack-zun12:54
*** jwcroppe has quit IRC13:07
*** jwcroppe has joined #openstack-zun13:08
*** jwcroppe has quit IRC13:12
*** jwcroppe has joined #openstack-zun13:28
*** lakerzhou2 has joined #openstack-zun13:29
*** adisky__ has quit IRC13:39
*** jwcroppe has quit IRC13:55
*** jwcroppe has joined #openstack-zun13:55
*** jwcroppe has quit IRC14:00
*** jwcroppe has joined #openstack-zun14:04
*** hongbin has joined #openstack-zun14:37
*** janki has quit IRC14:38
*** lakerzhou2 has quit IRC15:09
*** egonzalez has quit IRC15:10
*** lakerzhou2 has joined #openstack-zun15:11
*** arund has quit IRC15:50
openstackgerritMerged openstack/python-zunclient master: Return image ID in container commit  https://review.openstack.org/47087015:51
openstackgerritMerged openstack/python-zunclient master: OSC: return columns instead of using print_dict  https://review.openstack.org/47085015:55
openstackgerritMerged openstack/python-zunclient master: Set concurrency to 1 for OSC tempest tests  https://review.openstack.org/47103215:57
openstackgerritMerged openstack/python-zunclient master: Zunclient should escape special character  https://review.openstack.org/46937116:01
*** sudipto has joined #openstack-zun16:57
*** sudipto_ has joined #openstack-zun16:57
*** lakerzhou2 has quit IRC17:00
*** lakerzhou2 has joined #openstack-zun17:01
*** catintheroof has quit IRC17:37
lakerzhou2hongbin, can zun create container by specifying a neutron port instead of network-id via kuryr?17:37
lakerzhou2hongbin, currently nova only support creating VM with SR-IOV port, which was created already. Networkid is not good enough here, because VM-1 can connect to Nework-N (VLAN) via regular binding using libvirt, VM-2 can connect to the same network-N via direct binding (SR-IOV) even on the same host17:41
hongbinlakerzhou2: i think that is a good idea17:44
hongbinlakerzhou2: this feature sounds like part of the network-api BP: https://blueprints.launchpad.net/zun/+spec/network-rest-api , mkrai , see the requirement above17:46
hongbinlakerzhou2: what is the API sound look like from your perspective ?17:55
lakerzhou2hongbin, give me a few minutes please, i will get back to you17:56
hongbinlakerzhou2: sure, i need to go out to get a coffee as well17:56
*** sudipto has quit IRC18:04
*** sudipto_ has quit IRC18:04
lakerzhou2hongbin, nova boot vm1 --nic port_id=$port_id ... For zun, we should support 1) multiple endpoints; 2)attaching adding endpoints. I read somewhere docker container can only connect to one network at creation time, but container can attach to multiple network after creation18:12
lakerzhou2*attaching additional endpoints*18:12
lakerzhou2Hongbin, the question is what kuryr can support. The document (maybe out dated) saying kuryr listens to /NetworkDriver.CreateEndpoint then will fetch port from neutron to match endpoint options (CIDR).18:19
lakerzhou2Hongbin, In SRIOV case, we should add a new endpoint option (e.g. PciPassthrough=direct for example).18:25
lakerzhou2Hongbin, Kuryr will have to bind the port according to its endpoint option - for SR-IOV port, default veth pair binding is not good18:26
hongbinlakerzhou2: i see18:26
hongbinlakerzhou2: for the nic option and the dynamic attaching endpoints, i think those are good ideas18:28
hongbinactually, i will write it down to the whiteboard of the BP18:28
hongbinlakerzhou2: another problem is how to specify a neutron port when creating the container?18:29
hongbinlakerzhou2: the last problem is how to do the neutron port binding i guess18:30
lakerzhou2hongbin, I am trying to document all of these thoughts in spec. I am thinking to specify network as we do today, then add additional endpoint options.18:31
hongbinlakerzhou2: yes, then kuryr has to understand that endpoint options i guess18:32
lakerzhou2hongbin, there are kuryr dependency for sure if we use kuryr driver. But most challenge part for me is neutron port binding.18:33
hongbinlakerzhou2: kuryr is designed to be pluggable for port binding AFAIK18:34
lakerzhou2Hongbin, network creation, SR-IOV port creation are out-of the scope of kuryr and Zun18:34
hongbinlakerzhou2: here is a list of drivers: https://github.com/openstack/kuryr/tree/master/kuryr/lib/binding/drivers , i imagine another driver is needed18:34
lakerzhou2Hongbin, I was trying to read kuryr codes till 2am last night, still don't know how to do the binding for SR-IOV port18:35
lakerzhou2hongbin, veth pair binding is quite simple. I think the only change in kuryr is the create_host_iface()18:36
hongbinlakerzhou2: i am not familiar with this part as well...18:36
*** nicolasbock has quit IRC18:38
hongbinlakerzhou2: perhaps we could go to the #openstack-kuryr channel, and ping the kuryr ptl18:38
lakerzhou2hongbin, my plan is to walk through all the scenario and work with kuryr or neutron ppl to get it done later. For now, we will document it as a dependency.18:38
hongbinlakerzhou2: ack18:38
lakerzhou2Hongbin, sure, the earlier the better18:38
lakerzhou2hongbin, do you know PTL of kuryr?18:39
hongbinlakerzhou2: i know18:39
hongbinlakerzhou2: let me ping me right now18:39
lakerzhou2hongbin, thanks, I am join the channel now18:39
hongbinlakerzhou2: it is the evening time for him right now, not sure if he is still here18:40
lakerzhou2hongbin, no worries. I can ping him later. Basically, I believe we should use kuryr in stead of another remote driver. And there are some work required at kuryr side. Hopefully not big (not big as far as I can see).18:43
hongbinlakerzhou2: ack18:44
lakerzhou2hongbin, thanks for the discussion, I will go back to my doc. It takes way more time than I thought. But I am half way through.18:45
hongbinlakerzhou2: np18:47
*** mkrai has quit IRC18:59
openstackgerritHongbin Lu proposed openstack/zun master: [WIP] Make sandbox container optional  https://review.openstack.org/47163419:00
*** lakerzhou2 has quit IRC20:38
openstackgerritHongbin Lu proposed openstack/python-zunclient master: Add security groups to container create/run  https://review.openstack.org/47507621:21
*** mkrai has joined #openstack-zun21:32
*** jwcroppe has quit IRC22:20
*** yasemin has joined #openstack-zun22:26
openstackgerritHongbin Lu proposed openstack/zun master: Make sandbox container optional  https://review.openstack.org/47163422:50
*** jwcroppe has joined #openstack-zun22:54

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