Wednesday, 2014-01-29

*** rpothier has joined #heat00:04
*** derekh has quit IRC00:05
*** pablosan has quit IRC00:07
*** tsmadds has joined #heat00:13
*** jmckind has quit IRC00:15
*** derekh has joined #heat00:17
*** tsmadds has quit IRC00:17
*** achampion has quit IRC00:18
*** slong has joined #heat00:21
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Check resources are in expected states  https://review.openstack.org/5864100:29
*** matsuhashi has joined #heat00:30
*** andersonvom has quit IRC00:37
*** alexpilotti has quit IRC00:37
*** _ruhe is now known as ruhe00:43
*** sabeen1 has quit IRC00:49
*** radez_g0n3 is now known as radez00:54
*** tango|2 has quit IRC01:06
*** nati_uen_ has joined #heat01:07
*** akuznetsov has quit IRC01:09
*** nati_ueno has quit IRC01:10
*** blomquisg has quit IRC01:11
*** tsmadds has joined #heat01:14
openstackgerritSteve Baker proposed a change to openstack/heat: Implement HOT intrinsic function get_file  https://review.openstack.org/6616001:17
openstackgerritSteve Baker proposed a change to openstack/heat: Store files in the raw_template table  https://review.openstack.org/6761701:17
*** IlyaE has joined #heat01:18
*** tsmadds has quit IRC01:18
*** blomquisg has joined #heat01:25
sdakei'm off, evening folks01:31
*** blomquisg has quit IRC01:36
*** derekh has quit IRC01:36
*** nosnos has joined #heat01:37
*** arbylee1 has quit IRC01:38
*** pvaneck has quit IRC01:39
*** vijendar1 has quit IRC01:45
*** ruhe is now known as _ruhe01:48
*** IlyaE has quit IRC01:48
*** blomquisg has joined #heat01:50
openstackgerritRyo Miki proposed a change to openstack/heat: Add handle_update for OS::Neutron::RouterL3Agent  https://review.openstack.org/6930301:54
*** nati_uen_ has quit IRC02:01
*** nati_ueno has joined #heat02:09
*** nati_ueno has quit IRC02:10
openstackgerritSteve Baker proposed a change to openstack/python-heatclient: Add support for software config resources  https://review.openstack.org/5888502:12
*** tsmadds has joined #heat02:15
*** blomquisg has quit IRC02:18
*** tsmadds has quit IRC02:20
*** IlyaE has joined #heat02:27
*** _ruhe is now known as ruhe02:28
*** spzala has quit IRC02:31
*** blomquisg has joined #heat02:31
*** alexheneveld has joined #heat02:34
*** chandankumar_ has joined #heat02:36
*** nati_ueno has joined #heat02:39
*** nati_ueno has quit IRC02:40
*** nati_ueno has joined #heat02:40
*** erkules_ has joined #heat02:41
*** erkules has quit IRC02:43
*** chandankumar_ has quit IRC02:48
*** dmueller has joined #heat02:59
*** dmueller has quit IRC03:02
*** aignatov_ is now known as aignatov03:07
*** chandankumar_ has joined #heat03:10
*** tsmadds has joined #heat03:15
*** chandankumar_ has quit IRC03:16
*** tsmadds has quit IRC03:20
*** nati_uen_ has joined #heat03:28
*** arbylee has joined #heat03:29
*** matsuhashi has quit IRC03:30
*** matsuhashi has joined #heat03:31
*** radez is now known as radez_g0n303:31
*** nati_ueno has quit IRC03:31
*** andersonvom has joined #heat03:32
*** matsuhashi has quit IRC03:35
*** arbylee has quit IRC03:49
*** david-lyle has joined #heat04:05
*** arbylee has joined #heat04:05
*** nati_uen_ has quit IRC04:13
*** packet has quit IRC04:15
*** tsmadds has joined #heat04:16
*** harlowja is now known as harlowja_away04:16
*** nati_ueno has joined #heat04:20
*** tsmadds has quit IRC04:21
*** alexheneveld has quit IRC04:22
*** nati_ueno has quit IRC04:23
*** nati_ueno has joined #heat04:23
*** nati_uen_ has joined #heat04:24
*** nati_uen_ has quit IRC04:25
*** nati_uen_ has joined #heat04:26
*** alexheneveld has joined #heat04:28
*** nati_ueno has quit IRC04:28
*** faramir1 has joined #heat04:28
*** aignatov is now known as aignatov_04:41
*** kebray_ has joined #heat04:42
*** nkhare has joined #heat04:44
*** edmund1 has quit IRC04:44
*** matsuhashi has joined #heat04:55
*** dims has quit IRC04:55
*** achampion has joined #heat04:55
*** dims has joined #heat04:57
*** cmyster has joined #heat05:03
*** pvaneck has joined #heat05:04
cmystermorning05:04
*** dmueller has joined #heat05:07
*** nati_ueno has joined #heat05:14
*** tsmadds has joined #heat05:17
*** nati_uen_ has quit IRC05:18
*** tsmadds has quit IRC05:21
andersonvommorning!05:22
*** kebray_ has quit IRC05:23
*** SergeyLukjanov_ is now known as SergeyLukjanov05:28
*** nati_ueno has quit IRC05:30
*** nati_ueno has joined #heat05:32
*** nati_ueno has quit IRC05:32
*** nati_ueno has joined #heat05:33
*** matsuhashi has quit IRC05:34
*** matsuhashi has joined #heat05:35
*** nati_ueno has quit IRC05:35
*** nati_ueno has joined #heat05:36
*** nosnos has quit IRC05:38
*** nati_uen_ has joined #heat05:38
*** yogesh has joined #heat05:38
*** nosnos has joined #heat05:41
*** nati_ueno has quit IRC05:41
*** nati_uen_ has quit IRC06:03
*** achampion has quit IRC06:06
*** achampion has joined #heat06:08
openstackgerritJenkins proposed a change to openstack/heat: Imported Translations from Transifex  https://review.openstack.org/6450406:08
*** nati_ueno has joined #heat06:11
*** tsmadds has joined #heat06:18
*** tsmadds has quit IRC06:22
*** matsuhashi has quit IRC06:23
*** matsuhas_ has joined #heat06:25
*** arbylee has quit IRC06:34
*** e0ne has joined #heat06:41
*** akuznetsov has joined #heat06:42
*** nati_ueno has quit IRC06:45
*** nati_ueno has joined #heat06:46
*** dmueller has quit IRC06:46
*** alexheneveld has quit IRC06:46
*** e0ne has quit IRC06:47
*** nati_ueno has quit IRC06:51
*** IlyaE has quit IRC06:56
*** SergeyLukjanov is now known as SergeyLukjanov_06:58
*** topol has quit IRC06:59
*** akuznetsov has quit IRC07:08
*** faramir1 has quit IRC07:10
*** sdake has quit IRC07:14
*** tsmadds has joined #heat07:18
*** coolsvap has joined #heat07:19
*** tsmadds has quit IRC07:23
openstackgerrithuangtianhua proposed a change to openstack/python-heatclient: Fixes environment file using correct YAML format  https://review.openstack.org/6980207:28
*** sdake has joined #heat07:36
*** alexpilotti has joined #heat07:38
*** ifarkas has joined #heat07:41
*** achampion has quit IRC07:42
*** achampion has joined #heat07:42
*** amritanshu_RnD has joined #heat07:43
*** slong has quit IRC07:44
*** erkules_ is now known as erkules07:45
*** jistr has joined #heat07:45
*** rwsu has quit IRC07:46
*** rwsu has joined #heat07:50
*** akuznetsov has joined #heat07:50
therve'morning07:51
*** jprovazn has joined #heat07:54
*** blomquisg has quit IRC07:57
*** chandankumar_ has joined #heat07:59
*** aignatov_ is now known as aignatov07:59
*** alexpilotti has quit IRC08:01
*** chandankumar_ has quit IRC08:03
*** chandankumar_ has joined #heat08:05
openstackgerrithuangtianhua proposed a change to openstack/python-heatclient: "version" section should be required in template  https://review.openstack.org/6980808:05
*** blomquisg has joined #heat08:11
*** akuznets_ has joined #heat08:14
*** chandankumar_ has quit IRC08:15
*** akuznetsov has quit IRC08:16
*** akuznets_ has quit IRC08:16
*** pvaneck has quit IRC08:18
*** tsmadds has joined #heat08:19
*** jprovazn has quit IRC08:20
shardymorning08:23
*** tsmadds has quit IRC08:23
*** akuznetsov has joined #heat08:25
*** aignatov is now known as aignatov_08:27
skraynevmorning08:28
*** yogesh has quit IRC08:33
*** tomek_adamczewsk has joined #heat08:34
*** yogesh has joined #heat08:34
*** chandankumar_ has joined #heat08:41
*** akuznetsov has quit IRC08:41
*** akuznetsov has joined #heat08:41
*** jprovazn has joined #heat08:42
*** tomek_adamczewsk has quit IRC08:42
*** chandankumar_ has quit IRC08:46
*** giulivo has joined #heat08:50
*** tsmadds has joined #heat08:53
*** SergeyLukjanov_ is now known as SergeyLukjanov08:56
therveHum master is indeed broken in devstack09:02
*** matsuhas_ has quit IRC09:04
*** matsuhashi has joined #heat09:04
shardytherve: what's broken?09:05
therveshardy, Not sure yet, but https://bugs.launchpad.net/heat/+bug/127399309:06
therveWe're getting None for the environment somehow09:07
*** yogesh has quit IRC09:10
shardytherve: it's a python-heatclient issue09:11
therveshardy, Yeah that's just what I thought09:11
thervedca8aaf35e789ef3c90e3c9af6c1cd647280af5e it seems09:12
therveAlthough the server could probably be more robust too09:13
*** aignatov_ is now known as aignatov09:14
*** matsuhashi has quit IRC09:17
shardyThe parsing of file paths seems broken too09:18
shardy-f ./foo.yaml looks for /foo.yaml09:18
*** derekh has joined #heat09:18
*** matsuhashi has joined #heat09:19
therveHum really? It seems to work for me09:23
shardytherve: with a relative path?09:24
therveYeah09:24
shardyhmm09:24
therveFor the template file you meant right?09:24
*** SergeyLukjanov is now known as SergeyLukjanov_09:25
shardytherve: Yeah, weird, I'll dig a bit deeper to see why it's not working09:25
therveshardy, Do we use the revert hammer or should we fix it?09:25
shardytherve: Have you identified what the problem is?09:26
shardyIf it's an obvious fix, I'd rather fix than revert09:26
therveshardy, it sends "environment": null09:26
therveInstead of skipping the key09:26
cmysterhmmm09:29
*** e0ne has joined #heat09:30
cmysteradding to my tests09:30
openstackgerritSteven Hardy proposed a change to openstack/python-heatclient: Only add files/environment to create fields if not None  https://review.openstack.org/6982109:33
shardytherve: ^^ that fixes it for me09:33
*** coolsvap_away has joined #heat09:33
shardycmyster: really, we need gating scenario tests in tempest which create a stack with the client09:34
*** coolsvap has quit IRC09:34
therveshardy, Yeah. Would you be so kind to add tests? :)09:34
shardytherve: yeah, on it now :)09:34
cmystershadower: exactly what I am working on09:34
cmystertabulator++09:35
shardycmyster: great, have you taken some of the bugs we have filed for tempest tests?09:35
*** therve has left #heat09:35
*** therve has joined #heat09:35
shardycmyster: we started raising bugs in the heat launchpad, and tagging them with "tempest", so we can coordinate who's doing what re tempest09:35
shardy(the tempest folks don't want our stuff in their launchpad)09:36
cmysternot yet, for now I am working on something that creates stacks for my own use and then tests if it was created and its last status. eventually all those silly little bugs will be automated, but I need some time to do it09:36
shardycmyster: Ok, cool, just wanted to make you aware of where we're tracking the tempest heat stuff09:37
cmysterthere is a meeting today right?09:38
cmysterI'll write down all the things that I require for my needs.09:38
shardycmyster: yep09:38
cmysteralso, please note that most (all?) tests I will be running on environments as close as possible to what people might end up using.09:39
shardycmyster: well the environment shouldn't matter much, but to get tests into upstream tempest, they will have to work on devstack09:41
cmysterI see09:41
*** SergeyLukjanov_ is now known as SergeyLukjanov09:41
cmysterwell I;ll need to get acquainted with it then :)09:41
shardycmyster: That's what's used to bring up the openstack environment that tempest runs on09:42
skraynevabout meeting time. Is normal time today  (20.00 UTC)?09:42
shardycmyster: but FYI I've developed tempest tests on devstack and RDO and either can work, depending on the type of test09:42
cmysterindeed.09:42
shardyskraynev: yes09:43
skraynevshardy: gotcha. thx)09:43
cmysterummm09:43
cmysterI see it in may calendar at 21:30 gmt?09:44
*** coolsvap has joined #heat09:45
*** coolsvap_away has quit IRC09:48
*** matsuhashi has quit IRC09:51
*** nosnos has quit IRC09:51
*** e0ne_ has joined #heat09:53
*** e0ne has quit IRC09:56
*** akuznetsov has quit IRC10:01
*** akuznetsov has joined #heat10:01
openstackgerritSteven Hardy proposed a change to openstack/python-heatclient: Pass empty dict not None for empty environment  https://review.openstack.org/6982110:05
*** akuznetsov has quit IRC10:13
*** e0ne_ has quit IRC10:15
*** che-arne has joined #heat10:15
*** e0ne has joined #heat10:15
*** e0ne has quit IRC10:18
*** e0ne has joined #heat10:19
*** tsmadds has quit IRC10:19
*** tsmadds has joined #heat10:23
*** e0ne_ has joined #heat10:26
*** e0ne has quit IRC10:26
openstackgerritA change was merged to openstack/heat: Update oslo db  https://review.openstack.org/6775910:28
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/python-heatclient: Fix order of arguments in assertEqual  https://review.openstack.org/6403910:30
*** akuznetsov has joined #heat10:37
*** aignatov is now known as aignatov_10:37
*** akuznetsov has quit IRC10:40
*** akuznetsov has joined #heat10:42
*** e0ne has joined #heat10:44
*** akuznetsov has quit IRC10:46
*** akuznetsov has joined #heat10:47
*** e0ne_ has quit IRC10:48
*** akuznetsov has quit IRC10:48
*** e0ne has quit IRC10:51
DaveJ_Hi guys, I have a few heat templates (cloud formation)  I was considering converting them to hot templates, but couldn't find parity for some features.  Is there an equivalent of auto-scaling group resource in the hot template?  Or is it just the same type ?10:53
DaveJ_actually n/m just found it10:53
*** amrit_ has joined #heat10:55
*** amrit_ has quit IRC10:55
*** amritanshu_RnD has quit IRC10:55
*** amritanshu_RnD has joined #heat10:55
*** achampion has quit IRC10:56
*** achampion has joined #heat10:56
*** akuznetsov has joined #heat10:59
*** jistr has quit IRC10:59
*** dmakogon_ is now known as denis_makogon11:05
DaveJ_Ok - a new issue for me.  I want to use an autoscaling group to create Instances that have multiple interfaces11:06
DaveJ_but can't see any way to do this11:06
DaveJ_With a standard AWS::EC2::Instance type, there is a NetworkInterfaces field, that allows be to specify multiple interfaces11:06
DaveJ_but for the autoscaling launch configuration, this doesn't seem to be an option11:07
shardyDaveJ_: Hi, you can use the AWS compatible resource types in HOT templates, we've not yet got native equivalents of all resources11:08
DaveJ_shardy: Thanks - although it looks like it's not something the AWS resources support.11:08
DaveJ_I'm not particularly tied to the AWS resources or cloud formation, I'd happily switch to hot, if it can let me do what's required.11:09
DaveJ_shardy: I was looking at this sample:11:13
DaveJ_https://www.irccloud.com/pastebin/VcQOXA7411:13
DaveJ_So I was trying to figure out, does this use a resource called 'my_web_server' which is an EC2 instance as a template ?11:13
DaveJ_The issue for me would be that I need to create two new network interfaces for each instance11:14
shardyDaveJ_: The autoscaling group creates a nested stack, containing AWS::EC2::Instance resources, using the config specified in the LaunchConfig11:14
shardyI'm not sure if what you want is currently possible tbh11:14
DaveJ_Yeah - so the AWS::AutoScaling::LaunchConfiguration doesn't seem to have support for adding multiple interfaces.11:15
*** coolsvap has quit IRC11:15
shardyDaveJ_: I was thinking you could overrided the Instance resource with a provider template in the environment, but I don't think that will work because the environment isn't passed down to the nested stack11:15
shardyThere's an OS::Heat::ScaledResource definition in the autoscaling.py, but AFAICS that's just a placeholder and you still can't actually override the EC2 resource type (zaneb will know for sure when he wakes up)11:16
shardyYou could probably use the OS::Heat::ResourceGroup to create a static group of OS::Nova::Server resources with multiple networks, but then you lose the auto part11:18
DaveJ_I'd like the Autoscaling ability, but the ResourceGroup would be a start.11:19
*** jistr has joined #heat11:19
DaveJ_I might be able to blag some work hours to work on the ScaledResource, as this would be pretty essential to what we are trying to do.11:19
shardyDaveJ_: Cool, help appreciated, you probably want to talk to radix as he's currently working towards a more flexible autoscaling capability11:20
DaveJ_shardy:  Are there any hot template examples of autoscaling with webhooks and celometer alarms.  I had a look in the git repo, but couldn't see anything.  I guess ultimately that's what I want, to do, with instances that have multiple networks.11:20
DaveJ_shardy: Ok I'll see if I can get hold of him.11:20
shardyhttps://github.com/openstack/heat-templates/blob/master/cfn/F17/AutoScalingCeilometer.yaml11:20
DaveJ_shardy: thanks - I had looked at that, but couldn't understand where 'AlarmUrl' was defined.  Is that something that is automatically generated11:23
shardyDaveJ_: Yeah the {"Fn::GetAtt": [WebServerScaleUpPolicy, AlarmUrl]} gives you a pre-signed URL which ceilometer then hits when an alarm happens11:25
DaveJ_shardy: excellent thanks..11:25
*** akuznetsov has quit IRC11:30
*** asalkeld_ has joined #heat11:31
*** asalkeld has quit IRC11:32
*** nkhare has quit IRC11:32
*** e0ne has joined #heat11:39
*** e0ne has quit IRC11:40
*** e0ne has joined #heat11:41
*** nkhare has joined #heat11:42
*** e0ne has quit IRC11:42
*** e0ne has joined #heat11:42
*** aignatov_ is now known as aignatov11:43
*** e0ne has quit IRC11:47
*** akuznetsov has joined #heat11:49
*** akuznets_ has joined #heat11:53
*** akuznetsov has quit IRC11:55
*** tsmadds has quit IRC12:07
*** nkhare has quit IRC12:14
*** Michalik has quit IRC12:15
cmystershardy: got a few minutes ?12:21
shardycmyster: sure12:22
*** e0ne has joined #heat12:30
*** vijendar has joined #heat12:31
*** vijendar has quit IRC12:34
*** akuznets_ has quit IRC12:35
*** e0ne has quit IRC12:36
*** akuznetsov has joined #heat12:37
*** e0ne has joined #heat12:41
*** akuznetsov has quit IRC12:44
*** e0ne has quit IRC12:45
*** rpothier has quit IRC12:53
*** akuznetsov has joined #heat12:54
*** akuznetsov has quit IRC12:57
*** chandankumar_ has joined #heat13:01
*** chandankumar_ has quit IRC13:05
*** chandankumar_ has joined #heat13:07
*** david-lyle has quit IRC13:08
*** jdob has joined #heat13:08
*** akuznetsov has joined #heat13:09
*** aweiteka has joined #heat13:09
*** Tross has left #heat13:10
*** asalkeld_ has quit IRC13:11
openstackgerritThomas Herve proposed a change to openstack/heat: Add a validation step to parameters schema  https://review.openstack.org/6965513:11
openstackgerritPablo Andres Fuente proposed a change to openstack/heat: Prevent access Parameters key in template dict  https://review.openstack.org/6831213:20
*** asalkeld_ has joined #heat13:24
*** chandankumar_ has quit IRC13:30
*** jistr is now known as jistr|english13:30
*** e0ne has joined #heat13:31
*** akuznetsov has quit IRC13:35
*** andersonvom has quit IRC13:36
*** pafuent has joined #heat13:36
openstackgerritJun Jie Nan proposed a change to openstack/heat: Resource type for software configuration  https://review.openstack.org/6762113:43
*** akuznetsov has joined #heat13:45
*** Michalik has joined #heat13:46
*** che-arne|2 has joined #heat13:47
*** che-arne has quit IRC13:48
*** che-arne|2 is now known as che-arne13:49
*** topol has joined #heat13:51
*** rpothier has joined #heat13:55
*** cmyster has quit IRC13:55
openstackgerritSteven Hardy proposed a change to openstack/heat: API tolerate None environment string  https://review.openstack.org/6986814:00
*** tspatzier has joined #heat14:04
openstackgerritSteven Hardy proposed a change to openstack/heat: heat_keystoneclient: abstract admin_client to a property  https://review.openstack.org/6957014:08
openstackgerritSteven Hardy proposed a change to openstack/heat: heat_keystoneclient ensure admin client respects SSL options  https://review.openstack.org/6956814:08
openstackgerritSteven Hardy proposed a change to openstack/heat: heat_keystoneclient: Move v3 path logic to constructor  https://review.openstack.org/6956914:08
openstackgerritSteven Hardy proposed a change to openstack/heat: heat_keystoneclient convert get_ec2_keypair to v3 API  https://review.openstack.org/6753114:08
openstackgerritSteven Hardy proposed a change to openstack/heat: Purge remaining heat_keystoneclient v2 code  https://review.openstack.org/6753414:08
openstackgerritSteven Hardy proposed a change to openstack/heat: heat_keystoneclient convert delete_ec2_keypair to v3 API  https://review.openstack.org/6753214:08
openstackgerritSteven Hardy proposed a change to openstack/heat: Fix user and signal responder exception import  https://review.openstack.org/6753314:08
*** aignatov is now known as aignatov_14:19
*** aignatov_ is now known as aignatov14:21
*** spzala has joined #heat14:21
*** achampion has quit IRC14:25
*** achampion has joined #heat14:25
*** jasond` has joined #heat14:26
*** sdake has quit IRC14:28
*** sdake has joined #heat14:28
sdakemorning14:28
shardyHi sdake14:29
sdakehey shardy14:29
*** alexheneveld has joined #heat14:33
*** ruruj has joined #heat14:35
*** jmckind has joined #heat14:36
*** randallburt has joined #heat14:37
randallburtstevebaker: you about?14:38
shardyrandallburt: unlikely..14:38
sdakerandballburt itss about 3am in nz atm14:38
randallburt Whoops, thanks sdake14:38
randallburtJust FYI then, I'm traveling back from the glance summit today and will miss the IRC meeting. I look forward to being assigned random action items.14:40
sdakelol14:40
sdakehow did that go14:40
*** aignatov is now known as aignatov_14:40
sdakethey up for the registry or whatever it is called14:41
randallburtreally well. Everyone is down for expanding the scope and getting more things than images in the catalog.14:41
sdake7am here, need more caffeine14:41
sdakeyes catalog not ergistry thanks :)14:41
*** jistr|english is now known as jistr14:41
sdakecool that seems like the right place to put it14:42
randallburtlol. Np. WIP patches should hopefully start showing up there in the next couple of weeks14:42
*** aignatov_ is now known as aignatov14:43
randallburtGotta pack and head out. Later heaters o/14:43
*** randallburt has quit IRC14:43
*** blomquisg has quit IRC14:43
*** e0ne_ has joined #heat14:44
rurujhi14:46
*** SergeyLukjanov is now known as SergeyLukjanov_a14:46
*** e0ne has quit IRC14:46
*** SergeyLukjanov_a is now known as SergeyLukjanov_14:47
*** chandankumar_ has joined #heat14:48
*** jcru has joined #heat14:50
*** achampion has quit IRC14:50
*** arbylee has joined #heat14:50
*** andersonvom has joined #heat14:50
rurujregarding autoscaling...how can I evalutate only the ceilometer's data of those instances belonging to the stack and not the data from all instances?14:51
*** chandankumar_ has quit IRC14:51
*** arbylee has quit IRC14:52
*** arbylee has joined #heat14:52
*** SergeyLukjanov_ is now known as SergeyLukjanov14:53
*** SergeyLukjanov is now known as SergeyLukjanov_a14:53
*** chandankumar_ has joined #heat14:54
*** SergeyLukjanov_a is now known as SergeyLukjanov_14:55
*** sabeen has joined #heat14:57
*** blomquisg has joined #heat14:57
*** chandankumar_ has quit IRC14:57
openstackgerrithuangtianhua proposed a change to openstack/heat: I18N support for error message  https://review.openstack.org/6963014:57
*** tims1 has quit IRC14:59
*** chandankumar_ has joined #heat14:59
openstackgerritJun Jie Nan proposed a change to openstack/heat: A multipart cloud-init resource based on SoftwareConfig  https://review.openstack.org/6321514:59
*** SergeyLukjanov_ is now known as SergeyLukjanov14:59
*** alexheneveld has quit IRC15:00
*** chandankumar_ has quit IRC15:02
*** chandankumar_ has joined #heat15:04
*** jprovazn has quit IRC15:05
*** packet has joined #heat15:05
*** chandankumar_ has quit IRC15:07
openstackgerritJun Jie Nan proposed a change to openstack/heat: A multipart cloud-init resource based on SoftwareConfig  https://review.openstack.org/6321515:08
*** aignatov is now known as aignatov_15:08
*** tango|2 has joined #heat15:10
openstackgerritA change was merged to openstack/heat-templates: Updates for RHEL 6.5  https://review.openstack.org/6953915:18
*** pafuent has left #heat15:18
*** achampion has joined #heat15:18
*** e0ne_ has quit IRC15:18
*** e0ne has joined #heat15:18
*** bgorski has joined #heat15:19
*** IlyaE has joined #heat15:21
*** vt102 has quit IRC15:24
*** achampion has quit IRC15:26
*** achampion has joined #heat15:27
*** mestery has quit IRC15:29
*** aignatov_ is now known as aignatov15:31
*** radez_g0n3 is now known as radez15:32
*** tims has joined #heat15:33
*** radez is now known as radez_g0n315:33
*** chandankumar_ has joined #heat15:34
*** achampion has quit IRC15:36
*** chandankumar_ has quit IRC15:38
*** achampion has joined #heat15:38
*** chandankumar_ has joined #heat15:40
openstackgerritJun Jie Nan proposed a change to openstack/heat: A cloud-config resource based on SoftwareConfig  https://review.openstack.org/6321415:40
*** dims has quit IRC15:42
shardyAnyone have any ideas of how to stub a call in parser.Stack.store() without fixing every single test?15:42
shardyI was thinking I could add a fixture at the HeatTestCase level but it's not that clear how15:43
*** dims has joined #heat15:43
zanebDaveJ_: have a look at https://review.openstack.org/#/c/58358/ but bear in mind that the InstanceGroup is probably not where we want to expose this first... it _will_ be supported in the new autoscaling API that radix is working on (and in fact OS::Nova::Server will be the default there anyway)15:43
zanebshardy: which call?15:44
*** alexheneveld has joined #heat15:44
shardyzaneb: I've added a call to self.clients.keystone().create_stack_domain_project...15:44
*** chandankumar_ has quit IRC15:45
shardybut because it's not controlled via a config option, it affects every test (unlike the trusts call, where I've just stubbed in each test)15:45
*** rcleere has joined #heat15:45
zaneband you want that to _always_ be stubbed?15:45
*** edmund has joined #heat15:46
shardyzaneb: Ideally I'd like to provide a default fixture which causes FakeKeystoneClient to be returned, but then I'll have to figure out how to still allow specific tests (mostly test_heatclient I think) to stub the client itself (overriding the default fixture)15:47
*** chandankumar_ has joined #heat15:47
shardywe return heat.tests.fakes.FakeKeystoneClient in most tests anyway, just not consistently15:47
zanebsounds painful :(15:47
shardyit's hard to know how to manage changes like this which have such huge test fallout15:47
shardyyeah it is15:48
zanebwhat does this function do and why is it needed in every store() call?15:48
*** coolsvap has joined #heat15:49
shardyzaneb: It creates a new project (owned by the stack) in the heat "stack user" domain15:49
*** chandankumar_ has quit IRC15:49
shardyzaneb: I was going to use a flag to only create it if a StackUser resource exists in the stack, but then we have the problem of differentiating correctly on update (to avoid leaking pre domain-user keystone users which are in the default domain)15:50
*** radez_g0n3 is now known as radez15:50
shardyso the least-bad option appears to be to accept the overhead of always creating the project on stack create15:50
shardythen it's always there on update (post domain-users transition), or not (backwards compatibility with existing stacks in the DB)15:51
zanebhmmm15:52
zanebwhat if resources that needed it just asked their stack for it, and it was lazily created and cached if needed?15:52
*** akuznetsov has quit IRC15:54
shardyzaneb: That doesn't solve the update problem, unless we store the project in every resource which creates a user15:55
shardyI was trying to store the project ID with the stack, maybe that is a mistake15:55
shardy2mins, I'll post a wip patch15:56
*** metabro has quit IRC15:56
zaneboh, I think I understand now - the problem is updates of stacks created before this code existed15:56
shardyYeah, exactly15:57
zanebI missed that the first time I read it15:57
*** metabro has joined #heat15:57
shardyzaneb: Yeah, the probem wasn't evident to me until I was part-way through writing the code15:57
shardys/probem/problem15:57
zanebalways the way.16:01
openstackgerritSteven Hardy proposed a change to openstack/heat: Add new stack_user_domain config option  https://review.openstack.org/6989016:01
openstackgerritSteven Hardy proposed a change to openstack/heat: heat_keystoneclient add support for stack domain projects  https://review.openstack.org/6989116:01
openstackgerritSteven Hardy proposed a change to openstack/heat: Create stack user domain project for each new stack  https://review.openstack.org/6989216:01
shardyzaneb: It's the last patch, if you have time for a quick look :)16:01
therveWhat's going on with the 26 builder?16:09
shardytherve: https://bugs.launchpad.net/openstack-ci/+bug/127413516:09
therveshardy, Thanks just got it too :)16:09
*** pablosan has joined #heat16:10
shardyzaneb: Thinking about it, maybe I should just create a StackUser base class which stores the project in resource_data, then transition User and SignalResponder to use it, in such a way that they'll remain backwards compatible16:13
shardyprobably preferably to fixing 2000 tests16:13
shardypreferable even16:14
zanebok. I looked at the patch, but didn't really have any ideas. I don't think I understand the upgrade issue well enough to comment intelligently16:15
shardyzaneb: Ok, no worries, thanks16:16
* shardy goes to put some extra-strength coffee on :)16:17
zanebhttp://www.cnn.com/2014/01/29/us/winter-weather/ <- 2 inches of snow in Atlanta; people are sleeping in their cars16:19
zanebwould probably be funny if we didn't need to be on the other side of Atlanta asap :/16:20
DaveJ_zaneb: Thanks for that.  I'll try and get in touch with radix to see if there is anything we can do to help.16:21
DaveJ_zaneb:  I know that patch isn't intended to be merged, but I'd like to try it out to see if it solves my use-case.  Do you have a example template that used it ?16:24
zanebI do not, I never tested it at all16:24
*** mestery has joined #heat16:24
*** mestery has quit IRC16:24
zanebby example template, do you mean the one passed through the template url?16:25
*** aignatov is now known as aignatov_16:25
zanebyou can (in theory) get a suitable one from the API16:26
zanebthe url is something like /resource_types/AWS::EC2::Server/template16:26
zanebif forget details, but I believe heatclient now supports it16:27
*** mestery has joined #heat16:27
*** IlyaE has quit IRC16:29
shardyzaneb: Yeah it does, heat resource-template <resource type>16:33
*** cmyster has joined #heat16:34
*** SergeyLukjanov is now known as SergeyLukjanov_16:34
*** sabeen has quit IRC16:35
*** david-lyle has joined #heat16:41
*** edmund has quit IRC16:41
*** amritanshu_RnD has quit IRC16:43
*** nati_ueno has joined #heat16:45
*** mestery has quit IRC16:46
*** rcleere has quit IRC16:47
*** arbylee has quit IRC16:48
*** arbylee has joined #heat16:49
*** vt102 has joined #heat16:50
radixDaveJ_: heya :)16:52
radixsorry I've been so afk this week, I'm traveling16:52
openstackgerritA change was merged to openstack/heat: fix some flaws in heat documents  https://review.openstack.org/6924816:54
*** vt102 has quit IRC16:55
*** tango|2 has quit IRC16:57
*** vt102 has joined #heat16:57
*** vt102 has quit IRC16:58
*** vt102 has joined #heat16:59
*** rcleere has joined #heat17:00
*** e0ne_ has joined #heat17:01
*** e0ne has quit IRC17:01
*** vijendar has joined #heat17:01
*** vijendar has quit IRC17:01
*** vijendar has joined #heat17:01
*** SergeyLukjanov_ is now known as SergeyLukjanov17:04
*** jistr has quit IRC17:05
*** pshchelo has quit IRC17:05
*** scollier has joined #heat17:08
*** pvaneck has joined #heat17:12
*** sabeen has joined #heat17:12
*** kebray_ has joined #heat17:15
jasond`shardy: FYI https://bugs.launchpad.net/heat/+bug/127420117:16
*** lcheng has joined #heat17:17
shardyjasond`: we can't support v2 auth and fix bug #108926117:17
*** scollier has quit IRC17:18
shardyjasond`: What is your roadmap for moving to using keystone?17:18
jasond`shardy: no idea :/17:19
shardyjasond`: Perhaps you may want to bring this up at the meeting today, but I don't see how we can modify heat master to have compatibility with third-party auth solutions, the whole point is we're integrated with the rest of openstack17:20
jasond`shardy: was https://bugs.launchpad.net/heat/+bug/1089261 a problem before the move to v3?17:20
shardyjasond`: Yes, the whole reason for moving to v3 is to try to fix it (by creating the users in a separate domain)17:20
jasond`ah ok17:20
shardywell that an that keystone are about to deprecate v217:20
shardy(in juno)17:20
jasond`sure, i'll bring it up17:21
shardyso we need to move to v3 either way17:21
lchenghello all,  I'm trying out heat on devstack and hit an error. Have anyone seen this error: “Resource Create Failed: Error: Creation of Server Stack-Lin-Wikidatabase-O3453234324 Failed: No Valid Host Was Found”17:21
shardyjasond`: From an outsiders perspective, it would seem your best option is to just use keystone, but I know it may not be that simple17:21
lchengWould appreciate any guidance on how to troubleshoot the issue.17:22
*** mestery_ has joined #heat17:23
thervelcheng, It sounds like a nova issue.17:23
jasond`shardy: there's a whole history to it.  i'm pretty sure randall knows the reasons.  thanks17:23
*** tspatzier has quit IRC17:25
shardyjasond`: Ok, cool, added a few comments17:27
jasond`shardy: thanks17:28
*** alexpilotti has joined #heat17:29
*** SergeyLukjanov is now known as SergeyLukjanov_a17:29
*** andrew_plunk has joined #heat17:29
*** SergeyLukjanov_a is now known as SergeyLukjanov_17:30
*** nati_ueno has quit IRC17:31
lchengtherve, thanks I'll check the nova logs17:31
*** cmyster_ has joined #heat17:32
*** nati_ueno has joined #heat17:33
*** nati_ueno has quit IRC17:34
DaveJ_radix: No worries.  I was just interested to know the state of  OS::Heat::ScaledResource.  I have a use-case where we want to scale a server that has multiple network interfaces.  I can boot indvidual instances with a network_list, but this isn't supported in auto-scaling groups.17:34
DaveJ_shardy/zaneb had suggested that the ScaledResource might be where this could be eventually implemented and you were looking at it.17:35
*** IlyaE has joined #heat17:35
radixhmm17:35
*** e0ne_ has quit IRC17:35
radixDaveJ_: ScaledResource isn't a real resource, it's just something that's intended to be overridden by the environment17:35
*** cmyster has quit IRC17:35
radixso you can replace it with whatever you want in your environment file (including e.g. a nested stack that has multiple resources in it)17:36
DaveJ_radix:  ah ok, sounds like it would be perfect.17:36
zanebradix: so AIUI DaveJ_ wants to be able to scale OS::Nova::Server instead of AWS::EC2::Instance resources17:36
DaveJ_radix: how far along in development is it ?17:36
radixDaveJ_: it's already possible in master afaik?17:37
radixbut maybe there's something blocking practicl use?17:37
DaveJ_radix: I haven't tried it out yet - was just talking to shardy and zaneb early today.17:37
*** SergeyLukjanov_ is now known as SergeyLukjanov17:38
radixzaneb: isn't that already possible by overriding ScaledResource to be a template wrapping OS::Nova::Server?17:38
zanebradix: it was never my intention to have implemented that already17:38
radixI guess I'm not sure which part is missing17:38
zanebradix: no17:39
zanebhttps://review.openstack.org/#/c/58357/2/heat/engine/resources/autoscaling.py17:39
zanebso the environment used for the nested stack is fixed17:39
radixoh right, you can't override the environment17:39
zanebhttps://review.openstack.org/#/c/58358/1/heat/engine/resources/autoscaling.py is what's missing17:39
zanebbut that was not something we ever planned to merge, just a demonstration17:40
radixah ok17:40
therveI'm not sure why the first part got merge, though17:40
radixI guess it doesn't actually have an effect on anything by itself17:41
zanebradix: DaveJ_ mentioned he was going to try that patch. Although tbh there's a strong possibility of running into subtle issues, since it has never been tested17:41
radixzaneb: what if the autoscale code didn't override the environment, could it inherit the parent's environment and allow overriding ScaledResource?17:42
zanebnested stacks don't inherit the parent environment17:42
zanebthey get an empty environment anyway17:42
radixok17:42
zanebyou could override it in the default environment... except that we now explicitly pass the environment for the InstanceGroup17:43
DaveJ_zaneb:  that may be an issue for us - as we want to inject the address of a server in the parent stack into all scaled up instances (the location of a puppet master)17:44
zanebin any event, that wouldn't help, because a LaunchConfig looks like an AWS::EC2::Instance17:44
zanebso you need a provider template to translate17:44
zanebDaveJ_: inject how? through the UserData? that shouldn't be a problem17:45
*** aignatov_ is now known as aignatov17:50
DaveJ_zaneb: Yep - so I was just passing in the private ip of the AWS Instance via Userdata17:51
*** ruruj has quit IRC17:51
DaveJ_so basically - I'll start by trying out that patch.  where hopefully I just need to get a resource-template that I can pass in to test if this works17:52
DaveJ_I'll see how far I can get with that17:52
zaneb" so I was just passing in the private ip of the AWS Instance via Userdata" <- that should Just Work(TM)17:52
openstackgerritSwann Croiset proposed a change to openstack/heat: Fix event_type names for stack CRUD notifications  https://review.openstack.org/6937217:54
*** harlowja_away is now known as harlowja17:54
*** cody-somerville has joined #heat17:56
*** rcleere has quit IRC17:59
*** derekh has quit IRC18:01
*** cmyster has joined #heat18:02
*** arbylee has quit IRC18:02
*** aignatov is now known as aignatov_18:03
*** cmyster_ has quit IRC18:05
*** aignatov_ is now known as aignatov18:06
*** nati_ueno has joined #heat18:08
*** dmueller has joined #heat18:08
*** nati_ueno has quit IRC18:09
*** andersonvom has quit IRC18:10
*** mestery_ is now known as mestery18:10
*** arbylee has joined #heat18:14
*** alexheneveld has quit IRC18:19
therveI won't be able to join the meeting (as usual), so I wondered if we could talk about autoscaling now, or if the topic could be mentioned then and I'll catch up :)18:22
shardyradix: ^^18:23
therveI think we won't be able to do nearly as many blueprints as listed, so it'd be cool to have a minimal goal for icehouse18:23
*** bgorski has quit IRC18:23
shardytherve: +118:23
therveie, a native resource?18:23
therveIt looks like https://blueprints.launchpad.net/heat/+spec/as-intermediate-resources18:23
therveOne thing I don't see is a native signal API. Are there any discussions about that?18:24
*** pablosan has quit IRC18:25
therveAlso the native resource isn't specified formally anywhere that I know18:26
zanebtherve: not sure what you mean by 'signal api'?18:26
shardytherve: So the discussions we had a couple of weeks ago, IIRC resulted in the decision to aim for refactoring the internal interfaces such that some native resources can be developed (with a common, or at least partially common underlying implementation)18:26
thervezaneb, API backing "AlarmURL"18:26
shardyzaneb: The current signal stuff only works via the CFN API18:27
zaneboh, ok18:27
zanebthat sucks18:27
shardySo we either need to add something similar to the ReST API, or just update resource metadata directly18:27
shardyprobably just doing something similar to the CFN API, but with native auth will work18:28
therveThat sounds like a step18:28
shardyi.e use a domain user identity with a username/password in the headers instead of an ec2 signed URL18:28
therve(Except I'm not sure how it will integrate with ceilometer, but we'll see)18:28
*** dmueller has quit IRC18:29
shardytherve: Yeah the ceilometer interfaces will need some modification, but that doesn't seem like it should be too difficult, we'll just specify the request differently18:29
therveCool, I like your confidence :)18:29
therveAnyway, we should just manage expectations as close as we're getting to the deadline18:30
thervezaneb, Did you have a precise idea on how the ScalingGroup would look like?18:30
*** yogesh has joined #heat18:30
therveI guess it would use that ScaledResource pattern?18:30
*** nati_ueno has joined #heat18:30
zanebit should look like the current InstanceGroup, except with different properties and the code factored into a library18:31
thervezaneb, So you would keep LaunchConfig separated?18:32
*** cmyster_ has joined #heat18:32
zanebthe properties should be based on what we expect the API to look like when we've created it18:32
zanebwhich is fortunately the one thing that we _have_ spent a lot of time on18:32
thervezaneb, Has it been documented somewhere though?18:33
*** alexheneveld has joined #heat18:33
therveI saw lots of discussions, but that wiki page stayed the same18:33
zanebtherve: tbh it's not super-important. I would prefer a separate launch config because it gives you the flexibility to reuse them18:33
therve(https://wiki.openstack.org/wiki/Heat/AutoScaling fwiw)18:33
*** vijendar has quit IRC18:34
zanebthere was a giant ML thread iirc18:34
thervezaneb, Not super important, but any point of discussion should be cleared quickly18:34
zanebbut I don't think there was ever a firm conclusion18:34
zanebtherve: ++18:34
shardyI was expecting the details to get thrashed out via code reviews, but, uh, there's not been any code..18:35
shardy(yet) :)18:35
therveRight.18:35
therveReviews aren't the best medium for design discussions, though18:36
zanebtherve: I think a ML thread with pro/con just on the separate LaunchConfig would be a good idea18:36
shardytherve: No, but at some point you have to go from discussion to something concrete18:36
*** cmyster has quit IRC18:36
thervezaneb, Would it be? That's another week of talking...18:37
zanebreviews are good for some design discussions, in this case it's more of a user interface issue than an architecture issue, so the presence of code is not so critical to deciding it18:38
zanebtherve: I believe so. Keep it focussed on one topic so it doesn't get too wide-ranging. Lay out the pros and cons, make a recommendation, and see if anybody has more to add18:39
*** jprovazn has joined #heat18:41
lchenghi therve, saw this nova error on heat stack create: Message: No valid host was found18:41
lchengCode 50018:41
lchengDetails:18:41
lchengFile “/opt/stack/nova/nova/scheduler/filter_scheduler.py line 107, in schedule_run_instance raise exception.NoValidHost(reason=“”)18:41
lchengFollowed the instruction here: https://wiki.openstack.org/wiki/Heat/GettingStartedUsingDevstack to enable the heat service.18:42
zaneblcheng: sounds like a nova error18:42
lchengzaneb: If I tried to create an instance in nova, it works fine. If I go through heat to create the instance, I get this error.18:43
lchengI used the template reference in https://wiki.openstack.org/wiki/Heat/GettingStartedUsingDevstack18:43
*** andersonvom has joined #heat18:45
*** alexheneveld_ has joined #heat18:46
zaneblcheng: in those instructions you're creating an m1.large server... does that flavor work when you create it directly in Nova?18:46
*** alexheneveld has quit IRC18:47
*** alexheneveld_ is now known as alexheneveld18:47
lchengzaneb: I got the same error when I tried to create m1.large nova instance18:50
zaneblcheng: OK, so the problem is that you don't have enough resources to launch an m1.large18:51
lchengWonder if I create a heat stack with using m1.small?18:51
zanebgood plan18:51
*** gokrokve has joined #heat18:52
*** pafuent has joined #heat18:57
*** blomquisg has quit IRC18:59
*** cmyster has joined #heat19:02
*** cmyster_ has quit IRC19:06
*** e0ne has joined #heat19:11
*** blomquisg has joined #heat19:13
*** IlyaE has quit IRC19:14
*** cody-somerville has quit IRC19:15
*** lcheng has quit IRC19:15
*** alexpilotti has quit IRC19:16
*** rcleere has joined #heat19:16
*** gokrokve_ has joined #heat19:21
andrew_plunkHello everyone. The code in: https://github.com/openstack/heat/blob/master/heat/engine/hot.py#L262  causes an InvalidSchemaError when running a stack-list against a heat that has templates with 'String' instead of 'string'19:23
*** sabeen has quit IRC19:23
andrew_plunkthis schema change was introduced without a template database migration, and I am looking for advice on how to proceed with a fix19:23
openstackgerritIvan Kolodyazhny proposed a change to openstack/python-heatclient: Added missed files to create venv with run_tests.sh  https://review.openstack.org/6994519:23
andrew_plunkI would be fine with introducing a database migration, however uppercase S string is used elsewhere in heat. Are we only allowing lowercase s in hot?19:24
*** gokrokve has quit IRC19:25
*** achampion has quit IRC19:25
*** achampion has joined #heat19:27
*** akuznetsov has joined #heat19:30
*** cmyster_ has joined #heat19:32
*** pablosan has joined #heat19:33
*** cmyster_ has quit IRC19:35
*** cmyster has quit IRC19:36
timsandrew_plunk: I would vote to actually submit a patch that fixes this change to allow backwards compatible syntax in hot templates as an intermediate step19:37
timszaneb: ^^19:37
tims?19:37
timsThe problem is that there are templates and stack representations that currently allow Uppercase in the template for HOT syntax19:38
*** slong has joined #heat19:39
*** IlyaE has joined #heat19:39
*** pshchelo has joined #heat19:40
*** tspatzier has joined #heat19:42
*** sabeen has joined #heat19:42
zanebtims, andrew_plunk: yeah the problem is that the HOT implementation was just kind of hacked up for experimenting, and allows all kinds of things that won't be allowed in actual HOT19:46
zanebthe way I see it, it's a preview and breakage is to be expected19:47
zanebbut I suspect that's not the way people have been using it19:47
timszaneb: agreed, but in the meantime I think a slower transition between backwards incompatibilities is warranted19:47
zanebtims: in what time frame would you imagine removing it?19:49
timsafter Icehouse?19:49
*** jmckind has quit IRC19:49
*** DandyPandy_ is now known as DandyPandy19:49
*** pablosan has quit IRC19:50
*** pablosan has joined #heat19:50
pafuentSpamapS: Are you around?19:50
timszaneb: I probably missed the conversation about when we consider HOT production ready19:51
zanebtims: I'm not strongly opposed to such a patch, but I'm not sure what it buys us19:51
zanebit's been deprecated since forever19:51
zanebpeople are using it anyway; they won't stop until it breaks19:51
timstrue19:52
zanebwhenever we break it will be an unhappy time for those people19:52
zanebthe sooner we do it, the fewer of those people there will be19:52
timssomehow I missed the point when the syntax was deprecated19:52
timsso it would buy dumb people like me time to adjust my templates19:52
tims:P19:52
zanebwell, the point of HOT is to be not cloudformation19:52
zanebso anything that's not defined in HOT but happens to be accidentally inherited from CloudFormation is deprecated by definition19:53
zanebideally we would have raised errors about this from the beginning :(19:54
timszaneb: I agree19:54
timsso maybe it is better to just do a database migration19:54
timsI'll let andrew_plunk chime in on that19:54
timsit can be a gray area though, for people not as acquainted with Heat19:54
timsfor what is HOT and what is an accident19:54
*** akuznetsov has quit IRC19:55
stevebakermorning19:55
pafuentHi, if I need to upload a new patch for an abandoned review, Should I use the same change-id? Should I use the same topic?19:55
stevebakerpafuent: you can restore the change first19:55
zanebtims: I totally agree, it's really not obvious19:55
pafuentstevebaker: The thing is that my change is a quite different than the actual one,19:56
*** lcheng has joined #heat19:56
stevebakerpafuent: your call then. It probably doesn't matter19:57
pafuentstevebaker: Ok, thanks19:57
andrew_plunksorry I was in a meeting zaneb19:59
zanebnp19:59
stevebakerI've added it to the meeting agenda19:59
andrew_plunkthanks stevebaker20:00
andrew_plunkwe can just talk about it then I guess :)20:00
SpamapSpafuent: around now, but -> meeting :)20:03
pafuentSpamapS: Ok, no problem20:03
*** cmyster has joined #heat20:05
pafuentSpamapS: Sorry to bother, but I can't upload my patch over the chmouel one using the same Change-Id. It's ok if I upload as a new review and point to the new review in a comment?20:10
chmouelpafuent: feel free to just send a new review20:10
chmouelpafuent: with a new change-id20:10
pafuentchmouel: Ok, thanks20:11
SpamapSbut why?20:11
SpamapSwas it abandoned?20:11
SpamapSthe review history ise useful.20:11
pafuentSpamapS: Yes20:12
chmouelSpamapS:  well i don't mind since pafuent is the one who actually make it works, we can just link it in the review there?20:13
pafuentSpamapS: I can add links to both reviews20:13
*** alexheneveld_ has joined #heat20:14
SpamapSchmouel: can you activate the change again?20:14
SpamapSthen pafuent can just upload a new patch20:15
chmouelSpamapS: sure20:15
*** alexheneveld has quit IRC20:15
*** alexheneveld_ is now known as alexheneveld20:15
pafuentSpamapS: After that, I need to upload a new patch with the same ChangeId and the same topic, rigth?20:15
*** jdob has quit IRC20:15
chmouelit's restored.20:16
chmouelhttps://review.openstack.org/#/c/63839/20:16
SpamapSpafuent: right, ideally you would have started with 'git review -d 63839' and ammended that patch20:19
pafuentSpamapS: No, I started in a new branch20:20
pafuentSpamapS: My bad.20:20
SpamapSpafuent: easy enough to fix20:21
*** achampion has quit IRC20:21
chmouelpafuent: yeah you just need to adjust your change-id in your new commit to the one from review 63839 (I43464c6079bf7631fda0c4ab112b8f874c6190a3) gerrit will pick it up20:22
pafuentchmouel: If I do that, Gerrit will save the day and I don't need to do the amend thing?20:23
*** achampion has joined #heat20:23
chmouelpafuent: did you create two commit?20:23
chmouelor amended already?20:23
pafuentchmould: Not only one, but I started from master, not your change20:24
chmouelah ok cool so you just need to mach the change-id as i mentioned before20:24
chmoueland you should be good20:24
pafuentpafuent: Ok, thanks20:24
openstackgerritPablo Andres Fuente proposed a change to openstack/heat: New Event uuid column and autoincremental id  https://review.openstack.org/6383920:26
*** cmyster_ has joined #heat20:32
chmouelpafuent: is your change working again all dbs? (i.e: sqlite/postgresql/mysql)20:32
*** cmyster has quit IRC20:32
*** cmyster_ is now known as cmyster20:32
*** cmyster has quit IRC20:32
*** cmyster has joined #heat20:32
pafuentchmouel: I couldn't test for postgresql20:34
chmouelpafuent: oh ok, i guess jenkins would do it for you20:34
chmouelpafuent: it's pretty easy on devstack to test postgresql20:34
pafuentchmouel: But I remember a job with the postgresql word, so I hope it test it20:34
chmouelpafuent: yeah it will20:34
pafuentchmouel: What should I do? Change someting in localrc?20:35
chmouelpafuent: yeah just add that20:35
chmoueldisable_service mysql20:35
chmouelenable_service postgresql20:35
chmouelin your localrc20:35
chmouel(or local.conf)20:35
*** e0ne has quit IRC20:35
pafuentchmouel: Thanks, I'll try that today if I can. If not, I'll do tomorrow.20:36
chmouelpafuent: cool! hopefully by tomo jenkins will let u know if it passed under pgsql20:37
pafuentchmouel: :)20:38
*** jmckind has joined #heat20:42
*** lcheng has quit IRC20:44
*** lcheng has joined #heat20:44
*** coolsvap is now known as coolsvap_away20:46
openstackgerritPablo Andres Fuente proposed a change to openstack/heat: HOT templates get_attr allows extra attributes  https://review.openstack.org/6756220:46
*** lcheng has quit IRC20:48
tspatzierstevebaker: agree on the last think you wrote in #openstack-meeting re template guide21:01
*** andrew_plunk has quit IRC21:01
*** sabeen1 has joined #heat21:01
tspatzierthis in combination with some samples in heat-templates would be best21:01
shardytspatzier: +1, I think we need both21:01
tspatziershardy: yes, because most people start with copy and paste ;-)21:02
*** cmyster_ has joined #heat21:02
*** cmyster has quit IRC21:02
*** sabeen1 has quit IRC21:02
*** cmyster_ is now known as cmyster21:02
*** cmyster has joined #heat21:02
shardytspatzier: 100yy :)21:02
*** sabeen1 has joined #heat21:02
*** sabeen has quit IRC21:03
zanebspeaking of which, there's a template in heat-templates with String and Number type params21:04
zanebfixing now...21:04
cmysterright.21:04
cmysterwell21:04
*** asalkeld_ has quit IRC21:05
*** asalkeld has joined #heat21:06
cmysternn folks21:09
*** cmyster has quit IRC21:09
*** pshchelo has quit IRC21:09
*** lcheng has joined #heat21:10
tspatzierzaneb: good catch :-)21:10
openstackgerritAlexander Ignatov proposed a change to openstack/heat: Add .idea folder to .gitignore  https://review.openstack.org/6998121:11
*** pablosan has quit IRC21:13
*** pablosan has joined #heat21:14
tspatzierzaneb: I filed a bug to remove other inconsistencies from the HOT code: https://bugs.launchpad.net/heat/+bug/127428821:16
*** alexheneveld has quit IRC21:17
zanebtspatzier: thanks. I actually just noticed that for the first time today when I was looking at pafuent's patches21:17
tspatzieryes, this is some legacy from the early days of hot that we never cleaned up so far. Now is the time ;-)21:18
zanebtspatzier: wait... does it actually allow that in the template?21:19
tspatzierAs far as I know it does.21:19
zaneb__getitem__ always translates the key to lower case21:19
zanebso if the uppercase key is in the template, it won't be found?21:20
stevebakerSpamapS: could you take a look at my reply to https://review.openstack.org/#/c/66230/ ?21:20
zaneboh, lower-level things might be different21:20
tspatzierthere is _translate_resource method in HOTemplate which translates e.g. 'type' into 'Type' but it does not complain when 'Type' is already there.21:20
*** lcheng has quit IRC21:20
zanebtspatzier: quite right21:21
zanebyeah, we should knock that on the head21:21
*** andersonvom has quit IRC21:22
*** andersonvom has joined #heat21:22
tspatzierso to force people to write templates with lowercase keys only, I could add validation to HOTemplate but still keep the _translate method.21:23
tspatzierEven though longer term it would be nice to have an internal representation closer to hot, so with lowercase keys. Similar to what we have with the param schema now.21:24
zaneblonger term it would be nice to have an internal representation that is _irrelevant_ because we only interact with it through an API :)21:25
tspatziereven better :-)21:26
zanebfor now I think you can keep the translate and just not allow it to default to the current value21:26
tspatzieryes, this is what I'm going to do. This way everyone writing a template at least gets told something is wrong.21:27
tspatzier... well, not in line with the docs21:27
zanebtspatzier, andrew_plunk: what if instead of a series of database migrations, we just maintained a single script that went through your database looking for bad templates and fixed them up?21:28
zanebwhenever one of your old templates breaks, you run the script21:29
*** andrew_plunk has joined #heat21:29
zanebI'll send that again then21:30
zanebtspatzier, andrew_plunk: what if instead of a series of database migrations, we just maintained a single script that went through your database looking for bad templates and fixed them up?21:30
zanebthe script would be idempotent, obviously21:30
tspatzierzaneb: sounds good. If andrew_plunk is done with his first shot of the script, I could add to it in the patch for the bug I opened.21:30
andrew_plunkwhat would be the advantage?21:30
*** slong has quit IRC21:30
jasond`zaneb: i like that approach21:31
SpamapSstevebaker: thanks for clarifying ;-)21:31
andrew_plunkzaneb. I wonder why you would need to run it more than once?21:31
zanebandrew_plunk: db migrations are messy :(21:31
zaneband slow21:31
zanebandrew_plunk: because we'll probably continue to make more breaking changes21:32
jasond`andrew_plunk: seems like a lot less overhead than a new migration script every time a problem is found21:32
andrew_plunkthe script is going to be do the same work the db migration would do, without the benefit of the framework the migrations have21:32
zanebandrew_plunk: e.g. https://bugs.launchpad.net/heat/+bug/127428821:32
zanebandrew_plunk: the migrations framework is just sqlalchemy-migrate, you can use it anywhere21:33
zanebbesides, you don't want to make a schema migration, just a data migration21:33
zanebthat just requires ordinary sqlalchemy21:33
andrew_plunkthen people will have to know about a separate script to run instead of the database migration that we all know and love ;)21:34
andrew_plunkbecause a lot of our database is schema less, this kind of migration is going to become more and more common21:35
andrew_plunkso why not do the migration within the framework we have already setup within heat-manage.21:35
andrew_plunkwhen something breaks with the database I know exactly what I need to do to fix it, run db_sync21:35
zanebfair enough21:36
zanebmy concern is that this will be very slow, and may run multiple times in the course of an upgrade21:37
tspatzierandrew_plunk: whatever the solution will be, what is your timeline when you wan to work on this? Because I would wait for a first shot of it before fixing bug #127428821:37
andrew_plunkI am going to do my best to finish it today tspatzier21:37
andrew_plunkzaneb: Let me try it out, if it is really slow I will do it your way21:37
tspatzierandrew_plunk: wow, that's quick :-)21:38
andrew_plunkIt is my top priority tspatzier21:38
*** jprovazn has quit IRC21:38
andrew_plunkthis schema change is breaking my builds21:38
*** andersonvom has quit IRC21:39
tspatzierok, I'll stay tuned21:39
*** tspatzier has quit IRC21:43
*** pafuent has left #heat21:43
*** andrew_plunk has quit IRC21:45
*** e0ne has joined #heat21:46
*** e0ne has quit IRC21:50
*** pablosan has quit IRC21:54
*** andrew_plunk has joined #heat21:57
*** SergeyLukjanov is now known as SergeyLukjanov_22:01
*** cody-somerville has joined #heat22:13
*** aweiteka has quit IRC22:15
*** andersonvom has joined #heat22:19
*** jmckind has quit IRC22:21
jasond`does anybody know what would cause my heat to suddenly start using v1.0 of the compute API?22:21
*** lcheng has joined #heat22:22
*** lcheng has quit IRC22:23
*** andersonvom1 has joined #heat22:24
*** lcheng has joined #heat22:24
*** andersonvom has quit IRC22:25
*** cody-somerville has quit IRC22:25
*** tango|2 has joined #heat22:27
*** aignatov is now known as aignatov_22:27
*** slong has joined #heat22:30
stevebakerzaneb: can you take a look at the handle_update discussion in https://review.openstack.org/#/c/62287 ?22:31
stevebakerjasond`: an old nova client?22:31
sdakejasond` heat only uses python-novaclient, so any interaction with nova comes through that python client22:32
zanebugh, this OpenShift template is just an unholy mix of HOT and cfn all copied and pasted into the same file :(22:32
*** rpothier has quit IRC22:33
*** jasond` has quit IRC22:34
*** pablosan has joined #heat22:40
*** pablosan has quit IRC22:40
*** pablosan has joined #heat22:41
*** yogesh has quit IRC22:41
*** rcleere has quit IRC22:42
*** andersonvom1 is now known as andersonvom22:42
*** jcru has quit IRC22:48
openstackgerritZane Bitter proposed a change to openstack/heat-templates: Run dos2unix over template files  https://review.openstack.org/7000422:49
openstackgerritZane Bitter proposed a change to openstack/heat-templates: Fix incorrect HOT Parameter types  https://review.openstack.org/7000522:49
openstackgerritZane Bitter proposed a change to openstack/heat-templates: Fix cfn-isms in OpenShift Enterprise Neutron template  https://review.openstack.org/7000622:49
*** giulivo has quit IRC22:49
zanebeasy reviews if anyone wants 'em ^22:52
*** radez is now known as radez_g0n322:52
*** achampion has quit IRC22:52
sdakeoff to dinner with the folks b bl22:53
andersonvomhey folks, does anybody know why in heat/engine/clients.py we have a bunch of try/except for the case where the client is not available even though all of them are in our requirements.txt file?22:55
*** IlyaE has quit IRC22:55
zanebandersonvom: back in the day, they were optional22:55
zanebalso, not everybody runs devstack22:55
stevebakerzaneb: where do you install clients from now? RDO?22:56
zanebpypi22:56
zanebI've recently started using a venv to run unit tests22:57
*** IlyaE has joined #heat22:57
zanebso it installs them automatically, but I only started doing that recently22:57
andersonvomzaneb: but aren't we expecting people to  "pip install -r requirements.txt" anyway?22:58
zanebandersonvom: candidly, no, and if anyone deployed their cloud like that they should be strung up ;)22:59
andersonvomzaneb: cool. thanks!22:59
stevebakerunless its triplo AMIRITE?22:59
zanebpip is evil23:00
*** andrew_plunk has quit IRC23:02
*** IlyaE has quit IRC23:03
SpamapSpip is the worst python packaging tool.. except all the others.23:08
* SpamapS would like to apologize to the entire UK for that paraphrase23:08
zanebSpamapS: easy_install is the *worst* Python packaging tool23:10
zanebstevebaker: added comment on that review. Usual problem23:17
*** kebray_ has quit IRC23:17
zanebstevebaker: the every-api-call-must-be-a-resource problem23:18
*** achampion has joined #heat23:30
stevebakerzaneb: right23:31
andersonvomstevebaker: you there?23:32
*** sabeen1 has quit IRC23:32
andersonvomstevebaker: the issue (https://review.openstack.org/#/c/68746/) also applies to regular resources that may have unmet dependencies. Would it be enough to change the commit message to say "resources" as opposed to "contrib resources"?23:35
stevebakerandersonvom: Ah, i see23:35
*** achampion has quit IRC23:35
stevebakerMaybe the help should be "Force resources to be added to the registry even if their runtime dependencies are not met"23:36
*** tims has quit IRC23:36
andersonvomstevebaker: oh yeah... true! that as well!23:37
stevebakerwe could use that flag for core resources too, good for docs building23:37
andersonvomo/23:38
andersonvomyeah... that was our point precisely when asking about the clients.py try/except part23:38
stevebakerI can't find it now, but there was that blueprint to only register resources if the endpoint was published in the keystone catalog <- the flag could bypass that logic too23:39
*** tomek_adamczewsk has joined #heat23:41
*** achampion has joined #heat23:41
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Add Docker resources to docs  https://review.openstack.org/6875323:44
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Rename rackspace.rst to contrib.rst  https://review.openstack.org/6875223:44
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Add Rackspace resources summary to docs  https://review.openstack.org/6879323:44
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Add force_resource_mapping option  https://review.openstack.org/6874623:44
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Restructure Rackspace resources  https://review.openstack.org/6874723:44
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Reorganize docker-plugin for consistency  https://review.openstack.org/6874823:44
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Allow force registration of docker resource  https://review.openstack.org/6874923:44
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Allow force registration of Rackspace resources  https://review.openstack.org/6937423:44
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Change Docker resource mapping name  https://review.openstack.org/6875023:44
openstackgerritAnderson Mesquita proposed a change to openstack/heat: Generate docs for contrib plugins  https://review.openstack.org/6875123:44
*** tomek_adamczewsk has quit IRC23:52
*** dims has quit IRC23:53
*** IlyaE has joined #heat23:54
*** IlyaE has quit IRC23:55
*** cadenzajon has quit IRC23:57
zanebstevebaker: I sent an email to the mailing list about how to model resources in general... at least I will have something to point to next time ;)23:58
stevebakerzaneb: we should have a page in our developer guide23:59
zanebgood idea23:59
kebrayshardy, you around?23:59

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