*** donnyd has quit IRC | 00:18 | |
*** donnyd has joined #openstack-cyborg | 00:19 | |
*** TxGirlGeek has quit IRC | 00:26 | |
openstackgerrit | Brin Zhang proposed openstack/cyborg master: Add cyborg quickstart reference in devstack https://review.opendev.org/697419 | 00:32 |
---|---|---|
*** donnyd has quit IRC | 00:43 | |
*** ildikov has quit IRC | 00:43 | |
*** gmann has quit IRC | 00:43 | |
*** dustinc has quit IRC | 00:46 | |
*** masayukig has quit IRC | 00:49 | |
openstackgerrit | Merged openstack/cyborg master: Set ignore_basepython_conflict (fixes confusing pep8 message) https://review.opendev.org/696397 | 00:59 |
*** Li_Liu has quit IRC | 01:01 | |
*** zhurong has joined #openstack-cyborg | 01:08 | |
*** chenke has joined #openstack-cyborg | 01:13 | |
*** zhurong has quit IRC | 01:18 | |
*** zhurong has joined #openstack-cyborg | 01:28 | |
*** igordc has quit IRC | 01:42 | |
*** chenke has quit IRC | 01:53 | |
*** chenke has joined #openstack-cyborg | 02:43 | |
openstackgerrit | chenker proposed openstack/cyborg master: Remove useless code for V1 https://review.opendev.org/702785 | 02:59 |
*** s_shogo has joined #openstack-cyborg | 02:59 | |
chenke | Hi shogo | 02:59 |
s_shogo | Hi chenke | 03:00 |
brinzhang | hi | 03:00 |
chenke | will you update the client code recently | 03:00 |
chenke | hi bailin. | 03:00 |
s_shogo | Ues, correspond to sdk update, thank you for your review >> chenke | 03:01 |
s_shogo | s/Ues/Yes/ | 03:01 |
chenke | Ok. After you update. I will learn it again. | 03:01 |
brinzhang | When start the meetting? | 03:02 |
chenke | maybe we need to wait for sundar,xinran. | 03:02 |
brinzhang | yeah \o | 03:03 |
openstackgerrit | Brin Zhang proposed openstack/cyborg master: Add Cyborg quickstart reference in devstack https://review.opendev.org/697419 | 03:06 |
chenke | seems sundar is not here. | 03:12 |
*** Sundar has joined #openstack-cyborg | 03:15 | |
Sundar | Hi all | 03:15 |
s_shogo | Hi Sundar | 03:15 |
Sundar | Sorry, I am late | 03:15 |
brinzhang | hi | 03:15 |
Sundar | #startmeeting openstack-cyborg | 03:15 |
openstack | Meeting started Thu Jan 16 03:15:39 2020 UTC and is due to finish in 60 minutes. The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot. | 03:15 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 03:15 |
*** openstack changes topic to " (Meeting topic: openstack-cyborg)" | 03:15 | |
openstack | The meeting name has been set to 'openstack_cyborg' | 03:15 |
Sundar | #topic Who's here | 03:15 |
*** openstack changes topic to "Who's here (Meeting topic: openstack-cyborg)" | 03:15 | |
Sundar | #info Sundar | 03:15 |
s_shogo | #info s_shogo | 03:16 |
chenke | #info chenke | 03:16 |
brinzhang | #info brnzhang | 03:16 |
Sundar | For the agenda today, apart from the patch reviews, I was thinking about the security policy refresh, microversions, and the spec for instance ops on accelerators. | 03:17 |
Sundar | Do any of you have something else to add? | 03:18 |
brinzhang | I want to talk about http://lists.openstack.org/pipermail/openstack-discuss/2020-January/011990.html | 03:18 |
brinzhang | Do we need to use the launchpad | 03:18 |
Sundar | Oh yes. brinzhang, I was planning to reply to that | 03:19 |
Sundar | brinzhang, we had some discussions last year, and folks decided to go with storyboard. | 03:19 |
brinzhang | it's ok | 03:19 |
Sundar | We already have a lot at https://storyboard.openstack.org/#!/project_group/77 | 03:20 |
brinzhang | Either way, as long as it can be managed uniformly, iam ok | 03:20 |
Sundar | Is there any string advantage to launchpad over storyboard that you want to bring u? | 03:20 |
Sundar | *strong | 03:20 |
Sundar | Ok | 03:21 |
brinzhang | I think the lauchpad was more intuitive | 03:21 |
Sundar | Is it signficantly faster? I find storyboard to be very slow | 03:21 |
brinzhang | ore intuitive | 03:22 |
brinzhang | As far as I know, more people use launchpad than storyboard | 03:22 |
brinzhang | yeah, launchpad is faster than storyboard | 03:22 |
brinzhang | sometime the storyboard always stuck | 03:23 |
brinzhang | so I recommand to use launchpad | 03:23 |
Sundar | We don't seem to have a quorum today to decide that, or even discuss it at length | 03:23 |
Sundar | I suggest you can bring it up in a future IRC meeting. | 03:24 |
brinzhang | yeah, I think this need more people to join, we talk about next topic, i think | 03:24 |
Sundar | Sure, I'll add this to the meeting agenda at https://wiki.openstack.org/wiki/Meetings/CyborgTeamMeeting. | 03:25 |
chenke | Ye. | 03:25 |
Sundar | We don't have Yumeng today to discuss the policy refresh | 03:25 |
chenke | I think storyborad is clearer。 | 03:25 |
Sundar | There is a lot of discussions going to microversions. I'd just like to make sure we have a solid footing for the future | 03:26 |
Sundar | chenke, got it. I suspect folks will have different opinions. | 03:26 |
brinzhang | Reference the nova microversion, use microversion is more useful | 03:26 |
chenke | yumeng is out of office. | 03:26 |
Sundar | brinzhang: yes, microversions are necessary to add features and evolve the APis sommothly, which is important for a young project like Cyborg | 03:27 |
brinzhang | When a new feature changed in Cyborg, we can record this with a microversion | 03:27 |
chenke | Agree | 03:27 |
brinzhang | So about this spec, what are you worry about? Sundar | 03:28 |
Sundar | brinzhang: I think we all agree on the need for microversions. The questions would be, are we following the best practices etc. I invited Chris Dent etc. to review it. If you see any gaps, please bring it up. It is better to take the time and do this properly to avoid gaps. For example, adopting conventions like docmetning APi changes in release | 03:29 |
Sundar | notes, as you pointed out in the Nova patch review | 03:29 |
Sundar | *documenting | 03:29 |
brinzhang | I'm not sure if the existing Cyborg feature will be implemented after this SPEC merger, or in parallel with it? | 03:30 |
Sundar | The way I see it, we released v2.0 of APis in Train. Whatever comes out of Ussuri will/should be microversion-enabled and will probably be v2.1. After that, we have incremental changes. | 03:31 |
brinzhang | yes, we should consider more, such as the unit tests, documenting, functional test etc. | 03:31 |
Sundar | Until we get Nova integration done, any version bump in Cyborg is not of great significance as people are not yet putting it in production. | 03:32 |
brinzhang | Sundar, so while someone pushed a spec for Cyborg, we should require he/she to add a new microversion for that api (if the api changes) | 03:32 |
Sundar | Sure | 03:33 |
brinzhang | Sundar, Even so, we also need to maintain microversion step by step | 03:34 |
Sundar | The device v2 APis would be 2.0, because they are totally new. But existing APIs like device profiles and ARQs would get a bump if they are changed. | 03:34 |
brinzhang | This is my opion for this spec, I will review it again while I free, and refer the experience I was in Nova | 03:35 |
Sundar | Great, sounds good | 03:35 |
Sundar | #topic Spec for instance ops for accelerators | 03:36 |
*** openstack changes topic to "Spec for instance ops for accelerators (Meeting topic: openstack-cyborg)" | 03:36 | |
brinzhang | agree "The device v2 APis would be 2.0" | 03:36 |
Sundar | https://review.opendev.org/#/c/605237/ | 03:36 |
brinzhang | I was reviewed this spec, I agree with matt, this spec mostly seem description the documents?is right? | 03:37 |
Sundar | I wrote this last year as a way to get agreement between Nova and Cyborg sides. Some people did not see it as a spec per se, because it is not specifying any changes. It is only documented behavior. | 03:37 |
brinzhang | some operators are in nova | 03:37 |
Sundar | Yes, all these are instance ops in Nova, and the doc addresses what should happen if the instance has accelerators | 03:38 |
brinzhang | Sundar, yeah, it's document | 03:38 |
Sundar | It may make sense to retire it as a spec and make it a document | 03:38 |
brinzhang | yeah, it make sense | 03:38 |
Sundar | Ok, I'll take an action item for that | 03:39 |
brinzhang | I think we should ensure the accuracy of the description | 03:39 |
Sundar | Is something not accurate? | 03:39 |
brinzhang | So we need more nova core to review this, such as erifed, stephenfin, alex_xu, sean-k-moony to review it, make it better and better | 03:40 |
brinzhang | See my comment in PS4 | 03:40 |
Sundar | They have all reviewed it to some extent or the other in Train. If it is recast as a Cyborg doc, it will be tougher to get their review, but we can try. | 03:41 |
brinzhang | about the rebuild action, I think you should re-word | 03:41 |
Sundar | ok, will go by the review | 03:41 |
brinzhang | Maybe we can send this to the nova meeting | 03:42 |
brinzhang | Next, i will review again, and check all instance ops | 03:42 |
Sundar | I'll ask the Nova PTL if that is feasible, after I recast it as a doc | 03:42 |
brinzhang | Sundar Above is my opinion on this | 03:43 |
Sundar | Sure, anything else on this subject? | 03:43 |
brinzhang | another way, it may be possible to add this to Nova's document library as an auxiliary document using Cyborg as an instance | 03:44 |
Sundar | 'Using Cyborg as an instance' -- what does that mean? | 03:44 |
brinzhang | put this doc in the Nova's library, while some want to use Cyborg, they can read this | 03:45 |
brinzhang | of course, we also need put this in Cyborg's document library | 03:45 |
brinzhang | just an idea | 03:46 |
Sundar | Nova's handling of instance ops varies with Neutron ports, Cinder volumes, etc. I doubt that Nova will agree to put a project-specific doc in their list. In fact, it may make sense to keep it in Cyborg, because different Cyborg drivers may potentially support different ops . | 03:46 |
Sundar | For example, hypothetically, if there is a problem with doing instance rebuilds with GPU drivers, that caveat should be in Cyborg docs, right? | 03:47 |
brinzhang | Yeah, but I think nova also need this | 03:48 |
Sundar | Ok, no harm in asking them if they can do it. | 03:48 |
brinzhang | maybe, put it in Cyborg is appropriate | 03:49 |
brinzhang | more appropriate | 03:49 |
chenke | I think cyborg is the main one and Nova can briefly introduce it. | 03:49 |
Sundar | Sure | 03:49 |
Sundar | #topic Other topics | 03:51 |
*** openstack changes topic to "Other topics (Meeting topic: openstack-cyborg)" | 03:51 | |
Sundar | The next PTG will be in Vancouver, June 8-11 | 03:51 |
chenke | About this patch. https://review.opendev.org/#/c/702578/ | 03:51 |
Sundar | Early Bird Full Access costs $299 and lasts till Mar 2, 2020. | 03:52 |
Sundar | chenke: I started looking at your patch. It is highly needed. Thanks | 03:52 |
chenke | Ok. Thanks. | 03:52 |
s_shogo | I'm considering a gap between cyborg API spec and sdk's premise(mainly arq delete API,with multi choice of argument), | 03:52 |
brinzhang | Sundar I need to follow the company's decision whether to participate in this PTG | 03:52 |
Sundar | brinzhang: Sure, just mentioning it for awareness | 03:53 |
brinzhang | Just need have a topic, also a candidate speeker | 03:53 |
Sundar | Would you like to collaborate with one of us for a speech or a project update? | 03:54 |
Sundar | Anyways, we can discuss it offline | 03:54 |
Sundar | s_shogo: Yes | 03:54 |
Sundar | I think we had a brief discussion. The HTTP DELETE's query parameters can be used to unify both forms, right? | 03:55 |
brinzhang | about chenke's patch, i have a doubt, consistency of interface parameters, separated from rpcapi | 03:55 |
brinzhang | sundar, while you review this, please take a care | 03:55 |
Sundar | brinzhang: ok | 03:55 |
brinzhang | mianly about the 'context' parameter | 03:55 |
chenke | context is useless for create db device profile | 03:56 |
chenke | I removed it . https://review.opendev.org/#/c/702578/2/cyborg/db/sqlalchemy/api.py | 03:57 |
chenke | What are your thoughts | 03:57 |
Sundar | chenke: I think brinzhang is saying that, when we remove context as a parameter for db api, it may be good to remove it from other places which use it only for dbapi, such as the main API | 03:57 |
brinzhang | chenke, we can wait for Sundar reviewed this, then talk about | 03:57 |
s_shogo | Sundar: That depends on sdk's delete function, I'll clarify the gap and ask this again. | 03:57 |
Sundar | s_shogo: ok | 03:58 |
chenke | brinzhang ok. | 03:58 |
Sundar | brinzhang: Welcome again to Cyborg. Could you give us some background about yourself? Do you use accelerators like FPGAs in your company? | 03:58 |
chenke | I have to go for lunch. bye~ | 03:59 |
brinzhang | We are use GPU in Cyborg, FPGA in plan~ | 03:59 |
brinzhang | Now we are applying the fpga device :) | 04:00 |
brinzhang | chenke, bye | 04:00 |
Sundar | chenke: Bye | 04:00 |
Sundar | brinzhang: Cool. What important things are missing from Cyborg, apart from nova integ? | 04:01 |
Sundar | And could I ask if they are Intel FPGAs? ;) | 04:01 |
brinzhang | Sundar, I was gurduate from Zhengzhou Unviersity, now I am working in Inspur, mainly work in OpenStack (Community) | 04:01 |
brinzhang | Yes, in theory, Intel's fpga | 04:02 |
Sundar | brinzhang: Good to know that. Happy to have you in OpenStack and Cyborg. | 04:03 |
brinzhang | Sundar, thanks | 04:03 |
s_shogo | Welcome brinzhang (again) | 04:03 |
Sundar | Please feel free to reach out to me in WeChat or IRC if you need anything | 04:04 |
brinzhang | s_shogo thanks~ | 04:04 |
Sundar | Anything else, folks? | 04:04 |
brinzhang | Nothing for me today | 04:04 |
Sundar | I think the Chinese New year is coming up soon? | 04:04 |
Sundar | Should we hold a meeting next week? | 04:05 |
brinzhang | Yeah, next Thursday, next meeting I will not join, on the way (go home) | 04:05 |
Sundar | Ah ok. Advance wishes -- Shenyun Kwai le :) | 04:06 |
Sundar | Have a good day, everybody. | 04:06 |
s_shogo | There are question from nova member in this IRC, last FRIday - we had better to follow that http://eavesdrop.openstack.org/irclogs/%23openstack-cyborg/%23openstack-cyborg.2020-01-10.log.html | 04:06 |
brinzhang | Thanks | 04:06 |
Sundar | s_shogo: Sean and I talked on Nova IRC this morning | 04:07 |
s_shogo | Sundar OK,thanks. | 04:07 |
Sundar | But thanks for pointing it out to me | 04:07 |
Sundar | #endmeeting | 04:07 |
*** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)" | 04:07 | |
openstack | Meeting ended Thu Jan 16 04:07:41 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 04:07 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-01-16-03.15.html | 04:07 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-01-16-03.15.txt | 04:07 |
openstack | Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-01-16-03.15.log.html | 04:07 |
*** openstackgerrit has quit IRC | 04:12 | |
*** Sundar has quit IRC | 04:21 | |
*** masayukig has joined #openstack-cyborg | 04:21 | |
*** masayukig has quit IRC | 04:26 | |
*** gmann has joined #openstack-cyborg | 06:49 | |
*** donnyd has joined #openstack-cyborg | 06:52 | |
*** Li_Liu has joined #openstack-cyborg | 06:54 | |
*** dustinc has joined #openstack-cyborg | 07:03 | |
*** masayukig has joined #openstack-cyborg | 07:04 | |
*** openstackgerrit has joined #openstack-cyborg | 07:08 | |
openstackgerrit | chenker proposed openstack/cyborg master: P1: public interface define [Improve UT for cyborg/db] https://review.opendev.org/702807 | 07:08 |
openstackgerrit | chenker proposed openstack/cyborg master: P1: public interface define [Improve UT for cyborg/db] https://review.opendev.org/702807 | 07:12 |
openstackgerrit | chenker proposed openstack/cyborg master: Public interface define [Improve UT for cyborg/db] https://review.opendev.org/702807 | 07:22 |
openstackgerrit | chenker proposed openstack/cyborg master: Improve UT for cyborg/db device profile https://review.opendev.org/702578 | 07:25 |
*** ildikov has joined #openstack-cyborg | 07:25 | |
openstackgerrit | chenker proposed openstack/cyborg master: Public interface define [Improve UT for cyborg/db] https://review.opendev.org/702807 | 07:29 |
openstackgerrit | chenker proposed openstack/cyborg master: Improve UT for cyborg/db device profile https://review.opendev.org/702578 | 07:29 |
*** s_shogo has quit IRC | 07:35 | |
openstackgerrit | YumengBao proposed openstack/cyborg-specs master: add doc8 check https://review.opendev.org/702819 | 08:39 |
openstackgerrit | chenker proposed openstack/cyborg master: Improve UT for cyborg/db attach handle https://review.opendev.org/702825 | 09:06 |
*** davidsha has joined #openstack-cyborg | 09:46 | |
openstackgerrit | chenker proposed openstack/cyborg master: Remove useless code for V1 and fix uuid error https://review.opendev.org/702785 | 11:39 |
openstackgerrit | chenker proposed openstack/cyborg master: Public interface define [Improve UT for cyborg/db] https://review.opendev.org/702807 | 11:39 |
openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: testcase for FPGAExtARQ https://review.opendev.org/699153 | 11:42 |
openstackgerrit | ShaoHe Feng proposed openstack/cyborg master: UT for job manager https://review.opendev.org/700504 | 11:42 |
openstackgerrit | chenker proposed openstack/cyborg master: Remove useless get_test_accelerator method and fix uuid error https://review.opendev.org/702785 | 12:20 |
openstackgerrit | chenker proposed openstack/cyborg master: Public interface define [Improve UT for cyborg/db] https://review.opendev.org/702807 | 12:20 |
openstackgerrit | chenker proposed openstack/cyborg master: Remove useless get_test_accelerator method and fix uuid error https://review.opendev.org/702785 | 12:24 |
openstackgerrit | chenker proposed openstack/cyborg master: Public interface define [Improve UT for cyborg/db] https://review.opendev.org/702807 | 12:24 |
openstackgerrit | chenker proposed openstack/cyborg master: Improve UT for cyborg/db device profile https://review.opendev.org/702578 | 12:24 |
*** tetsuro has quit IRC | 12:45 | |
openstackgerrit | Brin Zhang proposed openstack/cyborg master: Add functional tests frame to the Cyborg https://review.opendev.org/702863 | 13:28 |
*** TxGirlGeek has joined #openstack-cyborg | 16:09 | |
*** davidsha has quit IRC | 17:24 | |
*** openstackgerrit has quit IRC | 18:58 | |
*** TxGirlGeek has quit IRC | 22:21 | |
*** TxGirlGeek has joined #openstack-cyborg | 23:08 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!