| zhenguo | morning mogan! | 00:12 |
|---|---|---|
| *** wanghao has joined #openstack-mogan | 00:39 | |
| *** openstackgerrit has joined #openstack-mogan | 00:45 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/mogan master: Updated from global requirements https://review.openstack.org/491291 | 00:45 |
| openstackgerrit | wanghao proposed openstack/mogan master: Manage existing BMs: Part-1 https://review.openstack.org/479660 | 00:58 |
| wanghao | morning, mogan! | 00:59 |
| *** litao__ has joined #openstack-mogan | 01:28 | |
| *** liusheng has joined #openstack-mogan | 01:47 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Fix aggregate metadata update/remove error https://review.openstack.org/491313 | 01:58 |
| openstackgerrit | Tao Li proposed openstack/mogan master: Manage existing BMs: Part-2 https://review.openstack.org/481544 | 02:09 |
| *** liusheng has quit IRC | 03:06 | |
| *** liusheng has joined #openstack-mogan | 03:07 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Fix aggregate metadata update/remove error https://review.openstack.org/491313 | 03:24 |
| openstackgerrit | Tao Li proposed openstack/mogan master: Manage existing BMs: Part-2 https://review.openstack.org/481544 | 04:22 |
| openstackgerrit | Merged openstack/mogan master: Updated from global requirements https://review.openstack.org/491291 | 04:37 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Specify image when do rebuilding https://review.openstack.org/490421 | 06:10 |
| openstackgerrit | wanghao proposed openstack/mogan master: Manage existing BMs: Part-1 https://review.openstack.org/479660 | 06:15 |
| openstackgerrit | wanghao proposed openstack/mogan master: Introduce Cinder client into Mogan https://review.openstack.org/489455 | 06:38 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Specify image when do rebuilding https://review.openstack.org/490421 | 07:00 |
| openstackgerrit | wanghao proposed openstack/mogan master: Introduce Cinder client into Mogan https://review.openstack.org/489455 | 07:06 |
| openstackgerrit | wanghao proposed openstack/mogan master: Manage existing BMs: Part-1 https://review.openstack.org/479660 | 07:08 |
| openstackgerrit | Tao Li proposed openstack/mogan master: Manage existing BMs: Part-2 https://review.openstack.org/481544 | 07:24 |
| wanghao | zhenguo: hi zhenguo, is there any issues more in changes in managing-existing-bms spec? | 07:37 |
| openstackgerrit | wanghao proposed openstack/mogan-specs master: Change adopt to manage in managing-existing-node spec https://review.openstack.org/490310 | 07:39 |
| openstackgerrit | Merged openstack/python-moganclient master: Enable some off-by-default checks https://review.openstack.org/477875 | 07:45 |
| openstackgerrit | Merged openstack/python-moganclient master: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/489269 | 07:46 |
| liusheng | wanghao, litao__ I prefer you make a dependency between your two patches, sinces there is overlap between your paches and will make conflicts. e.g. both of your patches added the API controller | 07:53 |
| zhenguo | wanghao: sorry, I was busy with some internal stuff, will go through it soon :D | 08:06 |
| wanghao | zhenguo: thanks | 08:29 |
| wanghao | liusheng: yes, we will merge it when one of us got merged. | 08:29 |
| wanghao | liusheng: Since we didn't sure which one will get merged first, so we both added the controller but it will be fixed when merge one of them. | 08:31 |
| liusheng | wanghao: why don't just let one depends on another one. | 08:32 |
| wanghao | liusheng: well it could, but just think now is fine too. | 08:38 |
| wanghao | liusheng: it's simple to fix the conflicts later. | 08:38 |
| liusheng | wanghao: ok, well.. just found there is overlap and make them dependent will make it friendly to review. | 08:52 |
| wanghao | liusheng: hah, I will keep same with litao__ | 08:53 |
| wanghao | liusheng: or guys can merge my patch first, it's simpler then litao__'s patch. : ) | 08:54 |
| liusheng | wanghao: reviewing :) | 08:54 |
| wanghao | liusheng: thanks | 08:54 |
| litao__ | liusheng: come on, reiew more and more ：） | 08:57 |
| litao__ | s/reiew/review/ | 08:58 |
| litao__ | liusheng, wanghao： I will commit another patch for managing server docs | 09:01 |
| liusheng | litao__: ok, thanks | 09:01 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Fix a typo https://review.openstack.org/491406 | 09:07 |
| liusheng | zhenguo: hi, have you finished you busy stuff ? :D | 09:14 |
| zhenguo | liusheng: hah | 09:31 |
| zhenguo | liusheng: just sent a test patch, our boss wants to learn how to contribute codes :D | 09:31 |
| liusheng | zhenguo: hah, which boss ? | 09:32 |
| zhenguo | liusheng: big boss | 09:32 |
| liusheng | zhenguo: lol | 09:32 |
| zhenguo | liusheng: we will get a new contributor soon | 09:33 |
| liusheng | zhenguo: just want to have a quick talk about replacing the "baremetal" in CLI, wanghao litao__ | 09:33 |
| liusheng | zhenguo: cool | 09:34 |
| zhenguo | liusheng: which keyword do you perfer | 09:35 |
| liusheng | zhenguo: baremachine, hah, but we didn't got a aggreement | 09:35 |
| zhenguo | liusheng: hah | 09:36 |
| liusheng | zhenguo: so that I can add commands about nodes, aggregates and others | 09:36 |
| zhenguo | liusheng: if we just keep the keyword 'baremetal' seems we only get a collision about nodes | 09:37 |
| zhenguo | liusheng: openstack baremetal compute nodes list? | 09:37 |
| liusheng | zhenguo: yes, and as you mentioned, it is easy to confuse users to consider we are Ironic | 09:38 |
| zhenguo | liusheng: openstack baremetal aggregate list? | 09:38 |
| zhenguo | liusheng: hah | 09:38 |
| zhenguo | liusheng: 'keyword' seems easy to change | 09:38 |
| liusheng | zhenguo: not sure, I mean after we have a certain choice, I can propose other changes | 09:39 |
| liusheng | zhenguo: sure | 09:39 |
| zhenguo | liusheng: seems we can't easy get a consensus :( | 09:39 |
| zhenguo | liusheng: create a poll for that? | 09:39 |
| liusheng | zhenguo: sounds good | 09:40 |
| liusheng | zhenguo: 1. keep using baremetal, 2 baremachine, 3. baremetal compute. any other candidate ? | 09:41 |
| zhenguo | liusheng; need others' feedback | 09:41 |
| liusheng | wanghao, litao__ any toughts about ^^ ? | 09:41 |
| zhenguo | liusheng: we can vote during next weekly meeting | 09:42 |
| liusheng | zhenguo: ok | 09:42 |
| * zhenguo away | 09:43 | |
| openstackgerrit | Merged openstack/mogan master: Add resource aggregates filed to flavor https://review.openstack.org/489151 | 16:00 |
| *** openstackstatus has quit IRC | 16:42 | |
| *** openstackstatus has joined #openstack-mogan | 16:44 | |
| *** ChanServ sets mode: +v openstackstatus | 16:44 | |
| *** litao__ has quit IRC | 18:17 | |
| openstackgerrit | Merged openstack/mogan master: Fix aggregate metadata update/remove error https://review.openstack.org/491313 | 19:54 |
| *** wanghao has quit IRC | 20:04 | |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org will be offline momentarily at 00:00 utc for a quick reconfiguration-related restart | 23:33 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!