Thursday, 2019-10-10

*** s_shogo has joined #openstack-cyborg02:14
*** tetsuro has quit IRC02:36
*** Yumeng has joined #openstack-cyborg02:41
*** chenke has joined #openstack-cyborg02:41
*** tetsuro has joined #openstack-cyborg02:45
*** xinranwang has joined #openstack-cyborg02:57
*** Sundar has joined #openstack-cyborg02:58
Sundar#startmeeting openstack-cyborg03:01
openstackMeeting started Thu Oct 10 03:01:45 2019 UTC and is due to finish in 60 minutes.  The chair is Sundar. Information about MeetBot at
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.03:01
*** openstack changes topic to " (Meeting topic: openstack-cyborg)"03:01
openstackThe meeting name has been set to 'openstack_cyborg'03:01
Sundar#topic Who's here03:01
*** openstack changes topic to "Who's here (Meeting topic: openstack-cyborg)"03:01
Yumeng#info Yumeng03:02
chenke#info chenke03:02
s_shogo#info s_shogo03:02
xinranwang#info xinranwang03:02
SundarHi all, hope you had a good holiday last week.03:02
Sundar#topic Status03:03
*** openstack changes topic to "Status (Meeting topic: openstack-cyborg)"03:03
SundarNothing much changed substantially last week. We got some doc patches into good shape.03:03
SundarPlease see:,
SundarThe async bind patch got merged. Whew, finally03:04
Sundar'Install OPAE tools': <-- this is specific to one driver but please do review it. Maybe you can look at the structure and style03:06
Sundar'Remove useless glance api_servers define': Unfortunately, I always see failures with this patch. I provided the logs and my cyborg.conf in that patch.03:07
SundarPlus some generic patches like
chenkeyes. This patch in analysis.03:07
SundarDo any of you have any comments or questions?03:08
chenkeI guess this error is because the previous issues.03:08
chenkeendpoint = utils.get_endpoint(ksa_adap)03:08
chenkeThe actually error is pointing here:  File "/opt/stack/cyborg/cyborg/image/", line 117, in get_api_servers\n    endpoint = utils.get_endpoint(ksa_adap)\n', u'  File "/opt/stack/cyborg/cyborg/common/", line 208, in get_endpoint\n    "interfaces: %s" % interfaces)\n03:09
Sundarchenke: Could you tell us what the previous issues are?03:10
chenkeThis code has not been executed03:10
chenkeSundar pls see this if condition03:11
SundarYes. So, the api_servers config seems necessary, right?03:11
chenkeThe api_servers hide the else condition.03:12
chenkenow when I delete this api_servers, the else condition raise error because "/opt/stack/cyborg/cyborg/common/", line 208, in get_endpoint"03:13
chenkeWhat I want to say is that this problem was hidden before.03:13
SundarGot it. So, the api_servers is mor elike a hack or workaround to hide the error in the 'else' clause03:14
chenkeAfter our meeting, I will find why the get_endpoint raise error.03:15
SundarThen the questions are: A. Do we have time to find the issue in the get_endpoint()? B. Is it worth fixing -- may be we should go straight to openstacksdk in U?03:15
chenkemaybe we could use openstacksdk.03:16
chenkeneed to have a try. I am not sure now. But eric suggest glance to use openstacksdk before.03:16
SundarYes, I believe we tried that approach and gave up because of a bug there. The opestacksdk patch which was suggested as a fix has since been abandoned.03:17
SundarAnyway, I agree this is a U-release conversation. Perhaps we should get started soon so that we can shoot for a resolution in November, early in the U cycle03:18
SundarCool, thanks Chenke03:20
SundarAll, please review the documentation patches. We need to close them for the next RC.03:21
SundarAlso, like I said, please review:
Sundar#topic PTG prep03:22
*** openstack changes topic to "PTG prep (Meeting topic: openstack-cyborg)"03:22
SundarI will create an etherpad and share with you. Please indicate who among you plan to attend.03:23
xinranwangnext RC is this week or 18 Oct03:23
Sundarxinranwang: The final RC is due this week. But I haven't seen a RC patch proposed yet.03:24
Sundar18 Oct is the release itself03:24
xinranwangSundar:  Ok, I will review it by this week.03:25
SundarYumeng: we need to work together for the project update.03:25
Yumengsundar: yes, I was about to ask.03:26
Yumengdo you have any plans about the structure?03:26
SundarJust curious, who among you plan to attend the PTG (after the Summit)?03:26
SundarYumeng: kind of. Can you access Google slides?03:27
xinranwangI will attend Summit and PTG, and I am preparing a demo at Intel booth during the Summit which show qat use case in cyborg and performance improvement.03:27
*** shaohe_feng has joined #openstack-cyborg03:27
shaohe_fenghi all.03:27
shaohe_fengsorry for late03:27
shaohe_fengsomething wrong with my pc.03:28
Sundarshaohe_feng: welcome, np03:28
chenkehi shaohe.03:28
shaohe_fengmorning chenke03:29
YumengSundar: google doc not always available. It will be appreciate if u can send me the file directly!03:29
SundarDO any of you have specific ideas for U release? Such as new drivers, or improvements to Cyborg? (Apart from nova integ, which IMO is most important.)03:29
s_shogoI'll attend the Summit and PTG (all day).03:29
SundarYumeng: got it, will do03:29
YumengI will attend Summit and PTG too03:29
SundarGreat, s_shogo and Yumeng03:30
SundarHope we'll get to meet chenke and zhurong too ;)03:30
xinranwangSundar:  from Intel side, there are  HDDL and QAT driver. We can do it in U release.03:30
chenkeI will attend Summit and PTG too.03:31
shaohe_fengYes, HDDL and QAT03:31
Sundar#topic AoB03:31
*** openstack changes topic to "AoB (Meeting topic: openstack-cyborg)"03:31
SundarAnything else to bring up today?03:31
shaohe_fengfor FPGA, should we support alias for function_id?03:31
shaohe_fengsuch as in DP:03:32
zhurongSundar hope to see you too03:32
Sundarshaohe: Yes, that would be a good addition. After nova integ.03:32
xinranwangYes, as shaohe said. it remains many TODOs and improvement in Cyborg, we can discuss during PTG.03:32
Sundarzhurong: Yes, I plan to be there03:32
SundarMy first trip to China :)03:33
chenkeSundar welcome to china.03:33
shaohe_feng"accel:function_id": "123_456_789_333", this is not friendly to user.03:33
*** licanwei has joined #openstack-cyborg03:33
shaohe_fengSundar welcome to china03:33
xinranwangwelcome to China :)03:34
shaohe_fengmaybe "accel:function_alias": "SmartNic_V1"03:34
shaohe_fengor other describe it.03:34
SundarYes, let's brainstorm on the function_id stuff. Perhaps I should create an etherpad for that.03:35
xinranwangSundar:  can you create a etherpad that we can add the idea on it.03:35
shaohe_fenga etherpad is good.03:35
SundarCool. Anything else, folks?03:35
shaohe_fenganother things, now the async job is in API.03:35
xinranwangAnd we can discuss them during the meeting03:35
shaohe_fengwe have issue, should be support async job in conductor or agent03:36
shaohe_fengpros and cons?03:36
SundarIMHO it should be in the conductor03:36
shaohe_fengOK, we can discuss and  improve it.  also put in etherpad03:37
*** licanwei has quit IRC03:37
SundarFrom the next meeting, we can start focusing more on the PTG.03:38
shaohe_fengalso another thing, a mini improve for factory.03:38
shaohe_feng^ #info link03:39
SundarDetails like this are probably better discussed as a patch proposal, rather than in etherpad, I think03:39
shaohe_fengthis is no in etherpad.03:39
shaohe_fengjust discuss it here03:39
shaohe_fengmini performance improvement03:40
shaohe_fengnow when I found it is a FPGA ARQ, I create a FPGA ARQ03:40
shaohe_fengthat means I access a DB again.03:40
shaohe_fengwe can cast base ARQ to FPGA ARQ to avoid the DB access03:41
Sundarshaohe_feng: Ok. Please feel free to propose a patch with your ideas.03:41
shaohe_fengdo you think is it worth to do it?03:42
shaohe_feng^ Sundar03:42
Sundarshaohe_feng: Yes. Not terribly sure about the casting across parent and child classes, though. That can be tricky.03:42
SundarTrying to avoid an extra db access makes sense.03:43
shaohe_fengjust a mini performance improvement.  and mini patch.03:43
shaohe_fengyou can see this example in the link03:43
shaohe_fengfrom math import pi03:44
shaohe_fengprint repr(c)03:44
shaohe_fengonly one line change:03:44
shaohe_fengc.__class__ = CirclePlus03:44
SundarPerhaps it is better to focus on more basic stuff, heading into U. We really need to focus on nova integ. One consistent feedback i hear is that CYborg code and patches tend to lack UT03:44
SundarWe completely lack functional testing with gabbi. These are not jazzy items but will help convince other developers and end users (operators) that we have good quality03:45
shaohe_fengIn cyborg: we just change one line the,  arqobj.__class__ = FPAGExaARQ03:45
shaohe_fengalso, we should list a task for testcase in the etherpad03:46
shaohe_fengtoo many testcase should be added03:46
shaohe_fengit is a huge task03:46
shaohe_fengHopeful everyone can take some testcase task.03:47
Sundarshaohe_feng: I like that. We should certainly spread the UT.03:48
SundarWhat do others think? Yumeng, chenke, zhurong, xinranwang03:48
SundarWe can split it into API, objects and db layer. By specific APIs (like device profile create), specific objects (e.g. ARQs, device profiles) and specific features (cross-service connections, etc.)03:50
Yumengyes agree,  it is urgent to improve UT and tempest case.03:51
SundarCool. We can follow up in the etherpad. Anything else today? Will wait for 30 seconds ...03:52
chenkenova integ and UT improve are the important things in U.03:52
SundarAgreed, along with the /v2/device,deployables API for listing inventory and programming03:53
SundarThanks everybody. Have a good week! Bye.03:54
*** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)"03:54
openstackMeeting ended Thu Oct 10 03:54:43 2019 UTC.  Information about MeetBot at . (v 0.1.4)03:54
openstackMinutes (text):
SundarAh, forgot to mention another priority: Cyborg client. :)03:55
*** Sundar has quit IRC03:56
s_shogoThanks, I'll continue the work for client and sdk in U release :)03:58
*** s_shogo has quit IRC03:58
*** shaohe_feng has quit IRC05:01
*** links has joined #openstack-cyborg05:47
*** chenke has quit IRC06:19
*** xinranwang has quit IRC07:07
*** FlorianFa has quit IRC07:24
*** Florian has joined #openstack-cyborg07:25
*** Florian has quit IRC09:19
*** FlorianFa has joined #openstack-cyborg09:19
*** Yumeng has quit IRC10:19
*** luyao has joined #openstack-cyborg11:07
*** openstackgerrit has joined #openstack-cyborg12:59
openstackgerritSundar Nadathur proposed openstack/cyborg master: Update docs: Part 1.
openstackgerritSundar Nadathur proposed openstack/cyborg master: Update docs: Part 2.
openstackgerritSundar Nadathur proposed openstack/cyborg master: Update docs: Part 3.
openstackgerritSundar Nadathur proposed openstack/cyborg master: Update docs: Part 1.
openstackgerritSundar Nadathur proposed openstack/cyborg master: Update docs: Part 2.
openstackgerritSundar Nadathur proposed openstack/cyborg master: Update docs: Part 3.
*** TxGirlGeek has joined #openstack-cyborg14:14
*** TxGirlGeek has quit IRC14:53
*** TxGirlGeek has joined #openstack-cyborg15:18
*** TxGirlGeek has quit IRC15:26
*** TxGirlGeek has joined #openstack-cyborg15:30
*** links has quit IRC16:01
*** efried is now known as efried_pto18:16
*** FlorianFa has quit IRC20:49
*** FlorianFa has joined #openstack-cyborg21:01
*** FlorianFa has quit IRC21:08
*** FlorianFa has joined #openstack-cyborg21:21
openstackgerritMerged openstack/cyborg master: Update master for stable/train
openstackgerritMerged openstack/cyborg master: Update driver section.
openstackgerritMerged openstack/cyborg master: API documentation update.
*** TxGirlGeek has quit IRC22:58

Generated by 2.15.3 by Marius Gedminas - find it at!