| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2019-07-04 | |||
| 12:23:16 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: improve intel FPGA driver https://review.opendev.org/669160 | |
| 13:00:45 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: do not diff stub devices https://review.opendev.org/669172 | |
| 13:04:56 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: do not diff stub devices https://review.opendev.org/669172 | |
| 13:04:57 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: improve intel FPGA driver https://review.opendev.org/669160 | |
| #openstack-cyborg - 2019-07-05 | |||
| 05:19:39 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: do not diff stub devices https://review.opendev.org/669172 | |
| 05:19:40 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: improve intel FPGA driver https://review.opendev.org/669160 | |
| 05:21:10 | openstackgerrit | zhurong proposed openstack/cyborg-tempest-plugin master: For test https://review.opendev.org/669249 | |
| 06:38:47 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: improve intel FPGA driver https://review.opendev.org/669160 | |
| 09:07:47 | openstackgerrit | YumengBao proposed openstack/cyborg master: enable branch selection in devstack installation https://review.opendev.org/669303 | |
| 09:24:53 | openstackgerrit | YumengBao proposed openstack/cyborg-tempest-plugin master: add zuul job to run cyborg tempest https://review.opendev.org/669010 | |
| 09:24:54 | openstackgerrit | YumengBao proposed openstack/cyborg-tempest-plugin master: add cyborg-nova-pilot branch job to run pilot related tempest https://review.opendev.org/669308 | |
| 09:25:44 | openstackgerrit | YumengBao proposed openstack/cyborg-tempest-plugin master: add zuul job to run cyborg tempest https://review.opendev.org/669010 | |
| #openstack-cyborg - 2019-07-08 | |||
| 02:04:51 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Change attach_info, cpid_info from sting to a dict. https://review.opendev.org/668868 | |
| 02:11:44 | openstackgerrit | Merged openstack/cyborg master: Remove the old generic driver https://review.opendev.org/658986 | |
| 03:42:56 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg-specs master: Specification for Cyborg Nova Placement interaction. https://review.opendev.org/603545 | |
| 06:20:28 | openstackgerrit | Xinran WANG proposed openstack/cyborg master: Change attach_info, cpid_info from string to a dict. https://review.opendev.org/668868 | |
| #openstack-cyborg - 2019-07-09 | |||
| 02:55:01 | Sundar | I have created Storyboard tasks, marked as critical/important, etc. Please update as necessary and grab the opportunities to own specific areas, by assigning ownership to yourself! https://storyboard.openstack.org/#!/project/openstack/cyborg | |
| 13:58:34 | openstackgerrit | Merged openstack/cyborg master: Add AICHIP and GENERIC type in DB and obj https://review.opendev.org/660548 | |
| 13:58:34 | openstackgerrit | Merged openstack/cyborg master: Add the new generic driver https://review.opendev.org/658987 | |
| 17:44:42 | openstackgerrit | Biwei Guo proposed openstack/cyborg-tempest-plugin master: End-to-end scenario test added Cleaned up code and all tempest tests passing except for Delete Device Profile bug fixed pep8 errors https://review.opendev.org/667231 | |
| 18:04:16 | openstackgerrit | Biwei Guo proposed openstack/cyborg-tempest-plugin master: End-to-end scenario test added Cleaned up code and all tempest tests passing except for Delete Device Profile bug fixed pep8 errors https://review.opendev.org/667231 | |
| #openstack-cyborg - 2019-07-10 | |||
| 02:55:20 | Sundar | IRC meeting in 5 min | |
| 03:01:52 | Sundar | #startmeeting openstack-cyborg | |
| 03:01:53 | 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. | |