| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2020-05-14 | |||
| 03:04:39 | openstack | Meeting started Thu May 14 03:04:38 2020 UTC and is due to finish in 60 minutes. The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 03:04:40 | openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 03:04:42 | openstack | The meeting name has been set to 'openstack_cyborg' | |
| 03:05:24 | Sundar | We haven't seen any RC patches for Cyborg yet, right? | |
| 03:05:24 | songwenping_ | hi | |
| 03:05:53 | Sundar | By now, I would have expected a release patch | |
| 03:06:21 | brinzhang_ | yes, we need a release patch | |
| 03:06:52 | brinzhang_ | and recently we have a lot patches need to be deal, https://review.opendev.org/#/q/project:openstack/cyborg+status:open | |
| 03:09:40 | Sundar | Hmm, Ussuri is already released: http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014847.html | |
| 03:10:37 | brinzhang_ | Sundar: I think we ignored this, I will check for cyborg and cyborgclient | |
| 03:11:25 | Sundar | The release patch was this: https://review.opendev.org/#/c/721400/ | |
| 03:11:32 | Sundar | Yumeng signed off. | |
| 03:12:32 | Sundar | This was the last commit: https://opendev.org/openstack/cyborg/commit/973716aa23d033df3a23aa2b34ce533f3f320c8f | |
| 03:13:24 | Sundar | Only 5 or so patches after that, so we didn't miss much | |
| 03:14:09 | Sundar | Ok, so we are already in Victoria cycle | |
| 03:14:41 | Sundar | I had submitted the Cyborg highlights for Ussuri back in early April. So, we are good there. | |
| 03:15:14 | brinzhang_ | Sundar: OK, I will push a commit for https://opendev.org/openstack/cyborg/commit/973716aa23d033df3a23aa2b34ce533f3f320c8f | |
| 03:15:16 | Sundar | Ussuri highlights puts Cyborg up there: https://releases.openstack.org/ussuri/highlights.html | |
| 03:16:20 | brinzhang_ | There are three patch belongs to Ussuri, so I would like to merge that to U https://review.opendev.org/#/c/723517/ | |
| 03:17:50 | Sundar | brinzhang_: It is already committed, right? | |
| 03:18:38 | brinzhang_ | Sundar: yes, these patch already committed by Sean McGinnis | |
| 03:18:40 | Sundar | I didn't fllow why the revert patches were submitted. Does anybody know the context? | |
| 03:18:46 | Sundar | *follow | |
| 03:19:05 | brinzhang_ | I pushed that in recently IRC meeting | |
| 03:19:42 | brinzhang_ | You said you can talk to Sean McGinnis by irc, maybe you forget Sundar | |
| 03:20:16 | Sundar | I thought Yumeng said she'd follow up in that discussion? | |
| 03:20:46 | brinzhang_ | I remember that another, said to gibi in nova channel | |
| 03:20:58 | Sundar | Anyway, if it is still unclear, I should ask Sean. | |
| 03:21:37 | brinzhang_ | Sundar, ok, after you talked him, then you can +A or -2 for this change, thanks | |
| 03:22:10 | Sundar | I put a reminder for myself. Will follow up. | |
| 03:22:28 | brinzhang_ | Sundar: thanks | |
| 03:22:40 | Sundar | NP. Anything specific for today? | |
| 03:23:43 | brinzhang_ | I think there more open status patches need to consider, whether in Cyborg, cyborgclient or cyborg-manila-plugin, such as bump hacking version patch | |
| 03:24:43 | Sundar | Could you give a pointer to this cyborg-manila-plugin? | |
| 03:25:38 | brinzhang_ | sorry.. cyborg-tempest-plugin ^^ | |
| 03:25:55 | brinzhang_ | spelling error :( | |
| 03:26:00 | Sundar | Oh ok, sure, np :) | |
| 03:26:11 | Sundar | I thought I missed something | |
| 03:27:15 | Sundar | Yup, there are quite a few patches. If you LMK the top patches to look at, I'll try to catch up after this meeting. | |
| 03:27:45 | Sundar | I think you mentioned the hacking patch: https://review.opendev.org/727483 | |
| 03:28:28 | brinzhang_ | right | |
| 03:28:48 | Sundar | Done. What else? | |
| 03:29:09 | Sundar | Looking at https://review.opendev.org/#/c/727603/ | |
| 03:29:23 | songwenping_ | Sundar: So far, do we only support launch one instance with accelerators? | |
| 03:30:02 | brinzhang_ | Sundar: I think https://review.opendev.org/#/c/727482/ double with https://review.opendev.org/#/c/727483 | |
| 03:30:10 | Sundar | No, we cna launch up to 10 instances together. Only the fake driver supports that so far. | |
| 03:31:07 | songwenping_ | If we launch more than one instances with accelerators, there may be problems in filter scheduler. pls see my patch:https://review.opendev.org/#/c/722651/ | |
| 03:32:55 | brinzhang_ | Sundar: yes https://review.opendev.org/#/c/727603/ reported by zuul check, thanks for +2 | |
| 03:33:10 | Sundar | songwenping_: I have launched 10 instances with fake driver in one host, using my Nova patches. This may be a concurrency issue. | |
| 03:36:48 | openstackgerrit | Merged openstack/cyborg master: Fix hacking min version to 3.0.1 https://review.opendev.org/727483 | |
| 03:39:13 | songwenping_ | Sundar: In my environment, it does not look like a concurrency issue. Are your rps in db is same as mine as description at:https://etherpad.opendev.org/p/filter_scheduler_issue_with_accelerators | |
| 03:39:50 | brinzhang_ | Sundar: https://opendev.org/openstack/cyborg/commit/973716aa23d033df3a23aa2b34ce533f3f320c8f this should 4.0.0.0rc2, right? | |
| 03:41:01 | Sundar | songwenping_: With the fake driver, it is one RP with 16 accelerator resources. In your case, you have two RPs with one resource each. May be that accounts for the difference. | |
| 03:42:01 | Sundar | brinzhang_: Does it matter? It got merged anyway, right? | |
| 03:42:47 | brinzhang_ | Sundar, yes, I will keeprc1 | |
| 03:43:50 | Sundar | songwenping_: Thanks for https://review.opendev.org/723865 . I gave a comment. | |
| 03:43:52 | songwenping_ | Sundar: ths. i will check the fake driver. | |
| 03:45:15 | brinzhang_ | Sundar: the release change, please check https://review.opendev.org/#/c/727916/ | |
| 03:47:20 | songwenping_ | Sundar: Thanks for your comment, i will add the 423 case. | |
| 03:48:55 | brinzhang_ | Sundar: does need I add this to stable/ussuri? now in master branch | |
| 03:50:05 | Sundar | brinzhang_: It is already in stable/ussuri, I think: https://opendev.org/openstack/cyborg/src/branch/stable/ussuri/.zuul.yaml | |
| 03:50:54 | brinzhang_ | Sundar: I know, I mean, the release patach https://review.opendev.org/#/c/727916/ | |
| 03:51:44 | brinzhang_ | Now, I submit in master branch, I am not sure whether need move to stable/ussuri in openstack/releases project | |
| 03:51:52 | Sundar | I don't understand why we need this patch. It was already merged into Ussuri. | |
| 03:52:16 | Sundar | That is, tox-bandit is a voting job in Ussuri | |
| 03:53:05 | brinzhang_ | Sundar: ok, I think I was wrong | |
| 03:53:28 | brinzhang_ | I will abandon this patch, Yumeng updated in https://review.opendev.org/#/c/721400/ | |
| 03:53:59 | Sundar | Yes | |
| 03:54:05 | brinzhang_ | I have nothing to talk | |
| 03:54:17 | Sundar | Anything else, folks? | |
| 03:54:50 | songwenping_ | Nothing to talk. | |
| 03:56:07 | Sundar | Thanks again for a successful Ussuri release. It has been my pleasure to server as PTL. From the next meeting, I hope Yumeng will drive the meeting. | |
| 03:56:22 | Sundar | I think it is lunch time for most of you. Enjoy your lunch, and have a good day! | |
| 03:56:38 | Sundar | #endmeeting | |
| 03:56:40 | openstack | Meeting ended Thu May 14 03:56:38 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 03:56:41 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-05-14-03.04.html | |
| 03:56:42 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-05-14-03.04.txt | |
| 03:56:43 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-05-14-03.04.log.html | |
| 04:09:28 | openstackgerrit | Merged openstack/cyborg master: Remove translation sections from setup.cfg https://review.opendev.org/723211 | |
| 07:04:04 | openstackgerrit | Wenping Song proposed openstack/cyborg master: add ut for get arqs list https://review.opendev.org/723865 | |
| 10:20:28 | openstackgerrit | Xinran WANG proposed openstack/cyborg-specs master: Add Intel QAT driver specification. https://review.opendev.org/728014 | |
| 10:52:57 | openstackgerrit | sean mooney proposed openstack/cyborg master: Optimise tox file https://review.opendev.org/708705 | |
| 11:17:20 | openstackgerrit | Wenping Song proposed openstack/cyborg master: add ut for get arqs list https://review.opendev.org/723865 | |
| 11:39:08 | openstackgerrit | sean mooney proposed openstack/cyborg master: Optimise tox file https://review.opendev.org/708705 | |
| 12:24:16 | openstackgerrit | melissaml proposed openstack/python-cyborgclient master: Remove translation sections from setup.cfg https://review.opendev.org/728063 | |
| #openstack-cyborg - 2020-05-15 | |||
| 01:28:07 | gmann | brinzhang: replied on this https://review.opendev.org/#/c/727482/1 | |
| 01:28:14 | gmann | we need that | |
| 01:31:56 | brinzhang_ | gmann: yeah, got it | |
| 01:31:58 | brinzhang_ | gmann: do you havie time to check a patch for functional test | |
| 01:32:02 | brinzhang_ | https://review.opendev.org/#/c/702863/ | |
| 01:32:10 | brinzhang_ | gmann:I cannot find which cased that issue, I left in comment | |
| 01:32:32 | brinzhang_ | gmann: sorry for disconnect | |
| 01:33:52 | gmann | functional test ? | |
| 01:33:57 | brinzhang_ | yeah | |
| 01:34:29 | brinzhang_ | mainly for configure, not real functional test | |
| 01:35:40 | gmann | you mean this one/ - https://review.opendev.org/#/c/702863/ | |
| 01:37:57 | openstackgerrit | Merged openstack/cyborg-tempest-plugin master: Fix hacking min version to 3.0.1 https://review.opendev.org/727482 | |
| 02:07:08 | brinzhang_ | gmann: yes, if you have free time, pls help me have a check ^ | |
| 02:07:57 | gmann | brinzhang_: sure, let this hacking this fixed and sometime next week i will check. | |
| 02:08:08 | gmann | hacking things | |