| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2021-08-20 | |||
| 06:04:11 | opendevreview | Merged openstack/python-cyborgclient master: Replace deprecated inspect.getargspec https://review.opendev.org/c/openstack/python-cyborgclient/+/804503 | |
| 06:10:29 | opendevreview | Merged openstack/cyborg master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/cyborg/+/804468 | |
| 08:25:41 | opendevreview | wangzhiguang proposed openstack/cyborg-tempest-plugin master: Add assertion information for batch delete device profile testcase https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/795199 | |
| 09:39:10 | opendevreview | wangzhiguang proposed openstack/cyborg-tempest-plugin master: Add assertion information for batch delete device profile testcase https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/795199 | |
| #openstack-cyborg - 2021-08-23 | |||
| 08:16:00 | opendevreview | Wenping Song proposed openstack/python-cyborgclient master: Fix the bug of creating arq with device profile group id required https://review.opendev.org/c/openstack/python-cyborgclient/+/805595 | |
| 08:45:04 | opendevreview | Wenping Song proposed openstack/python-cyborgclient master: Add description args for device profile create https://review.opendev.org/c/openstack/python-cyborgclient/+/803886 | |
| 08:49:01 | opendevreview | Wenping Song proposed openstack/cyborg master: Add description comment for device profile create https://review.opendev.org/c/openstack/cyborg/+/804052 | |
| 10:03:47 | qiujunting | https://review.opendev.org/c/openstack/nova/+/802225 | |
| 10:10:09 | brinzhang | qiujunting: you should move this topic to nova channel | |
| 10:37:26 | opendevreview | Merged openstack/python-cyborgclient master: Add checks for showing dp with name https://review.opendev.org/c/openstack/python-cyborgclient/+/802140 | |
| #openstack-cyborg - 2021-08-24 | |||
| 02:16:02 | opendevreview | Wenping Song proposed openstack/python-cyborgclient master: Fix the bug of making device_profile_group_id parameter is optional https://review.opendev.org/c/openstack/python-cyborgclient/+/805595 | |
| 06:04:41 | opendevreview | Wenping Song proposed openstack/python-cyborgclient master: Add test case for show arq https://review.opendev.org/c/openstack/python-cyborgclient/+/805781 | |
| #openstack-cyborg - 2021-08-26 | |||
| 06:04:20 | opendevreview | Wenping Song proposed openstack/python-cyborgclient master: Add test case for show arq https://review.opendev.org/c/openstack/python-cyborgclient/+/805781 | |
| 09:44:58 | opendevreview | Wenping Song proposed openstack/python-cyborgclient master: Add test case for show arq https://review.opendev.org/c/openstack/python-cyborgclient/+/805781 | |
| 10:55:55 | opendevreview | Merged openstack/python-cyborgclient master: Fix the bug of making device_profile_group_id parameter is optional https://review.opendev.org/c/openstack/python-cyborgclient/+/805595 | |
| 11:00:46 | opendevreview | Wenping Song proposed openstack/python-cyborgclient master: Add test case for show arq https://review.opendev.org/c/openstack/python-cyborgclient/+/805781 | |
| #openstack-cyborg - 2021-08-27 | |||
| 10:25:26 | opendevreview | Merged openstack/python-cyborgclient master: Add test case for show arq https://review.opendev.org/c/openstack/python-cyborgclient/+/805781 | |
| 10:41:36 | opendevreview | Merged openstack/cyborg master: Remove the redundant index page for APIs entry https://review.opendev.org/c/openstack/cyborg/+/804553 | |
| #openstack-cyborg - 2021-08-31 | |||
| 01:25:38 | opendevreview | Merged openstack/cyborg master: Erase previously collected coverage for cybrog https://review.opendev.org/c/openstack/cyborg/+/799019 | |
| #openstack-cyborg - 2021-09-01 | |||
| 07:16:20 | opendevreview | Merged openstack/cyborg-tempest-plugin master: Add assertion information for batch delete device profile testcase https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/795199 | |
| 11:59:57 | xinranwang | hi all | |
| 12:00:19 | brinzhang | hi | |
| 12:00:43 | chenke | hi all | |
| 12:01:23 | xinranwang | hi brinzhang | |
| 12:01:26 | xinranwang | hi chenke | |
| 12:01:35 | xinranwang | is songwenping around? | |
| 12:01:54 | songwenping | hi all | |
| 12:01:59 | songwenping | yes, i am here | |
| 12:02:10 | xinranwang | cool | |
| 12:02:16 | xinranwang | let's start the meeting | |
| 12:02:18 | brinzhang | ^ | |
| 12:02:26 | xinranwang | #startmeeting openstack-cyborg | |
| 12:02:26 | opendevmeet | Meeting started Wed Sep 1 12:02:26 2021 UTC and is due to finish in 60 minutes. The chair is xinranwang. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 12:02:26 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 12:02:26 | opendevmeet | The meeting name has been set to 'openstack_cyborg' | |
| 12:02:36 | xinranwang | #topic FF this week | |
| 12:03:35 | xinranwang | sep 3 is feature freeze, we need land all important patch before that | |
| 12:03:49 | brinzhang | yeah | |
| 12:04:06 | songwenping | ack | |
| 12:04:09 | xinranwang | do you have any patch need to be landed before that? | |
| 12:04:45 | xinranwang | please paste it here, let's do it as high priority | |
| 12:05:04 | chenke | no for me. thanks | |
| 12:06:10 | brinzhang | https://review.opendev.org/c/openstack/cyborg/+/804559 this microversion history page, IMO, it's important | |
| 12:07:20 | xinranwang | cool, please review it | |
| 12:07:22 | xinranwang | hi qiujunting | |
| 12:07:41 | qiujunting | hi xinranwang | |
| 12:08:06 | songwenping | brinzhang: passed | |
| 12:08:20 | brinzhang | cool, thanks | |
| 12:08:33 | brinzhang | hi qiujunting ^^ | |
| 12:08:36 | brinzhang | good afternoon | |
| 12:08:42 | chenke | hi qiujunting | |
| 12:08:52 | chenke | welcome | |
| 12:09:15 | qiujunting | hi brinzhang | |
| 12:09:20 | brinzhang | xinranwang: how about this patch https://review.opendev.org/c/openstack/cyborg/+/796594 | |
| 12:09:22 | songwenping | i have two patches that conflict with master | |
| 12:09:24 | qiujunting | hi chenke | |
| 12:09:44 | songwenping | i'll rebase soon. | |
| 12:10:01 | brinzhang | I try to test in my local env, but the environment cannot run fine | |
| 12:10:10 | brinzhang | s/try/tried | |
| 12:10:43 | xinranwang | eric is right, it is devstack who add the 'accelerator' in the url | |
| 12:11:17 | songwenping | no eric is wron | |
| 12:11:19 | songwenping | g | |
| 12:11:32 | xinranwang | but i think we should use accelerator in url by default, I mean, without devstack's setting | |
| 12:11:57 | songwenping | i've tested on my env | |
| 12:13:04 | songwenping | eric has wrong setting on his deployment env | |
| 12:13:47 | xinranwang | how did you test, with devstack or kolla? | |
| 12:14:29 | songwenping | none of them, openstack on k8s | |
| 12:15:45 | xinranwang | what deployment tools you use? | |
| 12:16:31 | songwenping | i use kolla-ansible | |
| 12:16:41 | songwenping | he's endpoint url is wrong | |
| 12:17:22 | xinranwang | i have use kolla ansible too, and url is without "accelerator" | |
| 12:18:29 | songwenping | we donnot need accelerator | |
| 12:18:33 | songwenping | with kolla-ansible | |
| 12:19:00 | songwenping | just use port 6666 instead | |
| 12:19:04 | xinranwang | yes | |
| 12:19:41 | xinranwang | but if we deploy by devstack, the url is with "accelerator", this is because devtack add it | |
| 12:20:12 | songwenping | with devstack we need 'accelerator' | |
| 12:20:24 | brinzhang | xinranwang: you mean, "accelerator" added by devstack? | |
| 12:20:30 | songwenping | devstack has no port | |
| 12:20:47 | xinranwang | brinzhang: yes | |
| 12:20:57 | songwenping | either we use 'accelerator' or 'port' is ok. | |
| 12:21:48 | songwenping | kolla-ansible is running cyborg with 'cyborg-api' process. | |
| 12:22:00 | xinranwang | shall we keep the url same either with kolla or devstack? | |
| 12:22:03 | songwenping | devstack use wsgi | |
| 12:22:43 | songwenping | no need | |
| 12:22:58 | songwenping | other components like nova and cinder is same with cyborg | |
| 12:23:07 | brinzhang | http://10.180.210.90:6666/accelerator/v2/devices/ | |
| 12:23:22 | brinzhang | # openstack --debug accelerator device list | |
| 12:23:34 | brinzhang | RESP BODY: {"devices": [{"rp_uuid": "03631f82-20a9-3f67-a29d-dc1abe4041bf", "vendor": "0xABCD", "uuid": "da024ddb-e940-4006-80e1-5bf1de8862fa", "links": [{"href": "http://10.180.210.90:6666/accelerator/v2/devices/da024ddb-e940-4006-80e1-5bf1de8862fa", "rel": "self"}], "in_use": false, "attribute_type": "FPGA", "created_at": "2021-08-25T06:58:58+00:00", "hostname": "control01", "updated_at": null, "name": "miss model info", "product_id": | |
| 12:23:34 | brinzhang | null, "vendor_value": null, "vendor_board_info": "fake_vendor_info", "model": "miss model info", "type": "FPGA", "id": 1, "address": "", "std_board_info": "{\"class\": \"Fake class\", \"device_id\": \"0xabcd\"}"}]} | |
| 12:23:34 | brinzhang | GET call to accelerator for http://10.180.210.90:6666/v2/devices used request id req-c1305a47-9003-4bdc-aee8-aa4c63a088e6 | |
| 12:23:56 | brinzhang | this is returned in kolla env | |
| 12:25:25 | songwenping | the links of devices returned is wrong | |
| 12:26:25 | xinranwang | ok, got it | |
| 12:26:34 | xinranwang | so we can keep it as it is | |
| 12:26:50 | xinranwang | songwenping could you please help to reply to eric | |
| 12:27:19 | songwenping | ok i'll talk with him tommorrow | |
| 12:27:38 | xinranwang | cool | |
| 12:27:41 | xinranwang | thanks songwenping | |
| 12:27:47 | xinranwang | any other patches? | |