openstackgerrit | xu-haiwei proposed stackforge/python-senlinclient: Fix help message for cluster_scale_in/out https://review.openstack.org/234737 | 00:05 |
---|---|---|
openstackgerrit | xu-haiwei proposed stackforge/senlin: Make 'grace_period' and 'destroy_after_deletion' work https://review.openstack.org/234736 | 00:16 |
*** pm90_ has joined #senlin | 00:49 | |
*** lawrancejing has joined #senlin | 00:54 | |
*** zhenguo has joined #senlin | 01:14 | |
*** jdandrea has quit IRC | 01:28 | |
*** Yanyanhu has joined #senlin | 01:33 | |
*** lixinhui has joined #senlin | 01:42 | |
*** lixinhui has quit IRC | 01:43 | |
*** elynn has joined #senlin | 01:51 | |
elynn | morning :) | 01:55 |
Qiming | morning | 02:02 |
Yanyanhu | morning | 02:02 |
*** Shijia has joined #senlin | 02:05 | |
*** Shijia has quit IRC | 02:07 | |
*** Shijia has joined #senlin | 02:08 | |
*** lixinhui has joined #senlin | 02:14 | |
*** Shijia_ has joined #senlin | 02:17 | |
*** Shijia has quit IRC | 02:17 | |
*** Shijia has joined #senlin | 02:24 | |
*** Shijia_ has quit IRC | 02:24 | |
*** Shijia_ has joined #senlin | 02:32 | |
*** Shijia has quit IRC | 02:32 | |
*** Qiming has quit IRC | 02:45 | |
*** Qiming has joined #senlin | 02:45 | |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: Make all cluster DB queries project_safe https://review.openstack.org/234683 | 02:56 |
elynn | I 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.yaml | 02:59 |
*** shu has joined #senlin | 03:12 | |
*** Qiming has quit IRC | 03:12 | |
*** Qiming has joined #senlin | 03:12 | |
xuhaiwei | elynn, so the senlin resource is landed? | 03:13 |
xuhaiwei | or you are just testing it? | 03:13 |
Qiming | xuhaiwei, IIUC, elynn is helping do a proof of concept and see it things can land smoothly | 03:16 |
*** Shijia has joined #senlin | 03:21 | |
*** Shijia_ has quit IRC | 03:21 | |
*** Liu has joined #senlin | 03:27 | |
elynn | I'm testing on it | 03:36 |
elynn | but still have some questions about profile properties. | 03:38 |
*** pm90_ has quit IRC | 03:59 | |
*** Liu has quit IRC | 04:03 | |
*** pm90_ has joined #senlin | 04:06 | |
*** pm90_ has quit IRC | 04:26 | |
openstackgerrit | Qiming Teng proposed stackforge/senlin: TODO item for actions API https://review.openstack.org/235084 | 05:11 |
Qiming | anyone please leave comments to the big tent proposal if any | 05:12 |
Qiming | I'm gonna submit it today to the TC governance for review | 05:13 |
Qiming | thanks | 05:13 |
*** pm90_ has joined #senlin | 05:17 | |
*** Shijia has quit IRC | 05:30 | |
*** Liu has joined #senlin | 05:32 | |
*** lawrancejing has quit IRC | 05:49 | |
*** lawrancejing has joined #senlin | 05:50 | |
*** Qiming has quit IRC | 06:00 | |
*** Qiming has joined #senlin | 06:10 | |
*** Liu has quit IRC | 06:34 | |
*** Liu has joined #senlin | 06:49 | |
*** Liu has quit IRC | 07:17 | |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: Remove two useless DB API interfaces https://review.openstack.org/235122 | 07:17 |
*** Qiming has quit IRC | 07:20 | |
*** Qiming has joined #senlin | 07:20 | |
*** Yanyan has joined #senlin | 07:30 | |
*** Yanyanhu has quit IRC | 07:33 | |
*** Liu has joined #senlin | 07:34 | |
openstackgerrit | Merged stackforge/senlin: TODO item for actions API https://review.openstack.org/235084 | 07:45 |
*** Shijia has joined #senlin | 07:50 | |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: Make all cluster DB queries project_safe https://review.openstack.org/234683 | 07:50 |
openstackgerrit | Qiming Teng proposed stackforge/senlin: Temporary proposal for big tent adoption https://review.openstack.org/234162 | 08:03 |
Qiming | zhenguo, there? | 08:04 |
zhenguo | Qiming: yeah | 08:04 |
Qiming | during the past weekly IRC meeting, we conducted a PTL election | 08:04 |
Qiming | I myself volunteered and got approval by the team | 08:05 |
Qiming | one thing I'd like to discuss with you (also it was raised in the meeting) is about senlin-dashboard | 08:06 |
zhenguo | Qiming: oh, definitely | 08:06 |
Qiming | do we need a PTL for that | 08:06 |
openstackgerrit | Merged stackforge/senlin: Remove two useless DB API interfaces https://review.openstack.org/235122 | 08:06 |
zhenguo | it's a sub project of senlin | 08:06 |
Qiming | I myself knows almost nothing about dashboard | 08:06 |
Qiming | I appreciate your kindness adding me to the core team, so that I can review patches from service interaction's perspective | 08:07 |
Qiming | since you are more experienced than I am in web development, and in how community organizes dashboard projects | 08:08 |
zhenguo | IMO, it's almost like python-senlinclient, as a sub project to senlin. so seems you PTL should lead all these projects. | 08:08 |
Qiming | I really want to hear your suggestions | 08:08 |
Qiming | I'd love to do anything to the dashboard | 08:08 |
Qiming | but I'm really not capable of making the right decisions | 08:09 |
zhenguo | yeah, you can give suggestions and requirements is enough I think. | 08:09 |
zhenguo | so, how's the dashboard project going now, sorry for not do anything these days. | 08:10 |
Qiming | if it is okay with you, and we are doing this following the community practice, let's do it | 08:10 |
Qiming | I'm seeing lawrancejing a hard worker recently | 08:10 |
zhenguo | yeah, I think it's common in community | 08:11 |
Qiming | cluster, node, profile operations are all being added and tuned | 08:11 |
lawrancejing | :) | 08:11 |
zhenguo | cool, thanks lawrancejing for that | 08:11 |
Qiming | maybe we can do this | 08:11 |
Qiming | we already have senlin-dashboard-cores group? | 08:12 |
zhenguo | yeah, in Tuskar, Magnum, Manila, we all have separate dashboard-core groups. | 08:13 |
Qiming | we can add senlin cores to senlin-dashbard-core group | 08:13 |
zhenguo | yeah. | 08:13 |
Qiming | and vice versa | 08:13 |
Qiming | that way we can accelerate code reviews | 08:13 |
zhenguo | yeah, and maybe we can add horizon-cores as well. | 08:14 |
Qiming | okay, so the suggestion is just keep the senlin-core and senlin-dashboard-core separate | 08:15 |
Qiming | recruit new members based on people's reviews and contributions | 08:15 |
Qiming | adding horizon-core to senlin-dashboard-core is good | 08:16 |
zhenguo | yeah | 08:16 |
Qiming | these two teams share more expertise in common | 08:16 |
Qiming | umm | 08:17 |
*** lawrancejing has quit IRC | 08:17 | |
*** lawrancejing has joined #senlin | 08:18 | |
openstackgerrit | Qiming Teng proposed stackforge/senlin: Temporary proposal for big tent adoption https://review.openstack.org/234162 | 08:20 |
Qiming | xuhaiwei, and Yanyan | 08:21 |
Yanyan | hi | 08:21 |
xuhaiwei | Qiming, liverbty release is today, are we lucky enough to join OpenStack in Liverbty | 08:21 |
Qiming | when I was drafting the big tent proposal, I was reading the proposal from kolla | 08:21 |
xuhaiwei | yes? | 08:21 |
Qiming | xuhaiwei, that will not happen, :( | 08:22 |
xuhaiwei | I am also reading the patch of kolla | 08:22 |
Yanyan | me too, actually :) | 08:22 |
Qiming | one of the things we were not doing well is about following the +2/+A rules | 08:22 |
xuhaiwei | sometime | 08:23 |
Qiming | the rule suggests that we need to have 2 +2s and 1 +A for patch to be merged | 08:23 |
Yanyan | yes, for some patches | 08:23 |
Qiming | sometimes, you know, we were breaking that rule | 08:23 |
Qiming | I myself have been doing that a lot | 08:24 |
Yanyan | yes, in cases the patch need fast approval | 08:24 |
* Qiming kicks himself | 08:24 | |
xuhaiwei | sometime I just can't understand the patch, so didn't do anything about it | 08:24 |
Yanyan | not your fault I believe... be cause we really don't have enough bandwidth to cover both developing and review sometimes | 08:24 |
Qiming | we have execuses: patch dependency, core team too small, urgency to fix a problem ... | 08:24 |
Qiming | but they are still not good practices | 08:25 |
Qiming | so .. on one hand, we need to do more reviews | 08:25 |
Yanyan | yes, definitely | 08:26 |
xuhaiwei | that might be a reason to block us? | 08:26 |
Qiming | on the other hand, please help recommend contributors to the team | 08:26 |
Qiming | no, it won't | 08:26 |
Qiming | there are proposals for projects that don't have a single line of code, :D | 08:26 |
Yanyan | sure, hope more people can join us | 08:27 |
xuhaiwei | maybe sometime we should not merge patch so quick, just wait for some new guys to review | 08:27 |
Qiming | in any community work, unless it is your full time job, people join and leave | 08:28 |
Qiming | that is common | 08:28 |
Yanyan | yea | 08:28 |
Qiming | so maybe we can lower the bar for people to become a core reviewer | 08:28 |
Qiming | let's keep our eyes open, :) | 08:29 |
zhenguo | Qiming: I have included senlin-core to senlin-dashboard-core groups. | 08:29 |
Qiming | thanks zhenguo | 08:29 |
zhenguo | :) | 08:30 |
xuhaiwei | some projects have a rule to become a core member, like review more than 3 patches a day during about 3 months | 08:30 |
Qiming | xuhaiwei, yes, that is a high bar for senlin, :) | 08:30 |
xuhaiwei | maybe we can make it 2 reviews for one month, something like that | 08:31 |
Qiming | hoho | 08:31 |
Qiming | all projects are watched: http://git.openstack.org/cgit/openstack/governance/tree/tools/teamstats.py | 08:32 |
xuhaiwei | anyway we want peopleo who can really help senlin | 08:32 |
Qiming | yes | 08:32 |
Yanyan | any contribution is welcome :) | 08:32 |
Qiming | line 139 in the above file is about the statistics they use to evaluate the diversity of a project | 08:32 |
xuhaiwei | but sometime people worked hard to be a core, but just stop working after being one | 08:33 |
Qiming | xuhaiwei, that is what I was saying: people join and leave | 08:33 |
Qiming | if it is not his/her full time job | 08:34 |
Yanyan | I guess sometimes this is caused by stupid evaluation rules enforced by some companies | 08:34 |
xuhaiwei | you are right :) | 08:34 |
Yanyan | getting support from boss to allow you do community job fulltime is not a easy task :) | 08:34 |
Qiming | right, contributions to a stackforge project? are you mad? | 08:34 |
Yanyan | so we are lucky to get chance to work here :P | 08:35 |
xuhaiwei | yap | 08:35 |
Qiming | oh, another change to the proposal | 08:36 |
Qiming | I put this into the big tent proposal: Team plans to excercise a PTL rotation among core contributors. | 08:37 |
Qiming | hope you won't mind, :) | 08:37 |
xuhaiwei | that is common | 08:37 |
Qiming | xuhaiwei, next cycle could be you | 08:38 |
Qiming | could be mr. newcomer | 08:39 |
Yanyan | sure, although we are still not ready now you know :) | 08:39 |
xuhaiwei | yes, that is a big change | 08:39 |
Qiming | you will | 08:39 |
Yanyan | yea, I believe we will some day :) | 08:39 |
Qiming | great! | 08:40 |
Qiming | cloning openstack/governance | 08:43 |
Yanyan | cool | 08:43 |
Qiming | let's have a try | 08:43 |
Yanyan | yep | 08:43 |
Yanyan | \o/ | 08:44 |
Qiming | be prepared though, it's near summit time | 08:44 |
Qiming | so the review may take time, some projects have cancelled weekly meeting | 08:44 |
Qiming | I am proposing it NOW because the Mitaka cycle has started | 08:45 |
Qiming | we want to be on board | 08:45 |
Yanyan | we are ready for a fight I think | 08:45 |
xuhaiwei | fight what? | 08:48 |
xuhaiwei | :) | 08:48 |
Yanyan | xuhaiwei, fight will those doubt and criticism :) | 08:53 |
Yanyan | will/with | 08:53 |
xuhaiwei | I found set_status method of cluster and node is often missed here and there, especially when action fails | 08:57 |
xuhaiwei | I am trying to fix it | 08:57 |
Qiming | xuhaiwei, yes, I found that as well | 08:58 |
Qiming | when a node fails creation, the status wasn't correct set, for example | 08:58 |
Qiming | it would be nice if you can help cleanse it | 08:58 |
xuhaiwei | I started with cluster action | 08:59 |
Qiming | one reminder is that you need to start from cluster_action's execute() method, not individual action handlers | 08:59 |
xuhaiwei | I have just finished the individual action handlers :((( | 09:00 |
Qiming | ... | 09:01 |
xuhaiwei | execute() and _execute() are no need to fix I think | 09:01 |
Qiming | then you learned something | 09:01 |
Qiming | https://review.openstack.org/235172 | 09:01 |
Yanyan | o/ | 09:07 |
Yanyan | hi, Qiming, ':' is missing in line2801 | 09:09 |
Qiming | what? | 09:10 |
xuhaiwei | the proposal patch | 09:12 |
Yanyan | Qiming, the patch check failed for nit error | 09:13 |
*** branw has joined #senlin | 09:15 | |
Qiming | ah I see | 09:15 |
Qiming | I noticed that line was displayed in a different color when being edited | 09:16 |
Qiming | didn't realize what kind of an error it is | 09:16 |
Qiming | :) | 09:16 |
Yanyan | ;) | 09:17 |
*** branw has quit IRC | 09:17 | |
*** branw has joined #senlin | 09:17 | |
*** Qiming has quit IRC | 09:22 | |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: Make all cluster DB queries project_safe https://review.openstack.org/234683 | 09:22 |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: [WIP]Make node related DB queries project safe https://review.openstack.org/235182 | 09:22 |
openstackgerrit | Yanyan Hu proposed stackforge/senlin: [WIP]Make node related DB queries project safe https://review.openstack.org/235182 | 09:24 |
*** Qiming has joined #senlin | 09:26 | |
*** branw_ has joined #senlin | 09:32 | |
*** branw has quit IRC | 09:36 | |
*** Liu has quit IRC | 09:36 | |
*** Yanyan has quit IRC | 09:55 | |
*** zhenguo has quit IRC | 10:09 | |
*** lixinhui has quit IRC | 10:10 | |
*** lixinhui has joined #senlin | 10:10 | |
lawrancejing | Hello Qiming, zhengguo, I will continue to work on senlin-dashboard, until most work done, include 70 ~ 90% unit / functional test | 10:11 |
Qiming | lawrancejing, thanks! | 10:11 |
*** lawrancejing has quit IRC | 10:12 | |
*** branw__ has joined #senlin | 10:32 | |
*** branw_ has quit IRC | 10:32 | |
*** pm90_ has quit IRC | 10:39 | |
*** lawrancejing has joined #senlin | 10:43 | |
*** branw_ has joined #senlin | 10:47 | |
*** branw__ has quit IRC | 10:51 | |
*** elynn has quit IRC | 11:08 | |
*** branw__ has joined #senlin | 12:02 | |
*** branw_ has quit IRC | 12:02 | |
openstackgerrit | lawrancejing proposed stackforge/senlin-dashboard: Add profile create test https://review.openstack.org/235286 | 12:13 |
*** branw_ has joined #senlin | 12:17 | |
*** branw__ has quit IRC | 12:21 | |
openstackgerrit | lawrancejing proposed stackforge/senlin-dashboard: Hide permission in profile table https://review.openstack.org/235289 | 12:21 |
*** branw__ has joined #senlin | 12:33 | |
*** branw_ has quit IRC | 12:33 | |
openstackgerrit | lawrancejing proposed stackforge/senlin-dashboard: Add node creation unit test https://review.openstack.org/235298 | 12:40 |
*** branw_ has joined #senlin | 12:48 | |
*** branw__ has quit IRC | 12:51 | |
*** jdandrea has joined #senlin | 12:54 | |
openstackgerrit | lawrancejing proposed stackforge/senlin-dashboard: Improve profile update form https://review.openstack.org/235312 | 12:56 |
openstackgerrit | lawrancejing proposed stackforge/senlin-dashboard: Add cluster creation unit test https://review.openstack.org/235326 | 13:13 |
*** branw__ has joined #senlin | 13:33 | |
*** branw_ has quit IRC | 13:33 | |
*** pm90_ has joined #senlin | 13:34 | |
*** Shijia has quit IRC | 13:42 | |
*** Shijia has joined #senlin | 13:42 | |
*** pm90_ has quit IRC | 13:47 | |
*** pm90_ has joined #senlin | 13:50 | |
*** lixinhui has quit IRC | 13:51 | |
openstackgerrit | lawrancejing proposed stackforge/senlin-dashboard: Add node details overview page https://review.openstack.org/235359 | 13:51 |
*** Shijia_ has joined #senlin | 13:51 | |
*** Shijia has quit IRC | 13:54 | |
*** Shijia_ has quit IRC | 14:01 | |
*** branw_ has joined #senlin | 14:02 | |
*** branw__ has quit IRC | 14:06 | |
*** lixinhui has joined #senlin | 14:13 | |
*** pm90__ has joined #senlin | 14:17 | |
*** lixinhui has quit IRC | 14:17 | |
*** pm90_ has quit IRC | 14:20 | |
*** lixinhui has joined #senlin | 14:40 | |
*** lixinhui has quit IRC | 14:44 | |
*** branw__ has joined #senlin | 14:47 | |
*** branw_ has quit IRC | 14:47 | |
*** lawrancejing has quit IRC | 14:51 | |
*** branw_ has joined #senlin | 15:02 | |
*** branw__ has quit IRC | 15:06 | |
*** lixinhui has joined #senlin | 15:07 | |
*** pm90__ has quit IRC | 15:08 | |
*** pm90_ has joined #senlin | 15:09 | |
*** pm90_ has quit IRC | 15:10 | |
*** pm90_ has joined #senlin | 15:10 | |
*** lixinhui has quit IRC | 15:12 | |
*** pm90__ has joined #senlin | 15:14 | |
*** branw_ has quit IRC | 15:17 | |
*** branw_ has joined #senlin | 15:17 | |
*** pm90_ has quit IRC | 15:17 | |
*** pm90__ has quit IRC | 15:26 | |
*** openstackgerrit has quit IRC | 15:31 | |
*** openstackgerrit has joined #senlin | 15:32 | |
*** branw__ has joined #senlin | 15:32 | |
*** lixinhui has joined #senlin | 15:34 | |
*** pratikmallya has joined #senlin | 15:35 | |
*** branw_ has quit IRC | 15:36 | |
*** lixinhui has quit IRC | 15:38 | |
openstackgerrit | OpenStack Proposal Bot proposed stackforge/senlin: Updated from global requirements https://review.openstack.org/234681 | 15:46 |
*** Qiming has quit IRC | 15:49 | |
*** lixinhui has joined #senlin | 16:01 | |
*** pratikmallya has quit IRC | 16:04 | |
*** lixinhui has quit IRC | 16:05 | |
*** branw_ has joined #senlin | 16:18 | |
*** branw__ has quit IRC | 16:18 | |
*** lixinhui has joined #senlin | 16:28 | |
*** branw__ has joined #senlin | 16:32 | |
*** lixinhui has quit IRC | 16:33 | |
*** branw_ has quit IRC | 16:36 | |
*** pratikmallya has joined #senlin | 16:40 | |
*** pratikma_ has joined #senlin | 16:46 | |
*** branw_ has joined #senlin | 16:47 | |
*** branw__ has quit IRC | 16:47 | |
*** pratikmallya has quit IRC | 16:49 | |
*** lixinhui has joined #senlin | 16:56 | |
*** lixinhui has quit IRC | 17:00 | |
*** pratikma_ has quit IRC | 17:06 | |
*** pratikmallya has joined #senlin | 17:16 | |
*** lixinhui has joined #senlin | 17:23 | |
*** lixinhui has quit IRC | 17:27 | |
*** branw__ has joined #senlin | 17:32 | |
*** branw_ has quit IRC | 17:36 | |
*** lixinhui has joined #senlin | 17:50 | |
*** lixinhui has quit IRC | 17:54 | |
*** lixinhui has joined #senlin | 18:17 | |
*** branw__ has quit IRC | 18:17 | |
*** branw__ has joined #senlin | 18:17 | |
*** lixinhui has quit IRC | 18:21 | |
*** branw_ has joined #senlin | 18:32 | |
*** branw__ has quit IRC | 18:36 | |
*** lixinhui has joined #senlin | 18:44 | |
*** lixinhui has quit IRC | 18:48 | |
*** lixinhui has joined #senlin | 19:11 | |
*** lixinhui has quit IRC | 19:16 | |
*** branw__ has joined #senlin | 19:17 | |
*** branw_ has quit IRC | 19:17 | |
*** lixinhui has joined #senlin | 19:38 | |
*** lixinhui has quit IRC | 19:42 | |
*** branw_ has joined #senlin | 20:02 | |
*** lixinhui has joined #senlin | 20:05 | |
*** branw__ has quit IRC | 20:06 | |
*** lixinhui has quit IRC | 20:09 | |
*** lixinhui has joined #senlin | 20:32 | |
*** lixinhui has quit IRC | 20:37 | |
*** branw__ has joined #senlin | 20:48 | |
*** branw_ has quit IRC | 20:48 | |
*** lixinhui has joined #senlin | 20:59 | |
*** branw_ has joined #senlin | 21:03 | |
*** lixinhui has quit IRC | 21:04 | |
*** branw__ has quit IRC | 21:06 | |
*** lixinhui has joined #senlin | 21:26 | |
*** lixinhui has quit IRC | 21:31 | |
*** lixinhui has joined #senlin | 21:54 | |
*** lixinhui has quit IRC | 21:58 | |
*** branw__ has joined #senlin | 22:02 | |
*** branw_ has quit IRC | 22:02 | |
*** branw_ has joined #senlin | 22:17 | |
*** pratikmallya has quit IRC | 22:18 | |
*** branw__ has quit IRC | 22:21 | |
*** lixinhui has joined #senlin | 22:21 | |
*** lixinhui has quit IRC | 22:25 | |
*** branw__ has joined #senlin | 22:32 | |
*** branw_ has quit IRC | 22:32 | |
*** branw_ has joined #senlin | 22:48 | |
*** lixinhui has joined #senlin | 22:48 | |
*** branw__ has quit IRC | 22:51 | |
*** lixinhui has quit IRC | 22:52 | |
*** Qiming has joined #senlin | 22:55 | |
*** pratikmallya has joined #senlin | 23:05 | |
*** lixinhui has joined #senlin | 23:15 | |
*** lixinhui has quit IRC | 23:19 | |
openstackgerrit | Merged stackforge/senlin-dashboard: Hide permission in profile table https://review.openstack.org/235289 | 23:27 |
*** branw_ has quit IRC | 23:34 | |
*** lixinhui has joined #senlin | 23:42 | |
*** Qiming_ has joined #senlin | 23:44 | |
*** jdandrea has quit IRC | 23:54 | |
*** Qiming has quit IRC | 23:54 | |
*** lixinhui has quit IRC | 23:54 | |
*** pratikmallya has quit IRC | 23:55 | |
*** yuanying has quit IRC | 23:55 | |
*** openstackgerrit has quit IRC | 23:55 | |
*** shu has quit IRC | 23:55 | |
*** xuhaiwei has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!