03:02:44 <Sundar> #startmeeting openstack-cyborg
03:02:45 <openstack> Meeting started Wed Jul 24 03:02:44 2019 UTC and is due to finish in 60 minutes.  The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot.
03:02:46 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
03:02:48 <openstack> The meeting name has been set to 'openstack_cyborg'
03:02:55 <Sundar> #topic Roll call
03:03:12 <Sundar> #info Sundar
03:03:19 <wangzhh> Hi all.
03:03:25 <Sundar> Hi Zhenghao
03:03:29 <wangzhh> #info wangzhh
03:03:55 <s_shogo> #info s_shogo
03:03:58 <Sundar> Let's wait a couple of minutes for people to join
03:04:14 <wangzhh> Yep.
03:04:16 <Yumeng> #info Yumeng
03:04:16 <Sundar> Hi s_shogo
03:05:07 <Sundar> Hi Yumeng
03:05:08 <Yumeng> hi all
03:05:29 <Sundar> I think we have waited enough. Let's get started.
03:05:33 <Sundar> #topic Specs
03:06:15 <Sundar> https://review.opendev.org/#/c/658263/ I suppose people have reviewed this. Thanks, Yumeng, for the +2. Can somebody give a workflow of +1?
03:08:08 <Sundar> Thanks, wangzhh
03:08:44 <Sundar> Can we also complete the review for https://review.opendev.org/603545 ?
03:09:29 <xinranwang> it looks fine for me. I can give a +@
03:09:31 <xinranwang> it looks fine for me. I can give a +2
03:10:02 <Sundar> Thanks, xinranwang. Appreciate it. I would also appreciate if somebody else can review it too.
03:10:17 <Sundar> Any other spec to talk about?
03:10:50 <Sundar> #topic Patches
03:11:10 <Sundar> https://review.opendev.org/#/q/project:openstack/cyborg+branch:master+topic:nova-integ This badly needs reviews!
03:12:11 <xinranwang> Is there any merge order of these patches?
03:12:26 <Sundar> The merge order is shown in the subject: P1, P2, etc.
03:12:33 <xinranwang> Ok, got it
03:12:37 <xinranwang> Thanks
03:12:50 <Sundar> NP
03:13:18 <Sundar> It looks like other folks are busy with their work :)
03:13:40 <xinranwang> Is Yumeng  here?
03:13:47 <Yumeng> yes
03:14:12 <openstackgerrit> Merged openstack/cyborg-specs master: Specification for Cyborg APIs Version 2.  https://review.opendev.org/658263
03:14:15 <xinranwang> Hi Yumeng
03:14:22 <Yumeng> Hi xinran
03:14:37 <Sundar> wangzhh, Yumeng: Please take a look when you can, s_shogo: Reviewing the specs and patches may be a oood way for you to get introduced to Cyborg code. :)
03:14:52 <Sundar> *good
03:15:07 <xinranwang> I noticed that you submit this patch https://review.opendev.org/#/c/669308/,  do need we this before pilot code merge to the master branch? Sundar
03:15:42 <xinranwang> * do we need
03:15:51 <wangzhh> OK. Sorry for meeting conflict. I can't reply in time.
03:16:20 <wangzhh> Will review it later.
03:16:55 <Sundar> I don't think we need tempest CI for the pilot branch, IMHO. No harm if we do it, but it is not an absolute necessity.
03:17:30 <Sundar> The tempest CI on the master will check the merged pilot code patches.
03:17:38 <Sundar> Thanks, wangzhh. Appreciate it.
03:18:09 <Sundar> There are other patches pending too. I'll review as much as I can by next week.
03:18:39 <Yumeng> thanks xinranwang, that's what I'm wanting to know.  Sundar, does that mean we will no longer need pilot patch in the near furture?
03:19:20 <xinranwang> Tempest plugin code is running with V2 API, it can not pass the test without pilot branch enabled for now. But after pilot branch merged, there will be no issues
03:19:23 <Yumeng> *pilot branch
03:19:54 <s_shogo> Thanks Sundar, I'll start review gradually.
03:21:05 <Sundar> Yumeng: I am thinking that, for networked FPGAs, we can either reuse the pilot branch or submit directly to master. There will be many changes depending on some ongoing Nova/Placement changes. But they are not needed for basic Cyborg.
03:21:23 <Sundar> #topic Python 3 migration
03:21:47 <Sundar> s_shogo: Thanks to you and ikuo for the first patch: https://review.opendev.org/#/c/667524/
03:22:12 <Sundar> Should we wait for more patches before starting the review?
03:23:37 <s_shogo> Yes, i'll post update patch , that includes zuul config.
03:23:45 <Sundar> Thanks
03:23:52 <Sundar> #topic Python client
03:24:17 <Sundar> s_shogo, could you please update us on the state of the client?
03:26:15 <s_shogo> I have completed that work in local environment, posting the patch is ready.
03:26:33 <Sundar> Thanks
03:26:38 <Sundar> #topic AoB
03:26:47 <Sundar> Is there anything else to discuss?
03:26:59 <Yumeng> Sundar, for the tempest CI, I agree with xinranwang, Tempest plugin code is running with V2 API, it can not pass the test without pilot branch enabled for now.
03:27:29 <Yumeng> So maybe we can have a pilot job for now since it doesn't need a lot effort
03:28:33 <Sundar> Yumeng: I am fine with that. However, the patches https://review.opendev.org/#/q/project:openstack/cyborg+branch:master+topic:nova-integ also enable V2 API in master. If they get merged, do we still need tempest for pilot branch?
03:30:22 <Yumeng> emmm..  as for this, let me think it over. I can leave you meesage
03:30:29 <Sundar> If you think it is easier or faster to enable it in pilot branch, that's fine.
03:31:34 <Sundar> I'll remove the -1s after this call.
03:31:48 <Sundar> Anything else, folks?
03:31:52 <Yumeng> Thanks Sundar
03:33:09 <Sundar> Thanks a lot, everybody! Have a good day. ;)  Bye.
03:33:13 <Sundar> #endmeeting