| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2018-03-21 | |||
| 14:51:14 | zhipeng | Sundar gr8t :) | |
| 14:51:26 | zhipeng | #topic rocky spec/patch discussion | |
| 14:51:52 | zhipeng | #link https://review.openstack.org/#/q/status:open+project:openstack/cyborg | |
| 14:52:37 | zhipeng | first up, Sundar's spec patch | |
| 14:53:13 | shaohe_feng_ | Sundar: good work | |
| 14:53:28 | Sundar | Shaohe, thanks :) | |
| 14:53:28 | shaohe_feng_ | but I have a question. why nova developer think the accelerator weigher call cause performance loss | |
| 14:53:39 | zhipeng | #link https://review.openstack.org/#/c/554717/ | |
| 14:53:59 | Sundar | I think the assumption was that the weigher will call into Cyborg REST API for each host | |
| 14:54:09 | Sundar | If the weigher is in Nova tree, that is true | |
| 14:54:25 | Sundar | But, if Cyborg keeps it, we have other options | |
| 14:54:27 | shaohe_feng_ | Sundar: why for each host? | |
| 14:54:51 | Sundar | The typical filter today operates per host | |
| 14:54:52 | shaohe_feng_ | Sundar: I have discussed it before. | |
| 14:55:18 | shaohe_feng_ | Sundar: the cyborg API will run on controller node. | |
| 14:55:32 | shaohe_feng_ | Sundar: we only call the api in controller node. | |
| 14:55:39 | shaohe_feng_ | just on api is OK. | |
| 14:56:13 | shaohe_feng_ | for example, the scheduler filter choose the suitable hosts | |
| 14:56:56 | shaohe_feng_ | and the scheduler weigher just call a API to query the accelerator infos of these hosts | |
| 14:57:05 | shaohe_feng_ | zhipeng: Li_Liu: right? | |
| 14:57:36 | zhipeng | that is still , per host | |
| 14:57:36 | Li_Liu | you mean the weigher is on the Cyborg controller side? | |
| 14:57:56 | shaohe_feng_ | zhipeng: no, we get list for filter API | |
| 14:58:32 | Sundar | Shaohe, yes, we could override the BaseWeigher and handle multiple hosts in one call. That call could invoke Cyborg REST API. | |
| 14:58:44 | shaohe_feng_ | for example: GET /cyborg/v1/accelerators?hosts=cyborg-1,cyborg-2&type=fpga | |
| 14:58:45 | Sundar | To me, it is not clear what the performance hit would be. | |
| 14:59:26 | Sundar | I suspect any performance hit would not be noticeable until we get to some scale | |
| 14:59:28 | Li_Liu | Does this involve the 2-stages scheduling problem we were trying to avoid? | |
| 15:00:10 | Sundar | There is no 2-stage scheduling here: the proposed filter/weigher is a typical one, which just filters hosts based on function calls. | |
| 15:00:26 | shaohe_feng_ | Sundar: yes, the scheduler has call placement several times, is there performance issue? | |
| 15:00:29 | zhipeng | Sundar I think Li_Liu meant for weigher in Nova | |
| 15:00:52 | zhipeng | shaohe_feng_ it is not the same thing | |
| 15:01:13 | shaohe_feng_ | they are both http request. | |
| 15:01:18 | zhipeng | anyways this has been discussed in extent with Nova team and let's stay with the conclusion | |
| 15:01:24 | Li_Liu | ok | |
| 15:01:30 | zhipeng | shaohe_feng_ we could discuss offline more with Alex | |
| 15:01:31 | shaohe_feng_ | OK | |
| 15:01:38 | zhipeng | but let's not dwell on it | |
| 15:01:51 | Sundar | May be I misunderstood :) We are proposing a weigher maintained in Cyborg tree, which the operator will configure in nova.conf. Is that a concern? | |
| 15:02:34 | shaohe_feng_ | weigher maintained in Cyborg tree, still need one cyborg api request, so also performance issue? | |
| 15:03:02 | zhipeng | Sundar I don't think that would be a concern | |
| 15:03:06 | Sundar | shaohe: I personally don't think so, but we'll check the data to assure everybody | |
| 15:03:24 | shaohe_feng_ | zhipeng: Li_Liu: do you think a weigher maintained in Cyborg tree is a good idea? | |
| 15:03:47 | zhipeng | yes, at the moment | |
| 15:04:09 | shaohe_feng_ | Sundar: you still need to tell cyborg which hosts need to weight | |
| 15:04:22 | shaohe_feng_ | on cyborg api call. | |
| 15:04:22 | Sundar | shaohe: This weigher is querying Cyborg DB. It is better to keep it in Cyborg | |
| 15:04:39 | Li_Liu | I agree with zhipeng | |
| 15:04:56 | zhipeng | the weigher will just talk to the conductor | |
| 15:05:02 | zhipeng | it is not blocking nova operations | |
| 15:05:06 | zhipeng | that is the point | |
| 15:05:59 | shaohe_feng_ | any the way, the api call will talk to conductor to query Cyborg DB | |
| 15:06:06 | Sundar | Shaohe, yes. The weigher gets a list of hosts. We could either introduce a new Cyborg API for that, or just have the weigher query the db directly | |
| 15:06:09 | shaohe_feng_ | no difference. | |
| 15:06:23 | zhipeng | shaohe_feng_ let's leave it offline | |
| 15:06:29 | shaohe_feng_ | zhipeng: OK. | |
| 15:06:46 | zhipeng | Sundar thx for the spec :) | |
| 15:06:57 | zhipeng | We will definitely review it more | |
| 15:07:51 | zhipeng | next up, Li Liu's patch | |
| 15:08:05 | zhipeng | #info Implemented the Objects and APIs for vf/pf | |
| 15:08:19 | zhipeng | #link https://review.openstack.org/552734 | |
| 15:08:59 | Sundar | Sorry, I need to leave for my next call. :( Will catch up from minutes | |
| 15:09:12 | Li_Liu | later | |
| 15:09:23 | zhipeng | Sundar no problem | |
| 15:09:40 | zhipeng | Sundar we need another discussion on k8s actions :) | |
| 15:10:11 | zhipeng | Li_Liu any update or comment on your patch ? | |
| 15:10:22 | zhipeng | things we need to be aware of ? | |
| 15:10:59 | Li_Liu | not really. I think shaohe needs to change some code in his resource tracker to adopt the change | |
| 15:11:35 | Li_Liu | other than that. one thing left over is to utilize the attribute table in deployables. This is still a missing piece | |
| 15:11:41 | shaohe_feng_ | Li_Liu: thanks for reminder. | |
| 15:12:12 | Li_Liu | I will keep working on that as well as 2 other specs | |
| 15:12:49 | Li_Liu | shaohe_feng_ np. let me know if you need any help on using my pf/vfs | |
| 15:12:49 | shaohe_feng_ | Li_Liu: 2 other specs include image management? | |
| 15:13:00 | shaohe_feng_ | Li_Liu: ok, thanks. | |
| 15:13:23 | Li_Liu | programability and image metadata standardization | |
| 15:13:44 | zhipeng | yes , big task on your shoulder :) | |
| 15:14:05 | Li_Liu | :) | |
| 15:15:45 | zhipeng | okey we've gone through our agenda list today | |
| 15:15:54 | zhipeng | I think we can end the meeting now :) | |
| 15:16:04 | zhipeng | and talk to you guys next week | |
| 15:18:11 | kosamara | bye | |
| 15:19:05 | Yumeng__ | bye | |
| 15:19:38 | zhipeng | #endmeeting | |
| 15:19:40 | openstack | Meeting ended Wed Mar 21 15:19:38 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 15:19:41 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2018/openstack_cyborg.2018-03-21-14.00.html | |
| 15:19:42 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2018/openstack_cyborg.2018-03-21-14.00.txt | |
| 15:19:43 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2018/openstack_cyborg.2018-03-21-14.00.log.html | |
| #openstack-cyborg - 2018-03-22 | |||
| 08:17:16 | openstackgerrit | YumengBao proposed openstack/cyborg-specs master: Initialize cyborg-specs repository https://review.openstack.org/554766 | |
| #openstack-cyborg - 2018-03-23 | |||
| 12:24:00 | NokMikeR | Hi, is there a list of cards or specific fpga boards thats currently supported by cyborg? | |
| 12:24:42 | zhipengh[m] | I think there is only Intel card now | |
| 12:25:25 | NokMikeR | IS there a specific model number for that card? | |
| 12:26:33 | zhipengh[m] | I can ask | |
| 12:26:54 | NokMikeR | Thank you | |
| #openstack-cyborg - 2018-03-26 | |||
| 21:00:54 | openstackgerrit | Merged openstack/cyborg master: Add default configuration files to data_files https://review.openstack.org/553463 | |
| 21:00:54 | openstackgerrit | Merged openstack/cyborg master: Remove etc/cyborg/cyborg.conf.sample from project https://review.openstack.org/553460 | |
| #openstack-cyborg - 2018-03-27 | |||
| 03:34:45 | Sundar_ | Hi Zhu Li | |
| #openstack-cyborg - 2018-03-29 | |||
| 15:30:04 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: bug fix: devstack can not start cyborg https://review.openstack.org/557742 | |
| 16:15:32 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: bug fix: devstack can not start cyborg https://review.openstack.org/557742 | |
| 20:41:46 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: bug fix: devstack can not start cyborg https://review.openstack.org/557742 | |
| #openstack-cyborg - 2018-03-30 | |||
| 19:29:15 | catalyst659 | .-. .-. | |