| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2019-09-10 | |||
| 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 | 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/ | |