| alexpilotti | sdake: hi | 00:00 |
|---|---|---|
| alexpilotti | sdake: I put together a wiki page doc with teh Windows specific Heat issues: http://wiki.cloudbase.it/heat-windows | 00:00 |
| *** gokrokve has joined #heat | 00:09 | |
| *** matsuhashi has joined #heat | 00:12 | |
| *** arbylee1 has quit IRC | 00:13 | |
| *** alexpilotti has quit IRC | 00:15 | |
| *** david-lyle has quit IRC | 00:16 | |
| *** david-lyle has joined #heat | 00:20 | |
| *** faramir1 has joined #heat | 00:24 | |
| *** achampion has joined #heat | 00:29 | |
| *** mikal_ is now known as mikal | 00:29 | |
| *** nosnos has joined #heat | 00:33 | |
| *** nosnos_ has joined #heat | 00:44 | |
| *** nosnos has quit IRC | 00:47 | |
| *** asalkeld has quit IRC | 00:48 | |
| *** pvaneck has quit IRC | 00:52 | |
| *** Tross has quit IRC | 00:53 | |
| *** nosnos_ has quit IRC | 00:54 | |
| *** nosnos has joined #heat | 00:54 | |
| *** gokrokve has quit IRC | 00:55 | |
| *** david_lyle_ has joined #heat | 00:55 | |
| *** Tross has joined #heat | 00:55 | |
| *** sjmc7 has quit IRC | 00:56 | |
| *** david-lyle has quit IRC | 00:59 | |
| *** asalkeld has joined #heat | 01:08 | |
| *** asalkeld has quit IRC | 01:14 | |
| *** asalkeld has joined #heat | 01:14 | |
| *** arbylee has joined #heat | 01:21 | |
| *** mestery has joined #heat | 01:25 | |
| *** mestery has quit IRC | 01:26 | |
| *** mestery has joined #heat | 01:26 | |
| *** TravT has quit IRC | 01:27 | |
| *** tomek_adamczewsk has joined #heat | 01:31 | |
| *** asalkeld has quit IRC | 01:31 | |
| sdake | alexpilotti sweet I'll have a read in the morning | 01:35 |
| *** tomek_adamczewsk has quit IRC | 01:38 | |
| *** david-lyle has joined #heat | 01:40 | |
| openstackgerrit | Ryo Miki proposed a change to openstack/heat: Add handle_update for OS::Neutron::RouterL3Agent https://review.openstack.org/69303 | 01:42 |
| *** david_lyle_ has quit IRC | 01:44 | |
| *** john-n-seattle2 has left #heat | 01:45 | |
| *** asalkeld has joined #heat | 01:45 | |
| *** mestery has quit IRC | 01:47 | |
| *** gokrokve has joined #heat | 01:47 | |
| *** nati_uen_ has joined #heat | 01:59 | |
| *** nati_uen_ has quit IRC | 01:59 | |
| *** gokrokve has quit IRC | 02:00 | |
| *** nati_uen_ has joined #heat | 02:00 | |
| *** alexheneveld has quit IRC | 02:01 | |
| *** alexheneveld has joined #heat | 02:01 | |
| *** alexheneveld has quit IRC | 02:02 | |
| *** nati_ueno has quit IRC | 02:02 | |
| *** gokrokve has joined #heat | 02:03 | |
| *** sgordon has joined #heat | 02:16 | |
| *** david-lyle has quit IRC | 02:29 | |
| *** erkules_ has joined #heat | 02:35 | |
| *** pablosan has quit IRC | 02:35 | |
| *** pablosan has joined #heat | 02:37 | |
| *** erkules has quit IRC | 02:37 | |
| *** WinnieTsang has quit IRC | 02:41 | |
| *** nati_ueno has joined #heat | 02:48 | |
| *** nati_ueno has quit IRC | 02:49 | |
| *** nati_ueno has joined #heat | 02:50 | |
| *** nati_uen_ has quit IRC | 02:51 | |
| *** IlyaE has quit IRC | 02:55 | |
| *** spzala has quit IRC | 03:05 | |
| openstackgerrit | Joe Gordon proposed a change to openstack/heat: Update heat.conf.sample for python-keystoneclient 0.5.0 https://review.openstack.org/70894 | 03:10 |
| *** nati_ueno has quit IRC | 03:11 | |
| *** WinnieTsang has joined #heat | 03:12 | |
| *** nati_ueno has joined #heat | 03:12 | |
| *** nati_ueno has quit IRC | 03:13 | |
| *** nati_ueno has joined #heat | 03:13 | |
| *** Tross has quit IRC | 03:15 | |
| *** faramir1 has quit IRC | 03:15 | |
| *** rpothier has joined #heat | 03:22 | |
| *** gokrokve has quit IRC | 03:26 | |
| *** gokrokve has joined #heat | 03:26 | |
| *** gokrokve has quit IRC | 03:30 | |
| *** IlyaE has joined #heat | 03:30 | |
| *** gokrokve has joined #heat | 03:32 | |
| *** gokrokve has quit IRC | 03:40 | |
| *** david-lyle has joined #heat | 03:57 | |
| *** gokrokve has joined #heat | 04:11 | |
| *** gokrokve_ has joined #heat | 04:13 | |
| *** harlowja is now known as harlowja_away | 04:13 | |
| *** harlowja_away is now known as harlowja | 04:15 | |
| *** gokrokve has quit IRC | 04:16 | |
| *** IlyaE has quit IRC | 04:17 | |
| *** gokrokve_ has quit IRC | 04:19 | |
| *** mattgriffin has joined #heat | 04:30 | |
| *** gokrokve has joined #heat | 04:32 | |
| *** arbylee has quit IRC | 04:32 | |
| *** achampion has quit IRC | 04:38 | |
| *** asalkeld has quit IRC | 04:41 | |
| *** nati_ueno has quit IRC | 04:44 | |
| *** IlyaE has joined #heat | 04:44 | |
| *** topol has joined #heat | 04:44 | |
| *** IlyaE has quit IRC | 04:49 | |
| *** sgordon has quit IRC | 04:54 | |
| *** IlyaE has joined #heat | 05:00 | |
| *** cmyster has joined #heat | 05:02 | |
| *** IlyaE has quit IRC | 05:06 | |
| *** dims has quit IRC | 05:11 | |
| *** mattgriffin has quit IRC | 05:11 | |
| *** dims has joined #heat | 05:13 | |
| *** nkhare has joined #heat | 05:17 | |
| *** erkules_ is now known as erkules | 05:18 | |
| *** slong has joined #heat | 05:19 | |
| *** slong_ has quit IRC | 05:20 | |
| *** coolsvap has joined #heat | 05:21 | |
| *** gokrokve has quit IRC | 05:24 | |
| *** mkollaro has joined #heat | 05:28 | |
| *** achampion has joined #heat | 05:28 | |
| cmyster | mornin' | 05:30 |
| *** gokrokve has joined #heat | 05:33 | |
| *** gokrokve has quit IRC | 05:38 | |
| *** rwsu has quit IRC | 05:41 | |
| *** asalkeld has joined #heat | 05:42 | |
| *** achampion has quit IRC | 05:46 | |
| *** Tross has joined #heat | 05:46 | |
| *** slong has quit IRC | 05:50 | |
| *** killer_prince has joined #heat | 05:52 | |
| *** IlyaE has joined #heat | 05:53 | |
| *** nosnos_ has joined #heat | 06:00 | |
| *** yogesh has joined #heat | 06:01 | |
| *** nosnos has quit IRC | 06:03 | |
| openstackgerrit | Jenkins proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/70517 | 06:07 |
| *** achampion has joined #heat | 06:11 | |
| *** e0ne has joined #heat | 06:23 | |
| *** e0ne has quit IRC | 06:27 | |
| *** yogesh has quit IRC | 06:27 | |
| *** rpothier_ has joined #heat | 06:29 | |
| *** john-n-seattle1 has joined #heat | 06:30 | |
| *** topol_ has joined #heat | 06:30 | |
| *** Tross1 has joined #heat | 06:31 | |
| *** coolsvap_away has joined #heat | 06:32 | |
| *** matsuhashi has quit IRC | 06:33 | |
| *** gokrokve has joined #heat | 06:33 | |
| *** matsuhashi has joined #heat | 06:33 | |
| *** SpamapS_ has joined #heat | 06:33 | |
| *** shadower_ has joined #heat | 06:34 | |
| *** lazzurs has joined #heat | 06:34 | |
| *** IlyaE_ has joined #heat | 06:35 | |
| *** brint_ has joined #heat | 06:37 | |
| *** john-n-s| has joined #heat | 06:37 | |
| *** gokrokve has quit IRC | 06:38 | |
| *** IlyaE has quit IRC | 06:38 | |
| *** Tross has quit IRC | 06:38 | |
| *** asalkeld has quit IRC | 06:38 | |
| *** coolsvap has quit IRC | 06:38 | |
| *** dims has quit IRC | 06:38 | |
| *** cmyster has quit IRC | 06:38 | |
| *** topol has quit IRC | 06:38 | |
| *** rpothier has quit IRC | 06:38 | |
| *** jasond has quit IRC | 06:38 | |
| *** zaneb has quit IRC | 06:38 | |
| *** IgorYozhikov has quit IRC | 06:38 | |
| *** pshchelo_ has quit IRC | 06:38 | |
| *** SpamapS has quit IRC | 06:38 | |
| *** lazzurs_ has quit IRC | 06:38 | |
| *** zpatten has quit IRC | 06:38 | |
| *** blomquisg has quit IRC | 06:38 | |
| *** john-n-s- has quit IRC | 06:38 | |
| *** dteselkin has quit IRC | 06:38 | |
| *** shadower has quit IRC | 06:38 | |
| *** brint has quit IRC | 06:38 | |
| *** mordred has quit IRC | 06:38 | |
| *** dteselkin_ has joined #heat | 06:38 | |
| *** zpatten_ has joined #heat | 06:38 | |
| *** zpatten_ is now known as zpatten | 06:39 | |
| *** IlyaE_ is now known as IlyaE | 06:39 | |
| *** brint_ is now known as brint | 06:39 | |
| *** cmyster has joined #heat | 06:41 | |
| *** cmyster has joined #heat | 06:41 | |
| *** Tross1 has quit IRC | 06:42 | |
| *** amritanshu_RnD has joined #heat | 06:42 | |
| *** Tross has joined #heat | 06:43 | |
| *** amritanshu_RnD is now known as Guest9317 | 06:43 | |
| *** topol_ has quit IRC | 06:44 | |
| *** harlowja is now known as harlowja_away | 06:44 | |
| *** pshchelo_ has joined #heat | 06:44 | |
| *** IgorYozhikov has joined #heat | 06:45 | |
| *** asalkeld has joined #heat | 06:45 | |
| *** zaneb has joined #heat | 06:45 | |
| *** blomquisg has joined #heat | 06:46 | |
| *** skraynev_ is now known as skraynev | 06:50 | |
| skraynev | Morning! | 06:50 |
| skraynev | zaneb, are you here? | 06:50 |
| *** mordred has joined #heat | 06:54 | |
| *** jasond has joined #heat | 06:54 | |
| *** coolsvap_away has quit IRC | 07:04 | |
| *** dims has joined #heat | 07:08 | |
| *** aignatov_ is now known as aignatov | 07:08 | |
| *** achampion has quit IRC | 07:08 | |
| *** yogesh has joined #heat | 07:09 | |
| *** saju_m has joined #heat | 07:12 | |
| *** asalkeld has quit IRC | 07:16 | |
| *** lcheng_ has joined #heat | 07:16 | |
| *** tspatzier has joined #heat | 07:19 | |
| *** slong has joined #heat | 07:19 | |
| *** coolsvap has joined #heat | 07:26 | |
| *** achampion has joined #heat | 07:32 | |
| *** asalkeld has joined #heat | 07:33 | |
| *** gokrokve has joined #heat | 07:33 | |
| *** akuznetsov has joined #heat | 07:37 | |
| *** gokrokve has quit IRC | 07:38 | |
| *** rwsu has joined #heat | 07:42 | |
| *** aignatov is now known as aignatov_ | 07:44 | |
| *** lcheng_ has quit IRC | 07:47 | |
| *** slong has quit IRC | 07:49 | |
| *** rwsu has quit IRC | 07:51 | |
| *** rwsu has joined #heat | 07:53 | |
| *** akuznetsov has quit IRC | 07:54 | |
| *** e0ne has joined #heat | 07:59 | |
| therve | SpamapS_, Hey, still around? | 07:59 |
| *** bvandenh has joined #heat | 08:05 | |
| *** jprovazn has joined #heat | 08:07 | |
| *** killer_prince is now known as lazy_prince | 08:08 | |
| *** IlyaE has quit IRC | 08:08 | |
| shardy | morning all | 08:08 |
| *** jistr has joined #heat | 08:11 | |
| *** pasquier-s has joined #heat | 08:12 | |
| *** akuznetsov has joined #heat | 08:13 | |
| cmyster | mornin | 08:13 |
| *** che-arne has quit IRC | 08:14 | |
| *** asalkeld has quit IRC | 08:20 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Bump python-keystoneclient to version 0.5.0 https://review.openstack.org/70930 | 08:26 |
| shardy | therve: any chance you can take a look at https://review.openstack.org/#/c/70894/ pls? | 08:28 |
| shardy | I think we need that or all currently queued merges will fail on the hacking check for heat.conf | 08:28 |
| shardy | e.g like https://review.openstack.org/#/c/69568/ | 08:29 |
| pshchelo_ | Morning! | 08:33 |
| *** gokrokve has joined #heat | 08:33 | |
| *** coolsvap has quit IRC | 08:36 | |
| *** gokrokve has quit IRC | 08:37 | |
| *** ifarkas has joined #heat | 08:41 | |
| skraynev | shardy: AFAIK, we decided to do sync update with Jenkins bot. | 08:43 |
| skraynev | shardy: I am not sure, that https://review.openstack.org/70930 make sense in this situation. | 08:44 |
| *** aignatov_ is now known as aignatov | 08:45 | |
| *** flwang has joined #heat | 08:49 | |
| shardy | skraynev: Yeah, in general we can wait for the bot, but I have patches waiting for 0.5.0 so was trying to get the bump in quicker | 08:50 |
| shardy | there's no rule AFAIK which says we *can't* bump manually, just that in general we don't have to | 08:50 |
| shardy | I'll abandon it anyway and we can waaait for the bot I guess | 08:51 |
| shardy | skraynev: context is, I've been waiting on this new keystonclient release for two weeks, so I'm a little impatient ;) | 08:52 |
| skraynev | shardy: he-he) you persuade me) I have not anything against this. Only want to know "how it works")) | 08:53 |
| skraynev | shardy, Yeah. AFAIK, your nervous also are related with other keystone patches (I mean moving to v3). Should it be finished for icehouse release? | 08:55 |
| skraynev | IMO, if it really help your works (for me keystone work really important), I would be glad add +1 for this patch ;) | 08:56 |
| *** lsmola has joined #heat | 08:57 | |
| shardy | skraynev: As of 0b073c5 the move to v3 is completed, but I'm still working on the instance-users BP implementation | 09:01 |
| *** slong has joined #heat | 09:02 | |
| shardy | skraynev: I'm expecting it all to be done for Icehouse, yes, but that's assuming I don't hit any more major keystone bugs ;) | 09:02 |
| skraynev | shardy: closed loop : do one thing -> fix bug and again :) | 09:05 |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Update heat.conf.sample https://review.openstack.org/70934 | 09:08 |
| *** flwang has left #heat | 09:12 | |
| skraynev | shardy, one question about keystone and this patch https://review.openstack.org/70934. I have seen some failed tests connected with heat.conf.sample. I have checked patches on review and have not found same changes. Could you confirm, that no any patches with same changes, please? | 09:12 |
| shardy | skraynev: uh, isnt' that a dupe of the patch you just +1'd? | 09:12 |
| *** yogesh has quit IRC | 09:15 | |
| skraynev | shardy. LOL. I have not had sleep enough.. facepalm | 09:17 |
| skraynev | shardy: Arrr. review two patch in parallel and then think, I may fix tests for this patch https://review.openstack.org/#/c/69303/ | 09:18 |
| therve | shardy, Can you explain the context of the conf patch? | 09:20 |
| therve | Oh hacking check | 09:20 |
| therve | That's dumb | 09:20 |
| skraynev | therve. yep. It breaks pep8 check for new patches. | 09:21 |
| *** coolsvap has joined #heat | 09:23 | |
| shardy | therve: Yeah we break anytime there's a new keystoneclient release which adds conf stuff to auth_token | 09:23 |
| therve | It doesn't sound great :) | 09:24 |
| shardy | I don't think auth_token gains options that frequently, so in reality not a huge problem, but yeah it's a bit inconvenient | 09:25 |
| shardy | it would be better if global requirements specified *the* version everyone had to use, rather than the minimum, then the bot could also propose conf changes with a version bump | 09:26 |
| skraynev | I think, it make sense | 09:29 |
| therve | shardy, What do you think of https://blueprints.launchpad.net/heat/+spec/native-signal-api ? | 09:33 |
| *** gokrokve has joined #heat | 09:33 | |
| shardy | therve: looks good, but do we really need both interfaces? | 09:34 |
| shardy | therve: I was thinking we'd just have a signal interface, which the waitcondition notification would also use | 09:35 |
| therve | shardy, You mean CFN and REST? | 09:35 |
| therve | Oh that | 09:35 |
| therve | Hum that's a good question, let me look at the code :) | 09:35 |
| therve | waitcondition returns something, I wonder how useful that is | 09:36 |
| shardy | therve: I was thinking we'd just have a native waitcondition-like resource based on SignalResponder | 09:37 |
| *** gokrokve has quit IRC | 09:38 | |
| therve | shardy, Isn't signal a bit better? | 09:38 |
| shardy | therve: Yes, that's what I'm saying - for the native API, drop waitcondition and just have a signal interface | 09:39 |
| therve | Ah OK I see | 09:39 |
| therve | Yeah sounds good to me. We can always add it later anyway if we feel that we missed something | 09:39 |
| *** newbie|4 has joined #heat | 09:40 | |
| therve | shardy, Unrelatedly, have you seen my comment on https://review.openstack.org/#/c/66304/ ? | 09:41 |
| shardy | therve: hmmm, Ok - for some reason I thought template validation created a parser.Stack and did validate() there | 09:43 |
| shardy | q | 09:43 |
| *** WinnieTsang has quit IRC | 09:44 | |
| *** giulivo has joined #heat | 09:44 | |
| shardy | therve: Actually the OpenStackClients auth_token property will already cache the token via self.keystone() | 09:46 |
| *** e0ne has quit IRC | 09:46 | |
| *** e0ne has joined #heat | 09:46 | |
| shardy | so I guess my performance argument isn't valid | 09:46 |
| therve | Well, I'm still creating a new client | 09:47 |
| *** emamdouh has joined #heat | 09:47 | |
| shardy | therve: but, like where I made the comment, it's *in* a stack object? | 09:48 |
| shardy | so you already have self.stack which has a client object? | 09:49 |
| therve | Indeed | 09:49 |
| therve | But I don't pass any of that? | 09:49 |
| therve | What's in context.auth_token? | 09:49 |
| *** e0ne_ has joined #heat | 09:50 | |
| *** e0ne has quit IRC | 09:50 | |
| shardy | Depends on the auth middleware, but it should contain a token is using auth_token | 09:50 |
| shardy | s/is/if | 09:50 |
| therve | Right, so not for user/password | 09:51 |
| therve | So your concern is right in that case, we're making an additional keystone request | 09:51 |
| shardy | But that validate_template you mentioned in service.py, that isn't doing constraint validation is it? | 09:52 |
| shardy | we only validate the constraints when we're passed parameters (at stack create time), when we'll have a stack object? | 09:52 |
| therve | Well we validate the schema of the constraints | 09:53 |
| therve | Like default values | 09:53 |
| shardy | Hmm, OK, well maybe we go with this for now and consider how it might be refactored later | 09:55 |
| shardy | some of the interfaces just seem a bit cumbersome | 09:55 |
| therve | Agreed. I like the separation of concerns, though. Having the stack there wouldn't feel right. | 09:57 |
| shardy | Yeah, maybe validate_template should be creating a parser.Stack (but only calling validate()) and passing the Clients object into Properties | 10:00 |
| shardy | but we figure that out later, if you're happy to go ahead with the current approach | 10:00 |
| therve | I think we can try that, it's not perfect but it has the benefit of being simple | 10:01 |
| therve | The keypair constraint patch would be trivial if it wasn't for the test fixes :) | 10:02 |
| *** e0ne_ has quit IRC | 10:02 | |
| *** e0ne has joined #heat | 10:02 | |
| shardy | Hah, every patch would be simple if it weren't for those pesky tests ;) | 10:02 |
| shardy | I had to change my whole approach to instance-users because my first attempt broke nearly every test | 10:03 |
| *** sergmelikyan has joined #heat | 10:04 | |
| *** alexheneveld has joined #heat | 10:06 | |
| *** e0ne has quit IRC | 10:06 | |
| *** e0ne has joined #heat | 10:07 | |
| *** e0ne has quit IRC | 10:10 | |
| *** e0ne has joined #heat | 10:10 | |
| *** aignatov is now known as aignatov_ | 10:13 | |
| *** metabro has quit IRC | 10:22 | |
| *** sergmelikyan has quit IRC | 10:24 | |
| *** yogesh has joined #heat | 10:25 | |
| *** jistr has quit IRC | 10:27 | |
| *** yogesh has quit IRC | 10:30 | |
| *** coolsvap has quit IRC | 10:30 | |
| *** gokrokve has joined #heat | 10:33 | |
| *** gokrokve has quit IRC | 10:37 | |
| *** mkollaro has quit IRC | 10:42 | |
| *** slong has quit IRC | 10:44 | |
| *** lazy_prince has quit IRC | 10:46 | |
| *** cody-somerville has joined #heat | 10:46 | |
| *** jamieh has joined #heat | 10:49 | |
| *** jistr has joined #heat | 10:50 | |
| *** e0ne has quit IRC | 10:50 | |
| *** e0ne has joined #heat | 10:50 | |
| *** che-arne has joined #heat | 10:52 | |
| *** russellb has joined #heat | 11:00 | |
| *** aignatov_ is now known as aignatov | 11:01 | |
| *** asalkeld has joined #heat | 11:02 | |
| *** e0ne has quit IRC | 11:10 | |
| *** ppetit has joined #heat | 11:24 | |
| *** sergmelikyan has joined #heat | 11:25 | |
| *** david-lyle has quit IRC | 11:29 | |
| *** nosnos_ has quit IRC | 11:30 | |
| *** jamieh has quit IRC | 11:32 | |
| *** gokrokve has joined #heat | 11:33 | |
| *** gokrokve has quit IRC | 11:38 | |
| *** asalkeld has quit IRC | 11:50 | |
| *** asalkeld has joined #heat | 11:57 | |
| *** tomek_adamczewsk has joined #heat | 11:59 | |
| *** aignatov is now known as aignatov_ | 12:00 | |
| *** rpothier_ has quit IRC | 12:02 | |
| *** nkhare has quit IRC | 12:04 | |
| *** sgordon has joined #heat | 12:06 | |
| *** aignatov_ is now known as aignatov | 12:12 | |
| *** sergmelikyan has quit IRC | 12:14 | |
| *** sergmelikyan has joined #heat | 12:14 | |
| *** e0ne has joined #heat | 12:17 | |
| *** nkhare has joined #heat | 12:20 | |
| *** bada_ has joined #heat | 12:21 | |
| *** tspatzier has quit IRC | 12:21 | |
| *** aignatov is now known as aignatov_ | 12:22 | |
| *** e0ne_ has joined #heat | 12:24 | |
| *** e0ne has quit IRC | 12:24 | |
| *** aignatov_ is now known as aignatov | 12:24 | |
| *** bada has quit IRC | 12:25 | |
| *** e0ne has joined #heat | 12:29 | |
| openstackgerrit | A change was merged to openstack/heat: Update heat.conf.sample for python-keystoneclient 0.5.0 https://review.openstack.org/70894 | 12:29 |
| *** e0ne__ has joined #heat | 12:30 | |
| *** e0ne___ has joined #heat | 12:31 | |
| *** e0ne__ has quit IRC | 12:31 | |
| *** asalkeld has quit IRC | 12:31 | |
| *** e0ne_ has quit IRC | 12:32 | |
| *** e0ne___ has quit IRC | 12:32 | |
| *** e0ne_ has joined #heat | 12:32 | |
| *** nkhare has quit IRC | 12:33 | |
| *** gokrokve has joined #heat | 12:33 | |
| openstackgerrit | Jenkins proposed a change to openstack/heat: Updated from global requirements https://review.openstack.org/70965 | 12:33 |
| *** e0ne has quit IRC | 12:34 | |
| *** matsuhashi has quit IRC | 12:36 | |
| *** matsuhashi has joined #heat | 12:36 | |
| *** gokrokve has quit IRC | 12:37 | |
| *** matsuhashi has quit IRC | 12:41 | |
| *** sgordon has quit IRC | 12:42 | |
| *** tspatzier has joined #heat | 13:04 | |
| *** e0ne has joined #heat | 13:05 | |
| *** e0ne_ has quit IRC | 13:08 | |
| *** e0ne_ has joined #heat | 13:13 | |
| openstackgerrit | Pablo Andres Fuente proposed a change to openstack/heat: HOT templates get_param allows extra attributes https://review.openstack.org/70710 | 13:16 |
| *** e0ne has quit IRC | 13:16 | |
| *** sgordon has joined #heat | 13:17 | |
| *** alexpilotti has joined #heat | 13:18 | |
| *** jprovazn has quit IRC | 13:20 | |
| *** kebray has joined #heat | 13:27 | |
| *** lekha has joined #heat | 13:28 | |
| *** jdob has joined #heat | 13:30 | |
| *** achampion has quit IRC | 13:31 | |
| *** pafuent has joined #heat | 13:31 | |
| *** gokrokve has joined #heat | 13:33 | |
| *** andersonvom has joined #heat | 13:33 | |
| *** aweiteka has joined #heat | 13:34 | |
| *** andersonvom has quit IRC | 13:35 | |
| *** andersonvom has joined #heat | 13:36 | |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Alter stack_count_all_by_tenant to stack_count_all https://review.openstack.org/70853 | 13:38 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Replace stack_get_all_by_tenant with stack_get_all https://review.openstack.org/70852 | 13:38 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Unscoped List Stacks https://review.openstack.org/63041 | 13:38 |
| *** gokrokve has quit IRC | 13:38 | |
| *** viktors has joined #heat | 13:41 | |
| *** saju_m has quit IRC | 13:41 | |
| *** jprovazn has joined #heat | 13:44 | |
| *** ifarkas has quit IRC | 13:51 | |
| *** pasquier-s_ has joined #heat | 13:51 | |
| *** ifarkas has joined #heat | 13:52 | |
| *** rpothier has joined #heat | 14:00 | |
| *** rpothier has quit IRC | 14:01 | |
| openstackgerrit | Thomas Herve proposed a change to openstack/heat: Implement native signal API https://review.openstack.org/70986 | 14:02 |
| *** rpothier has joined #heat | 14:02 | |
| *** dims has quit IRC | 14:03 | |
| *** sgordon has quit IRC | 14:04 | |
| *** cmyster has quit IRC | 14:05 | |
| *** sgordon has joined #heat | 14:05 | |
| *** lsmola has quit IRC | 14:05 | |
| *** tspatzier has quit IRC | 14:13 | |
| openstackgerrit | Pablo Andres Fuente proposed a change to openstack/heat: HOT templates get_param allows extra attributes https://review.openstack.org/70710 | 14:14 |
| *** mkollaro has joined #heat | 14:18 | |
| *** achampion has joined #heat | 14:22 | |
| *** lsmola has joined #heat | 14:22 | |
| *** jasond has quit IRC | 14:29 | |
| *** jcru has joined #heat | 14:30 | |
| *** sabeen has joined #heat | 14:33 | |
| *** gokrokve has joined #heat | 14:33 | |
| *** andersonvom has quit IRC | 14:34 | |
| *** spzala has joined #heat | 14:34 | |
| *** andersonvom has joined #heat | 14:34 | |
| *** akuznetsov has quit IRC | 14:35 | |
| *** vijendar has joined #heat | 14:36 | |
| *** gokrokve has quit IRC | 14:38 | |
| *** arbylee has joined #heat | 14:39 | |
| *** lsmola has quit IRC | 14:41 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Add personality files property to Server resource https://review.openstack.org/68685 | 14:42 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Make Server compatible with Nova Key Pair resource https://review.openstack.org/68230 | 14:42 |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Native Nova Server compatibility for Cloud Server https://review.openstack.org/58043 | 14:42 |
| *** jasond has joined #heat | 14:42 | |
| *** coolsvap has joined #heat | 14:42 | |
| *** tspatzier has joined #heat | 14:46 | |
| *** jmckind has joined #heat | 14:47 | |
| *** wchrisj has quit IRC | 14:48 | |
| *** jdob_ has joined #heat | 14:49 | |
| *** jdob has quit IRC | 14:49 | |
| *** lsmola has joined #heat | 14:53 | |
| *** tspatzier has quit IRC | 14:55 | |
| *** sgordon has quit IRC | 14:58 | |
| *** radez_g0n3 is now known as radez | 14:59 | |
| *** IlyaE has joined #heat | 14:59 | |
| *** lsmola has quit IRC | 15:05 | |
| shardy | jpeeler: Hey, around? | 15:08 |
| *** mattgriffin has joined #heat | 15:08 | |
| *** sgordon has joined #heat | 15:10 | |
| *** jdob has joined #heat | 15:13 | |
| *** pafuent has left #heat | 15:13 | |
| *** tspatzier has joined #heat | 15:15 | |
| *** lsmola has joined #heat | 15:18 | |
| *** jdob_ has quit IRC | 15:19 | |
| *** jistr has quit IRC | 15:23 | |
| *** jistr has joined #heat | 15:23 | |
| *** tspatzier has quit IRC | 15:24 | |
| sdake | morning | 15:28 |
| *** gokrokve has joined #heat | 15:29 | |
| *** gokrokve_ has joined #heat | 15:30 | |
| *** Guest9317 has quit IRC | 15:31 | |
| shardy | Hi sdake | 15:33 |
| *** gokrokve has quit IRC | 15:33 | |
| *** wchrisj has joined #heat | 15:34 | |
| *** david-lyle has joined #heat | 15:34 | |
| sdake | hey shardy | 15:36 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add API support for stack preview https://review.openstack.org/68406 | 15:36 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add StackResource specific preview behavior https://review.openstack.org/68407 | 15:36 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add preview for LoadBalancer https://review.openstack.org/68412 | 15:36 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add engine preview stack with Template and Params https://review.openstack.org/65576 | 15:36 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add preview for NestedStack https://review.openstack.org/68408 | 15:36 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add preview for ResourceGroup https://review.openstack.org/68409 | 15:36 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add preview for InstanceGroup https://review.openstack.org/68410 | 15:36 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add preview for TemplateResource https://review.openstack.org/68411 | 15:36 |
| *** TravT has joined #heat | 15:36 | |
| jpeeler | shardy: hey i'm here | 15:37 |
| alexpilotti | sdake: hi | 15:37 |
| shardy | jpeeler: Oh Hi, sorry, I was going to ask you to re-approve https://review.openstack.org/#/c/69568/, but it looks like my re approve worked | 15:38 |
| shardy | jpeeler: Thanks for the reviews btw - I don't know if you spotted it, but there one between the two you approved: | 15:40 |
| shardy | https://review.openstack.org/#/c/69569/ | 15:40 |
| jpeeler | yeah i meant to approve that one | 15:42 |
| *** tims has joined #heat | 15:43 | |
| *** tims1 has joined #heat | 15:44 | |
| *** tims has quit IRC | 15:47 | |
| alexpilotti | hi guys, I put together some points related to Heat + Windows: http://wiki.cloudbase.it/heat-windows | 15:49 |
| *** aignatov is now known as aignatov_ | 15:50 | |
| *** jprovazn is now known as jprovazn_afk | 15:51 | |
| alexpilotti | the only urgent point is to decide wether we add Windows support in heat-cfntools or we just go with a separate "fork" of http://aws.amazon.com/developertools/AWS-CloudFormation/4026240853893296 | 15:52 |
| *** hk_peter has joined #heat | 16:04 | |
| therve | jasond: That change you're making for keypairs is pretty bad. Can't we find another way? | 16:05 |
| hk_peter | Hey guy, do you want to join the project Titan team, to develop a mature management tool for openstack? http://peter.kingofcoders.com | 16:05 |
| *** IlyaE has quit IRC | 16:05 | |
| *** pablosan has quit IRC | 16:06 | |
| *** tspatzier has joined #heat | 16:07 | |
| *** pablosan has joined #heat | 16:08 | |
| jasond | therve: if you can do it better, i'll abandon my patch | 16:08 |
| therve | jasond: well thinking out loud, if we introduce a resource reference, it would work, right? | 16:09 |
| jasond | therve: not sure i follow. can you give an example? | 16:11 |
| therve | Sure one sec | 16:11 |
| therve | jasond: http://paste.openstack.org/show/62455/ for example | 16:12 |
| *** randallburt has joined #heat | 16:15 | |
| *** randallburt has quit IRC | 16:15 | |
| *** randallburt has joined #heat | 16:16 | |
| jasond | therve: ah, i see. so did you mean to use a new OS::Nova::Server property, "keypair", in that example? | 16:16 |
| radix | morning | 16:17 |
| therve | jasond: Well yeah, I don't think it would work with the current property | 16:17 |
| *** tspatzier has quit IRC | 16:17 | |
| therve | We could also introduce an attribute name on the keypair resource, but it's a bit backward | 16:17 |
| *** tspatzier has joined #heat | 16:17 | |
| jasond | therve: that was the point of my change, to make it work without introducing a new property | 16:17 |
| therve | jasond: Sure, but the implementation is horrible. | 16:18 |
| jasond | therve: i'm open to suggestions | 16:18 |
| therve | We need to get rid of those iterate the whole stack thing, not add more | 16:18 |
| therve | Well that's what I'm doing :) | 16:19 |
| jasond | so you're saying adding a new property is preferable to iterating the stack? | 16:20 |
| radix | therve: was that "get_resources" a typo for "get_resource"? | 16:20 |
| therve | radix, Yes | 16:20 |
| radix | or is there a new hot function I didn't know about | 16:20 |
| radix | ah ok :) | 16:20 |
| therve | jasond: I'd rather have that yes. It's just my opinion though | 16:21 |
| hk_peter | hey guys, is there any person in charge of developing admin interface for heat in dashbaord? | 16:21 |
| openstackgerrit | Billy Keith Box proposed a change to openstack/heat: Verify setenforce is executable https://review.openstack.org/71019 | 16:21 |
| zaneb | hk_peter: I believe it's tims1 | 16:21 |
| hk_peter | cool, actually i want to develop an admin interface for heat in Titan, checkout my opensource management tool http://peter.kingofcoders.com | 16:22 |
| jasond | therve: okay, thanks. i'll see if steve baker has an opinion. i can't decide which would be less confusing for users | 16:22 |
| therve | jasond: https://etherpad.openstack.org/p/icehouse-summit-heat-exorcism fwiw, I think zaneb would agree with me that it's relevant | 16:22 |
| hk_peter | to see if anybody who familiar with heat can help out the design the interface. | 16:23 |
| zaneb | therve, jasond: do you have link handy to the patch you're discussing? | 16:23 |
| jasond | zaneb: https://review.openstack.org/#/c/68230/ | 16:23 |
| *** tspatzier has quit IRC | 16:23 | |
| zaneb | ta | 16:23 |
| zaneb | jasond: that doesn't solve the problem where it's declared as a Nova keypair resource in a _different_ stack (e.g. passed to a nested stack) | 16:26 |
| zaneb | jasond: which sounds like an extremely common case | 16:26 |
| jasond | ah, good point | 16:26 |
| *** hk_peter has quit IRC | 16:27 | |
| *** pablosan has quit IRC | 16:27 | |
| jasond | zaneb: any opinion on adding a new "keypair" property? | 16:28 |
| *** pafuent has joined #heat | 16:30 | |
| zaneb | jasond: I'm not clear on how that helps... | 16:30 |
| zaneb | actually, looking at http://paste.openstack.org/show/62455/ I'm not actually clear on what it means | 16:30 |
| *** pablosan has joined #heat | 16:31 | |
| zaneb | fwiw I think patch set 2 looked ok | 16:33 |
| jasond | zaneb: also relevant https://review.openstack.org/#/c/70319/ | 16:33 |
| radix | yeah I'm not sure if you're going to reasonably still be able to check for keypairs ever | 16:34 |
| *** IlyaE has joined #heat | 16:34 | |
| *** topol has joined #heat | 16:34 | |
| zaneb | for now, I think radix is probably right :( | 16:35 |
| zaneb | trouble is, our validation code sucks | 16:35 |
| zaneb | what we really need is type-tagging to follow data all the way through the system | 16:36 |
| radix | hmm | 16:36 |
| zaneb | so if we got this value from Ref on a Nova Keypair resource, we know it has type "keypair", even when we don't know the value yet, and we track that all the way through nested stacks &c. &c. | 16:37 |
| radix | yeah, that would be cool | 16:37 |
| radix | zaneb: do you see a use for that outside of validation? | 16:37 |
| zaneb | but we are a looooooong way from being able to implement anything like that | 16:37 |
| zaneb | prolly not, but it would be great for validation | 16:38 |
| zaneb | spoiler alert: I am working on changing how functions are resolved, so that we just stick a lazily-evaluated function object into the parse tree, instead of converting it to a value as soon as we can | 16:39 |
| *** lsmola has quit IRC | 16:39 | |
| zaneb | I think that's a prerequisite for type-checked validation... but it still doesn't get us very close | 16:39 |
| radix | nice | 16:39 |
| jasond | zaneb: i'll revert that review to patchset 3. thanks | 16:39 |
| openstackgerrit | A change was merged to openstack/heat: heat_keystoneclient ensure admin client respects SSL options https://review.openstack.org/69568 | 16:40 |
| therve | It feels weird that removing validation is the answer here | 16:42 |
| zaneb | therve: btw just yesterday we had a user in here asking how to pass security groups into a nested stack, and we had to tell him it's on the exorcism list of things that are not supported | 16:42 |
| therve | jasond: I may miss something, but if you don't introduce a dependency between the resources, you can have a problem | 16:42 |
| jasond | therve: Ref creates a dependency | 16:43 |
| therve | jasond: Do you have a Ref? | 16:43 |
| jasond | therve: see example 2 http://paste.openstack.org/show/62459/ | 16:45 |
| zaneb | Ref returns new_keypair.id | 16:45 |
| *** tomek_adamczewsk has quit IRC | 16:45 | |
| * zaneb didn't know that keypairs had IDs, but that looks like it will do the trick | 16:46 | |
| therve | jasond: So does that work right now? | 16:46 |
| *** SergeyLukjanov is now known as SergeyLukjanov_a | 16:46 | |
| therve | I'm confused | 16:47 |
| *** jistr has quit IRC | 16:47 | |
| jasond | it does work. (reading code to understand how) | 16:48 |
| zaneb | therve: it works except that the validation will barf IIUC | 16:48 |
| zaneb | because the keypair hasn't been created at the time of validation | 16:48 |
| *** pablosan has quit IRC | 16:49 | |
| *** pablosan has joined #heat | 16:49 | |
| *** edmund has joined #heat | 16:49 | |
| therve | Ah so FnGetRefId returns None or something | 16:50 |
| zaneb | something like that | 16:50 |
| zaneb | actually I think it returns the resource name | 16:50 |
| zaneb | have I mentioned that our validation code is terrible? | 16:50 |
| *** mestery has joined #heat | 16:51 | |
| *** mestery has quit IRC | 16:51 | |
| *** lsmola has joined #heat | 16:52 | |
| *** rcleere has joined #heat | 16:52 | |
| *** mestery has joined #heat | 16:53 | |
| openstackgerrit | Pablo Andres Fuente proposed a change to openstack/heat: HOT templates get_param allows extra attributes https://review.openstack.org/70710 | 16:53 |
| therve | Maybe we could override KeyPair FnGetRefId to return None, then the validation would do the right thing I think | 16:55 |
| *** akuznetsov has joined #heat | 16:56 | |
| *** nkhare has joined #heat | 16:58 | |
| *** e0ne_ has quit IRC | 16:59 | |
| *** akuznetsov has quit IRC | 17:00 | |
| *** tspatzier has joined #heat | 17:01 | |
| *** mestery has quit IRC | 17:02 | |
| *** akuznetsov has joined #heat | 17:02 | |
| *** randallburt has quit IRC | 17:03 | |
| *** rockstar has joined #heat | 17:03 | |
| *** mestery has joined #heat | 17:03 | |
| *** mestery has quit IRC | 17:05 | |
| *** mestery has joined #heat | 17:06 | |
| *** bvandenh has quit IRC | 17:06 | |
| radix | except | 17:08 |
| radix | I think IDs can't be non-strings | 17:08 |
| *** slong has joined #heat | 17:09 | |
| zaneb | None usually gets translated to an empty string somewhere in the function resolution process, I think | 17:09 |
| *** ppetit has quit IRC | 17:09 | |
| *** mestery_ has joined #heat | 17:10 | |
| funzo_ | radez: zaneb: therve: it's probably easier for me to have the conversation here so I can get your consolidated input. I would like have support for OS::Nova::Server resources in autoscaling for the icehouse release. I can go the route of doing a simple refactor of scaling groups, but I think it might conflict with the work that is currently going on. | 17:10 |
| radix | funzo_: hmm, I think it should be fine | 17:10 |
| therve | https://blueprints.launchpad.net/heat/+spec/as-lib is where things are noted | 17:11 |
| *** mestery_ has quit IRC | 17:11 | |
| therve | It looks a bit stuck, but I'm sure radix will do something about it | 17:11 |
| *** mestery has quit IRC | 17:11 | |
| radix | hmm | 17:11 |
| radix | so it seems like that could be done independently, right? | 17:12 |
| radix | isn't the main thing that needs to happen a continuation of zane's idea about parameterizing the nested stack's environment? | 17:12 |
| *** mestery has joined #heat | 17:12 | |
| therve | Yeah maybe | 17:12 |
| *** ifarkas has quit IRC | 17:12 | |
| funzo_ | radez: I was just talking with zaneb about some quick paths to accomplish what I'd like to do, but that *might* conflict with something else in the works. | 17:13 |
| zaneb | funzo_: s/radez/radix/ ;) | 17:13 |
| funzo_ | heh, thx | 17:13 |
| radix | I think it's fine | 17:13 |
| funzo_ | radez: i'm SO sorry | 17:13 |
| *** arbylee has quit IRC | 17:14 | |
| funzo_ | radez: therve: zaneb: ok, so you guys are fine with me adding a launchconfig resource that looks like an OS::Nova::Server instead of the AWS::EC2::Instance? | 17:14 |
| funzo_ | lol, radez again | 17:14 |
| radix | hmm | 17:15 |
| zaneb | radix: so I think funzo could implement what he wants by creating a native LaunchConfig resource and subclassing the scaling group to map ScaledResource to a different type in the environment | 17:15 |
| radix | oh, I see | 17:15 |
| radix | zaneb: what about your environment idea? | 17:15 |
| radix | I mean, didn't you suggest allowing the user to override it? | 17:15 |
| zaneb | but that's _not_ going to look like the native resources that therve is working on | 17:15 |
| therve | I'm not working on anything :) | 17:16 |
| radix | but that sounds fine anyway | 17:16 |
| zaneb | radix: only as a demonstration of the mechanism, not as a serious proposal for something to be merged | 17:16 |
| radix | I see | 17:16 |
| funzo_ | zaneb: ideally, whatever ends up upstream would accomplish the same goal | 17:17 |
| funzo_ | zaneb: If this wouldn't be worthy of upstream acceptance, what would be? and will it happen for icehouse? | 17:17 |
| zaneb | right, it's just a question of whether we can get that done for icehouse | 17:17 |
| funzo_ | zaneb: if I get this working asap, can we get it in icehouse, and juno can just have something that replaces it? | 17:18 |
| funzo_ | or is that too much of a support nightmare | 17:18 |
| zaneb | the upstream goal is a superset of what you want, and what you want hasn't up to this point been regarded as on the path to where we're going | 17:18 |
| funzo_ | zaneb: and the superset isn't in the icehouse timeframe, correct? | 17:19 |
| zaneb | I hate creating resource types that we know are not what we want long term and then having to support them forever :/ | 17:19 |
| funzo_ | the bp says icehouse, but therve just said he's not working on it | 17:20 |
| zaneb | funzo_: well, it's targeting for icehouse | 17:20 |
| radix | the majority of the whole AS stuff is post-icehouse | 17:20 |
| zaneb | but looking decidedly questionable | 17:20 |
| radix | er, s/stuff/design/ | 17:20 |
| radix | I pretty much just expect to get the basic library code done | 17:20 |
| therve | funzo_, I'm hoping radix is :) | 17:20 |
| funzo_ | this reminds me of "who's on first" | 17:21 |
| zaneb | radix, therve: so we're saying that we're not expecting to even get native resources for Icehouse? | 17:21 |
| radix | zaneb: I'm not sure if we'll get super useful ones, maybe just an InstanceGroup-like thing | 17:21 |
| zaneb | I think it's fair to say the API part has been punted to Juno | 17:21 |
| therve | zaneb, I'd like to. It's just not progressing well | 17:21 |
| zaneb | is there anything I can do to help it progress? | 17:22 |
| radix | our team at RS grew to help with the work, but I have been doing a lot of introductory stuff and so it's actually slowed things down | 17:22 |
| radix | like last week I was in SF having lots of conversations with cyli and rockstar | 17:22 |
| therve | zaneb, Maybe update the wiki spec so that it reflects what we want to have for icehouse? | 17:23 |
| zaneb | yeah, that's unfortunately inevitable | 17:23 |
| radix | therve: you mean the "Autoscaling" page? | 17:23 |
| therve | radix, yes. Or maybe a new simpler one | 17:23 |
| radix | ok | 17:23 |
| funzo_ | zaneb: radix: therve: ok so for the icehouse timeframe, to achieve native resource autoscaling... take the approach that won't match the intended upstream impl? | 17:27 |
| funzo_ | ..or stay tuned. | 17:27 |
| therve | funzo_, I'm not sure what that means | 17:27 |
| zaneb | funzo_: luckily resource types are pluggable :) | 17:28 |
| funzo_ | therve: I'm asking if I need to have autoscaling work with OS::Nova::Server resources in the icehouse timeframe, should I take the path of implementation zaneb mentioned, or is there something else that will be done for icehouse | 17:28 |
| *** alexheneveld has quit IRC | 17:29 | |
| funzo_ | zaneb: right, but taking this approach potentially puts us in a long term resource support place that we don't want to be in, as you mentioned. | 17:29 |
| funzo_ | my end goal is two fold, to wow people with demos and to be able to say that OS now supports native resource autoscaling. | 17:29 |
| zaneb | funzo_: that's the advantage of allowing out-of-tree plugins; you can write it and not have to support it upstream forever | 17:30 |
| funzo_ | zaneb: ok - so you're saying implement the resource the way you've mentioned and that should fall in line with the existing plugin architecture? | 17:31 |
| funzo_ | zaneb: sorry for my ignorance, I'll have to look into how that works | 17:31 |
| zaneb | funzo_: I'm saying you can implement the resource yourself without having to get it accepted into upstream Heat to integrate it (just drop it into /usr/lib/heat/plugins or whatever it is) | 17:32 |
| funzo_ | zaneb: k, I'll start with that. and just so i'm understanding this correctly, there is nothing we can ship with the icehouse release that would accomplish native resource autoscaling, correct? | 17:33 |
| *** mestery_ has joined #heat | 17:34 | |
| zaneb | yes, schedule pressure is making it look very unlikely that we would be able to ship anything like this in time for icehouse :/ | 17:34 |
| *** jrist has quit IRC | 17:35 | |
| funzo_ | okie doke, I'll go with the plugin approach. thanks for the time guys | 17:35 |
| *** newbie|4 has quit IRC | 17:37 | |
| andersonvom | therve: you still around? | 17:37 |
| therve | Yes | 17:38 |
| *** mestery has quit IRC | 17:38 | |
| andersonvom | therve: could you take a look at my comments re the patch https://review.openstack.org/#/c/68746/ and explain a little what you had in mind? | 17:39 |
| *** TravT has quit IRC | 17:40 | |
| *** TravT has joined #heat | 17:40 | |
| therve | andersonvom, I was thinking like a real python variable. Don't we import code when generating doc? | 17:40 |
| *** arbylee1 has joined #heat | 17:41 | |
| andersonvom | therve: we do, but we'd have then a global variable that would be used? | 17:41 |
| andersonvom | therve: we use conf.py to set up what we need to generate the docs, if you want to take a look | 17:42 |
| *** SpamapS_ is now known as SpamapS | 17:43 | |
| *** andrew_plunk has joined #heat | 17:43 | |
| therve | I'm sorry, I'm not really proposing a solution | 17:43 |
| therve | Just that having a conf variable to fix our doc process doesn't feel right | 17:44 |
| *** akuznetsov has quit IRC | 17:44 | |
| abramley | I have a quick heat template question that hopefully somebody can help me with. In my template I am creating several AWS::RDS::DBInstance resources and then within another resource I want to obtain the address of the DBInstance so I am using { "Fn::GetAtt" : [ <resource>, "Endpoint.Address" ] } - but this is giving me an empty string rather than the address of the DBInstance. | 17:44 |
| SpamapS | therve: here now. | 17:44 |
| abramley | Is this something that is covered by tests and known to work? I did also try one of the sample templates and had the same issue. | 17:45 |
| therve | SpamapS, Hi. I ended up thinking really hard and answered your email :) | 17:45 |
| andersonvom | therve: maybe it's not the best way to go, but I can't think of any other way to go about it, since having to make all resources aware of the fact that we may want to generate docs is bad, IMO. | 17:46 |
| SpamapS | therve: cool, just now starting to read. | 17:46 |
| *** sgran has quit IRC | 17:46 | |
| *** julienvey has quit IRC | 17:46 | |
| *** stannie has quit IRC | 17:46 | |
| *** IlyaE has quit IRC | 17:46 | |
| therve | andersonvom, what about installing the requirements? | 17:48 |
| *** stannie has joined #heat | 17:48 | |
| *** sgran has joined #heat | 17:48 | |
| *** jrist has joined #heat | 17:49 | |
| *** julienvey has joined #heat | 17:49 | |
| *** IlyaE has joined #heat | 17:49 | |
| *** pvaneck has joined #heat | 17:49 | |
| andersonvom | therve: I don't know I like that idea that much. 1) I don't think folks running docs.openstack.org would like having to install all dependencies to generate the docs there. 2) from a dev's perspective, I wouldn't like to have to install everything so that I can read about it and figure out whether I want to use it or not. | 17:49 |
| andersonvom | #2 being in regards to generating the docs locally, of course. | 17:50 |
| *** akuznetsov has joined #heat | 17:50 | |
| therve | I don't think 2) stands | 17:50 |
| therve | We should ask for 1) | 17:51 |
| radix | hm | 17:51 |
| *** akuznetsov has quit IRC | 17:51 | |
| radix | oh, I missed that response | 17:52 |
| andersonvom | therve: why not? when we start having several different plugins in contrib/, is it really reasonable to expect someone to install all requirement from all plugins, just so they can have docs for them? even though they could be generated without installing these deps? | 17:53 |
| therve | andersonvom, docs are hosted. You need to install sphinx to generate them locally, you can install some dependencies | 17:54 |
| *** gokrokve_ has quit IRC | 17:54 | |
| therve | You need to install all other clients afaict | 17:55 |
| *** akuznetsov has joined #heat | 17:55 | |
| *** gokrokve_ has joined #heat | 17:55 | |
| *** jrist has quit IRC | 17:56 | |
| arbylee1 | therve sphinx is already part of test-requirements though | 17:56 |
| andersonvom | therve: well... but you need sphynx to generate anything. you don't need pyrax, or docker-py, or whatever other module that may be required by any plugin. | 17:56 |
| *** IlyaE has quit IRC | 17:56 | |
| arbylee1 | therve if you take a look at clients.py, swiftclient, neutronclient, etc are all optionally installed | 17:57 |
| *** sgordon has quit IRC | 17:57 | |
| therve | arbylee1, Are they if you generate docs though? | 17:57 |
| andersonvom | therve: if they're not installed, docs won't be generated for them either. | 17:57 |
| therve | Right | 17:58 |
| therve | Anyway, that's just my opinion, you can find 2 other cores to approve it | 17:58 |
| *** harlowja_away is now known as harlowja | 17:59 | |
| *** randallburt has joined #heat | 18:01 | |
| *** nkhare has quit IRC | 18:02 | |
| *** TravT has quit IRC | 18:03 | |
| *** nati_ueno has joined #heat | 18:04 | |
| *** WinnieTsang has joined #heat | 18:05 | |
| *** SergeyLukjanov_a is now known as SergeyLukjanov | 18:05 | |
| *** mattgriffin1 has joined #heat | 18:05 | |
| pafuent | abramley: I think Endpoint.Address isn't a valid argument, and for that reason you are getting an empty string | 18:06 |
| *** mattgriffin has quit IRC | 18:07 | |
| pafuent | abramley: You can try this to access the PublicIP of a DB instance | 18:07 |
| pafuent | abramley: https://github.com/openstack/heat-templates/blob/master/cfn/F18/WordPress_Single_Instance.template#L137 | 18:07 |
| *** slong has quit IRC | 18:07 | |
| *** e0ne has joined #heat | 18:08 | |
| abramley | I looked in the AWS_RDS_DBInstance.yaml template and at the bottom I see: | 18:08 |
| abramley | Outputs: | 18:08 |
| abramley | Endpoint.Address: {'Fn::GetAtt': [DatabaseInstance, PublicIp]} | 18:08 |
| abramley | Endpoint.Port: {Ref: Port} | 18:08 |
| pafuent | abramley: Yes, that are Outputs, not attributes. Fn::GetAtt is used to access resource atributtes | 18:09 |
| *** jrist has joined #heat | 18:09 | |
| abramley | pafuent - I did try the PublicIp too with no success | 18:10 |
| shardy | pafuent: they are mapped to attributes, it's a provider template | 18:10 |
| pafuent | shardy: Ahhh. I didn't know that | 18:10 |
| abramley | pafuent - I was wondering if this was something related to it being a nested template ? | 18:10 |
| *** akuznetsov has quit IRC | 18:11 | |
| shardy | abramley: If you create a stack with just an AWS::EC2::Instance in it, do you get an IP for PublicIp? | 18:11 |
| pafuent | abramley: I think someone talk about that in IRC yesterday. Maybe someone else could help you. I don't know about nested templates. | 18:11 |
| pafuent | abramley: Sorry | 18:11 |
| radez | funzo_: nice to hear from you... I just ate lunch with erosen, we decided you're going to join us for a tough mudder soon | 18:12 |
| funzo_ | radez: sweet! | 18:12 |
| abramley | shardy - I will try - there is also the possibility that this is an issue with my installation since I am actually using heat that is installed in the Trove Redstack dev environment | 18:13 |
| *** tspatzier has quit IRC | 18:15 | |
| *** tspatzier has joined #heat | 18:15 | |
| *** tspatzier has quit IRC | 18:15 | |
| *** coolsvap is now known as coolsvap_away | 18:16 | |
| *** akuznetsov has joined #heat | 18:28 | |
| *** akuznetsov has quit IRC | 18:29 | |
| *** rpothier_ has joined #heat | 18:31 | |
| *** e0ne_ has joined #heat | 18:33 | |
| *** akuznetsov has joined #heat | 18:34 | |
| *** sgordon has joined #heat | 18:35 | |
| *** e0ne has quit IRC | 18:35 | |
| *** akuznetsov has quit IRC | 18:35 | |
| *** rpothier has quit IRC | 18:35 | |
| *** akuznetsov has joined #heat | 18:35 | |
| *** yogesh has joined #heat | 18:35 | |
| *** gokrokve_ has quit IRC | 18:35 | |
| *** akuznetsov has joined #heat | 18:36 | |
| *** akuznetsov has quit IRC | 18:36 | |
| *** gokrokve has joined #heat | 18:36 | |
| *** akuznetsov has joined #heat | 18:36 | |
| *** che-arne has quit IRC | 18:37 | |
| *** mestery_ has quit IRC | 18:38 | |
| *** gokrokve has quit IRC | 18:41 | |
| *** sdake has quit IRC | 18:42 | |
| *** arbylee3 has joined #heat | 18:42 | |
| *** jistr has joined #heat | 18:42 | |
| *** arbylee1 has quit IRC | 18:42 | |
| *** akuznetsov has quit IRC | 18:42 | |
| *** sdake has joined #heat | 18:43 | |
| *** sdake has quit IRC | 18:43 | |
| *** sdake has joined #heat | 18:43 | |
| *** dims has joined #heat | 18:44 | |
| *** jprovazn_afk is now known as jprovazn | 18:50 | |
| *** IlyaE has joined #heat | 18:50 | |
| *** arbylee1 has joined #heat | 18:51 | |
| *** gokrokve has joined #heat | 18:55 | |
| *** sdake has quit IRC | 18:55 | |
| *** arbylee3 has quit IRC | 18:56 | |
| *** arbylee2 has joined #heat | 18:59 | |
| *** arbylee2 has quit IRC | 18:59 | |
| *** arbylee2 has joined #heat | 18:59 | |
| *** gokrokve has quit IRC | 19:00 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Make Server compatible with Nova Key Pair resource https://review.openstack.org/68230 | 19:02 |
| openstackgerrit | A change was merged to openstack/heat: heat_keystoneclient: Move v3 path logic to constructor https://review.openstack.org/69569 | 19:02 |
| openstackgerrit | A change was merged to openstack/heat: heat_keystoneclient: abstract admin_client to a property https://review.openstack.org/69570 | 19:02 |
| *** sabeen1 has joined #heat | 19:03 | |
| *** arbylee1 has quit IRC | 19:03 | |
| abramley | shardy - if I change my template to use AWS::EC2::Instance instead of RDS::DBInstance then it does work if I do Fn::GetAtt with PublicIp | 19:03 |
| *** sabeen has quit IRC | 19:04 | |
| *** mestery has joined #heat | 19:04 | |
| *** gokrokve has joined #heat | 19:06 | |
| *** sabeen1 has quit IRC | 19:08 | |
| *** sabeen has joined #heat | 19:08 | |
| *** e0ne_ has quit IRC | 19:09 | |
| *** gokrokve has quit IRC | 19:11 | |
| *** gokrokve has joined #heat | 19:14 | |
| *** shadower_ is now known as shadower | 19:18 | |
| *** sdake has joined #heat | 19:19 | |
| *** sdake has quit IRC | 19:19 | |
| *** sdake has joined #heat | 19:19 | |
| *** skraynev_ has joined #heat | 19:21 | |
| skraynev_ | zaneb, are you around? | 19:22 |
| zaneb | skraynev_: hi | 19:22 |
| skraynev_ | zaneb: hi) | 19:23 |
| *** WinnieTsang has left #heat | 19:23 | |
| skraynev_ | zaneb: I only want ask your opinion about https://review.openstack.org/#/c/70358/ | 19:23 |
| zaneb | skraynev_: but I have an irc meeting with ttx in 7 minutes... | 19:23 |
| zaneb | looking... | 19:24 |
| skraynev_ | zaneb: now or after 7 min? | 19:24 |
| zaneb | meeting starts in... 6 minutes | 19:24 |
| *** alexheneveld has joined #heat | 19:24 | |
| zaneb | so as long as this is a <6 minute question, no problem ;) | 19:25 |
| skraynev_ | Oh. ok. I try | 19:25 |
| skraynev_ | What do you think about this solution for validation outputs? | 19:25 |
| skraynev_ | is it possible realize with other aproach? | 19:26 |
| *** TravT has joined #heat | 19:26 | |
| skraynev_ | Only this two question. I will be thankful if you look this patch and then, when you will have gree time give me answers | 19:27 |
| zaneb | skraynev_: is there a reason that just calling resolve_runtime_data wouldn't work? | 19:27 |
| zaneb | obviously it wouldn't return any useful data, but it should find any errors like this | 19:28 |
| zaneb | fwiw what I am currently working on is to collapse the static/runtime thing into a single pass with lazily-evaluated functions | 19:29 |
| zaneb | if that works then this code would end up getting removed/replaced | 19:29 |
| skraynev_ | it works, but after stack creatung | 19:31 |
| *** vijendar has quit IRC | 19:31 | |
| *** alexheneveld has quit IRC | 19:31 | |
| *** klwilson has quit IRC | 19:31 | |
| skraynev_ | hm. it's interesting. I think, that now meeting time for you unfortunatelly. Could we continue it tomorrow or through email? | 19:33 |
| skraynev_ | because it's difficult catch you, while in my time zone night time don't come | 19:33 |
| zaneb | ttx is not about, so we can continue the discussion if you are available now | 19:33 |
| *** klwilson has joined #heat | 19:33 | |
| zaneb | skraynev_: how is check_resource_attributes different from template.resolve_attributes? I'm concerned that we are duplicating code here | 19:35 |
| skraynev_ | zaneb: in template.resolve_attributes we check, that resource currently in one in correct state. | 19:37 |
| zaneb | oh :( | 19:37 |
| skraynev_ | if I try to call this function without changes > GetAttr function will be skipped | 19:37 |
| skraynev_ | so, for me the main problem is checking before we start create. | 19:38 |
| skraynev_ | hm, in other side: May I try call template.resolve_attributes during creation process? | 19:39 |
| skraynev_ | I am not sure, that know right place for it. | 19:39 |
| *** alexheneveld has joined #heat | 19:41 | |
| *** sdake has quit IRC | 19:42 | |
| *** sdake has joined #heat | 19:43 | |
| *** sdake has quit IRC | 19:44 | |
| *** sdake has joined #heat | 19:44 | |
| *** sdake has quit IRC | 19:44 | |
| *** sdake has joined #heat | 19:44 | |
| *** sdake has quit IRC | 19:44 | |
| *** sdake has joined #heat | 19:44 | |
| *** alexheneveld has quit IRC | 19:45 | |
| zaneb | shardy: ping re https://blueprints.launchpad.net/heat/+spec/x-auth-trust if you are about | 19:46 |
| *** randallburt has quit IRC | 19:47 | |
| *** IlyaE has quit IRC | 19:47 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Handle API limit exception in OS::Trove::Instance https://review.openstack.org/71082 | 19:49 |
| sdake | anyone seen this problem with devstack + nova networking http://paste.fedoraproject.org/74412/54301613 | 19:51 |
| *** aignatov_ is now known as aignatov | 19:52 | |
| *** randallburt has joined #heat | 19:53 | |
| zaneb | no, but that's a superb error message :D | 19:53 |
| *** randallburt1 has joined #heat | 19:54 | |
| sdake | yes, most excellent | 19:54 |
| sdake | next up pulling the iproute sources and figuring out what the hell that is supposed to mean | 19:54 |
| sdake | I guess it makes sense to somebody | 19:54 |
| sdake | all started when I upgraded... | 19:55 |
| * sdake groans | 19:55 | |
| *** randallburt has quit IRC | 19:57 | |
| larsks | sdake: If i'm reading "ip addr help" correctly, you can't specify flags like "dynamic" to "add" or "delete". | 19:57 |
| larsks | sdake: ...but it also suggests the same thing about "scope", which is patently false. So who knows? | 19:58 |
| larsks | Pulling "dynamic" out of the command line makes it work. | 19:58 |
| *** skraynev_ has quit IRC | 20:00 | |
| *** skraynev_ has joined #heat | 20:00 | |
| *** jistr has quit IRC | 20:03 | |
| *** klwilson has quit IRC | 20:04 | |
| *** klwilson has joined #heat | 20:04 | |
| *** sgordon has quit IRC | 20:04 | |
| *** skraynev_ has quit IRC | 20:05 | |
| *** IlyaE has joined #heat | 20:08 | |
| pafuent | I change the state of a BP as 'Implemented' and it disappeared, it's ok? | 20:17 |
| pafuent | Should I have done that? | 20:17 |
| *** vijendar has joined #heat | 20:22 | |
| *** vijendar has quit IRC | 20:22 | |
| *** vijendar has joined #heat | 20:22 | |
| *** mestery has quit IRC | 20:23 | |
| *** randallburt1 has quit IRC | 20:26 | |
| *** e0ne has joined #heat | 20:28 | |
| sdake | larsks yes I noticed that when I gave it a go without - so I disabled dynamic on the interface via ip with same results | 20:31 |
| sdake | big pile of software setting magic bits - ugh | 20:31 |
| *** gokrokve has quit IRC | 20:37 | |
| *** pafuent has left #heat | 20:38 | |
| *** gokrokve has joined #heat | 20:38 | |
| *** gokrokve has quit IRC | 20:43 | |
| *** slong has joined #heat | 20:44 | |
| *** denis_makogon has quit IRC | 20:44 | |
| *** dmakogon_ has joined #heat | 20:44 | |
| *** sdake has quit IRC | 20:48 | |
| funzo_ | stevebaker: I noticed the priority was changed on https://blueprints.launchpad.net/heat/+spec/autoscaling-parameters | 20:48 |
| funzo_ | stevebaker: can you offer any insight into the prioritization of the one? | 20:49 |
| *** sdake has joined #heat | 20:50 | |
| *** sdake has quit IRC | 20:50 | |
| *** sdake has joined #heat | 20:50 | |
| *** asalkeld has joined #heat | 20:53 | |
| *** randallburt has joined #heat | 20:55 | |
| *** sgrasley has joined #heat | 20:57 | |
| SpamapS | hm | 20:57 |
| SpamapS | I'm seeing FK constraint errors trying to delete stacks. | 20:58 |
| SpamapS | DELETE FROM stack WHERE stack.id = '1002e2fb-c558-481a-a357-98c360c8ab15' | 20:58 |
| SpamapS | Foreign key constraint fails for table `heat`.`stack_lock`: | 20:58 |
| SpamapS | , CONSTRAINT `stack_lock_ibfk_1` FOREIGN KEY (`stack_id`) REFERENCES `stack` (`id`) | 20:58 |
| *** scollier has joined #heat | 20:58 | |
| *** jrist has quit IRC | 20:58 | |
| SpamapS | That seems like there's a path through to DELETE where we aren't locking. | 20:58 |
| *** nati_ueno has quit IRC | 20:59 | |
| randallburt | Or we're forgetting to release the lock before we actually delete the stack. | 20:59 |
| *** asalkeld has quit IRC | 20:59 | |
| SpamapS | randallburt: equally possible for sure. | 20:59 |
| randallburt | jasond`: thoughts? ^^^ | 20:59 |
| SpamapS | I think that is it actually | 20:59 |
| SpamapS | because this is the purge command | 21:00 |
| SpamapS | which does not check for locks | 21:00 |
| SpamapS | because it really shouldn't have to :) | 21:00 |
| randallburt | shouldn't it? if I'm trying to purge during an update/create for example? | 21:00 |
| *** nati_ueno has joined #heat | 21:00 | |
| randallburt | seems as racy as any other operation. | 21:01 |
| jasond | i'm looking into it | 21:02 |
| *** sdake has quit IRC | 21:02 | |
| jasond | yeah, the release should be called just after the delete. i'm surprised i haven't seen this | 21:05 |
| *** gokrokve has joined #heat | 21:08 | |
| jasond | if we release the lock before deleting, that introduces another race condition | 21:08 |
| jasond | SpamapS: any chance you can show me how to reproduce that? | 21:09 |
| SpamapS | jasond: We have a very long running cloud, and I'm just finding this in the mysql innodb list of failed fks | 21:09 |
| *** gokrokve_ has joined #heat | 21:10 | |
| jasond | oh ok. if you come across it again, please let me know | 21:10 |
| SpamapS | mysql> select count(*) from stack s inner join stack_lock sl on sl.stack_id = s.id where s.deleted_at is not null\G | 21:11 |
| SpamapS | *************************** 1. row *************************** | 21:11 |
| SpamapS | count(*): 43 | 21:11 |
| SpamapS | jasond: I have 43 deleted stacks that have locks | 21:11 |
| *** radez is now known as radez_g0n3 | 21:11 | |
| *** jrist has joined #heat | 21:11 | |
| *** asalkeld has joined #heat | 21:12 | |
| jasond | oh ok, i have 2 | 21:12 |
| *** gokrokve has quit IRC | 21:13 | |
| *** scollier has quit IRC | 21:13 | |
| *** gokrokve_ has quit IRC | 21:14 | |
| *** aignatov is now known as aignatov_ | 21:15 | |
| SpamapS | jasond: perhaps something goes wrong and the thread manager never calls the release. | 21:16 |
| *** sdake has joined #heat | 21:17 | |
| *** sdake has quit IRC | 21:17 | |
| *** sdake has joined #heat | 21:17 | |
| SpamapS | jasond: realistically, we should probably let purge just delete those locks too. But we should also not leave dangling locks around. | 21:17 |
| SpamapS | jasond: want a bug report? | 21:17 |
| * sdake launches nuke at nova | 21:17 | |
| jasond | SpamapS: sure | 21:18 |
| *** sdake has quit IRC | 21:19 | |
| *** sdake has joined #heat | 21:19 | |
| *** sdake has quit IRC | 21:19 | |
| *** sdake has joined #heat | 21:19 | |
| *** alexpilotti has quit IRC | 21:22 | |
| SpamapS | jasond: https://bugs.launchpad.net/heat/+bug/1276336 | 21:27 |
| jasond | SpamapS: thanks | 21:28 |
| *** alexheneveld has joined #heat | 21:38 | |
| *** mestery has joined #heat | 21:39 | |
| *** mestery has quit IRC | 21:40 | |
| *** mestery has joined #heat | 21:40 | |
| gondoi | is openstack in general opposed to vagrant? | 21:51 |
| gondoi | I'm curious if it would be okay to add a Vagrantfile to heat | 21:51 |
| *** jdob has quit IRC | 21:53 | |
| *** ScottL1 has joined #heat | 21:55 | |
| *** sdake has quit IRC | 21:55 | |
| ScottL1 | Are there any good documents or examples on the use of autoscaling in a multi-network environment? | 21:56 |
| ScottL1 | I need to constrain which networks the ip addresses are chosen from | 21:57 |
| ScottL1 | any pointers or advise would be welcome | 21:58 |
| *** jprovazn has quit IRC | 21:59 | |
| *** e0ne has quit IRC | 22:04 | |
| *** david-lyle has quit IRC | 22:04 | |
| *** sdake has joined #heat | 22:16 | |
| *** alexheneveld has quit IRC | 22:16 | |
| *** blomquisg has quit IRC | 22:20 | |
| *** mestery has quit IRC | 22:24 | |
| sdake | gondoi i haven't heard any opposition to vagrant on the ml | 22:26 |
| sdake | how were you thinking of adding it to heat? | 22:26 |
| gondoi | sdake: well just as a segmented dev environment for running/testing and verification on linux distros, etc | 22:27 |
| sdake | gondoi I know this might be a bigger job, but I expect devstack folks would appreciatee that more :) | 22:27 |
| gondoi | ensures a consistent environment as well as long as you have vagrant | 22:28 |
| sdake | gondoi most folks run heat either from distro packaging or devstack | 22:28 |
| gondoi | you bet.. i'm just curious because i've seen vagrant repos for heat and other projects when it typically makes more sense to have it inside the project repo | 22:28 |
| sdake | not sure how vagrant could fit into distro packaging, but devstack definately | 22:28 |
| sdake | I haven't actually tried vagrant so I don't know precisely how it works | 22:29 |
| gondoi | yeah, you could even use devstack inside the vagrant instance | 22:29 |
| gondoi | it's really good for clean room type developing... | 22:29 |
| gondoi | if you are having problems with your env, you can vagrant destroy && vagrant up and start over from scratch with a brand new vm | 22:30 |
| *** rpothier_ has quit IRC | 22:30 | |
| *** jcru has quit IRC | 22:34 | |
| *** aweiteka has quit IRC | 22:37 | |
| ScottL1 | Create Failed Resource Create Failed: Error: Resource Create Failed: Badrequest: Multiple Possible Networks Found, Use A Network Id To Be More Specific. (Http 400) | 22:38 |
| *** dims has quit IRC | 22:39 | |
| *** sgordon has joined #heat | 22:43 | |
| radix | zaneb: yo, you around? | 22:46 |
| zaneb | howdy | 22:47 |
| radix | zaneb: what would you think of something like this for a patch? it doesn't align exactly with the list you wrote, but it seemed like a pretty logical step: http://paste.openstack.org/show/62495/ | 22:47 |
| radix | it just refactors _create_template to not rely on self.instances | 22:48 |
| radix | and be a staticmethod | 22:48 |
| zaneb | without digging into the details, that looks like a size of patch which would be comfortable to review :) | 22:49 |
| *** gokrokve has joined #heat | 22:49 | |
| radix | I tried very hard not to do anything extra, not even refactor that pointless nested function in _create_template :P | 22:49 |
| zaneb | so assuming it works, +1 ;) | 22:50 |
| * zaneb has long forgotten the exact sequence of patches I suggested anyway ;) | 22:51 | |
| shardy | zaneb: Hey, sorry been afk for a bit and saw I got pinged re x-auth-trust | 22:51 |
| zaneb | shardy: hey, np. I added it to the agenda for the meeting tomorrow anyway | 22:52 |
| zaneb | but I guess it's at the alt time so you will probably miss it | 22:52 |
| shardy | zaneb: The keystone stuff hasn't landed, so we can't implement it: | 22:52 |
| shardy | https://review.openstack.org/#/c/56243/ | 22:52 |
| zaneb | the question was, are we still expecting to implement that in Icehouse | 22:52 |
| zaneb | i.e. do we need to push keystone folks to do it | 22:53 |
| shardy | I see the keystone BP has been marked implemented but that is a mistake AFAICS | 22:53 |
| shardy | zaneb: No, it's low priority for Heat as far as I'm concerned | 22:53 |
| shardy | only one person has ever asked for that functionality | 22:53 |
| radix | that seems pretty weird | 22:53 |
| zaneb | shardy: dolphm just marked it as implemented because of https://review.openstack.org/#/c/57481/ | 22:54 |
| radix | why would you want to limit the number of tokens instead of just the duration of the trust? it seems like that would lead to a lot of problems by e.g. changing the frequency of a scheduled scaling operation | 22:54 |
| shardy | That's updating the API docs, but not the implementation | 22:54 |
| zaneb | shardy: which appears to be what we need | 22:54 |
| shardy | the patch I linked actually implements it | 22:55 |
| zaneb | oh, right | 22:55 |
| * zaneb didn't look that far | 22:55 | |
| shardy | but it's been stuck in an infinite review loop for a long time | 22:55 |
| *** dims_ has joined #heat | 22:55 | |
| zaneb | weird that they would merge the docs before the patch | 22:55 |
| zaneb | why does it say "Fix for bug #1250617" | 22:56 |
| zaneb | ??? | 22:56 |
| shardy | Yeah, they seem to have a weird workflow - stuff seems to go into identity-api, then code, then the official docs | 22:56 |
| shardy | invariably it ends up a mess and nothing matches | 22:56 |
| zaneb | shardy: and all 3 are marked as closing the bug | 22:57 |
| shardy | Yeah, it's all wrong | 22:58 |
| radix | hmm, what was the syntax for partial-implementation of a blueprint? I don't see it on https://wiki.openstack.org/wiki/Gerrit_Workflow | 22:58 |
| radix | just one for partial-bug | 22:58 |
| sdake | i'm off for the day | 22:59 |
| shardy | radix: You just tag bluprint: blah lots of times and manually change the BP status when completed | 22:59 |
| radix | hmm | 22:59 |
| sdake | this got my devstack working -> http://paste.fedoraproject.org/74485/54757139 | 22:59 |
| sdake | enjoy | 22:59 |
| radix | stevebaker requested I use "partial blueprint ..." last time | 22:59 |
| radix | shardy: so you mean I should avoid "Implements:"? | 23:00 |
| shardy | radix: For blueprints there is not automatic way of changing the state AFAIK, only bugs | 23:00 |
| shardy | s/not/no | 23:00 |
| *** topol has quit IRC | 23:01 | |
| *** harlowja is now known as harlowja_away | 23:01 | |
| radix | I dont think that answers my question | 23:02 |
| *** andersonvom1 has joined #heat | 23:02 | |
| *** andersonvom has quit IRC | 23:03 | |
| shardy | radix: you can use Implements if you like | 23:03 |
| *** jmckind has quit IRC | 23:04 | |
| radix | ok. I don't know if it actually means anything, that's just what Gerrit_Workflow says to do | 23:04 |
| shardy | I don't think it has any special meaning like Partial-Bug though | 23:04 |
| radix | alrighty | 23:05 |
| *** IlyaE has quit IRC | 23:06 | |
| shardy | radix: I think there are a few formats which work, just check your BP whiteboard after posting the patches to make sure they've been linked correctly | 23:06 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Rename Marconi plugin package https://review.openstack.org/71141 | 23:06 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Add Docker resources to docs https://review.openstack.org/68753 | 23:07 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Rename rackspace.rst to contrib.rst https://review.openstack.org/68752 | 23:07 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Add Rackspace resources summary to docs https://review.openstack.org/68793 | 23:07 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Add force_resource_mapping option https://review.openstack.org/68746 | 23:07 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Restructure Rackspace resources https://review.openstack.org/68747 | 23:07 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Reorganize docker-plugin for consistency https://review.openstack.org/68748 | 23:07 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Allow force registration of docker resource https://review.openstack.org/68749 | 23:07 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Allow force registration of Rackspace resources https://review.openstack.org/69374 | 23:07 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Change Docker resource mapping name https://review.openstack.org/68750 | 23:07 |
| openstackgerrit | Richard Lee proposed a change to openstack/heat: Generate docs for contrib plugins https://review.openstack.org/68751 | 23:07 |
| shardy | zaneb: Thanks for following up with dolphm, I'm going to call it a night | 23:09 |
| zaneb | shardy: np, looks like he already has as well ;) | 23:09 |
| zaneb | \o | 23:09 |
| openstackgerrit | Christopher Armstrong proposed a change to openstack/heat: Refactor _create_template to not rely on instances https://review.openstack.org/71143 | 23:11 |
| *** jasond has quit IRC | 23:12 | |
| *** harlowja_away is now known as harlowja | 23:13 | |
| *** mestery has joined #heat | 23:17 | |
| *** mestery has quit IRC | 23:17 | |
| *** mestery has joined #heat | 23:17 | |
| *** andersonvom1 is now known as andersonvom | 23:18 | |
| *** rcleere has quit IRC | 23:26 | |
| *** randallburt has quit IRC | 23:27 | |
| *** jasond has joined #heat | 23:29 | |
| *** sabeen has quit IRC | 23:29 | |
| *** andrew_plunk has quit IRC | 23:29 | |
| *** sgordon has quit IRC | 23:32 | |
| *** arbylee2 has quit IRC | 23:32 | |
| *** sgordon has joined #heat | 23:34 | |
| *** yogesh has quit IRC | 23:37 | |
| *** achampion has quit IRC | 23:38 | |
| *** andersonvom has quit IRC | 23:41 | |
| *** sgordon has quit IRC | 23:48 | |
| *** wchrisj has quit IRC | 23:54 | |
| *** wchrisj has joined #heat | 23:54 | |
| *** sgordon has joined #heat | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!