03:01:52 #startmeeting openstack-cyborg 03:01:54 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 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 03:01:57 The meeting name has been set to 'openstack_cyborg' 03:02:05 #topic Roll call 03:02:19 #info Sundar 03:02:30 #info Biwei 03:02:35 #info s_shogo 03:02:45 Hi Yumeng, Biwei, xinranwang, s_shogo 03:02:51 Hi all 03:02:56 #info xinranwang 03:03:56 #topic Spec review 03:04:09 #link https://opendev.org/openstack/cyborg-specs/src/branch/master/specs/train/cyborg-accelerator-driver.rst 03:04:26 This spec is not in the right location: should be under train/approved 03:04:56 The formatting is also broken, as we can see 03:05:03 hi all 03:05:09 #info Yumeng 03:06:06 Can somebody get this fixed? Ideally, the original author will do so. If not, anybody else can take it up. Thoughts? 03:07:57 I can take a look. 03:08:23 Thanks, Yumeng. You could also ping yikun, the original author, to see what he thinks. 03:08:45 ok will do 03:09:00 oops, I will fix soon 03:09:05 #info yikun 03:09:23 yikun, welcome and thanks 03:09:36 #link https://review.opendev.org/#/q/status:open+project:openstack/cyborg-specs+owner:Sundar+after:2019-07-01 03:10:12 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 So, I may update the Nova-Cyborg spec, with this content 03:10:46 Yikun Jiang proposed openstack/cyborg-specs master: Move the spec into right location https://review.opendev.org/669974 03:11:28 yikun: That was fast! The formatting also needs some updates 03:12:00 Sundar: sure, thanks for reminder 03:12:04 #topic Tempest CI 03:12:52 For calling tempest CI, we need a Nova patch as well. Yumeng, can you or your colleagues help us with that? 03:13:19 do you mean upstream ci with fake driver? 03:13:50 or a real third party ci? 03:14:06 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 So, this is just the patch to set up Zuul gate in Nova to invoke this 03:15:52 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 Yumeng, is that fine? 03:16:32 We intend to bring up a VM, but perhaps it will have no device? 03:17:03 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 That will cause the virt driver to ignore it silently and bring up a VM without it. 03:17:47 That will tell us the entire flow is good though there ais no host device in the VM's xml 03:18:34 xinranwang, Biwei: Please add or clarify as needed 03:18:44 aha.. got it. clear enough 03:18:52 thanks sundar 03:19:11 I will do the zuul patch 03:19:22 Great. Thanks, Yumeng! 03:19:47 #topic Getting pilot into master 03:20:01 yes Sundar is right, need modify virt driver as well 03:20:48 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 *pilot 03:22:31 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 If anybody has objections, please LMK 03:24:16 Agree, maybe separate into small patch is easier to review too :) 03:24:28 Agree, and small patches is better to me 03:25:20 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 Anyway, np, we'll do it the proper way this time 03:25:38 can we separate according to APIs, for example, Deviceprofile API with all db functions and test cases, and so on. 03:25:49 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 Hope to get this done by this week. 03:27:08 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 #topic Storyboard task update 03:27:41 yikun, I was thinking of splitting them into API layer, objects layer and db layer, as before 03:27:59 Any better ideas? 03:28:11 it's ok to me 03:28:46 Storyboard: #link https://storyboard.openstack.org/#!/project/openstack/cyborg 03:29:09 I have updated the old tasks and added new ones. Please review and add/modify as needed 03:30:30 Any comments? 03:32:02 #topic AoB 03:32:31 I am out of office July 15 through July 22. 03:32:50 I may not be able to join next week. But you are all welcome to meet as needed. 03:33:11 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 So, I will repost that with fix of tests and so on. 03:33:59 s_shogo: ikuo posted it to stable/stein branch. Please post it to master branch instead. 03:34:38 sorry, I also fix that. 03:34:47 Thanks. NP :) 03:34:58 Anything else, all? 03:36:07 Have a good day, everybody! Bye. 03:36:11 #endmeeting