| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2018-09-26 | |||
| 14:11:57 | Coco_gao | nova only use VARs, and will update VARs' status by calling os-acc, I think | |
| 14:12:06 | xinran | what is VAR | |
| 14:12:50 | Coco_gao | We should have a discuss on what field still remained and what else should be removed as well as the new table field | |
| 14:13:53 | Li_Liu | Do you and XiaoHei need any help with this? | |
| 14:14:36 | Coco_gao | we may need Sundar's help on VAR table | |
| 14:15:22 | xinran | what is VAR ？ | |
| 14:16:06 | Coco_gao | I think I can make a draft then we can discuss it together, Since Sundar wrote the recent Specs on device discovery and interactions with nova, he may have some advice I think. | |
| 14:16:18 | Coco_gao | Virtual Accelerator Record (VAR) | |
| 14:17:25 | xinran | Coco_gao: thanks, | |
| 14:17:34 | Li_Liu | ok, Sundar is busy with his specs right now most likely | |
| 14:17:58 | Li_Liu | I will talk with him and see how to balance his work | |
| 14:18:11 | xinran | Seems there will be a changement at db layer? | |
| 14:18:17 | Li_Liu | I have a feeling we have too much work depend on him right now | |
| 14:18:31 | Li_Liu | xinran, yes. | |
| 14:19:10 | Coco_gao | Yes I think so, because he wrote the specs | |
| 14:19:37 | xinran | Li_Liu: could you please elaborate it or is there any docs concerning that? | |
| 14:19:37 | wangzhh | Yeap, most of specs. | |
| 14:20:53 | Coco_gao | We should follow the spec right? | |
| 14:21:00 | Li_Liu | Regarding the db change, I think Coco_gao is still drafting it | |
| 14:21:34 | Li_Liu | but the you can find related information here: https://etherpad.openstack.org/p/cyborg-ptg-stein | |
| 14:21:56 | Coco_gao | But I and xiaohe talked to Sundar that he should not write the implementation details on his spec. Who write the code decide the details | |
| 14:22:19 | xinran | OK | |
| 14:22:23 | xinran | thanks | |
| 14:22:29 | Li_Liu | Coco_gao, thist's the right way | |
| 14:23:28 | Li_Liu | Ok, move on. | |
| 14:23:38 | sum12 | Li_Liu: were there any notes taken during the discussion with neutron team during PTG? | |
| 14:24:20 | Li_Liu | sum12, there are some notes here | |
| 14:24:21 | Li_Liu | https://etherpad.openstack.org/p/fpga-networking | |
| 14:24:45 | sum12 | thanks | |
| 14:25:02 | Coco_gao | sum12, also https://etherpad.openstack.org/p/cyborg-ptg-stein-summary | |
| 14:26:11 | Li_Liu | I saw Xinran is have 4 patches waiting :P | |
| 14:26:21 | Li_Liu | I will take a look and provide comments | |
| 14:26:29 | sum12 | thanks Cato_gao | |
| 14:26:44 | Li_Liu | hope to clear them up soon | |
| 14:26:53 | xinran | Li_Liu: thank | |
| 14:27:03 | xinran | *thanks | |
| 14:27:24 | xinran | And I will submit another patch of os-acc soon | |
| 14:27:29 | Li_Liu | I am still working on FPGA programming api. Just solve the glance client problem should provide a updated patch some within the week | |
| 14:27:50 | Li_Liu | xinran, you are our super hero right now~~ | |
| 14:28:10 | xinran | About parse the flavor.extra_spec to the acceptable format to cyborg api | |
| 14:28:26 | xinran | Li_Liu: lol | |
| 14:28:53 | xinran | But these patches still need ovo land first | |
| 14:29:31 | Li_Liu | right, you might need to sync up with Coco and Xiaohei for that | |
| 14:30:03 | Li_Liu | Coco_gao wangzhh | |
| 14:30:07 | xinran | Li_Liu: yes, I have pull their patch to local and tested | |
| 14:30:30 | Coco_gao | the DB evolution is a huge change if we merge two tables(accelerators and deployables) as Sundar's suggest. Should we keep the old one? | |
| 14:30:32 | wangzhh | :) Xiaohei... | |
| 14:30:38 | wangzhh | Got it. | |
| 14:31:32 | Coco_gao | and evolve to the new version? | |
| 14:32:21 | wangzhh | Agree. I suggest keep the old one now. | |
| 14:33:41 | Coco_gao | everything will be changed after db evolution, the ovo, my patch on device object should also be rewritten. | |
| 14:34:06 | Li_Liu | I agree, let's keep the old one and decide when to remove it later | |
| 14:35:07 | xinran | The old one is the current one? | |
| 14:35:16 | Coco_gao | yes | |
| 14:35:16 | wangzhh | Coco, not really. Most of them can be reused. And we should hava a quick start. | |
| 14:35:23 | xinran | I mean acc dep these 2 tables | |
| 14:35:33 | Coco_gao | yes. | |
| 14:38:33 | xinran | Ok, but my placement repot patch depends on ovo now | |
| 14:42:41 | Li_Liu | ok, I think we are clear on the work item and their dependencies | |
| 14:42:58 | Li_Liu | Let's do this :) | |
| 14:43:04 | Li_Liu | #topic AoB | |
| 14:44:32 | Coco_gao | If we don't merge the two tables, then DB evolution is not urgent. Maybe I can do something more urgent right now. | |
| 14:46:52 | Li_Liu | Coco_gao, I think xinran is worried if she's implementing based on old DB design, once you new DB is merged, she might have to rewrite the code over again | |
| 14:47:18 | Coco_gao | I think so, not only xinran's code. | |
| 14:47:49 | Coco_gao | we finish rocky's target first? | |
| 14:48:00 | xinran | My code is now based on coco’s ovo patch :) | |
| 14:48:38 | Li_Liu | i see | |
| 14:49:02 | Coco_gao | Any feedback pls contact me directly, thanks xinran | |
| 14:49:10 | Li_Liu | Coco_gao, yes, try to finish up R's left over first | |
| 14:49:40 | xinran | So I think it’s ok for me if we use ovo design | |
| 14:51:26 | Coco_gao | I think the urgent thing is the docs~~ | |
| 14:51:26 | Li_Liu | alright, let's wrap up. | |
| 14:51:49 | Li_Liu | Coco_gao I will work with Sundar on that to speed it up | |
| 14:52:27 | Coco_gao | I can join from this week, feel free to contact me if I can help. | |
| 14:52:37 | Li_Liu | sure | |
| 14:52:41 | Li_Liu | thanks Coco | |
| 14:52:55 | Li_Liu | have a night night guys | |
| 14:52:59 | Li_Liu | #endmeeting | |
| 14:53:01 | wangzhh | Installation guide is urgent. | |
| 14:53:02 | openstack | Meeting ended Wed Sep 26 14:52:59 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 14:53:03 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2018/openstack_cyborg.2018-09-26-14.05.html | |
| 14:53:04 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2018/openstack_cyborg.2018-09-26-14.05.txt | |
| 14:53:05 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2018/openstack_cyborg.2018-09-26-14.05.log.html | |
| 14:53:06 | wangzhh | Bye | |
| 14:53:11 | Li_Liu | 88 | |
| 14:53:24 | xinran | bye | |
| 14:53:26 | Coco_gao | Bye | |
| 19:33:32 | Sundar | efried: Please ping me when you can | |
| 19:38:01 | efried | Sundar: Hi there. | |
| 19:42:33 | Sundar | Hi Eric, in the nova spec 603955, you said that having Cyborg claim a device by default (without explicit whitelisting) is a security risk. Can you elaborate? | |
| 19:45:02 | Sundar | Line 108 in https://review.openstack.org/#/c/603955 | |
| 20:26:41 | Sundar | @efried: I'll be around for another 90 minutes. I'll step out after that and should be back by 5 pm PDT. | |
| 20:27:31 | efried | sorry I missed your response earlier. ffr best to tag me | |
| 20:27:52 | efried | Not claiming, but exposing | |
| 20:28:34 | efried | If you expose, say, the controller that's handling the root disk of the management partition, and someone manages to attach it to the VM, your end user (the owner of the VM) could wreak havoc. | |
| 20:28:48 | efried | Sundar: ^^ :) | |
| 20:29:32 | Sundar | Ah, yes, will make sure to tag you :) | |
| 20:30:18 | Sundar | efried: My statement was "if the operator has installed and configured Cyborg drivers, he has explicitly enabled the devices managed by those drivers." | |
| 20:30:38 | Sundar | Why is that a security risk? | |
| 20:31:02 | efried | Sundar: that should be fine. | |