Earlier  
Posted Nick Remark
#openstack-cyborg - 2019-09-09
23:57:50 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: P5: Basic changes for API layer. https://review.opendev.org/670466
23:57:51 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: P7: Changes for ARQs and ExtARQs. https://review.opendev.org/670468
23:57:51 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: P6: Changes for device profiles. https://review.opendev.org/670467
23:57:52 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: P9: Enable v2 API in devstack. https://review.opendev.org/670470
23:57:52 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: P8: Changes for programming. https://review.opendev.org/670469
23:57:53 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: Notify Nova when binding finished https://review.opendev.org/674520
#openstack-cyborg - 2019-09-10
00:43:44 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: python3 patch (including zuul config) https://review.opendev.org/667524
00:56:09 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: Enable api v2 policy check https://review.opendev.org/678177
08:26:34 openstackgerrit Xinran WANG proposed openstack/cyborg master: bug fixing: let FPGA driver report correct traits when no SRIOV enabled https://review.opendev.org/680953
10:21:59 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: [WIP] async job https://review.opendev.org/681005
10:34:48 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: [WIP] async job https://review.opendev.org/681005
11:52:47 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: [WIP] async job https://review.opendev.org/681005
13:09:00 openstackgerrit Merged openstack/cyborg master: P5: Basic changes for API layer. https://review.opendev.org/670466
13:09:02 openstackgerrit Merged openstack/cyborg master: P5.5: Additional changes to db layer. https://review.opendev.org/677115
13:09:03 openstackgerrit Merged openstack/cyborg master: P6: Changes for device profiles. https://review.opendev.org/670467
13:09:09 openstackgerrit Merged openstack/cyborg master: P7: Changes for ARQs and ExtARQs. https://review.opendev.org/670468
13:16:05 openstackgerrit Merged openstack/cyborg master: P8: Changes for programming. https://review.opendev.org/670469
13:16:07 openstackgerrit Merged openstack/cyborg master: P9: Enable v2 API in devstack. https://review.opendev.org/670470
13:27:13 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: [WIP] async job https://review.opendev.org/681005
14:02:31 openstackgerrit chenker proposed openstack/cyborg master: P5: Fix pep8 error in cyborg/accelerator https://review.opendev.org/679175
14:37:22 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: [WIP] async job https://review.opendev.org/681005
#openstack-cyborg - 2019-09-11
02:10:01 openstackgerrit chenker proposed openstack/cyborg master: P6: Fix pep8 error in cyborg/agent and cyborg/db https://review.opendev.org/679193
03:20:26 openstackgerrit zhurong proposed openstack/cyborg master: Make ResourceNotFound as NotFound exception https://review.opendev.org/681369
07:08:43 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: Notify Nova when all ARQs are resolved for an instance. https://review.opendev.org/674520
07:21:51 openstackgerrit Shogo Saito proposed openstack/python-cyborgclient master: v2 API Support(using openstack-sdk) https://review.opendev.org/681391
07:35:07 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: Notify Nova when all ARQs are resolved for an instance. https://review.opendev.org/674520
08:27:33 openstackgerrit Merged openstack/cyborg master: Add Huawei Ascend driver https://review.opendev.org/660874
08:51:54 openstackgerrit Shogo Saito proposed openstack/python-cyborgclient master: v2 API Support(using openstack-sdk) https://review.opendev.org/681391
10:06:54 openstackgerrit ShaoHe Feng proposed openstack/cyborg master: [WIP] async job https://review.opendev.org/681005
10:19:05 openstackgerrit YumengBao proposed openstack/cyborg master: Implement privsep boilerplate in cyborg. https://review.opendev.org/673957
14:56:12 openstackgerrit Sundar Nadathur proposed openstack/cyborg master: Notify Nova when all ARQs are resolved for an instance. https://review.opendev.org/674520
#openstack-cyborg - 2019-09-12
01:01:12 openstackgerrit Shogo Saito proposed openstack/python-cyborgclient master: v2 API Support(using openstack-sdk) https://review.opendev.org/681391
01:25:28 openstackgerrit YumengBao proposed openstack/cyborg master: Implement privsep boilerplate in cyborg. https://review.opendev.org/673957
03:03:25 Sundar Hi all
03:03:53 chenke Hi ~
03:03:59 wangzhh Hi all.
03:04:03 s_shogo Hi all
03:04:25 Sundar Hi chenke, wangzhh, s_shogo
03:04:31 xinranwang Hi all
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 xinranwang Yumeng: should we merge 676665 before 667231?
03:24:59 Yumeng so do we just follow the current way or change it ? what do you think?
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.

Earlier   Later