| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2020-01-09 | |||
| 03:08:09 | Yumeng | I will focus on reviewing too, these days | |
| 03:08:10 | Sundar | I just called out a few patches that I think may need some discussion | |
| 03:08:18 | Sundar | #topic Patches | |
| 03:08:19 | Sundar | https://review.opendev.org/#/c/699099/ | |
| 03:08:49 | Sundar | One comment was: "Can devices in cyborg be associated to specific projects?" | |
| 03:09:01 | Sundar | This is a use case worth discussing here, right? | |
| 03:10:52 | Sundar | What do you guys think of this use case? Yumeng, chenke, s_shogo, wangzhh | |
| 03:11:16 | Li_Liu | #info Li_Liu | |
| 03:11:57 | Yumeng | sundar: are you thinking Lance Bragstad's question? Can devices in cyborg be associated to specific projects? If so, this check should ensure the user has reader authorization on the project in addition to the 'reader' role, correct? | |
| 03:12:19 | Sundar | Hi Li, xinranwang: we are discussing a comment in https://review.opendev.org/#/c/699099/ i.e. "Can devices in cyborg be associated to specific projects?" | |
| 03:12:22 | xinranwang | Hi all, sorry for late | |
| 03:12:23 | Sundar | Yumeng: yes | |
| 03:12:44 | xinranwang | #info xinranwang | |
| 03:13:01 | Sundar | xinranwang: np | |
| 03:13:44 | xinranwang | I think devices can be shared between different project, what do you guys think? | |
| 03:13:52 | Li_Liu | Hi Sundar | |
| 03:13:55 | chenke | I don't think it is necessary. | |
| 03:14:01 | chenke | Sundar | |
| 03:14:33 | Yumeng | agree with Xinran | |
| 03:14:37 | Li_Liu | The "project" concept here is in the Openstack context right? | |
| 03:14:41 | wangzhh | xinranwang, Agree. | |
| 03:15:23 | s_shogo | xinranwang , +1 | |
| 03:15:25 | wangzhh | Li_Liu, Yes | |
| 03:15:29 | Sundar | By default, a device can be shared across different projects. The questions seems to be whether there is value in allowing the admin to allocate a set of devices for a specific project. | |
| 03:16:03 | Li_Liu | Agree, since cpu/memory do not associate with project neither | |
| 03:16:35 | chenke | Agree | |
| 03:16:50 | chenke | Acceleration resources should be the same as CPU. | |
| 03:16:59 | xinranwang | agree | |
| 03:17:19 | Sundar | Ok, if we decide not to do this in Ussuri and change our mind later, that is not a problem, AFAICS. I don't see any upgrade issues. Anybody has a different thought? | |
| 03:19:52 | Sundar | Apparently not. So, let us agree to not have an option of dedicating a device to a project. | |
| 03:20:08 | Li_Liu | :P | |
| 03:20:25 | Sundar | I'll take that as a yes :) | |
| 03:20:52 | Sundar | Next patch: https://review.opendev.org/670973 Bump openstackdocs theme | |
| 03:21:18 | Sundar | This has been sitting for a while. I am not an expert here. But it seems safe to merge this change. What do you all think? | |
| 03:22:24 | Sundar | Li_Liu, Yumeng, chenke, xinranwang, s_shogo, wangzhh: ^ | |
| 03:22:46 | chenke | Agree Merge +1. | |
| 03:23:39 | Sundar | Ok, chenke, let's both review it and merge it. Thanks. | |
| 03:23:47 | chenke | Ok. | |
| 03:24:18 | Sundar | Drawing all your attention to the ML thread: http://lists.openstack.org/pipermail/openstack-discuss/2020-January/011802.html | |
| 03:24:45 | Sundar | I haven;t caught up fully, and will respond soon-ish. If you have any thoughts, we can discuss here. | |
| 03:26:09 | Li_Liu | That seems pretty long. I will try to read through it as well | |
| 03:26:17 | Sundar | Sure | |
| 03:26:23 | Sundar | #topic AoB | |
| 03:26:30 | chenke | http://lists.openstack.org/pipermail/openstack-discuss/2020-January/011803.html | |
| 03:26:52 | chenke | zhipeng huang had an email reply this. | |
| 03:28:31 | Sundar | Yes, chenke. But do we all agree with Arkady's idea? Any points not addressed in the email thread so far. BTW, Arkady was at the PTG and raised issues re. firmware updates, which i captured in https://etherpad.openstack.org/p/cyborg-ussuri-programming-apis | |
| 03:29:45 | chenke | I haven't read this email carefully. I will learn your response. | |
| 03:29:50 | Sundar | In my understanding, Ironic deals with devices as just PCI devices, but does inspect their properties | |
| 03:30:33 | Sundar | A Cyborg driver can do that. I am not sure that an entire Cyborg driver or its equivalent can be included in the Ironic inspection process. | |
| 03:30:57 | Sundar | However, Arkady brings up issues regarding server lifecycle management, warranties, etc. | |
| 03:31:33 | xinranwang | Yes, ironic can only inspect general pci device, but for example, cyborg can discover vf/pf topology and AFU ID as well. | |
| 03:31:50 | xinranwang | will read through it later | |
| 03:31:57 | Sundar | s/but does inspect/but does NOT inspect/ | |
| 03:32:34 | Sundar | Anything else to cover today? | |
| 03:32:44 | Yumeng | Sundar: seems need more thinking. I will dig the ML more. and Re:https://review.opendev.org/#/c/701518/ Yes. I was testing with a real GPU | |
| 03:33:49 | Sundar | Yumeng: good to know | |
| 03:34:00 | Yumeng | I was testing GPU driver and pls help to review. Li_Liu, chenke, xinranwang, s_shogo, Sundar,wangzhh: ^ | |
| 03:34:10 | Yumeng | https://review.opendev.org/#/c/701518/ | |
| 03:34:14 | xinranwang | Yumeng: ok, will review it later | |
| 03:34:29 | Yumeng | ^_^ thanks! | |
| 03:34:45 | Sundar | Please keep continuing the good work, folks! Thanks a lot. Happy New Year, and have a good day! | |
| 03:34:49 | Sundar | Bye | |
| 03:34:51 | wangzhh | Yumeng, Yep, some comments in line. | |
| 03:34:52 | Yumeng | bye | |
| 03:34:54 | Sundar | #endmeeting | |
| 03:34:56 | wangzhh | Bye | |
| 03:34:56 | openstack | Meeting ended Thu Jan 9 03:34:53 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 03:34:57 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-01-09-03.03.html | |
| 03:34:58 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-01-09-03.03.txt | |
| 03:34:59 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-01-09-03.03.log.html | |
| 03:35:15 | Yumeng | wangzhh: Thanks! I will check. :) | |
| #openstack-cyborg - 2020-01-10 | |||
| 02:07:01 | openstackgerrit | chenker proposed openstack/cyborg master: Use ResourceNotFound replace AttachHandleNotFound https://review.opendev.org/700264 | |
| 06:36:26 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Implementation of Device v2 API https://review.opendev.org/695648 | |
| 14:42:04 | sean-k-mooney | what is the repo name for the cyborg client i assuemt here is an osc pluging i should install? | |
| 14:42:55 | sean-k-mooney | just grabing lunch so ill check back in a while but im going to try and deploy the nova integration poc when i get back and play around with it | |
| 14:43:27 | sean-k-mooney | i jsut installed with the cyborg devstack plugin but it does not seam to have installed the client | |
| 22:16:27 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: Send a separate bind event to Nova for each ARQ in an instance. https://review.opendev.org/698846 | |
| #openstack-cyborg - 2020-01-11 | |||
| 19:39:42 | openstackgerrit | Merged openstack/cyborg master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/670973 | |
| #openstack-cyborg - 2020-01-13 | |||
| 02:58:24 | openstackgerrit | chenker proposed openstack/cyborg master: Use ResourceNotFound replace AttachHandleNotFound https://review.opendev.org/700264 | |
| 08:03:12 | openstackgerrit | Xinran WANG proposed openstack/cyborg-specs master: Specification of Cyborg API microversion. https://review.opendev.org/699149 | |
| #openstack-cyborg - 2020-01-14 | |||
| 02:26:04 | openstackgerrit | Merged openstack/cyborg master: Use ResourceNotFound replace DeviceProfileNotFound https://review.opendev.org/700262 | |
| 03:03:37 | openstackgerrit | Merged openstack/cyborg master: Implementation of Device v2 API https://review.opendev.org/695648 | |
| 03:12:24 | openstackgerrit | YumengBao proposed openstack/cyborg master: Update gpu driver https://review.opendev.org/701518 | |
| 03:35:17 | openstackgerrit | chenker proposed openstack/cyborg master: Use ResourceNotFound replace DeviceNotFound https://review.opendev.org/700263 | |
| 03:35:27 | openstackgerrit | chenker proposed openstack/cyborg master: Use ResourceNotFound replace DeployableNotFound https://review.opendev.org/701329 | |
| 03:58:50 | openstackgerrit | Merged openstack/cyborg master: Introduce bandit security linter https://review.opendev.org/700568 | |
| 06:21:02 | openstackgerrit | YumengBao proposed openstack/cyborg master: Set default value in get fpga trait https://review.opendev.org/702353 | |
| 08:38:38 | openstackgerrit | Merged openstack/cyborg master: Add created_time and update_time for device profile API https://review.opendev.org/700973 | |
| 13:48:36 | openstackgerrit | Li Liu proposed openstack/cyborg master: Added initial dummt functional test https://review.opendev.org/702428 | |
| 13:52:34 | openstackgerrit | Li Liu proposed openstack/cyborg master: Added initial dummt functional test https://review.opendev.org/702428 | |
| #openstack-cyborg - 2020-01-15 | |||
| 03:28:47 | openstackgerrit | chenker proposed openstack/cyborg master: Improve UT for DB device profile https://review.opendev.org/702578 | |
| 04:39:26 | openstackgerrit | Merged openstack/cyborg master: Use ResourceNotFound replace DeviceNotFound https://review.opendev.org/700263 | |
| 04:39:29 | openstackgerrit | Merged openstack/cyborg master: Use ResourceNotFound replace ExtArq and Attribute NotFound https://review.opendev.org/701331 | |
| 06:01:43 | openstackgerrit | chenker proposed openstack/cyborg master: Use ResourceNotFound replace DeployableNotFound https://review.opendev.org/701329 | |
| 06:11:36 | brinzhang | chenke: hi， I saw you push this patch https://review.opendev.org/#/c/702578/, I have a doubt, should we remove the context from device profile API? | |
| 06:11:36 | brinzhang | chenke: hi， I saw you push this patch https://review.opendev.org/#/c/702578/, I have a doubt, should we remove the context from device profile API? | |
| 07:45:24 | openstackgerrit | YumengBao proposed openstack/cyborg master: Update gpu driver https://review.opendev.org/701518 | |
| 08:30:42 | openstackgerrit | chenker proposed openstack/cyborg master: Improve UT for DB device profile https://review.opendev.org/702578 | |
| 09:28:10 | openstackgerrit | Merged openstack/cyborg master: Use ResourceNotFound replace AttachHandleNotFound https://review.opendev.org/700264 | |
| 09:31:19 | openstackgerrit | Merged openstack/cyborg master: Use ResourceNotFound replace DeployableNotFound https://review.opendev.org/701329 | |