Monday, 2018-02-12

*** Qiming has quit IRC01:09
*** Qiming has joined #senlin01:13
*** chenyb4 has joined #senlin01:18
*** liyi has joined #senlin01:20
*** pzchen has joined #senlin01:20
*** liyi has quit IRC01:30
*** liyi has joined #senlin01:40
*** liyi has quit IRC01:44
openstackgerritchenyb4 proposed openstack/senlin master: Fix health manager event listener policy detach error  https://review.openstack.org/52189202:05
openstackgerritQiming Teng proposed openstack/senlin master: Turn py27 gate on  https://review.openstack.org/54315602:07
chenyb4hi, Qiming For this patch, I suggest that there is a bug fix before ruijie is not optimized.  https://review.openstack.org/52189202:07
QimingThis English sentence make completely no sense02:20
QimingWhat is your suggestion?02:20
QimingWhat bug fix you are referring to?02:20
QimingWhat is to be not optimized?02:21
Qiming"A bug fix before ruijie"?02:21
chenyb4The health manager event listener has stopped errors.When we detach event listener or stop engine service, the listener does not stop.02:28
chenyb4This bug is always there.02:29
QimingSo you mean, there is a bug02:30
QimingThat bug happens when health manager stops a listener02:30
QimingListeners cannot be successfully stopped02:30
QimingRuijie posted comments that your fix is not correct02:31
chenyb4When too many clusters add health event policies, more event listeners will be enabled. Ruijie considers all listening to share a listener event listener, but it is not implemented at this time.02:31
QimingI believe he tried, just hasn't completed it yet?02:31
chenyb4yes02:32
QimingCreating and destroying listeners are not trivial operations02:32
QimingIf there is clearly a right way to get things done, how about we solve the problem using the correct solution?02:33
chenyb4in this patch, Ruijie finished part of the work. https://review.openstack.org/#/c/529457/02:37
chenyb4This bug is very dangerous.02:40
chenyb4So, 1. We can continue to finish the rest of ruijie's work.02:40
chenyb42. Solve the bug with this patch first. https://review.openstack.org/52189202:41
chenyb4Ruijie seems to have done most of the work.02:47
chenyb4https://review.openstack.org/#/c/529235/ https://review.openstack.org/#/c/529279/  https://review.openstack.org/#/c/529457/02:47
QimingYes, chenyb4 I'm aware of the thread Ruijie has been working on, and that is the reason I was asking why we are coming up with a shortcut solution now03:18
QimingIdeally, we should finish that thread03:18
*** liyi has joined #senlin03:19
chenyb4Yes, I will take the time to look at ruijie commit in detail. The completion of the remaining part will take time and it will take good test.03:22
chenyb4Ceilometer notification data acquisition can give me some advice.03:23
*** liyi has quit IRC03:24
Qimingyup03:25
QimingBut I really don't think we should solve this problem using a premature solution03:25
openstackgerritchenpengzi proposed openstack/senlin master: Update mysql connection  https://review.openstack.org/54333303:45
openstackgerritchenyb4 proposed openstack/python-senlinclient master: Change event and node module describe error.  https://review.openstack.org/54333604:04
*** liyi has joined #senlin04:19
*** jmlowe has joined #senlin04:20
*** jmlowe_ has quit IRC04:22
*** liyi has quit IRC04:24
openstackgerritchenyb4 proposed openstack/python-senlinclient master: Change event and node module describe error.  https://review.openstack.org/54333605:03
Qimingchenyb4, http://logs.openstack.org/56/543156/3/check/senlin-dsvm-tempest-py27-api/7c06f39/logs/screen-sl-eng.txt.gz#_Feb_12_02_38_09_25343205:07
QimingPatch #542076 has been breaking py27 api test05:08
chenyb4 I see05:08
Qiminghttp://logs.openstack.org/76/542076/3/check/senlin-dsvm-tempest-py27-api/27adb9c/logs/screen-sl-eng.txt.gz#_Feb_09_16_01_57_48256105:09
QimingIt has been there when the #542076 patch was checked05:10
chenyb4On February 8, was the merge this patch.  https://review.openstack.org/#/c/542076/05:17
chenyb4hi, Qiming  https://review.openstack.org/#/c/542692/ this patch05:19
chenyb4Qiming, scaling policy miss VERSIONS, maybe05:19
QimingOkay05:19
QimingMerging #54269205:20
QimingIt has been there for several days05:20
QimingI didn't check the details, please help keep an eye on it05:21
chenyb4Ok05:23
openstackgerritblkart proposed openstack/python-senlinclient master: Retrieve profile type by type name and version  https://review.openstack.org/54081005:31
*** liyi has joined #senlin05:38
*** liyi has quit IRC05:42
openstackgerritMerged openstack/senlin master: Add scaling policy version support message  https://review.openstack.org/54269205:52
openstackgerritMerged openstack/senlin master: Update mysql connection  https://review.openstack.org/54333305:52
*** liyi has joined #senlin06:48
*** liyi has quit IRC06:52
chenyb4https://review.openstack.org/#/c/543156/ senlin-dsvm-tempest-py27-api check success07:19
chenyb4http://logs.openstack.org/56/543156/3/check/senlin-dsvm-tempest-py35-integration/465f3bc/logs/screen-sl-eng.txt.gz#_Feb_12_06_27_09_444772  It seems to be related to the zaqar message release.07:37
QimingYes, both integration tests are failing due to this problem07:39
QimingThe weird thing is .... Zaqar functional test has been skipped07:41
openstackgerritQiming Teng proposed openstack/senlin master: Turn py27 API gate on  https://review.openstack.org/54315607:47
QimingLooks like we have to postpone turning on the integration tests again07:47
*** AlexeyAbashkin has joined #senlin07:51
openstackgerritQiming Teng proposed openstack/senlin-tempest-plugin master: Skip integration tests for lifecycle hooks  https://review.openstack.org/54336807:52
QimingLet's see if the above patch will make the integration gate green again.07:53
Qiming^07:53
chenyb4ok08:00
openstackgerritQiming Teng proposed openstack/senlin master: Last release note for Queens  https://review.openstack.org/54337208:08
openstackgerritMerged openstack/senlin master: Fix a https url issue  https://review.openstack.org/54252708:26
openstackgerritMerged openstack/senlin master: Fix grammar error  https://review.openstack.org/54316908:26
openstackgerritMerged openstack/senlin master: remove no use define and spell error  https://review.openstack.org/54319808:29
*** liyi has joined #senlin08:56
openstackgerritQi Peng proposed openstack/python-senlinclient master: Remove unnecessary definition  https://review.openstack.org/54338509:00
*** liyi has quit IRC09:01
*** chenyb4 has quit IRC10:06
*** pzchen has quit IRC10:08
*** liyi has joined #senlin10:16
*** liyi has quit IRC10:20
openstackgerritMerged openstack/senlin master: Turn py27 API gate on  https://review.openstack.org/54315610:40
*** liyi has joined #senlin11:15
*** liyi has quit IRC11:20
*** jmlowe has quit IRC12:27
openstackgerritMerged openstack/senlin master: Last release note for Queens  https://review.openstack.org/54337212:37
*** jmlowe has joined #senlin15:23
*** jmlowe has quit IRC15:28
*** jmlowe has joined #senlin15:30
*** AlexeyAbashkin has quit IRC17:36
*** jmlowe has quit IRC18:44
*** jmlowe has joined #senlin19:04
*** jmlowe has quit IRC19:06
*** jmlowe has joined #senlin19:06
*** jmlowe has quit IRC19:08
*** jmlowe has joined #senlin19:09
*** jmlowe has quit IRC19:15
*** jmlowe has joined #senlin19:22
*** jmlowe has quit IRC19:38
*** jmlowe has joined #senlin19:41
*** JudeC has joined #senlin20:27
*** jmlowe has quit IRC22:14
*** AlexeyAbashkin has joined #senlin23:21
*** JudeC has quit IRC23:26
*** AlexeyAbashkin has quit IRC23:26
*** JudeC has joined #senlin23:27
*** JudeC has quit IRC23:27
*** JudeC has joined #senlin23:28
*** JudeC has quit IRC23:30

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