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:56 <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?
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:23 <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 <xinranwang> #endmeeting