*** wanghao has joined #openstack-mogan | 00:27 | |
*** wanghao has quit IRC | 00:27 | |
*** wanghao has joined #openstack-mogan | 00:28 | |
*** wanghao has quit IRC | 00:41 | |
*** wanghao has joined #openstack-mogan | 00:43 | |
*** wanghao has quit IRC | 00:43 | |
*** wanghao has joined #openstack-mogan | 00:45 | |
*** Xinran has quit IRC | 01:02 | |
*** Xinran has joined #openstack-mogan | 01:03 | |
zhenguo | morning mogan! | 01:06 |
---|---|---|
wanghao | morning! | 01:09 |
*** liujiong has joined #openstack-mogan | 01:10 | |
liusheng | morning | 01:12 |
*** wanghao has quit IRC | 01:17 | |
*** wanghao has joined #openstack-mogan | 01:18 | |
*** wanghao_ has joined #openstack-mogan | 01:27 | |
*** wanghao has quit IRC | 01:31 | |
*** litao__ has joined #openstack-mogan | 01:38 | |
*** wanghao_ has quit IRC | 02:05 | |
*** wanghao has joined #openstack-mogan | 02:06 | |
*** openstackgerrit has joined #openstack-mogan | 02:09 | |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Adds `disabled` field for flavors. https://review.openstack.org/474441 | 02:09 |
*** wanghao_ has joined #openstack-mogan | 02:43 | |
*** wanghao has quit IRC | 02:46 | |
*** wanghao_ has quit IRC | 02:57 | |
*** wanghao has joined #openstack-mogan | 03:04 | |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Revert "Add ComputeDisk object" https://review.openstack.org/475215 | 03:06 |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Revert "Add ComputeDisk object" https://review.openstack.org/475215 | 03:07 |
*** wanghao has quit IRC | 03:10 | |
*** wanghao has joined #openstack-mogan | 03:11 | |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Revert "Add ComputeDisk object" https://review.openstack.org/475215 | 05:06 |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add `extra_specs` to flavor create JSON schema https://review.openstack.org/475232 | 05:24 |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add `extra_specs` to flavor create JSON schema https://review.openstack.org/475232 | 06:25 |
openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test cases for console https://review.openstack.org/474498 | 07:07 |
openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test cases for server interface attaching https://review.openstack.org/474835 | 07:11 |
openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test cases for server networks API https://review.openstack.org/474829 | 07:11 |
*** wanghao_ has joined #openstack-mogan | 07:31 | |
*** wanghao has quit IRC | 07:33 | |
*** wanghao_ has quit IRC | 07:35 | |
*** wanghao has joined #openstack-mogan | 07:36 | |
*** openstackgerrit has quit IRC | 07:48 | |
*** wanghao_ has joined #openstack-mogan | 07:58 | |
*** wanghao_ has quit IRC | 07:59 | |
*** wanghao_ has joined #openstack-mogan | 07:59 | |
*** wanghao has quit IRC | 08:00 | |
*** wanghao has joined #openstack-mogan | 08:14 | |
*** wanghao_ has quit IRC | 08:16 | |
zhenguo | liusheng, sahohe_feng, wanghao, litao__: review time | 08:32 |
wanghao | zhenguo: yeah | 08:32 |
litao__ | got it | 08:32 |
liusheng | zhenguo: ok | 08:33 |
*** openstackgerrit has joined #openstack-mogan | 08:36 | |
openstackgerrit | Zhenguo Niu proposed openstack/mogan-specs master: New flavor for baremetal servers https://review.openstack.org/454113 | 08:36 |
zhenguo | litao__: please find my reply here https://review.openstack.org/#/c/474441/ | 08:37 |
litao__ | zhenguo: If the flavor is disabled , could we get it from API? | 08:39 |
zhenguo | litao__: yes | 08:39 |
zhenguo | litao__: need to add a filter later | 08:39 |
litao__ | zhenguo: So if the flavor is disabled , user can use it for creating servers? | 08:40 |
zhenguo | litao__: in fact not, we will raise flavor not found exception | 08:40 |
litao__ | zhenguo: I think the exception message will confuse users | 08:43 |
zhenguo | litao__: we will finally not let common users list the disabled flavors | 08:44 |
litao__ | zhenguo: In fact , the flavor exists | 08:44 |
zhenguo | litao__: you mean raise something like "the flavor is disabled"? | 08:45 |
litao__ | zhenguo: yeah, If so , that's ok. | 08:45 |
zhenguo | litao__: ok | 08:45 |
liusheng | hey guys, except for the floatingip association/disassociation tempest test which I am working on, our tempest tests have covered all the APIs | 08:47 |
zhenguo | liusheng: great | 08:47 |
wanghao | liusheng: cool! | 08:59 |
wanghao | guys, is there any comments more in specify neutron port patch? I'd like to land it this week. | 09:00 |
openstackgerrit | liusheng proposed openstack/mogan master: WIP: Tempest: add tempest test for server floatingip association/disassociation https://review.openstack.org/475302 | 09:06 |
zhenguo | wanghao: will take a look | 09:09 |
wanghao | zhenguo: thanks very much | 09:09 |
zhenguo | wanghao: np | 09:09 |
wanghao | zhenguo: hah | 09:10 |
wanghao | zhenguo: I leave a comment for disabled flavors | 09:11 |
wanghao | zhenguo: I think we may need to change list and get flavor APIs too. | 09:11 |
wanghao | zhenguo: since we need to filter the disabled flavor to common user. | 09:11 |
wanghao | zhenguo: just let admin to see all flavors. | 09:11 |
zhenguo | wanghao: yes, but that will be done later :D | 09:12 |
zhenguo | wanghao: as we still discuss whether we need to add filters to list api or leverage searchlight | 09:12 |
zhenguo | wanghao: but admin can also create servers, lol | 09:12 |
wanghao | zhenguo: yes, but they can't use disabled flavor too. | 09:13 |
wanghao | zhenguo: just can see it | 09:13 |
zhenguo | wanghao: yes, so I plan to add a 'disabled' filter | 09:13 |
wanghao | zhenguo: that will be great in our API, and I didn't get how to use searchlight in Mogan. | 09:14 |
wanghao | zhenguo: you mean we didn't need our own list api? | 09:14 |
zhenguo | wanghao: seems yes | 09:14 |
zhenguo | wanghao: but still under discussion, as we are non official projects | 09:15 |
zhenguo | wanghao: so personally I think we still need to add filters to list API in Pike and Xinran is also work on server list now | 09:15 |
wanghao | zhenguo: okay | 09:16 |
zhenguo | wanghao: so for the disabled patch, I don't introduce filters | 09:16 |
zhenguo | wanghao: we will add that support later | 09:16 |
wanghao | zhenguo: ok, make sense to me, in my mind, I'd like we keep the list api. | 09:17 |
zhenguo | wanghao: hah, agree | 09:17 |
wanghao | zhenguo: we can hold it until nova or other project want to use searchlight. | 09:17 |
zhenguo | wanghao: sure | 09:17 |
zhenguo | hi all, I updated the new flavor specs, please have a look | 09:18 |
zhenguo | the resouces filed has been changed to use a dict object type, and I plan to not only include node resouce_class, we may also need to add disk/nic resouce class later | 09:19 |
zhenguo | maybe the baremetal node should be a nested resource provider | 09:19 |
zhenguo | we can leverage Placement nested resouce provider parent->children pattern after it landed | 09:20 |
zhenguo | liusheng, wanghao, liao__: ^^ | 09:21 |
wanghao | zhenguo: why could be a nested resource provider? | 09:22 |
wanghao | zhenguo: do not use the whole hardware resource in a bm? | 09:22 |
zhenguo | wanghao: I also want to expose disk and nic resouces | 09:22 |
zhenguo | wanghao: yes | 09:23 |
zhenguo | wanghao: yes, for disks and nics we may only use a part of them | 09:23 |
wanghao | zhenguo: but left part could be use anymore... | 09:24 |
wanghao | coudn't be use anymore | 09:24 |
zhenguo | wanghao: yes, couldn't be used by others | 09:24 |
wanghao | zhenguo: is there use case for this? I mean user only want to use a part of hardware of a bm. | 09:25 |
wanghao | zhenguo: maybe he can extend the resource later? | 09:25 |
zhenguo | wanghao: no, not for users, I want to expose childer resources to scheduler | 09:25 |
wanghao | zhenguo: that I see, | 09:26 |
zhenguo | wanghao: I want to treat node/disk/nic as separate resouce but with a relation ship as parent-childern | 09:26 |
zhenguo | wanghao: need to think more about this, but the flavor changes can make it more flexible, even if we don't want to use nested resources | 09:27 |
wanghao | zhenguo: okay I see, I just feel some weired about node this parent and disk/nic is childern. | 09:29 |
zhenguo | wanghao: hah, | 09:30 |
wanghao | zhenguo: but I agree with that we should including those resource. | 09:30 |
zhenguo | wanghao: there will be a spec later for move to placement | 09:30 |
*** wanghao has quit IRC | 09:33 | |
zhenguo | liusheng: do you think the node notion includes nics and disks? | 09:35 |
liusheng | zhenguo: maybe, even it is not necessary for us, maybe it is useful in the future for valence or cloudboot ? | 09:36 |
liusheng | sorry I didn't attend your disscussion | 09:38 |
zhenguo | liusheng: hah, yes, | 09:39 |
zhenguo | liusheng: do you have the link for jay pipes presentation during the bug smash | 09:39 |
liusheng | zhenguo: did he sent it publick ? | 09:40 |
zhenguo | liusheng: not sure, hah | 09:40 |
liusheng | zhenguo: if you need, may we can ask other guys our team | 09:40 |
zhenguo | liusheng: hah, no rush | 09:40 |
* zhenguo brb | 09:41 | |
*** liujiong has quit IRC | 10:05 | |
openstackgerrit | liusheng proposed openstack/mogan master: WIP: Tempest: add tempest test for server floatingip association/disassociation https://review.openstack.org/475302 | 10:20 |
openstackgerrit | Merged openstack/mogan master: Revert "Add ComputeDisk object" https://review.openstack.org/475215 | 10:29 |
openstackgerrit | Merged openstack/mogan master: Tempest: add tempest test cases for console https://review.openstack.org/474498 | 10:30 |
openstackgerrit | Merged openstack/mogan master: Tempest: add tempest test cases for server networks API https://review.openstack.org/474829 | 10:30 |
*** Kevin_Zheng has quit IRC | 10:31 | |
zhenguo | https://blueprints.launchpad.net/mogan/+spec/track-resources-using-placement | 11:11 |
zhenguo | shaohe_feng: are you still around? | 11:21 |
liusheng | zhenguo: hi zhenguo, are you working on importing placement ? | 11:31 |
zhenguo | liusheng: not yet, just registered a bp | 11:31 |
liusheng | zhenguo: oh, I am going to have a try :D | 11:32 |
zhenguo | liusheng: hah, great | 11:32 |
zhenguo | liusheng: please assign the bp to you, hah | 11:33 |
liusheng | it is ok, we can work together, do we need a spec ? | 11:33 |
zhenguo | liusheng: sure, we do need a spec | 11:34 |
zhenguo | liusheng: maybe I can help to draft a spec, and you start the POC? | 11:35 |
liusheng | zhenguo: ok, I am try to do some draft code test firstly | 11:35 |
liusheng | zhenguo: yes, I am starting the PoC | 11:35 |
liusheng | zhenguo: lol | 11:35 |
zhenguo | liusheng: lol | 11:35 |
zhenguo | liusheng: ncie | 11:35 |
* zhenguo away | 11:52 | |
openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test for server floatingip association/disassociation https://review.openstack.org/475302 | 11:55 |
*** litao__ has quit IRC | 12:08 | |
openstackgerrit | Merged openstack/mogan master: Adds `disabled` field for flavors. https://review.openstack.org/474441 | 12:09 |
openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test cases for server interface attaching https://review.openstack.org/474835 | 12:12 |
openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test for server floatingip association/disassociation https://review.openstack.org/475302 | 12:15 |
openstackgerrit | Merged openstack/mogan master: Add `extra_specs` to flavor create JSON schema https://review.openstack.org/475232 | 12:21 |
*** liusheng has quit IRC | 16:31 | |
*** liusheng has joined #openstack-mogan | 16:32 | |
*** zhangyang_ has joined #openstack-mogan | 18:12 | |
*** zhangyang has quit IRC | 18:19 | |
*** zhangyang_ is now known as zhangyang | 18:19 | |
*** Xinran has quit IRC | 18:21 | |
*** Xinran has joined #openstack-mogan | 18:22 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!