| gmann | brinzhang_: done | 00:04 |
|---|---|---|
| gmann | that is needed for flag ignore_basepython_conflict | 00:04 |
| brinzhang_ | ignore_basepython_conflict add in python 3.1.1 version? | 00:05 |
| brinzhang_ | gmann: ok, I know, +2 for that patch. | 00:06 |
| gmann | brinzhang_: thanks | 00:06 |
| gmann | brinzhang_: this one also - https://review.opendev.org/#/c/704076/ | 00:07 |
| gmann | updated the tox env | 00:07 |
| brinzhang_ | gmann: done too. | 00:07 |
| gmann | thanks | 00:08 |
| brinzhang_ | I think I should upgrade cyborg project too, will do | 00:08 |
| openstackgerrit | Brin Zhang proposed openstack/cyborg master: Upgrade minversion to match python3 https://review.opendev.org/716796 | 00:11 |
| brinzhang_ | gmann: pls review this patch, I upgrade minversion in tox | 00:11 |
| gmann | +1. i hope py37 timeout things are solved. | 00:14 |
| brinzhang_ | I think that's the py37 failed reason previous | 00:15 |
| *** links has joined #openstack-cyborg | 02:17 | |
| *** Sundar has joined #openstack-cyborg | 02:46 | |
| *** tetsuro has quit IRC | 02:50 | |
| *** s_shogo has joined #openstack-cyborg | 02:51 | |
| *** xinranwang has joined #openstack-cyborg | 02:57 | |
| *** chenke has joined #openstack-cyborg | 02:59 | |
| *** links has quit IRC | 03:00 | |
| Sundar | #startmeeting openstack-cyborg | 03:00 |
| openstack | Meeting started Thu Apr 2 03:00:44 2020 UTC and is due to finish in 60 minutes. The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot. | 03:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 03:00 |
| *** openstack changes topic to " (Meeting topic: openstack-cyborg)" | 03:00 | |
| openstack | The meeting name has been set to 'openstack_cyborg' | 03:00 |
| Sundar | Hi all | 03:00 |
| Sundar | #topic Roll call | 03:01 |
| *** openstack changes topic to "Roll call (Meeting topic: openstack-cyborg)" | 03:01 | |
| Sundar | o/ Sundar | 03:01 |
| s_shogo | Hi all | 03:01 |
| brinzhang_ | o/ | 03:01 |
| xinranwang | Hi all | 03:01 |
| Sundar | Just the 3 of us? May be we'll wait a couple of min. | 03:01 |
| *** Yumeng has joined #openstack-cyborg | 03:01 | |
| chenke | Hi all | 03:02 |
| chenke | o/ | 03:02 |
| Yumeng | hi chenke, hi all | 03:02 |
| Sundar | Hi all | 03:02 |
| Sundar | Good, let's get started | 03:02 |
| Sundar | First the good news! | 03:02 |
| Sundar | The Nova integration is essentially done! https://review.opendev.org/#/q/project:openstack/nova+bp/nova-cyborg-interaction+(status:merged%20OR%20status:open) | 03:03 |
| s_shogo | Congrats !!! | 03:03 |
| Sundar | I am just finsihing up on the release notes and a patch for a corner case. | 03:03 |
| Yumeng | bravo!! such a great effort! | 03:03 |
| chenke | Great! | 03:04 |
| brinzhang_ | nice job | 03:04 |
| xinranwang | Congrats | 03:04 |
| xinranwang | Thanks for your efforts | 03:04 |
| Sundar | It has been a long journey -- we started in Rocky release, then we had the cyborg-nova pilot code, got the spec merged, and this patch series has been going for about a year. | 03:04 |
| Sundar | Thanks :) I am relieved. | 03:05 |
| Sundar | I am also happy that many companies plan to use Cyborg and, in fact, have already been using it. | 03:06 |
| chenke | a big step. | 03:06 |
| Sundar | Some telco companies in China also look forward to Cyborg with some networking support. | 03:06 |
| Yumeng | Yes, it's not easy. Great news for us! | 03:07 |
| Sundar | The best part is, all of you are developers and also users. That is the ideal situation: users guide the development of the project. | 03:07 |
| brinzhang_ | We are using now | 03:07 |
| Yumeng | brinzhang_: great! so does ZTE. | 03:09 |
| Sundar | The part that is not so good for me is that I have moved onto another role. So, I can spend only my spare time outside of work on Cyborg. I will continue to review patches and conduct the meetings (unless somebody else wants to take that over). But I may not have time to write lots of new patches. | 03:10 |
| Sundar | It is a bit tough for me to step aside from Cyborg, but that is the way forward. | 03:10 |
| brinzhang_ | Sundar: That's enough, I know that will take your free time | 03:11 |
| brinzhang_ | thanks | 03:11 |
| Sundar | I will complete the 4 open patches here: https://review.opendev.org/#/q/status:open+project:openstack/cyborg+branch:master+owner:Sundar | 03:12 |
| Sundar | Re. https://review.opendev.org/684456, I think xinranwang and I agree that it is not necessary now. | 03:12 |
| Sundar | Is there any other important piece left for me, apart from completing the PTL duties for U? | 03:13 |
| s_shogo | >https://review.opendev.org/684456 I approve that, to install OPAE in other way is ok, IMO. | 03:15 |
| Sundar | s_shogo: Good, thanks. | 03:15 |
| Sundar | If any of you think of something I have missed, you can LMK later too. | 03:16 |
| Sundar | For the rest of this meeting, we should probably prioritize the patches that need to merge before Apr 9. | 03:16 |
| Sundar | #topic Patches before Apr 9 | 03:16 |
| *** openstack changes topic to "Patches before Apr 9 (Meeting topic: openstack-cyborg)" | 03:16 | |
| Yumeng | Thank you Sundar! You've contributed many to Cyborg. It's hard to say en end,but you've done a good begining and good end. | 03:16 |
| Sundar | Thanks, Yumeng | 03:17 |
| Sundar | https://review.opendev.org/#/q/status:open+project:openstack/cyborg+branch:master | 03:17 |
| chenke | I will review your patches as soon as possble. | 03:18 |
| Sundar | The most important patches are the ones I am working on. Umm, no, just kidding :) | 03:18 |
| Yumeng | I will also review them. | 03:18 |
| Sundar | It is the microversion patch: https://review.opendev.org/696860 | 03:18 |
| Yumeng | haha. | 03:19 |
| xinranwang | Yes, please review it when you got time. | 03:19 |
| Yumeng | will do. | 03:19 |
| Sundar | What else would we consider highest priority? | 03:19 |
| xinranwang | I also submit a patch completing v2 api doc | 03:20 |
| xinranwang | https://review.opendev.org/715980 please review this too :) | 03:21 |
| chenke | Hi sundar. I found a bug when deleing device profile before a vm(with fake-device) was deleted. | 03:21 |
| Sundar | xinranwang: I agree it is important but we can also do it after Apr 9, if we need to. | 03:22 |
| Sundar | chenke: Interesting, what's the bug? | 03:22 |
| chenke | That's because the foriegn key constraint. | 03:22 |
| Sundar | The original idea was to allow the device profile deletion when the VM is running, but we don't allow it today | 03:22 |
| Sundar | chenke: Yes, the ARQs refer to the device profile entry | 03:23 |
| Sundar | But they don't need to -- we copy the device profile groups into the ARQs, so we can remove that link | 03:23 |
| chenke | So, is it reasonable to throw this error? | 03:23 |
| chenke | No. Just copy the device profile groups is not enough. | 03:23 |
| chenke | the error like this: `cyborg`.`extended_accelerator_requests`, CONSTRAINT `extended_accelerator_requests_ibfk_1` FOREIGN KEY (`device_profile_id`) | 03:24 |
| Sundar | chenke: For now, the shortest improvement would be to block it at the API level rather than throw an error | 03:25 |
| Sundar | For the future, we may need more changes to use the device profile groups copied into the ARQ | 03:26 |
| Sundar | However, there is a fundamental problem. | 03:26 |
| Sundar | On the Nova side, the operator may do a hard reboot or other ops for the VM anytime, and those ops will delete the old ARQs, and create new ones from the device profile name. | 03:26 |
| Sundar | So, if the devic eprofile has been deleted, that is a problem. | 03:26 |
| Sundar | Any thoughts? | 03:29 |
| chenke | Ok. | 03:29 |
| chenke | So we should keep the device profile. | 03:30 |
| Sundar | So, let us block the deletion at the device profile API level rather than throw an error | 03:30 |
| chenke | Agree. | 03:30 |
| *** tetsuro has joined #openstack-cyborg | 03:30 | |
| Sundar | Cool. | 03:31 |
| Sundar | BTW, I will be proposing a patch shortly for CYborg's highlights for the U release. I'll share that link. | 03:31 |
| Yumeng | yes ,agree now we just blocl the API level. but later maybe we can do like nova flavor. create a copy in nova for ops like reboot | 03:31 |
| Yumeng | but that needs nova approval.. :- | 03:32 |
| Sundar | Yumeng: The instance object has a copy of the flavor inside it, but does not have the ARQs. It has to call Cyborg with the device profile name to create the ARQ for some instance ops. | 03:33 |
| Sundar | Yes, it needs more changes in Nova | 03:33 |
| Sundar | We may need to make ARQs into an OVO, so it can be stored in the instance object. | 03:33 |
| Sundar | Plus, db changes in Nova | 03:34 |
| Yumeng | yes. | 03:34 |
| chenke | actually that's hard. | 03:35 |
| Sundar | Yup | 03:36 |
| Sundar | That's why I didn;t do it :) | 03:36 |
| chenke | Ok. I have no other needed to be discussed. | 03:37 |
| Sundar | Anything else, folks? | 03:38 |
| brinzhang_ | one patch | 03:38 |
| brinzhang_ | Sundar, chenke, s_shogo: I think this is why py37 timeout previous https://review.opendev.org/#/c/716796/1/tox.ini | 03:39 |
| brinzhang_ | the ignore_basepython_conflict flag introduced in 3.1.1, before we are in 2.0 | 03:39 |
| brinzhang_ | that I upgrade this to 3.1.1 to adopt python3 | 03:40 |
| chenke | good catch. | 03:41 |
| Sundar | Yea, makes sense to me. | 03:41 |
| s_shogo | good | 03:41 |
| Sundar | #topic AoB | 03:42 |
| *** openstack changes topic to "AoB (Meeting topic: openstack-cyborg)" | 03:42 | |
| Sundar | Anything else? | 03:42 |
| *** igordc has quit IRC | 03:42 | |
| brinzhang_ | Nothing else, many thing talked in wechat, and have done of mine ^^ | 03:43 |
| Sundar | Thanks a lot, folks. Have a good day, and stay safe away from the virus! | 03:43 |
| Sundar | #endmeeting | 03:44 |
| *** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)" | 03:44 | |
| openstack | Meeting ended Thu Apr 2 03:44:00 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 03:44 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-02-03.00.html | 03:44 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-02-03.00.txt | 03:44 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-02-03.00.log.html | 03:44 |
| s_shogo | Me too. I'll update the deployable programming patch, please review it. > folks | 03:44 |
| chenke | Thanks all. Bye. will do it. | 03:44 |
| s_shogo | bye | 03:45 |
| *** s_shogo has quit IRC | 03:45 | |
| Yumeng | bye | 03:47 |
| *** Sundar has quit IRC | 03:50 | |
| openstackgerrit | Merged openstack/cyborg master: Upgrade minversion to match python3 https://review.opendev.org/716796 | 04:48 |
| openstackgerrit | Merged openstack/cyborg master: Add Cyborg API Reference to Cyborg index page https://review.opendev.org/711869 | 04:53 |
| *** links has joined #openstack-cyborg | 05:25 | |
| openstackgerrit | Brin Zhang proposed openstack/cyborg master: Optimize device profiles APIs https://review.opendev.org/712049 | 06:35 |
| *** chenke has quit IRC | 06:46 | |
| openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: Add hostname to FakeDevice name to support multi-host deployment https://review.opendev.org/693784 | 06:58 |
| *** tetsuro_ has joined #openstack-cyborg | 07:10 | |
| *** tetsuro has quit IRC | 07:13 | |
| *** tetsuro_ has quit IRC | 07:14 | |
| openstackgerrit | Merged openstack/cyborg master: Add comments for deletion of ARQs by instance or ARQ UUIDs. https://review.opendev.org/714782 | 07:20 |
| *** tetsuro has joined #openstack-cyborg | 07:40 | |
| openstackgerrit | Merged openstack/cyborg master: Optimize device profiles APIs https://review.opendev.org/712049 | 07:52 |
| *** tetsuro has quit IRC | 09:14 | |
| openstackgerrit | Brin Zhang proposed openstack/cyborg master: Remove future imports https://review.opendev.org/716910 | 10:05 |
| openstackgerrit | Brin Zhang proposed openstack/cyborg master: Use unittest.mock instead of third party mock https://review.opendev.org/716916 | 10:21 |
| *** brinzhang_ has quit IRC | 10:24 | |
| *** brinzhang_ has joined #openstack-cyborg | 10:24 | |
| openstackgerrit | Brin Zhang proposed openstack/cyborg master: hacking: force explicit import of python's mock https://review.opendev.org/716920 | 10:28 |
| *** brinzhang has joined #openstack-cyborg | 10:58 | |
| *** brinzhang_ has quit IRC | 11:01 | |
| *** brinzhang_ has joined #openstack-cyborg | 11:21 | |
| *** brinzhang has quit IRC | 11:24 | |
| *** Yumeng has quit IRC | 11:35 | |
| *** xinranwang has quit IRC | 11:46 | |
| *** links has quit IRC | 15:03 | |
| openstackgerrit | Merged openstack/cyborg-tempest-plugin master: Remove py27 from tox env list https://review.opendev.org/704076 | 17:31 |
| *** igordc has joined #openstack-cyborg | 21:05 | |
| *** brinzhang has joined #openstack-cyborg | 21:34 | |
| *** brinzhang_ has quit IRC | 21:37 | |
| *** openstackstatus has quit IRC | 22:56 | |
| *** openstack has joined #openstack-cyborg | 22:59 | |
| *** ChanServ sets mode: +o openstack | 22:59 | |
| *** brinzhang_ has joined #openstack-cyborg | 23:15 | |
| *** brinzhang has quit IRC | 23:18 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!