| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2021-07-29 | |||
| 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 | opendevmeet | Log: https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-07-29-02.34.log.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 | Minutes: https://meetings.opendev.org/meetings/openstack_cyborg/2021/openstack_cyborg.2021-07-29-02.34.html | |
| 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 | xinranwang | #endmeeting | |
| 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 | |
| 02:27:57 | songwenping_ | long time to see | |
| 02:30:24 | chenke | seems xinran not here | |
| 02:35:56 | xinranwang | hi all | |
| 02:36:00 | xinranwang | sorry for late | |
| 02:36:20 | songwenping_ | hi all | |
| 02:38:09 | xinranwang | chenke has quit | |
| 02:38:19 | xinranwang | let's start first | |
| 02:38:28 | opendevmeet | The meeting name has been set to 'openstack_cyborg' | |
| 02:38:28 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 02:38:28 | opendevmeet | Meeting started Thu Aug 5 02:38:28 2021 UTC and is due to finish in 60 minutes. The chair is xinranwang. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 02:38:28 | xinranwang | #startmeeting openstack-cyborg | |
| 02:38:34 | xinranwang | #roll call | |
| 02:38:39 | xinranwang | #topic roll call | |
| 02:38:47 | xinranwang | #info xinranwang | |
| 02:39:03 | songwenping_ | #info songwenping_ | |
| 02:39:23 | songwenping_ | he offline again | |
| 02:45:00 | chenke | hi all | |
| 02:45:16 | chenke | how are you guys | |
| 02:45:35 | songwenping_ | hi chenke longtime to see | |
| 02:45:53 | chenke | haha | |
| 02:46:00 | chenke | thanks | |
| 02:46:06 | chenke | please go on | |
| 02:46:41 | songwenping_ | xinran is offine? | |
| 02:46:49 | xinranwang | sorry, I am back | |
| 02:46:59 | xinranwang | Hi chenke , no time no see | |
| 02:47:05 | xinranwang | s /no/long | |
| 02:47:25 | xinranwang | today, I have nothing specail want bring up. | |
| 02:47:36 | xinranwang | do you guys need any help? | |
| 02:48:28 | songwenping_ | one question | |
| 02:48:59 | songwenping_ | why the zuul is fail about this patch | |
| 02:49:01 | songwenping_ | https://review.opendev.org/c/openstack/cyborg/+/797403 | |
| 02:49:21 | songwenping_ | as the oslo.db is the latest version | |
| 02:49:53 | songwenping_ | except the oslo.db on devstack env is not the latest | |
| 02:51:55 | songwenping_ | one spec need to review: https://review.opendev.org/c/openstack/cyborg-specs/+/803103 | |
| 02:52:19 | xinranwang | songwenping_: it now passed zuul | |
| 02:52:52 | xinranwang | ok, thanks for reminding | |