Earlier  
Later
New Search
Posted Nick Remark
#openstack-cyborg - 2020-04-16
04:04:43 shaohe_feng I agree with you.
04:04:47 brinzhang songwenping's just make this as a bug fix
04:04:52 Sundar brinzhang and all: first, is it an important issue to fix in Ussuri?
04:04:57 shaohe_feng and I have discuss with xinranwang for it.
04:05:28 shaohe_feng we are all same options with you.
04:06:24 shaohe_feng make cyborg db correctly firstly?
04:06:40 shaohe_feng brinzhang agree?
04:07:29 shaohe_feng or you can list some pros and cons to make the conclusion
04:07:49 xinranwang The root reason is that now we coupling placement and db update, and we can not avoid this dependency. IMHO, we should discuss the decoupling in V. Now, we just make sure that there is less risk to have data inconsistency.
04:08:31 chenke +1
04:08:32 brinzhang I am not sure whether is good for me, when would like to fix this issue? shaohe_feng
04:09:28 brinzhang xinranwang: agree +1
04:09:47 shaohe_feng OK， let's xinranwang to make a good design firstly
04:09:59 songwenping xinranwang: agree +1
04:10:06 Sundar xinranwang: are you voting to merge the patch now, and revisit it later?
04:10:34 brinzhang xinranwang's mean is to agree to merge this patch, then in V we should do Sundar suggestion, right?
04:10:40 shaohe_feng maybe a new spec for it firstly, and we can discuss it base on the new spec.
04:11:26 xinranwang I think we can merge it to let U have less risk of data inconsistency
04:12:13 Sundar all ok with that?
04:12:32 Yumeng agree +1
04:12:53 Sundar shaohe_feng, s_shogo ^
04:13:03 s_shogo agree, +1
04:13:15 chenke some advice inline .
04:13:25 chenke https://review.opendev.org/#/c/718584/9/cyborg/conductor/manager.py
04:13:26 brinzhang Yumeng, xinranwang: please add this to V release plan
04:13:27 xinranwang I have thought about the decoupling a bit, there's lots of things to discuss,several gaps as I mentioned in last meeting. We should think more and discusss more to have a solution of decoupling.
04:13:32 shaohe_feng I give up vote.
04:13:55 shaohe_feng stay neutral
04:14:05 Sundar Ok, great. Please mention this in commit message of this patch.
04:14:36 shaohe_feng if we merge it, please leave details note in the patch
04:14:47 brinzhang xinranwang: can you leave comments in this patch? that songwenping can update that
04:14:57 shaohe_feng such as FIXME or NOTE in it.
04:15:14 brinzhang That TODO(), I think
04:15:17 Sundar Anything else, folks? We are over the time.
04:15:19 xinranwang shaohe_feng: yes, understand, it is a hard decision, we have a long way to go lol...
04:15:30 xinranwang brinzhang: sure
04:15:47 brinzhang xinranwang: thanks
04:16:07 brinzhang it's time to launch, end meetting?
04:16:13 shaohe_feng OK.
04:16:28 shaohe_feng more drivers are welcome this new release right?
04:16:36 Sundar Great. Thanks a lot, everybody! Great progress this cycle. We have another week to make even more contributions. Take care and stay safe!
04:16:39 Sundar #endmeeting
04:16:41 openstack Meeting ended Thu Apr 16 04:16:39 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
04:16:43 openstack Minutes: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-16-03.02.html
04:16:44 openstack Minutes (text): http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-16-03.02.txt
04:16:45 openstack Log: http://eavesdrop.openstack.org/meetings/openstack_cyborg/2020/openstack_cyborg.2020-04-16-03.02.log.html
04:16:47 shaohe_feng ^ Sundar
04:17:27 Sundar shaohe_feng: Yes, newer drivers are always welcome, hopefully with CI support so that they can be maintained.
06:29:37 openstackgerrit YumengBao proposed openstack/cyborg master: Refactor v2 arq api https://review.opendev.org/696089
06:57:32 openstackgerrit Shogo Saito proposed openstack/cyborg master: Programming support (v2 Deployable API) https://review.opendev.org/698190
09:07:47 openstackgerrit Merged openstack/cyborg master: Refactor v2 arq api https://review.opendev.org/696089
10:35:20 openstackgerrit Wenping Song proposed openstack/cyborg master: revert device and deployable when resource provider create fail https://review.opendev.org/718584
11:54:38 openstackgerrit Wenping Song proposed openstack/cyborg master: revert device and deployable when resource provider create fail https://review.opendev.org/718584
12:38:16 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: Ascend driver:[B602:subprocess_popen_with_shell_equals_true] https://review.opendev.org/720456
12:42:59 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: Ascend driver:[B602:subprocess_popen_with_shell_equals_true] https://review.opendev.org/720456
12:43:56 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: [B104:hardcoded_bind_all_interfaces] https://review.opendev.org/720149
13:25:24 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: SPDK driver:[B602:subprocess_popen_with_shell_equals_true] https://review.opendev.org/720475
13:48:51 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: [B108:hardcoded_tmp_directory] https://review.opendev.org/720143
13:48:51 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: [B104:hardcoded_bind_all_interfaces] https://review.opendev.org/720149
13:48:52 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: Ascend driver:[B602:subprocess_popen_with_shell_equals_true] https://review.opendev.org/720456
13:48:52 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: SPDK driver:[B602:subprocess_popen_with_shell_equals_true] https://review.opendev.org/720475
13:48:53 openstackgerrit YumengBao proposed openstack/cyborg master: Change bandit job from non-voting to voting https://review.opendev.org/720479
#openstack-cyborg - 2020-04-17
00:10:44 openstackgerrit Brin Zhang proposed openstack/cyborg master: revert device and deployable when resource provider create fail https://review.opendev.org/718584
03:14:28 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: SPDK driver:[B602:subprocess_popen_with_shell_equals_true] https://review.opendev.org/720475
07:19:58 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: [B108:hardcoded_tmp_directory] https://review.opendev.org/720143
18:38:54 openstackgerrit Andreas Jaeger proposed openstack/python-cyborgclient master: Update docs building https://review.opendev.org/720804
18:39:25 openstackgerrit Andreas Jaeger proposed openstack/python-cyborgclient master: Update docs building https://review.opendev.org/720804
#openstack-cyborg - 2020-04-20
06:01:41 openstackgerrit Brin Zhang proposed openstack/cyborg master: hacking: force explicit import of python's mock https://review.opendev.org/716920
#openstack-cyborg - 2020-04-21
11:51:19 openstackgerrit YumengBao proposed openstack/cyborg master: Fix bandit error: [B108:hardcoded_tmp_directory] https://review.opendev.org/720143
11:56:25 openstackgerrit YumengBao proposed openstack/cyborg master: Change bandit job from non-voting to voting https://review.opendev.org/720479
12:04:33 openstackgerrit YumengBao proposed openstack/cyborg master: Change bandit job from non-voting to voting https://review.opendev.org/720479
#openstack-cyborg - 2020-04-22
10:26:03 openstackgerrit Brin Zhang proposed openstack/cyborg master: revert device and deployable when resource provider create fail https://review.opendev.org/718584
11:43:00 openstackgerrit Merged openstack/cyborg master: revert device and deployable when resource provider create fail https://review.opendev.org/718584
#openstack-cyborg - 2020-04-23
03:01:44 Sundar Hi all
03:01:58 Sundar #startmeeting openstack-cyborg
03:01:59 openstack Meeting started Thu Apr 23 03:01:58 2020 UTC and is due to finish in 60 minutes. The chair is Sundar. Information about MeetBot at http://wiki.debian.org/MeetBot.
03:02:00 openstack Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
03:02:02 openstack The meeting name has been set to 'openstack_cyborg'
03:02:09 s_shogo Hi all
03:02:13 songwenping_ Hi all
03:02:28 xinranwang Hi all
03:03:21 Yumeng #info Yumeng
03:03:24 brinzhang hi all
03:03:28 Yumeng hi all
03:03:40 Sundar Good, we have a quorum.
03:04:04 Sundar Does anybody have a specific thing to discuss?
03:04:09 Sundar *any
03:04:10 chenke hi all
03:04:13 chenke #info chenke
03:04:23 xinranwang #info xinranwang
03:04:40 s_shogo #info s_shogo
03:05:08 brinzhang #info brinzhang
03:06:04 Yumeng hi all, pls help to review and merge https://review.opendev.org/#/q/topic:fix-bandit-check-failures+(status:open+OR+status:merged)
03:06:07 brinzhang I have some summary work in my company, so there few work take here in this meeting, I am sorry
03:06:38 brinzhang Yumeng: I reviewed the first patch, I hope you can add the test case to cover your changes
03:06:43 Sundar Looking at https://review.opendev.org/#/q/status:open+project:openstack/cyborg+branch:master
03:06:48 Yumeng brinzhang: I just replied your comments: https://review.opendev.org/#/c/720143/4/cyborg/agent/manager.py
03:06:55 Sundar Agree with Yumeng -- bandit fixes are important
03:07:22 Sundar I'll review them after this meeting
03:07:30 brinzhang Yumeng: I mean, you should add test case to test https://review.opendev.org/#/c/720143/4/cyborg/agent/manager.py@52

Earlier   Later