| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2021-09-01 | |||
| 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 | GET call to accelerator for http://10.180.210.90:6666/v2/devices used request id req-c1305a47-9003-4bdc-aee8-aa4c63a088e6 | |
| 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 | 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: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? | |
| 12:28:21 | songwenping | no from me | |
| 12:28:23 | brinzhang | please check whether it is right, if it's right, the response body will have a wrong value | |
| 12:28:36 | brinzhang | it's also a bug | |
| 12:28:41 | brinzhang | https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/796795 | |
| 12:29:08 | brinzhang | this patch, IMO, it's ready to go, we dont need the *tls-proxy* tag | |
| 12:29:41 | xinranwang | it is go | |
| 12:29:46 | xinranwang | *good | |
| 12:29:55 | chenke | tls means what? | |
| 12:30:09 | chenke | load balance? | |
| 12:30:49 | brinzhang | chenke: please refer https://etherpad.opendev.org/p/support-tls-default | |
| 12:30:50 | xinranwang | I think it is for service's communication via tls | |
| 12:31:55 | chenke | ok. I will see it. thanks | |
| 12:32:24 | brinzhang | nothing for me | |
| 12:32:26 | brinzhang | xinranwang | |
| 12:32:32 | xinranwang | brinzok | |
| 12:32:34 | xinranwang | ok | |
| 12:32:35 | chenke | yes xinran. it's like ssl | |
| 12:32:49 | chenke | tls like ssl. I agree merge it. | |
| 12:34:03 | xinranwang | cool | |
| 12:34:11 | xinranwang | #topic PTG | |
| 12:34:34 | xinranwang | we have not too much topic for PTG, please continue add topic ther | |
| 12:34:46 | opendevreview | Wenping Song proposed openstack/cyborg master: Fix the arq post api doc https://review.opendev.org/c/openstack/cyborg/+/804937 | |
| 12:35:26 | xinranwang | cool | |
| 12:35:32 | xinranwang | #AoB | |
| 12:35:37 | xinranwang | #topic AoB | |
| 12:35:48 | xinranwang | anything else you want bring up ? | |
| 12:36:13 | brinzhang | I will add some topic to the etherpad while I am free | |
| 12:36:16 | opendevreview | Wenping Song proposed openstack/cyborg master: Fix the wrong link https://review.opendev.org/c/openstack/cyborg/+/804938 | |
| 12:36:45 | songwenping | noting form myside | |
| 12:36:48 | xinranwang | how efficient songwenping is ! | |
| 12:36:50 | chenke | no for me | |
| 12:36:56 | brinzhang | nothing from me | |
| 12:37:11 | songwenping | xinranwang: just rebase master, fast work. | |
| 12:38:15 | opendevreview | Merged openstack/cyborg-tempest-plugin master: Enable tls-proxy support in test jobs https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/796795 | |
| 12:38:59 | xinranwang | ok, let's wrap up the meeting | |
| 12:39:03 | xinranwang | thanks guys | |
| 12:39:07 | opendevmeet | Log: https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-09-01-12.02.log.html | |
| 12:39:07 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-09-01-12.02.txt | |
| 12:39:07 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-09-01-12.02.html | |
| 12:39:07 | opendevmeet | Meeting ended Wed Sep 1 12:39:07 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 12:39:07 | xinranwang | #endmeeting | |
| 12:39:14 | xinranwang | have a good night | |
| 12:39:23 | brinzhang | bye | |