| *** wanghao_ has quit IRC | 00:36 | |
| zhenguo | morning mogan! | 00:56 |
|---|---|---|
| *** kevinz has joined #openstack-mogan | 00:59 | |
| *** wanghao has joined #openstack-mogan | 01:29 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add check for availability zone when set resources https://review.openstack.org/463438 | 02:04 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan-ui master: Fix no availability zone error https://review.openstack.org/463439 | 02:11 |
| *** wanghao has quit IRC | 02:15 | |
| *** wanghao has joined #openstack-mogan | 02:16 | |
| *** wanghao has quit IRC | 02:16 | |
| openstackgerrit | Merged openstack/mogan-ui master: Fix no availability zone error https://review.openstack.org/463439 | 02:24 |
| zhenguo | liusheng: wrt the flavor access, do you think common users should get flavor with access info, like it's a private flavor, a common users would see who can access it | 02:54 |
| liusheng | zhenguo: let me try in Nova :) | 02:56 |
| zhenguo | liusheng: nova doens't return such things | 02:56 |
| zhenguo | liusheng: also the extra specs not return with flavor list/show | 02:56 |
| zhenguo | liusheng: how about just keeping flavor and access as two set of APIs | 02:57 |
| zhenguo | liusheng: and on client side, we can call them separately if we want to show access info for admin | 02:57 |
| liusheng | zhenguo: yes, but OSC command: flavor show return it | 02:57 |
| zhenguo | liusheng: I think it call several APIs | 02:57 |
| liusheng | zhenguo: sounds a bit reasonable | 02:58 |
| zhenguo | liusheng: and seems access info should not be exposed to normal users, wdyt? | 02:58 |
| zhenguo | liusheng: let me check our policy for such APIs | 02:59 |
| liusheng | zhenguo: so just abandon my patch ? lol | 02:59 |
| liusheng | zhenguo: I am ok with that | 02:59 |
| zhenguo | liusheng: hah, sorry, but we can wait for a while | 02:59 |
| liusheng | zhenguo: ok | 02:59 |
| zhenguo | liusheng: our API policy is not denied normal users to list flavor access info | 03:01 |
| liusheng | zhenguo: you mean the default rules ? | 03:02 |
| zhenguo | liusheng: yes, I added it, but not check nova's policy | 03:02 |
| liusheng | zhenguo: maybe only admin | 03:02 |
| zhenguo | liusheng: maybe, need to confirm what's nova policies for this | 03:02 |
| liusheng | zhenguo: ok, our flavor object don't have project and user properties | 03:03 |
| zhenguo | liusheng: aha, yes | 03:04 |
| zhenguo | liusheng: for private flavor, access is the extenstion for project attribute :D | 03:05 |
| zhenguo | liusheng: btw, our UI now support server create/delete and keypairs management, but only with the basic functionalities. | 03:07 |
| liusheng | zhenguo: awesome, cool! | 03:07 |
| liusheng | lol | 03:07 |
| zhenguo | liusheng: maybe we can show during the Bug Smash | 03:08 |
| liusheng | zhenguo: we can show some thing in bug smash | 03:08 |
| liusheng | yes | 03:08 |
| zhenguo | liusheng: hah, | 03:08 |
| zhenguo | liusheng: please have a look at this https://review.openstack.org/#/c/463438/ when you got time, I changed the default az to mogan from nova :D | 03:09 |
| liusheng | zhenguo: ok, sure | 03:10 |
| zhenguo | liusheng: thanks | 03:10 |
| * zhenguo brb | 03:11 | |
| *** wanghao has joined #openstack-mogan | 03:17 | |
| *** wanghao has quit IRC | 03:22 | |
| openstackgerrit | Merged openstack/mogan master: Add check for availability zone when set resources https://review.openstack.org/463438 | 03:46 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: [POC] New flavor https://review.openstack.org/457178 | 03:51 |
| *** wanghao has joined #openstack-mogan | 04:18 | |
| *** wanghao has quit IRC | 04:22 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan-ui master: Add server ips column https://review.openstack.org/463465 | 05:26 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Detach interface for server https://review.openstack.org/461767 | 05:46 |
| *** wanghao has joined #openstack-mogan | 06:19 | |
| *** wanghao has quit IRC | 06:24 | |
| zhenguo | liusheng: how about changing server.extra to server.metadata? | 07:10 |
| liusheng | zhenguo: they represent same things, right ? | 07:11 |
| liusheng | zhenguo: ok | 07:11 |
| zhenguo | liusheng: yes, same thing, just change the name | 07:11 |
| zhenguo | liusheng: extra seems not meaningful :D | 07:11 |
| liusheng | zhenguo: ok, sounds reasonable | 07:11 |
| liusheng | haha | 07:11 |
| zhenguo | lol | 07:11 |
| zhenguo | liusheng: for simple, maybe just change the API object | 07:12 |
| liusheng | zhenguo: don't change the db field ? | 07:12 |
| zhenguo | liusheng: I remember metadata is an observed name, not sure | 07:13 |
| zhenguo | s/observed/reserved | 07:13 |
| liusheng | in mogan ? | 07:14 |
| liusheng | zhenguo: in mogan ? | 07:14 |
| zhenguo | liusheng: no, not related to mogan | 07:14 |
| liusheng | zhenguo: oh, many projects, have the concept metadata, seems. | 07:15 |
| zhenguo | liusheng: IIRC, I tried it before, | 07:15 |
| liusheng | zhenguo: lol | 07:15 |
| zhenguo | liusheng: do you like to try it again? | 07:15 |
| zhenguo | lol | 07:15 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan-ui master: Add server ips column https://review.openstack.org/463465 | 07:18 |
| *** wanghao has joined #openstack-mogan | 07:20 | |
| *** wanghao has quit IRC | 07:24 | |
| openstackgerrit | Merged openstack/mogan-ui master: Add server ips column https://review.openstack.org/463465 | 07:33 |
| *** wanghao has joined #openstack-mogan | 08:17 | |
| *** wanghao has quit IRC | 08:21 | |
| *** wanghao has joined #openstack-mogan | 08:49 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Following up patch of 7d0f8ab36074c38327d47280c7f3ef6bbaad7f1a https://review.openstack.org/463496 | 08:50 |
| zhenguo | liusheng: just found a wrong renaming for instance to server: https://review.openstack.org/463496 | 08:51 |
| *** wanghao has quit IRC | 08:52 | |
| liusheng | zhenguo: good catch, hah my mistake | 08:52 |
| zhenguo | liusheng: hah | 08:52 |
| *** wanghao has joined #openstack-mogan | 08:52 | |
| *** wanghao has quit IRC | 08:53 | |
| *** wanghao has joined #openstack-mogan | 08:53 | |
| *** wanghao has quit IRC | 08:54 | |
| *** wanghao has joined #openstack-mogan | 08:54 | |
| *** wanghao has quit IRC | 08:55 | |
| *** wanghao has joined #openstack-mogan | 08:55 | |
| *** wanghao has quit IRC | 08:55 | |
| *** wanghao has joined #openstack-mogan | 08:56 | |
| *** wanghao has quit IRC | 08:56 | |
| *** wanghao has joined #openstack-mogan | 08:56 | |
| *** wanghao has quit IRC | 08:57 | |
| liusheng | zhenguo: hi zhenguo, about the scheduler_hints, currently, we don't have any filter to use the scheduler_hints. do you think it is ok to support pass the scheduler_hints to scheduler for now. and in the future if we have the rquirement then we add filter using scheduler_hints ? | 09:01 |
| zhenguo | liusheng: yeah, it's ok | 09:01 |
| liusheng | zhenguo: ok, btw, we should consider to persistent the request_spec data to db, since it provide convenience, and nova have support that | 09:02 |
| zhenguo | liusheng: yes, I seee nova has a request spec object, but I didn't find why | 09:03 |
| zhenguo | liusheng: for us, request spec is only used when scheduing | 09:04 |
| liusheng | zhenguo: that is what I said | 09:04 |
| liusheng | zhenguo: yes, but it's better to persistent that info, since that can be use to query the scheduler_hints after a server created | 09:05 |
| liusheng | zhenguo: but, seems Mogan don't need to support migration, hal | 09:05 |
| liusheng | hah | 09:05 |
| zhenguo | liusheng: sure, hah | 09:05 |
| liusheng | zhenguo: if we don't persistent request_spec, some scheduling related info such as scheduler_hints will lost after a server created | 09:06 |
| zhenguo | liusheng: does that have any side effect if lost the hints? | 09:07 |
| zhenguo | liusheng: as I understand scheduler hints is only used to give some hints to scheduler, correct me if I'm wrong :D | 09:07 |
| liusheng | zhenguo: in Nova, that may effect instance migration. and also may provide a way to query the sheduling parameter about a specific instance creation | 09:08 |
| liusheng | zhenguo: I am not sure, seems our products have similar requirement | 09:09 |
| liusheng | zhenguo: yes | 09:09 |
| zhenguo | liusheng: our products? | 09:09 |
| zhenguo | liusheng: seems now we can just ignore the migration case | 09:10 |
| liusheng | zhenguo: yes, so the only use case I can find out that may there is need to query the scheduling paramters about a server after creation | 09:11 |
| zhenguo | liusheng: hmm. why we need to do that? | 09:11 |
| liusheng | zhenguo: and I just confirmed with zhenyu, he also said that is two main use cases | 09:11 |
| zhenguo | liusheng: maybe we can just not save it, only find out true requirements, wdyt? | 09:12 |
| liusheng | zhenguo: yes, I am ok at least for now :D | 09:16 |
| zhenguo | liusheng: thanks | 09:16 |
| zhenguo | liusheng: when bulding the UI, I find we really have many things to do | 09:16 |
| liusheng | zhenguo: hah | 09:28 |
| liusheng | zhenguo: may you can just add them to our tasks list | 09:29 |
| zhenguo | liusheng: yes, I just added one, add filters to list API :D | 09:29 |
| zhenguo | liusheng: and many things blocked by my new flavor task, will focus on it this week | 09:30 |
| liusheng | zhenguo: how about use searchlight to perform that ? | 09:30 |
| zhenguo | liusheng: sounds good, but not familiar with SL :D | 09:31 |
| liusheng | zhenguo: we have a SL core. lol | 09:31 |
| zhenguo | lol | 09:31 |
| zhenguo | liusheng: but seems it require us to send full notifications | 09:31 |
| liusheng | zhenguo: yes | 09:32 |
| zhenguo | liusheng: not sure it's easy for us now to achieve it | 09:32 |
| zhenguo | liusheng: and seems SL will not accept a non official project | 09:33 |
| Kevin_Zheng | you can use api call first | 09:33 |
| liusheng | zhenguo: once we have basical stable frame, we can add more notification | 09:33 |
| zhenguo | liusheng: maybe you can ask Kevin_Zheng to see whether we can leverage SL | 09:35 |
| liusheng | zhenguo: yes, he just said that we can implement an own SL plugin firstly. lol | 09:35 |
| Kevin_Zheng | not very hard | 09:35 |
| Kevin_Zheng | maybe 100 loc | 09:35 |
| zhenguo | liusheng: if so, maybe you can try | 09:35 |
| zhenguo | Kevin_Zheng: really? lol | 09:36 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Following up patch of 7d0f8ab36074c38327d47280c7f3ef6bbaad7f1a https://review.openstack.org/463496 | 09:49 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: [POC] New flavor https://review.openstack.org/457178 | 09:55 |
| *** wanghao has joined #openstack-mogan | 09:57 | |
| *** wanghao has quit IRC | 10:02 | |
| *** kevinz has quit IRC | 10:19 | |
| *** wanghao has joined #openstack-mogan | 10:58 | |
| *** wanghao has quit IRC | 11:03 | |
| openstackgerrit | liusheng proposed openstack/mogan master: WIP: Add support for scheduler_hints https://review.openstack.org/463534 | 11:37 |
| *** wanghao has joined #openstack-mogan | 11:38 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: [POC] New flavor https://review.openstack.org/457178 | 11:43 |
| openstackgerrit | Merged openstack/mogan master: Following up patch of 7d0f8ab36074c38327d47280c7f3ef6bbaad7f1a https://review.openstack.org/463496 | 11:45 |
| *** wanghao has quit IRC | 11:54 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Add support for scheduler_hints https://review.openstack.org/463534 | 12:00 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add create flavor json schema check https://review.openstack.org/463544 | 12:06 |
| openstackgerrit | liusheng proposed openstack/python-moganclient master: Add support for scheduler hint https://review.openstack.org/463553 | 12:23 |
| *** kevinz has joined #openstack-mogan | 13:19 | |
| *** kevinz has quit IRC | 13:27 | |
| *** kevinz has joined #openstack-mogan | 13:28 | |
| *** kevinz has quit IRC | 15:26 | |
| *** kevinz has joined #openstack-mogan | 16:08 | |
| *** kevinz has quit IRC | 16:13 | |
| *** liusheng has quit IRC | 18:30 | |
| *** liusheng has joined #openstack-mogan | 18:30 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!