Earlier  
Posted Nick Remark
#openstack-cyborg - 2018-05-16
14:23:47 shaohe_feng rodrigo introduce intel OPAE API.
14:23:54 Sundar I can come in the afternoon
14:24:32 ildikov Sundar: it's ok
14:24:35 shaohe_feng Also some improvement on accelerator driver.
14:24:58 ildikov Sundar: it's more about answering students' questions on Cyborg as we have a few indicated that they are interested in the project
14:25:01 zhipeng shaohe_feng anywhere i can find the minutes ?
14:25:22 ildikov zhipeng: thanks for the mention! :)
14:25:57 zhipeng ildikov don't mention it (no pun intended :P)
14:26:32 ildikov zhipeng: my bad :)
14:26:47 zhipeng :P
14:27:04 zhipeng shaphe_feng i think I can dig out the minutes :)
14:27:13 zhipeng thanks for hosting the meeting, it has a great start
14:27:14 shaohe_feng zhipeng, it upload the log automaticly
14:27:28 zhipeng did xilinx people join ?
14:27:34 shaohe_feng let me find it.
14:27:46 shaohe_feng zhipeng, xilinx people not joined
14:28:04 shaohe_feng Li_Liu, will connect with them
14:28:04 zhipeng okey thx
14:28:19 zhipeng Li_Liu any update on the doc team side ?
14:28:25 shaohe_feng hopeful next time, they can join.
14:28:40 Li_Liu no response..
14:28:54 Li_Liu is Dutch still the contact?
14:29:20 shaohe_feng zhipeng, http://eavesdrop.openstack.org/meetings/openstack_cyborg_driver/2018/openstack_cyborg_driver.2018-05-14-14.01.log.html
14:29:25 shaohe_feng it is here.
14:29:32 zhipeng thx man :)
14:29:45 zhipeng Li_Liu no he left Xilinx
14:29:51 zhipeng Chuck Song is the contact
14:30:02 Li_Liu hmm. ok
14:32:39 Sundar FYI, I am readying a patch for driver-agent API update. It would be good to know who is working on what area
14:32:42 zhipeng Li_Liu any update on the doc side ?
14:33:09 zhipeng SUndar i think lenovo team could help on that
14:33:15 Li_Liu zhipeng, not yet, I will start org it after the summit
14:33:54 zhipeng okey
14:34:56 zhipeng #topic critical rocky spec review
14:38:22 zhipeng #info python-cyborgclient update
14:38:25 zhipeng #link https://review.openstack.org/#/c/565023/
14:38:44 zhipeng and #link https://review.openstack.org/568799
14:38:50 shaohe_feng zhipeng, as we discussed
14:38:57 zhipeng this took large chunk of time last meeting
14:38:57 shaohe_feng I have change the command to:
14:39:08 zhipeng so plz review it and if ok let's land it now
14:39:13 shaohe_feng openstack accelerator list
14:39:20 shaohe_feng ^ Sundar
14:40:03 shaohe_feng and change the service type to accelerator
14:40:37 Sundar shaohe: Sure. I did +1 just now
14:41:40 shaohe_feng thanks
14:41:48 Li_Liu I will give a +2
14:42:36 shaohe_feng we can add more command base on this patch
14:42:50 shaohe_feng then users can easy to use cyborg
14:42:57 zhipeng yes :)
14:43:01 zhipeng this is a big help
14:43:15 Li_Liu been waiting for this.
14:43:19 Li_Liu this is great
14:43:25 Sundar shaohe: https://review.openstack.org/#/c/565023/ looks good. There are still some references to deprecation. I'll try to understand that and post comments.
14:43:50 Sundar May be we can remove the deprecated stuff later too
14:44:08 shaohe_feng yes. let it work first.
14:44:14 shaohe_feng then others can works on it.
14:44:27 shaohe_feng it is a base patch for other commands.
14:44:53 shaohe_feng I notice NokMikeR need cyborg client.
14:45:09 zhipeng okey :)
14:45:52 zhipeng #info quota specs
14:46:04 zhipeng xinran__ do you have any further updates ?
14:48:06 xinran__ I have update the spec
14:49:10 xinran__ And also submitted the patch
14:49:10 xinran__ Now I think quota usage can
14:49:10 xinran__ work
14:49:24 xinran__ When users create or delete one accelerator quota usage will also be updated
14:50:04 shaohe_feng great.
14:50:05 zhipeng sounds great
14:51:25 xinran__ Need more review now
14:53:10 Sundar OK, will review it soon.
14:53:42 xinran__ Sundar: thanks
14:54:16 shaohe_feng thanks xinran__ , will review it.
14:54:26 zhipeng #info fpga programming spec
14:54:30 Sundar We need to understand what it means to reserve an accelerator in Cyborg without informing Nova
14:54:38 zhipeng Li_Liu you think this is in good shape ?
14:55:16 Li_Liu I think so
14:55:28 Li_Liu Sundar, do you have further comments?
14:55:39 Li_Liu I addressed your comments in my last patch
14:55:46 Sundar Yea, what does it mean for the driver to reserve an accelerator?
14:56:30 Sundar I will post comments on the patch
14:56:33 zhipeng Sundar prepare for future usage ?
14:57:24 Li_Liu are you talking about the programming spec?
14:58:15 Sundar zhipeng: I will post comments after thinking some more
14:58:29 Sundar Li_Liu: I think we are talking of Xinran's quota spec
14:58:55 xinran__ Sundar: you mean quota reserve or ...
14:59:05 zhipeng Sundar regarding the fpga programming spec ?
14:59:26 shaohe_feng :)
14:59:50 Sundar xinran: yes, in your patch: self._driver.reserve(...)
15:00:19 Sundar zhipeng: I will review all the specs and post comments :)
15:00:35 zhipeng okey :)
15:01:05 shaohe_feng I think admin can set the reserve. if reserve, cyborg should not report to placement?
15:01:53 edleafe shaohe_feng: you can report total=1 and reserved=1
15:02:10 edleafe at least with the bug fix that's currently being worked on
15:02:25 shaohe_feng edleafe, got it. thanks.
15:02:46 shaohe_feng edleafe, which bug?
15:03:10 xinran__ Sundar: we need to reserve quota usage first, check delta+current usage < 0 or not, but quota usage will not be updated at this time. and after create accelerator successfully, we need call commit to update quota usage.
15:03:33 edleafe shaohe_feng: https://review.openstack.org/#/c/568613/
15:03:59 edleafe shaohe_feng: it's still being debated whether this is a bug fix or a behavior change :)
15:04:11 xinran__ Did I get your point?

Earlier   Later