Thursday, 2019-09-12

openstackgerritShogo Saito proposed openstack/python-cyborgclient master: v2 API Support(using openstack-sdk)
openstackgerritYumengBao proposed openstack/cyborg master: Implement privsep boilerplate in cyborg.
*** TxGirlGeek has quit IRC01:45
*** s_shogo has joined #openstack-cyborg02:28
*** chenke has joined #openstack-cyborg02:30
*** chunxiu has joined #openstack-cyborg02:51
*** changzhi has joined #openstack-cyborg02:53
*** xinranwang has joined #openstack-cyborg02:57
*** Yumeng has joined #openstack-cyborg03:01
*** Sundar has joined #openstack-cyborg03:02
SundarHi all03:03
*** wangzhh has joined #openstack-cyborg03:03
chenkeHi ~03:03
wangzhhHi all.03:03
s_shogoHi all03:04
SundarHi chenke, wangzhh, s_shogo03:04
xinranwangHi all03:04
Sundar#startmeeting openstack-cyborg03:04
openstackMeeting started Thu Sep 12 03:04:35 2019 UTC and is due to finish in 60 minutes.  The chair is Sundar. Information about MeetBot at
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.03:04
*** openstack changes topic to " (Meeting topic: openstack-cyborg)"03:04
openstackThe meeting name has been set to 'openstack_cyborg'03:04
wangzhhHi Sundar.03:04
SundarThis is the week of Train-3 milestone.03:05
SundarWe'll look at the status, pending patches and future plans. Does anybody have any opens to be addressed?03:06
Sundar#topic Status so far03:06
*** openstack changes topic to "Status so far (Meeting topic: openstack-cyborg)"03:06
s_shogomaybe, py3 patch.03:06
Yumeng#info Yumeng03:07
SundarGood news: We got nearly all the nova-integ patches merged. Only the Nova notification patch remains.03:07
YumengThanks Sundar for your great efforts03:07
SundarIMHO, that is close too. We have some cleanup to do in the way we connect to other services. But we can fix that after this week too.03:08
SundarYumeng: Thanks. All your contributions helped too.03:08
SundarThe next most important things after Nova notifn would be tempest ( and Python 3.03:09
SundarDO you al agree? Other patches are important too, but they cna be merged after this week too03:09
xinranwanghow about async bind patch?03:10
SundarThat also can come in after this week, IMHO. I don;t think it is ready to merge in next 2 days.03:11
SundarNot so good news: It has been an uphill climb to get reviews from nova developers. The view I am hearing is that Nova patches for Cyborg need more sustained review before they can merge.03:12
SundarSo far, the fact that most patches haven;t merged was a reason why reviewes were not getting done. Also, tempest CI was a prereq. We are very close to getting them all done , I think.03:13
SundarAny thoughts or comments?03:13
SundarIt seems to me that Python 3 patch from Shogo is also close. Has nybody checked it out in their local env?03:14
s_shogoI checked py3 patch with latest version(P9 merged), it works well.03:15
Sundars_shogo: Great. I'll also try it just in case. I'd invite others to check too. Do you think we can merge it by Friday?03:16
s_shogoSundar: I think so, there seems to be no big issue.03:17
SundarGreat. BTW, here are my Cycle highlights for Train. They include Python 3 too.03:19
Sundar#topic Tempest03:19
*** openstack changes topic to "Tempest (Meeting topic: openstack-cyborg)"03:19
SundarWe are supposed to show the tempest CI as a prerequisite, like I said.03:20
SundarYumeng: thanks a lot for all that you and your team have done. Is  critical to merge this week?03:20
SundarDefinitely, we need to merge soon.03:21
Yumengsundar: i will update today. I think we should merge asap03:22
Yumengagreed . also 66723103:22
SundarGreat. Glad to see your progress on privsep - but I think we can even do that in the next couple of weeks.03:22
SundarNo need to burn yourself out :)03:23
YumengEric and sean suggest we we don't need to have a cyborg.privsep.* sub module at all03:23
Yumengjust a top level cyborg/ module that init the context and the privileged function should be in normal modules with privileged appended as a prefix or suffix.03:23
Yumengbut the current fact is nova and cinder still have cyborg.privsep.* sub module03:24
Yumengso do we just follow the current way or change it ? what do you think?03:24
xinranwangYumeng:  should we merge 676665 before 667231?03:24
SundarYes. I have heard it be said even nova's use of rootwrap etc. is not a good template to follow. Sometimes we are expected to do better than than the legacy projects because we are new.03:25
SundarYumeng: IMHO, it is better to take some time and do this right, because it will be tough to revisit this again. So, it may be better to follow the experienced input.03:26
Yumengyes, pleas merge 676665 before 66723103:26
YumengSundar: ok. then I will reconsider it. do as they suggest.03:28
SundarThanks a lot, Yumeng.03:28
SundarYumeng: Re. 676665, There is a -1 on that. What does it take to get it resolved?03:28
YumengI will do that03:28
SundarExcellent. Thanks :)03:29
SundarWhat should we do about ?03:29
Sundar(That is Python 3. I jumped to a different topic.)03:30
Yumengand to all, please help to review and merge this bug fix
s_shogoThat was included in the py3 patch.
chenkeYe. That was  part of s_shogo's patch.03:31
SundarSo, it should just be abandoned, I suppose.03:31
s_shogoSundar: That's right.03:32
SundarYumeng, wangzhh: Re., Has GPU been tested with the nova-integ patches?03:32
YumengI have tested with part of the nova-integ patches.03:35
wangzhhSorry, I don't have GPU env. But I think this part should be OK. Because there is little change this part.03:35
wangzhhYumeng, Great.03:35
SundarOk. Has anybody launched a VM with real GPU attached? Oherise, how do we know it works?03:35
Yumengwangzhh: yes a tiny fix. pls help to review \0/03:36
*** shaohe_feng has joined #openstack-cyborg03:36
shaohe_fenghi all03:36
chenkehi shaohe.03:37
SundarHi shaohe_feng03:37
*** chunxiu has quit IRC03:37
SundarTo summarize the most important patches for this week:03:37
SundarNova notification:
SundarPython 3:
SundarTempest: ,
YumengSundar: emmmm... I tested in a devstack env. not a real GPU env03:37
SundarDo we all agree that these are the important patches to merge this week?03:38
chenkeZhurong has some suggestions on Nova notification, but I suggest we merge this patch first, and then have time to modify it.03:40
Sundarchenke: Totally agreed. Zhurong's point is important and I am working on it.03:40
Sundarchenke: Thanks to you and zhurong for lots of active contribution! It'll be good to merge your patches by next week, IMHO, to help further development.03:40
Sundar#topic AoB03:41
*** openstack changes topic to "AoB (Meeting topic: openstack-cyborg)"03:41
SundarAnything else, folks?03:41
shaohe_fengwill be test on really evn today03:42
SundarCool, thanks Shaohe.03:42
shaohe_fengstill some bug, I think it can works today.03:42
shaohe_fengit is some  complex.03:43
shaohe_fengbut it implemented as we discussion.03:43
shaohe_fengwelcome RFC.03:43
SundarSure. All, please review and help merge the important patches above. Have a good day!03:44
wangzhhDo we need to merge this patch before nova integrate?
Sundarwangzhh: Nova developers have not asked for that. They tend to take security for granted, so this may yet come up. It is important to merge that too.03:45
shaohe_fengwe need this patch, but maybe nova integrate does not need it .03:46
*** changzhi has quit IRC03:46
wangzhhFIne, so plz also help to review this patch.03:46
SundarSure, will do03:46
shaohe_fengwhen is the dead line of async job?03:47
shaohe_fengit it really not easy to test it on really env.03:47
shaohe_fengtoo many steps03:48
Sundarshaohe_feng: Given that this week is milestone 3, we have to see how things go from here. I agree with you. It is better to take time and ensure it works well.03:48
SundarIt does not have to merge this week, if that makes you feel better :)03:49
shaohe_fengOK, I will do my best to make it works well today.03:49
SundarGreat. Thanks, shaohe_feng03:49
shaohe_fengfor tomorrow is our Moon festival03:49
SundarI see. Happy Moon festival to all of you!03:50
SundarAll right, see you all next week. Bye :)03:51
*** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)"03:51
openstackMeeting ended Thu Sep 12 03:51:09 2019 UTC.  Information about MeetBot at . (v 0.1.4)03:51
openstackMinutes (text):
*** Sundar has quit IRC03:51
*** chenke has quit IRC03:53
*** s_shogo has left #openstack-cyborg04:10
*** links has joined #openstack-cyborg04:34
*** efried_zzz has quit IRC05:34
*** TxGirlGeek has joined #openstack-cyborg06:15
*** TxGirlGeek has quit IRC06:17
openstackgerritYumengBao proposed openstack/cyborg master: add cyborg-tempest job and remove cyborg-devstack job
*** xinranwang has quit IRC07:56
*** wangzhh has quit IRC08:03
*** links has quit IRC08:18
*** links has joined #openstack-cyborg08:20
openstackgerritYumengBao proposed openstack/cyborg master: Implement privsep boilerplate in cyborg.
openstackgerritYumengBao proposed openstack/cyborg master: Implement privsep boilerplate in cyborg.
*** efried has joined #openstack-cyborg09:00
*** FlorianFa has quit IRC09:13
*** links has quit IRC09:40
*** links has joined #openstack-cyborg09:46
*** Yumeng has quit IRC09:47
openstackgerritYumengBao proposed openstack/cyborg master: Implement privsep boilerplate in cyborg.
*** efried is now known as efried_afk13:36
*** links has quit IRC14:24
*** TxGirlGeek has joined #openstack-cyborg16:03
*** efried_afk is now known as efried16:56
openstackgerritMerged openstack/cyborg master: add cyborg-tempest job and remove cyborg-devstack job
*** TxGirlGeek has quit IRC19:17
*** TxGirlGeek has joined #openstack-cyborg19:17
*** TxGirlGeek has quit IRC19:27
*** TxGirlGeek has joined #openstack-cyborg19:52
*** TxGirlGeek has quit IRC19:54
*** TxGirlGeek has joined #openstack-cyborg19:56
*** TxGirlGeek has quit IRC20:45
*** TxGirlGeek has joined #openstack-cyborg20:59
*** dustinc has joined #openstack-cyborg21:23
*** efried has quit IRC21:50
*** efried has joined #openstack-cyborg21:52
*** TxGirlGeek has quit IRC23:38

Generated by 2.15.3 by Marius Gedminas - find it at!