| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2018-12-12 | |||
| 02:27:11 | Coco_gao | The remaining tasks in the list may owned by zhenghao. | |
| 02:27:35 | Coco_gao | Or, you can talk with him. | |
| 02:33:01 | Yumeng_ | ok, no pb. if he was busy I can help | |
| 02:41:21 | Coco_gao | By the way, what's new time for the IRC meeting? | |
| 02:47:56 | Yumeng_ | haha 11:00 | |
| 02:49:37 | Yumeng_ | I also thought it was 10:00. | |
| 03:01:19 | Li_Liu | Hi Sundar | |
| 03:01:26 | Sundar | Hi Li | |
| 03:01:27 | Li_Liu | Let's wait for others | |
| 03:01:31 | Sundar | Sure | |
| 03:02:32 | Li_Liu | maybe wait for another 3 mins | |
| 03:02:55 | Yumeng_ | hi Sundar Li :) | |
| 03:03:11 | Li_Liu | Hi Yumeng :) | |
| 03:04:04 | jiapei | o/ | |
| 03:04:43 | xinranwang | Hi all | |
| 03:04:54 | Sundar | Hi Yumeng_ and all | |
| 03:05:38 | Li_Liu | #startmeeting openstack-cyborg | |
| 03:05:39 | openstack | Meeting started Wed Dec 12 03:05:38 2018 UTC and is due to finish in 60 minutes. The chair is Li_Liu. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 03:05:40 | openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 03:05:42 | openstack | The meeting name has been set to 'openstack_cyborg' | |
| 03:05:48 | Li_Liu | #topic Roll Call | |
| 03:05:56 | Li_Liu | #info Li_Liu | |
| 03:06:03 | Li_Liu | Let's get started | |
| 03:06:10 | xinranwang | #info xinranwang | |
| 03:06:14 | Sundar | #info Sundar | |
| 03:06:20 | Li_Liu | I hope the new time slot works better for most of the people | |
| 03:07:08 | Li_Liu | #topic status updates | |
| 03:07:23 | Yumeng_ | # info Yumeng_ | |
| 03:07:30 | Li_Liu | Sundar, what's new on your side? | |
| 03:08:35 | Sundar | The feature branches for CYborg and its client have been created. Will be submitting code for a prorotype that shows how Nova could call into CYborg | |
| 03:09:13 | Li_Liu | that's great. So would help push the Nova-Cyborg interaction spec right? | |
| 03:09:16 | Sundar | This is intended to show developers how the interaction in the spec will play out in practice | |
| 03:09:22 | Sundar | I hope so :) | |
| 03:10:06 | Li_Liu | Once created, let's find some time to talk with Nova folks directly | |
| 03:10:11 | Li_Liu | let's push together | |
| 03:10:42 | Li_Liu | xinrangwang, anything you wanna share? | |
| 03:10:55 | Sundar | I also need to develop and push the Nova patches that do the actual invocation | |
| 03:10:56 | Li_Liu | https://review.openstack.org/#/c/624340/ | |
| 03:11:44 | Li_Liu | xinranwang, is that one ready to be reviewed? | |
| 03:12:24 | Li_Liu | Sundar, maybe we can show them how it should work first? | |
| 03:12:26 | xinranwang | Li_Liu: nope, it depends on the DB schemas the ovo object | |
| 03:13:03 | xinranwang | it's just the initial version | |
| 03:13:31 | Li_Liu | xinranwang, the spec should be ready anytime, does it also depends on Coco/zhenghao's work? | |
| 03:14:39 | Sundar | Li_Liu: Yes, the pilot code for Cyborg and its client will be submitted first. | |
| 03:15:16 | Li_Liu | great | |
| 03:16:24 | xinranwang | yes, because the input param for conductor is ovo object | |
| 03:16:41 | Li_Liu | Sundar, I think https://review.openstack.org/#/c/615462/ can be merged | |
| 03:17:03 | Li_Liu | xinranwang, ok\ | |
| 03:17:08 | xinranwang | Li_Liu: Btw, I have submitted our POC code with workflow -1 | |
| 03:17:49 | Li_Liu | xinranwang, right, understand | |
| 03:17:58 | Li_Liu | thanks a lot for the work | |
| 03:18:14 | Sundar | Li_Liu: The db spec has progressed a lot. Only a couple of points left, which you wanted to address offline. I replied by email about half hour ago. | |
| 03:18:52 | Coco_gao | I received you email | |
| 03:19:00 | Li_Liu | Sundar, I saw that one, I think i address them in my patch already | |
| 03:19:01 | Yumeng_ | Sundar, for tasks in Client and DB https://storyboard.openstack.org/#!/story/2004248. which one has a higher priority currently? | |
| 03:19:13 | Coco_gao | I will update the OVO object patch as well as submmit New DB patch. | |
| 03:19:34 | Li_Liu | Coco_gao, are you gonna work on the DB object based on the new db spec? | |
| 03:20:42 | xinranwang | Coco_gao: I can take over the diff part in conductor if you don't have enough time :) | |
| 03:20:59 | Coco_gao | Yes. | |
| 03:21:01 | Coco_gao | Li | |
| 03:21:06 | Sundar | The table currently named ARQ actually has ExtARQ data, since it includes Cyborg-private fields. That's why I said the table is better named as ExtARQ. ExtARQ is a superset of ARQ | |
| 03:21:26 | Li_Liu | that seems a lot of work, you might wanna offload them | |
| 03:22:01 | Coco_gao | xinran, I have the diff function in the old ovo patch, so it won't take long if the db object is ready. | |
| 03:22:26 | Li_Liu | Sundar, you mean there will not be a ARQ table then. only ExtARQ | |
| 03:22:51 | Sundar | Li_Liu: Yes | |
| 03:22:56 | Coco_gao | Li, I think the ExtARQ combines old ARQ and ExtARQ | |
| 03:23:02 | Li_Liu | ok, I will make the change right now | |
| 03:23:11 | Sundar | Li_Liu: Thanks | |
| 03:24:20 | Sundar | For the question of whether attach handles go with deployables or devices, one could argue both ways but I thought it is easier if it goes at the device level. if we want to keep it in the current form, it may require an upgrade later. As long we are all ok with that, I am fine | |
| 03:24:24 | Coco_gao | Li_Liu, I will implement table device and deployable and attribute. Other DB work might owned by zhenghao or anyone else? | |
| 03:24:57 | Sundar | Coco_gao: It may make sense to split the db work, since there are so many. Thanks for suggesting that. | |
| 03:24:59 | Li_Liu | Coco_gao, I will talk with zheng hao and xinran for the work load | |
| 03:25:07 | Yumeng_ | coco I have can help | |
| 03:25:22 | Coco_gao | OK, you can discuss with zhenghao about other DB works. | |
| 03:25:31 | Coco_gao | Thanks, Yumeng_ | |
| 03:25:31 | xinranwang | Coco_gao: ok, this patch is an initial version of conductor update, https://review.openstack.org/#/c/624340/, plz review it to see if it fits your requirements. It need more functions to be implemented. | |
| 03:25:43 | Li_Liu | Sundar, I kinda lean towards let attachHandle reference to Device now | |
| 03:25:51 | Coco_gao | #info Coco_gao | |
| 03:26:42 | Sundar | Li_Liu: OK :) It is all a bit hypothetical because the devices in Stein train frame will work fine either way. | |
| 03:27:38 | Sundar | BTW, even if we put in device level, we have to figure out how to handle a PCI card wih 2 FPGAs, if that ever becomes a thing | |
| 03:27:54 | Li_Liu | sure | |
| 03:28:32 | Coco_gao | xinranwang, I will check your patch, thank you. | |
| 03:28:37 | Li_Liu | ok, I think it's great we can finalize the spec and can push things further | |
| 03:29:28 | Li_Liu | #topic AoB | |
| 03:29:57 | Li_Liu | On top of what we have discussed, do you guys have anything to add? | |
| 03:30:11 | xinranwang | Sundar: do you have any concern about the POC code I put in the community? | |
| 03:30:51 | Coco_gao | Sundar, You mean that cyborg client code is ready? | |
| 03:31:04 | Sundar | xinranwang: My main concern is this is an outdated poc, and it may cause confusion. | |
| 03:31:35 | Coco_gao | or we still need other one to implement that? | |
| 03:31:35 | Sundar | Coco_gao: I have made some changes to Cyborg client too. I will put that on a feature branch too. But it is still WIP | |
| 03:32:38 | xinranwang | Sundar: I understand your concern, it should be clarified to nova that these patches is for reference only | |
| 03:32:52 | Sundar | BTW, clarification: The prototype is not the final code. There will be many TODOs. Our developers are free to re-implement, or fill the gaps while leveraging the prototype | |
| 03:33:34 | Coco_gao | Yes , I think so. | |
| 03:34:03 | Sundar | Coco_gao, Zhenghao and possibly Yumeng_ are working on the db. Xinran is working on the agent/conductor API and the diff. Is that correct? | |
| 03:34:47 | Li_Liu | Sundar I think so | |
| 03:35:10 | Yumeng_ | Sundar: yes, next time please also email me(bao.yumeng@zte.com.cn) if there is any update about the specificaiton :) | |
| 03:35:30 | Coco_gao | Or add yumeng to the reviewer. | |
| 03:35:33 | Sundar | Yumeng_: Sure :) Thanks for helping! | |
| 03:35:40 | Li_Liu | We did not have you before? Yumeng_ | |
| 03:35:48 | Li_Liu | We will in the future | |