Earlier  
Posted Nick Remark
#openstack-cyborg - 2020-02-20
03:58:14 brinzhang_ thanks all
03:58:43 shaohe_feng if no, please help to review the serial patches https://review.opendev.org/#/c/708655/7
03:58:59 shaohe_feng I have give a enough information in the patches
03:59:14 brinzhang_ Yumeng: yes, it time lunch :)
03:59:17 shaohe_feng about the history and the root cause
03:59:32 shaohe_feng OK, let's end the meeting.
03:59:36 shaohe_feng have good lunch
03:59:42 chenke Ok.
03:59:49 chenke bye~
04:00:05 shaohe_feng Thanks All.
04:00:18 shaohe_feng #endmeeting
04:00:20 openstack Meeting ended Thu Feb 20 04:00:18 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
04:00:21 openstack Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-02-20-03.15.html
04:00:23 openstack Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-02-20-03.15.txt
04:00:24 openstack Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-02-20-03.15.log.html
04:04:13 openstackgerrit Brin Zhang proposed openstack/cyborg master: Fixed define incorrectly of the exc msg https://review.opendev.org/708796
#openstack-cyborg - 2020-02-22
10:22:38 openstackgerrit YumengBao proposed openstack/cyborg master: Add cpid_type to db schema https://review.opendev.org/709286
#openstack-cyborg - 2020-02-24
06:47:19 openstackgerrit YumengBao proposed openstack/cyborg master: Update cpid_type allowed values in db schema https://review.opendev.org/709286
09:43:26 openstackgerrit YumengBao proposed openstack/cyborg master: Fix cpid_type of gpu device with improper value https://review.opendev.org/709433
09:43:26 openstackgerrit YumengBao proposed openstack/cyborg master: Update cpid_type allowed values in db schema https://review.opendev.org/709286
15:50:17 openstackgerrit Merged openstack/cyborg master: Avoid creating a root provider when parent is not found https://review.opendev.org/708726
17:06:09 openstackgerrit Merged openstack/cyborg master: Fix exceptions defined with improper _msg_fmt https://review.opendev.org/708728
22:13:55 openstackgerrit sean mooney proposed openstack/cyborg master: move setting to devstack/settings https://review.opendev.org/708770
22:13:55 openstackgerrit sean mooney proposed openstack/cyborg master: add support for multi node deployments to fake driver https://review.opendev.org/708674
22:13:56 openstackgerrit sean mooney proposed openstack/cyborg master: simplify devstack plugin https://review.opendev.org/708773
22:13:56 openstackgerrit sean mooney proposed openstack/cyborg master: support standard devstack plugin cloning https://review.opendev.org/708771
22:13:57 openstackgerrit sean mooney proposed openstack/cyborg master: move functions from devstack/plugin.sh to devstack/lib/cyborg https://review.opendev.org/709620
22:59:46 openstackgerrit sean mooney proposed openstack/cyborg master: add support for installing cyborg client https://review.opendev.org/709629
23:19:00 openstackgerrit sean mooney proposed openstack/cyborg master: move setting to devstack/settings https://review.opendev.org/708770
23:19:00 openstackgerrit sean mooney proposed openstack/cyborg master: add support for multi node deployments to fake driver https://review.opendev.org/708674
23:19:01 openstackgerrit sean mooney proposed openstack/cyborg master: move functions from devstack/plugin.sh to devstack/lib/cyborg https://review.opendev.org/709620
23:19:01 openstackgerrit sean mooney proposed openstack/cyborg master: support standard devstack plugin cloning https://review.opendev.org/708771
23:19:02 openstackgerrit sean mooney proposed openstack/cyborg master: add support for installing cyborg client https://review.opendev.org/709629
23:19:02 openstackgerrit sean mooney proposed openstack/cyborg master: simplify devstack plugin https://review.opendev.org/708773
#openstack-cyborg - 2020-02-25
00:28:22 openstackgerrit sean mooney proposed openstack/cyborg master: WIP add cyborg multinode job https://review.opendev.org/709641
00:45:25 openstackgerrit sean mooney proposed openstack/cyborg master: move functions from devstack/plugin.sh to devstack/lib/cyborg https://review.opendev.org/709620
00:45:26 openstackgerrit sean mooney proposed openstack/cyborg master: WIP add cyborg multinode job https://review.opendev.org/709641
00:45:26 openstackgerrit sean mooney proposed openstack/cyborg master: add support for installing cyborg client https://review.opendev.org/709629
00:45:26 openstackgerrit sean mooney proposed openstack/cyborg master: simplify devstack plugin https://review.opendev.org/708773
14:59:47 openstackgerrit Dan Smith proposed openstack/cyborg master: Fix setting cyborg agent hostname https://review.opendev.org/709749
19:52:50 openstackgerrit sean mooney proposed openstack/cyborg master: WIP add cyborg multinode job https://review.opendev.org/709641
20:37:39 openstackgerrit sean mooney proposed openstack/cyborg master: WIP add cyborg multinode job https://review.opendev.org/709641
23:36:34 openstackgerrit sean mooney proposed openstack/cyborg master: WIP add cyborg multinode job https://review.opendev.org/709641
#openstack-cyborg - 2020-02-26
08:13:53 openstackgerrit Brin Zhang proposed openstack/cyborg master: Delete sanbox directory https://review.opendev.org/709974
09:30:56 openstackgerrit Brin Zhang proposed openstack/cyborg master: WIP: add api_samples_test_base to functional https://review.opendev.org/709992
#openstack-cyborg - 2020-02-27
03:00:05 Sundar Hello all
03:00:17 Sundar #startmeeting openstack-cyborg
03:00:18 chenke Hello sudnar.
03:00:19 openstack Meeting started Thu Feb 27 03:00:17 2020 UTC and is due to finish in 60 minutes. The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot.
03:00:20 openstack Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
03:00:22 openstack The meeting name has been set to 'openstack_cyborg'
03:00:24 chenke #info chenke
03:00:35 Sundar #info Sundar
03:00:38 Yumeng #info Yumeng
03:00:43 brinzhang #info brinzhang
03:00:45 xinranwang #info xinranwang
03:01:32 Sundar Do we have any major topics for today? I'd like to provide a status update on Nova integ after other discussion.
03:02:07 xinranwang I have one topic about microversion
03:02:49 Sundar Ok, xinranwang. Anything else to discuss, before we get int that?
03:02:50 xinranwang Now I implemented a decorator api_version to check the microversion.
03:03:34 Yumeng I only have gpu_driver_improve patches need your reivew: https://review.opendev.org/#/q/status:open+project:openstack/cyborg+branch:master+topic:gpu-driver-improve. nothing else.
03:03:35 xinranwang please see https://review.opendev.org/#/c/696860/
03:04:17 brinzhang I want to talk about the functional tests, I found there are so many work should to do, anyone have some sugestion? or any simple project we can reference?
03:05:14 Sundar xinranwang: do you have a question or something to discuss about that patch?
03:06:27 xinranwang Yes
03:06:34 xinranwang Now I implemented a decorator api_version to check the microversion.
03:06:38 brinzhang Sundar, xinranwang, I think mainly is https://review.opendev.org/#/c/696860/3/cyborg/api/controllers/base.py@85
03:08:13 xinranwang Do you think we should have a schema check in this patch, IMO, I think schema check is based on microversion support, we can do it later.
03:08:36 xinranwang thanks brinzhang to paste the link
03:09:42 shaohe_feng it can be another patch.
03:09:53 Sundar "This decorator MUST appear first (the outermost
03:10:14 shaohe_feng schema check should not only for microversion
03:10:24 shaohe_feng it should for all APIs.
03:11:52 brinzhang I don't think we should rewrite the same API interface every time while we need to add a new microversion, which will cause a lot of code redundancy.
03:14:04 xinranwang brinzhang: Yes, I understand your concern. For now, we have only one microversion, (the v2.1 is my PoC code, will be merge to v2.0), so there is only one API function.
03:14:32 brinzhang xinranwang: Yeah, understand
03:15:06 s_shogo #info s_shogo
03:15:20 xinranwang What I suggest to do, is to support the microverison firstly, and them we can add schema check in another patch.
03:17:21 shaohe_feng I'd like to talk one things, one patch for one issues. except the nits fix.
03:18:13 shaohe_feng https://review.opendev.org/#/c/693784/1/devstack/lib/cyborg@159
03:18:26 shaohe_feng ^ this is a example
03:18:32 Li_Liu #info Li_Liu
03:19:22 Yumeng agree with xinranwang. we can have a bp of API shema check later or in next release.
03:19:23 shaohe_feng we have already know there are some issues in devstack config
03:20:12 shaohe_feng about half years ago.
03:20:36 Sundar xinranwang: I think we all agree with Brin's concern, and that we all agree there should be a schema check. You just want it in another patch, right?
03:20:39 shaohe_feng but why we can not continue on it?
03:21:01 shaohe_feng we always want to one patch to fix many issues.
03:21:10 shaohe_feng fix one by one
03:21:19 shaohe_feng this will be a fast way.
03:21:36 shaohe_feng and you can see, https://review.opendev.org/#/c/709749/1/devstack/lib/cyborg
03:21:54 xinranwang Sundar: yes, exactly. As Yumeng said, we should have another bp and patch to implement it.
03:22:20 shaohe_feng It is also devstack fix. we have config it right before.
03:22:37 Sundar Ok, I agree, xinranwang, Yumeng
03:22:41 shaohe_feng Mix many issues together.
03:22:54 shaohe_feng really a bad idea.
03:23:30 xinranwang It seems brinzhang was dropped off, I will talk to him later. Thanks guys.
03:23:40 Sundar s_shogo: No sure what you are trying to say. The link you gave is for an old patch set. Are you saying we should enforce one patch for one issue?
03:24:57 s_shogo Sundar: IMHO, the mention intended to shaohe_feng ?
03:25:27 shaohe_feng yes, one by one.

Earlier   Later