Thursday, 2015-10-15

openstackgerritxu-haiwei proposed stackforge/python-senlinclient: Fix help message for cluster_scale_in/out  https://review.openstack.org/23473700:05
openstackgerritxu-haiwei proposed stackforge/senlin: Make 'grace_period' and 'destroy_after_deletion' work  https://review.openstack.org/23473600:16
*** pm90_ has joined #senlin00:49
*** lawrancejing has joined #senlin00:54
*** zhenguo has joined #senlin01:14
*** jdandrea has quit IRC01:28
*** Yanyanhu has joined #senlin01:33
*** lixinhui has joined #senlin01:42
*** lixinhui has quit IRC01:43
*** elynn has joined #senlin01:51
elynnmorning :)01:55
Qimingmorning02:02
Yanyanhumorning02:02
*** Shijia has joined #senlin02:05
*** Shijia has quit IRC02:07
*** Shijia has joined #senlin02:08
*** lixinhui has joined #senlin02:14
*** Shijia_ has joined #senlin02:17
*** Shijia has quit IRC02:17
*** Shijia has joined #senlin02:24
*** Shijia_ has quit IRC02:24
*** Shijia_ has joined #senlin02:32
*** Shijia has quit IRC02:32
*** Qiming has quit IRC02:45
*** Qiming has joined #senlin02:45
openstackgerritYanyan Hu proposed stackforge/senlin: Make all cluster DB queries project_safe  https://review.openstack.org/23468302:56
elynnI try to use heat to create cluster and profile and it worked, and I'm using this template https://github.com/lynic/templates/blob/master/senlin/template/cp.yaml02:59
*** shu has joined #senlin03:12
*** Qiming has quit IRC03:12
*** Qiming has joined #senlin03:12
xuhaiweielynn, so the senlin resource is landed?03:13
xuhaiweior you are just testing it?03:13
Qimingxuhaiwei, IIUC, elynn is helping do a proof of concept and see it things can land smoothly03:16
*** Shijia has joined #senlin03:21
*** Shijia_ has quit IRC03:21
*** Liu has joined #senlin03:27
elynnI'm testing on it03:36
elynnbut still have some questions about profile properties.03:38
*** pm90_ has quit IRC03:59
*** Liu has quit IRC04:03
*** pm90_ has joined #senlin04:06
*** pm90_ has quit IRC04:26
openstackgerritQiming Teng proposed stackforge/senlin: TODO item for actions API  https://review.openstack.org/23508405:11
Qiminganyone please leave comments to the big tent proposal if any05:12
QimingI'm gonna submit it today to the TC governance for review05:13
Qimingthanks05:13
*** pm90_ has joined #senlin05:17
*** Shijia has quit IRC05:30
*** Liu has joined #senlin05:32
*** lawrancejing has quit IRC05:49
*** lawrancejing has joined #senlin05:50
*** Qiming has quit IRC06:00
*** Qiming has joined #senlin06:10
*** Liu has quit IRC06:34
*** Liu has joined #senlin06:49
*** Liu has quit IRC07:17
openstackgerritYanyan Hu proposed stackforge/senlin: Remove two useless DB API interfaces  https://review.openstack.org/23512207:17
*** Qiming has quit IRC07:20
*** Qiming has joined #senlin07:20
*** Yanyan has joined #senlin07:30
*** Yanyanhu has quit IRC07:33
*** Liu has joined #senlin07:34
openstackgerritMerged stackforge/senlin: TODO item for actions API  https://review.openstack.org/23508407:45
*** Shijia has joined #senlin07:50
openstackgerritYanyan Hu proposed stackforge/senlin: Make all cluster DB queries project_safe  https://review.openstack.org/23468307:50
openstackgerritQiming Teng proposed stackforge/senlin: Temporary proposal for big tent adoption  https://review.openstack.org/23416208:03
Qimingzhenguo, there?08:04
zhenguoQiming: yeah08:04
Qimingduring the past weekly IRC meeting, we conducted a PTL election08:04
QimingI myself volunteered and got approval by the team08:05
Qimingone thing I'd like to discuss with you (also it was raised in the meeting) is about senlin-dashboard08:06
zhenguoQiming: oh, definitely08:06
Qimingdo we need a PTL for that08:06
openstackgerritMerged stackforge/senlin: Remove two useless DB API interfaces  https://review.openstack.org/23512208:06
zhenguoit's a sub project of senlin08:06
QimingI myself knows almost nothing about dashboard08:06
QimingI appreciate your kindness adding me to the core team, so that I can review patches from service interaction's perspective08:07
Qimingsince you are more experienced than I am in web development, and in how community organizes dashboard projects08:08
zhenguoIMO, it's almost like python-senlinclient, as a sub project to senlin. so seems you PTL should lead all these projects.08:08
QimingI really want to hear your suggestions08:08
QimingI'd love to do anything to the dashboard08:08
Qimingbut I'm really not capable of making the right decisions08:09
zhenguoyeah, you can give suggestions and requirements is enough I think.08:09
zhenguoso, how's the dashboard project going now, sorry for not do anything these days.08:10
Qimingif it is okay with you, and we are doing this following the community practice, let's do it08:10
QimingI'm seeing lawrancejing a hard worker recently08:10
zhenguoyeah, I think it's common in community08:11
Qimingcluster, node, profile operations are all being added and tuned08:11
lawrancejing:)08:11
zhenguocool, thanks lawrancejing for that08:11
Qimingmaybe we can do this08:11
Qimingwe already have senlin-dashboard-cores group?08:12
zhenguoyeah, in Tuskar, Magnum, Manila, we all have separate dashboard-core groups.08:13
Qimingwe can add senlin cores to senlin-dashbard-core group08:13
zhenguoyeah.08:13
Qimingand vice versa08:13
Qimingthat way we can accelerate code reviews08:13
zhenguoyeah, and maybe we can add horizon-cores as well.08:14
Qimingokay, so the suggestion is just keep the senlin-core and senlin-dashboard-core separate08:15
Qimingrecruit new members based on people's reviews and contributions08:15
Qimingadding horizon-core to senlin-dashboard-core is good08:16
zhenguoyeah08:16
Qimingthese two teams share more expertise in common08:16
Qimingumm08:17
*** lawrancejing has quit IRC08:17
*** lawrancejing has joined #senlin08:18
openstackgerritQiming Teng proposed stackforge/senlin: Temporary proposal for big tent adoption  https://review.openstack.org/23416208:20
Qimingxuhaiwei, and Yanyan08:21
Yanyanhi08:21
xuhaiweiQiming, liverbty release is today, are we lucky  enough to join OpenStack in Liverbty08:21
Qimingwhen I was drafting the big tent proposal, I was reading the proposal from kolla08:21
xuhaiweiyes?08:21
Qimingxuhaiwei, that will not happen, :(08:22
xuhaiweiI am also reading the patch of kolla08:22
Yanyanme too, actually :)08:22
Qimingone of the things we were not doing well is about following the +2/+A rules08:22
xuhaiweisometime08:23
Qimingthe rule suggests that we need to have 2 +2s and 1 +A for patch to be merged08:23
Yanyanyes, for some patches08:23
Qimingsometimes, you know, we were breaking that rule08:23
QimingI myself have been doing that a lot08:24
Yanyanyes, in cases the patch need fast approval08:24
* Qiming kicks himself08:24
xuhaiweisometime I just can't understand the patch, so didn't do anything about it08:24
Yanyannot your fault I believe... be cause we really don't have enough bandwidth to cover both developing and review sometimes08:24
Qimingwe have execuses: patch dependency, core team too small, urgency to fix a problem ...08:24
Qimingbut they are still not good practices08:25
Qimingso .. on one hand, we need to do more reviews08:25
Yanyanyes, definitely08:26
xuhaiweithat might be a reason to block us?08:26
Qimingon the other hand, please help recommend contributors to the team08:26
Qimingno, it won't08:26
Qimingthere are proposals for projects that don't have a single line of code, :D08:26
Yanyansure, hope more people can join us08:27
xuhaiweimaybe sometime we should not merge patch so quick, just wait for some new guys to review08:27
Qimingin any community work, unless it is your full time job, people join and leave08:28
Qimingthat is common08:28
Yanyanyea08:28
Qimingso maybe we can lower the bar for people to become a core reviewer08:28
Qiminglet's keep our eyes open, :)08:29
zhenguoQiming: I have included senlin-core to senlin-dashboard-core groups.08:29
Qimingthanks zhenguo08:29
zhenguo:)08:30
xuhaiweisome projects have a rule to become a core member, like review more than 3 patches a day during about 3 months08:30
Qimingxuhaiwei, yes, that is a high bar for senlin, :)08:30
xuhaiweimaybe we can make it 2 reviews for one month, something like that08:31
Qiminghoho08:31
Qimingall projects are watched: http://git.openstack.org/cgit/openstack/governance/tree/tools/teamstats.py08:32
xuhaiweianyway we want peopleo who can really help senlin08:32
Qimingyes08:32
Yanyanany contribution is welcome :)08:32
Qimingline 139 in the above file is about the statistics they use to evaluate the diversity of a project08:32
xuhaiweibut sometime people worked hard to be a core, but just stop working after being one08:33
Qimingxuhaiwei, that is what I was saying: people join and leave08:33
Qimingif it is not his/her full time job08:34
YanyanI guess sometimes this is caused by stupid evaluation rules enforced by some companies08:34
xuhaiweiyou are right :)08:34
Yanyangetting support from boss to allow you do community job fulltime is not a easy task :)08:34
Qimingright, contributions to a stackforge project? are you mad?08:34
Yanyanso we are lucky to get chance to work here :P08:35
xuhaiweiyap08:35
Qimingoh, another change to the proposal08:36
QimingI put this into the big tent proposal: Team plans to excercise a PTL rotation among core contributors.08:37
Qiminghope you won't mind, :)08:37
xuhaiweithat is common08:37
Qimingxuhaiwei, next cycle could be you08:38
Qimingcould be mr. newcomer08:39
Yanyansure, although we are still not ready now you know :)08:39
xuhaiweiyes, that is a big change08:39
Qimingyou will08:39
Yanyanyea, I believe we will some day :)08:39
Qiminggreat!08:40
Qimingcloning openstack/governance08:43
Yanyancool08:43
Qiminglet's have a try08:43
Yanyanyep08:43
Yanyan\o/08:44
Qimingbe prepared though, it's near summit time08:44
Qimingso the review may take time, some projects have cancelled weekly meeting08:44
QimingI am proposing it NOW because the Mitaka cycle has started08:45
Qimingwe want to be on board08:45
Yanyanwe are ready for a fight I think08:45
xuhaiweifight what?08:48
xuhaiwei:)08:48
Yanyanxuhaiwei, fight will those doubt and criticism :)08:53
Yanyanwill/with08:53
xuhaiweiI found set_status method of cluster and node is often missed here and there, especially when action fails08:57
xuhaiweiI am trying to fix it08:57
Qimingxuhaiwei, yes, I found that as well08:58
Qimingwhen a node fails creation, the status wasn't correct set, for example08:58
Qimingit would be nice if you can help cleanse it08:58
xuhaiweiI started with cluster action08:59
Qimingone reminder is that you need to start from cluster_action's execute() method, not individual action handlers08:59
xuhaiweiI have just finished the individual action handlers :(((09:00
Qiming...09:01
xuhaiweiexecute() and _execute() are no need to fix I think09:01
Qimingthen you learned something09:01
Qiminghttps://review.openstack.org/23517209:01
Yanyano/09:07
Yanyanhi, Qiming, ':' is missing in line280109:09
Qimingwhat?09:10
xuhaiweithe proposal patch09:12
YanyanQiming, the patch check failed for nit error09:13
*** branw has joined #senlin09:15
Qimingah I see09:15
QimingI noticed that line was displayed in a different color when being edited09:16
Qimingdidn't realize what kind of an error it is09:16
Qiming:)09:16
Yanyan;)09:17
*** branw has quit IRC09:17
*** branw has joined #senlin09:17
*** Qiming has quit IRC09:22
openstackgerritYanyan Hu proposed stackforge/senlin: Make all cluster DB queries project_safe  https://review.openstack.org/23468309:22
openstackgerritYanyan Hu proposed stackforge/senlin: [WIP]Make node related DB queries project safe  https://review.openstack.org/23518209:22
openstackgerritYanyan Hu proposed stackforge/senlin: [WIP]Make node related DB queries project safe  https://review.openstack.org/23518209:24
*** Qiming has joined #senlin09:26
*** branw_ has joined #senlin09:32
*** branw has quit IRC09:36
*** Liu has quit IRC09:36
*** Yanyan has quit IRC09:55
*** zhenguo has quit IRC10:09
*** lixinhui has quit IRC10:10
*** lixinhui has joined #senlin10:10
lawrancejingHello Qiming, zhengguo, I will continue to work on  senlin-dashboard, until most work done, include 70 ~ 90% unit / functional test10:11
Qiminglawrancejing, thanks!10:11
*** lawrancejing has quit IRC10:12
*** branw__ has joined #senlin10:32
*** branw_ has quit IRC10:32
*** pm90_ has quit IRC10:39
*** lawrancejing has joined #senlin10:43
*** branw_ has joined #senlin10:47
*** branw__ has quit IRC10:51
*** elynn has quit IRC11:08
*** branw__ has joined #senlin12:02
*** branw_ has quit IRC12:02
openstackgerritlawrancejing proposed stackforge/senlin-dashboard: Add profile create test  https://review.openstack.org/23528612:13
*** branw_ has joined #senlin12:17
*** branw__ has quit IRC12:21
openstackgerritlawrancejing proposed stackforge/senlin-dashboard: Hide permission in profile table  https://review.openstack.org/23528912:21
*** branw__ has joined #senlin12:33
*** branw_ has quit IRC12:33
openstackgerritlawrancejing proposed stackforge/senlin-dashboard: Add node creation unit test  https://review.openstack.org/23529812:40
*** branw_ has joined #senlin12:48
*** branw__ has quit IRC12:51
*** jdandrea has joined #senlin12:54
openstackgerritlawrancejing proposed stackforge/senlin-dashboard: Improve profile update form  https://review.openstack.org/23531212:56
openstackgerritlawrancejing proposed stackforge/senlin-dashboard: Add cluster creation unit test  https://review.openstack.org/23532613:13
*** branw__ has joined #senlin13:33
*** branw_ has quit IRC13:33
*** pm90_ has joined #senlin13:34
*** Shijia has quit IRC13:42
*** Shijia has joined #senlin13:42
*** pm90_ has quit IRC13:47
*** pm90_ has joined #senlin13:50
*** lixinhui has quit IRC13:51
openstackgerritlawrancejing proposed stackforge/senlin-dashboard: Add node details overview page  https://review.openstack.org/23535913:51
*** Shijia_ has joined #senlin13:51
*** Shijia has quit IRC13:54
*** Shijia_ has quit IRC14:01
*** branw_ has joined #senlin14:02
*** branw__ has quit IRC14:06
*** lixinhui has joined #senlin14:13
*** pm90__ has joined #senlin14:17
*** lixinhui has quit IRC14:17
*** pm90_ has quit IRC14:20
*** lixinhui has joined #senlin14:40
*** lixinhui has quit IRC14:44
*** branw__ has joined #senlin14:47
*** branw_ has quit IRC14:47
*** lawrancejing has quit IRC14:51
*** branw_ has joined #senlin15:02
*** branw__ has quit IRC15:06
*** lixinhui has joined #senlin15:07
*** pm90__ has quit IRC15:08
*** pm90_ has joined #senlin15:09
*** pm90_ has quit IRC15:10
*** pm90_ has joined #senlin15:10
*** lixinhui has quit IRC15:12
*** pm90__ has joined #senlin15:14
*** branw_ has quit IRC15:17
*** branw_ has joined #senlin15:17
*** pm90_ has quit IRC15:17
*** pm90__ has quit IRC15:26
*** openstackgerrit has quit IRC15:31
*** openstackgerrit has joined #senlin15:32
*** branw__ has joined #senlin15:32
*** lixinhui has joined #senlin15:34
*** pratikmallya has joined #senlin15:35
*** branw_ has quit IRC15:36
*** lixinhui has quit IRC15:38
openstackgerritOpenStack Proposal Bot proposed stackforge/senlin: Updated from global requirements  https://review.openstack.org/23468115:46
*** Qiming has quit IRC15:49
*** lixinhui has joined #senlin16:01
*** pratikmallya has quit IRC16:04
*** lixinhui has quit IRC16:05
*** branw_ has joined #senlin16:18
*** branw__ has quit IRC16:18
*** lixinhui has joined #senlin16:28
*** branw__ has joined #senlin16:32
*** lixinhui has quit IRC16:33
*** branw_ has quit IRC16:36
*** pratikmallya has joined #senlin16:40
*** pratikma_ has joined #senlin16:46
*** branw_ has joined #senlin16:47
*** branw__ has quit IRC16:47
*** pratikmallya has quit IRC16:49
*** lixinhui has joined #senlin16:56
*** lixinhui has quit IRC17:00
*** pratikma_ has quit IRC17:06
*** pratikmallya has joined #senlin17:16
*** lixinhui has joined #senlin17:23
*** lixinhui has quit IRC17:27
*** branw__ has joined #senlin17:32
*** branw_ has quit IRC17:36
*** lixinhui has joined #senlin17:50
*** lixinhui has quit IRC17:54
*** lixinhui has joined #senlin18:17
*** branw__ has quit IRC18:17
*** branw__ has joined #senlin18:17
*** lixinhui has quit IRC18:21
*** branw_ has joined #senlin18:32
*** branw__ has quit IRC18:36
*** lixinhui has joined #senlin18:44
*** lixinhui has quit IRC18:48
*** lixinhui has joined #senlin19:11
*** lixinhui has quit IRC19:16
*** branw__ has joined #senlin19:17
*** branw_ has quit IRC19:17
*** lixinhui has joined #senlin19:38
*** lixinhui has quit IRC19:42
*** branw_ has joined #senlin20:02
*** lixinhui has joined #senlin20:05
*** branw__ has quit IRC20:06
*** lixinhui has quit IRC20:09
*** lixinhui has joined #senlin20:32
*** lixinhui has quit IRC20:37
*** branw__ has joined #senlin20:48
*** branw_ has quit IRC20:48
*** lixinhui has joined #senlin20:59
*** branw_ has joined #senlin21:03
*** lixinhui has quit IRC21:04
*** branw__ has quit IRC21:06
*** lixinhui has joined #senlin21:26
*** lixinhui has quit IRC21:31
*** lixinhui has joined #senlin21:54
*** lixinhui has quit IRC21:58
*** branw__ has joined #senlin22:02
*** branw_ has quit IRC22:02
*** branw_ has joined #senlin22:17
*** pratikmallya has quit IRC22:18
*** branw__ has quit IRC22:21
*** lixinhui has joined #senlin22:21
*** lixinhui has quit IRC22:25
*** branw__ has joined #senlin22:32
*** branw_ has quit IRC22:32
*** branw_ has joined #senlin22:48
*** lixinhui has joined #senlin22:48
*** branw__ has quit IRC22:51
*** lixinhui has quit IRC22:52
*** Qiming has joined #senlin22:55
*** pratikmallya has joined #senlin23:05
*** lixinhui has joined #senlin23:15
*** lixinhui has quit IRC23:19
openstackgerritMerged stackforge/senlin-dashboard: Hide permission in profile table  https://review.openstack.org/23528923:27
*** branw_ has quit IRC23:34
*** lixinhui has joined #senlin23:42
*** Qiming_ has joined #senlin23:44
*** jdandrea has quit IRC23:54
*** Qiming has quit IRC23:54
*** lixinhui has quit IRC23:54
*** pratikmallya has quit IRC23:55
*** yuanying has quit IRC23:55
*** openstackgerrit has quit IRC23:55
*** shu has quit IRC23:55
*** xuhaiwei has quit IRC23:55

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