03:04:35 <Sundar> #startmeeting openstack-cyborg
03:04:36 <openstack> Meeting 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 http://wiki.debian.org/MeetBot.
03:04:37 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
03:04:40 <openstack> The meeting name has been set to 'openstack_cyborg'
03:04:46 <wangzhh> Hi Sundar.
03:04:55 <chenke> \0
03:05:24 <Sundar> This is the week of Train-3 milestone.
03:06:01 <Sundar> We'll look at the status, pending patches and future plans. Does anybody have any opens to be addressed?
03:06:33 <Sundar> #topic Status so far
03:06:44 <s_shogo> maybe, py3 patch.
03:07:04 <Yumeng> #info Yumeng
03:07:14 <Sundar> Good news: We got nearly all the nova-integ patches merged. https://review.opendev.org/#/q/project:openstack/cyborg+branch:master+topic:nova-integ Only the Nova notification patch remains.
03:07:59 <Yumeng> Thanks Sundar for your great efforts
03:08:05 <Sundar> IMHO, 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:29 <Sundar> Yumeng: Thanks. All your contributions helped too.
03:09:15 <Sundar> The next most important things after Nova notifn would be tempest (https://review.opendev.org/#/c/667231/) and Python 3.
03:09:36 <Sundar> DO you al agree? Other patches are important too, but they cna be merged after this week too
03:10:26 <xinranwang> how about async bind patch?
03:11:10 <Sundar> That also can come in after this week, IMHO. I don;t think it is ready to merge in next 2 days.
03:12:10 <Sundar> Not 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:13:11 <Sundar> So 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:34 <Sundar> Any thoughts or comments?
03:14:21 <Sundar> It seems to me that Python 3 patch from Shogo is also close. Has nybody checked it out in their local env?
03:15:21 <s_shogo> I checked py3 patch with latest version(P9 merged), it works well.
03:16:19 <Sundar> s_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:17:52 <s_shogo> Sundar: I think so, there seems to be no big issue.
03:18:38 <chenke> +1
03:19:15 <Sundar> Great. BTW, here are my Cycle highlights for Train. https://review.opendev.org/#/c/681283/ They include Python 3 too.
03:19:40 <Sundar> #topic Tempest
03:20:23 <Sundar> We are supposed to show the tempest CI as a prerequisite, like I said.
03:20:59 <Sundar> Yumeng: thanks a lot for all that you and your team have done. Is https://review.opendev.org/#/c/676665/  critical to merge this week?
03:21:58 <Sundar> Definitely, we need to merge https://review.opendev.org/#/c/667231/ soon.
03:22:10 <Yumeng> sundar: i will update today. I think we should merge asap
03:22:47 <Yumeng> agreed . also 667231
03:22:55 <Sundar> Great. Glad to see your progress on privsep - but I think we can even do that in the next couple of weeks.
03:23:12 <Sundar> No need to burn yourself out :)
03:23:23 <Yumeng> about https://review.opendev.org/#/c/673957/
03:23:50 <Yumeng> Eric and sean suggest we we don't need to have a cyborg.privsep.* sub module at all
03:23:59 <Yumeng> just a top level cyborg/privsep.py module that init the context and the privileged function should be in normal modules with privileged appended as a prefix or suffix.
03:24:29 <Yumeng> but the current fact is nova and cinder still have cyborg.privsep.* sub module
03:24:59 <Yumeng> so do we just follow the current way or change it ? what do you think?
03:24:59 <xinranwang> Yumeng:  should we merge 676665 before 667231?
03:25:07 <Sundar> Yes. 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:26:47 <Sundar> Yumeng: 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:53 <Yumeng> yes, pleas merge 676665 before 667231
03:28:01 <Yumeng> Sundar: ok. then I will reconsider it. do as they suggest.
03:28:14 <Sundar> Thanks a lot, Yumeng.
03:28:24 <Sundar> Yumeng: Re. 676665, There is a -1 on that. What does it take to get it resolved?
03:28:56 <Yumeng> I will do that
03:29:08 <Sundar> Excellent. Thanks :)
03:29:59 <Sundar> What should we do about https://review.opendev.org/#/c/675990/ ?
03:30:16 <Sundar> (That is Python 3. I jumped to a different topic.)
03:31:06 <Yumeng> and to all, please help to review and merge this bug fix https://review.opendev.org/#/c/675059/
03:31:10 <s_shogo> That was included in the py3 patch. https://review.opendev.org/#/c/667524/
03:31:45 <chenke> Ye. That was  part of s_shogo's patch.
03:31:49 <Sundar> So, it should just be abandoned, I suppose.
03:32:09 <chenke> Agree
03:32:13 <s_shogo> Sundar: That's right.
03:32:19 <Sundar> Cool.
03:32:58 <Sundar> Yumeng, wangzhh: Re. https://review.opendev.org/#/c/675059/, Has GPU been tested with the nova-integ patches?
03:35:15 <Yumeng> I have tested with part of the nova-integ patches.
03:35:24 <wangzhh> Sorry, I don't have GPU env. But I think this part should be OK. Because there is little change this part.
03:35:35 <wangzhh> Yumeng, Great.
03:35:43 <Sundar> Ok. Has anybody launched a VM with real GPU attached? Oherise, how do we know it works?
03:35:47 <Sundar> *Otherwise
03:36:30 <Yumeng> wangzhh: yes a tiny fix. pls help to review \0/
03:36:57 <shaohe_feng> hi all
03:37:13 <chenke> hi shaohe.
03:37:15 <Sundar> Hi shaohe_feng
03:37:32 <Sundar> To summarize the most important patches for this week:
03:37:41 <Sundar> Nova notification: https://review.opendev.org/674520
03:37:48 <Sundar> Python 3: https://review.opendev.org/#/c/667524/
03:37:55 <Sundar> Tempest: https://review.opendev.org/#/c/676665/ , https://review.opendev.org/#/c/667231/
03:37:57 <Yumeng> Sundar: emmmm... I tested in a devstack env. not a real GPU env
03:38:17 <Sundar> Do we all agree that these are the important patches to merge this week?
03:38:47 <Yumeng> agreed'
03:39:16 <chenke> +1
03:40:01 <chenke> Zhurong has some suggestions on Nova notification, but I suggest we merge this patch first, and then have time to modify it.
03:40:30 <Sundar> chenke: Totally agreed. Zhurong's point is important and I am working on it.
03:40:59 <Sundar> chenke: 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:41:28 <Sundar> #topic AoB
03:41:38 <Sundar> Anything else, folks?
03:42:00 <shaohe_feng> https://review.opendev.org/#/c/681005/
03:42:20 <shaohe_feng> will be test on really evn today
03:42:41 <Sundar> Cool, thanks Shaohe.
03:42:42 <shaohe_feng> still some bug, I think it can works today.
03:43:10 <shaohe_feng> it is some  complex.
03:43:22 <shaohe_feng> but it implemented as we discussion.
03:43:30 <shaohe_feng> welcome RFC.
03:44:56 <Sundar> Sure. All, please review and help merge the important patches above. Have a good day!
03:44:57 <wangzhh> Do we need to merge this patch before nova integrate? https://review.opendev.org/#/c/678177/
03:45:55 <Sundar> wangzhh: 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:46:04 <shaohe_feng> we need this patch, but maybe nova integrate does not need it .
03:46:31 <wangzhh> FIne, so plz also help to review this patch.
03:46:39 <shaohe_feng> sure,
03:46:40 <Sundar> Sure, will do
03:47:09 <shaohe_feng> when is the dead line of async job?
03:47:48 <shaohe_feng> it it really not easy to test it on really env.
03:48:00 <shaohe_feng> too many steps
03:48:30 <Sundar> shaohe_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:49:05 <Sundar> It does not have to merge this week, if that makes you feel better :)
03:49:30 <shaohe_feng> OK, I will do my best to make it works well today.
03:49:49 <Sundar> Great. Thanks, shaohe_feng
03:49:51 <shaohe_feng> for tomorrow is our Moon festival
03:50:27 <Sundar> I see. Happy Moon festival to all of you!
03:50:58 <shaohe_feng> Thanks,
03:51:04 <Sundar> All right, see you all next week. Bye :)
03:51:09 <Sundar> #endmeeting