| *** edmondsw has quit IRC | 00:17 | |
| *** tetsuro has joined #openstack-cyborg | 00:51 | |
| *** Sundar has joined #openstack-cyborg | 00:53 | |
| *** tetsuro has quit IRC | 00:53 | |
| *** tetsuro has joined #openstack-cyborg | 00:56 | |
| *** edmondsw has joined #openstack-cyborg | 01:43 | |
| *** tetsuro has quit IRC | 02:05 | |
| *** tetsuro has joined #openstack-cyborg | 02:18 | |
| *** tetsuro has quit IRC | 02:24 | |
| *** tetsuro has joined #openstack-cyborg | 02:35 | |
| *** Li_Liu has joined #openstack-cyborg | 02:37 | |
| *** tetsuro has quit IRC | 02:40 | |
| *** openstackgerrit has joined #openstack-cyborg | 02:42 | |
| openstackgerrit | coco-Gao proposed openstack/cyborg master: Add New Driver OVO. https://review.openstack.org/624138 | 02:42 |
|---|---|---|
| *** tetsuro has joined #openstack-cyborg | 02:47 | |
| *** xinranwang has joined #openstack-cyborg | 02:58 | |
| *** wangzhh has joined #openstack-cyborg | 02:58 | |
| *** Coco_gao has joined #openstack-cyborg | 02:58 | |
| Coco_gao | Hi all | 02:58 |
| *** HongboZhao has joined #openstack-cyborg | 02:59 | |
| yikun | o/ | 02:59 |
| Li_Liu | Hi Coco | 02:59 |
| Kevin_Zheng | o/ | 02:59 |
| Li_Liu | Hi guys | 02:59 |
| wangzhh | Hi all. | 03:00 |
| Li_Liu | let's wait for sundar for a bit | 03:00 |
| Li_Liu | #startmeeting openstack-cyborg | 03:02 |
| openstack | Meeting started Wed Feb 27 03:02:08 2019 UTC and is due to finish in 60 minutes. The chair is Li_Liu. Information about MeetBot at http://wiki.debian.org/MeetBot. | 03:02 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 03:02 |
| *** openstack changes topic to " (Meeting topic: openstack-cyborg)" | 03:02 | |
| openstack | The meeting name has been set to 'openstack_cyborg' | 03:02 |
| Li_Liu | Let's get started | 03:02 |
| Li_Liu | Let's get started | 03:02 |
| *** tetsuro has quit IRC | 03:02 | |
| Li_Liu | #topic Roll Call | 03:02 |
| Li_Liu | #info Li_Liu | 03:02 |
| Sundar | #info Sundar | 03:02 |
| wangzhh | #info wangzhh | 03:02 |
| *** Yumeng_ has joined #openstack-cyborg | 03:03 | |
| yikun | o/ | 03:03 |
| Li_Liu | #topic Status Tracking | 03:03 |
| *** openstack changes topic to "Status Tracking (Meeting topic: openstack-cyborg)" | 03:03 | |
| Yumeng_ | #info Yumeng | 03:03 |
| Yumeng_ | hi all | 03:03 |
| Li_Liu | Hi yumeng | 03:03 |
| Li_Liu | I saw coco just committed the OVO patch | 03:04 |
| Li_Liu | thanks a lot coco | 03:04 |
| Li_Liu | let' | 03:04 |
| Li_Liu | let's try to merge it asap | 03:04 |
| Li_Liu | So that xinran and I can start working on the conductor | 03:04 |
| wangzhh | Cool. | 03:04 |
| Li_Liu | Is Sundar here? | 03:05 |
| Coco_gao | Still, I got something to discuss with Sundar. | 03:05 |
| Sundar | Li_Liu: Yes | 03:05 |
| Sundar | I agree | 03:05 |
| Coco_gao | Hi Sundar | 03:05 |
| Sundar | Hi Coco_gao | 03:05 |
| Li_Liu | Sundar, if we wanna get a working cyborg-nova integration | 03:06 |
| Coco_gao | #info Coco_gao | 03:06 |
| Li_Liu | can we start something now | 03:06 |
| Li_Liu | ? | 03:06 |
| Coco_gao | I think so | 03:06 |
| Sundar | Li_Liu: I think it is a long shot. It has got some review, but we cannot expect the Nova side to remain as is after March 8, as it goes through reviews | 03:07 |
| Sundar | So, the details of the APIs would change, even if the overall form does not | 03:07 |
| Sundar | i.e. we may still have create ARQ, ind ARQ etc. But the API signatures may change | 03:07 |
| Sundar | which means versioning it again | 03:08 |
| Sundar | *bind ARQ | 03:08 |
| xinranwang | Hi all, sorry for late | 03:08 |
| *** tetsuro has joined #openstack-cyborg | 03:08 | |
| Li_Liu | can we go ahead with a patched based implementation? | 03:08 |
| Li_Liu | so that we can demo this in the summit | 03:09 |
| Sundar | LiLiu, Coco_gao, all: Please do review the CYborg pilot code and Nova patches. That will at least reduce the chances that we get surprised later | 03:09 |
| Li_Liu | and apply later changes after we finalize thing with nova | 03:09 |
| Coco_gao | Ok, I will. | 03:09 |
| wangzhh | OK, sundar. | 03:10 |
| Li_Liu | Sundar, I will review after get in the conductor code :) | 03:10 |
| Sundar | Li_Liu: I have plans for a demo with the patches. I am working on getting resource commitments within my co. | 03:10 |
| Sundar | I think we can do a demo, even if it is not in the form that will finally merge | 03:10 |
| Li_Liu | ok | 03:11 |
| Li_Liu | sounds good | 03:11 |
| Coco_gao | I will fix the unit test part. | 03:11 |
| Coco_gao | Li_Liu, as for the conductor diff, feel free to discuss with us. | 03:12 |
| Li_Liu | Coco_gao | 03:12 |
| Li_Liu | sure, that part is owned by xinran and I | 03:13 |
| Coco_gao | Sundar, as for my new driver ovo, I still got something to discuss with you. | 03:13 |
| Coco_gao | can the attach_handles have same attach_info, but the attach_type is different? | 03:13 |
| Li_Liu | xinranwang, I think we don't really need 2 people working on it | 03:13 |
| Sundar | Coco_gao: Sure. Do you want to do that here, or do you prefer Zoom? | 03:13 |
| Li_Liu | do you have some cycles during the day? | 03:14 |
| Coco_gao | Sundar, maybe zoom or Email | 03:14 |
| Sundar | Coco_gao: attach_type is 'PCI', 'mediated_device', etc. SO, if the types are different ,the info will be totally different | 03:14 |
| *** tetsuro has quit IRC | 03:15 | |
| xinranwang | Li_Liu: yes, I agree. I can take over this part if needed | 03:15 |
| Sundar | E.g. For PCI, attach_info is BDF: {"domain": "0", bud: "5E", ... } | 03:15 |
| Sundar | For mediated devices, it will be a UUID that is dynamically created | 03:16 |
| Sundar | Coco_gao: Sure. Please feel free to shoot an email, and I can also send a zoom invite | 03:16 |
| Li_Liu | xinranwang, sure. do you have eta for it? | 03:16 |
| Coco_gao | Sundar, thank you. | 03:16 |
| wangzhh | Sundar, so one deployable can report 2 or more attach_handle? | 03:16 |
| wangzhh | *attach_handles | 03:16 |
| 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 |
| Coco_gao | Sundar, pls help review the new drive ovo patch. | 03:17 |
| Sundar | wangzhh: This is the situation when one deployable has 2 or more accelerator resources. Then each one needs a separate attach handle | 03:17 |
| Coco_gao | I hope I didn't mis-understanding your design. | 03:18 |
| Sundar | Coco_gao: On it after this meeting | 03:18 |
| xinranwang | Li_Liu: what's eta? | 03:18 |
| Li_Liu | "Estimated Time of Accomplishing" | 03:19 |
| Li_Liu | :P | 03:19 |
| xinranwang | lol | 03:19 |
| 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:20 |
| wangzhh | Sundar: The 'accelerator resources' here meas vf or pf here? | 03:21 |
| Li_Liu | Code freeze is next week | 03:21 |
| Sundar | wangzhh: We have made a distinction between accelerator resources, which is just a number, from PF (controlpath_id) and VF (attach_handle) | 03:21 |
| Li_Liu | wangzhh, do you have to wait for conductor code for your piece? | 03:22 |
| wangzhh | Sundar: Got it. | 03:22 |
| wangzhh | Li_liu: No, just the function name. I can hack it. | 03:23 |
| Li_Liu | ok, let's get wangzhh and xinranwang's patch in before the code freeze | 03:23 |
| Sundar | wangzhh: Good. If it helps, please see https://docs.google.com/presentation/d/1Anud3Qbcb0P3G245HpHduHhslx1MJljGD6wqPDy7o9E/edit#slide=id.g44d3e3519f_4_96 | 03:24 |
| Li_Liu | I might also fix the FPGA programming api as well. but that should be a minor change | 03:24 |
| 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 |
| wangzhh | Sundar, Thx. | 03:24 |
| xinranwang | Li_Liu: Ah yes, but conductor need to call ovo's function as I understood. | 03:24 |
| Li_Liu | xinranwang, that's in coco's patch right? | 03:25 |
| Coco_gao | The function in OVO is used for diff, I can add that part. | 03:26 |
| 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 |
| Sundar | because we use the term 'device_id' in the db layer for something else | 03:26 |
| Li_Liu | Coco_gao, do you need help for them? | 03:27 |
| 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 |
| Sundar | It doesn't have to be done for Stein, though | 03:28 |
| Li_Liu | Sundar, you are right, should not be too much effort | 03:28 |
| Coco_gao | Li_Liu, I will ask zhenghao for help | 03:28 |
| Li_Liu | since we kinda claimed reprogramming API works in last summit | 03:29 |
| Li_Liu | Coco_gao, thanks, also thanks wangzhh :) | 03:29 |
| *** tetsuro has joined #openstack-cyborg | 03:30 | |
| xinranwang | Coco_gao: So I will sync with you and wangzhh to start working on conductor part in same time then. | 03:30 |
| Yumeng_ | Coco_gao, I can also help. just LMK if you need help | 03:31 |
| 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:32 |
| Coco_gao | Yes, we'd better work in parallel | 03:33 |
| *** shoahe_feng has joined #openstack-cyborg | 03:33 | |
| shoahe_feng | #info shaohe | 03:33 |
| shoahe_feng | morning Coco_gao, evening Li_Liu | 03:34 |
| Coco_gao | hi shaohe | 03:34 |
| Li_Liu | sounds we have pretty good idea on the task before the code freeze | 03:34 |
| Li_Liu | hi shaohe_feng | 03:34 |
| Li_Liu | Let's keep working them and be open to communication through out the freezing phase | 03:35 |
| 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 |
| Li_Liu | #topic AoB | 03:36 |
| *** openstack changes topic to "AoB (Meeting topic: openstack-cyborg)" | 03:36 | |
| Li_Liu | Let's call the meeting then. Thank you guys for the efforts | 03:37 |
| Li_Liu | Have a good day/night | 03:37 |
| Li_Liu | #endmeeting | 03:37 |
| *** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)" | 03:37 | |
| 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 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-02-27-03.02.html | 03:37 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-02-27-03.02.txt | 03:37 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-02-27-03.02.log.html | 03:37 |
| *** Sundar has left #openstack-cyborg | 03:38 | |
| *** Li_Liu has quit IRC | 03:39 | |
| openstackgerrit | Yongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg https://review.openstack.org/639487 | 03:46 |
| *** diga has joined #openstack-cyborg | 03:47 | |
| openstackgerrit | Yongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg https://review.openstack.org/639487 | 03:53 |
| openstackgerrit | Yongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg https://review.openstack.org/639487 | 03:56 |
| *** tetsuro has quit IRC | 04:02 | |
| *** tetsuro has joined #openstack-cyborg | 04:03 | |
| *** tetsuro has quit IRC | 04:04 | |
| openstackgerrit | Yongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg https://review.openstack.org/639487 | 04:34 |
| openstackgerrit | coco-Gao proposed openstack/cyborg master: Add New Driver OVO. https://review.openstack.org/624138 | 05:02 |
| *** tetsuro has joined #openstack-cyborg | 05:38 | |
| *** xinranwang has quit IRC | 05:47 | |
| *** tetsuro has quit IRC | 06:05 | |
| *** tetsuro has joined #openstack-cyborg | 06:08 | |
| *** tetsuro has quit IRC | 06:31 | |
| *** tetsuro has joined #openstack-cyborg | 07:07 | |
| *** Coco_gao has quit IRC | 07:17 | |
| *** shoahe_feng has quit IRC | 07:24 | |
| openstackgerrit | Merged openstack/cyborg master: Add OS-limitation in the devstack_setup doc https://review.openstack.org/627613 | 07:43 |
| openstackgerrit | Merged openstack/cyborg master: Replacing the HTTP protocal with HTTPS in devstack_setup.rst. https://review.openstack.org/617778 | 07:43 |
| *** wangzhh has quit IRC | 07:48 | |
| *** tetsuro has quit IRC | 07:55 | |
| *** helenafm has joined #openstack-cyborg | 08:38 | |
| *** diga has quit IRC | 08:56 | |
| openstackgerrit | Yongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient to cyborg requirements https://review.openstack.org/639487 | 09:31 |
| *** HongboZhao has quit IRC | 09:37 | |
| openstackgerrit | Xinran WANG proposed openstack/cyborg master: Bug fix: Associate controlpath with device_id https://review.openstack.org/639610 | 09:40 |
| openstackgerrit | Xinran WANG proposed openstack/cyborg master: Bug fix: Associate controlpath with device_id https://review.openstack.org/639610 | 09:51 |
| openstackgerrit | Xinran WANG proposed openstack/cyborg master: Bug fix: Change object definition according to the newest DB. https://review.openstack.org/639610 | 11:05 |
| *** tetsuro has joined #openstack-cyborg | 11:57 | |
| *** tetsuro has quit IRC | 11:59 | |
| *** davidsha has joined #openstack-cyborg | 14:02 | |
| openstackgerrit | Merged openstack/cyborg master: Bug fix: Change object definition according to the newest DB. https://review.openstack.org/639610 | 14:32 |
| *** cdent has joined #openstack-cyborg | 15:16 | |
| efried | cdent: o/ | 15:16 |
| cdent | hola | 15:16 |
| *** Sundar has joined #openstack-cyborg | 15:16 | |
| efried | Since Berlin, Sundar has been working hard to put together a PoC series for the cyborg/nova interaction | 15:17 |
| * cdent remembers | 15:17 | |
| efried | including writing the v2.0 APIs in cyborg itself | 15:17 |
| efried | We wrote up an etherpad with a summary of the APIs that are used in the Poc | 15:17 |
| 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 |
| efried | would you be willing to help out there? | 15:18 |
| Sundar | o/, cdent and efried | 15:18 |
| efried | Not looking for a detailed review, and not of the source code, just of the interfaces | 15:18 |
| cdent | yes I would be willing. Are there some reviews in progress you can add me to? Might also add edleafe, elmiko, dtantsur | 15:18 |
| * cdent nods | 15:19 | |
| efried | cdent: It's here: https://etherpad.openstack.org/p/nova-cyborg-api-objects | 15:19 |
| cdent | comments on the etherpad presumable the right way to go? Is there a schedule or is "soon" good enough? | 15:20 |
| 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 |
| efried | What we don't want is for some major API-side thing to distract from the nova reviews. | 15:20 |
| cdent | I understand very well that concern | 15:21 |
| efried | The API is certainly close enough to demonstrate what the nova side will look like. | 15:21 |
| efried | But... yeah, we've all seen this happen. | 15:21 |
| cdent | I should be able to have a proper look before end of week, probably tomorrow | 15:21 |
| efried | Thank you very much cdent | 15:22 |
| cdent | happy to help move this stuff. | 15:22 |
| efried | The API code is here (in a cyborg pilot branch): https://review.openstack.org/#/c/626419/ | 15:22 |
| efried | But the interface is the most important thing rn. | 15:22 |
| 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:25 |
| Sundar | efried: Sure | 15:26 |
| cdent | ✔ | 15:26 |
| *** helenafm has quit IRC | 15:42 | |
| * edleafe catches up | 16:02 | |
| Sundar | Thanks, efried and cdent | 16:02 |
| edleafe | I'll add it to my TODO list | 16:02 |
| efried | Thanks edleafe | 16:06 |
| *** tetsuro has joined #openstack-cyborg | 16:40 | |
| *** tetsuro has quit IRC | 16:51 | |
| *** davidsha has quit IRC | 17:09 | |
| *** dims has quit IRC | 17:35 | |
| *** efried is now known as efried_afk | 17:47 | |
| *** dims has joined #openstack-cyborg | 17:48 | |
| *** Sundar is now known as DarthNau | 17:50 | |
| *** efried_afk is now known as efried | 18:32 | |
| *** DarthNau has quit IRC | 19:25 | |
| *** Yumeng_ has quit IRC | 20:06 | |
| *** cdent has quit IRC | 20:24 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!