Wednesday, 2014-07-02

openstackgerritSteve Baker proposed a change to openstack/heat: Order intrinsic function docs alphabetically  https://review.openstack.org/10271700:00
openstackgerritSteve Baker proposed a change to openstack/heat: HOT intrinisic function list_join like Fn::Join  https://review.openstack.org/10271800:00
*** david-lyle has quit IRC00:05
*** david-lyle has joined #heat00:06
*** IlyaE has quit IRC00:09
*** david-lyle has quit IRC00:10
*** tango has quit IRC00:16
openstackgerritA change was merged to openstack/heat: Fix pagination of stack-list  https://review.openstack.org/10160500:22
*** gokrokve has quit IRC00:24
*** dims__ has quit IRC00:25
*** arbylee has joined #heat00:34
*** daneyon has quit IRC00:39
*** daneyon has joined #heat00:39
*** daneyon has quit IRC00:39
*** shakamunyi has quit IRC00:42
*** dims__ has joined #heat00:43
*** alexheneveld has quit IRC00:45
*** achampion has joined #heat00:45
*** ccrouch has joined #heat00:50
*** kfox1111 has quit IRC00:55
*** TravT|2 has quit IRC00:57
*** nati_ueno has quit IRC00:59
*** randallburt has quit IRC01:00
*** shakamunyi has joined #heat01:00
*** tiantian has joined #heat01:00
*** m_22 has joined #heat01:01
*** nati_ueno has joined #heat01:01
openstackgerritliusheng proposed a change to openstack/heat: Use None instead of mutables in method params defaults  https://review.openstack.org/10370701:09
*** nati_ueno has quit IRC01:09
openstackgerritSteve Baker proposed a change to openstack/heat: Catch NotFound for every stack delete db operation  https://review.openstack.org/10371501:10
openstackgerritSteve Baker proposed a change to openstack/heat: Sleep before stopping threads for delete  https://review.openstack.org/10371601:10
*** nati_ueno has joined #heat01:10
tiantianmorning01:10
stevebakerzaneb: ^ is another crack at the delete race logging error, take a look at the commit message01:10
*** shakamunyi has quit IRC01:20
*** nosnos has joined #heat01:49
*** gokrokve has joined #heat01:49
*** nosnos has quit IRC01:55
*** nosnos has joined #heat01:55
*** Yanyan has joined #heat02:04
*** nati_ueno has quit IRC02:07
*** nati_ueno has joined #heat02:10
*** andersonvom has joined #heat02:12
elynnHi morning all :)02:16
*** m_22 has quit IRC02:17
*** danielbruno has quit IRC02:17
*** nati_ueno has quit IRC02:21
*** nati_ueno has joined #heat02:21
*** zhiyan_ is now known as zhiyan02:22
*** nati_ueno has quit IRC02:22
*** zhiyan is now known as zhiyan_02:30
*** Qiming has joined #heat02:32
*** arbylee1 has joined #heat02:35
*** arbylee has quit IRC02:35
*** arbylee has joined #heat02:37
*** arbylee1 has quit IRC02:37
*** zhiyan_ is now known as zhiyan02:42
*** Qiming has quit IRC02:47
*** harlowja is now known as harlowja_away02:50
*** ajc_ has joined #heat02:50
zanebstevebaker: so the issue is the second time in a row that you stop the delete (i.e. the third consecutive delete command)?02:52
stevebakerzaneb: its timing sensitive, so its hard to tell if it is the third or fourth02:53
zanebok, but it can't happen on the second is I guess the point02:54
stevebakerzaneb: I could reproduce it with just 2 deletes. Even trying different delays between each call02:55
zanebthen I'm still confused :D02:56
*** killer_prince is now known as lazy_prince02:59
*** lazy_prince is now known as killer_prince03:00
zanebon the 2nd delete, the probability that the *first* delete will finish within 0.2s is ~003:00
stevebakerI couldn't reproduce it with 2 deletes, but that doesn't mean it can't be reproduced.03:01
stevebakerzaneb: unless its an empty stack, such as with the test in question03:01
*** nati_ueno has joined #heat03:02
zanebok, I could believe that :)03:03
zanebso we're acknowledging that this doesn't fix any real problems, it just makes tempest stop complaining03:04
zanebstevebaker: why remove the threadgroup.wait() though? that throws us into a polling loop when we could be wait()ing03:06
*** dims__ has quit IRC03:09
stevebakerzaneb: it seemed to not do anything. graceful=True does a wait anyway, and graceful=False each thread is killed. My interpretation of kill docs was that calls to thread.wait would immediately raise GreenletExit after kill has been called http://eventlet.net/doc/modules/greenthread.html (maybe I'm wrong about the "immediately")03:10
*** ramishra has joined #heat03:11
zanebyeah, it will always raise GreenletExit after that, but not immediately - it will still block until the thread finishes03:11
zanebyou're right that it makes no difference in the end, because we sleep() until the lock is released anyway03:12
zanebbut the current thread could get rescheduled multiple times during that, unnecessarily03:12
*** cmyster has joined #heat03:13
zanebI'll leave a review comment03:13
stevebakerzaneb: yeah, I'll put the wait back03:14
zanebthe eventlet documentation threw me the first 3 times I read it too03:16
*** cmyster is now known as rabbi_cmyster03:16
zanebthey're pretty poor03:16
openstackgerritSteve Baker proposed a change to openstack/heat: Sleep before stopping threads for delete  https://review.openstack.org/10371603:16
*** rabbi_cmyster is now known as cmyster03:16
cmystermorning03:16
zanebstevebaker: thanks. I'm inclined to merge, but still struggling to figure out the root cause03:19
stevebakerI'm putting it down to internal SQLAlechemy-mumble-mumble not stopping gracefully when its thread gets killed03:20
*** zhiyan is now known as zhiyan_03:22
cmystera zombie...03:22
cmysterkill it with fire...03:22
zanebthat's a definite posibility. though when its thread gets killed, GreenletExit gets raised in it... so it definitely gets the chance to handle it gracefully03:22
zanebstevebaker: we think it's the lock release that is failing?03:23
*** zhiyan_ is now known as zhiyan03:24
*** nosnos has quit IRC03:27
*** zhiyan has left #heat03:29
*** zhiyan has joined #heat03:37
openstackgerritZhang Yang proposed a change to openstack/heat: Refactor unit test test_neutron_autoscaling  https://review.openstack.org/10406103:38
*** zhiyan is now known as zhiyan_03:39
zanebstevebaker: I've almost convinced myself it's a minor bug in SQLAlchemy03:39
*** zhiyan_ is now known as zhiyan03:40
*** TravT has joined #heat03:45
*** zhiyan is now known as zhiyan_03:46
*** sab has joined #heat03:49
*** zhiyan_ is now known as zhiyan03:50
*** Qiming has joined #heat03:54
skraynevGood morning!03:58
cmystermorning04:01
openstackgerritA change was merged to openstack/heat: Fix pass incorrect volume_size while building bdm  https://review.openstack.org/10382204:02
*** andersonvom has quit IRC04:03
*** TravT has quit IRC04:03
openstackgerritliusheng proposed a change to openstack/heat: Fix H201 violations and re-enable gating  https://review.openstack.org/10277804:04
*** rakesh_hs has joined #heat04:05
*** andersonvom has joined #heat04:06
stevebakerzaneb: agreed, and the error itself is not technically a race, but the condition that leads to it is04:16
*** rakesh_hs has quit IRC04:16
lifelessstevebaker: hey q for you04:17
stevebakerlifeless: yup04:17
lifelessstevebaker: how can I, in a CFN template (sorry ;)) go from a OS::Neutron::Port -> the default route on the subnet04:17
*** rakesh_hs has joined #heat04:17
lifelessstevebaker: specifically I want the gateway IP on the subnet from the first fixed_ip from the port object04:17
lifelessstevebaker: presume the subnet already exists (provider network)04:18
*** tango has joined #heat04:18
lifelessstevebaker: I can pull out the subnet_id easily enough04:18
lifelessstevebaker: http://paste.ubuntu.com/7734743/04:19
stevebakerlifeless: off hand, I don't know04:19
lifelessstevebaker: I might be wrong, but I'm thinking its not possible04:20
lifelessstevebaker: k, thanks04:20
lifelessstevebaker: should I file a bug, do you think ?04:20
stevebakerlifeless: if you can't dig it out of the show attribute for the port then we could always add an attribute that exposes that04:21
stevebakerlifeless: sure, file a bug04:21
lifelessthe port just has the subnet_id, not the subnet object AFAICT04:22
lifelesshmm04:22
lifelesshttps://bugs.launchpad.net/heat/+bug/1286128 claims to be able to do this04:22
uvirtbotLaunchpad bug 1286128 in heat "Allow neutron network lookup by name" [Medium,Fix committed]04:22
* lifeless digs deeper04:22
stevebakerany attribute which takes a network or subnet can now use uuid *or* name04:23
lifelesssorry for the burnt cycles. I may end up filing a doc bug somewhere04:23
lifelessstevebaker: its not passing in thats a problem ;)04:23
lifelessstevebaker: its reading-out04:23
stevebakeryeah04:23
lifelessstevebaker: we pass in network. Network has N subnets.04:23
lifelessstevebaker: one of those subnets is chosen. gateway ip is on that subnet04:24
*** lazy_prince has joined #heat04:25
*** kebray has joined #heat04:27
*** rushiagr_ has joined #heat04:30
*** rushiagr has joined #heat04:31
*** rushiagr has quit IRC04:34
*** bvivek has joined #heat04:36
*** rakesh_hs2 has joined #heat04:36
*** rakesh_hs has quit IRC04:36
*** dims__ has joined #heat04:37
*** ramishra has quit IRC04:37
*** nosnos has joined #heat04:42
*** dims__ has quit IRC04:42
*** IlyaE has joined #heat04:42
*** Guest1036 has quit IRC04:43
*** lazy_prince has quit IRC04:46
*** killer_prince is now known as lazy_prince04:47
*** ramishra has joined #heat04:49
*** kebray has quit IRC04:53
*** kebray has joined #heat04:54
*** andersonvom has quit IRC05:00
*** dims__ has joined #heat05:02
*** dims__ has quit IRC05:07
openstackgerritSirushti Murugesan proposed a change to openstack/heat: Remove newline from message to avoid truncating API response  https://review.openstack.org/10406805:07
openstackgerritSirushti Murugesan proposed a change to openstack/heat: Avoid truncating API Response  https://review.openstack.org/10406805:09
*** kebray has quit IRC05:14
*** tiantian has quit IRC05:18
*** rushiagr has joined #heat05:24
*** rwsu has quit IRC05:24
*** cody-somerville has quit IRC05:27
*** cody-somerville has joined #heat05:27
*** shakamunyi has joined #heat05:31
*** shakamunyi has quit IRC05:32
*** shakamunyi has joined #heat05:32
*** rushiagr has quit IRC05:35
*** rushiagr has joined #heat05:36
*** chandan_kumar has joined #heat05:38
*** sab has quit IRC05:39
*** sab has joined #heat05:42
*** shakamunyi has quit IRC05:42
*** shakamunyi has joined #heat05:50
*** shakamunyi has quit IRC05:50
*** bvivek has quit IRC05:53
*** bvivek has joined #heat05:56
openstackgerritOpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex  https://review.openstack.org/8975006:02
*** dims__ has joined #heat06:03
*** nati_ueno has quit IRC06:03
*** IlyaE has quit IRC06:03
*** dims__ has quit IRC06:08
*** mohits has joined #heat06:09
*** unmeshg has joined #heat06:09
*** nkhare has joined #heat06:11
*** jstrachan has joined #heat06:14
*** tiantian has joined #heat06:14
*** nati_ueno has joined #heat06:16
*** bvivek2 has joined #heat06:25
*** bvivek has quit IRC06:25
*** gokrokve has quit IRC06:29
*** rushiagr_ has joined #heat06:35
*** rushiagr has quit IRC06:35
*** tango has quit IRC06:38
*** ishant has joined #heat06:45
*** julienvey has joined #heat06:50
*** nati_ueno has quit IRC06:51
*** rushiagr_ has quit IRC06:52
*** TonyBurn_ has joined #heat06:56
*** tomek_adamczewsk has joined #heat06:58
*** avquadri has joined #heat06:59
*** gokrokve has joined #heat07:00
therveGood morning!07:01
*** gokrokve_ has joined #heat07:02
*** gokrokv__ has joined #heat07:04
*** rushiagr_ has joined #heat07:04
*** gokrokve has quit IRC07:04
*** jamiehannaford has joined #heat07:04
*** gokrokve_ has quit IRC07:07
*** julienvey has quit IRC07:07
openstackgerritEthan Lynn proposed a change to openstack/heat: Implement Flavor Extra-specs Key/Value Pairs  https://review.openstack.org/10209407:07
*** gokrokv__ has quit IRC07:08
*** tspatzier has joined #heat07:09
cmystermorning therve07:11
*** mohits has quit IRC07:13
*** bvivek2 has quit IRC07:14
*** mohits has joined #heat07:17
*** maxskew has quit IRC07:19
openstackgerritEthan Lynn proposed a change to openstack/heat: Implement PrivateIpAddress property for EC2::Instance  https://review.openstack.org/9858007:20
*** tomek_adamczewsk has quit IRC07:21
*** pasquier-s has joined #heat07:22
skraynevtherve: hi, could you please check bug with 1336225. This id is mentioned in  https://review.openstack.org/#/c/103842/. I can not find this bug too, but possibly it's related with rights.07:22
Qiminggood afternoon07:23
therveskraynev, It was tagged as security07:23
skraynevtherve: I suppose, that it's invisible according to some security policies.07:23
therveAlthough that's not the right process for security issues :/07:24
skraynevtherve: Do you have some docs how to do id in right way?07:24
therveskraynev, https://wiki.openstack.org/wiki/VulnerabilityManagement07:25
*** maxskew has joined #heat07:25
therveskraynev, I think the bug is public now, you should try again07:25
skraynevtherve: thank you for the link. also you are right, it works for me now :)07:26
*** bvivek has joined #heat07:28
*** pas-ha has joined #heat07:33
pas-hamorning all07:33
lifelessstevebaker: filed https://bugs.launchpad.net/heat/+bug/133665607:37
uvirtbotLaunchpad bug 1336656 in heat "existing subnet not looked up on port?" [Undecided,New]07:37
cmystermorning pas-ha, Qiming07:37
Qimingcmyster, hi07:38
*** jcoufal has joined #heat07:38
pas-hacmyster, hi07:38
thervelifeless, Isn't there a typo? gatewway_ipa ?07:39
therveSorry gateway_ipa07:40
therveMaybe it's a new beer brewed by neutron07:40
lifelesstherve: that was a test I did, shouldn't have let it leak in07:42
lifelesstherve: heat doesn't throw on bad attribute names07:42
thervelifeless, Not with Fn::Select no07:42
*** mkerrin has joined #heat07:45
lifelesstherve: makes it hard to see whats up :)07:45
lifelesstherve: anyhow, just rechecking in case I fumble fingered07:45
thervelifeless, I don't think what you're trying to do works07:47
cmysterquestion: I have a stack with an issue in it, and via horizon I click on it. I see in horizon's log that "The server has either erred or is incapable of performing the requested operation." but the HTTP code is 500. What is the bug here? the code or the message ?07:47
therveGetting fixed_ips retrieve a dict with subnet_id, not a subnet object07:47
lifelesstherve: I know it doesn't work, the question is should it :)07:47
lifelesstherve: see https://review.openstack.org/#/c/82853/16/heat/engine/resources/neutron/port.py07:47
lifelesstherve: and https://bugs.launchpad.net/heat/+bug/128612807:47
uvirtbotLaunchpad bug 1286128 in heat "Allow neutron network lookup by name" [Medium,Fix committed]07:48
lifelesstherve: "07:48
lifeless    Additional properties 'network' and 'subnet' are added to07:48
lifeless    the resources (existing properties 'network_id' and"07:48
thervelifeless, That's a property, not an attribute07:48
*** tomek_adamczewsk has joined #heat07:50
Qiminghi, therve07:51
therve(It's also not an object, but a name)07:51
*** ifarkas has joined #heat07:51
lifelesstherve: anyhow, so - I get that it doesn't work today. Seems desirable for it work though, no?07:52
lifelessor there to be some way to specify that I need an object representing that subnet somehow (and I can't know the subnet a-priori, since its chosen by Neutron)07:53
thervelifeless, Probably? Neutron is hard, I have no idea.07:54
lifelesstherve: lol07:54
lifelesstherve: landscape was hard :) Neutron is a walk in the park.07:54
thervelifeless, It was designed correctly though :)07:55
therveNeutron implicit relationships between resources is just awful07:55
Qimingtherve: I'm wondering if we can make OS::stack_id (maybe parent_stack_id as well, if any) and OS::stack_name implicit entries in Nova server metadata.07:56
lifelesshmm I saw the thread on it but honestly didn't really get the issue07:56
therveQiming, Why?07:56
*** lsmola__ has joined #heat07:57
Qimingtherve: it could be a convenience for in-server operations, but I'm not sure it will expose some security risks07:57
therveQiming, I'd be tempted to just document it. Explicit is always better.07:57
Qimingtherve: it will eliminate the 'matching_metadata' property we need to specify for AutoscalingGroup07:57
thervelifeless, Anyway do you want to rephrase that bug or should I?07:58
therveQiming, I don't really want to eliminate that07:58
tspatzierQiming, therve: there is a patch in review that does this, and it has a config option that brings with it the documentation.07:59
Qimingtherve, any reasons why we require template authors to specify server.metadata and matching_metadata manually?  I know it is not the only way to filter out metrics from ceilometer side.07:59
*** tspatzier has quit IRC08:00
therveQiming, I prefer it to be explicit.08:01
therveThere are many ways you can write those rules, it's not clear why we should pick one08:01
therveIt's really 2 lines in the template most of the time08:02
Qimingtspatzier, oh? will check it08:02
*** gokrokve has joined #heat08:04
*** dims__ has joined #heat08:06
*** tspatzier has joined #heat08:07
shardymorning all08:08
thervetspatzier, Can you give the link to the patch08:09
therveHi shardy08:09
*** gokrokve has quit IRC08:09
lifelesstherve: how would you like it rephrased? And sure.08:10
tspatziertherve: sure - but I just realised that the patch actually does that for scheduler hints, not for metadata. But could be very similar. One second ...08:10
tspatziertherve, Qiming:  https://review.openstack.org/#/c/9688908:11
*** dims__ has quit IRC08:11
*** loki184 has joined #heat08:11
thervelifeless, Something like "Can't access subnet attributes from a port resource"08:11
cmystermorning shardy,08:11
lifelesstherve: retitled08:11
lifelesstherve: if you want more clarity, my brain is halting, so I'll let you do that ;)08:12
*** mkollaro has joined #heat08:14
*** julienvey has joined #heat08:18
*** rakesh_hs2 has quit IRC08:19
*** julienvey has quit IRC08:22
*** rakesh_hs has joined #heat08:22
*** bvivek has quit IRC08:22
thervetspatzier, Why is it a global configuration switch? It doesn't make much sense to me08:26
Qimingtspatzier, just took a quick glance over the patch, I agree to therve on the global tags08:27
therveI guess it's provider oriented?08:27
skraynevtherve: do not you mind if missed place for deprecated client call be fixed in depended patch?08:29
thervemissed place?08:29
skraynevtherve: I mean https://review.openstack.org/#/c/97973/1508:29
skraynevread last comment08:29
ramishra:q!08:30
*** gokrokve has joined #heat08:30
ramishraoops.. sorry08:30
skraynevskraynev: I missed it early :(08:30
*** ramishra has quit IRC08:30
therveI guess that's fine if it's just a warning08:30
*** ramishra has joined #heat08:30
cmystershardy: did you find anything interesting with my keypair template ?08:31
tspatziertherve: right, this is a provider thing. The use case is that the provider has some nova schedule plugin that evaluates the hints to make placement decision based on the fact that servers belong to one stack08:31
tspatzier... and that seemed to make sense to me.08:31
therveMokay08:31
skraynevtherve: glad hear it.08:32
shardycmyster: yes, basically the name should be optional08:34
*** gokrokve has quit IRC08:34
shardythen you can just get_resource: the keypair resource from the server resource08:34
openstackgerritliusheng proposed a change to openstack/heat: Use None instead of mutables in method params defaults  https://review.openstack.org/10370708:34
shardyactually, you can do that now, but you need to ensure the parameter name for the key is unique08:34
*** mkollaro has quit IRC08:35
shardywe should just default to the heat generated unique resource name08:35
shardycmyster: So basically, you don't need the depends_on08:35
liushengelynn: Hi, thanks for your review about https://review.openstack.org/10370  a new ps has been uploaded :)08:36
cmystershardy: I see.08:36
shardycmyster: http://paste.fedoraproject.org/114921/4042902008:36
shardyI'll post a patch later which makes the name optional and defaults to the resource name (which contains a random suffix)08:37
cmystershardy: on that (almost) subject, I created a stack when some commented out lines, keypair related, and when horizon is asking for that stack we return 500 to it. I think that it should be 400 in this case:http://pastebin.test.redhat.com/21903908:39
cmysterI was trying to be a smart arse...08:39
shardycmyster: lol ;)08:40
shardyYeah 500 is never the right response, please can you raise a bug with a reproduce procedure?08:40
cmysterand yes I amended that particular template exactly as you08:40
cmysteroh08:40
cmysteryes I'll try to check that file history...08:41
*** derekh_ has joined #heat08:41
*** bvivek has joined #heat08:41
*** FrancoisBillant has joined #heat08:43
elynnliusheng, I will review soon :)08:43
*** jamiehannaford has quit IRC08:44
*** jamiehannaford has joined #heat08:44
*** jcoufal has quit IRC08:48
*** bvandenh has joined #heat08:48
openstackgerritKanagaraj Manickam proposed a change to openstack/heat: Don't expose password in heat-keystone-setup-domain logs  https://review.openstack.org/10384208:51
*** jamiehannaford has quit IRC08:52
*** jamiehannaford has joined #heat08:56
*** julienvey_ has joined #heat09:00
*** gokrokve has joined #heat09:02
*** gokrokve has quit IRC09:03
*** gokrokve has joined #heat09:04
*** dims__ has joined #heat09:07
openstackgerritRabi Mishra proposed a change to openstack/python-heatclient: Alter misleading cli command 'resource-template'  https://review.openstack.org/10411809:08
*** gokrokve has quit IRC09:09
*** dims__ has quit IRC09:12
*** saju_m has joined #heat09:12
openstackgerritRabi Mishra proposed a change to openstack/python-heatclient: Alter misleading cli command 'resource-template'  https://review.openstack.org/10411809:12
*** alexheneveld has joined #heat09:17
*** titish_maryam has joined #heat09:18
*** mkollaro has joined #heat09:19
*** titish_maryam has left #heat09:19
*** bgorski has joined #heat09:20
*** andrearosa has quit IRC09:22
*** andreaf has joined #heat09:24
*** tomek_adamczewsk has quit IRC09:24
*** tomek_adamczewsk has joined #heat09:27
*** ramishra has quit IRC09:27
*** PragadeeswaranS has joined #heat09:28
*** ramishra has joined #heat09:35
*** loki184 has quit IRC09:36
*** liusheng is now known as liusheng-leave09:39
*** andrearosa has joined #heat09:40
*** rakesh_hs has quit IRC09:41
cmystershardy: I hit a snag with the template as you pasted it ^09:48
cmysterkey_name is heat_test_key yet when I try to create a stack it is telling me that the key heat_key is already created ?09:48
*** sorantis has joined #heat09:55
*** rushiagr_ has quit IRC09:56
shardycmyster: well presumably you already have a key with that name?09:56
zhangyangshardy:As I know, nove spec will be freezed at july 3, how about heat?09:56
cmysterno, its something different,09:57
Qimingzhangyang, I don't think that will apply to heat09:57
shardyzhangyang: we are not imposing a strict spec freeze like nova09:57
cmysterI am looking at it, I created a stack that might have made this one catch sometihng,09:58
cmysterI'll let you know :)09:58
shardybut if folks have major features they want in Juno, now is the time to propose them ;)09:58
zhangyang<shardy>: ok, thanks!09:58
*** Dev_Jin has joined #heat10:01
*** gokrokve has joined #heat10:02
*** Dev_Jin has quit IRC10:02
*** Dev_Jin has joined #heat10:03
*** Qiming has quit IRC10:04
Dev_JinHi.. I am trying to configure AutoScalingGroup and ScalingPolicy using heat.. I am able to launch VM as per the policy and scale-in scale-out also working.. I have a use case where I want to monitor instances created as a part of AutoScalingGroup.. but what I am seeing is all the instance (which are part of group and which are not part of the group) have identical information in terms of project_id, user_id etc.. how do I differentiate which instance are pa10:06
Dev_Jinof group and which are ouside group?10:06
Dev_JinI read somewhere about metadata injection.. how does it work?10:06
*** gokrokve has quit IRC10:07
*** dims__ has joined #heat10:08
*** jcoufal has joined #heat10:13
*** dims__ has quit IRC10:13
*** Yanyan has quit IRC10:15
zhangyang<Dev_Jin>: you can see the instance metadata, if the instancs belongs to a scalinggroup, the group info could be found in metadata, this also is how celiometer monitor autoscalinggroup10:18
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::NodeGroupTemplate resource  https://review.openstack.org/10028810:18
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement sahara client plugin  https://review.openstack.org/10058810:18
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::Cluster resource  https://review.openstack.org/7233610:18
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::ClusterTemplate resource  https://review.openstack.org/10150910:18
*** ramishra has quit IRC10:21
*** ramishra_ has joined #heat10:25
Dev_Jinthanks for reply zhangyang.. instance metadata is empty. I am executing nova show <instance id>10:29
Dev_Jinam I missing something?10:29
*** dims__ has joined #heat10:32
*** ramishra_ has quit IRC10:39
*** rakesh_hs has joined #heat10:42
*** Qiming has joined #heat10:47
zhangyangDev_Jin:did your scalinggroup have a tag start with 'metering.'? if that, heat won't add the groupname tag.10:47
Dev_JinI see Tags property in AWS::AutoScaling::AutoScalingGroup .. but I am using OS::Heat::AutoScalingGroup.. and I don’t see any tags there..10:51
*** tiantian has quit IRC10:54
Dev_Jinzhangyang: I see Tags property in AWS::AutoScaling::AutoScalingGroup .. but I am using OS::Heat::AutoScalingGroup.. and I don’t see any tags there..10:55
zhangyangif you use OS::Heat::AutoScalingGroup, I think you need to add Tag for your resource definition, I'm not sure.10:55
*** gokrokve has joined #heat11:02
*** loki184 has joined #heat11:05
*** mkollaro has quit IRC11:05
*** bvivek has quit IRC11:08
*** gokrokve has quit IRC11:08
*** mohits has quit IRC11:14
*** rushiagr_ has joined #heat11:19
*** ishant has quit IRC11:21
*** pasquier-s has quit IRC11:26
*** avquadri has quit IRC11:39
*** sgordon has joined #heat11:40
*** loki184 has quit IRC11:41
*** sab has quit IRC11:44
*** nkhare has quit IRC11:45
*** mkollaro has joined #heat11:46
cmystertherve: high? really?11:46
cmyster(the 500 error bug)11:47
thervecmyster, Yeah11:54
*** pasquier-s has joined #heat11:54
therveIt's most likely a regression because of the oslo.messaging port11:54
openstackgerritZhang Yang proposed a change to openstack/heat: Get members from DB when deleting loadbalancer  https://review.openstack.org/8935911:55
openstackgerritZhang Yang proposed a change to openstack/heat: Refactor unit test test_neutron_autoscaling  https://review.openstack.org/10406111:55
cmysterk11:57
therveshardy, Hey, around? There is an issue with oslo.messaging errors I think11:58
*** ajc_ has quit IRC11:58
*** nosnos has quit IRC11:59
cmystertherve: I am trying to introduce a similar issue when horizon is trying to get a stack info on a faulty stack. but just like this, it usually works11:59
cmysterarrrg, reproduce*11:59
thervecmyster, Don't introduce issues :D11:59
cmysterhahaha11:59
*** nosnos has joined #heat11:59
cmysterever pretty ones ?11:59
shardytherve: H11:59
shardyHi11:59
therveshardy, So exceptions which are not defined in heat.common.exceptions raise 50012:00
therveshardy, Any idea what to do?12:00
shardytherve: there's a mapping for each API, sec12:00
shardyhttps://github.com/openstack/heat/blob/master/heat/api/middleware/fault.py#L5712:01
shardyhttps://github.com/openstack/heat/blob/master/heat/api/aws/exception.py#L26812:01
shardyI assume there are some missing new types to add to those?12:01
*** julienvey_ has quit IRC12:01
shardywe fall through to a 500 if the type isn't defined12:02
*** julienvey_ has joined #heat12:02
therveHum I didn't know this was required12:02
*** gokrokve has joined #heat12:02
* cmyster smells a new automated test12:02
*** julienvey_ is now known as julienvey12:03
*** nosnos has quit IRC12:04
cmysterbbl12:05
*** julienvey has quit IRC12:06
*** jdandrea has joined #heat12:06
*** julienvey has joined #heat12:07
*** gokrokve has quit IRC12:07
*** bvandenh has quit IRC12:08
*** tomek_adamczewsk has quit IRC12:08
unmeshgHi, I came across a bug of supporting 'auto' as mountpoint12:08
unmeshgthere are issues in using it, which I have summarized in the bug https://bugs.launchpad.net/heat/+bug/126034512:09
uvirtbotLaunchpad bug 1260345 in heat "OS::Cinder::VolumeAttachment does not support "auto" value for mountpoint" [Medium,Triaged]12:09
unmeshgI have also enlisted possible approaches for resolving it12:09
shardyunmeshg: last time I checked the issue was that it doesn't work in cinder rather than a heat specific issue12:10
unmeshgshardy: I tried it on master branch (on my devstack setup )12:10
unmeshgit works fine from nova CLI12:11
*** tomek_adamczewsk has joined #heat12:11
unmeshgalso works fine if the mountpoint is defined 'null' (instead of 'auto' ) in the heat template12:11
unmeshgplease suggest the best approach12:13
shardyunmeshg: Oh right, I'm remembering incorrectly, I think my issue was when you actually specify mountpoint it's not respected12:13
shardyunmeshg: Looks like zaneb already has - if we make the property optional with no default then we'll pass None to novalclient if it's not specified12:14
shardywhich will do the right thing without needing to specify anything won't it?12:14
*** TonyBurn_ has quit IRC12:15
*** alexpilotti has joined #heat12:15
unmeshgyes, with the docs specifying that approach, it would be best!12:16
*** sorantis has quit IRC12:20
unmeshgshardy: thanks12:20
*** unmeshg has quit IRC12:21
*** unmeshg has joined #heat12:22
*** maxskew has quit IRC12:22
openstackgerritA change was merged to openstack/heat: Don't expose password in heat-keystone-setup-domain logs  https://review.openstack.org/10384212:23
*** maxskew has joined #heat12:24
openstackgerritOpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements  https://review.openstack.org/9650712:24
*** jdob has joined #heat12:25
*** sorantis has joined #heat12:26
*** achampion has quit IRC12:27
openstackgerritOpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements  https://review.openstack.org/9650712:33
therveshardy, So the issue is not there12:37
therveIt's a combination of allowed_remote_exceptions and exception default arguments12:37
*** erecio has joined #heat12:43
*** rushiagr_ is now known as rushiagr12:48
*** unmeshg has left #heat12:48
*** dims__ has quit IRC12:52
*** dims__ has joined #heat12:53
*** radez_g0n3 is now known as radez12:55
jdandreaQ: Mappings (HeatTemplateFormatVersion '2012-12-12') ... are they superseded by anything equivalent in heat_template_version '2013-05-23' ... ? (If so, I beg your pardon - I'm missing it!)12:56
shardyjdandrea: HeatTemplateFormatVersion is yaml rendered CFN12:58
jdandreaExactly.12:58
shardyheat_template_version is the native HOT format12:58
jdandreaSomeone here asked me if the HOT format had an equivalent to mappings. *scratches head* :)12:58
shardyjdandrea: No it doesn't12:58
jdandreaok12:58
shardyjdandrea: it has the concept of an environment instead12:58
jdandreaAh, true!12:59
shardywhere you can specify parameters which in most cases could replace the mappings usage in CFN templates12:59
jdandreaSo not the exact same thing, but ... yes. That.12:59
jdandreaTerrific.12:59
*** gokrokve has joined #heat13:02
*** chandan_kumar is now known as chandankumar13:03
*** blues-man has joined #heat13:04
*** gokrokve has quit IRC13:07
*** pafuent has joined #heat13:11
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: UnitTests: fix mock assertions usage  https://review.openstack.org/10418713:14
openstackgerritThomas Herve proposed a change to openstack/heat: Fix serialization of InvalidSchemaError  https://review.openstack.org/10418913:19
*** DaveJ__ has joined #heat13:22
*** andersonvom has joined #heat13:30
blues-manhello13:31
*** aweiteka has joined #heat13:33
*** swygue has joined #heat13:34
saju_mHi13:35
Qimingshardy?13:36
saju_mHow to invoke the classes of puppet module "heat" ? What is the order ?  I tried this http://dpaste.com/1V4YN00   , but not working .13:37
saju_mi invoked the class "heat" in the init.pp,  https://github.com/stackforge/puppet-heat/blob/master/manifests/init.pp13:38
therveEmilienM, do you know?  ^^13:38
saju_mbut that did't install any heat services13:38
shardyQiming: hi13:39
Qiminghi, shardy, not sure if this needs a core team review? https://review.openstack.org/#/c/96507/13:39
*** arbylee has quit IRC13:40
EmilienMtherve: saju_m looking now13:40
*** arbylee has joined #heat13:40
saju_m<EmilienM> thanks13:40
EmilienMsaju_m: you should switch on #puppet-openstack13:40
therveQiming, It's a regular review, so yeah13:40
Qimingshardy: 30 patch sets now, I don't know the rules for reviewing/approving these changes13:40
EmilienMsaju_m: you should not make order13:40
EmilienMit's within the module13:41
tspatziershardy: do you have a minute? I would have some questions to a real authentication expert :)13:41
Qimingshardy, but it is still there to be merged13:41
saju_m<EmilienM> ok, let me ask this in #puppet-openstack13:41
*** robsparker has joined #heat13:42
* Qiming is all ears to authentication topic, because he is puzzled as well ...13:42
*** rbuilta has joined #heat13:42
*** jpeeler has quit IRC13:45
*** jpeeler has joined #heat13:45
*** jpeeler has quit IRC13:45
*** jpeeler has joined #heat13:45
*** lazy_prince is now known as killer_prince13:45
*** rakesh_hs has quit IRC13:50
*** DandyPandy has joined #heat13:51
*** rushiagr has quit IRC13:56
*** tango has joined #heat13:59
*** IlyaE has joined #heat14:02
*** gokrokve has joined #heat14:02
*** ckmvishnu has joined #heat14:04
*** JAGDISH__ has joined #heat14:05
*** gokrokve has quit IRC14:07
bgorskils14:08
zanebfail :D14:09
*** achampion has joined #heat14:09
bgorski:)14:10
*** rwsu has joined #heat14:10
*** erecio has quit IRC14:10
*** BillArnold has quit IRC14:11
*** andersonvom has quit IRC14:14
*** BillArnold_ has joined #heat14:14
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement CFN CancelUpdateStack action  https://review.openstack.org/10285214:26
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement cancelling of stack update  https://review.openstack.org/10285114:26
*** gokrokve has joined #heat14:26
*** gokrokve_ has joined #heat14:27
*** gokrokve has quit IRC14:31
*** zz_gondoi is now known as gondoi14:31
shardytspatzier: Hi14:33
shardysorry, was afk for a few minutes14:33
tspatzierhey shardy, no problem. I'm in a call at the moment. Will ping you when done.14:34
*** sabeen2 has joined #heat14:34
shardytspatzier: Ok, np14:34
*** jcoufal has quit IRC14:34
*** jcoufal has joined #heat14:36
*** sabeen2 has quit IRC14:37
*** sabeen1 has quit IRC14:38
*** sabeen2 has joined #heat14:38
*** sorantis has quit IRC14:38
*** mkollaro has quit IRC14:40
*** saju_m has quit IRC14:42
*** pasquier-s has quit IRC14:43
*** gokrokve_ has quit IRC14:43
*** gokrokve has joined #heat14:47
*** daneyon has joined #heat14:47
*** daneyon has quit IRC14:47
*** daneyon has joined #heat14:48
*** sorantis has joined #heat14:50
*** andreaf has quit IRC14:51
*** arunrajan has joined #heat14:54
*** arunrajan has quit IRC14:54
*** arunrajan has joined #heat14:54
*** dims__ has quit IRC14:55
*** pas-ha has quit IRC14:58
*** nkhare has joined #heat15:00
*** saju_m has joined #heat15:01
*** rushiagr has joined #heat15:02
*** Dev_Jin has quit IRC15:09
*** gokrokve has quit IRC15:10
*** ckmvishnu has quit IRC15:10
*** gokrokve has joined #heat15:11
*** Dev_Jin has joined #heat15:15
*** PragadeeswaranS has quit IRC15:16
openstackgerritSteven Hardy proposed a change to openstack/heat: Add OS::Ironic::Chassis resource  https://review.openstack.org/10313315:16
openstackgerritSteven Hardy proposed a change to openstack/heat: Add Ironic resource base class  https://review.openstack.org/10422215:16
openstackgerritSteven Hardy proposed a change to openstack/heat: Add OS::Ironic::Node resource  https://review.openstack.org/10422315:16
openstackgerritSteven Hardy proposed a change to openstack/heat: Add OS::Ironic::Port resource  https://review.openstack.org/10422415:16
*** ericgoncz has joined #heat15:17
*** kebray has joined #heat15:18
jasondshardy: hey15:18
*** packet has joined #heat15:19
openstackgerritA change was merged to openstack/heat: Use None instead of mutables in method params defaults  https://review.openstack.org/10370715:19
shardyjasond: Hi!15:20
jasondshardy: i'm trying to understand how update works with wait conditions15:20
jasondit seems like AWS doesn't support it15:20
jasondwere you trying to support updates in the wait condition resource you're working on?15:21
*** randallburt has joined #heat15:21
*** randallburt has quit IRC15:21
*** randallburt has joined #heat15:21
shardyjasond: I was planning to, but the version posted currently doesn't15:21
shardyjasond: I was thinking add a "update" boolean property for the handle, which makes it behave like UpdateWaitConditionHandle on update15:22
*** dims__ has joined #heat15:23
jasondah, i guess i need to see how UpdateWaitConditionHandle works15:23
shardyjasond: It does a slightly weird thing overriding the base-class update method15:24
shardysee the original review SpamapS posted for details of why, but basically handle_update isn't called if you don't change the properties15:24
*** rushiagr has quit IRC15:25
*** Dev_Jin has quit IRC15:25
shardyjasond: did you have time to think about the base-class and interfaces discussed in your review?15:25
*** gokrokve has quit IRC15:25
shardyIt would be nice to make the properties for the native API based resource and the swift one compatible15:26
shardyso they can be easily interchangeable and/or mapped via the global environment15:26
jasondi did.  there are some parts of that base class that aren't relevant to the swift resource15:26
jasondswift-backed wait condition15:26
*** rushiagr has joined #heat15:26
shardyjasond: yup, I assumed we might need further refactoring, it was mostly the interfaces I was concerned about15:27
shardywe can always resolve duplication later :)15:27
jasondthey're aligned now.  see my update from yesterday15:27
*** dims__ has quit IRC15:27
shardyjasond: ah!  Thanks, I'm a bit behind on reviews this week15:27
*** dims__ has joined #heat15:27
jasondnp15:28
jasondalthough the swiftwaitconditionhandle has a timeout property15:28
jasondi guess it could use a default15:28
jasondi have to run to a meeting, be back in half an hour15:29
*** saju_m has quit IRC15:29
tspatziershardy: ping15:32
*** gondoi is now known as zz_gondoi15:32
shardytspatzier: Hi15:32
*** sorantis has quit IRC15:33
tspatziershardy: I have an authentication related issue with software config and heat in standalone multi cloud mode - not sure if anyone tried that so far ...15:33
*** sorantis has joined #heat15:33
shardytspatzier: probably not, auth for signalling gets hard in standalone mode15:33
tspatzierare you familiar with how access to metadata and signalling works for software config when using cfn?15:33
shardytspatzier: yes, pretty much15:34
shardynot so much the specifics of standalone or multi-cloud though tbh15:34
tspatziershardy: good, I thought so :)15:34
*** KanagarajM has joined #heat15:34
*** zz_gondoi is now known as gondoi15:35
tspatziershardy: So here is what I tried and what the result was: I set up heat standalone multi cloud and deployed a template with software config to a remove devstack. For that I have to specify the auth-url on the command line, and I had configured it before as an allowed uri in heat.conf15:35
*** sorantis has quit IRC15:35
tspatzierI guess this also generated EC2 tokens for the instances so the in-instance tools can query the metadata. and the remote keystone is used for this token generation.15:36
tspatzierNow when the instances talk back to the standalone heat engine, it tries to validate the tokens against its local keystone ... which fails. Unless I hardcode the remote auth uri in some more properties in heat.conf. But that seems to void the multi-cloud idea, right?15:38
shardytspatzier: Yeah, heat in standalone mode uses the auth_password middleware doesn't it?15:38
shardyso the ec2tokens auth won't work15:39
tspatzierI'm not too familiar with all that but just found out that the token validation seems to read the auth uri properties from the config file.15:39
openstackgerritA change was merged to openstack/heat: UnitTests: fix mock assertions usage  https://review.openstack.org/10418715:40
tspatziershardy: will the same scenario be possible once the native heat api and signalling works?15:40
openstackgerritA change was merged to openstack/heat: use register_cli_opts instead of register_opts  https://review.openstack.org/10384715:40
shardytspatzier: Yes the problem is the signed URL generated with the keypair needs to be validated via the ec2tokens auth middleware, which presumably isn't configured if you're in standalone mode15:40
shardytspatzier: You may be better using the HEAT_SIGNAL transport instead, because that uses a username and randomly generated password (which get deployed inside the instance)15:41
openstackgerritOpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements  https://review.openstack.org/9650715:42
*** jcoufal has quit IRC15:42
tspatziershardy: so in the case of HEAT_SIGNAL the standalone heat engine will not have to go to keystone (using the wrong one in standalone mode) and verify?15:42
shardytspatzier: by default the agents probably get a token from keystone then send that to heat, so they'd need to be configured to instead send the username/password so the auth_password middleware would work15:42
shardytspatzier: the problem is the other middlewares need to be configured with an admin service user which does the token validation15:44
*** chandankumar has quit IRC15:44
shardyand you don't have that in standalone mode15:44
shardytspatzier: python-heatclient can be configured to send the username/password to heat, but I'm not sure about the os-* agents15:45
*** tomek_adamczewsk has quit IRC15:45
shardytspatzier: probably a question for SpamapS/stevebaker15:45
tspatziershardy: I think since the heat collector plugin could do it and os-collect-config would not care.15:46
tspatziershardy: so when the heat client sends username and password, where would the standalone heat engine go an verify it?15:46
shardytspatzier: I assume you don't have the stack user domain configured, and that the user creating the stack is admin and just creates the users in the same project as the stack?15:47
tspatziershardy: I think so, at least I did not so any extra config in my environment.15:48
shardytspatzier: Ok, the standalone use-case may actually mean we need to maintain the legacy non-domain paths15:48
shardyor come up with another way to authenticate signals from instances in standalone mode15:48
shardyunfortunately the cost of keeping things working in standalone mode has turned out to be very high :(15:49
tspatziershardy: or find another mechanism for sharing software deployment metadata and deployment state in another way. I think this came up at the summit, but we never dived into it.15:50
shardytspatzier: If you can get the instance sending the username/password, the auth_password middleware you're already using should work15:50
*** nati_ueno has joined #heat15:50
shardytspatzier: Yeah, just getting this problem solved for the integrated (non standalone) case is really hard, making it work for both cases is pretty difficult15:51
shardytspatzier: I will think about it and try to find time to test some things though15:51
tspatziershardy: ok, then I can try to get the heat transport running and see how far I get. Seems like there are a few things to be fixed os-* and the image element15:51
shardytspatzier: Ok, sounds good, please let me know how it goes :)15:52
tspatziershardy: will do. and if you have some thoughts, I am eager to hear them :)15:52
tspatziershardy: on standalone in general: is this supposed to continue to be supported? sounds like it causes pain to keep everything in synch.15:53
*** elynn_ has joined #heat15:54
shardytspatzier: I don't know, probably a good topic for a meeting discussion, but I know some folks want to keep it15:54
SpamapSshardy: I think it is o-k to put limitations on standalone like "you have to use cfn for signals"15:54
shardySpamapS: the issue in this case is cfn signals won't work, so we have to make the agents send the username/password and use the native transport (with auth_password middleware configured)15:55
shardyThat should be a viable workaround for tspatzier, I think15:55
shardypretty sure nobody has tried this yet though ;)15:55
SpamapSshardy: what if we made standalone users run a standalone keystone?15:56
*** elynn_ has quit IRC15:56
tspatziershardy: I'm trying to find a way this work and also can be supported going forward. If we come up with a solution that has restriction, i.e. we do not support all kinds of possible option for standalone, that is fine.15:57
shardySpamapS: I guess, just sounds like yet more complexity tbh15:57
*** rakesh_hs has joined #heat15:57
SpamapSshardy: also to be clear, Helion's public cloud (aka hpcloud.net) will have a Heat service at some point. We're just not sure when that is. Then I will basically say "standalone can die"15:58
SpamapSshardy: I'm happy to sign HP people up to do any hard work to keep standalone alive.15:58
SpamapSshardy: If you hadn't noticed, our engineers are starting to get their feet wet in the code, so they'll be around to help with things now. :)15:58
shardySpamapS: sure, and I do understand some folks want standalone, but it's always been this kinda partially functional thing IMO, with lots of gotchas and stuff which doesn't work15:59
shardybut if folks are prepared to maintain all-the-workarounds, then fair enough :)15:59
*** daneyon_ has joined #heat15:59
shardySpamapS: yup, I've seen, good news :)15:59
SpamapSshardy: partially functioning for all heat use cases, but fully functioning for HP Rover's use case, is fine.15:59
SpamapSthus far they do not use updates or signalling16:00
SpamapSthough I think there is interest in signals16:00
tspatzierSpamapS: we stumbled over signals when trying to do software config with standalone16:00
shardyI know I've resisted it for the integrated case, but the standalone case seems like an argument for the random-string based auth (managed by heat) which some folks wanted16:01
*** arunrajan has quit IRC16:01
shardyTrouble is, if we do that it'll get used in non-standalone16:01
*** daneyon has quit IRC16:02
shardymaybe everything should be bounced via swift in standalone mode?16:02
shardye.g using the transport jasond is working on?16:02
tspatziershardy: that's also something I thought about. have to think about it more though16:03
shardytspatzier: Ok, I guess getting the user/password thing working is the first step, then we can think about alternative transports :)16:03
tspatziershardy: yes, sounds like a plan.16:04
*** arunrajan has joined #heat16:04
tspatzierSpamapS: I saw there are some patches in review to make heat the default transport in os-* . what is the way to get those changes when I build an image with diskimage-builder? Doesn't it pull released versions of os-* ?16:05
*** killer_prince is now known as lazy_prince16:05
*** gokrokve has joined #heat16:06
*** daneyon_ has quit IRC16:07
*** bgorski has quit IRC16:10
*** tspatzier has quit IRC16:10
*** blues-man has quit IRC16:12
*** nkhare has quit IRC16:16
*** IlyaE has quit IRC16:16
*** jcoufal has joined #heat16:20
*** alexheneveld has quit IRC16:21
openstackgerritWilliam C. Arnold proposed a change to openstack/heat: Implements stack lifecycle scheduler hints  https://review.openstack.org/9688916:22
*** Qiming has quit IRC16:24
*** daneyon has joined #heat16:27
*** noTHD has quit IRC16:27
*** Qiming has joined #heat16:28
KanagarajMheat is having REST API for list_stacks and with_count->no.of stacks16:30
*** david-lyle has joined #heat16:32
KanagarajMin the implementation, there are two rpc call made between api and engine service, this could be improved by introducing another rpc call list_stacks_with_metadata, where more details about stack list could be returned such as 'count'16:32
KanagarajMwill this should be a blue print or a defect?16:33
*** nati_ueno has quit IRC16:33
*** lazy_prince is now known as killer_prince16:34
SpamapSKanagarajM: defect.16:34
KanagarajMok thanks16:34
*** lsmola__ has quit IRC16:34
*** ifarkas has quit IRC16:34
*** JAGDISH__ has quit IRC16:36
*** JAGDISH__ has joined #heat16:38
*** Qiming has quit IRC16:42
*** julienvey has quit IRC16:42
*** FrancoisBillant has quit IRC16:43
KanagarajMSpamapS: i have filed a bug for the same https://bugs.launchpad.net/heat/+bug/1336857  and i have mentioned the return value of the new method list_stacks_with_metadata., could you validate it once and kindly suggest me if we could provide additional details as well. thanks.16:44
uvirtbotLaunchpad bug 1336857 in heat "improve list_stacks with count API implementation to use single rpc call" [Undecided,New]16:44
*** arbylee has quit IRC16:46
*** arbylee1 has joined #heat16:46
openstackgerritSteven Hardy proposed a change to openstack/heat: heat_keystoneclient add get_user_token  https://review.openstack.org/10288516:46
openstackgerritSteven Hardy proposed a change to openstack/heat: stack user add _user_token  https://review.openstack.org/10288716:46
openstackgerritSteven Hardy proposed a change to openstack/heat: clients make heat_url public  https://review.openstack.org/10288616:46
openstackgerritSteven Hardy proposed a change to openstack/heat: Convert WaitConditionHandle to use handle_signal  https://review.openstack.org/10135116:46
openstackgerritSteven Hardy proposed a change to openstack/heat: Update waitcondition API to use signal RPC interface  https://review.openstack.org/10135216:46
openstackgerritSteven Hardy proposed a change to openstack/heat: Refactor waitcondition resources to allow easier subclassing  https://review.openstack.org/10135316:46
openstackgerritSteven Hardy proposed a change to openstack/heat: Add an OS::Heat::WaitCondition resource  https://review.openstack.org/10135416:46
openstackgerritSteven Hardy proposed a change to openstack/heat: Add native WaitConditionHandle resource  https://review.openstack.org/10288816:46
*** Qiming has joined #heat16:48
*** zhiyan is now known as zhiyan_16:50
*** daneyon has quit IRC16:50
*** daneyon has joined #heat16:51
*** zhiyan_ is now known as zhiyan16:51
*** harlowja_away is now known as harlowja16:52
*** arunrajan has quit IRC16:55
*** IlyaE has joined #heat16:55
*** Qiming has quit IRC16:57
morganfainbergSpamapS, ping re: https://review.openstack.org/#/c/99225/ i responded to your inline comment about the tests. trying to figure out if we can do that better, but i am not seeing a way.16:59
morganfainbergSpamapS, i'll plan on fixing the commit to note the new options/option rename today17:00
*** Dev_Jin has joined #heat17:03
*** stannie has quit IRC17:05
*** arbylee1 has quit IRC17:08
*** arbylee has joined #heat17:08
*** gondoi is now known as zz_gondoi17:08
*** zhiyan is now known as zhiyan_17:08
*** bharath has joined #heat17:09
bharathHey I am new to heat, May I know why can't invisible_to_admin user create a stack ?17:10
*** derekh_ has quit IRC17:11
*** zz_gondoi is now known as gondoi17:11
SpamapSmorganfainberg: you can just split the tests into two classes, one that makes the call, and one that doesn't.17:12
*** loki184 has joined #heat17:13
morganfainbergSpamapS, how does that work with inheriting the tests from the parent?17:13
morganfainbergSpamapS or are you saying we should not do that17:13
SpamapSmorganfainberg: split the parent17:13
morganfainbergSpamapS, AH!17:13
SpamapSmorganfainberg: note that this is _not_ a requirement.17:13
morganfainbergSpamapS, now that makes sense (forgive my lack of coffee [and shabu shabu] filled brain)17:14
morganfainbergSpamapS, nah, i like it.17:14
SpamapSmorganfainberg: but less test code is less headache IMO :)17:14
morganfainbergSpamapS, it's a good approach and makes things a bit less headachy17:14
SpamapSnice17:14
morganfainbergSpamapS, ++++++++17:14
morganfainbergSpamapS, need to do lunch again17:15
*** tango has quit IRC17:16
*** gondoi is now known as zz_gondoi17:16
*** IlyaE has quit IRC17:17
*** SnowDust has joined #heat17:20
*** arbylee has quit IRC17:37
*** arbylee has joined #heat17:37
*** DaveJ__ has quit IRC17:38
*** sab has joined #heat17:40
*** sab has quit IRC17:44
*** arbylee has quit IRC17:44
*** arbylee has joined #heat17:45
*** Dev_Jin has quit IRC17:45
*** openstackgerrit has quit IRC17:49
*** openstackgerrit has joined #heat17:50
*** denis_makogon has quit IRC17:51
*** dmakogon_ has joined #heat17:51
*** KanagarajM has quit IRC17:52
*** bandarji has joined #heat17:52
*** jstrachan has quit IRC17:55
*** samuelmz_ is now known as samuelmz17:55
*** saju_m has joined #heat17:56
*** loki184 has quit IRC17:57
*** rushiagr has quit IRC17:57
*** bharath has quit IRC17:58
*** daneyon_ has joined #heat17:59
*** tomek_adamczewsk has joined #heat17:59
samuelmzHi, I've a heat template in which I've a OS::Nova::Server and a OS::Heat::AutoScalingGroup. Both have a script into the user_data section. How can I configure the template to execute the AutoScalingGroup user_data *after* the Nova::Server one? Any thought on this?18:00
*** tango has joined #heat18:01
*** daneyon has quit IRC18:01
*** jdandrea has quit IRC18:02
*** nati_ueno has joined #heat18:05
*** bandarji has quit IRC18:05
*** bandarji has joined #heat18:05
*** packet has quit IRC18:06
*** arunrajan has joined #heat18:09
*** jdandrea has joined #heat18:09
*** IlyaE has joined #heat18:10
*** bandarji has quit IRC18:10
*** bandarji has joined #heat18:10
*** bandarji has quit IRC18:11
*** arbylee has quit IRC18:11
*** shakamunyi has joined #heat18:14
*** packet has joined #heat18:19
*** arunrajan has left #heat18:20
openstackgerritDavid J Hu proposed a change to openstack/python-heatclient: Add keystone v3 auth support  https://review.openstack.org/9272818:23
*** jdandrea has quit IRC18:26
morganfainbergSpamapS, trying to split this code up, there is a lot more change here than i thought when you recommended the split (test_heatclient)18:27
morganfainbergSpamapS, let me make sure it passes tests and i'll post it. let me know what you think.18:28
*** kebray has quit IRC18:30
*** jamiehannaford has quit IRC18:31
*** kebray has joined #heat18:31
*** jamiehannaford has joined #heat18:32
BillArnold_I see no agenda for a heat meeting today at 2000 UTC. Is there a meeting today? Want to suggest an item, isolation of openstack AZs in practice.18:32
SpamapSmorganfainberg: cool18:37
*** dims__ has quit IRC18:37
*** arbylee has joined #heat18:37
morganfainbergstevebaker, shardy, I think heat should stop gating on sample config being up to date. if say oslo.messaging (or keystoneclient) changed options that you guys relied on all changes would get blocked/wedged until someone pushes an update sample config18:38
morganfainbergshardy, stevebaker, most other projects have droped that from pep8 for that very reason18:38
SpamapSBillArnold_: there should be one today, but perhaps zaneb just hasn't added it?18:38
openstackgerritMorgan Fainberg proposed a change to openstack/heat: Support using domain_name instead of domain_id  https://review.openstack.org/9922518:38
*** arbylee has quit IRC18:39
*** arbylee has joined #heat18:39
morganfainbergSpamapS, not sure i like that ^ as much as the previous iteration18:39
zanebBillArnold_, SpamapS: my bad, added.18:40
SpamapSmorganfainberg: no.. that did not turn out as awesome as I'd have expected.18:43
BillArnold_zaneb thanks. This derives from the discussions on the mailing list about autoscaling groups that Mike Spreitzer started yesterday.18:43
zanebI figured :)18:44
*** IlyaE has quit IRC18:44
morganfainbergSpamapS, yeah. it felt like a rabbit hole, but wanted to make sure i wasn't crazy before i said ... "yeah lets not do that"18:44
*** nati_ueno has quit IRC18:44
morganfainbergSpamapS, I say we go back one patchset and just fix the commit message. any complaints?18:45
*** nati_ueno has joined #heat18:45
*** dims__ has joined #heat18:47
*** nati_uen_ has joined #heat18:53
*** nati_ueno has quit IRC18:56
*** rbuilta has quit IRC18:57
*** cj has joined #heat18:57
*** kebray has quit IRC18:58
*** harlowja is now known as harlowja_away18:59
openstackgerritMorgan Fainberg proposed a change to openstack/heat: Support using domain_name instead of domain_id  https://review.openstack.org/9922519:00
openstackgerritJason Dunsmore proposed a change to openstack/heat: Add utility functions for Swift TempURLs  https://review.openstack.org/10430219:02
*** harlowja_away is now known as harlowja19:03
cjhello. I'm trying to create a HOT template with the OS::Glance::Image resource type but Heat is reporting an error "Unknown resource type: OS::Glance::Image". I have the latest of the master DevStack branch for Heat. Any suggestions on how I can "register" this resource type?  I was thinking it would be known by default19:10
*** tomek_adamczewsk has quit IRC19:15
*** tomek_adamczewsk has joined #heat19:16
*** jamiehannaford has quit IRC19:21
openstackgerritJason Dunsmore proposed a change to openstack/heat: OS::Swift::Signal resource  https://review.openstack.org/9694719:22
*** IlyaE has joined #heat19:23
*** dims__ has quit IRC19:33
*** DandyPandy_ has joined #heat19:35
*** DandyPandy has quit IRC19:36
*** sorantis has joined #heat19:37
*** cj has quit IRC19:39
*** cj has joined #heat19:39
*** vijendar has joined #heat19:43
*** dims has joined #heat19:44
*** SnowDust has quit IRC19:46
*** dims_ has joined #heat19:48
*** dims has quit IRC19:49
*** shakamunyi has quit IRC19:52
*** dims_ has quit IRC19:54
*** dims has joined #heat19:54
gpocentekstevebaker: hi! Do you me if I take other https://review.openstack.org/#/c/95979/ to add the doc bit?19:55
*** pas-ha has joined #heat19:55
gpocentekstevebaker: that is "Do you mind*"...19:56
*** IlyaE has quit IRC19:57
*** jdob has quit IRC19:57
*** jdob has joined #heat19:58
*** SnowDust has joined #heat19:58
*** dims_ has joined #heat19:58
zanebmeeting time20:00
*** tspatzier has joined #heat20:00
stevebakergpocentek, its yours! thanks20:00
*** dims has quit IRC20:00
skraynevstevebaker: new patches with stevedore kill me... reinstall heat for adding new plugin ;)20:04
gpocentekstevebaker: stevebaker thanks20:04
*** IlyaE has joined #heat20:05
*** dims_ has quit IRC20:05
*** jmckind has joined #heat20:05
*** jmckind has quit IRC20:05
*** rakesh_hs has quit IRC20:06
stevebakerskraynev, but really, how often do you add a new plugin?20:06
*** dims has joined #heat20:06
*** shakamunyi has joined #heat20:07
skraynevstevebaker: yeah, rarely. I just get so big emotions after testing patches :)20:09
*** jmckind has joined #heat20:09
*** harlowja is now known as harlowja_away20:09
skraynevstevebaker: So now I prefer use latest instead of downloading one by one.20:10
*** SnowDust has quit IRC20:11
*** radez is now known as radez_g0n320:13
bodepdhow do you specify arbitrary events and block for events in userdata (like you can in cfn)?20:16
bodepdjust a pointer to the docs would be greatly appreciated :) I'm sure I just missed it20:17
*** cj has quit IRC20:19
*** vijendar has quit IRC20:20
openstackgerritGauvain Pocentek proposed a change to openstack/heat: Makefile option to build xml docs  https://review.openstack.org/9597920:21
randallburtskraynev:  did you publish those comments for https://review.openstack.org/#/c/97975/16? I'm not seeing them.20:23
randallburtskraynev:  or were you referring to stevebaker's comments?20:23
skraynevrandallburt: second20:23
openstackgerritJason Dunsmore proposed a change to openstack/heat: OS::Swift::Signal resource  https://review.openstack.org/9694720:23
randallburtskraynev:  ah, ok. I'll remove −1 then. I just wanted to make sure I wasn't missing something.20:23
skraynevrandallburt: no, you have not missed something :)20:24
*** achampion has quit IRC20:30
openstackgerritMorgan Fainberg proposed a change to openstack/heat: Support using domain_name instead of domain_id  https://review.openstack.org/9922520:30
*** alexpilotti has quit IRC20:31
*** swygue has quit IRC20:32
openstackgerritYing Li proposed a change to openstack/heat: Move calculating new capacity to a function  https://review.openstack.org/9947020:33
*** aweiteka has quit IRC20:35
*** harlowja_away is now known as harlowja20:39
*** swygue has joined #heat20:42
*** swygue has quit IRC20:46
*** swygue has joined #heat20:46
*** sorantis has quit IRC20:48
*** zz_gondoi is now known as gondoi20:50
*** tomek_adamczewsk has quit IRC20:52
*** IlyaE has quit IRC20:52
*** IlyaE has joined #heat20:54
SpamapSdid I miss anything important in the meeting?20:55
SpamapSgot stuck running errands forever :-P20:55
*** dims has quit IRC20:55
pas-haSpamapS, everybody review stevebaker series :)20:55
SpamapSyeah I need to review a lot20:56
SpamapSzaneb: is there a group rate for hotel rooms or anything for the meetup?20:57
skraynevSpamapS, and reminding about booking for mid-cycle meetup20:57
*** tomek_adamczewsk has joined #heat20:57
zanebSpamapS: no. experience from TripleO is that it is a giant PITA for little benefit20:57
*** alexheneveld has joined #heat20:58
SpamapSzaneb: agreed20:58
* skraynev go to sleep20:58
SpamapSzaneb: HP also has good standing rates with most hotel chains so sometimes the negotiated rate is worse. ;)20:58
*** lurker5 has joined #heat20:58
* SpamapS watches those well oiled corporate muscles gleam in the sun20:59
zanebSpamapS: yeah, the group rate is worse than the RH rate for those two hotels20:59
SpamapSever marvelled at it's BEAUTY20:59
*** dims has joined #heat21:00
zanebif you call them up and ask for the Red Hat rate, you will quite possibly get it21:00
ekarlsobtw, I know I missed the meeting. but is there plans to get partial updates to stacks during Junp ?21:02
ekarlsocurrently one needs to resubmit the whole thing which is tedious when u want to say swap out flavors or images21:03
ekarlsofor upgrades / capacity increase21:03
stevebakerekarlso: that could be implemented entirely in python-heatclient21:04
ekarlsostevebaker: rly ?21:05
ekarlsofar as I understood heat api is missing PATCH support21:05
stevebakerekarlso: heatclient could fetch the previous template if it is not supplied on the cli21:05
SpamapSekarlso: heatclient could just fetch the current template and submit the changes21:05
SpamapS:)21:05
ekarlsoSpamapS: suuure21:06
ekarlsobut the heat api requires u to also pass all params no ?21:06
ekarlso..21:06
SpamapSyes, and fetching the stack shows them21:06
*** gokrokve has quit IRC21:06
randallburtekarlso:  a pet peeve of mine as well, but again heatclient could sort it.21:06
SpamapSno doubt PATCH would be more efficient21:06
SpamapSand should be implemented21:07
ekarlsoSpamapS: So, how would you then do if it's a password or secret ? ;P21:07
SpamapSbut .. bigger fish to fry21:07
ekarlsoit's tedious... :p21:07
randallburtSpamapS:  agreed, plus hidden params break the client solution21:07
randallburtekarlso:  jinx :)21:07
ekarlsowe're trying to upgrade for example a database instance, that requires us to pass in the whole params list each time which could have been as simple as passing in a new imasge id21:08
*** gokrokve has joined #heat21:08
randallburtekarlso:  do you know if there's an existing bp for this? IIRC, Tim Schnell may have submitted one a while back.21:09
SpamapSekarlso: nobody thinks it is a bad idea.21:09
SpamapSekarlso: but AFAIK, nobody is driving such work ATM21:09
ekarlsorandallburt: there's a bug at least I think that pas-ha linked me once21:09
ekarlsoSpamapS: I just asked if it was something for J21:09
SpamapSyeah the bug is at least a year old21:09
randallburtekarlso:  k21:09
SpamapSekarlso: it could be... if it had developers assigned. :)21:10
ekarlsoI think it would make life easier for #tripleo stuff as well :p21:10
SpamapShttps://bugs.launchpad.net/heat/+bug/122482821:10
uvirtbotLaunchpad bug 1224828 in tuskar "heat stack-update wants all parameters and the template all over again" [Medium,Triaged]21:10
SpamapSekarlso: agreed. TripleO's Heat resources are focused on convergence.21:10
zanebekarlso: there is a plan to fix the part about having to resubmit params. tango is working on it21:10
SpamapSbecause zomg if we don't start now21:11
*** saju_m has quit IRC21:11
ekarlsozaneb: PATCH support or ?21:11
SpamapSzaneb: tango should take ownership of that bug then21:11
zanebekarlso: not PATCH support, no21:11
ekarlsozaneb: boo21:11
zanebhttps://blueprints.launchpad.net/heat/+spec/troubleshooting-low-level-control21:11
ekarlsoSpamapS: convergence vs ?21:11
SpamapSekarlso: vs. workflow21:12
zanebSpamapS: I'll link that bug to the blueprint21:12
SpamapSekarlso: https://review.openstack.org/#/c/95907/21:12
zaneboh, it is linked already21:12
tangoekarlso: please take a look at the bp to see if it does what you need, or add comment to the whiteboard21:12
SpamapSzaneb: troubleshooting-low-level-control is a really broad topic.. :P21:12
* SpamapS facepalms at Launchpad putting linked blueprints below subscribed parties21:13
*** tspatzier has quit IRC21:13
tangoSpamapS: Yeah, agree, I should have come up with a more precise name21:13
ekarlsostill dont see how that makes it more convenient to say update a image or a password even and not requiring the whole param mess21:13
zanebSpamapS: yeah, it was originally something else altogether. I pretty much hijacked that blueprint to make it this21:13
*** dims_ has joined #heat21:13
ekarlsoif u have 30+ parameters it's a bit pain having to reconstruct it for update21:14
SpamapSekarlso: environment files FTW21:14
ekarlsoSpamapS: rly ?21:14
zanebekarlso: the point of that bug/bp is that you won't have to pass the parameters again21:14
randallburtand while we're on the subject, lets also be able to omit the template as well.21:14
randallburtSpamapS:  boo.21:14
SpamapSrandallburt: #lean21:14
*** IlyaE has quit IRC21:14
zanebrandallburt: why? you can always get the template from the API21:14
SpamapSdo what you can today to enable a use case today. :)21:15
randallburtzaneb:  because lazy21:15
*** dims has quit IRC21:15
SpamapSok, I think its time I went into the cone of silence21:15
randallburtlol21:15
ekarlsoheat stack-update mystack -P 'password=meh' ...21:15
SpamapSekarlso: heat stack-update mystack -e all-my-password-params.json   ;)21:15
zaneblike stevebaker said, python-heatclient can handle that21:15
ekarlsosure env files are ok, but if you're doing this for other services that use heat then u don't store env files all around.21:15
SpamapSekarlso: We agree. No argument is necessary. _WORK_ is necessary.21:16
SpamapSand the PATCH method in particular will be more useful when convergence happens because you can just patch individual params and the template independent of one another if you don't care about atomicity21:16
SpamapSand thus, I go to the cone of silence to focus21:17
ekarlsoSpamapS: what's teh timeframe for that ? ;P21:17
ekarlsoSpamapS: also for a newbie at this, convergence vs ?21:17
*** lurker5 has quit IRC21:18
randallburtekarlso:  too late. cone of silence.21:18
ekarlsohah21:18
*** dims_ has quit IRC21:18
*** IlyaE has joined #heat21:18
randallburtekarlso:  and atm, there's no real timeframe for that other than starting the convergence work now and having "something" around it for Juno. That doesn't mean PATCH in Juno. Beyond that, its kinda up to the community and folks willing to work on it.21:19
*** pafuent has left #heat21:19
*** tomek_adamczewsk has quit IRC21:19
ekarlsorandallburt: what's convergence btw?21:20
randallburtekarlso:  your best bet atm is to look at the relevant specs and bps and weigh in on those. Your bestest bet is to submit your own specs and patches.21:20
randallburtekarlso:  1 sec, will link the spec.21:20
ekarlsorandallburt: u mean the low level thing tango / zaneb talked about ?21:21
*** tomek_adamczewsk has joined #heat21:21
randallburtekarlso:  https://review.openstack.org/#/c/95907/21:21
randallburtekarlso:  also, we had some discussions at the summit and on the ML that would help clarify, but don't have the links for those ATM.21:21
ekarlsotango: it doesn't seem to be the same as doing a PATCH towards updating a stack though +21:24
*** sgordon` has quit IRC21:27
tangoekarlso: can you point me to some info on PATCH?21:27
*** sgordon has quit IRC21:27
*** DandyPandy_ has quit IRC21:28
*** sgordon` has joined #heat21:28
ekarlsotango: it's the HTTP Verb used when doing a partial update, that's what u do when u stack-update no ?21:28
randallburtekarlso:  stack update is PUT IIRC.21:29
*** IlyaE has quit IRC21:29
ekarlsorandallburt: even when you're updating 1 param ?21:29
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Add resource attrs and properties to stack-preview  https://review.openstack.org/9447921:29
tangoekarlso: I guess you mean using the concept of PATCH in http for stack-update parameters?21:30
ekarlsotango: yeah21:31
*** jdob has quit IRC21:31
tangoekarlso: so you don't have to list all the parameters again, change only a few and let the rest be the same21:31
ekarlsoit's silly to pass all the information if u just want to update 1 passowrd...21:31
*** dims has joined #heat21:31
ekarlsoadds complexity to things using it ..21:31
ekarlsoif I knew where to look I would allmost be tempted to try to add support for it :p21:32
*** daneyon_ has quit IRC21:32
*** daneyon has joined #heat21:32
tangoekarlso: That's what I have in mind for the bp.  I know the bp sounds different but the idea is to make it easy to reuse info currently in the DB21:33
*** IlyaE has joined #heat21:33
ekarlsotango: if u have a service that's using heat to setupresources aka trove and u say want to upgrade the cluster instances aka changing flavor it would be uber easier to just pass in instanceX_flavor = 'm1.large'21:34
ekarlsovs having to first get existing, ensure all params are really the correct type (if not it breaks when calling the stacks.update(), populating passwords again and so on ..21:34
ekarlsooh and also sometimes some params are implicit so u need to filter those too21:35
*** tomek_adamczewsk has quit IRC21:36
tangoekarlso: I can update the bp to add this use case.21:37
ekarlsotango: it would be awesome if it coudl incorporate partial updates like supporting just updating 1 parameter21:38
ekarlsoor even updating the template w/o parameters using existing parameters21:38
tangoekarlso: It's a little further generalization from what Zane suggested, but it seems useful.21:38
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Add resource attrs and properties to stack-preview  https://review.openstack.org/9447921:39
ekarlsotango: it would make services using heat and probably others as well easier ;p21:40
tangoekarlso: right, that case is currently covered.  How would you suggest the CLI option for patching just one parameter, to avoid ambiguity?21:41
*** tomek_adamczewsk has joined #heat21:41
tangoekarlso: we can't use ---parameters again since it would be ambiguous21:43
ekarlsotango: good q ;)21:43
ekarlsotango: btw, a PATCH can be more then 1 parameter, it implies a partial update..21:43
tangoekarlso: right, I would expect a partial list21:44
stevebakerlifeless: btw, by making this comment I'm not fobbing off your request https://bugs.launchpad.net/heat/+bug/1336656/comments/121:44
uvirtbotLaunchpad bug 1336656 in heat "Can't access subnet attributes from a port resource" [Medium,Triaged]21:44
*** jmckind has quit IRC21:44
ekarlsotango: it could be u would update f ex flavor and image of a instance21:44
ekarlsoor multiple passwords21:45
ekarlsoor users21:45
ekarlsoanywho's tango it would super awesome if it was added, I gotta go hit the sack :)21:46
tangoekarlso: ok, will do21:46
ekarlsotango: i'll owe u a beer / dinner @ paris if u do ;p21:46
tangoekarlso: hehe, will code for beer :)21:47
*** lindsayk1 has joined #heat21:48
stevebakerekarlso: btw there is this https://bugs.launchpad.net/heat/+bug/122482821:49
uvirtbotLaunchpad bug 1224828 in tuskar "heat stack-update wants all parameters and the template all over again" [Medium,Triaged]21:49
stevebakeroh, thats already been pasted21:49
ekarlsostevebaker: that was what I meant earlier.21:49
ekarlso:)21:49
randallburttango/ekarlso FWIW, unless this is implemented in the API and not just in the client, you only solve the problem for folks using python-heatclient. Some folks use other language bindings or the ReST api directly.21:51
stevebakerrandallburt: yes, PATCH would be preferred21:52
ekarlsorandallburt: I was hoping for API side as well :)21:52
ekarlsobut nighty folks (:21:53
ekarlsotoo late here :|21:53
*** kebray has joined #heat21:55
tangorandallburt: agree, I am doing both client and API21:56
randallburttango:  cool, thanks!21:56
*** arbylee has quit IRC21:56
*** BillArnold_ has quit IRC21:59
*** JAGDISH__ has quit IRC21:59
*** arbylee has joined #heat22:01
*** lindsayk1 has quit IRC22:03
*** ccrouch has quit IRC22:04
*** maxskew has quit IRC22:05
*** pas-ha has quit IRC22:09
*** kebray has quit IRC22:10
*** kebray has joined #heat22:11
*** lindsayk1 has joined #heat22:14
*** pas-ha has joined #heat22:15
*** shakamunyi has quit IRC22:17
*** ccrouch has joined #heat22:17
*** ericgoncz has quit IRC22:19
*** tomek_adamczewsk has quit IRC22:19
*** shakamunyi has joined #heat22:20
*** gondoi is now known as zz_gondoi22:24
*** DandyPandy has joined #heat22:25
*** DandyPandy has quit IRC22:25
*** DandyPandy has joined #heat22:26
*** pas-ha has quit IRC22:26
*** sorantis has joined #heat22:26
*** zhangyang has quit IRC22:27
*** pas-ha has joined #heat22:28
*** zhangyang has joined #heat22:28
*** pas-ha has quit IRC22:29
openstackgerritA change was merged to openstack/heat: Guaranteed password complexity using OS::Heat::RandomString  https://review.openstack.org/10163422:33
*** lindsayk1 has quit IRC22:38
*** lindsayk1 has joined #heat22:40
*** lindsayk1 has joined #heat22:41
*** DandyPandy has quit IRC22:52
*** david-lyle has quit IRC22:53
tangozaneb: hi Zane22:53
zanebhey22:53
openstackgerritA change was merged to openstack/heat-specs: Add specification to implement EC2 instance bdm  https://review.openstack.org/10213922:53
tangoI ran into an error scenario with stack-update which looks like a bug22:54
*** david-lyle has joined #heat22:54
tangowant to run it by you to see if I am doing something weird22:54
tangoSo I deploy a template win 1 resource (a server)22:54
tangothen I run stack-update to add a second resource with a parameter.  The update fails, and in the DB, the stack entry shows the template reverting to the previous version with 1 resource22:56
tangobut the parameters column contains the parameters from the failed stack-update22:56
openstackgerritA change was merged to openstack/heat-specs: Specification for filtering stacks by status for cfn API  https://review.openstack.org/10114022:57
tangoafterward, any CLI on this stack fails with the message:  ERROR: The Parameter (mycustomprop_one) was not defined in template.22:57
tangoIt fails on validating the parameters, because the previous template is used, which does not have the new resource with this parameter22:58
tangoEven stack-delete would not work22:58
tangoYou have this fix pending:  https://review.openstack.org/#/c/101288/22:59
tangoI wonder if it's related22:59
*** david-lyle has quit IRC22:59
*** tomek_adamczewsk has joined #heat22:59
*** blamar_ has joined #heat23:01
*** blamar has quit IRC23:01
*** blamar_ is now known as blamar23:01
*** arbylee has quit IRC23:01
*** blamar has quit IRC23:04
*** tomek_adamczewsk has quit IRC23:05
*** swygue has quit IRC23:11
*** shakamunyi has quit IRC23:11
*** arbylee has joined #heat23:19
*** JAGDISH__ has joined #heat23:24
*** ccrouch has quit IRC23:26
*** sabeen2 has quit IRC23:26
*** swygue has joined #heat23:28
*** m_22 has joined #heat23:37
*** jcoufal has quit IRC23:48
*** nati_uen_ has quit IRC23:49
*** arbylee has quit IRC23:51
*** swygue has quit IRC23:52
lifelessstevebaker: didn't think you were for a second23:53
openstackgerritOpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements  https://review.openstack.org/9650723:54
*** sorantis has quit IRC23:55

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