*** achampion has quit IRC | 00:10 | |
*** ramishra has joined #heat | 00:21 | |
*** ramishra has quit IRC | 00:26 | |
*** asalkeld has quit IRC | 00:34 | |
*** Qiming has joined #heat | 00:47 | |
*** EricGonczer_ has joined #heat | 00:53 | |
*** alexheneveld has quit IRC | 01:01 | |
*** ramishra has joined #heat | 01:22 | |
*** EricGonczer_ has quit IRC | 01:22 | |
*** erkules_ has joined #heat | 01:25 | |
*** ramishra has quit IRC | 01:26 | |
*** erkules has quit IRC | 01:27 | |
*** EricGonczer_ has joined #heat | 01:32 | |
*** EricGonczer_ has quit IRC | 01:33 | |
*** EricGonczer_ has joined #heat | 01:47 | |
*** killer_prince has quit IRC | 01:53 | |
*** lazy_prince has joined #heat | 01:53 | |
*** cody-somerville_ has quit IRC | 02:01 | |
openstackgerrit | Ishant Tyagi proposed a change to openstack/heat: Add DB model for resource graph and also persist the graph https://review.openstack.org/128590 | 02:16 |
---|---|---|
*** ramishra has joined #heat | 02:23 | |
*** ramishra has quit IRC | 02:28 | |
*** dimsum_ has joined #heat | 02:31 | |
*** dsneddon has quit IRC | 02:35 | |
*** dsneddon has joined #heat | 02:35 | |
*** jyoti-ranjan has joined #heat | 02:43 | |
*** EricGonczer_ has quit IRC | 02:53 | |
*** dimsum_ has quit IRC | 03:21 | |
*** ramishra has joined #heat | 03:23 | |
*** ramishra has quit IRC | 03:28 | |
*** ramishra has joined #heat | 03:36 | |
*** ramishra has quit IRC | 03:41 | |
*** jyoti-ranjan has quit IRC | 03:42 | |
*** LiJiansheng has joined #heat | 03:48 | |
*** dimsum_ has joined #heat | 04:22 | |
*** dimsum_ has quit IRC | 04:27 | |
*** ramishra has joined #heat | 04:37 | |
*** ramishra has quit IRC | 04:41 | |
*** boris-42 has quit IRC | 04:56 | |
*** zero1 has joined #heat | 05:02 | |
*** dimsum_ has joined #heat | 05:32 | |
*** dimsum_ has quit IRC | 05:37 | |
*** mohits has joined #heat | 05:38 | |
*** ramishra has joined #heat | 05:38 | |
*** ramishra has quit IRC | 05:42 | |
*** serg_melikyan has joined #heat | 05:49 | |
*** sabeen1 has joined #heat | 05:59 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/128188 | 06:01 |
*** pasquier-s has joined #heat | 06:25 | |
*** zero1 has quit IRC | 06:28 | |
*** serg_melikyan has quit IRC | 06:34 | |
*** pasquier-s has quit IRC | 06:35 | |
*** miguelgrinberg_ has joined #heat | 06:36 | |
*** ramishra has joined #heat | 06:38 | |
*** ramishra has quit IRC | 06:43 | |
*** pasquier-s has joined #heat | 06:49 | |
*** lazy_prince is now known as killer_prince | 07:05 | |
*** pasquier-s has quit IRC | 07:09 | |
*** Qiming has quit IRC | 07:15 | |
*** mohits has quit IRC | 07:22 | |
*** sabeen1 has quit IRC | 07:22 | |
*** killer_prince is now known as lazy_prince | 07:25 | |
*** ramishra has joined #heat | 07:40 | |
*** ramishra has quit IRC | 07:44 | |
*** cmyster_away is now known as cmyster | 07:48 | |
*** alexpilotti has joined #heat | 07:51 | |
cmyster | morning | 08:03 |
*** lazy_prince is now known as killer_prince | 08:07 | |
*** alexpilotti has quit IRC | 08:15 | |
*** ramishra has joined #heat | 08:40 | |
*** ramishra has quit IRC | 08:45 | |
*** rushiagr_away is now known as rushiagr | 09:05 | |
*** dimsum_ has joined #heat | 09:09 | |
*** rushiagr is now known as rushiagr_away | 09:12 | |
*** dimsum_ has quit IRC | 09:14 | |
*** serg_melikyan has joined #heat | 09:35 | |
*** killer_prince is now known as lazy_prince | 09:35 | |
*** serg_melikyan has quit IRC | 09:39 | |
*** alexpilotti has joined #heat | 09:40 | |
*** ramishra has joined #heat | 09:41 | |
*** ramishra has quit IRC | 09:45 | |
*** robklg has joined #heat | 10:17 | |
*** robklg has quit IRC | 10:26 | |
*** alexheneveld has joined #heat | 10:30 | |
*** Qiming has joined #heat | 10:40 | |
*** serg_melikyan has joined #heat | 10:40 | |
*** alexheneveld has quit IRC | 10:41 | |
*** ramishra has joined #heat | 10:42 | |
*** fbo has quit IRC | 10:43 | |
*** serg_melikyan has quit IRC | 10:45 | |
*** serg_melikyan has joined #heat | 10:46 | |
*** ramishra has quit IRC | 10:46 | |
*** alexpilotti has quit IRC | 10:47 | |
*** serg_melikyan has quit IRC | 10:50 | |
*** alexheneveld has joined #heat | 10:51 | |
*** LiJiansheng has quit IRC | 10:52 | |
*** fbo has joined #heat | 11:01 | |
*** alexheneveld has quit IRC | 11:11 | |
*** lazy_prince is now known as killer_prince | 11:19 | |
*** erkules_ is now known as erkules | 11:28 | |
*** ramishra has joined #heat | 11:43 | |
*** ramishra has quit IRC | 11:47 | |
*** dimsum_ has joined #heat | 11:59 | |
*** dimsum_ has quit IRC | 12:03 | |
*** mtreinish has quit IRC | 12:11 | |
*** tspatzier has joined #heat | 12:23 | |
*** dimsum_ has joined #heat | 12:37 | |
*** alexpilotti has joined #heat | 12:39 | |
*** ramishra has joined #heat | 12:43 | |
*** ramishra has quit IRC | 12:48 | |
*** xhu has joined #heat | 12:53 | |
*** xhu has quit IRC | 12:54 | |
*** EricGonczer_ has joined #heat | 13:15 | |
*** EricGonczer_ has quit IRC | 13:19 | |
*** mtreinish has joined #heat | 13:24 | |
*** xhu has joined #heat | 13:26 | |
*** alexpilotti has quit IRC | 13:37 | |
*** xhu has quit IRC | 14:10 | |
*** serg_melikyan has joined #heat | 14:17 | |
*** EricGonczer_ has joined #heat | 14:18 | |
*** serg_melikyan has quit IRC | 14:20 | |
*** serg_melikyan has joined #heat | 14:20 | |
*** serg_melikyan has quit IRC | 14:24 | |
*** tspatzier has quit IRC | 14:27 | |
*** EricGonczer_ has quit IRC | 14:29 | |
*** robklg has joined #heat | 14:30 | |
*** robklg has quit IRC | 14:40 | |
*** cmyster has quit IRC | 14:41 | |
*** killer_prince has quit IRC | 14:50 | |
*** killer_prince has joined #heat | 14:57 | |
*** killer_prince is now known as lazy_prince | 14:57 | |
*** serg_melikyan has joined #heat | 15:02 | |
*** serg_melikyan has quit IRC | 15:09 | |
*** serg_melikyan has joined #heat | 15:15 | |
*** dimsum_ has quit IRC | 15:16 | |
*** dimsum_ has joined #heat | 15:16 | |
*** dimsum_ has quit IRC | 15:21 | |
*** dimsum_ has joined #heat | 15:23 | |
*** dimsum_ has quit IRC | 15:29 | |
*** zaneb has quit IRC | 15:29 | |
*** zaneb has joined #heat | 15:30 | |
*** Drago has joined #heat | 15:37 | |
*** serg_melikyan has quit IRC | 15:40 | |
*** mspreitz has joined #heat | 15:43 | |
*** Drago has quit IRC | 15:43 | |
*** Drago has joined #heat | 15:43 | |
*** dimsum_ has joined #heat | 15:53 | |
*** sabeen has joined #heat | 15:54 | |
*** serg_melikyan has joined #heat | 15:54 | |
*** serg_melikyan has quit IRC | 15:59 | |
*** Qiming has quit IRC | 16:03 | |
*** tspatzier has joined #heat | 16:05 | |
*** mspreitz has quit IRC | 16:16 | |
*** serg_melikyan has joined #heat | 16:24 | |
*** serg_melikyan has quit IRC | 16:27 | |
*** serg_melikyan has joined #heat | 16:27 | |
*** mkollaro has joined #heat | 16:38 | |
*** sabeen has quit IRC | 16:39 | |
*** EricGonczer_ has joined #heat | 16:43 | |
*** serg_melikyan has quit IRC | 16:45 | |
*** serg_melikyan has joined #heat | 16:52 | |
*** Drago has quit IRC | 17:14 | |
*** dimsum_ has quit IRC | 17:19 | |
*** dimsum_ has joined #heat | 17:19 | |
*** sdake_ has joined #heat | 17:21 | |
*** hdd has joined #heat | 17:23 | |
*** dimsum_ has quit IRC | 17:24 | |
*** EricGonczer_ has quit IRC | 17:32 | |
*** cody-somerville_ has joined #heat | 17:34 | |
*** Drago has joined #heat | 17:37 | |
*** pscheie has quit IRC | 17:37 | |
*** Goneri has quit IRC | 17:37 | |
*** Adri2000 has quit IRC | 17:37 | |
*** gooblja has quit IRC | 17:37 | |
*** zz_gondoi has quit IRC | 17:37 | |
*** Adri2000 has joined #heat | 17:37 | |
*** pscheie has joined #heat | 17:38 | |
*** Goneri has joined #heat | 17:38 | |
*** gooblja_ has joined #heat | 17:39 | |
*** gondoi has joined #heat | 17:40 | |
*** serg_melikyan has quit IRC | 17:44 | |
*** Drago has quit IRC | 17:44 | |
*** Drago has joined #heat | 17:44 | |
*** serg_melikyan has joined #heat | 17:44 | |
*** sabeen1 has joined #heat | 17:51 | |
*** serg_melikyan has quit IRC | 17:54 | |
*** serg_melikyan has joined #heat | 18:03 | |
*** serg_melikyan has joined #heat | 18:17 | |
*** hdd has quit IRC | 18:45 | |
*** mcwoods has quit IRC | 19:12 | |
*** boris-42 has joined #heat | 19:24 | |
*** dimsum_ has joined #heat | 19:26 | |
*** dimsum_ has quit IRC | 19:37 | |
*** kebray has joined #heat | 19:37 | |
*** dimsum_ has joined #heat | 19:37 | |
*** dimsum_ has quit IRC | 19:42 | |
*** dimsum_ has joined #heat | 19:45 | |
*** dimsum_ has quit IRC | 19:53 | |
*** dimsum_ has joined #heat | 19:53 | |
*** dimsum_ has quit IRC | 19:58 | |
*** sdake_ has quit IRC | 20:00 | |
*** dimsum_ has joined #heat | 20:04 | |
*** dimsum_ has quit IRC | 20:07 | |
*** dimsum_ has joined #heat | 20:08 | |
*** dimsum_ has quit IRC | 20:12 | |
*** dimsum_ has joined #heat | 20:20 | |
*** mkollaro has quit IRC | 20:25 | |
*** fbo has quit IRC | 20:26 | |
*** serg_melikyan has quit IRC | 20:32 | |
*** fbo has joined #heat | 20:34 | |
openstackgerrit | Steve Baker proposed a change to openstack/heat-specs: Heat functional tests https://review.openstack.org/110171 | 20:35 |
*** dimsum_ has quit IRC | 20:42 | |
*** dimsum_ has joined #heat | 20:43 | |
*** fbo has quit IRC | 20:43 | |
*** fbo has joined #heat | 20:44 | |
*** tspatzier has quit IRC | 20:46 | |
*** dimsum_ has quit IRC | 20:47 | |
*** dimsum_ has joined #heat | 21:08 | |
*** tomek_adamczewsk has joined #heat | 21:22 | |
openstackgerrit | A change was merged to openstack/heat-specs: Heat functional tests https://review.openstack.org/110171 | 21:29 |
*** kebray has quit IRC | 21:35 | |
openstackgerrit | A change was merged to openstack/heat-templates: Use utf-8 encoding when writing config scripts https://review.openstack.org/128907 | 21:37 |
*** Qiming has joined #heat | 21:38 | |
*** sabeen1 has quit IRC | 21:39 | |
*** Drago has quit IRC | 21:41 | |
*** alexheneveld has joined #heat | 22:01 | |
*** tomek_adamczewsk has quit IRC | 22:03 | |
*** tomek_adamczewsk has joined #heat | 22:09 | |
Qiming | stevebaker, are you around? | 22:18 |
*** alexpilotti has joined #heat | 22:26 | |
*** dimsum_ has quit IRC | 22:28 | |
*** dimsum_ has joined #heat | 22:28 | |
*** dimsum_ has quit IRC | 22:32 | |
*** dimsum_ has joined #heat | 22:34 | |
*** asalkeld has joined #heat | 22:34 | |
asalkeld | morning | 22:39 |
Qiming | 6:40 am here, morning | 22:39 |
Qiming | asalkeld, need some advices from you on https://review.openstack.org/#/c/119619/ | 22:40 |
asalkeld | ok, Qiming just getting started here | 22:41 |
Qiming | I saw no further reviews on it since last meeting | 22:41 |
Qiming | should I abandon it and rebase the other changes onto master? | 22:41 |
Qiming | you know, a lot of discussions and revisions were targeting resource groups recently | 22:42 |
asalkeld | Qiming, have you looked at all at flattening the class structure, is it feasible? | 22:42 |
Qiming | yes, asalkeld, I thought about it | 22:42 |
Qiming | my initial understanding is that it won't be easy task. even if some logics are extracted into util functions, those functions may still need to access class members | 22:43 |
asalkeld | well you should pass everything as a param | 22:45 |
Qiming | the reason I proposed #119619 was that I want Heat AutoScalingResourceGroup a direct subclass of ResourceGroup, if that makes any senses at all | 22:45 |
asalkeld | Qiming, another option is to have a MixinClass | 22:46 |
Qiming | heat/scaling/cooldown is already a mixin | 22:46 |
Qiming | but I don't see it an elegant solution, ;) | 22:46 |
asalkeld | i am not a fan of the current resource group class | 22:47 |
asalkeld | it's super ugly | 22:48 |
Qiming | I understand the benefits to extract resize(), adjust(), ... into utils, but I am doubting how wide the interface will become | 22:48 |
* Qiming nods | 22:48 | |
asalkeld | def _handle_repl_val(self, res_name, val): | 22:49 |
asalkeld | def _do_prop_replace(self, res_name, res_def_template): | 22:50 |
*** alexpilotti has quit IRC | 22:50 | |
Qiming | Once all resize, adjust logics are moved up to the resource group level, the subclasses would be much cleaner | 22:50 |
Qiming | asalkeld, blame stevebaker then, :) | 22:50 |
asalkeld | imho no one needs to inherit from resource group | 22:50 |
asalkeld | (if you can pull those methods out) | 22:50 |
Qiming | in other words, you mean I should abandon the reorg specs and the patches? | 22:51 |
asalkeld | Qiming, well what if you just didn't do this one | 22:52 |
asalkeld | so pull the logic that you need out | 22:52 |
asalkeld | and use it from the other classes | 22:53 |
asalkeld | (qiming let me check your spec) | 22:53 |
Qiming | great. I am feeling embarassed for having not making any progresses for so long as a contributor | 22:55 |
asalkeld | don't stress | 22:55 |
asalkeld | this should be easy | 22:55 |
asalkeld | StackResource -> AutoScalingGroup | 22:56 |
asalkeld | StackResource -> AWSAutoScalingGroup | 22:56 |
*** alexpilotti has joined #heat | 22:57 | |
asalkeld | then grab the resize functionality from instance group (as a util) | 22:57 |
asalkeld | and what ever you need from ResourceGroup | 22:57 |
asalkeld | and put that into a util | 22:57 |
Qiming | I see, so you want a *complete* separation | 22:57 |
asalkeld | sure, why not | 22:58 |
asalkeld | also zaneb suggested instance group inherit from AutoScalingGroup | 22:58 |
asalkeld | fixed scaling is a special kind of autoscaling | 22:58 |
Qiming | I gave zaneb's suggestion a thought, but didn't quite understand the intention | 22:59 |
asalkeld | so what he is saying is: | 22:59 |
Qiming | autoscaling is a automated scaling, isn't it? | 22:59 |
asalkeld | 1. you really don't need instance group | 22:59 |
asalkeld | (i added it a *long* time ago for ease of use) | 22:59 |
asalkeld | you can put it on top of autoscaling group - basically with less options | 23:00 |
Qiming | yep, that's what I intended | 23:00 |
asalkeld | Qiming, yeah and instance group is manual scaling | 23:00 |
asalkeld | so it was for people that didn't want to get confused with the logic of "desired/min/max size" | 23:01 |
Qiming | so autoscalinggroup seems a special case of instance-group, given that it is making the scaling triggered by a 'policy' thing? | 23:01 |
Qiming | got it | 23:01 |
Qiming | it is a implementation details problem | 23:02 |
asalkeld | instance group basically is an autoscaling group, just with less options | 23:02 |
asalkeld | so yeah one issue is our resource types are user facing | 23:03 |
asalkeld | so we have 2 issues | 23:03 |
asalkeld | development view and user view | 23:03 |
asalkeld | mixed together | 23:03 |
Qiming | right | 23:03 |
asalkeld | my suggestion is to not have much logic in the class it's self | 23:04 |
asalkeld | but rather a library of functions that you use to implement the class | 23:04 |
Qiming | okay, let me try work on that and add you to the reviewer list | 23:05 |
asalkeld | cool | 23:05 |
Qiming | should I revise the specs as a first step? | 23:05 |
asalkeld | you will see the unit tests will be lovely | 23:05 |
asalkeld | (actually look like unit tests) | 23:05 |
Qiming | I am already loving the split of testcases | 23:05 |
asalkeld | Qiming, imo do the code first and make sure you can do what we want | 23:06 |
asalkeld | then update the spec | 23:06 |
Qiming | okay, got it | 23:06 |
asalkeld | the spec is both design and a statement of work | 23:06 |
Qiming | alright, making it consistent with the code is what I cared | 23:07 |
asalkeld | once we see the code, i see no problem with changing the design as long as we update the spec later | 23:07 |
asalkeld | yip | 23:07 |
asalkeld | Qiming, i am sorry you got messed around | 23:08 |
asalkeld | certainly not my intension | 23:08 |
Qiming | I'm feeling happy that I can move on | 23:08 |
asalkeld | cool | 23:08 |
asalkeld | i think at summit there is a review of the spec process, as this seems to happen quite a bit | 23:09 |
asalkeld | and it's frustrating | 23:09 |
Qiming | it is normal, people always have different opinions | 23:10 |
asalkeld | https://etherpad.openstack.org/p/kilo-crossproject-summit-topics | 23:10 |
asalkeld | line: 69 | 23:10 |
Qiming | will read it later | 23:11 |
Qiming | one more thing | 23:11 |
asalkeld | yip | 23:11 |
Qiming | about cross-project discussions | 23:11 |
asalkeld | yeah, got any suggestions | 23:11 |
Qiming | previously there had been a discussion about the interaction between Heat and Ceilometer | 23:11 |
Qiming | it was about trust creation, shardy and therve know the details | 23:12 |
asalkeld | ok, it would also be nice to have qazar alarms | 23:12 |
Qiming | I don't know if it has been settled, or should it appear in the agenda | 23:12 |
asalkeld | or notification alarms | 23:12 |
asalkeld | well since it's just 2 projects, maybe we could just grab some of the ceilometer guys and have a quick chat ? | 23:13 |
Qiming | I'm interested in the conclusion because I want to experiment having instances sent out heartbeat to ceilometer directly | 23:13 |
Qiming | which would be a OpenStack native agent, replacing the cfn-push-stats script | 23:14 |
asalkeld | Qiming, well huge +1 to a native metric sender | 23:14 |
asalkeld | maybe that just needs a ceilometer spec and someone to do it | 23:15 |
Qiming | the agent need two things to do its job: credentials, ceilometer endpoint | 23:15 |
asalkeld | (and agreement from us on the approach) | 23:15 |
asalkeld | yip | 23:15 |
Qiming | current solution that creates a stack domain user won't work, because ceilometer knows nothing about it | 23:16 |
asalkeld | we need to be able to send at the user that created the stack | 23:17 |
asalkeld | so ceilo can match the metrics | 23:18 |
Qiming | that's why I'm eager to know the outputs/conclusions from the cross-project discussion | 23:18 |
asalkeld | yip | 23:18 |
Qiming | if we pass the stack-owner account into an instance, things will become much easier | 23:19 |
Qiming | I think shardy has a lot more to add in that | 23:20 |
Qiming | have to run, ttyl | 23:23 |
asalkeld | later | 23:27 |
*** Qiming has quit IRC | 23:27 | |
*** serg_melikyan has joined #heat | 23:33 | |
*** serg_melikyan has quit IRC | 23:38 | |
*** EricGonczer_ has joined #heat | 23:39 | |
*** sdake_ has joined #heat | 23:43 | |
*** EricGonczer_ has quit IRC | 23:43 | |
*** EricGonczer_ has joined #heat | 23:43 | |
*** EricGonczer_ has quit IRC | 23:44 | |
*** tomek_adamczewsk has quit IRC | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!