| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-cyborg - 2019-07-10 | |||
| 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 | |
| 03:36:13 | openstack | Meeting ended Wed Jul 10 03:36:11 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 03:36:14 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-07-10-03.01.html | |
| 03:36:15 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-07-10-03.01.txt | |
| 03:36:16 | openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2019/openstack_cyborg.2019-07-10-03.01.log.html | |
| 03:37:19 | yikun | ^ here are two easy fixes, I guess we could get these merged soon | |
| 03:37:19 | yikun | https://review.opendev.org/#/c/660603/ a fix on devstack agent enabled_drivers config | |
| 03:37:19 | yikun | https://review.opendev.org/#/c/660353/ a fix of gpu discover | |
| 03:38:29 | yikun | https://review.opendev.org/#/c/668868/ and xinran's patch can also be merged | |
| 03:40:23 | Li_Liu | Patch for enabling external event for Nova | |
| 03:40:24 | Li_Liu | https://review.opendev.org/#/c/669975/ | |
| 03:48:16 | openstackgerrit | Merged openstack/cyborg master: Fix re defination of GPU https://review.opendev.org/660353 | |
| 03:48:17 | openstackgerrit | Merged openstack/cyborg master: Fix the section of enabled_drivers https://review.opendev.org/660603 | |
| 04:05:36 | openstackgerrit | Merged openstack/cyborg master: Change attach_info, cpid_info from string to a dict. https://review.opendev.org/668868 | |
| 06:20:12 | openstackgerrit | Yikun Jiang proposed openstack/cyborg-specs master: Move the spec into right location https://review.opendev.org/669974 | |
| 06:57:44 | openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add Huawei Ascend driver https://review.opendev.org/660874 | |
| 07:25:34 | openstackgerrit | Yikun Jiang proposed openstack/cyborg master: Add the fake driver https://review.opendev.org/665318 | |
| 08:57:25 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: add a new driver for hddl https://review.opendev.org/670015 | |
| 09:10:56 | openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: add a new driver for hddl https://review.opendev.org/670015 | |
| 16:37:19 | openstackgerrit | Merged openstack/cyborg-tempest-plugin master: add zuul job to run cyborg tempest https://review.opendev.org/669010 | |
| 22:31:50 | openstackgerrit | Biwei Guo proposed openstack/cyborg-tempest-plugin master: End-to-end scenario test added https://review.opendev.org/667231 | |
| #openstack-cyborg - 2019-07-11 | |||
| 22:56:57 | openstackgerrit | Merged openstack/cyborg master: bug fix: deploy every cyborg components correctly https://review.opendev.org/659292 | |
| #openstack-cyborg - 2019-07-12 | |||
| 08:15:01 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P2: Changes for db layer, constants and exceptions. https://review.opendev.org/670463 | |
| 08:15:01 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P1: Disable old tests and add base for APIv2 tests. https://review.opendev.org/670462 | |
| 08:15:02 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P4: Basic changes for objects layer. https://review.opendev.org/670465 | |
| 08:15:02 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P3: Changes for services. https://review.opendev.org/670464 | |
| 08:15:03 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P6: Changes for device profiles. https://review.opendev.org/670467 | |
| 08:15:03 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P5: Basic changes for API layer. https://review.opendev.org/670466 | |
| 08:15:04 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P8: Changes for programming. https://review.opendev.org/670469 | |
| 08:15:04 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P7: Changes for ARQs and ExtARQs. https://review.opendev.org/670468 | |
| 08:15:05 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg master: P9: Enable v2 API in devstack. https://review.opendev.org/670470 | |
| 08:41:32 | openstackgerrit | Sundar Nadathur proposed openstack/cyborg-specs master: Specification for Cyborg APIs Version 2. https://review.opendev.org/658263 | |