Thursday, 2017-08-03

*** wanghao has joined #openstack-mogan00:47
zhenguomorning mogan!00:56
*** litao__ has joined #openstack-mogan00:56
zhenguoliusheng: hi, please add your server group patches to https://etherpad.openstack.org/p/MoganWhiteBoard01:08
liushengzhenguo: ok01:08
openstackgerritliusheng proposed openstack/mogan master: Remove the unused BaremetalAggregateClient class  https://review.openstack.org/49026701:26
openstackgerritXinran WANG proposed openstack/mogan master: Don't return unuseful BM flavor info to common users.  https://review.openstack.org/48955302:14
zhenguoliusheng, shaohe_feng: please also do some reviews on specs and client, thanks!03:01
liushengzhenguo: ok03:03
openstackgerritXinran WANG proposed openstack/mogan master: Don't return unuseful BM flavor info to common users.  https://review.openstack.org/48955303:04
wanghaozhenguo: sorry,  I left for a while.   Since there is no constraint to use manage or adopt,  I just think both is ok03:04
zhenguowanghao: how about just follow cinder03:05
wanghaozhenguo: of course,  Cinder use manage/unmanage,  and also list managable volumes03:05
zhenguowanghao: hah03:05
wanghaozhenguo: but Ironic use adopt03:05
wanghaozhenguo: I'm ok with both TBH.03:06
zhenguowanghao: ironic is not for users, hah03:06
wanghaozhenguo: emm,  I'm ok with manage03:07
wanghaozhenguo: I will update the spec.03:07
zhenguowanghao: ok, thanks03:07
wanghaozhenguo: seems in the first patches of spec,  I use 'manage', hahaha.03:08
zhenguowanghao: lol03:08
openstackgerritXinran WANG proposed openstack/mogan master: Don't return unuseful BM flavor info to common users.  https://review.openstack.org/48955303:09
* zhenguo brb03:09
wanghaozhenguo: sometime, when you go to look back,  you just find the first is the best. : )03:09
zhenguowanghao: hah,03:21
zhenguoliusheng: we will not provide server group update API ,right?03:24
liushengzhenguo: just didn't fond Nova provide that03:25
zhenguoliusheng: hah, yes, seem we don't need that03:25
liushengzhenguo: hah03:25
openstackgerritwanghao proposed openstack/mogan-specs master: Change adopt to manage in managing-existing-node spec  https://review.openstack.org/49031003:28
openstackgerritliusheng proposed openstack/mogan master: Add API tests for server group  https://review.openstack.org/49032804:06
*** wxy has quit IRC04:51
openstackgerritwanghao proposed openstack/mogan-specs master: Change adopt to manage in managing-existing-node spec  https://review.openstack.org/49031005:50
wanghaohi guys,  I have updated the spec according to our discussion in meeting, plz have a review.05:51
*** dims has quit IRC05:52
*** dims has joined #openstack-mogan05:54
zhenguowanghao: ok, thanks06:08
openstackgerritliusheng proposed openstack/mogan master: Add API tests for server group  https://review.openstack.org/49032806:14
zhenguowanghao: https://www.youtube.com/watch?v=0d6b2RO8WFw , it's quite like our managing running servers06:14
* zhenguo brb06:22
*** wanghao has quit IRC06:22
*** wanghao has joined #openstack-mogan06:22
wanghaozhenguo: yes,  it adopt the bm into Nova.06:26
wanghaozhenguo: seems they use a tool kit and fake driver in Ironic to finish the job.06:43
zhenguowanghao: seems a fake driver in nova when booting the server, then translate to their own driver to manage ironic06:44
*** wanghao_ has joined #openstack-mogan06:44
zhenguothey can also support graphical console06:46
*** wanghao has quit IRC06:48
openstackgerritliusheng proposed openstack/mogan master: Add API tests for server group  https://review.openstack.org/49032807:01
*** wanghao has joined #openstack-mogan07:05
*** wanghao_ has quit IRC07:05
*** wanghao_ has joined #openstack-mogan07:09
*** wanghao has quit IRC07:13
openstackgerritMerged openstack/mogan master: Don't return unuseful BM flavor info to common users.  https://review.openstack.org/48955307:51
*** openstackgerrit has quit IRC08:18
zhenguoliusheng: seems ironic PTL +2ed your patch08:44
liushengzhenguo: oh, the patch is stuck a long time. hah08:50
zhenguoliusheng: hah, but seems we don't relay on that now08:51
liushengzhenguo: not very sure .. but we can install ironic without Nova08:51
zhenguoliusheng: hah08:51
*** wxy has joined #openstack-mogan08:53
*** dims has quit IRC09:04
*** dims has joined #openstack-mogan09:08
*** openstackgerrit has joined #openstack-mogan09:55
openstackgerritZhenguo Niu proposed openstack/mogan master: Add checks for aggregate availability_zone  https://review.openstack.org/48987609:55
*** wanghao has joined #openstack-mogan10:38
*** wanghao_ has quit IRC10:41
openstackgerritXinran WANG proposed openstack/mogan master: Specify image when do rebuilding  https://review.openstack.org/49042110:53
dimszhenguo : folks : is mogan having an election too? at least a PTL nomination like cyborg - http://lists.openstack.org/pipermail/openstack-dev/2017-August/120550.html11:04
zhenguodims: yes sir, thanks for the kindly reminder :)11:06
*** wanghao has quit IRC11:14
openstackgerritXinran WANG proposed openstack/mogan master: Specify image when do rebuilding  https://review.openstack.org/49042111:14
*** wanghao has joined #openstack-mogan11:14
*** wanghao has quit IRC11:21
*** wanghao has joined #openstack-mogan11:21
*** wanghao has quit IRC11:26
*** litao__ has quit IRC11:59
* zhenguo away12:21
dimszhenguo : i left some tips for tagging release on the cyborg channel. talk to zhipeng please as well, since both cyborg and mogan are in the same boat12:48
dimszhenguo : please join #openstack-release channel and we can talk more there if you have any questions12:49
*** harlowja has quit IRC19:57
*** harlowja has joined #openstack-mogan21:15

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