03:00:04 #startmeeting zun 03:00:05 Meeting started Tue Aug 29 03:00:04 2017 UTC and is due to finish in 60 minutes. The chair is hongbin. Information about MeetBot at http://wiki.debian.org/MeetBot. 03:00:06 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 03:00:08 The meeting name has been set to 'zun' 03:00:11 #link https://wiki.openstack.org/wiki/Zun#Agenda_for_2017-08-29_0300_UTC Today's agenda 03:00:14 #topic Roll Call 03:00:17 O/ 03:00:18 o/ 03:00:19 Namrata 03:00:21 hi 0/ 03:00:22 o/ 03:00:28 o/ 03:00:30 hi 03:00:47 thanks for joining the meeting Shunli spn Namrata kiennt mkrai kiseok7 03:01:03 let's get started 03:01:06 #topic Announcements 03:01:23 i have no announcement , anyone else has? 03:02:09 seem no 03:02:17 #topic Cinder integration 03:02:31 for this bp, there are some progress last week 03:02:49 i am working on two WIP patches, which i think it is ready to test 03:03:07 #link https://review.openstack.org/#/c/473115/ 03:03:25 #link https://review.openstack.org/#/c/491271/ 03:03:45 the first one is a big patch, i will split it into several smaller patches 03:04:32 basically, this is how it works: zun run --mount source=,destination= 03:04:56 the --mount option can be used multiple times to bind mount multiple volumes 03:05:31 any question so far? 03:05:59 hongbin: you are not using volume api, how mount is different from volume api ? 03:06:18 diga: i am using cinder api 03:06:26 okay 03:06:47 it means its a direct integration with Zun 03:06:53 yes 03:06:57 got it 03:07:54 ok, it looks there is no more question 03:08:09 then, we move to the next topic 03:08:19 #topic Introduce container composition (kevinz) 03:08:26 #link https://blueprints.launchpad.net/zun/+spec/introduce-compose 03:08:36 kevinz cannot join the meeting today 03:08:51 he uploaded several patches about capsule last week 03:08:51 for cinder integration using fuxi, I will update the patch today, some error needs to fix which I will update today. Due to some personal reason I couldn't work since 2 month 03:09:06 diga: ack 03:09:35 diga: you could leverage the code of my patches when integrating with fuxi 03:09:42 hongbin: Yeah 03:09:53 hongbin: will take a look at it today 03:10:06 hongbin: and update my code accordingly 03:10:34 back to the capsule topic 03:10:58 kevin has submitted some unit tests patches last week, all are merged 03:11:19 that is all for this bp 03:11:30 any question for this topic? 03:12:06 ok, advance topic 03:12:08 #topic Add support for clear container (mkrai) 03:12:13 #link https://blueprints.launchpad.net/zun/+spec/support-secure-container 03:12:24 mkrai: want to lead the discussion of this topic? 03:12:30 hongbin: sure 03:12:59 The patches for supporting a new runtime parameter in zun create/run API were merged 03:13:25 awesome 03:13:27 There is a question on whether we should allow non-admin users to select runtime or not 03:13:30 yay! 03:13:47 What do team think about it? 03:13:59 yes, i raise this suggestion 03:14:07 i could explain the rational a bit 03:14:23 i see the --runtime option as a dangourous operation 03:14:32 mkrai: did you mean changing runtime for spinning containers by non-admin users? 03:14:41 spn: Yes 03:15:00 because users could use it to choose a runtime that is more secure or less secure 03:15:17 i think we need a way to restrict the choice of runtime 03:15:42 option 1: introduce a config (i.e. enabled_runtime) to specify a list of choosible runtime 03:15:55 hongbin: I am not sure I understood that argument. Why shouldnt users be allowed to change his runtime to secure or non-secure 03:15:57 option 2: disallow non-admin user to specify the --runtime option 03:16:22 hongbin: My opinion was to let non-admin users also choose runtime so that they have the flexibilty to run their container with any of the avialable option 03:16:26 spn: i assume there are two 'runtime': docker or clear container 03:16:55 spn: if user can choose docker over clear container, i assume it is less secure 03:17:20 spn: for example, if i am a public cloud provider, i will enforce clear container as the only runtime 03:17:37 spn: make sense? 03:17:46 hongbin: I agree what you said. but not every user may need a clear container all the time based on his requirment. 03:18:16 spn: I agree on that, the users might want to change the runtime based on their requirement 03:18:17 mkrai: ack 03:18:18 but yes for public cloud there should be an option for admin to restrict it 03:18:37 then, how about option #1 03:19:04 so why dont we drop in an option for admin so that he can decide whether on this cloud a user can change container or not 03:19:22 spn: +1 03:19:30 imagine this situation where on the horizon UI, if a user is allowed to change container 03:19:57 he gets an option to click on a check box or something like that which specifies type of container also as an option. 03:20:21 cc is one type of an option , future can be any other type of runtime 03:20:45 spn: sound good +1 03:20:52 hongbin: I didn't get the option #1 clearly 03:21:12 mkrai: there is a config called "enabled_runtime" 03:21:14 hongbin: Does it mean that it can contain both docker and clear container ? 03:21:32 mkrai: if we had a config option like allowed_docker_runtimes = " docker, coe" 03:21:38 etc 03:21:38 And this might vary for other projects like just docker or just clear container 03:21:42 mkrai: yes, it could be enabled_runtime=docker,cc, or enabled_runtime=cc 03:22:03 Ok so that non-admins have right to choose from the list? 03:22:10 yes 03:22:15 yes.. allowed list 03:22:20 and admin decides the list 03:22:32 if admin says only coe just one option is shown to users 03:22:32 But this option will be applied to all the projects 03:23:02 mkrai: can this be forced upon project basis? 03:23:10 how to handle case when a admin in a project wants a different list than other project? 03:23:23 spn: Yes i am also thinking of the same case 03:24:02 hongbin: Does it makes sense? 03:24:33 mkrai: frankly, i couldn't think of use case that requires a per-project allowed list 03:25:10 like a finance team in the company which uses containers should be forced with cc 03:25:16 not all need it 03:25:30 ok, that make sense 03:25:47 then, the question is how to do it :) 03:25:55 if its needs to be project specific than it should be zun command line tunable 03:26:04 spn: +1 03:26:14 hongbin: Right 03:26:30 spn: yes, that should work 03:26:56 Ahh I am sensing a new kind of resource in Zun :D but that might not be needed 03:27:16 hongbin: spn we might need to store info with some resource 03:27:32 store the enabled_driver info in db 03:28:32 yes if we go that path 03:28:43 I guess it should be info of compute node. 03:28:46 I will start to work on this implementation and present the idea in next meeting 03:28:58 I will discuss with spn 03:28:59 mkrai: +1 03:29:10 ok, sound good 03:29:12 Shunli: ack 03:29:28 as the clear container should run in clear linux, right? 03:29:44 Shunli: it can run on other OS also 03:29:50 Like Ubuntu Cent OS etc 03:30:08 mkrai: but the image is special 03:30:21 mkrai: thx ,ack 03:30:22 it should have a particular kernel version isn;t it 03:30:28 spn: Yes we need to install clear container on each compute node 03:30:54 mkrai: I am talking about changes inside the glance image? if any 03:31:18 spn: Which glance image? 03:31:32 sorry I meant the docker image which runs on cc 03:32:13 It is same as the normal image 03:32:24 mkrai: ok got it 03:32:57 hongbin: that's all from me :) 03:33:02 thanks mkrai 03:33:12 thanks mkrai 03:33:19 mkrai: thanks for brining this discussion :) 03:33:32 #topic NFV use cases (lakerzhou) 03:33:35 Thank you all for a good discussion :) 03:33:40 #link https://etherpad.openstack.org/p/zun-nfv-use-cases 03:33:59 for this one, Shunli is working on several patches about pci 03:34:29 Shunli: do you have more details to add? 03:34:52 i'm struggled on the unit test last week. so the pci patch is a bit slow 03:35:12 just uploaded a pci device db model yesterday 03:35:36 no more progress about the pci feature. 03:35:42 #link https://review.openstack.org/#/c/498286/ 03:35:58 Shunli: i think those are good progress 03:36:10 Shunli: thanks for the work 03:36:44 hongbin: the api version controller breaks the unit. 03:37:03 i'm cannot solve it, need someone help on it. 03:37:03 Shunli: which patch? 03:37:24 the network detach api patch. it's random. 03:37:42 even after i add the http header of api version for ut 03:37:57 it's still fails some times. 03:38:07 this one? https://review.openstack.org/#/c/493787/ 03:38:16 https://review.openstack.org/#/c/493787/ 03:38:56 not sure if someone familiar with the pecan route, can digg into this problem. 03:39:18 Shunli: I also added the api-version in header for unittest 03:39:23 otherwise it failed 03:39:46 however, the gate looks all pass 03:39:55 yes, it success some times, some times fail. 03:40:27 ok, we can work on that offline 03:40:47 ok. 03:41:05 all, any other comment on this topic? 03:41:33 #topic Open Discussion 03:41:53 all, any topic that you want to discuss with the team? 03:42:01 hi, i have one 03:42:05 https://etherpad.openstack.org/p/zun-multihost-problems 03:42:08 #link https://etherpad.openstack.org/p/zun-multihost-problems 03:43:16 zun multi-host scenario has some problems and I need advice. 03:43:27 * hongbin is reading the etherpad 03:46:11 kiennt: i couldn't figure out what was wrong , need to find some time to setup the environment to reproduce the error 03:47:18 kiennt: i will get back to that after finishing the cinder bp 03:48:57 hongbin: Basically, Zun doesn't pass pool_id because subnet doesn't have subnetpool_id. So Kuryr will try to create new kuryr subnetpool (which already created in the 1st node) 03:49:22 Therefore it will raise exception Another pool with same cirdr 03:49:37 yes 03:50:39 however, i don't have a solution in mind right now :) 03:51:20 perhaps this bp will help 03:51:22 #link https://blueprints.launchpad.net/kuryr-libnetwork/+spec/existing-subnet 03:51:27 kiennt: exist if detects no pool id by zun? 03:51:33 exit* 03:51:34 does neutron has the tag plugin enabled? 03:52:08 hongbin: np, it's a bit tricky :D thanks for the link 03:52:19 Shunli: yes, it does 03:52:36 Shunli: tag plugin is enabled after pike 03:52:59 it should be called "tag extension" 03:53:04 https://review.openstack.org/#/c/441024/ 03:53:12 yes, tag extension 03:54:46 ok, let's work on the project this week, and rediscuss it at the next meeting 03:54:52 spn: yes, if zun can't pass pool_id, kuryr will create its subnetpool. 03:55:06 hongbin: yes 03:55:08 exist if detects no pool id by zun? 03:55:08 exit* 03:55:22 oh, wrong copy paste 03:55:27 :) 03:55:28 sorry, thank you all 03:55:51 any other topic to discuss? 03:56:13 none from me 03:56:20 no 03:56:29 ok 03:56:40 all, thanks for joining the meeting, see you next week 03:56:43 #endmeeting