Wednesday, 2019-02-27

*** edmondsw has quit IRC00:17
*** tetsuro has joined #openstack-cyborg00:51
*** Sundar has joined #openstack-cyborg00:53
*** tetsuro has quit IRC00:53
*** tetsuro has joined #openstack-cyborg00:56
*** edmondsw has joined #openstack-cyborg01:43
*** tetsuro has quit IRC02:05
*** tetsuro has joined #openstack-cyborg02:18
*** tetsuro has quit IRC02:24
*** tetsuro has joined #openstack-cyborg02:35
*** Li_Liu has joined #openstack-cyborg02:37
*** tetsuro has quit IRC02:40
*** openstackgerrit has joined #openstack-cyborg02:42
openstackgerritcoco-Gao proposed openstack/cyborg master: Add New Driver OVO.
*** tetsuro has joined #openstack-cyborg02:47
*** xinranwang has joined #openstack-cyborg02:58
*** wangzhh has joined #openstack-cyborg02:58
*** Coco_gao has joined #openstack-cyborg02:58
Coco_gaoHi all02:58
*** HongboZhao has joined #openstack-cyborg02:59
Li_LiuHi Coco02:59
Li_LiuHi guys02:59
wangzhhHi all.03:00
Li_Liulet's wait for sundar for a bit03:00
Li_Liu#startmeeting openstack-cyborg03:02
openstackMeeting 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.03:02
*** openstack changes topic to " (Meeting topic: openstack-cyborg)"03:02
openstackThe meeting name has been set to 'openstack_cyborg'03:02
Li_LiuLet's get started03:02
Li_LiuLet's get started03:02
*** tetsuro has quit IRC03:02
Li_Liu #topic Roll Call03:02
Li_Liu#info Li_Liu03:02
Sundar#info Sundar03:02
wangzhh#info wangzhh03:02
*** Yumeng_ has joined #openstack-cyborg03:03
Li_Liu#topic Status Tracking03:03
*** openstack changes topic to "Status Tracking (Meeting topic: openstack-cyborg)"03:03
Yumeng_#info Yumeng03:03
Yumeng_hi all03:03
Li_LiuHi yumeng03:03
Li_LiuI saw coco just committed the OVO patch03:04
Li_Liuthanks a lot coco03:04
Li_Liulet's try to merge it asap03:04
Li_LiuSo that xinran and I can start working on the conductor03:04
Li_LiuIs Sundar here?03:05
Coco_gaoStill, I got something to discuss with Sundar.03:05
SundarLi_Liu: Yes03:05
SundarI agree03:05
Coco_gaoHi Sundar03:05
SundarHi Coco_gao03:05
Li_LiuSundar, if we wanna get a working cyborg-nova integration03:06
Coco_gao#info Coco_gao03:06
Li_Liucan we start something now03:06
Coco_gaoI think so03:06
SundarLi_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 reviews03:07
SundarSo, the details of the APIs would change, even if the overall form does not03:07
Sundari.e. we may still have create ARQ, ind ARQ etc. But the API signatures may change03:07
Sundarwhich means versioning it again03:08
Sundar*bind ARQ03:08
xinranwang Hi all, sorry for late03:08
*** tetsuro has joined #openstack-cyborg03:08
Li_Liucan we go ahead with a patched based implementation?03:08
Li_Liuso that we can demo this in the summit03:09
SundarLiLiu, Coco_gao, all: Please do review the CYborg pilot code and Nova patches. That will at least reduce the chances that we get surprised later03:09
Li_Liuand apply later changes after we finalize thing with nova03:09
Coco_gaoOk, I will.03:09
wangzhhOK, sundar.03:10
Li_LiuSundar, I will review after get in the conductor code :)03:10
SundarLi_Liu: I have plans for a demo with the patches. I am working on getting resource commitments within my co.03:10
SundarI think we can do a demo, even if it is not in the form that will finally merge03:10
Li_Liusounds good03:11
Coco_gaoI will fix the unit test part.03:11
Coco_gaoLi_Liu, as for the conductor diff,  feel free to discuss with us.03:12
Li_Liusure, that part is owned by xinran and I03:13
Coco_gaoSundar, as for my new driver ovo, I still got something to discuss with you.03:13
Coco_gaocan the attach_handles have same attach_info, but the attach_type is different?03:13
Li_Liuxinranwang, I think we don't really need 2 people working on it03:13
SundarCoco_gao: Sure. Do you want to do that here, or do you prefer Zoom?03:13
Li_Liudo you have some cycles during the day?03:14
Coco_gaoSundar, maybe zoom or Email03:14
SundarCoco_gao: attach_type is 'PCI', 'mediated_device', etc. SO, if the types are different ,the info will be totally different03:14
*** tetsuro has quit IRC03:15
xinranwangLi_Liu:  yes, I agree. I can take over this part if needed03:15
SundarE.g. For PCI, attach_info is BDF: {"domain": "0", bud: "5E", ... }03:15
SundarFor mediated devices, it will be a UUID that is dynamically created03:16
SundarCoco_gao: Sure. Please feel free to shoot an email, and I can also send a zoom invite03:16
Li_Liuxinranwang, sure. do  you have eta for it?03:16
Coco_gaoSundar, thank you.03:16
wangzhhSundar, so one deployable can report 2 or more attach_handle?03:16
Sundarwangzhh: Yes. Today's products don't need it (at least for the ones I know about), but it is possible in the future03:17
Coco_gaoSundar, pls help review the new drive ovo patch.03:17
Sundarwangzhh: This is the situation when one deployable has 2 or more accelerator resources. Then each one needs a separate attach handle03:17
Coco_gaoI hope I didn't mis-understanding your design.03:18
SundarCoco_gao: On it after this meeting03:18
xinranwangLi_Liu:  what's eta?03:18
Li_Liu"Estimated Time of Accomplishing"03:19
xinranwangLi_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
wangzhhSundar:  The 'accelerator resources' here meas vf or pf here?03:21
Li_LiuCode freeze is next week03:21
Sundarwangzhh: We have made a distinction between accelerator resources, which is just a number, from PF (controlpath_id) and VF (attach_handle)03:21
Li_Liuwangzhh, do you have to wait for conductor code for your piece?03:22
wangzhhSundar: Got it.03:22
wangzhhLi_liu: No, just the function name. I can  hack it.03:23
Li_Liuok, let's get wangzhh and xinranwang's patch in before the code freeze03:23
Sundarwangzhh: Good. If it helps, please see
Li_LiuI might also fix the FPGA programming api as well. but that should be a minor change03:24
Coco_gaoSundar, 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
wangzhhSundar, Thx.03:24
xinranwangLi_Liu:  Ah yes,  but conductor need to call ovo's function as I understood.03:24
Li_Liuxinranwang, that's in coco's patch right?03:25
Coco_gaoThe function in OVO is used for diff, I can add that part.03:26
SundarCoco_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
Sundarbecause we use the term 'device_id' in the db layer for something else03:26
Li_LiuCoco_gao, do you need help for them?03:27
SundarLi_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
SundarIt doesn't have to be done for Stein, though03:28
Li_Liu Sundar, you are right, should not be too much effort03:28
Coco_gaoLi_Liu, I will ask zhenghao for help03:28
Li_Liusince we kinda claimed reprogramming API works in last summit03:29
Li_LiuCoco_gao, thanks, also thanks wangzhh :)03:29
*** tetsuro has joined #openstack-cyborg03:30
xinranwangCoco_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 help03:31
xinranwangdo 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_gaoYes, we'd better work in parallel03:33
*** shoahe_feng has joined #openstack-cyborg03:33
shoahe_feng#info shaohe03:33
shoahe_fengmorning Coco_gao, evening Li_Liu03:34
Coco_gaohi shaohe03:34
Li_Liusounds we have pretty good idea on the task before the code freeze03:34
Li_Liuhi shaohe_feng03:34
Li_LiuLet's keep working them and be open to communication through out the freezing phase03:35
Li_Liuzoom meeting room is open all the time, in case you guys want to discuss anything, just ping us on wechat and drop in zoom03:36
Li_Liu#topic AoB03:36
*** openstack changes topic to "AoB (Meeting topic: openstack-cyborg)"03:36
Li_LiuLet's call the meeting then. Thank you guys for the efforts03:37
Li_LiuHave a good day/night03:37
*** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)"03:37
openstackMeeting ended Wed Feb 27 03:37:49 2019 UTC.  Information about MeetBot at . (v 0.1.4)03:37
openstackMinutes (text):
*** Sundar has left #openstack-cyborg03:38
*** Li_Liu has quit IRC03:39
openstackgerritYongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg
*** diga has joined #openstack-cyborg03:47
openstackgerritYongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg
openstackgerritYongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg
*** tetsuro has quit IRC04:02
*** tetsuro has joined #openstack-cyborg04:03
*** tetsuro has quit IRC04:04
openstackgerritYongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient for cyborg
openstackgerritcoco-Gao proposed openstack/cyborg master: Add New Driver OVO.
*** tetsuro has joined #openstack-cyborg05:38
*** xinranwang has quit IRC05:47
*** tetsuro has quit IRC06:05
*** tetsuro has joined #openstack-cyborg06:08
*** tetsuro has quit IRC06:31
*** tetsuro has joined #openstack-cyborg07:07
*** Coco_gao has quit IRC07:17
*** shoahe_feng has quit IRC07:24
openstackgerritMerged openstack/cyborg master: Add OS-limitation in the devstack_setup doc
openstackgerritMerged openstack/cyborg master: Replacing the HTTP protocal with HTTPS in devstack_setup.rst.
*** wangzhh has quit IRC07:48
*** tetsuro has quit IRC07:55
*** helenafm has joined #openstack-cyborg08:38
*** diga has quit IRC08:56
openstackgerritYongjun Bai proposed openstack/cyborg master: Fix:add python-glanceclient to cyborg requirements
*** HongboZhao has quit IRC09:37
openstackgerritXinran WANG proposed openstack/cyborg master: Bug fix: Associate controlpath with device_id
openstackgerritXinran WANG proposed openstack/cyborg master: Bug fix: Associate controlpath with device_id
openstackgerritXinran WANG proposed openstack/cyborg master: Bug fix: Change object definition according to the newest DB.
*** tetsuro has joined #openstack-cyborg11:57
*** tetsuro has quit IRC11:59
*** davidsha has joined #openstack-cyborg14:02
openstackgerritMerged openstack/cyborg master: Bug fix: Change object definition according to the newest DB.
*** cdent has joined #openstack-cyborg15:16
efriedcdent: o/15:16
*** Sundar has joined #openstack-cyborg15:16
efriedSince Berlin, Sundar has been working hard to put together a PoC series for the cyborg/nova interaction15:17
* cdent remembers15:17
efriedincluding writing the v2.0 APIs in cyborg itself15:17
efriedWe wrote up an etherpad with a summary of the APIs that are used in the Poc15:17
efriedand 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
efriedwould you be willing to help out there?15:18
Sundaro/, cdent and efried15:18
efriedNot looking for a detailed review, and not of the source code, just of the interfaces15:18
cdentyes I would be willing. Are there some reviews in progress you can add me to? Might also add edleafe, elmiko, dtantsur15:18
* cdent nods15:19
efriedcdent: It's here:
cdentcomments on the etherpad presumable the right way to go? Is there a schedule or is "soon" good enough?15:20
efriedI 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
efriedWhat we don't want is for some major API-side thing to distract from the nova reviews.15:20
cdentI understand very well that concern15:21
efriedThe API is certainly close enough to demonstrate what the nova side will look like.15:21
efriedBut... yeah, we've all seen this happen.15:21
cdentI should be able to have a proper look before end of week, probably tomorrow15:21
efriedThank you very much cdent15:22
cdenthappy to help move this stuff.15:22
efriedThe API code is here (in a cyborg pilot branch):
efriedBut the interface is the most important thing rn.15:22
efriedSundar: 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
Sundarefried: Sure15:26
*** helenafm has quit IRC15:42
* edleafe catches up16:02
SundarThanks, efried and cdent16:02
edleafeI'll add it to my TODO list16:02
efriedThanks edleafe16:06
*** tetsuro has joined #openstack-cyborg16:40
*** tetsuro has quit IRC16:51
*** davidsha has quit IRC17:09
*** dims has quit IRC17:35
*** efried is now known as efried_afk17:47
*** dims has joined #openstack-cyborg17:48
*** Sundar is now known as DarthNau17:50
*** efried_afk is now known as efried18:32
*** DarthNau has quit IRC19:25
*** Yumeng_ has quit IRC20:06
*** cdent has quit IRC20:24

Generated by 2.15.3 by Marius Gedminas - find it at!