| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2019-06-12 | |||
| 03:44:15 | ikuo_o | yes, It include adopting openstacksdk. | |
| 03:44:32 | Sundar | ok | |
| 03:44:46 | ikuo_o | we are preparing write storyboard, pls wait. | |
| 03:44:51 | Sundar | Re. Cyborg patches, I approved the generic driver and got it merged. Thanks, yikun | |
| 03:45:14 | Sundar | Also, approved a bunch of small patches on fixing opendev URL or email addresses | |
| 03:46:11 | yikun | Sundar: thanks, but we should first https://review.opendev.org/#/c/658986 first, :) | |
| 03:46:39 | Sundar | I have partial fixes for the pilot code comments. Unfortunately, I could not dedicate a few days to get this done completely. | |
| 03:47:37 | Sundar | yikun: Agreed. Missed that | |
| 03:48:14 | Sundar | Will refresh the pilot code this week with whatever I have. | |
| 03:48:33 | Sundar | @all, please do review the specs | |
| 03:48:59 | Sundar | Can we aim to get one set of reviews by this week? | |
| 03:49:36 | Sundar | #link https://review.opendev.org/#/q/status:open+project:openstack/cyborg-specs+owner:Sundar | |
| 03:50:56 | Sundar | wangzhh, Li_Liu, yikun, xinranwang, all: Can we aim to get one set of reviews for the specs by this week? | |
| 03:51:59 | ikuo_o | sorry, poor connection | |
| 03:52:25 | Sundar | NP, ikuo_o. I was asking if we can get the specs reviewed by this week. | |
| 03:53:04 | ikuo_o | ok, I will check logs. | |
| 03:53:21 | Sundar | #topic AoB | |
| 03:53:43 | Sundar | I think we can skip the Zoom call tomorrow, and use that time to get the specs reviewed | |
| 03:54:41 | ikuo_o | all right, thank you Sundar. | |
| 03:55:07 | Sundar | Thanks, ikuo_o and all. Have a good day. Bye. | |
| 03:55:10 | Sundar | #endmeeting | |
| 03:55:12 | openstack | Meeting ended Wed Jun 12 03:55:10 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 03:55:13 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-06-12-03.02.html | |
| 03:55:14 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-06-12-03.02.txt | |
| 03:55:16 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-06-12-03.02.log.html | |
| 16:02:55 | Sundar | yikun, xinranwang, Biwei: Please see http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-06-12.log.html#t2019-06-12T15:55:18 | |
| #openstack-cyborg - 2019-06-14 | |||
| 07:14:39 | openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add the fake driver https://review.opendev.org/665318 | |
| 07:43:51 | openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add the fake driver https://review.opendev.org/665318 | |
| #openstack-cyborg - 2019-06-17 | |||
| 14:02:48 | openstackgerrit | chenker proposed openstack/cyborg-tempest-plugin master: Switch to the new canonical constraints URL on master https://review.opendev.org/665678 | |
| 14:06:28 | openstackgerrit | chenker proposed openstack/cyborg-specs master: Switch to the new canonical constraints URL on master https://review.opendev.org/665679 | |
| #openstack-cyborg - 2019-06-18 | |||
| 01:56:17 | openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add the fake driver https://review.opendev.org/665318 | |
| 07:56:40 | openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Fix the section of enabled_drivers https://review.opendev.org/660603 | |
| 08:08:59 | openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add AICHIP and GENERIC type in DB and obj https://review.opendev.org/660548 | |
| 08:09:26 | openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add Huawei Ascend driver https://review.opendev.org/660874 | |
| #openstack-cyborg - 2019-06-19 | |||
| 03:00:08 | Coco_gao | Hi all | |
| 03:00:17 | Biwei | Hi | |
| 03:01:46 | Sundar | #startmeeting openstack-cyborg | |
| 03:01:47 | openstack | Meeting started Wed Jun 19 03:01:46 2019 UTC and is due to finish in 60 minutes. The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 03:01:48 | openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 03:01:50 | openstack | The meeting name has been set to 'openstack_cyborg' | |
| 03:01:58 | Sundar | #topic Roll call | |
| 03:02:16 | Sundar | #info Sundar | |
| 03:02:19 | Biwei | #info Biwei | |
| 03:02:28 | Coco_gao | #info Coco_gao | |
| 03:02:49 | Sundar | Hi Biwei, Coco_gao | |
| 03:02:57 | Biwei | Hi Sundar | |
| 03:03:30 | Sundar | Let's wait a minute for folks to join | |
| 03:03:35 | Sundar | Hi wangzhh | |
| 03:03:41 | Coco_gao | I am sorry that I may miss two week's meeting | |
| 03:03:47 | wangzhh | Hi Sundar. :) | |
| 03:04:14 | Sundar | Coco_gao: You will miss next 2 weeks? | |
| 03:04:31 | Coco_gao | Nope | |
| 03:04:34 | Coco_gao | I missed | |
| 03:04:55 | Sundar | Ah, got it. NP. | |
| 03:04:58 | Coco_gao | last two week's meeting. | |
| 03:05:02 | Sundar | Agenda: https://wiki.openstack.org/wiki/Meetings/CyborgTeamMeeting#Agenda | |
| 03:05:49 | Sundar | Please take a look. | |
| 03:05:55 | ikuo_o | #info ikuo_o | |
| 03:06:10 | Sundar | Anything to be added to the agenda? | |
| 03:06:45 | Sundar | #topic Device profile names | |
| 03:07:16 | Sundar | One question that has come up, esp. in the review of https://review.opendev.org/#/c/626057/, is whether device profiles are unique, or should be unique | |
| 03:07:56 | Sundar | My answer is in the response to the comment in Line 466 of https://review.opendev.org/#/c/626057/10/cyborg/db/sqlalchemy/api.py | |
| 03:08:06 | Sundar | Please take a look first. Then, we'll discuss. | |
| 03:09:06 | Sundar | Can you all see it? | |
| 03:09:48 | wangzhh | Yep. | |
| 03:11:17 | wangzhh | As my comment in line. The name should be unique. | |
| 03:11:35 | Coco_gao | In the db, it was unique | |
| 03:12:24 | Coco_gao | I see Li posted that the name should not be used to identify. | |
| 03:12:59 | Sundar | wangzhh: https://review.opendev.org/gitweb?p=openstack/cyborg.git;f=cyborg/db/sqlalchemy/models.py;hb=refs/changes/57/626057/10#l183 | |
| 03:13:04 | Sundar | It is unique in the db | |
| 03:13:51 | Sundar | Coco_gao: given my response, do you agree that names must be unique and should be used to identify DPs? I am ok with using UUIDs also, if there is a need | |
| 03:15:43 | wangzhh | Cool. I think it's better to identify DP with UUID. | |
| 03:15:49 | Coco_gao | For other projects, we usually use UUIDs not the names. | |
| 03:16:52 | Coco_gao | +1 better to use UUID | |
| 03:17:27 | Sundar | Yes. But, if we use UUIDs in all APIs, the operator would have to embed DP uuids in flavors, instead of names. | |
| 03:18:26 | wangzhh | We just use UUIDs in internal API. For the rest api, name is more friendly. | |
| 03:18:50 | ikuo_o | Can we find UUIDs from name by using APIs? | |
| 03:19:14 | wangzhh | Just like nova. We can support nova show VM_UUID/VN_NAME. | |
| 03:19:17 | Sundar | ikuo_o: Yes, at the cost of a db access | |
| 03:19:38 | ikuo_o | thanks. | |
| 03:20:08 | Sundar | wangzhh: Internal APIs -- e.g. from API layer to object layer -- can use UUIDs. The problem then is that the API layer needs to do a db lookup to get the uuid from the name, right? | |
| 03:20:26 | wangzhh | Yes. | |
| 03:20:58 | Coco_gao | So the name should also be unique? | |
| 03:21:22 | Coco_gao | one uuid corrosponds to one name? | |
| 03:21:47 | Sundar | Coco_gao: The name has to be unique anyway. If not, when the operator puts a dp name in a flavor, it would be ambiguous. | |
| 03:21:56 | Sundar | Yes, one uuid == one name | |
| 03:22:40 | Sundar | I think an API to rename a device profile would have to use UUIDs. We don't have that today. | |
| 03:23:00 | Sundar | We can do that in the future once Cyborg is is use | |
| 03:23:05 | Sundar | *in use | |
| 03:23:10 | Coco_gao | Agree | |
| 03:23:28 | wangzhh | +1 Great. | |
| 03:23:38 | Coco_gao | UUIDs is not urgent before Cyborg is in use | |
| 03:24:23 | Coco_gao | If we have time, better to add that, but that's not urgent so far. | |
| 03:25:21 | Sundar | One caveat: we have /v2/device_profiles/{name} as a valid URL | |
| 03:25:46 | Sundar | So, if we want a specific UUID, it would have to be a query like: /v2/device_profiles?uuid={uuid} | |
| 03:26:35 | Sundar | But Nova does not use the former: it only does /v2/device_profiles?nam={name} | |
| 03:26:48 | Sundar | So, we have some flexibility to change the former | |
| 03:29:57 | Sundar | Any thoughts? | |
| 03:31:10 | Coco_gao | You mean we should focus on the name API then see if we need UUID API? | |
| 03:31:19 | Sundar | Yes | |