Thursday, 2020-01-16

*** donnyd has quit IRC00:18
*** donnyd has joined #openstack-cyborg00:19
*** TxGirlGeek has quit IRC00:26
openstackgerritBrin Zhang proposed openstack/cyborg master: Add cyborg quickstart reference in devstack  https://review.opendev.org/69741900:32
*** donnyd has quit IRC00:43
*** ildikov has quit IRC00:43
*** gmann has quit IRC00:43
*** dustinc has quit IRC00:46
*** masayukig has quit IRC00:49
openstackgerritMerged openstack/cyborg master: Set ignore_basepython_conflict (fixes confusing pep8 message)  https://review.opendev.org/69639700:59
*** Li_Liu has quit IRC01:01
*** zhurong has joined #openstack-cyborg01:08
*** chenke has joined #openstack-cyborg01:13
*** zhurong has quit IRC01:18
*** zhurong has joined #openstack-cyborg01:28
*** igordc has quit IRC01:42
*** chenke has quit IRC01:53
*** chenke has joined #openstack-cyborg02:43
openstackgerritchenker proposed openstack/cyborg master: Remove useless code for V1  https://review.opendev.org/70278502:59
*** s_shogo has joined #openstack-cyborg02:59
chenkeHi shogo02:59
s_shogoHi chenke03:00
brinzhanghi03:00
chenkewill you update the client code recently03:00
chenkehi bailin.03:00
s_shogoUes, correspond to sdk update, thank you for your review >> chenke03:01
s_shogos/Ues/Yes/03:01
chenkeOk. After you update. I will learn it again.03:01
brinzhangWhen start the meetting?03:02
chenkemaybe we need to wait for sundar,xinran.03:02
brinzhangyeah \o03:03
openstackgerritBrin Zhang proposed openstack/cyborg master: Add Cyborg quickstart reference in devstack  https://review.opendev.org/69741903:06
chenkeseems sundar is not here.03:12
*** Sundar has joined #openstack-cyborg03:15
SundarHi all03:15
s_shogoHi Sundar03:15
SundarSorry, I am late03:15
brinzhanghi03:15
Sundar#startmeeting openstack-cyborg03:15
openstackMeeting 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.03:15
*** openstack changes topic to " (Meeting topic: openstack-cyborg)"03:15
openstackThe meeting name has been set to 'openstack_cyborg'03:15
Sundar#topic Who's here03:15
*** openstack changes topic to "Who's here (Meeting topic: openstack-cyborg)"03:15
Sundar#info Sundar03:15
s_shogo#info s_shogo03:16
chenke#info chenke03:16
brinzhang#info brnzhang03:16
SundarFor 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
SundarDo any of you have something else to add?03:18
brinzhangI want to talk about http://lists.openstack.org/pipermail/openstack-discuss/2020-January/011990.html03:18
brinzhangDo we need to use the launchpad03:18
SundarOh yes. brinzhang, I was planning to reply to that03:19
Sundarbrinzhang, we had some discussions last year, and folks decided to go with storyboard.03:19
brinzhangit's ok03:19
SundarWe already have a lot at https://storyboard.openstack.org/#!/project_group/7703:20
brinzhangEither way, as long as it can be managed uniformly, iam ok03:20
SundarIs there any string advantage to launchpad over storyboard that you want to bring u?03:20
Sundar*strong03:20
SundarOk03:21
brinzhangI think the lauchpad was more intuitive03:21
SundarIs it signficantly faster? I find storyboard to be very slow03:21
brinzhangore intuitive03:22
brinzhangAs far as I know, more people use launchpad than storyboard03:22
brinzhangyeah, launchpad is faster than storyboard03:22
brinzhangsometime the storyboard always stuck03:23
brinzhangso I recommand to use launchpad03:23
SundarWe don't seem to have a quorum today to decide that, or even discuss it at length03:23
SundarI suggest you can bring it up in a future IRC meeting.03:24
brinzhangyeah, I think this need more people to join, we talk about next topic, i think03:24
SundarSure, I'll add this to the meeting agenda at https://wiki.openstack.org/wiki/Meetings/CyborgTeamMeeting.03:25
chenkeYe.03:25
SundarWe don't have Yumeng today to discuss the policy refresh03:25
chenkeI think storyborad is clearer。03:25
SundarThere is a lot of discussions going to microversions. I'd just like to make sure we have a solid footing for the future03:26
Sundarchenke, got it. I suspect folks will have different opinions.03:26
brinzhangReference the nova microversion, use microversion is more useful03:26
chenkeyumeng is out of office.03:26
Sundarbrinzhang: yes, microversions are necessary to add features and evolve the APis sommothly, which is important for a young project like Cyborg03:27
brinzhangWhen a new feature changed in Cyborg, we can record this with a microversion03:27
chenkeAgree03:27
brinzhangSo about this spec, what are you worry about? Sundar03:28
Sundarbrinzhang: 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 release03:29
Sundarnotes, as you pointed out in the Nova patch review03:29
Sundar*documenting03:29
brinzhangI'm not sure if the existing Cyborg feature will be implemented after this SPEC merger, or in parallel with it?03:30
SundarThe 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
brinzhangyes, we should consider more, such as the unit tests, documenting, functional test etc.03:31
SundarUntil 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
brinzhangSundar, 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
SundarSure03:33
brinzhangSundar, Even so, we also need to maintain microversion step by step03:34
SundarThe 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
brinzhangThis is my opion for this spec, I will review it again while I free, and refer the experience I was in Nova03:35
SundarGreat, sounds good03:35
Sundar#topic Spec for instance ops for accelerators03:36
*** openstack changes topic to "Spec for instance ops for accelerators (Meeting topic: openstack-cyborg)"03:36
brinzhangagree "The device v2 APis would be 2.0"03:36
Sundarhttps://review.opendev.org/#/c/605237/03:36
brinzhangI was reviewed this spec, I agree with matt, this spec mostly seem description the documents?is right?03:37
SundarI 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
brinzhangsome operators are in nova03:37
SundarYes, all these are instance ops in Nova, and the doc addresses what should happen if the instance has accelerators03:38
brinzhangSundar, yeah, it's document03:38
SundarIt may make sense to retire it as a spec and make it a document03:38
brinzhangyeah, it make sense03:38
SundarOk, I'll take an action item for that03:39
brinzhangI think we should ensure the accuracy of the description03:39
SundarIs something not accurate?03:39
brinzhangSo we need more nova core to review this, such as erifed, stephenfin, alex_xu, sean-k-moony to review it, make it better and better03:40
brinzhangSee my comment in PS403:40
SundarThey 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
brinzhangabout the rebuild action, I think you should re-word03:41
Sundarok, will go by the review03:41
brinzhangMaybe we can send this to the nova meeting03:42
brinzhangNext, i will review again, and check all instance ops03:42
SundarI'll ask the Nova PTL if that is feasible, after I recast it as a doc03:42
brinzhangSundar Above is my opinion on this03:43
SundarSure, anything else on this subject?03:43
brinzhanganother way, it may be possible to add this to Nova's document library as an auxiliary document using Cyborg as an instance03:44
Sundar'Using Cyborg as an instance' -- what does that mean?03:44
brinzhangput this doc in the Nova's library, while some want to use Cyborg, they can read this03:45
brinzhangof course, we also need put this in Cyborg's document library03:45
brinzhangjust an idea03:46
SundarNova'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
SundarFor example, hypothetically, if there is a problem with doing instance rebuilds with GPU drivers, that caveat should be in Cyborg docs, right?03:47
brinzhangYeah, but I think nova also need this03:48
SundarOk, no harm in asking them if they can do it.03:48
brinzhangmaybe, put it in Cyborg is appropriate03:49
brinzhangmore appropriate03:49
chenkeI think cyborg is the main one and Nova can briefly introduce it.03:49
SundarSure03:49
Sundar#topic Other topics03:51
*** openstack changes topic to "Other topics (Meeting topic: openstack-cyborg)"03:51
SundarThe next PTG will be in Vancouver, June 8-1103:51
chenkeAbout this patch. https://review.opendev.org/#/c/702578/03:51
SundarEarly Bird Full Access costs $299 and lasts till Mar 2, 2020.03:52
Sundarchenke: I started looking at your patch. It is highly needed. Thanks03:52
chenkeOk.  Thanks.03:52
s_shogoI'm considering a gap between cyborg API spec and sdk's premise(mainly arq delete API,with multi choice of argument),03:52
brinzhangSundar I need to follow the company's decision whether to participate in this PTG03:52
Sundarbrinzhang: Sure, just mentioning it for awareness03:53
brinzhangJust need have a topic, also a candidate speeker03:53
SundarWould you like to collaborate with one of us for a speech or a project update?03:54
SundarAnyways, we can discuss it offline03:54
Sundars_shogo: Yes03:54
SundarI think we had a brief discussion. The HTTP DELETE's query parameters can be used to unify both forms, right?03:55
brinzhangabout chenke's patch, i have a doubt, consistency of interface parameters, separated from rpcapi03:55
brinzhangsundar, while you review this, please take a care03:55
Sundarbrinzhang: ok03:55
brinzhangmianly about the 'context' parameter03:55
chenkecontext is useless for create db device profile03:56
chenkeI removed it . https://review.opendev.org/#/c/702578/2/cyborg/db/sqlalchemy/api.py03:57
chenkeWhat are your thoughts03:57
Sundarchenke: 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 API03:57
brinzhangchenke, we can wait for Sundar reviewed this, then talk about03:57
s_shogoSundar: That depends on sdk's delete function, I'll clarify the gap and ask this again.03:57
Sundars_shogo: ok03:58
chenkebrinzhang ok.03:58
Sundarbrinzhang: Welcome again to Cyborg. Could you give us some background about yourself? Do you use accelerators like FPGAs in your company?03:58
chenkeI have to go for lunch.  bye~03:59
brinzhangWe are use GPU in Cyborg, FPGA in plan~03:59
brinzhangNow we are applying the fpga device :)04:00
brinzhangchenke, bye04:00
Sundarchenke: Bye04:00
Sundarbrinzhang: Cool. What important things are missing from Cyborg, apart from nova integ?04:01
SundarAnd could I ask if they are Intel FPGAs? ;)04:01
brinzhangSundar, I was gurduate from Zhengzhou Unviersity, now I am working in Inspur, mainly work in OpenStack (Community)04:01
brinzhangYes, in theory, Intel's fpga04:02
Sundarbrinzhang: Good to know that. Happy to have you in OpenStack and Cyborg.04:03
brinzhangSundar, thanks04:03
s_shogoWelcome brinzhang (again)04:03
SundarPlease feel free to reach out to me in WeChat or IRC if you need anything04:04
brinzhangs_shogo thanks~04:04
SundarAnything else, folks?04:04
brinzhangNothing for me today04:04
SundarI think the Chinese New year is coming up soon?04:04
SundarShould we hold a meeting next week?04:05
brinzhangYeah, next Thursday, next meeting I will not join, on the way (go home)04:05
SundarAh ok. Advance wishes -- Shenyun Kwai le :)04:06
SundarHave a good day, everybody.04:06
s_shogoThere 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.html04:06
brinzhangThanks04:06
Sundars_shogo: Sean and I talked on Nova IRC this morning04:07
s_shogoSundar OK,thanks.04:07
SundarBut thanks for pointing it out to me04:07
Sundar#endmeeting04:07
*** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)"04:07
openstackMeeting ended Thu Jan 16 04:07:41 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)04:07
openstackMinutes:        http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-01-16-03.15.html04:07
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-01-16-03.15.txt04:07
openstackLog:            http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-01-16-03.15.log.html04:07
*** openstackgerrit has quit IRC04:12
*** Sundar has quit IRC04:21
*** masayukig has joined #openstack-cyborg04:21
*** masayukig has quit IRC04:26
*** gmann has joined #openstack-cyborg06:49
*** donnyd has joined #openstack-cyborg06:52
*** Li_Liu has joined #openstack-cyborg06:54
*** dustinc has joined #openstack-cyborg07:03
*** masayukig has joined #openstack-cyborg07:04
*** openstackgerrit has joined #openstack-cyborg07:08
openstackgerritchenker proposed openstack/cyborg master: P1: public interface define [Improve UT for cyborg/db]  https://review.opendev.org/70280707:08
openstackgerritchenker proposed openstack/cyborg master: P1: public interface define [Improve UT for cyborg/db]  https://review.opendev.org/70280707:12
openstackgerritchenker proposed openstack/cyborg master: Public interface define [Improve UT for cyborg/db]  https://review.opendev.org/70280707:22
openstackgerritchenker proposed openstack/cyborg master: Improve UT for cyborg/db device profile  https://review.opendev.org/70257807:25
*** ildikov has joined #openstack-cyborg07:25
openstackgerritchenker proposed openstack/cyborg master: Public interface define [Improve UT for cyborg/db]  https://review.opendev.org/70280707:29
openstackgerritchenker proposed openstack/cyborg master: Improve UT for cyborg/db device profile  https://review.opendev.org/70257807:29
*** s_shogo has quit IRC07:35
openstackgerritYumengBao proposed openstack/cyborg-specs master: add doc8 check  https://review.opendev.org/70281908:39
openstackgerritchenker proposed openstack/cyborg master: Improve UT for cyborg/db attach handle  https://review.opendev.org/70282509:06
*** davidsha has joined #openstack-cyborg09:46
openstackgerritchenker proposed openstack/cyborg master: Remove useless code for V1 and fix uuid error  https://review.opendev.org/70278511:39
openstackgerritchenker proposed openstack/cyborg master: Public interface define [Improve UT for cyborg/db]  https://review.opendev.org/70280711:39
openstackgerritShaoHe Feng proposed openstack/cyborg master: testcase for FPGAExtARQ  https://review.opendev.org/69915311:42
openstackgerritShaoHe Feng proposed openstack/cyborg master: UT for job manager  https://review.opendev.org/70050411:42
openstackgerritchenker proposed openstack/cyborg master: Remove useless get_test_accelerator method and fix uuid error  https://review.opendev.org/70278512:20
openstackgerritchenker proposed openstack/cyborg master: Public interface define [Improve UT for cyborg/db]  https://review.opendev.org/70280712:20
openstackgerritchenker proposed openstack/cyborg master: Remove useless get_test_accelerator method and fix uuid error  https://review.opendev.org/70278512:24
openstackgerritchenker proposed openstack/cyborg master: Public interface define [Improve UT for cyborg/db]  https://review.opendev.org/70280712:24
openstackgerritchenker proposed openstack/cyborg master: Improve UT for cyborg/db device profile  https://review.opendev.org/70257812:24
*** tetsuro has quit IRC12:45
openstackgerritBrin Zhang proposed openstack/cyborg master: Add functional tests frame to the Cyborg  https://review.opendev.org/70286313:28
*** TxGirlGeek has joined #openstack-cyborg16:09
*** davidsha has quit IRC17:24
*** openstackgerrit has quit IRC18:58
*** TxGirlGeek has quit IRC22:21
*** TxGirlGeek has joined #openstack-cyborg23:08

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!