Wednesday, 2019-08-14

*** xinranwang has joined #openstack-cyborg02:42
*** wangzhh has joined #openstack-cyborg02:52
openstackgerritXinran WANG proposed openstack/cyborg master: Notify Nova when binding finished  https://review.opendev.org/67452002:53
openstackgerritXinran WANG proposed openstack/cyborg master: Notify Nova when binding finished  https://review.opendev.org/67452002:55
*** Sundar has joined #openstack-cyborg02:57
openstackgerritXinran WANG proposed openstack/cyborg master: Notify Nova when binding finished  https://review.opendev.org/67452002:59
*** Yumeng has joined #openstack-cyborg03:00
SundarHi Yumeng03:01
YumengHi Sundar03:01
Yumenghi all03:01
SundarLet's wait for a couple of minutes.03:01
Yumengok03:01
wangzhhHi all.03:03
SundarHi wangzhh03:03
Sundar#startmeeting openstack-cyborg03:03
openstackMeeting started Wed Aug 14 03:03:18 2019 UTC and is due to finish in 60 minutes.  The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot.03:03
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.03:03
*** openstack changes topic to " (Meeting topic: openstack-cyborg)"03:03
wangzhhHi Sundar.03:03
openstackThe meeting name has been set to 'openstack_cyborg'03:03
Sundar#topic Roll call03:03
*** openstack changes topic to "Roll call (Meeting topic: openstack-cyborg)"03:03
Sundaro/03:03
Yumeng#info Yumeng03:03
wangzhh#info wangzhh03:03
SundarSeems like thin attendance today, at least to start with03:04
Sundar#topic New time for weekly IRC meeting03:04
*** openstack changes topic to "New time for weekly IRC meeting (Meeting topic: openstack-cyborg)"03:04
xinranwang#info xinranwang03:04
Yumenghi xinran03:05
xinranwangHi all :)03:05
SundarWe did a Doodle poll to pick the best time for al: https://doodle.com/poll/6t279f9y6msztz7x03:05
SundarThere were 2 slots with 6 votes each. I pinged Shaohe and Coco individually. Both agreed on Thu 11 am China time.03:06
yikun#info yikun03:07
SundarAny objections to choosing that as the new time?03:07
SundarSince no objections were raised, I'll send a notice. From next week onwards, this is the new meeting time.03:08
Sundar#topic os-acc discontinuance03:09
*** openstack changes topic to "os-acc discontinuance (Meeting topic: openstack-cyborg)"03:09
SundarA patch was filed to fix Python 3 issues in os-acc. To avoid such hassles, I propose to discontinue os-acc. If there are no objections, I'll start the process of retiring the repository right away.03:10
SundarAre there any objections?03:10
SundarSince no objections were raised, I'll send a message in openstack-discuss and start the process.03:12
SundarWe are following the agenda in: https://wiki.openstack.org/wiki/Meetings/CyborgTeamMeeting#Agenda03:12
SundarSince we don't have NTT folks here today, we'll jump ahead by one topic.03:12
Sundar#topic Tempest CI03:13
*** openstack changes topic to "Tempest CI (Meeting topic: openstack-cyborg)"03:13
SundarOur (my?) current plan is to kick off tempest with the fake driver for Nova patches and tempest with real FPGA hardware for Cyborg patches. Do you think we should both types of CI for both kinds of patches?03:14
SundarYumeng, wangzhh, xinranwang, yikun: ^03:14
SundarPretty silent group today :)03:17
wangzhhThinking...03:18
wangzhhDoes nova guys agree with only tempest with real hardware?03:19
SundarNova wants the fake driver tempest primarily.03:19
SundarCyborg may also want the same.03:20
yikunthe real device driver is implemented as the 3rd party ci?03:20
SundarYumeng: Yes, for Intel FPGAs03:20
SundarSorry, yikun :)03:20
Yumengfor the real FPGA hardware for Cyborg patches,  we mainly use it to test the  discover and report part, while for the fake driver for Nova patches, we mainly use it to test the attach_detach part,right ?03:21
yikunOK, no objection from me, I think it's same as the original plan, isn't it?03:21
wangzhhUh-huh, it's better to imply them one by one, fake driver first?03:21
SundarIt was not clear to me that we agreed on fake driver tempest for Cyborg patches too. If so, do we need to file a new patch for that?03:22
SundarYumeng: The fake driver part will test the whole flow from Nova till Cyborg agent, using the fake driver. WHile 3p CI with real hardware will ensure real FPGA driver and functionality like programming are not broken03:23
Sundarwangzhh: Agreed - let's start with fake driver for both. We will have 3p CI going for Cyborg 'soon'.03:24
xinranwang I consider the fake driver is a solution for nova-side requirements, it mainly aims at nova-cyborg interaction.03:24
Sundarxinranwang: When we submit a Cyborg patch, we want to ensure that does not break Nova-Cyborg interaction either.03:26
SundarNova requirements are the forcing function, but it helps to have it for Cyborg too, right?03:26
xinranwangSo we will always keep it, even if we finished the 3p CI?03:28
SundarAre you advocating that we do only the 3p CI for Cyborg? That would be the minimum.03:30
SundarThat is what I have currently planned for. Just checking in with all of you.03:31
xinranwangNo, I just want make sure the fake driver is a long term solution or not.03:31
SundarOK. The minimum for Nova is fake-driver. The minimum for Cyborg, IMHO, is 3p CI with real hardware. As long as the hardware setup is always available and functional, we don't need the fake driver tempest for Cyborg. Are we all aligned on that?03:33
Sundarwangzhh, yikun, Yumeng, xinranwang: ^03:35
xinranwangAgree with you03:35
wangzhhAgree +103:35
Yumengyes,agree with that.03:36
SundarGreat, we have a quorum. Let's stick with the minimum for Train and build on it later, if needed03:37
Sundar#agreed Will implement only fake-driver tempest for Nova patches and 3p CI with Intel FPGA for Cyborg patches in Train.03:37
SundarYumeng: Can you update https://review.opendev.org/#/c/670999/ to reflect the comments? i.e. mark it as WIP, and place a dependency on my patch series?03:37
Yumengok,np03:38
SundarThanks03:38
Sundar#topic Storyboard tasks03:38
*** openstack changes topic to "Storyboard tasks (Meeting topic: openstack-cyborg)"03:38
SundarThere are 3 important tasks: RBAC, Cyborg functional tests, Nova functional tests that mock Cyborg API03:38
SundarFor RBAC, can any of you sign up? It is important for security. Our common/policy.py is all for v1 API. But it needs to be fixed for v2.03:39
*** shaohe_feng has joined #openstack-cyborg03:39
Sundarwangzhh or yikun: Will be ideal if one of you can pick up :)  because Yumeng and Xinran have chosen other activities03:41
wangzhhI can take policy releated work next week.03:42
SundarExcellent, wangzhh. Thanks :)03:42
wangzhhNp :)03:42
SundarNova functional tests that mock Cyborg API: I think we need this to be done for the Nova patches to merge.03:43
SundarNobody has raised this yet but I remember this from PTG.03:44
Sundaryikun: If this is a game of musical chairs, you are the last person left. :)03:44
SundarOK, no pressure. We need to revisit this later.03:47
Sundar#topic AoB03:47
*** openstack changes topic to "AoB (Meeting topic: openstack-cyborg)"03:47
SundarPlease note that the IRC meeting time shifts next week as agreed.03:48
SundarPython 3 is super-important for Train. Apparently, Cyborg missed adding Py3 tests, which was a goal in Pike release.03:49
SundarIf we make Python 3 job as a voting job, we have to expend work now to keep it all fixed. If we make it a non-voting job, there is a risk because there is apparently a plan to disable all py27 testing early in next cycle.03:50
Sundarwangzhh, Yumeng, xinranwang, yikun: what do you favor?03:51
xinranwangIt depends on how much time we left. But I do know there's some bugs need to fix when i run py3 tests.03:53
SundarIf we have time, we also have Cyborg functional tests to do -- basic for good quality.03:54
xinranwangthose are all nice to have. Let's focus on critical path.03:54
xinranwangWe need more review on nova-cyborg integ patches and placement report patch.03:55
SundarYes. IIUC, you are suggesting to keep Python 3 jobs as no-voting for now and make them voting in the next cycle, right?03:56
xinranwangYes, after we finish the important things, if we have more time, we can work on it in this release as well, but it's not obligate. That's my suggestion :)03:58
SundarGot it. Any objections?03:58
SundarAny objections from wangzhh, yikun, Yumeng?03:59
wangzhhFine with me.03:59
SundarOK, let's go with that plan.04:00
SundarThanks, everybody. Have a good day! Bye.04:00
Sundar#endmeeting04:00
*** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)"04:00
shaohe_fengcritical path is high priority04:00
openstackMeeting ended Wed Aug 14 04:00:32 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)04:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-08-14-03.03.html04:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-08-14-03.03.txt04:00
openstackLog:            http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-08-14-03.03.log.html04:00
shaohe_fengSundar https://review.opendev.org/#/c/673228/04:00
Sundarshaohe_feng: Good to see you here. Yes.04:00
shaohe_fenghas fix the python3 bug04:01
SundarYou mean you have a fix for some Py3 bug?04:01
shaohe_fengyou mentioned in the email04:01
shaohe_fengthat the python3 ci can not work04:02
shaohe_fengI looks into this bug04:02
shaohe_fengand fix it.04:02
shaohe_fenghttps://www.python.org/dev/peps/pep-0479/#sub-proposal-decorator-to-explicitly-request-current-behaviour04:03
SundarAh I see. Thanks, shaohe_feng.04:03
shaohe_fengand this is office python3 link to tell us what's has changed.04:04
shaohe_fengI add the link in the patch, to let others know this issue more easily.04:05
shaohe_fengSundar: I need to go, bye.04:06
SundarBye04:07
*** Sundar has quit IRC04:07
*** shaohe_feng has quit IRC04:12
*** links has joined #openstack-cyborg04:26
*** links has quit IRC05:09
*** links has joined #openstack-cyborg05:13
*** Yumeng has quit IRC05:46
*** jraju__ has joined #openstack-cyborg05:54
*** links has quit IRC05:55
*** xinranwang has quit IRC07:21
openstackgerritYumengBao proposed openstack/cyborg master: Implement privsep boilerplate in cyborg.  https://review.opendev.org/67395707:32
*** helenafm has joined #openstack-cyborg08:00
*** wangzhh has quit IRC08:01
*** helenafm has quit IRC10:58
*** jraju__ has quit IRC13:35
*** helenafm has joined #openstack-cyborg14:14
*** helenafm has quit IRC15:06
*** efried has quit IRC20:45
*** efried has joined #openstack-cyborg20:46
openstackgerritBiwei Guo proposed openstack/cyborg-tempest-plugin master: End-to-end scenario test added  https://review.opendev.org/66723123:23

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!