| *** tetsuro has joined #openstack-cyborg | 00:08 | |
| openstackgerrit | Merged openstack/cyborg-specs master: Change the doc8 check mode https://review.opendev.org/748798 | 00:57 |
|---|---|---|
| *** tetsuro has quit IRC | 00:58 | |
| *** tetsuro has joined #openstack-cyborg | 00:59 | |
| *** tetsuro has quit IRC | 01:03 | |
| openstackgerrit | Wenping Song proposed openstack/cyborg master: Signal to conductor service to apply patch accelerator requests. https://review.opendev.org/751957 | 01:30 |
| *** tetsuro has joined #openstack-cyborg | 01:32 | |
| *** tetsuro has quit IRC | 01:37 | |
| *** brinzhang_ has joined #openstack-cyborg | 02:10 | |
| *** brinzhang0 has quit IRC | 02:13 | |
| openstackgerrit | Shogo Saito proposed openstack/cyborg master: Add releasenote for programming method https://review.opendev.org/752358 | 02:26 |
| *** swp20 has joined #openstack-cyborg | 02:53 | |
| *** Yumeng has joined #openstack-cyborg | 02:55 | |
| openstackgerrit | Brin Zhang proposed openstack/cyborg master: WIP: Add microversion instructions https://review.opendev.org/743722 | 03:02 |
| *** chenke has joined #openstack-cyborg | 03:02 | |
| chenke | Hi yumeng. | 03:02 |
| *** s_shogo has joined #openstack-cyborg | 03:02 | |
| swp20 | Hi all | 03:03 |
| brinzhang_ | \o | 03:04 |
| Yumeng | hi all | 03:04 |
| s_shogo | Hi all | 03:05 |
| Yumeng | hi s_shogo. let's wait one more second, I will ask if xinran comes. | 03:06 |
| s_shogo | OK | 03:06 |
| *** xinranwang__ has joined #openstack-cyborg | 03:07 | |
| xinranwang__ | Hi all | 03:07 |
| Yumeng | hi xinran | 03:07 |
| Yumeng | ok. we've got enough people. let's get started | 03:08 |
| Yumeng | #startmeeting openstack-cyborg | 03:08 |
| openstack | Meeting started Thu Sep 17 03:08:51 2020 UTC and is due to finish in 60 minutes. The chair is Yumeng. Information about MeetBot at http://wiki.debian.org/MeetBot. | 03:08 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 03:08 |
| *** openstack changes topic to " (Meeting topic: openstack-cyborg)" | 03:08 | |
| openstack | The meeting name has been set to 'openstack_cyborg' | 03:08 |
| Yumeng | #topic Roll call | 03:09 |
| *** openstack changes topic to "Roll call (Meeting topic: openstack-cyborg)" | 03:09 | |
| Yumeng | #info Yumeng | 03:09 |
| swp20 | #info swp20 | 03:09 |
| xinranwang__ | #info xinranwang__ | 03:09 |
| s_shogo | #info s_shogo | 03:09 |
| brinzhang_ | #info brinzhang_ | 03:09 |
| Yumeng | cool | 03:10 |
| Yumeng | #topic Agenda | 03:10 |
| *** openstack changes topic to "Agenda (Meeting topic: openstack-cyborg)" | 03:10 | |
| chenke | #info chenke | 03:10 |
| Yumeng | we are approaching final rc. Code freeze is Oct 05 - Oct 09, but we actually may just have 13 days to go before RC cut because of the Chinese National Holiday(Oct 01 - Oct 08). | 03:11 |
| xinranwang__ | wow, it's approching | 03:12 |
| Yumeng | yep ^ time flies. | 03:12 |
| Yumeng | and we now have one big feature to merge. the program API patch: https://review.opendev.org/#/c/698190/ | 03:13 |
| s_shogo | yes > program API patch | 03:14 |
| s_shogo | I read your comment just now, I'll add the TODO soon. | 03:14 |
| s_shogo | : Yumeng | 03:14 |
| Yumeng | Great! | 03:14 |
| swp20 | s_shogo: https://review.opendev.org/#/c/698190/11/cyborg/agent/rpcapi.py@L49 do we need always to change the version to RPC_API_VERSION? | 03:15 |
| swp20 | i think the 1.0 is the min version. | 03:15 |
| swp20 | we shouldnot change it. | 03:16 |
| s_shogo | That is not mandatory one, IMO. | 03:16 |
| brinzhang_ | agree with swp20, we should change the version in fpga_program() | 03:18 |
| s_shogo | But there seems to be warning may be outputted if there is difference between manager.py's microversion and RPC versions. | 03:18 |
| swp20 | if we change it, other api may be effected. | 03:18 |
| brinzhang_ | but it seems the 1.0 is not exist | 03:18 |
| Yumeng | "manager.py's microversion and RPC versions", will these two have conflicts? | 03:20 |
| *** links has joined #openstack-cyborg | 03:21 | |
| s_shogo | Yumeng that may be in my specific problem, I'll check that. | 03:21 |
| brinzhang_ | s_shogo: keep the rpc version in 1.0 is better than to change it to 1.1, it is backward compatible | 03:22 |
| brinzhang_ | I think | 03:22 |
| Yumeng | Thanks s_shogo. | 03:23 |
| s_shogo | brinzhang_ : as before , you mentioned the 1.1 in the patch, that is old one? | 03:23 |
| brinzhang_ | may I missed this said by swp20 | 03:24 |
| s_shogo | OK, to change that to 1.0 is not difficult task, | 03:25 |
| brinzhang_ | Yumeng, s_shogo: RPC_API_VERSION exist in manager.py and rpcapi.py, in the manager.py do we need it? | 03:26 |
| brinzhang_ | It seems like no need to record that in AgentManager, right? | 03:27 |
| Yumeng | yes I thinks so. | 03:27 |
| Yumeng | just checked another project, pls see https://github.com/openstack/manila/blob/479f5f1938bd212982cd4a2d25464b757ca12e2b/manila/scheduler/manager.py | 03:28 |
| brinzhang_ | A question is in https://review.opendev.org/#/c/698190/11/cyborg/agent/manager.py@46, do we need to init the target for new rpc api version? | 03:28 |
| Yumeng | ops, seems manila.SchedulerManager recorded RPC_API_VERSION but didn't actually use it. | 03:29 |
| brinzhang_ | https://github.com/openstack/manila/blob/479f5f1938bd212982cd4a2d25464b757ca12e2b/manila/scheduler/rpcapi.py#L28-L53 | 03:30 |
| xinranwang__ | Shall we make conductor manager and rpcapi have same verison? | 03:31 |
| brinzhang_ | the change rpc api always keep in latest version init in target in manila schdeuler rpcapi | 03:31 |
| Yumeng | brinzhang_,s_shogo: cinder may have a better example in using RPC_API_VERSION in both rpc.py and manager.py. FIR:https://github.com/openstack/cinder/blob/19caf5651b1272cba46396c53bd5cbb6a312def5/cinder/manager.py , https://github.com/openstack/cinder/blob/3c29c3846ee4e0a43e74f7328642ace218f00d18/cinder/rpc.py#L202 | 03:34 |
| brinzhang_ | from these example, we can sure the Target need the latest RPC version, it can support 1.0~latest | 03:36 |
| brinzhang_ | target = messaging.Target(topic=self.TOPIC, version=self.RPC_API_VERSION) | 03:37 |
| Yumeng | xinranwang__, I think conductor rpcapi_version can be different from that of agent. | 03:37 |
| brinzhang_ | s_shogo: please test in you local for the program API, and changed the target | 03:37 |
| s_shogo | ok, I try that. | 03:38 |
| brinzhang_ | How to understand Yumeng's doubt? | 03:39 |
| swp20 | s_shogo: i change conductor rpcapi target version to latest, and create dp with error: HttpException: 500: Server Error for url: http://10.180.249.199/accelerator/v2/device_profiles, Remote error: UnsupportedVersion Endpoint does not support RPC version 1.1. Attempted method: device_profile_create | 03:41 |
| s_shogo | now, I have encountered the same error. that returns 500. | 03:41 |
| s_shogo | swp20 | 03:41 |
| s_shogo | ( in fpga_program | 03:42 |
| swp20 | we should research this. | 03:42 |
| brinzhang_ | IMO, that need to have a deep search, and make sure what we should do, waht do you think, s_shogo, Yumeng, xinranwang_, swp20? | 03:43 |
| s_shogo | Toward Victoria release, to change the RPC version 1.0 temporary is one way ,IMO. | 03:44 |
| xinranwang__ | s | 03:44 |
| xinranwang__ | s_shogo: yes I agree this | 03:44 |
| s_shogo | (that is not perfect solution , this is temporary .) | 03:44 |
| brinzhang_ | ok | 03:45 |
| xinranwang__ | don't block on this | 03:45 |
| xinranwang__ | we can backport this to V when we figure it out | 03:46 |
| s_shogo | that's good, I agree the backport: xinranwang_ | 03:46 |
| brinzhang_ | We can let this go, but need to make sure it's ok for this release, at least, we dont introduce an error | 03:47 |
| swp20 | ok. aggree | 03:47 |
| Yumeng | yes, agree. don't block on this | 03:47 |
| Yumeng | ok.. let's move to next topic. | 03:49 |
| s_shogo | ok, I will change that, and commit soon. | 03:49 |
| Yumeng | thanks s_shogo | 03:49 |
| s_shogo | thanks all | 03:49 |
| brinzhang_ | ok | 03:49 |
| brinzhang_ | *me will left 30mins | 03:49 |
| Yumeng | here is just an announcement: What to do next week? review and merge docs, release note, bug fix, and necessary enhancement patches mainly for Victoria delivery. | 03:50 |
| Yumeng | and when I was check doc page this morning. I wanna discuss with you guys what is not good with the current doc page? what we can do in the next few days? https://docs.openstack.org/cyborg/latest/ | 03:51 |
| Yumeng | I was writing the doc for the cyborg new policy as a usage guide. and will submit a very basic framework of that. | 03:53 |
| Yumeng | do we need to update the driver's usage guide? I think the testing report is already good enough, but how can users know this can be also a usage guide? | 03:55 |
| Yumeng | Another page I can think of is the CLI page. I will update this. https://docs.openstack.org/cyborg/latest/cli/index.html. | 03:58 |
| Yumeng | That's all I can think of for now. | 03:59 |
| s_shogo | I think the lack of cli document is problem, too. | 04:00 |
| s_shogo | I'm trying to add the cyborg plugin's description to the osc document , now. https://review.opendev.org/#/c/751986/ | 04:00 |
| Yumeng | Just mention this, doesn't have to point out all of them at one. if you guys think of other pages, just ping us at wechat. | 04:01 |
| Yumeng | s_shogo: nice! do you need helpt in asking review? | 04:01 |
| s_shogo | Yumeng now,I'm trying fix the Zuul errors. after that, I will ask your review :) | 04:02 |
| Yumeng | sure. | 04:02 |
| *** songwenping_ has joined #openstack-cyborg | 04:03 | |
| Yumeng | cool. Thank you, guys! We've done a lot in Victoria! | 04:03 |
| Yumeng | Do you guys want to bring up anything else? | 04:04 |
| Yumeng | if nothing else, let's wrap up this meeting and see you all again next week! | 04:05 |
| Yumeng | bye | 04:05 |
| Yumeng | #endmeeting | 04:05 |
| *** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)" | 04:05 | |
| openstack | Meeting ended Thu Sep 17 04:05:28 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 04:05 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-09-17-03.08.html | 04:05 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-09-17-03.08.txt | 04:05 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-09-17-03.08.log.html | 04:05 |
| *** swp20 has quit IRC | 04:06 | |
| *** songwenping__ has joined #openstack-cyborg | 04:09 | |
| *** songwenping_ has quit IRC | 04:12 | |
| *** songwenping__ has quit IRC | 04:25 | |
| *** songwenping__ has joined #openstack-cyborg | 04:26 | |
| *** songwenping__ has quit IRC | 05:03 | |
| openstackgerrit | Shogo Saito proposed openstack/cyborg master: Programming support (v2 Deployable API) https://review.opendev.org/698190 | 05:09 |
| *** tetsuro has joined #openstack-cyborg | 05:14 | |
| *** songwenping__ has joined #openstack-cyborg | 05:38 | |
| *** chenke has quit IRC | 05:52 | |
| *** s_shogo has quit IRC | 06:10 | |
| openstackgerrit | YumengBao proposed openstack/cyborg master: Add trait check for POST Device Profile https://review.opendev.org/752257 | 06:41 |
| openstackgerrit | YumengBao proposed openstack/cyborg master: Add rc check for POST Device Profile https://review.opendev.org/752269 | 06:42 |
| openstackgerrit | YumengBao proposed openstack/cyborg master: Add rc check for POST Device Profile https://review.opendev.org/752269 | 06:44 |
| *** jraju__ has joined #openstack-cyborg | 06:59 | |
| *** links has quit IRC | 07:00 | |
| *** xinranwang__ has quit IRC | 07:36 | |
| openstackgerrit | YumengBao proposed openstack/cyborg master: [goal] Fixing lower constraints for Ubuntu Focal https://review.opendev.org/752391 | 08:34 |
| openstackgerrit | YumengBao proposed openstack/python-cyborgclient master: [goal] Fixing lower constraints for Ubuntu Focal https://review.opendev.org/752399 | 08:52 |
| openstackgerrit | YumengBao proposed openstack/cyborg-tempest-plugin master: [goal] Fixing lower constraints for Ubuntu Focal https://review.opendev.org/752401 | 09:00 |
| openstackgerrit | Merged openstack/python-cyborgclient master: Update master for stable/victoria https://review.opendev.org/750922 | 09:11 |
| openstackgerrit | Merged openstack/python-cyborgclient master: Add Python3 wallaby unit tests https://review.opendev.org/750923 | 09:23 |
| *** tetsuro has quit IRC | 09:25 | |
| *** efried has quit IRC | 11:49 | |
| *** efried has joined #openstack-cyborg | 11:51 | |
| *** efried has quit IRC | 12:01 | |
| *** brinzhang has joined #openstack-cyborg | 12:31 | |
| *** brinzhang_ has quit IRC | 12:36 | |
| *** efried has joined #openstack-cyborg | 13:00 | |
| *** Yumeng has quit IRC | 15:48 | |
| *** jraju__ has quit IRC | 16:28 | |
| *** brinzhang_ has joined #openstack-cyborg | 16:46 | |
| *** brinzhang has quit IRC | 16:50 | |
| *** irclogbot_3 has quit IRC | 17:58 | |
| *** irclogbot_3 has joined #openstack-cyborg | 18:01 | |
| *** openstackgerrit has quit IRC | 19:21 | |
| *** brinzhang0 has joined #openstack-cyborg | 22:14 | |
| *** brinzhang_ has quit IRC | 22:17 | |
| *** openstackgerrit has joined #openstack-cyborg | 22:22 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/cyborg master: Change default policy file from JSON to YAML https://review.opendev.org/752576 | 22:22 |
| gmann | brinzhang0: Yumeng ^^ a pre work for new policy shipment | 22:24 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!