Thursday, 2020-04-09

*** brinzhang_ has quit IRC00:01
*** brinzhang has joined #openstack-cyborg00:11
brinzhanggmann: I would like to add the contributing from https://review.opendev.org/#/c/712246/5/doc/source/contributor/contributing.rst to common contribution docs, but I found there already exist, but it is hard to find,00:41
brinzhanggmann: and it is not on the same page as the use of git, which makes me very uncomfortable.00:41
brinzhangusing git page https://docs.openstack.org/contributors/common/git.html#id100:42
brinzhangusing gerrit paage https://docs.openstack.org/contributors/common/setup-gerrit.html#what-is-gerrit00:42
openstackgerritWenping Song proposed openstack/cyborg master: revert device and deployable when resource provider create fail  https://review.opendev.org/71858400:42
brinzhangIMO, they are should in one page, or need some super link00:43
*** xinranwang has joined #openstack-cyborg02:02
openstackgerritYumengBao proposed openstack/cyborg master: Refactor v2 arq api  https://review.opendev.org/69608902:52
*** Yumeng has joined #openstack-cyborg02:55
*** s_shogo has joined #openstack-cyborg02:59
Yumenghi xinranwang,xinranwang and Li_Liu03:03
xinranwangHi03:03
Yumenghi hi.03:03
Yumenghi Li_Liu03:03
Yumenghi brinzhang03:04
s_shogoHi all03:04
brinzhangHi Yumeng and all, today I have some need to do right now, I am sorry cannot participate in today's meetting03:05
*** chenke has joined #openstack-cyborg03:05
Yumengok. no worries. let's get started03:05
Yumeng#startmeeting openstack-cyborg03:05
openstackMeeting started Thu Apr  9 03:05:59 2020 UTC and is due to finish in 60 minutes.  The chair is Yumeng. Information about MeetBot at http://wiki.debian.org/MeetBot.03:06
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.03:06
chenkeHi all. sorry for later.03:06
*** openstack changes topic to " (Meeting topic: openstack-cyborg)"03:06
openstackThe meeting name has been set to 'openstack_cyborg'03:06
chenke#info chenke03:06
xinranwang#info xinranwang03:06
Yumeng#topic Roll call03:06
*** openstack changes topic to "Roll call (Meeting topic: openstack-cyborg)"03:06
Yumeng#info Yumeng03:06
s_shogo#info s_shogo03:06
YumengI suppose the agenda for today is reviewing pending patches03:07
Yumenglet's go through by priorities from high03:08
xinranwangThe microverison patch has been merged, thank you all for reviewing :)03:08
chenkegood job.03:08
Yumenggreat! Thanks Xinran for the efforts ! and v2 doc need review https://review.opendev.org/#/c/715980/03:09
xinranwangYes, please review this doc patch as well. thanks Yumeng03:10
Yumengs_shogo and xinranwang, do you have any other patches need review?03:11
xinranwangNo other patches from m03:11
xinranwang*me03:12
Yumengok03:12
openstackgerritYumengBao proposed openstack/cyborg master: Refactor v2 arq api  https://review.opendev.org/69608903:12
s_shogoThis one. I'm working with Sundar's comment. https://review.opendev.org/#/c/698190/03:12
chenkeOk. will reivew it.03:13
s_shogoShaohe may have opinion about this programming patch, as the previous meeting : I can ask him in wechat.03:14
s_shogothank you, chenke03:14
YumengI notice shaohe also has several patches realted to programming https://review.opendev.org/#/q/owner:shaohe.feng%2540intel.com+status:open03:14
Yumengare they related to yours? s_shogo03:15
s_shogoAt a glance, It seems to be driver layer, my patch provides api.03:16
s_shogoI'll investigate and check them after this meeting, thanks.03:16
Yumengok. seems different. I will ask him if need merged in this release.03:16
s_shogook.03:17
Yumengso there also left Brin's patches https://review.opendev.org/#/q/owner:zhangbailin%2540inspur.com+status:open03:17
Yumengand mine two: arq refactor https://review.opendev.org/#/c/696089/, and add policy role https://review.opendev.org/#/c/699102/03:18
Yumengplease help to review! not big change.03:19
Yumengchenke: do you have any pending patches?03:20
chenkeoh. no patch for me. But a patch needs us wo notice.03:20
xinranwangOk, will review them.03:20
chenkehttps://review.opendev.org/#/c/718584/1/cyborg/conductor/manager.py03:20
chenkeThis patch seems help cyborg to sync data with placement. We should consider whether it's needed or not.03:21
xinranwangYes, I just saw this.03:23
xinranwangIMO, it is reasonable.03:24
Yumeng+1 agree with Brin's comment.  xinranwang: do we plan to decouple placement report in next release?03:25
chenkeIt looks okay, but needs further analysis.03:26
xinranwangI think it depends on users' demand, if we want cyborg run as a standalone project, yes, we should decouple them03:27
chenkeIt looks okay, but needs further analysis. To be honest, the current piece of code looks a bit redundant. About placement reporting and cyborg database update are mixed together. But I have no better idea about decoupling them.03:29
xinranwangchenke:  yes, that's what I am thinking too.  If we want decouple them, it will generate more redundant code, because both of them need to do the diff.03:30
chenkeYe. seems no better code to make report and sync cyborg db.  Let's these code stay as it is.03:32
Yumengemmm,seems a problem to choose a less evil one.  let's keep it as one topic of the PTG.03:32
Yumeng#topic AoB03:33
*** openstack changes topic to "AoB (Meeting topic: openstack-cyborg)"03:33
YumengAnything else?03:33
brinzhanghttps://review.opendev.org/#/c/718584/1/cyborg/conductor/manager.py are you talking this patch?03:33
Yumengyes brinzhang03:33
xinranwangYumeng:  Sure, btw, do we have ptg etherpad where we can put the topic in, I'd like add some :)03:34
brinzhangI dont think we should keep current logical, if so, maybe when we do next cycle to add a device, we cannot get the device occupied in obj03:34
chenkeHi brin.  "we cannot get the device occupied in obj" means what?03:35
Yumengxinranwang: ok, will let you know later. I will discuss with Sundar, and maybe we can create one!03:35
xinranwangchenke:  I think this patch  does solve problems, it does not change the main logic, just revert the cyborg DB. I think it is useful.03:36
chenkeAgree.03:36
brinzhangIMHO, the logic of the current code in this file makes it complicated to read. If possible, I hope to be able to refactor it.03:36
xinranwangYumeng:  thanks´╝ü03:36
brinzhangxinranwang, right, it just resolve the problem, not change any logical03:36
chenkebrinzhang. Agree. If we have better solution. I want to talk about it.03:37
Yumengbrinzhang: yes. let's discuss it more detaily.03:37
brinzhangYumeng: We can take https://review.opendev.org/#/q/status:open+project:openstack/cyborg+branch:master+topic:unittest.mock to PTG03:37
xinranwangbrinzhang:  Yes, we are trying to find a way to decouple the code. There is still some gaps. We'd better let it be like it is(the main logic) in this release, and we can discuss  in PTG.03:38
brinzhangSundar think we cannot do this right now, but whatever we shuold do sooner or later03:38
brinzhangyes xinranwang, https://review.opendev.org/#/c/718584 isnot change it's logical, we can looked into later03:39
Yumengok. I will paste the PTG etherpad later. let's add topics.03:39
brinzhanghttps://review.opendev.org/#/c/717519/.03:39
brinzhanghttps://review.opendev.org/#/c/717520/03:39
brinzhangabove two patches, I think we should merge, because of droped py27 from Cyborg03:40
brinzhangmany projects are done of this cleanup03:40
xinranwangAnother thing: There is still some operations are not supported in nova like shelve/unshelve. I'd like to investigate it and do this part in next release if possible.  I will add this topic to PTG etherpad too.03:41
YumengGreat! Thanks Xinran!03:41
Yumengwill review them Brin.03:41
Yumengif nothing else. Let's wrap up this meeting and I'll see you all again next week03:41
YumengThank you all. have a good day!03:42
xinranwangThanks Yumeng03:42
xinranwangBye bye all03:42
YumengBye03:42
Yumeng#endmeeting03:42
*** openstack changes topic to "Pending patches (Meeting topic: openstack-cyborg)"03:42
openstackMeeting ended Thu Apr  9 03:42:42 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)03:42
openstackMinutes:        http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-09-03.05.html03:42
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-09-03.05.txt03:42
openstackLog:            http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-09-03.05.log.html03:42
s_shogobye03:42
*** s_shogo has quit IRC03:42
*** brinzhang_ has joined #openstack-cyborg04:14
*** brinzhang has quit IRC04:17
*** brinzhang has joined #openstack-cyborg04:43
*** brinzhang_ has quit IRC04:46
*** brinzhang_ has joined #openstack-cyborg04:49
*** brinzhang has quit IRC04:53
*** brinzhang has joined #openstack-cyborg04:54
*** brinzhang_ has quit IRC04:57
*** brinzhang_ has joined #openstack-cyborg05:13
*** brinzhang_ has quit IRC05:15
*** brinzhang_ has joined #openstack-cyborg05:15
*** brinzhang has quit IRC05:16
*** brinzhang has joined #openstack-cyborg05:20
*** brinzhang_ has quit IRC05:23
*** brinzhang_ has joined #openstack-cyborg05:32
*** brinzhang has quit IRC05:36
*** brinzhang has joined #openstack-cyborg05:49
*** brinzhang has quit IRC05:51
*** brinzhang has joined #openstack-cyborg05:51
*** brinzhang_ has quit IRC05:52
*** chenke has quit IRC05:54
*** brinzhang_ has joined #openstack-cyborg07:13
*** brinzhang has quit IRC07:16
*** Yumeng has quit IRC07:20
openstackgerritBrin Zhang proposed openstack/cyborg master: Bump openstacksdk 0.46.0  https://review.opendev.org/71860607:27
openstackgerritMerged openstack/cyborg-tempest-plugin master: Update to hacking 3.0  https://review.opendev.org/71751907:30
openstackgerritBrin Zhang proposed openstack/cyborg master: revert device and deployable when resource provider create fail  https://review.opendev.org/71858407:31
openstackgerritBrin Zhang proposed openstack/cyborg master: Refactor v2 arq api  https://review.opendev.org/69608907:31
*** jraju__ has joined #openstack-cyborg07:35
openstackgerritBrin Zhang proposed openstack/cyborg master: Bump openstacksdk>=0.46.0  https://review.opendev.org/71860607:39
*** brinzhang has joined #openstack-cyborg07:51
*** brinzhang_ has quit IRC07:54
openstackgerritBrin Zhang proposed openstack/cyborg master: Bump openstacksdk>=0.46.0  https://review.opendev.org/71860608:12
openstackgerritBrin Zhang proposed openstack/cyborg master: Refactor v2 arq api  https://review.opendev.org/69608908:13
openstackgerritBrin Zhang proposed openstack/cyborg master: revert device and deployable when resource provider create fail  https://review.opendev.org/71858408:14
*** brinzhang_ has joined #openstack-cyborg08:49
*** brinzhang has quit IRC08:52
*** brinzhang has joined #openstack-cyborg08:58
*** brinzhang_ has quit IRC09:01
openstackgerritWenping Song proposed openstack/cyborg master: revert device and deployable when resource provider create fail  https://review.opendev.org/71858409:30
*** brinzhang_ has joined #openstack-cyborg09:59
*** brinzhang has quit IRC10:02
openstackgerritWenping Song proposed openstack/cyborg master: revert device and deployable when resource provider create fail  https://review.opendev.org/71858410:06
openstackgerritYumengBao proposed openstack/cyborg master: Bump openstacksdk>=0.46.0  https://review.opendev.org/71860610:32
openstackgerritYumengBao proposed openstack/cyborg master: Refactor v2 arq api  https://review.opendev.org/69608910:32
openstackgerritYumengBao proposed openstack/cyborg master: Bump openstacksdk>=0.46.0  https://review.opendev.org/71860610:41
openstackgerritYumengBao proposed openstack/cyborg master: revert device and deployable when resource provider create fail  https://review.opendev.org/71858410:41
openstackgerritYumengBao proposed openstack/cyborg master: Refactor v2 arq api  https://review.opendev.org/69608910:42
openstackgerritWenping Song proposed openstack/cyborg master: sync data for placement  https://review.opendev.org/71866211:07
*** igordc has quit IRC12:18
*** xinranwang has quit IRC12:30
*** brinzhang has joined #openstack-cyborg16:57
*** brinzhang_ has quit IRC17:00
*** jraju__ has quit IRC17:15
openstackgerritAndreas Jaeger proposed openstack/cyborg-specs master: Cleanup py27 support and docs  https://review.opendev.org/71878319:47

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