| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2019-02-27 | |||
| 03:16:27 | Coco_gao | Sundar, thank you. | |
| 03:16:38 | wangzhh | *attach_handles | |
| 03:17:05 | Sundar | wangzhh: Yes. Today's products don't need it (at least for the ones I know about), but it is possible in the future | |
| 03:17:32 | Coco_gao | Sundar, pls help review the new drive ovo patch. | |
| 03:17:53 | Sundar | wangzhh: This is the situation when one deployable has 2 or more accelerator resources. Then each one needs a separate attach handle | |
| 03:18:05 | Coco_gao | I hope I didn't mis-understanding your design. | |
| 03:18:07 | Sundar | Coco_gao: On it after this meeting | |
| 03:18:42 | xinranwang | Li_Liu: what's eta? | |
| 03:19:30 | Li_Liu | "Estimated Time of Accomplishing" | |
| 03:19:34 | Li_Liu | :P | |
| 03:19:47 | xinranwang | lol | |
| 03:20:54 | xinranwang | Li_Liu: hmmm, i think i can start working on this once OVO part has done, and will submit at least one patch in the following week. | |
| 03:21:05 | wangzhh | Sundar: The 'accelerator resources' here meas vf or pf here? | |
| 03:21:37 | Li_Liu | Code freeze is next week | |
| 03:21:51 | Sundar | wangzhh: We have made a distinction between accelerator resources, which is just a number, from PF (controlpath_id) and VF (attach_handle) | |
| 03:22:09 | Li_Liu | wangzhh, do you have to wait for conductor code for your piece? | |
| 03:22:36 | wangzhh | Sundar: Got it. | |
| 03:23:15 | wangzhh | Li_liu: No, just the function name. I can hack it. | |
| 03:23:48 | Li_Liu | ok, let's get wangzhh and xinranwang's patch in before the code freeze | |
| 03:24:28 | Sundar | wangzhh: Good. If it helps, please see https://docs.google.com/presentation/d/1Anud3Qbcb0P3G245HpHduHhslx1MJljGD6wqPDy7o9E/edit#slide=id.g44d3e3519f_4_96 | |
| 03:24:29 | Li_Liu | I might also fix the FPGA programming api as well. but that should be a minor change | |
| 03:24:42 | Coco_gao | Sundar, I think the original design is that attach_handle can be VF or PF, and controlpath_id is only used for identify a device? Do we change now? | |
| 03:24:55 | wangzhh | Sundar, Thx. | |
| 03:24:58 | xinranwang | Li_Liu: Ah yes, but conductor need to call ovo's function as I understood. | |
| 03:25:42 | Li_Liu | xinranwang, that's in coco's patch right? | |
| 03:26:08 | Coco_gao | The function in OVO is used for diff, I can add that part. | |
| 03:26:27 | Sundar | Coco_gao: No. We always had a distinction. In the link I posted earlier, the only change I made was to change the term 'DeviceID' to 'ControlPath_ID' | |
| 03:26:48 | Sundar | because we use the term 'device_id' in the db layer for something else | |
| 03:27:14 | Li_Liu | Coco_gao, do you need help for them? | |
| 03:28:14 | Sundar | Li_Liu: I suspect the programming API needs some changes. It is tied to the device path now. But shuld be tied to deployable UUID or some such, right? | |
| 03:28:32 | Sundar | It doesn't have to be done for Stein, though | |
| 03:28:49 | Li_Liu | Sundar, you are right, should not be too much effort | |
| 03:28:58 | Coco_gao | Li_Liu, I will ask zhenghao for help | |
| 03:29:11 | Li_Liu | since we kinda claimed reprogramming API works in last summit | |
| 03:29:43 | Li_Liu | Coco_gao, thanks, also thanks wangzhh :) | |
| 03:30:19 | xinranwang | Coco_gao: So I will sync with you and wangzhh to start working on conductor part in same time then. | |
| 03:31:15 | Yumeng_ | Coco_gao, I can also help. just LMK if you need help | |
| 03:32:49 | xinranwang | do you think it is better to work in parallel, cause time's rushing by. I can hack the ovo function call in conductor if needed. | |
| 03:33:37 | Coco_gao | Yes, we'd better work in parallel | |
| 03:33:51 | shoahe_feng | #info shaohe | |
| 03:34:21 | shoahe_feng | morning Coco_gao, evening Li_Liu | |
| 03:34:30 | Coco_gao | hi shaohe | |
| 03:34:39 | Li_Liu | sounds we have pretty good idea on the task before the code freeze | |
| 03:34:47 | Li_Liu | hi shaohe_feng | |
| 03:35:27 | Li_Liu | Let's keep working them and be open to communication through out the freezing phase | |
| 03:36:20 | Li_Liu | zoom meeting room is open all the time, in case you guys want to discuss anything, just ping us on wechat and drop in zoom | |
| 03:36:45 | Li_Liu | #topic AoB | |
| 03:37:24 | Li_Liu | Let's call the meeting then. Thank you guys for the efforts | |
| 03:37:38 | Li_Liu | Have a good day/night | |
| 03:37:49 | Li_Liu | #endmeeting | |
| 03:37:51 | openstack | Meeting ended Wed Feb 27 03:37:49 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 03:37:52 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-02-27-03.02.html | |
| 03:37:53 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-02-27-03.02.txt | |
| 03:37:54 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-02-27-03.02.log.html | |
| 03:46:31 | openstackgerrit | Yongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg https://review.openstack.org/639487 | |
| 03:53:36 | openstackgerrit | Yongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg https://review.openstack.org/639487 | |
| 03:56:30 | openstackgerrit | Yongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg https://review.openstack.org/639487 | |
| 04:34:04 | openstackgerrit | Yongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg https://review.openstack.org/639487 | |
| 05:02:57 | openstackgerrit | coco-Gao proposed openstack/cyborg master: Add New Driver OVO. https://review.openstack.org/624138 | |
| 07:43:38 | openstackgerrit | Merged openstack/cyborg master: Add OS-limitation in the devstack_setup doc https://review.openstack.org/627613 | |
| 07:43:43 | openstackgerrit | Merged openstack/cyborg master: Replacing the HTTP protocal with HTTPS in devstack_setup.rst. https://review.openstack.org/617778 | |
| 09:31:17 | openstackgerrit | Yongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient to cyborg requirements https://review.openstack.org/639487 | |
| 09:40:21 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Bug fix: Associate controlpath with device_id https://review.openstack.org/639610 | |
| 09:51:09 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Bug fix: Associate controlpath with device_id https://review.openstack.org/639610 | |
| 11:05:28 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Bug fix: Change object definition according to the newest DB. https://review.openstack.org/639610 | |
| 14:32:53 | openstackgerrit | Merged openstack/cyborg master: Bug fix: Change object definition according to the newest DB. https://review.openstack.org/639610 | |
| 15:16:43 | efried | cdent: o/ | |
| 15:16:47 | cdent | hola | |
| 15:17:06 | efried | Since Berlin, Sundar has been working hard to put together a PoC series for the cyborg/nova interaction | |
| 15:17:22 | efried | including writing the v2.0 APIs in cyborg itself | |
| 15:17:43 | efried | We wrote up an etherpad with a summary of the APIs that are used in the Poc | |
| 15:18:10 | efried | and were hoping to get some API-oriented eyeballs on it to make sure there aren't any egregious violations of overall stylisticness or OpenStack APIness. | |
| 15:18:23 | efried | would you be willing to help out there? | |
| 15:18:24 | Sundar | o/, cdent and efried | |
| 15:18:41 | efried | Not looking for a detailed review, and not of the source code, just of the interfaces | |
| 15:18:57 | cdent | yes I would be willing. Are there some reviews in progress you can add me to? Might also add edleafe, elmiko, dtantsur | |
| 15:19:10 | efried | cdent: It's here: https://etherpad.openstack.org/p/nova-cyborg-api-objects | |
| 15:20:01 | cdent | comments on the etherpad presumable the right way to go? Is there a schedule or is "soon" good enough? | |
| 15:20:37 | efried | I don't think there's super urgency. We want to start socializing the nova-side PoC itself, but we can do that in its current form. | |
| 15:20:51 | efried | What we don't want is for some major API-side thing to distract from the nova reviews. | |
| 15:21:01 | cdent | I understand very well that concern | |
| 15:21:07 | efried | The API is certainly close enough to demonstrate what the nova side will look like. | |
| 15:21:15 | efried | But... yeah, we've all seen this happen. | |
| 15:21:43 | cdent | I should be able to have a proper look before end of week, probably tomorrow | |
| 15:22:23 | efried | Thank you very much cdent | |
| 15:22:43 | cdent | happy to help move this stuff. | |
| 15:22:49 | efried | But the interface is the most important thing rn. | |
| 15:22:49 | efried | The API code is here (in a cyborg pilot branch): https://review.openstack.org/#/c/626419/ | |
| 15:25:32 | efried | Sundar: If you get some time, you might as well add the other APIs the nova code will need - which I think is just 'unbind' and 'delete arq', right? | |
| 15:26:14 | Sundar | efried: Sure | |
| 15:26:16 | cdent | ✔ | |
| 16:02:29 | Sundar | Thanks, efried and cdent | |
| 16:02:30 | edleafe | I'll add it to my TODO list | |
| 16:06:26 | efried | Thanks edleafe | |
| #openstack-cyborg - 2019-02-28 | |||
| 07:46:57 | openstackgerrit | Merged openstack/cyborg master: Add New Driver OVO. https://review.openstack.org/624138 | |
| #openstack-cyborg - 2019-03-03 | |||
| 03:58:06 | openstackgerrit | wangzhh proposed openstack/cyborg master: Report data to conducor https://review.openstack.org/640615 | |
| 04:01:54 | openstackgerrit | wangzhh proposed openstack/cyborg master: Report data to conductor https://review.openstack.org/640615 | |
| 05:04:02 | openstackgerrit | Li Liu proposed openstack/cyborg master: Modify the deployable's api for new database work https://review.openstack.org/640616 | |
| #openstack-cyborg - 2019-03-04 | |||
| 08:03:33 | openstackgerrit | coco-Gao proposed openstack/cyborg master: Fix bug: deployable object related bugs. https://review.openstack.org/640689 | |
| 12:27:07 | openstackgerrit | Merged openstack/cyborg master: Fix bug: deployable object related bugs. https://review.openstack.org/640689 | |