| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2019-08-14 | |||
| 03:19:15 | wangzhh | Does nova guys agree with only tempest with real hardware? | |
| 03:19:43 | Sundar | Nova wants the fake driver tempest primarily. | |
| 03:20:08 | Sundar | Cyborg may also want the same. | |
| 03:20:15 | yikun | the real device driver is implemented as the 3rd party ci? | |
| 03:20:30 | Sundar | Yumeng: Yes, for Intel FPGAs | |
| 03:20:42 | Sundar | Sorry, yikun :) | |
| 03:21:14 | Yumeng | for 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:15 | yikun | OK, no objection from me, I think it's same as the original plan, isn't it? | |
| 03:21:47 | wangzhh | Uh-huh, it's better to imply them one by one, fake driver first? | |
| 03:22:07 | Sundar | It 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:23:17 | Sundar | Yumeng: 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 broken | |
| 03:24:27 | Sundar | wangzhh: Agreed - let's start with fake driver for both. We will have 3p CI going for Cyborg 'soon'. | |
| 03:24:56 | xinranwang | I consider the fake driver is a solution for nova-side requirements, it mainly aims at nova-cyborg interaction. | |
| 03:26:13 | Sundar | xinranwang: When we submit a Cyborg patch, we want to ensure that does not break Nova-Cyborg interaction either. | |
| 03:26:43 | Sundar | Nova requirements are the forcing function, but it helps to have it for Cyborg too, right? | |
| 03:28:27 | xinranwang | So we will always keep it, even if we finished the 3p CI? | |
| 03:30:19 | Sundar | Are you advocating that we do only the 3p CI for Cyborg? That would be the minimum. | |
| 03:31:07 | Sundar | That is what I have currently planned for. Just checking in with all of you. | |
| 03:31:28 | xinranwang | No, I just want make sure the fake driver is a long term solution or not. | |
| 03:33:10 | Sundar | OK. 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:35:18 | Sundar | wangzhh, yikun, Yumeng, xinranwang: ^ | |
| 03:35:24 | xinranwang | Agree with you | |
| 03:35:47 | wangzhh | Agree +1 | |
| 03:36:08 | Yumeng | yes,agree with that. | |
| 03:37:00 | Sundar | Great, we have a quorum. Let's stick with the minimum for Train and build on it later, if needed | |
| 03:37:36 | Sundar | #agreed Will implement only fake-driver tempest for Nova patches and 3p CI with Intel FPGA for Cyborg patches in Train. | |
| 03:37:45 | Sundar | Yumeng: 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:38:18 | Yumeng | ok,np | |
| 03:38:23 | Sundar | Thanks | |
| 03:38:31 | Sundar | #topic Storyboard tasks | |
| 03:38:52 | Sundar | There are 3 important tasks: RBAC, Cyborg functional tests, Nova functional tests that mock Cyborg API | |
| 03:39:48 | Sundar | For 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:41:15 | Sundar | wangzhh or yikun: Will be ideal if one of you can pick up :) because Yumeng and Xinran have chosen other activities | |
| 03:42:27 | wangzhh | I can take policy releated work next week. | |
| 03:42:38 | Sundar | Excellent, wangzhh. Thanks :) | |
| 03:42:50 | wangzhh | Np :) | |
| 03:43:47 | Sundar | Nova functional tests that mock Cyborg API: I think we need this to be done for the Nova patches to merge. | |
| 03:44:14 | Sundar | Nobody has raised this yet but I remember this from PTG. | |
| 03:44:44 | Sundar | yikun: If this is a game of musical chairs, you are the last person left. :) | |
| 03:47:50 | Sundar | OK, no pressure. We need to revisit this later. | |
| 03:47:58 | Sundar | #topic AoB | |
| 03:48:17 | Sundar | Please note that the IRC meeting time shifts next week as agreed. | |
| 03:49:08 | Sundar | Python 3 is super-important for Train. Apparently, Cyborg missed adding Py3 tests, which was a goal in Pike release. | |
| 03:50:59 | Sundar | If 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:51:33 | Sundar | wangzhh, Yumeng, xinranwang, yikun: what do you favor? | |
| 03:53:00 | xinranwang | It depends on how much time we left. But I do know there's some bugs need to fix when i run py3 tests. | |
| 03:54:04 | Sundar | If we have time, we also have Cyborg functional tests to do -- basic for good quality. | |
| 03:54:48 | xinranwang | those are all nice to have. Let's focus on critical path. | |
| 03:55:13 | xinranwang | We need more review on nova-cyborg integ patches and placement report patch. | |
| 03:56:53 | Sundar | Yes. IIUC, you are suggesting to keep Python 3 jobs as no-voting for now and make them voting in the next cycle, right? | |
| 03:58:32 | xinranwang | Yes, 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:52 | Sundar | Got it. Any objections? | |
| 03:59:28 | Sundar | Any objections from wangzhh, yikun, Yumeng? | |
| 03:59:55 | wangzhh | Fine with me. | |
| 04:00:17 | Sundar | OK, let's go with that plan. | |
| 04:00:29 | Sundar | Thanks, everybody. Have a good day! Bye. | |
| 04:00:32 | Sundar | #endmeeting | |
| 04:00:33 | shaohe_feng | critical path is high priority | |
| 04:00:34 | openstack | Meeting ended Wed Aug 14 04:00:32 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 04:00:35 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-08-14-03.03.html | |
| 04:00:36 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-08-14-03.03.txt | |
| 04:00:37 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-08-14-03.03.log.html | |
| 04:00:47 | shaohe_feng | Sundar https://review.opendev.org/#/c/673228/ | |
| 04:00:55 | Sundar | shaohe_feng: Good to see you here. Yes. | |
| 04:01:01 | shaohe_feng | has fix the python3 bug | |
| 04:01:31 | Sundar | You mean you have a fix for some Py3 bug? | |
| 04:01:53 | shaohe_feng | you mentioned in the email | |
| 04:02:12 | shaohe_feng | that the python3 ci can not work | |
| 04:02:37 | shaohe_feng | I looks into this bug | |
| 04:02:41 | shaohe_feng | and fix it. | |
| 04:03:21 | shaohe_feng | https://www.python.org/dev/peps/pep-0479/#sub-proposal-decorator-to-explicitly-request-current-behaviour | |
| 04:03:42 | Sundar | Ah I see. Thanks, shaohe_feng. | |
| 04:04:34 | shaohe_feng | and this is office python3 link to tell us what's has changed. | |
| 04:05:20 | shaohe_feng | I add the link in the patch, to let others know this issue more easily. | |
| 04:06:32 | shaohe_feng | Sundar: I need to go, bye. | |
| 04:07:24 | Sundar | Bye | |
| 07:32:08 | openstackgerrit | YumengBao proposed openstack/cyborg master: Implement privsep boilerplate in cyborg. https://review.opendev.org/673957 | |
| 23:23:33 | openstackgerrit | Biwei Guo proposed openstack/cyborg-tempest-plugin master: End-to-end scenario test added https://review.opendev.org/667231 | |
| #openstack-cyborg - 2019-08-15 | |||
| 10:16:36 | openstackgerrit | YumengBao proposed openstack/cyborg master: add cyborg-tempest job and remove cyborg-devstack job https://review.opendev.org/676665 | |
| #openstack-cyborg - 2019-08-16 | |||
| 03:12:52 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Placement report https://review.opendev.org/659233 | |
| #openstack-cyborg - 2019-08-17 | |||
| 14:33:06 | openstackgerrit | Akira Yoshiyama proposed openstack/cyborg master: Fix CyborgObject.as_dict() for Python 3.x (py3 fixes: 1/2) https://review.opendev.org/677066 | |
| 14:44:08 | openstackgerrit | Akira Yoshiyama proposed openstack/cyborg master: Use list(filter) for Python 3.x (py3 fixes: 2/2) https://review.opendev.org/677067 | |
| #openstack-cyborg - 2019-08-19 | |||
| 06:40:18 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P6: Changes for device profiles. https://review.opendev.org/670467 | |
| 06:40:19 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P7: Changes for ARQs and ExtARQs. https://review.opendev.org/670468 | |
| 06:40:19 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P8: Changes for programming. https://review.opendev.org/670469 | |
| 06:40:20 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P9: Enable v2 API in devstack. https://review.opendev.org/670470 | |
| 06:40:20 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P5.5: Additional changes to db layer. https://review.opendev.org/677115 | |
| 07:50:48 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Notify Nova when binding finished https://review.opendev.org/674520 | |
| 09:23:05 | openstackgerrit | YumengBao proposed openstack/cyborg master: add device_profile 'name' duplicate check https://review.opendev.org/677154 | |
| 10:51:39 | openstackgerrit | YumengBao proposed openstack/cyborg master: add device_profile 'name' duplicate check https://review.opendev.org/677154 | |
| #openstack-cyborg - 2019-08-20 | |||
| 02:25:57 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Notify Nova when binding finished https://review.opendev.org/674520 | |
| 09:05:16 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: fake driver bug fix https://review.opendev.org/677436 | |
| 11:20:54 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Placement report https://review.opendev.org/659233 | |
| 15:22:14 | gmann | zhipeng: or any other core: i cannot find Sundar on irc to discuss the py3 patch. as per train community goal we need to make cyborg run on py3 and this patch seems fixing all - https://review.opendev.org/#/c/667524/ | |
| 15:22:59 | gmann | can we have direction on this if it is ok to make py3 jobs voting, if so i can update the patch. NOTE: start of U is our target to remove the py27 support | |
| #openstack-cyborg - 2019-08-21 | |||
| 06:28:48 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P5.5: Additional changes to db layer. https://review.opendev.org/677115 | |
| 06:28:49 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P6: Changes for device profiles. https://review.opendev.org/670467 | |
| 06:28:49 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P7: Changes for ARQs and ExtARQs. https://review.opendev.org/670468 | |
| 06:28:50 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P8: Changes for programming. https://review.opendev.org/670469 | |
| 06:28:50 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P9: Enable v2 API in devstack. https://review.opendev.org/670470 | |