| *** wanghao has joined #openstack-mogan | 00:43 | |
| *** litao__ has joined #openstack-mogan | 01:01 | |
| zhenguo | morning mogan! | 01:02 |
|---|---|---|
| litao__ | morning | 01:02 |
| openstackgerrit | liusheng proposed openstack/mogan master: Fix the config sample generation https://review.openstack.org/491742 | 01:10 |
| zhenguo | liusheng, shaohe_feng, wanghao, litao__, Xinran, luyao, zhangyang: please have a look at the Mogan PTL candidacy mail and help to cast your +1/-1 vote there, and if you also want to do self nominations, it should be done by tomorrow, thanks! | 01:22 |
| liusheng | zhenguo: ok | 01:23 |
| litao__ | zhenguo: OK | 01:23 |
| litao__ | zhenguo: The destack gateway failed? | 01:39 |
| zhenguo | litao__: no | 01:40 |
| litao__ | zhenguo: ok， I recheck | 01:41 |
| zhenguo | litao__: ok | 01:41 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add aggregates filters when do scheduling https://review.openstack.org/491695 | 01:46 |
| zhenguo | liusheng: do you know how to handle default az on scheduling in nova? | 01:47 |
| liusheng | zhenguo: seems a config option, let me ask | 01:48 |
| zhenguo | liusheng: yes, there's a config option, but how do consider it when scheduling | 01:49 |
| liusheng | zhenguo: there is a az filter to ckeck that. and will use the default value if didn't set | 01:50 |
| zhenguo | liusheng: not sure how can we achieve that | 01:51 |
| zhenguo | liusheng: as we need to pass member_of filter to list rps, not default az aggregate there | 01:51 |
| zhenguo | liusheng: I am a bit confused with default az, and find it may bring more confusion to users, how about just getting rid of it | 01:53 |
| liusheng | zhenguo: I am just thinking about that | 01:53 |
| zhenguo | liusheng: if cloud operators don't set az, it should be None | 01:53 |
| liusheng | zhenguo: I think it due to what's the meaning we want to give to az | 01:54 |
| zhenguo | liusheng: yes | 01:55 |
| liusheng | zhenguo: do you think if Nova will move az implemenation to placement based on rp traits ? | 01:56 |
| liusheng | zhenguo: once placement get nature | 01:56 |
| zhenguo | liusheng: seems not | 01:56 |
| zhenguo | liusheng: maybe take years, hah | 01:56 |
| liusheng | zhenguo: hah, because I think az is actually a tag of a host | 01:57 |
| zhenguo | liusheng: nova moves to use aggregate metadata to represent that | 01:58 |
| liusheng | zhenguo: if we want to keep az support, what's the relationship between aggregate and az will be ? | 01:58 |
| zhenguo | liusheng: az is just a metadata of aggregate | 01:59 |
| zhenguo | liusheng: not special for us, | 01:59 |
| zhenguo | liusheng: the only difference with other aggs is that common users can get it | 02:00 |
| liusheng | zhenguo: oh, got it | 02:00 |
| zhenguo | liusheng: everything related to group in mogan will be based on aggregate | 02:00 |
| zhenguo | zhangyang: will it affect you if we get rid of defautl az | 02:01 |
| liusheng | zhenguo: so aggregate seems is the smallest unit ? | 02:01 |
| zhenguo | liusheng: yes | 02:01 |
| liusheng | zhenguo: no, just discuss its necessity. hah | 02:01 |
| liusheng | zhenguo: not sure if it has use case in the future | 02:02 |
| zhenguo | liusheng: you mean default az? | 02:02 |
| zhenguo | liusheng: yes, aggregate is the samllest uinit | 02:03 |
| liusheng | zhenguo: not only | 02:03 |
| liusheng | zhenguo: I mean the az support feature | 02:03 |
| liusheng | zhenguo: you mean get rid of defaut az, or whole az feature ? | 02:03 |
| zhenguo | liusheng: default az | 02:03 |
| liusheng | zhenguo: oh, sorry, hah | 02:04 |
| zhenguo | liusheng: hah | 02:04 |
| liusheng | zhenguo: seems it is no effect | 02:05 |
| zhenguo | liusheng: AZ is needed, at least our portal will let users choose which az they want to put the server in | 02:05 |
| liusheng | zhenguo: yes | 02:05 |
| litao__ | zhenguo: I recheck my patch and no trigger Jenkins | 02:05 |
| zhenguo | litao__: you can find it on zuul.openstack.org? | 02:06 |
| zhenguo | liusheng: ok, let me get rid of it, lol | 02:06 |
| liusheng | zhenguo: ok | 02:06 |
| zhangyang | zhenguo: actually we are still using a older version of mogan... cause we tend to do the scheduling part on our own for now... and maybe we shall put these things together after our deadline and we'll catch up with you guys... | 02:09 |
| zhenguo | zhangyang: hah, ok, we will finsih the aggregates and server group tasks soon, then you can consider to update your deployment | 02:10 |
| litao__ | zhenguo: ok | 02:11 |
| zhangyang | zhenguo: ok, now busy copying bdm from nova 😵 | 02:12 |
| zhenguo | zhangyang: we just registered a bp for that :D | 02:12 |
| zhangyang | zhenguo: hah, no too much difference from nova this part as far as i can see | 02:15 |
| zhenguo | zhangyang: if so, we can also start to copy that hah | 02:15 |
| zhangyang | zhenguo: ah...then it's so sad that we have to copy it twice 😭 | 02:16 |
| zhenguo | zhangyang: yes, that's because we are not based on the same code :( but it will be better later | 02:17 |
| zhangyang | zhenguo: too low efficency before... if i had finished these code earlier maybe things can get better... | 02:21 |
| zhenguo | zhangyang: yes, we need to improve the current situation, maybe you can do an upgrade after mogan pike release. then you can report bugs and backport patches to your branch | 02:23 |
| zhangyang | zhenguo: yes, hope things will move on nicely | 02:24 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add ServerGroup object https://review.openstack.org/488909 | 02:24 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add server groups db models and api https://review.openstack.org/488298 | 02:24 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add API tests for server group https://review.openstack.org/490328 | 02:24 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add ServerGroup API interfaces https://review.openstack.org/489850 | 02:24 |
| zhenguo | zhangyang: let's try to get there | 02:25 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Get rid of default availability zone https://review.openstack.org/491968 | 02:35 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Specify image when do rebuilding https://review.openstack.org/490421 | 02:58 |
| * litao__ zhenguo: This patch needs your advice： https://review.openstack.org/#/c/481544/ | 02:59 | |
| * litao__ zhenguo: This patch needs your advice： https://review.openstack.org/#/c/481544/ | 03:00 | |
| litao__ | test | 03:01 |
| litao__ | zhenguo: This patch needs your advice： https://review.openstack.org/#/c/481544/ | 03:01 |
| zhenguo | litao__: ok, will check it soon :D | 03:02 |
| zhenguo | litao__: hi, I just left some comments on your patch | 03:40 |
| zhenguo | litao__: why we need to go through the workflow of creating a new server? | 03:41 |
| zhenguo | litao__: for managing servers, we just need to check the specified flavor, networks available, and create a db entry, it's very simple as my understand | 03:42 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Get rid of default availability zone https://review.openstack.org/491968 | 03:50 |
| litao__ | zhenguo: see my reply | 06:15 |
| zhenguo | litao__: it's weird | 06:15 |
| zhenguo | litao__: why we need the workflow | 06:15 |
| zhenguo | litao__: just because call ironic driver api? | 06:15 |
| litao__ | zhenguo: Mainly I think it is also creating a new server actully | 06:16 |
| zhenguo | litao__: just db entry | 06:16 |
| litao__ | zhenguo: Also parameter check， quota check. etc | 06:17 |
| zhenguo | litao__: yes, you need to check the parameters, but need to check the whether it's available, different with creating server | 06:17 |
| zhenguo | litao__: like you need to check the network port existing instead of creating a new port | 06:17 |
| zhenguo | litao__: and it should be a rpc call instead of cast, as we don't have long time proces running | 06:19 |
| * litao__ zhenguo: Yes , like what you said, It is also be a standalone worklow | 06:19 | |
| litao__ | zhenguo: Yes , like what you said, It is also be a standalone worklow | 06:20 |
| litao__ | zhenguo: Maybe it is more clear | 06:20 |
| litao__ | zhenguo: I will try to refactor the codes | 06:28 |
| zhenguo | litao__: ok, thanks! | 06:28 |
| openstackgerrit | Merged openstack/mogan master: [Trivialfix]Fix typos in mogan https://review.openstack.org/491770 | 06:31 |
| openstackgerrit | liusheng proposed openstack/python-moganclient master: Add node aggregate commands support https://review.openstack.org/492002 | 06:50 |
| openstackgerrit | liusheng proposed openstack/python-moganclient master: Add node aggregate commands support https://review.openstack.org/492002 | 07:00 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add ServerGroup object https://review.openstack.org/488909 | 08:41 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add server groups db models and api https://review.openstack.org/488298 | 08:41 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add API tests for server group https://review.openstack.org/490328 | 08:41 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add ServerGroup API interfaces https://review.openstack.org/489850 | 08:41 |
| *** wanghao_ has joined #openstack-mogan | 08:44 | |
| *** wanghao has quit IRC | 08:46 | |
| *** zhuli has quit IRC | 08:47 | |
| *** kong has quit IRC | 08:47 | |
| *** dims has quit IRC | 08:47 | |
| *** Xinran has quit IRC | 08:47 | |
| *** luyao has quit IRC | 08:47 | |
| *** zhangyang has quit IRC | 08:47 | |
| *** liusheng has quit IRC | 08:47 | |
| *** shaohe_feng has quit IRC | 08:47 | |
| *** RuiChen has quit IRC | 08:47 | |
| *** lin_yang has quit IRC | 08:47 | |
| *** litao__ has quit IRC | 08:47 | |
| *** Jeffrey4l has quit IRC | 08:47 | |
| *** wxy has quit IRC | 08:47 | |
| *** openstackgerrit has quit IRC | 08:47 | |
| *** wanghao_ has quit IRC | 08:47 | |
| *** ChanServ has quit IRC | 08:47 | |
| *** Kevin_Zheng has quit IRC | 08:47 | |
| *** zhenguo has quit IRC | 08:47 | |
| *** openstack has joined #openstack-mogan | 13:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!