03:01:52 <Sundar> #startmeeting openstack-cyborg
03:01:54 <openstack> Meeting started Wed Jul 10 03:01:52 2019 UTC and is due to finish in 60 minutes.  The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot.
03:01:55 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
03:01:57 <openstack> The meeting name has been set to 'openstack_cyborg'
03:02:05 <Sundar> #topic Roll call
03:02:19 <Sundar> #info Sundar
03:02:30 <Biwei> #info Biwei
03:02:35 <s_shogo> #info s_shogo
03:02:45 <Sundar> Hi Yumeng, Biwei, xinranwang, s_shogo
03:02:51 <xinranwang> Hi all
03:02:56 <xinranwang> #info xinranwang
03:03:56 <Sundar> #topic Spec review
03:04:09 <Sundar> #link https://opendev.org/openstack/cyborg-specs/src/branch/master/specs/train/cyborg-accelerator-driver.rst
03:04:26 <Sundar> This spec is not in the right location: should be under train/approved
03:04:56 <Sundar> The formatting is also broken, as we can see
03:05:03 <Yumeng> hi all
03:05:09 <Yumeng> #info Yumeng
03:06:06 <Sundar> Can somebody get this fixed? Ideally, the original author will do so. If not, anybody else can take it up. Thoughts?
03:07:57 <Yumeng> I can take a look.
03:08:23 <Sundar> Thanks, Yumeng. You could also ping yikun, the original author, to see what he thinks.
03:08:45 <Yumeng> ok will do
03:09:00 <yikun> oops, I will fix soon
03:09:05 <yikun> #info yikun
03:09:23 <Sundar> yikun, welcome and thanks
03:09:36 <Sundar> #link https://review.opendev.org/#/q/status:open+project:openstack/cyborg-specs+owner:Sundar+after:2019-07-01
03:10:12 <Sundar> Thanks for the reviews. I'll respond to the comments. The instance ops spec should really be reviewed by both Cyborg and Nova developers, but we are getting only some eyeballs from Nova side
03:10:30 <Sundar> So, I may update the Nova-Cyborg spec, with this content
03:10:46 <openstackgerrit> Yikun Jiang proposed openstack/cyborg-specs master: Move the spec into right location  https://review.opendev.org/669974
03:11:28 <Sundar> yikun: That was fast! The formatting also needs some updates
03:12:00 <yikun> Sundar: sure, thanks for reminder
03:12:04 <Sundar> #topic Tempest CI
03:12:52 <Sundar> For calling tempest CI, we need a Nova patch as well. Yumeng, can you or your colleagues help us with that?
03:13:19 <Yumeng> do you mean upstream ci with fake driver?
03:13:50 <Yumeng> or a real third party ci?
03:14:06 <Sundar> Yes. Upstream CI for now. Xinranwang and Biwei are doing most of the plugin development. You have helped a lot with the tempest infrastructure.
03:14:25 <Sundar> So, this is just the patch to set up Zuul gate in Nova to invoke this
03:15:52 <Yumeng> ok. and one more question: About the "upstream ci with fake driver", what do we want to check at the end ? do we just need to check that the xml has pci device or we need to see a fake VM with pci device ?
03:15:56 <Sundar> Yumeng, is that fine?
03:16:32 <Sundar> We intend to bring up a VM, but perhaps it will have no device?
03:17:03 <Sundar> My understanding is, I'll modify the Nova virt driver to accept attach hand;es with attach_handle_type == 'TestPCI' or something like that
03:17:23 <Sundar> That will cause the virt driver to ignore it silently and bring up a VM without it.
03:17:47 <Sundar> That will tell us the entire flow is good though there ais no host device in the VM's xml
03:18:34 <Sundar> xinranwang, Biwei: Please add or clarify as needed
03:18:44 <Yumeng> aha.. got it. clear enough
03:18:52 <Yumeng> thanks sundar
03:19:11 <Yumeng> I will do the zuul patch
03:19:22 <Sundar> Great. Thanks, Yumeng!
03:19:47 <Sundar> #topic Getting pilot into master
03:20:01 <xinranwang> yes  Sundar is right, need modify virt driver as well
03:20:48 <Sundar> The patches in plot branch were merged into a big patch after we voted on it. However, I continue to get questions on that approach from folks outside Cyborg .
03:20:53 <Sundar> *pilot
03:22:31 <Sundar> So, it may be safest and least controversial to introduce the pilot code in master as a set of small patches. You can review either branch.
03:22:56 <Sundar> If anybody has objections, please LMK
03:24:16 <xinranwang> Agree, maybe separate into small patch is easier to review too :)
03:24:28 <yikun> Agree, and small patches is better to me
03:25:20 <Sundar> Yes, other projects prefer smaller patches. That's why I was surprised on the request to merge them into a big patch and that it got unanimous votes. :)
03:25:37 <Sundar> Anyway, np, we'll do it the proper way this time
03:25:38 <xinranwang> can we separate according to APIs, for example, Deviceprofile API with all db functions and test cases, and so on.
03:25:49 <Sundar> I think I have addressed most of the important comments. I have intentionally left many factors as TODO, so that we can all split the tasks among ourselves after the merge
03:26:07 <Sundar> Hope to get this done by this week.
03:27:08 <yikun> and then hope our small patches can be merged step by step, that means it's better make them independent of others.
03:27:11 <Sundar> #topic Storyboard task update
03:27:41 <Sundar> yikun, I was thinking of splitting them into API layer, objects layer and db layer, as before
03:27:59 <Sundar> Any better ideas?
03:28:11 <yikun> it's ok to me
03:28:46 <Sundar> Storyboard: #link https://storyboard.openstack.org/#!/project/openstack/cyborg
03:29:09 <Sundar> I have updated the old tasks and added new ones. Please review and add/modify as needed
03:30:30 <Sundar> Any comments?
03:32:02 <Sundar> #topic AoB
03:32:31 <Sundar> I am out of office July 15 through July 22.
03:32:50 <Sundar> I may not be able to join next week. But you are all welcome to meet as needed.
03:33:11 <s_shogo> About python3 migration.Ikuo posted a patch(py3 migration patch) before, but that doesn't include changes related to tests. ( zuul runs that with py2.7)
03:33:34 <s_shogo> So, I will repost that with fix of  tests and so on.
03:33:59 <Sundar> s_shogo: ikuo posted it to stable/stein branch. Please post it to master branch instead.
03:34:38 <s_shogo> sorry, I also fix that.
03:34:47 <Sundar> Thanks. NP :)
03:34:58 <Sundar> Anything else, all?
03:36:07 <Sundar> Have a good day, everybody! Bye.
03:36:11 <Sundar> #endmeeting