| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2018-08-22 | |||
| 14:36:00 | shaohe_feng_robo | but pleacement report depend on driver | |
| 14:36:08 | xinran | I just trying to understand if all these should be done in Rocky or not? | |
| 14:36:30 | shaohe_feng_robo | not sure all drivers are ready. | |
| 14:36:32 | sum12 | wangzhh: couple you point me to a piece ? I was expecting it to be a separate project | |
| 14:36:45 | shaohe_feng_robo | Sundar: what's the status of the new driver? | |
| 14:37:07 | xinran | yes placement report depend on driver, did coco already have some works on it ? | |
| 14:37:57 | shaohe_feng_robo | Li_Liu: for interaction, edleafe can help on nova side, | |
| 14:38:24 | sum12 | found it | |
| 14:39:01 | xinran | Li_Liu: yes, is it urgent? | |
| 14:39:21 | wangzhh | sum12, OK. :) | |
| 14:39:35 | Li_Liu | xinran, just do it on your own pace, but do you have a eta tho? | |
| 14:40:17 | openstackgerrit | Nguyen Hai proposed openstack/python-cyborgclient master: Update reno for stable/rocky https://review.openstack.org/586830 | |
| 14:40:19 | shaohe_feng_robo | Li_Liu: who report to placement? | |
| 14:40:22 | Coco_gao | what kind of data placement need? I just store all of the data. | |
| 14:40:29 | shaohe_feng_robo | agent or conductor? | |
| 14:40:39 | Sundar | shaohe_feng: The driver developer is apparently busy with other stuff. He has reached a stage where rpm and deb packages for OPAE libraries get packaged with the tree. I'll sync up with him and help in testing. | |
| 14:41:00 | Coco_gao | besides, the driver-specific traits is not support right now. | |
| 14:41:02 | openstackgerrit | Nguyen Hai proposed openstack/python-cyborgclient master: import zuul job settings from project-config https://review.openstack.org/592840 | |
| 14:41:11 | openstackgerrit | Nguyen Hai proposed openstack/python-cyborgclient master: switch documentation job to new PTI https://review.openstack.org/592841 | |
| 14:41:11 | shaohe_feng_robo | Sundar: good. thanks . | |
| 14:41:16 | openstackgerrit | Nguyen Hai proposed openstack/python-cyborgclient master: add python 3.6 unit test job https://review.openstack.org/592842 | |
| 14:41:21 | xinran | Li_Liu: i am afraid that it's quite hard to implement all in this release ... | |
| 14:41:54 | sum12 | xinran: agreed | |
| 14:41:57 | Coco_gao | yeah, I think the work to heavy for xinran | |
| 14:42:01 | Coco_gao | too | |
| 14:42:22 | Li_Liu | shaohe_feng_robo, agent report to cyborg, then cyborg uses the placement client to report to placement, right? | |
| 14:42:24 | Coco_gao | since the time is limited before Rocky | |
| 14:42:33 | xinran | Coco_gao: have you already do some work on placement report which related to your current work? | |
| 14:42:41 | shaohe_feng_robo | Li_Liu: OK. | |
| 14:43:20 | xinran | Li_Liu: yes, cyborg use placement client to report resource | |
| 14:43:21 | Li_Liu | xinran, I know, as I said, just do it on your own pace. but could you give estimations on them? | |
| 14:43:22 | Coco_gao | I didn't report to placement by now. I | |
| 14:44:37 | Coco_gao | I just collected the driver data, store in db. | |
| 14:45:20 | shaohe_feng_ | every card is a sub-provider | |
| 14:45:25 | wangzhh | Coco, After diff. :) | |
| 14:45:50 | xinran | Li_Liu: I will probably submit first version of new api allowing nova/cyborg interaction next week | |
| 14:46:29 | Li_Liu | Coco_gao, that should be good for now. next we will figure out during the reporting step, how to invoke placement client apis | |
| 14:46:31 | shaohe_feng_ | Li_Liu: the placement report does not need many code | |
| 14:46:46 | Li_Liu | xinran, sounds good | |
| 14:46:47 | shaohe_feng_ | just need to know the deloyable well. | |
| 14:47:07 | Li_Liu | shaohe_feng_, yup | |
| 14:47:11 | xinran | to be clear, did we agreed to improve the current API, I mean, add a new API like DeployableActions to do alloc and dealloc? | |
| 14:47:56 | wangzhh | xinran, I suggest to imporve them in S release. | |
| 14:47:59 | shaohe_feng_ | xinran: you can introduce your new API briefly | |
| 14:48:47 | Coco_gao | Li_Liu, where can I store the acc_topology data? | |
| 14:49:10 | xinran | wangzhh: yes that what i desired | |
| 14:49:17 | xinran | shaohe_feng_: sure | |
| 14:50:52 | Sundar | Folks, I want to clarify that all commits now should be considered provisional. We may need to change them substantially to meet the use cases and Nova requirements | |
| 14:51:04 | Li_Liu | Coco_gao, the acc_topology is not to be saved tho. It helps you to structure the deployable relationships | |
| 14:51:39 | sum12 | wangzhh: do you plan to pull in any patches for the Rocky release for os-acc ? | |
| 14:52:19 | Li_Liu | Coco_gao, in case it's not clear to you, we can discuss it further once you feel better | |
| 14:52:26 | shaohe_feng_ | Sundar: will you work on the image upload ? auto parser the bitstream files, and generate the glance metadata? | |
| 14:52:46 | wangzhh | sum12: No. | |
| 14:53:49 | xinran | I am trying to add a new API representing all actions of deployables e.g. allocation/deallocation. for example: def post(self, body) with body like {"FPGA":"vendor_id":"","type":""...,"GPU","vendor_id":""...}. And in this method, deployable_list will be call with filters. | |
| 14:54:54 | wangzhh | shaohe_feng_, Before upload image, we need download first, in agent. Do you think so? | |
| 14:54:57 | Sundar | Shaohe: Sure. | |
| 14:56:03 | xinran | To support batch alloc/dealloc, there will be a loop to call deployable_list(...,filters) that's my current solution. Does anyone of you have some better suggestion? | |
| 14:57:54 | wangzhh | xinran, for batch alloc/dealloc, we should have new APIs, and I think it's better to have a spec. | |
| 14:58:18 | wangzhh | We should disscuss how to handle failure. | |
| 14:58:45 | Li_Liu | wangzhh, you mean the looping case? | |
| 14:59:17 | wangzhh | Li_Liu, No, for the new design. | |
| 14:59:59 | Coco_gao | LI_Liu, I just mess up,can we list the remaining work and the timeline for that? | |
| 15:00:16 | xinran | there is no spec currently | |
| 15:00:49 | wangzhh | Oh, Sorry. Misunderstand u. xinran. | |
| 15:01:21 | Li_Liu | Coco_gao, you mean the remaining work on the agent reporting? | |
| 15:01:41 | wangzhh | Current implemetion | |
| 15:01:43 | Coco_gao | on cyborg rocky. | |
| 15:01:56 | wangzhh | doesn't need spec. | |
| 15:01:59 | Sundar | I need to leave now for the next call. Thank you all. Bfn! | |
| 15:04:30 | Li_Liu | Coco_gao, for rocky, in my opinion, I hope we can merge the ovo stuff that you are currently working on. anything on top of that is a bonus :) | |
| 15:05:15 | Li_Liu | plus, xinran's new api and my programming api | |
| 15:05:36 | Li_Liu | wangzhh, what's left on your plate for Rocky? | |
| 15:05:48 | xinran | there are 1. driver discovery and data structure. 2. placement report. 3. nova integration (new api) and what else ? | |
| 15:06:33 | wangzhh | Agent diff and save, related to Coco's patch. | |
| 15:06:47 | Li_Liu | I thought the placement client report is finished didn't it? | |
| 15:07:00 | Li_Liu | wangzhh, sounds good | |
| 15:07:54 | Coco_gao | but I am afraid, Sundar's Spec have difference with my code. | |
| 15:08:02 | wangzhh | Li_liu, you mean we have finished reporting to placement? | |
| 15:08:34 | xinran | Sundar's spec mention that diff should be done in conductor. Did we have an agreement on it? | |
| 15:08:34 | Coco_gao | the api | |
| 15:08:36 | wangzhh | Not yet. xinran said , she is working on it. | |
| 15:08:54 | Li_Liu | https://github.com/openstack/cyborg/blob/master/cyborg/services/client/report.py | |
| 15:09:06 | wangzhh | xinran, not in current release. | |
| 15:09:17 | Li_Liu | these code from shaohe is the placement client right? | |
| 15:09:36 | Coco_gao | nope, zhipeng think it should be done on agent in this release | |
| 15:09:51 | xinran | ok thanks | |
| 15:10:09 | shaohe_feng_ | Li_Liu: yes, client | |
| 15:10:18 | wangzhh | Yes, but we should intergrate them in agent. | |
| 15:10:33 | Coco_gao | I think so | |
| 15:10:37 | Li_Liu | yup, it's just the matter when to use the client | |
| 15:10:50 | shaohe_feng_ | wangzhh: we do not need to download | |
| 15:11:28 | Coco_gao | I think it should be used after diff. | |
| 15:11:52 | wangzhh | shaohe_feng_, so what if we want to program? | |
| 15:12:01 | xinran | Coco_gao: yes, exactly | |
| 15:12:24 | wangzhh | Shouldn't we download bitstream first? | |
| 15:13:40 | shaohe_feng_ | no | |
| 15:13:43 | Coco_gao | Bye, I need to go bed. | |
| 15:13:58 | shaohe_feng_ | Coco_gao: have a good dream. | |
| 15:14:01 | Li_Liu | bye, Coco_gao, take a good rest | |
| 15:14:51 | Li_Liu | shaohe_feng_, wangzhh, are you guys talking about the same step here? | |