Thursday, 2017-08-10

*** wanghao has joined #openstack-mogan00:39
zhenguomorning mogan!00:50
liushengmorning01:23
zhenguoliusheng: would you perfer to vote for the CLI keyword during this meeting?01:26
liushengzhenguo: yes, I think it should be01:26
zhenguoliusheng: maybe we can hold on for a while, as replacing the keyword is easy :D01:27
zhenguoliusheng: I hope we all on the same page instead of voting01:27
liushengzhenguo: :)01:28
liushengzhenguo: ok01:28
zhenguoliusheng: seems openstack CLI can autocomplete01:28
liushengzhenguo: yes01:29
zhenguoliusheng: personally I still think baremachine vs baremetal seems a bit weird01:29
*** litao__ has joined #openstack-mogan01:29
liushengzhenguo: but there isn't a better choice01:30
zhenguoliusheng: how about just keep the baremetal01:30
liushengzhenguo: .. you told us it is unreasonable. lol01:30
zhenguoliusheng: use 'baremetal compute' or 'baremeal management' instead01:30
liushengzhenguo: ok01:31
liushengzhenguo: oh01:31
zhenguoliusheng: or use baremetal-compute01:31
liushengzhenguo: but that make some long commands and is not friendly to use01:31
zhenguoliusheng: it will be autocomplete01:31
liushengzhenguo: autocomplete dosn't metter01:33
*** RuiChen has joined #openstack-mogan01:37
zhenguohi all, the weekly meeting will happen soon, please move to #openstack-meeting01:59
openstackgerritliusheng proposed openstack/python-moganclient master: Add node aggregate commands support  https://review.openstack.org/49200202:56
*** wanghao has quit IRC02:58
*** wanghao has joined #openstack-mogan02:58
*** wanghao has quit IRC02:59
*** wanghao has joined #openstack-mogan02:59
*** wanghao has quit IRC03:00
*** wanghao has joined #openstack-mogan03:00
*** wanghao has quit IRC03:01
*** wanghao has joined #openstack-mogan03:01
*** wanghao has quit IRC03:01
*** wanghao has joined #openstack-mogan03:02
*** wanghao has quit IRC03:02
*** wanghao has joined #openstack-mogan03:02
*** wanghao has quit IRC03:03
*** wanghao has joined #openstack-mogan03:04
zhenguoliusheng: I would like to move baremetal dir out from engine, place it like network, image, and volume resources, https://review.openstack.org/#/c/492084/03:16
liushengzhenguo: ok, let's follow Nova :D03:18
zhenguoliusheng: hah03:18
openstackgerritZhenguo Niu proposed openstack/mogan master: Replace default_availability_zone with default_schedule_zone  https://review.openstack.org/49236603:39
zhenguoliusheng: for the server group API schema, I find the policies is an array but seems we can only specify one rule from affinity and anti-affinity now, that's reserved for future we may introduce more policies, right?03:50
liushengzhenguo: hmm... seems I was careless, since Nova have other policies support, and I only stoled from Nova but just remove other policies..03:52
zhenguoliusheng: hah, but maybe it's ok to keep it03:52
zhenguoliusheng: for future use03:52
liushengzhenguo: ok03:53
* zhenguo brb03:53
openstackgerritliusheng proposed openstack/python-moganclient master: Add commands for aggregate node actions  https://review.openstack.org/49237004:12
openstackgerritZhenguo Niu proposed openstack/mogan master: Remove rp aggregate when the aggregate get removed  https://review.openstack.org/49206305:03
*** litao__ has quit IRC05:19
openstackgerritZhenguo Niu proposed openstack/mogan master: Replace default_availability_zone with default_schedule_zone  https://review.openstack.org/49236605:20
liushengzhenguo: hi did you found the mail from Tony Breeds ? do we need a pike release ?06:25
zhenguoliusheng: yes, we do06:38
zhenguoliusheng: we should create the branch soon?06:45
liushengzhenguo: seems need to release recently06:45
zhenguoliusheng: how? not sure why we are listed there as a non-official project06:46
liushengzhenguo: as the mail mentioned we can use the tool to create a stable branch in openstack/releases06:46
liushengzhenguo: I am not sure, but seems it don't has the limitation06:46
zhenguoliusheng: but we haven't finish our work now :(06:48
liushengzhenguo: yes, I also concern about that :(06:48
liushengzhenguo: but I guess it is ok we don't release now06:49
openstackgerritliusheng proposed openstack/mogan master: Remove unused methods in common/paths and common/rpc  https://review.openstack.org/49239706:53
zhenguoliusheng: seems they will cut the requirements soon for pike, so if we don't create the branch, we will not get requirements update for pike06:54
openstackgerritliusheng proposed openstack/mogan master: Remove unused methods in common/paths and common/rpc  https://review.openstack.org/49239706:54
liushengzhenguo: seems yes06:54
zhenguoliusheng: seems they will cut requirements on Monday06:57
liushengzhenguo: if we release later, does it matter ?06:57
zhenguoliusheng: not sure, we will not get update from requirements for Pike?06:58
liushengzhenguo: may need to make change in requirement repo06:59
zhenguoliusheng: not sure if we can change the stable branch07:00
zhenguoliusheng: do you think it's ok if we don't get requirements update for pike branch?07:02
zhenguoliusheng: seems it's not matter for our first release, haha07:02
liushengzhenguo: yes, I dont' think anyone will use our pike release in production. hah07:03
zhenguoliusheng: hah, so it's ok, and we can add the requirements manually if there's big issue07:04
liushengzhenguo: ok07:04
zhenguoliusheng: I think we are listed on the email just because we subscribed the requirements07:05
zhenguoliusheng: they just kindly remind us07:05
liushengzhenguo: yes, the mail mentioned07:05
liushengzhenguo: yes07:05
zhenguoliusheng: I need this https://review.openstack.org/#/c/492366/ landed first before making aggregates filter happen on scheduling.07:08
liushengzhenguo: ok, seems we lack of core reviewer now :(07:08
zhenguoliusheng: very07:09
zhenguoliusheng: shaohe_feng will come back to be active soon07:09
liushengzhenguo: ok, good news07:09
zhenguoliusheng, shaohe_feng: as cores, let's try to go through all patches in mogan, python-moganclient, and mogan-specs everyday!07:11
zhenguoliusheng, shaohe_feng: seems it's hard, lol07:12
liushenghah07:12
liushengzhenguo: about your above patch, do you think if we should put the config option into mogan/conf/schedluer.py than engine.py ?07:13
zhenguoliusheng: in nova, it's in compute07:14
zhenguoliusheng: and seems it's make sense, as we need to set it to server availabiilty zone07:14
liushengzhenguo: nova has a availability_zone.py07:14
zhenguoliusheng: but that option is in compute conf07:14
zhenguoliusheng: we used it on engine, and set it to server, then just pass the value to scheduler07:15
liushengzhenguo: seems it is mainly used in scheduler..07:16
zhenguoliusheng: I dont' mean the value used for07:16
liushengzhenguo: oh07:17
zhenguoliusheng: we extract the option in engine07:17
liushengzhenguo: ok07:17
zhenguoliusheng: hah07:17
zhenguoliusheng: seriously, although it's hard to go through patches everyday, but we should try, at least during theses days before pike release, that makes mogan moves along07:19
liushengzhenguo: ok, sure07:19
zhenguoliusheng: thanks, I will talk to shaohe_feng later07:20
liushengzhenguo: ok07:20
openstackgerritZhenguo Niu proposed openstack/mogan master: Replace default_availability_zone with default_schedule_zone  https://review.openstack.org/49236607:22
zhenguoliusheng: wrt https://review.openstack.org/#/c/492002/ , you will add aggregate set/unset support in a following up patch, right?07:53
liushengzhenguo: you mean the aggregate node comands ?07:55
zhenguoliusheng: no, I mean aggregate update07:55
liushengzhenguo: https://review.openstack.org/#/c/492370/07:55
liushengzhenguo: oh, let me check07:55
zhenguoliusheng: name and metadata update with set/unset07:55
liushengzhenguo: ok, I will add it to this patch07:56
zhenguoliusheng: ok ,thanks07:56
openstackgerritZhenguo Niu proposed openstack/mogan master: Do not allow update flavor internal attributes  https://review.openstack.org/49243808:56
zhenguoliusheng: just find we have nics attribute in server, but there's no cpu and memory there09:12
liushengzhenguo: do we need to add that ?09:26
zhenguoliusheng: for a server object, seems it's the basic information09:26
liushengzhenguo: but we dicussed to avoid exposing that info to end users09:27
liushengzhenguo: only flavor resource class09:27
zhenguoliusheng: yes,09:27
zhenguoliusheng: so users still need to get the flavor to see the description09:28
zhenguoliusheng: we use resource class for scheuling doesn't mean we dont' need to show basic properties to users09:29
zhenguoliusheng: wrt nics, I think it should not be added with neutron network09:31
zhenguoliusheng: a server has fixed nics, tenant network is just some attribute of the nic09:31
zhenguoliusheng: it's a nic instead of network09:32
liushengzhenguo: sorry, I am busy in a nother thing now :(10:08
zhenguoliusheng: hah10:08
openstackgerritZhenguo Niu proposed openstack/mogan master: Make disabled flavors not available for common users  https://review.openstack.org/49248811:51
openstackgerritMerged openstack/mogan master: Remove unused methods in common/paths and common/rpc  https://review.openstack.org/49239713:31
openstackgerritMerged openstack/mogan master: Remove rp aggregate when the aggregate get removed  https://review.openstack.org/49206313:43
openstackgerritMerged openstack/mogan master: Normalize objects file name  https://review.openstack.org/49209213:43
openstackgerritMerged openstack/mogan master: Improve resources updating process  https://review.openstack.org/48735113:43
openstackgerritMerged openstack/mogan master: Fix the config sample generation  https://review.openstack.org/49174213:44
openstackgerritMerged openstack/mogan master: Add server groups db models and api  https://review.openstack.org/48829813:54
openstackgerritMerged openstack/mogan master: Add ServerGroup object  https://review.openstack.org/48890913:55
openstackgerritMerged openstack/mogan master: Add ServerGroup API interfaces  https://review.openstack.org/48985013:55
openstackgerritMerged openstack/mogan master: Add API tests for server group  https://review.openstack.org/49032813:58
*** wanghao has quit IRC18:13
*** wanghao has joined #openstack-mogan18:21
*** openstackgerrit has quit IRC19:03

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