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 | 07:16 |
---|---|---|
xinranwang | hi all | 11:59 |
*** brinzhang_ is now known as brinzhang | 12:00 | |
brinzhang | hi | 12:00 |
chenke | hi all | 12:00 |
xinranwang | hi brinzhang | 12:01 |
xinranwang | hi chenke | 12:01 |
xinranwang | is songwenping around? | 12:01 |
songwenping | hi all | 12:01 |
songwenping | yes, i am here | 12:01 |
xinranwang | cool | 12:02 |
xinranwang | let's start the meeting | 12:02 |
brinzhang | ^ | 12:02 |
xinranwang | #startmeeting openstack-cyborg | 12:02 |
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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 12:02 |
opendevmeet | The meeting name has been set to 'openstack_cyborg' | 12:02 |
xinranwang | #topic FF this week | 12:02 |
xinranwang | sep 3 is feature freeze, we need land all important patch before that | 12:03 |
brinzhang | yeah | 12:03 |
songwenping | ack | 12:04 |
xinranwang | do you have any patch need to be landed before that? | 12:04 |
xinranwang | please paste it here, let's do it as high priority | 12:04 |
chenke | no for me. thanks | 12:05 |
brinzhang | https://review.opendev.org/c/openstack/cyborg/+/804559 this microversion history page, IMO, it's important | 12:06 |
xinranwang | cool, please review it | 12:07 |
xinranwang | hi qiujunting | 12:07 |
qiujunting | hi xinranwang | 12:07 |
songwenping | brinzhang: passed | 12:08 |
brinzhang | cool, thanks | 12:08 |
brinzhang | hi qiujunting ^^ | 12:08 |
brinzhang | good afternoon | 12:08 |
chenke | hi qiujunting | 12:08 |
chenke | welcome | 12:08 |
qiujunting | hi brinzhang | 12:09 |
brinzhang | xinranwang: how about this patch https://review.opendev.org/c/openstack/cyborg/+/796594 | 12:09 |
songwenping | i have two patches that conflict with master | 12:09 |
qiujunting | hi chenke | 12:09 |
songwenping | i'll rebase soon. | 12:09 |
brinzhang | I try to test in my local env, but the environment cannot run fine | 12:10 |
brinzhang | s/try/tried | 12:10 |
xinranwang | eric is right, it is devstack who add the 'accelerator' in the url | 12:10 |
songwenping | no eric is wron | 12:11 |
songwenping | g | 12:11 |
xinranwang | but i think we should use accelerator in url by default, I mean, without devstack's setting | 12:11 |
songwenping | i've tested on my env | 12:11 |
songwenping | eric has wrong setting on his deployment env | 12:13 |
xinranwang | how did you test, with devstack or kolla? | 12:13 |
songwenping | none of them, openstack on k8s | 12:14 |
xinranwang | what deployment tools you use? | 12:15 |
songwenping | i use kolla-ansible | 12:16 |
songwenping | he's endpoint url is wrong | 12:16 |
xinranwang | i have use kolla ansible too, and url is without "accelerator" | 12:17 |
songwenping | we donnot need accelerator | 12:18 |
songwenping | with kolla-ansible | 12:18 |
songwenping | just use port 6666 instead | 12:19 |
xinranwang | yes | 12:19 |
xinranwang | but if we deploy by devstack, the url is with "accelerator", this is because devtack add it | 12:19 |
songwenping | with devstack we need 'accelerator' | 12:20 |
brinzhang | xinranwang: you mean, "accelerator" added by devstack? | 12:20 |
songwenping | devstack has no port | 12:20 |
xinranwang | brinzhang: yes | 12:20 |
songwenping | either we use 'accelerator' or 'port' is ok. | 12:20 |
songwenping | kolla-ansible is running cyborg with 'cyborg-api' process. | 12:21 |
xinranwang | shall we keep the url same either with kolla or devstack? | 12:22 |
songwenping | devstack use wsgi | 12:22 |
songwenping | no need | 12:22 |
songwenping | other components like nova and cinder is same with cyborg | 12:22 |
brinzhang | http://10.180.210.90:6666/accelerator/v2/devices/ | 12:23 |
brinzhang | # openstack --debug accelerator device list | 12:23 |
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 |
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 |
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 |
brinzhang | this is returned in kolla env | 12:23 |
songwenping | the links of devices returned is wrong | 12:25 |
xinranwang | ok, got it | 12:26 |
xinranwang | so we can keep it as it is | 12:26 |
xinranwang | songwenping could you please help to reply to eric | 12:26 |
songwenping | ok i'll talk with him tommorrow | 12:27 |
xinranwang | cool | 12:27 |
xinranwang | thanks songwenping | 12:27 |
xinranwang | any other patches? | 12:27 |
songwenping | no from me | 12:28 |
brinzhang | please check whether it is right, if it's right, the response body will have a wrong value | 12:28 |
brinzhang | it's also a bug | 12:28 |
brinzhang | https://review.opendev.org/c/openstack/cyborg-tempest-plugin/+/796795 | 12:28 |
brinzhang | this patch, IMO, it's ready to go, we dont need the *tls-proxy* tag | 12:29 |
xinranwang | it is go | 12:29 |
xinranwang | *good | 12:29 |
chenke | tls means what? | 12:29 |
chenke | load balance? | 12:30 |
brinzhang | chenke: please refer https://etherpad.opendev.org/p/support-tls-default | 12:30 |
xinranwang | I think it is for service's communication via tls | 12:30 |
chenke | ok. I will see it. thanks | 12:31 |
brinzhang | nothing for me | 12:32 |
brinzhang | xinranwang | 12:32 |
xinranwang | brinzok | 12:32 |
xinranwang | ok | 12:32 |
chenke | yes xinran. it's like ssl | 12:32 |
chenke | tls like ssl. I agree merge it. | 12:32 |
xinranwang | cool | 12:34 |
xinranwang | #topic PTG | 12:34 |
xinranwang | we have not too much topic for PTG, please continue add topic ther | 12:34 |
opendevreview | Wenping Song proposed openstack/cyborg master: Fix the arq post api doc https://review.opendev.org/c/openstack/cyborg/+/804937 | 12:34 |
xinranwang | cool | 12:35 |
xinranwang | #AoB | 12:35 |
xinranwang | #topic AoB | 12:35 |
xinranwang | anything else you want bring up ? | 12:35 |
brinzhang | I will add some topic to the etherpad while I am free | 12:36 |
opendevreview | Wenping Song proposed openstack/cyborg master: Fix the wrong link https://review.opendev.org/c/openstack/cyborg/+/804938 | 12:36 |
songwenping | noting form myside | 12:36 |
xinranwang | how efficient songwenping is ! | 12:36 |
chenke | no for me | 12:36 |
brinzhang | nothing from me | 12:36 |
songwenping | xinranwang: just rebase master, fast work. | 12:37 |
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 |
xinranwang | ok, let's wrap up the meeting | 12:38 |
xinranwang | thanks guys | 12:39 |
xinranwang | #endmeeting | 12:39 |
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 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-09-01-12.02.html | 12:39 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-09-01-12.02.txt | 12:39 |
opendevmeet | Log: https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-09-01-12.02.log.html | 12:39 |
xinranwang | have a good night | 12:39 |
brinzhang | bye | 12:39 |
opendevreview | Merged openstack/cyborg master: Add description comment for device profile create https://review.opendev.org/c/openstack/cyborg/+/804052 | 12:49 |
opendevreview | Merged openstack/cyborg master: Add the microversion history docs https://review.opendev.org/c/openstack/cyborg/+/804559 | 12:49 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!