Earlier  
Posted Nick Remark
#openstack-cyborg - 2021-07-22
03:10:49 xinranwang ok, let's wrap up the meeing
03:11:06 songwenping bye
03:11:08 xinranwang take care everyone , stay safe
03:11:16 xinranwang #endmeeting
03:11:16 opendevmeet Meeting ended Thu Jul 22 03:11:16 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
03:11:16 opendevmeet Minutes: https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-07-22-02.38.html
03:11:16 opendevmeet Minutes (text): https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-07-22-02.38.txt
03:11:16 opendevmeet Log: https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-07-22-02.38.log.html
#openstack-cyborg - 2021-07-23
02:06:05 opendevreview Wenping Song proposed openstack/cyborg master: Delete trait from placement https://review.opendev.org/c/openstack/cyborg/+/790937
08:12:28 opendevreview Wenping Song proposed openstack/cyborg master: Delete trait from placement https://review.opendev.org/c/openstack/cyborg/+/790937
11:07:44 opendevreview Eric Xie proposed openstack/python-cyborgclient master: Hide the traceback when deleting a non-existent ARQ https://review.opendev.org/c/openstack/python-cyborgclient/+/802004
#openstack-cyborg - 2021-07-24
07:43:29 opendevreview Eric Xie proposed openstack/python-cyborgclient master: Add checks for showing dp with name https://review.opendev.org/c/openstack/python-cyborgclient/+/802140
#openstack-cyborg - 2021-07-26
04:48:24 opendevreview Eric Xie proposed openstack/python-cyborgclient master: Add checks for showing dp with name https://review.opendev.org/c/openstack/python-cyborgclient/+/802140
04:55:15 opendevreview Eric Xie proposed openstack/python-cyborgclient master: Hide the traceback when deleting a non-existent ARQ https://review.opendev.org/c/openstack/python-cyborgclient/+/802004
#openstack-cyborg - 2021-07-29
02:32:51 xinranwang hi all
02:33:02 brinzhang_ hi
02:33:10 songwenping_ hi all
02:33:17 brinzhang_ at 11:00, I have a meeting
02:33:43 xinranwang ok
02:33:48 brinzhang_ in each Thursday in fruture it will be happened :(
02:33:56 xinranwang let's start the meeting quickly
02:34:04 xinranwang #startmeeting
02:34:04 opendevmeet xinranwang: Error: A meeting name is required, e.g., '#startmeeting Marketing Committee'
02:34:11 xinranwang #startmeeting openstack-cyborg
02:34:11 opendevmeet Meeting started Thu Jul 29 02:34:11 2021 UTC and is due to finish in 60 minutes. The chair is xinranwang. Information about MeetBot at http://wiki.debian.org/MeetBot.
02:34:11 opendevmeet Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
02:34:11 opendevmeet The meeting name has been set to 'openstack_cyborg'
02:34:16 xinranwang #topic roll call
02:34:41 brinzhang_ #info brinzhang
02:34:46 xinranwang #info xinranwang
02:35:45 xinranwang #topic pending patches
02:35:46 songwenping_ #info songwenping_
02:36:26 xinranwang let's go through the pending patches
02:36:49 xinranwang 1st: https://review.opendev.org/c/openstack/cyborg/+/790937
02:37:38 xinranwang I think this one is good for me. each time we delete trart for on rp, we should check if we need to delete the trait as well.
02:39:14 xinranwang brinzhang_ what do you think about this
02:39:32 xinranwang 2nd: https://review.opendev.org/c/openstack/cyborg/+/797403
02:41:45 brinzhang_ for delete the trait I think it's ok, I will review it at afternoon
02:41:59 xinranwang brinzhang_: ok, thanks
02:42:49 brinzhang_ the sec I have talked with wenping, I suggest to combine the duplicate exception
02:43:09 xinranwang what do you mean by combine
02:43:35 songwenping_ combine name exist and uuid exist exception
02:44:00 brinzhang_ yes
02:44:04 songwenping_ to resource existed exception
02:44:35 songwenping_ need we distingush them
02:45:03 xinranwang it should be 2 different exceptions
02:45:28 brinzhang_ if columns just only exist with name, using string will be better,but it'a an array now, just using arr[0] will be have a risk
02:46:06 brinzhang_ we can using log + rase exception to warning the user/developer
02:46:17 xinranwang i think the array is not for one colume, it records all duplicated columes' name
02:46:27 xinranwang so it is a array
02:47:32 brinzhang_ if so, why are you just call the first key arr[0]?
02:48:34 xinranwang we didn't use arr[0] now, the if condition is: if 'name' in ['name']
02:49:18 xinranwang calling arr[0] is a workaround because the value of the list is not 'name', but "xxxxxxname" instead
02:49:24 songwenping_ just for verify the tempest, we cannot use arr[0] in fact.
02:49:43 xinranwang songwenping_: +1
02:50:45 songwenping_ we should find why the e.columns is different, some env is ok.
02:51:12 brinzhang_ but now changed with if 'name' in e.columns[0] it will be always using the fist
02:51:18 brinzhang_ is right?
02:52:10 songwenping_ we donnot plan to merge arr[0]
02:52:32 xinranwang it just a work around. I think we should find out how this array formed
02:52:41 songwenping_ yes
02:53:06 songwenping_ why the exception columns is ['name']
02:53:17 songwenping_ s/is/isnot
02:54:27 xinranwang do you think we can try to use 10.0.0,
02:55:04 songwenping_ perhaps has no relate with the oslo.db version
02:56:02 songwenping_ need to look into it.
02:56:25 xinranwang ok, let's discuss offline
02:56:32 songwenping_ yes let's continue
02:56:35 xinranwang #topic AoB
02:56:52 xinranwang anything else you wanna bring up?
02:57:30 songwenping_ eric xie commit one patch
02:58:13 xinranwang you mean the url related one?
02:58:22 songwenping_ i am finding
02:59:05 songwenping_ https://review.opendev.org/c/openstack/python-cyborgclient/+/802140
02:59:24 songwenping_ we donnot support device profile show with name
03:00:23 songwenping_ should we support to do this?
03:00:49 xinranwang oh, yes
03:00:54 xinranwang good point.
03:00:56 songwenping_ the patch is not reasonable
03:01:10 xinranwang I think we should support using name
03:01:53 songwenping_ i talked with him to discuss in the ptg, maybe we need a spec to do this.
03:02:59 xinranwang cool, thanks wenping
03:03:24 songwenping_ no problem
03:03:47 xinranwang anything else?
03:03:55 songwenping_ nothing
03:04:11 xinranwang ok, if nothing else, let's wrap up the meeting
03:04:41 songwenping_ ok
03:04:43 songwenping_ bye
03:04:44 xinranwang #endmeeting
03:04:44 opendevmeet Meeting ended Thu Jul 29 03:04:44 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
03:04:44 opendevmeet Minutes: https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-07-29-02.34.html
03:04:44 opendevmeet Minutes (text): https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-07-29-02.34.txt
03:04:44 opendevmeet Log: https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-07-29-02.34.log.html
03:04:51 xinranwang bye, have a nice day
08:45:01 opendevreview Wenping Song proposed openstack/cyborg master: update oslo.db version to 10.0.0 https://review.opendev.org/c/openstack/cyborg/+/802839
#openstack-cyborg - 2021-07-30
09:58:31 opendevreview Merged openstack/cyborg master: Delete trait from placement https://review.opendev.org/c/openstack/cyborg/+/790937
#openstack-cyborg - 2021-07-31
08:19:21 opendevreview Eric Xie proposed openstack/cyborg-specs master: Add Xilinx FPGA driver specification https://review.opendev.org/c/openstack/cyborg-specs/+/803103
08:55:24 opendevreview Eric Xie proposed openstack/cyborg-specs master: Add Xilinx FPGA driver specification https://review.opendev.org/c/openstack/cyborg-specs/+/803103
#openstack-cyborg - 2021-08-05
02:10:51 chenke hi
02:27:49 songwenping_ hi chenke

Earlier   Later