Thursday, 2014-06-19

*** DandyPandy has quit IRC00:01
*** ericgoncz has quit IRC00:13
openstackgerritKevin Fox proposed a change to openstack/heat: Add a OS::Nova::ServerGroup resource.  https://review.openstack.org/10099500:16
*** andersonvom has joined #heat00:20
*** matsuhashi has joined #heat00:20
*** tango has quit IRC00:21
*** Qiming has joined #heat00:25
*** randallburt has joined #heat00:26
*** nati_ueno has quit IRC00:26
*** randallburt has quit IRC00:26
*** randallburt has joined #heat00:26
kfox1111hi Randall00:27
randallburthey00:27
*** andersonvom has quit IRC00:28
*** ramishra has joined #heat00:28
kfox1111thanks for the review.00:28
randallburtnp00:28
kfox1111I think I have everything done now.00:28
kfox1111Way more work then I would have hoped. :/00:29
kfox1111A good learning experience though.00:29
randallburtit always is. (work, that is ;)00:29
randallburtcan you shoot me a link? I lost the email.00:29
kfox1111https://review.openstack.org/#/c/100995/00:30
randallburtlooking00:30
*** zaneb has quit IRC00:31
*** ramishra has quit IRC00:32
sjmc7anyone still here?00:42
sjmc7:)00:42
sjmc7stevebaker, you got a sec?00:42
*** nosnos has joined #heat00:43
randallburtkfox1111:  a couple more minor nits, but LGTM. sorry about the older tests as examples.00:44
*** zaneb has joined #heat00:44
sjmc7zaneb - sorry to bug you again00:45
randallburtits like you summoned him sjmc7 ;)00:45
sjmc7but real quick, the problem appears to be in update_and_save in the sqlalchemy interface - adding session.merge(self) causes DB changes to get committed properly00:45
sjmc7randallburt :)  i fear i have outstayed my welcome today00:46
*** dims_ has quit IRC00:46
randallburt:D00:46
sjmc7i'll file a bug for this00:46
randallburtsjmc7:  k.00:46
sjmc7and once more swear at ORMs in general00:47
*** dims_ has joined #heat00:48
*** kfox1111 has quit IRC00:48
*** DandyPandy has joined #heat00:50
*** DandyPandy has quit IRC00:51
openstackgerritRandall Burt proposed a change to openstack/heat: Update to latest version of pyrax and add Swift support  https://review.openstack.org/9465800:55
openstackgerritRandall Burt proposed a change to openstack/heat: Account for differences in Rackspace Cloud Glance  https://review.openstack.org/9633700:55
*** sjmc7 has quit IRC00:57
*** TravT has quit IRC01:01
*** dims_ has quit IRC01:05
openstackgerritOpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements  https://review.openstack.org/9650701:07
*** m_22 has joined #heat01:08
*** spzala has quit IRC01:10
*** nanjj`` has joined #heat01:11
*** samstav has joined #heat01:13
*** nanjj`` has quit IRC01:18
*** tiantian has joined #heat01:19
tiantianmorning all :)01:19
*** RockKuo_Office has joined #heat01:21
*** Qiming has quit IRC01:23
*** m_22 has quit IRC01:23
*** gokrokve has quit IRC01:25
*** gokrokve has joined #heat01:25
liushengmorning01:28
*** ramishra has joined #heat01:29
*** mestery has joined #heat01:29
*** gokrokve has quit IRC01:30
*** ramishra has quit IRC01:34
*** mestery has quit IRC01:34
*** bgorski has quit IRC01:39
SpamapSstevebaker: FYI: https://review.openstack.org/10107301:46
*** ericgoncz has joined #heat01:46
SpamapSstevebaker: testing using os-collect-config locally.. we'll see how it goes. :)01:46
* SpamapS calls it a day01:47
*** mestery has joined #heat01:54
*** DandyPandy has joined #heat01:57
*** gokrokve has joined #heat01:59
*** Tross has quit IRC02:00
*** zhangyang has quit IRC02:03
*** tiantian has quit IRC02:04
*** liusheng has quit IRC02:04
*** liusheng has joined #heat02:04
*** zhangyang has joined #heat02:04
*** tiantian has joined #heat02:04
elynnmorning guys :)02:07
*** matsuhashi has quit IRC02:07
*** matsuhashi has joined #heat02:08
*** matsuhashi has joined #heat02:08
*** arbylee has joined #heat02:12
*** mestery_ has joined #heat02:20
*** mestery_ has quit IRC02:21
zhangyangstevebaker:this patch needs your review, https://review.openstack.org/#/c/100830/02:21
*** mestery_ has joined #heat02:22
*** mestery has quit IRC02:23
stevebakerzhangyang: does that change stop all the deprecation warnings in a test run?02:24
stevebakerzhangyang: I suspect the barbican resources landed at the same time, which also do the old way02:25
*** DandyPandy has quit IRC02:26
*** ramishra has joined #heat02:29
*** bnemec has quit IRC02:30
*** samstav has quit IRC02:31
openstackgerrithuangtianhua proposed a change to openstack/heat: Fix count for stack-list while show deleted  https://review.openstack.org/10107802:31
*** achampion has quit IRC02:32
tiantianstevebaker: would you please to review the patch: https://review.openstack.org/#/c/100804/ thanks02:33
*** ramishra has quit IRC02:35
*** ericgoncz_ has joined #heat02:36
*** ericgoncz has quit IRC02:37
*** ericgoncz_ is now known as ericgoncz02:37
*** ramishra has joined #heat02:38
zhangyang<stevebaker>: just saw your commit, I will do it as you suggested, and I just studied the barbican code, it seems the metadata isn't used. Do I miss something?02:44
stevebakerzhangyang: if you've fixed all the deprecation warnings then ignore me02:44
zhangyangalright, tks.02:45
*** ericgoncz has quit IRC02:45
*** mestery_ is now known as mestery02:48
*** ramishra has quit IRC02:51
*** rpothier has joined #heat02:53
*** ramishra_ has joined #heat02:56
*** achampion has joined #heat02:58
elynnhi, anyone can review these two patches? https://review.openstack.org/#/c/100191/ https://review.openstack.org/#/c/100178/02:59
*** ramishra has joined #heat03:00
*** gokrokve has quit IRC03:00
*** ramishra_ has quit IRC03:01
*** achampion has quit IRC03:03
*** samstav has joined #heat03:13
*** cmyster has joined #heat03:27
*** rwsu has quit IRC03:27
cmystermorning03:28
cmysterstevebaker: evening, here ?03:28
stevebakercmyster: afternoon03:29
cmysterhey, Jenkins is finally playing nice, can you please check https://review.openstack.org/#/c/89790/ if there is anything that I need to add ?03:29
*** gokrokve has joined #heat03:33
*** ramishra has quit IRC03:33
*** zhiyan_ is now known as zhiyan03:34
*** gokrokve_ has joined #heat03:36
stevebakercmyster: looks good03:36
* SpamapS realizes he has not run devstack in over a year. :-P03:37
*** blamar has joined #heat03:38
*** blamar has quit IRC03:38
*** matsuhashi has quit IRC03:38
cmysterthanks stevebaker,03:39
*** matsuhashi has joined #heat03:39
cmysterSpamapS: its easy, stack.sh, wait for it to break, fix it locally, unstack, stack, repeat ;)03:39
*** gokrokve has quit IRC03:39
cmysterJ/K its working rather well the last few attempts I had with it over the last few weeks03:40
SpamapScmyster: I refuse to subject my laptop to stack.sh03:41
SpamapScmyster: I'll run it in hpcloud instances. :)03:41
* cmyster does it on bare metal03:41
*** akuznetsov has joined #heat03:43
*** matsuhashi has quit IRC03:45
*** matsuhashi has joined #heat03:45
*** matsuhashi has quit IRC03:46
*** jyoti_ranjan has joined #heat03:47
*** nosnos has quit IRC04:02
*** mestery has quit IRC04:06
*** mestery has joined #heat04:07
*** sab has joined #heat04:09
*** mestery has quit IRC04:11
*** zhangyang has quit IRC04:24
*** zhangyang has joined #heat04:24
*** randallburt has quit IRC04:26
openstackgerrithuangtianhua proposed a change to openstack/heat: Fix count for stack-list while show deleted  https://review.openstack.org/10107804:30
*** samstav has quit IRC04:35
*** matsuhashi has joined #heat04:37
*** kopparam has joined #heat04:42
*** kirankv has joined #heat04:42
*** nosnos has joined #heat04:43
*** akuznetsov has quit IRC04:53
*** nkhare has joined #heat04:56
*** matsuhashi has quit IRC04:56
*** matsuhashi has joined #heat04:57
*** akuznetsov has joined #heat04:58
*** achampion has joined #heat05:05
*** kopparam has quit IRC05:11
*** kopparam has joined #heat05:11
*** achampion has quit IRC05:12
*** alexpilotti has joined #heat05:12
*** nkhare has quit IRC05:21
*** nati_ueno has joined #heat05:23
skraynevgood morning05:27
elynnHi stevebaker , SpamapS, skraynev do you have time to review these two patches? https://review.openstack.org/#/c/98042/ https://review.openstack.org/#/c/98580/05:29
elynnNeed your feedback :)05:29
skraynevelynn: eyah, I planned to do it after review "client-as-plugins" patches. IMO, looks good, but I wanted to test it else ;)05:31
elynnthx skraynev :)05:31
elynnreview codes is a hard work.05:33
*** tiantian has quit IRC05:34
*** rakesh_hs2 has joined #heat05:35
*** bmahalakshmi has joined #heat05:37
*** liusheng has quit IRC05:43
*** akuznetsov has quit IRC05:43
*** liusheng has joined #heat05:43
*** Demitar has joined #heat05:50
*** harlowja is now known as harlowja_away05:54
*** nkhare has joined #heat05:55
*** Demitar has quit IRC05:55
*** morganfainberg has quit IRC05:57
*** morganfainberg has joined #heat05:58
*** lazy_prince has joined #heat06:00
*** kopparam has quit IRC06:01
*** alexpilotti has quit IRC06:01
*** alexpilotti has joined #heat06:01
*** kopparam has joined #heat06:01
openstackgerritOpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex  https://review.openstack.org/8975006:02
*** akuznetsov has joined #heat06:09
skraynevelynn: +1. and very important :)06:12
*** rakesh_hs2 has quit IRC06:15
*** rakesh_hs2 has joined #heat06:16
elynnthx skraynev :)06:22
*** tiantian has joined #heat06:22
*** andreaf has joined #heat06:22
*** jprovazn has joined #heat06:32
*** kopparam has quit IRC06:42
*** e0ne has joined #heat06:45
openstackgerrithuangtianhua proposed a change to openstack/heat: Fix error count for stack-list while show deleted  https://review.openstack.org/10107806:59
*** elynn has quit IRC07:02
*** andreaf has quit IRC07:03
*** achampion has joined #heat07:03
*** gokrokve_ has quit IRC07:05
*** PhilK has quit IRC07:05
*** e0ne has quit IRC07:06
*** PhilK_ has joined #heat07:07
*** shardy_afk is now known as shardy07:08
shardymorning all07:08
*** jcoufal has joined #heat07:10
openstackgerritZhang Yang proposed a change to openstack/heat: Hide deprecate warnings for metadata tests  https://review.openstack.org/10083007:11
therveGood morning!07:11
cmystermorning07:11
*** Michalik- has joined #heat07:12
*** gokrokve has joined #heat07:15
*** ajc_ has joined #heat07:17
*** mcwoods_ has quit IRC07:17
*** zhangyang has quit IRC07:19
*** zhangyang has joined #heat07:19
*** gokrokve has quit IRC07:19
*** kragniz_ has joined #heat07:24
*** baffle_ has joined #heat07:24
*** kragniz has quit IRC07:27
*** baffle has quit IRC07:27
*** dteselkin has quit IRC07:27
*** dteselkin has joined #heat07:28
*** achampion has quit IRC07:30
*** gokrokve has joined #heat07:35
*** mcwoods has joined #heat07:36
pas-hamorning all07:40
*** gokrokve has quit IRC07:40
openstackgerrithuangtianhua proposed a change to openstack/python-heatclient: Show physical_resource_id in resource-list  https://review.openstack.org/10112507:44
*** kopparam has joined #heat07:52
cmystershardy: mind checking https://review.openstack.org/#/c/89790/ ? jenkins is in a rare small working window07:59
*** alexpilotti has quit IRC08:00
*** kopparam has quit IRC08:01
*** nati_ueno has quit IRC08:01
*** kopparam has joined #heat08:01
*** chandan_kumar has quit IRC08:01
*** alexpilotti has joined #heat08:02
*** e0ne has joined #heat08:05
openstackgerritSergey Kraynev proposed a change to openstack/heat: Adding validation algorithm for get attr functions  https://review.openstack.org/8248808:06
*** e0ne has quit IRC08:13
*** jyoti_ranjan has quit IRC08:15
*** e0ne has joined #heat08:23
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::NodeGroupTemplate resource  https://review.openstack.org/10028808:27
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement sahara client plugin  https://review.openstack.org/10058808:27
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::Cluster resource  https://review.openstack.org/7233608:27
*** gokrokve has joined #heat08:27
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::NodeGroupTemplate resource  https://review.openstack.org/10028808:29
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement sahara client plugin  https://review.openstack.org/10058808:29
openstackgerritPavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::Cluster resource  https://review.openstack.org/7233608:29
*** gokrokve has quit IRC08:32
*** elynn has joined #heat08:33
sabI am getting an error "AttributeError: 'Module_six_moves_urllib_parse' object has no attribute 'SplitResult'", can anyone help me?08:35
*** andreaf has joined #heat08:37
elynnsab, maybe you can paste your log here :)08:37
elynnand how you get this error.08:38
*** jyoti_ranjan has joined #heat08:39
sabelynn, http://paste.openstack.org/show/84466/08:39
*** maishsk has joined #heat08:40
sabthis is while just starting up using devstack,08:40
* maishsk says good morning !08:40
maishskanyone here able to help with a question about docker and heat?08:40
*** Qiming has joined #heat08:41
maishskkeep on getting this message08:42
maishskERROR: Unknown resource Type : DockerInc::Docker::Container08:43
elynnsab, how do you get this error?08:43
elynncreate a stack?08:43
openstackgerrithuangtianhua proposed a change to openstack/heat-specs: Events pagination and sorting Specification  https://review.openstack.org/10113308:43
skraynevmaishsk: looks like your have not such resource08:43
elynnsab, what version of six library do you use?08:44
skraynevmaishsk: AFAIK, all resources in contrib directory are not avaliable by default08:44
*** nosnos has quit IRC08:44
shardycmyster: done, lgtm!08:44
maishskskraynev: I dont need to have docker installed on the controller - if I remember correctly08:44
maishskskraynev: I followed instructions from here08:45
maishskhttps://github.com/openstack/heat/tree/master/contrib/docker/docker08:45
maishskskraynev: my heat.conf has this plugin_dirs=/root/heat/contrib/docker/docker08:46
*** saju_m has joined #heat08:46
maishskthe github is not correct08:46
cmysterwoot08:46
*** nosnos has joined #heat08:46
cmysterthanks shardy,08:46
skraynevmaishsk: no, it's correct08:47
maishskthere is no plugin folder08:47
cmysternow I need someone from tempest to +1 it as well ?08:47
skraynevmaishsk: you may check that this plugin was loaded. look in heat-engine.log for this information.08:48
sabelynn, its 1.5.208:48
shardycmyster: You need two tempest-cores to review it, I've just added them all as reviewers08:49
sabelynn, heat-eng doesn't even start  :)08:49
elynnsab, I think you hit this bug, https://bugs.launchpad.net/devstack/+bug/131632808:50
uvirtbotLaunchpad bug 1316328 in devstack "Stack trace when stack.sh hits "cinder-manage db sync"" [Undecided,Fix released]08:50
maishskskraynev: I dont understand - are the instructions on Github correct?08:50
maishskskraynev: there is no plugin folder08:51
maishskskraynev: or should it be set that way - regardless?08:51
skraynevmaishsk:  instructions on Github is correct. It's right way08:51
elynnsab, you can try to upgrade six to 1.6.0 and see if this error still exist08:51
shardymaishsk: is /root accessible to the stack user?08:52
sabelynn, this is after I did a git pull yesterday or today morning. Yes, I will try updating08:52
shardyI would assume not, so shouldn't that be /opt/stack/heat/contrib...08:52
maishskskraynev: Even though there is plugin folder?08:52
maishskskraynev: I am not using devstack08:53
skraynevmaishsk: Hm. I think, that it should work for you.08:54
*** rakesh_hs2 has quit IRC08:54
skraynevmaishsk: my suggestion is to check that plugin was registered.08:54
*** julienvey has joined #heat08:54
maishskI dont think it was08:54
maishskskraynev: the folder is not there - so the instructions will not work08:55
maishsk[root@icontrol ~(keystone_veeas)]# cd heat/contrib/docker-plugin/plugin08:55
maishsk-su: cd: heat/contrib/docker-plugin/plugin: No such file or directory08:55
maishsk[root@icontrol ~(keystone_veeas)]# cd heat/contrib/docker/docker/08:56
openstackgerrithuangtianhua proposed a change to openstack/heat-specs: Events pagination and sorting Specification  https://review.openstack.org/10113308:56
maishskskraynev: that works08:56
skraynevit's normal08:57
skraynevhave you any log files? In this case log for heat-engine process.08:57
*** rakesh_hs2 has joined #heat08:58
maishski do08:58
openstackgerrithuangtianhua proposed a change to openstack/heat-specs: Events pagination and sorting Specification  https://review.openstack.org/10113308:59
tiantianshardy: would you review https://review.openstack.org/#/c/101078/ ths08:59
maishskskraynev: see http://paste.openstack.org/show/84467/ - just restarted the service09:00
maishskskraynev: I have removed the IP's from the logs...09:01
maishskskraynev: I don't see anything there regarding docker whil restarting heat-engine09:02
*** piyush has joined #heat09:02
skraynevmaishsk: ok, let me look..09:03
sabelynn, great, thanks . its works :)09:04
elynnsab, np :)09:04
maishskskraynev: Thanks!09:05
skraynevmaishsk: hm . really looks, like docker plugin was not registered.09:05
maishskskraynev: which takes me back to the question - how can I point in the heat.conf file to a folder that does not exist09:06
maishskskraynev: which is why I think that the README is not correct09:07
*** Demitar has joined #heat09:07
openstackgerrithuangtianhua proposed a change to openstack/heat-specs: Events pagination and sorting Specification  https://review.openstack.org/10113309:09
skraynevmaishsk: Oh. Now I understand you fully09:09
skraynevmaishsk: so it's bug in documentation.09:09
maishskskraynev: OK09:10
maishsk:)09:10
skraynevmaishsk: possibly it was missed, when it was renamed :)09:10
maishskskranyev: so it should be heat/contrib/docker/docker/09:10
maishsk ?09:10
skraynevmaishsk: I think so09:11
maishskOK09:11
maishskshould I will put it a code change for the documentation?09:11
*** e0ne_ has joined #heat09:11
skraynevmaishsk: Try to do it and if it works correct for you, please upload a fix09:11
maishskskraynev: so that is the problem - it does not09:12
maishskskraynev: as you can see from the log...09:12
tiantianshardy: hi, I write a spec for a bp, and add it in the 'juno' floder, the jenkins run failed, the error is "document isn't included in any toctree", so I think I can't add the 'juno' floder for the heat-spec09:12
skraynevmaishsk: I mean test, that docker/docker works for you :)09:13
*** e0ne has quit IRC09:13
maishskskraynev: It does not09:13
*** openstackgerrit has quit IRC09:14
*** elynn has quit IRC09:14
maishskskraynev: requirements are met - see http://paste.openstack.org/show/84469/09:14
*** piyush has quit IRC09:15
*** Fayablazer has joined #heat09:17
*** Demitar_ has joined #heat09:17
skraynevmaishsk: do you think that problem is not only in directory name?09:19
*** Demitar has quit IRC09:19
maishskskraynev: I dont know - I cannot get it to work...09:20
skraynevmaishsk: ok, as I understood you right you have docker plugin for this path09:20
skraynev/root/heat/contrib/docker/docker09:21
skraynevright?09:21
maishskskraynev: correct09:21
skraynevand same path is provided in config file?09:21
skraynevplugin_dirs=/root/heat/contrib/docker/docker09:22
maishskskraynev: in heat.conf ---> plugin_dirs=/root/heat/contrib/docker/docker09:22
skraynevmaishsk: and I hope, that it's not single line in heat.conf ?09:24
maishskskraynev: no it is not the only line in heat.conf09:24
skraynevfooh.. good news :)09:24
cmysterthanks shardy, I noted the names, I'll remember to add them to other patches as well09:25
skraynevnext. have heat access to this file ?09:25
shardycmyster: You can just add tempest-core in the add reviewer box09:25
shardysame for all projects09:25
shardyI'd suggest not doing it unless a patch is ready and has been posted for some time though09:26
shardyas it'll spam everyone with notifications via email09:26
cmysterkk09:27
skraynevmaishsk: in my case I have such log http://paste.openstack.org/show/84470/ , after adding plugin_dirs=../contrib/docker/docker in heat.conf09:27
skraynevmaishsk: line 17 means, that it was registered09:28
maishskskraynev: looking to see if permissions are correct.09:28
maishskone sec09:28
*** gokrokve has joined #heat09:28
*** Demitar_ has quit IRC09:28
skraynevmaishsk: It's my last guess about your problem :(09:29
shardymaishsk: is the heat-engine actually running as root though?09:30
maishskno it's not09:30
maishskskraynev: heat     14399  1.5  0.0 291340 72340 ?        Ss   08:58   0:30 /usr/bin/python /usr/bin/heat-engine09:31
shardythen it's not going to be able to access a plugin in /root then is it?09:31
maishskshardy: Nope.... ;)09:31
*** gokrokve has quit IRC09:33
skraynevmaishsk: I do not use root, because my repo is placed in not root directory:)09:33
*** Demitar has joined #heat09:35
*** cmyster has quit IRC09:35
*** e0ne_ has quit IRC09:38
skraynevshardy: about readme for contrib resources. How do you think: should we also fix rewriting whole heat.conf file ?09:38
*** e0ne has joined #heat09:39
skraynevshardy: https://review.openstack.org/#/c/101144/2/contrib/docker/docker/README.md09:39
shardyskraynev: lol, yes ;)09:40
skraynevshardy: IMO, using ">" for writing plugin_dirs in heat.conf looks wrong09:40
*** nosnos has quit IRC09:40
skraynevshardy: I copy-pasted these commands and got awesome result with short heat.conf :)09:41
*** jprovazn has quit IRC09:42
*** e0ne has quit IRC09:43
*** matsuhashi has quit IRC09:44
*** matsuhashi has joined #heat09:44
*** matsuhashi has quit IRC09:46
*** matsuhashi has joined #heat09:47
*** akuznetsov has quit IRC09:52
maishsk:)09:54
*** Qiming has quit IRC09:58
maishskskraynev: added that to the change.10:00
maishskskraynev: ok - some progress - but now have a new error.10:03
maishskskraynev: http://paste.openstack.org/show/84475/10:04
*** cdent has joined #heat10:07
skraynevmaishsk: wait a minute, I will look10:09
skraynevmaishsk: I have only one idea: You use old code for main services (where attributes module have not own schema class), and docker resource is a new code.10:13
*** jprovazn has joined #heat10:14
maishskskraynev: that could be - how can I verify?10:14
skraynevmaishsk: make git log in both repositories that you use. and compare date of last commit10:16
skraynevmaishsk: cd /root/heat; git log10:16
skraynevand cd <directory with heat, that installed in you system>; git log10:17
tiantianbye all:)10:17
skraynevin case with devstack it's /opt/stack/heat10:17
skraynevtiantian: bye10:17
maishskskraynev: the installation is from a package10:18
maishskMetadata-Version: 1.110:18
maishskName: heat10:18
maishskVersion: 2014.110:18
maishskSummary: OpenStack Orchestration10:18
maishskHome-page: http://www.openstack.org/10:18
maishskAuthor: OpenStack10:18
maishskAuthor-email: openstack-dev@lists.openstack.org10:18
maishskLicense: UNKNOWN10:18
maishskDescription: ====10:18
skraynevmaishsk: yes, this version has not new attribute schema10:20
maishskskraynev: joy... :(10:20
skraynevmaishsk: so you have two ways: use new package or downgrade /root/heat repo10:21
skraynevI prefer second10:21
maishskwill try10:21
skraynevcd /root/heat; git checkout -t aa6fa5e975606576c759ed14d1e4d67186e2a37710:22
*** Demitar_ has joined #heat10:25
*** Demitar has quit IRC10:28
*** julienvey has quit IRC10:28
*** gokrokve has joined #heat10:29
*** gokrokve_ has joined #heat10:31
*** Demitar has joined #heat10:32
*** gokrokve has quit IRC10:33
*** Demitar_ has quit IRC10:34
*** gokrokve_ has quit IRC10:35
*** Qiming has joined #heat10:51
*** Demitar_ has joined #heat10:53
*** Demitar has quit IRC10:57
*** zhiyan is now known as zhiyan_10:57
*** Demitar has joined #heat10:59
*** Demitar_ has quit IRC10:59
*** matsuhashi has quit IRC11:00
*** matsuhashi has joined #heat11:01
*** e0ne has joined #heat11:05
*** matsuhashi has quit IRC11:05
*** sorantis has joined #heat11:06
*** dmakogon_ has quit IRC11:15
*** dmakogon_ has joined #heat11:15
*** matsuhashi has joined #heat11:16
*** Demitar_ has joined #heat11:18
*** Demitar has quit IRC11:21
*** maishsk has quit IRC11:23
*** tiantian has quit IRC11:24
*** Demitar has joined #heat11:30
*** gokrokve has joined #heat11:32
*** Demitar_ has quit IRC11:33
*** RockKuo_Office has quit IRC11:35
*** gokrokve has quit IRC11:37
*** akuznetsov has joined #heat11:39
*** dmakogon_ is now known as denis_makogon11:43
*** Rajalakshmi has quit IRC11:45
*** Demitar_ has joined #heat11:48
*** Demitar has quit IRC11:49
*** rpothier has quit IRC11:49
*** ajc_ has quit IRC11:50
*** Demitar has joined #heat11:50
*** ajc_ has joined #heat11:50
*** Demitar_ has quit IRC11:53
*** sab has quit IRC11:53
*** bmahalakshmi has quit IRC11:54
*** julienvey has joined #heat11:54
*** ajc_ has quit IRC11:55
*** BillArnold has quit IRC11:56
*** Demitar_ has joined #heat11:58
*** kopparam has quit IRC11:59
*** Demitar has quit IRC12:01
*** mkollaro has quit IRC12:03
*** nkhare has quit IRC12:04
*** mkollaro has joined #heat12:05
afaranhaHi all, how can I init an instance with ubuntu user and password (instead of the security key) using the heat template?12:10
*** matsuhashi has quit IRC12:13
*** matsuhashi has joined #heat12:14
*** Demitar has joined #heat12:15
*** Demitar_ has quit IRC12:17
*** matsuhas_ has joined #heat12:18
*** matsuhashi has quit IRC12:18
*** rbuilta has joined #heat12:23
*** kashyapk has joined #heat12:25
*** sorantis has quit IRC12:25
*** gokrokve has joined #heat12:32
*** kashyapk has quit IRC12:34
*** kashyapk has joined #heat12:35
*** sorantis has joined #heat12:35
shardyafaranha: Here's one way to do it:12:35
shardyhttp://paste.fedoraproject.org/111104/4031813212:35
*** DandyPandy has joined #heat12:36
shardyYou can pass cloud-config data which tells cloud-init to setup the password12:36
*** DandyPandy has quit IRC12:36
*** DandyPandy has joined #heat12:37
*** gokrokve has quit IRC12:37
*** asalkeld has quit IRC12:37
*** kashyapk has quit IRC12:39
*** Demitar_ has joined #heat12:39
*** john-n-seattle has joined #heat12:42
*** Demitar has quit IRC12:42
*** rpothier has joined #heat12:43
*** erecio has joined #heat12:44
*** andreaf has quit IRC12:46
*** mkollaro1 has joined #heat12:48
*** mkollaro has quit IRC12:48
afaranhashardy: Thanks!12:53
*** jmckind has joined #heat12:56
*** jcoufal has quit IRC12:56
*** Demitar has joined #heat12:59
*** Demitar_ has quit IRC13:01
*** mestery has joined #heat13:09
*** mestery has quit IRC13:09
*** mestery has joined #heat13:10
*** Demitar_ has joined #heat13:10
*** Demitar has quit IRC13:12
*** pafuent has joined #heat13:18
*** blomquisg has joined #heat13:19
*** Demitar has joined #heat13:21
*** Demitar_ has quit IRC13:22
*** jcoufal has joined #heat13:23
*** andreaf has joined #heat13:24
*** kashyapk has joined #heat13:25
*** Demitar_ has joined #heat13:28
*** Demitar has quit IRC13:30
*** gokrokve has joined #heat13:33
*** blues-man has joined #heat13:35
*** Demitar has joined #heat13:35
blues-manhello13:35
*** julienvey has quit IRC13:35
*** norman is now known as wpf13:36
*** DandyPandy has quit IRC13:37
*** gokrokve has quit IRC13:37
*** DandyPandy has joined #heat13:38
*** arbylee has quit IRC13:38
*** Demitar_ has quit IRC13:39
*** kashyapk has quit IRC13:40
*** sgordon has joined #heat13:40
*** sgordon has quit IRC13:40
*** sgordon has joined #heat13:40
*** kashyapk has joined #heat13:40
*** m_22 has joined #heat13:44
*** kashyapk has quit IRC13:44
*** jyoti_ranjan has quit IRC13:45
*** jergerber has joined #heat13:45
*** sballe has joined #heat13:47
*** anteaya has quit IRC13:51
*** anteaya has joined #heat13:53
*** zhiyan_ is now known as zhiyan13:54
*** DandyPandy has quit IRC13:57
afaranhashardy: Could you please explain me again the AWS::CloudFormation::WaitCondition? I already have an instance configured with neutron and accessing internet, but still failing in this resource13:58
*** andersonvom has joined #heat13:59
*** TravT has joined #heat13:59
*** julienvey has joined #heat14:00
afaranhathis condition is to wait until the instance creation is complete? If so I think I should just put the time to 2000 (+- 1400s to complete an instance creation here)14:00
shardyafaranha: It waits until you send a signal from the instance14:03
shardyso if the time until you send that signal is 1400s, you're right, you need >1400s or it will fail14:03
*** sorantis has quit IRC14:04
*** arbylee has joined #heat14:04
afaranhaUnderstand, thank you! I was just checking because is quite expensive to just try if it works14:04
*** gokrokve has joined #heat14:04
*** arbylee has quit IRC14:08
*** arbylee has joined #heat14:09
*** e0ne has quit IRC14:10
*** kashyapk has joined #heat14:11
*** bnemec has joined #heat14:14
*** sjmc7 has joined #heat14:14
*** zz_gondoi is now known as gondoi14:15
*** kashyapk has quit IRC14:15
*** e0ne has joined #heat14:17
*** matsuhas_ has quit IRC14:17
jasondhm, gerrit stopped resetting the votes on new patchsets14:18
*** mcwoods has quit IRC14:19
*** Demitar_ has joined #heat14:20
*** Demitar has quit IRC14:22
*** tango has joined #heat14:22
*** matsuhashi has joined #heat14:23
*** tspatzier has joined #heat14:24
*** sabeen has joined #heat14:24
*** jdob has joined #heat14:33
*** rakesh_hs2 has quit IRC14:33
*** matsuhashi has quit IRC14:33
*** matsuhashi has joined #heat14:33
*** rwsu has joined #heat14:34
*** DandyPandy has joined #heat14:34
*** kirankv has quit IRC14:37
*** dims has joined #heat14:37
*** matsuhashi has quit IRC14:38
*** matsuhashi has joined #heat14:40
*** noTHD has quit IRC14:41
*** jergerber has quit IRC14:41
pas-hajasond, there's something more strange than that14:42
pas-hamore like if it was a conflict-less rebase that it does not reset. but that was yesterday...14:43
jasondhopefully it's just a bug14:46
*** noTHD has joined #heat14:55
shardypas-ha: That's expected, there's trivial rebase detection which re-applies previous reviews (but not approval) if a simple rebase has been performed14:56
pas-hayes, but I think I saw some inconsistencies when it is a series of dependent patches and something changes down the line14:57
*** e0ne has quit IRC14:59
*** e0ne has joined #heat15:00
zanebI think the trivial rebase detector script has gone away, and that functionality is now built in to Gerrit15:00
*** david-lyle has joined #heat15:02
jasondapparently, it doesn't always do the right thing https://review.openstack.org/#/c/94957/15:02
*** noTHD has quit IRC15:02
*** david-lyle has quit IRC15:03
zanebjasond: those two patchsets are the same... how could it do anything else?15:04
*** david-lyle has joined #heat15:04
jasondzaneb: i added the correct dependency in patchset 215:04
*** e0ne has quit IRC15:04
*** jprovazn is now known as jprovazn_afk15:05
zanebright, but the heuristic is that if two patch sets are the same, they keep their votes15:05
jasondis that the new behavior or did it always work that way?15:06
zanebthe only alternatives are (a) never keep the votes, even for trivial rebases, or (b) build an artificial intelligence system that can interpret comments15:06
zanebit's new to gerrit, but the trivial rebase script always worked that way15:06
zanebthe only difference is the script used to leave a comment to say what it was doing15:07
sjmc7i vote b)15:07
zanebrofl15:07
*** matsuhashi has quit IRC15:07
*** m_22 has quit IRC15:07
*** matsuhashi has joined #heat15:07
jasondoh ok.  for some reason i thought it only kept the votes if the commit message changed15:08
sjmc7zaneb - the problem with database commits not happening (and thus software config stack-updates still not working) is due to the object not getting added to the sqlalchemy session in some situations15:08
zanebsjmc7: ooooh, interesting15:08
sjmc7i filed https://bugs.launchpad.net/heat/+bug/1331872, but it means this was broken for over a month15:08
uvirtbotLaunchpad bug 1331872 in heat "Changes in update_and_save not always committed" [Undecided,New]15:08
zanebtell me more15:08
sjmc7so i'm surprised nobody found it15:08
zanebhow do you know it was broken for over a month?15:09
sjmc7the update-and-save code was added may 515:09
zanebsjmc7: of 201215:10
sjmc7ah :)15:11
sjmc7i'll blame that on 8pm15:11
sjmc7it would only affect situations where the session wasn't already attached to the object15:11
zaneblol15:11
sjmc7so it may just be that this happens to be triggering it15:11
*** matsuhashi has quit IRC15:11
zanebwe *should* be using one common session for everything in a request15:11
*** kashyapk has joined #heat15:12
zanebit's stored in the context15:12
therveYou don't really want long-lived transations though15:12
sjmc7it's calling get_session rather than using one already attached to the model15:12
*** blamar has joined #heat15:12
zaneboh, I am not passing the context from parser.Stack15:13
zanebmy fault15:13
zanebeasily fixed15:13
* zaneb mutters about optional parameters15:14
zanebgoing afk for a few minutes, but will post a patch when I get back15:14
sjmc7no worries15:15
*** kashyapk has quit IRC15:16
*** Demitar has joined #heat15:18
*** kgriffs|afk is now known as kgriffs15:18
*** Demitar_ has quit IRC15:19
*** jcoufal has quit IRC15:21
*** TravT has quit IRC15:26
*** Demitar has quit IRC15:30
*** denis_makogon has quit IRC15:32
*** dmakogon_ has joined #heat15:33
*** TravT has joined #heat15:35
*** mkollaro1 has quit IRC15:35
*** Demitar has joined #heat15:36
zanebback15:37
*** ramishra has joined #heat15:38
*** harset has joined #heat15:39
harsethi15:39
*** mkollaro has joined #heat15:39
*** ramishra_ has joined #heat15:41
harsethi i'm trying to run a stack with this template http://pastebin.com/61K84AwY, but it fails with an error : TypeError: unhashable type: 'dict'  http://pastebin.com/B7dJhU4M .. any idea?How can I spot the problem?!15:42
sjmc7can you repaste the template, harset?15:42
zanebsjmc7: remove the comma and it works15:43
sjmc78pm!15:43
*** ramishra has quit IRC15:43
harsetsjmc7, why?15:43
zanebharset: your problem is that you have an unholy mix of HOT and cfn syntax15:44
zanebharset: change Ref to get_resource or get_param, as appropriate15:44
zaneband Fn::GetAtt to get_attr15:45
*** saju_m has quit IRC15:45
harsetzaneb, so eg i must change LaunchConfigurationName: {Ref: launch} to LaunchConfigurationName: {get_resource: launch} ?!15:46
*** mkollaro has quit IRC15:46
zanebharset: yes, you should. although looking at the code, it appears we do allow Ref15:47
zanebso it must be another problem15:47
zanebFn::GetAtt is the problem. use get_attr instead15:49
zanebon lines 66 & 8215:49
harsetzaneb, wow this fix it :) thank you really much!15:51
zanebnp15:52
zanebwe should probably do a bit more validation there, to give a better error message15:52
harsetzaneb, yes :)15:53
*** alexpilotti has quit IRC15:55
*** alexpilotti has joined #heat15:55
zanebharset: if you file a bug, I'll fix it right now. deal?15:56
sjmc7zaneb, my other fires are dying out, i'll fix the stack update thing15:56
zanebsjmc7: testing a patch right now, but there are unit test failures15:57
*** e0ne has joined #heat15:57
sjmc7ah, ok, great (not the failures). will test when you're done15:57
zaneb:(15:57
zanebit'a like the unit tests only work without context and the real world only works with it. sigh15:59
harsetzaneb, absolutely not, I am grateful for the work you do. I simply agree with you about the  cryptic error message :|15:59
zanebharset: I'm offering to fix the cryptic error message if you file a bug report about it16:00
sjmc7:)16:00
*** lazy_prince has quit IRC16:00
*** alexpilotti has quit IRC16:01
harsetzaneb, ah ok,now I understood :) .. where is the right place to file a bug report? https://launchpad.net/heat here?!16:01
sjmc7yep, harset16:01
zanebyep16:01
*** e0ne has quit IRC16:01
sjmc7which tests are failing, zaneb?16:02
harsetok I'll do after the work in office :)16:02
zanebsjmc7: basically all of the stack update ones16:02
*** e0ne has joined #heat16:03
zanebsjmc7: http://paste.fedoraproject.org/111175/31937971 <- there is the patch if you want to try it16:03
harsetmmm... seems I have another proble...the alarm is on http://pastebin.com/PkDaQL1M but no scaling occours :| ..where I can look to spot the problem?16:03
sjmc7the last three days have been like the software twilight zone. nothing surprises me any more16:03
*** Qiming has quit IRC16:04
*** david-lyle has quit IRC16:04
*** jcoufal has joined #heat16:06
*** david-lyle has joined #heat16:07
*** packet has joined #heat16:09
*** Demitar_ has joined #heat16:15
zanebharset: check the heat-engine log. most likely you are encountering the same issue that sjmc7 and others were having yesterday16:17
zanebwe think that was due to https://bugs.launchpad.net/tripleo/+bug/133144516:17
uvirtbotLaunchpad bug 1331445 in tripleo "Refusing to bootstrap mysql cluster until role is known. (missing heat metadata) (dup-of: 1331720)" [Critical,Triaged]16:17
uvirtbotLaunchpad bug 1331720 in heat "missing TripleO heat metadata" [High,In progress]16:17
zaneband the offending commit there has been reverted, so you might want to update your code and try again16:17
sjmc7zaneb-  that patch fixes the update issue16:17
*** blamar has quit IRC16:18
*** Demitar has quit IRC16:18
zanebsjmc7: cool! so I just have to make the unit tests work :/16:19
*** bgorski has joined #heat16:20
sjmc7yeah.. i'm puzzled that this change breaks them16:22
*** nati_ueno has joined #heat16:23
*** Fayablazer has quit IRC16:23
zanebsjmc7: I expect that somewhere in the unit tests, things are also being done without a context, and they all ended up in the same session as long as nothing used a context16:23
harsetzaneb, I can't spot any error in the heat-engine.log http://pastebin.com/r2KBFqnA16:25
zanebharset: is there nothing after that? looks like the alarm is not being triggered at all16:30
*** cdent_ has joined #heat16:31
*** julienvey has quit IRC16:32
*** cdent has quit IRC16:32
*** cdent_ is now known as cdent16:32
harsetzaneb, nothing :( ..but the alarm is triggered http://pastebin.com/PkDaQL1M (state==alarm)16:33
harsethere http://pastebin.com/vDtYwdXS the resource list16:34
zanebyeah, so somewhere between Ceilometer and heat-engine it is getting lost16:34
zanebI'd check the ceilometer and heat-api-cfn logs to see how far it is getting16:35
*** kfox1111 has joined #heat16:35
harsetzaneb, heat-api (I haven't an heat-api-cfn log) shows only16:36
harsetINFO urllib3.connectionpool [-] Starting new HTTP connection (1): 10.0.0.9816:36
zanebiirc the URL returned from AWS::AutoScaling::ScalingPolicy will be to heat-api-cfn, so the fact you're not running that is probably the issue16:38
harsetzaneb, here http://pastebin.com/gbfcmtxZ the resource show for CPUAlarmHigh16:39
harsetzaneb, how can I check if I'm running all the necessary services?16:40
zanebok, I'm wrong. port 8004 is heat-api, not heat-api-cfn16:41
zaneboh, sorry, no16:41
*** jcoufal has quit IRC16:41
zanebthat is just the resource link16:41
zanebwe can't see the alarm url16:41
*** jcoufal has joined #heat16:42
zanebharset: are you using devstack?16:42
harsetzaneb, no i'm using packstack on a centos16:42
zanebah, ok16:42
harsetpackstack Icehouse 2014.1.1dev106816:42
*** e0ne has quit IRC16:43
zanebtelnet localhost 800016:43
zanebshould give you a fair idea if heat-api-cfn is running16:43
*** e0ne has joined #heat16:43
zanebalthough the fact that there's no log file for it is pretty conclusive already16:43
harsetzaneb, netstat -natp |grep 8000 show me nothing16:43
shardyharset: did you specify --os-heat-cfn-install=y when running packstack?16:45
harsetzaneb, CONFIG_HEAT_CFN_INSTALL=n .. no :(16:45
zanebit seems pretty clear that you haven't got heat-api-cfn installed/running16:45
zanebharset: ok, so edit your packstack answer file and re-run packstack16:45
shardyharset: You can re-run packstack after setting that in the answerfile16:45
shardyThen you'll need to make a minor fix to the heat.conf:16:46
shardyhttps://bugzilla.redhat.com/show_bug.cgi?id=110639416:46
uvirtbotshardy: Error: Could not parse XML returned by bugzilla.redhat.com: HTTP Error 404: Not Found16:46
harsetcan I use both cfn and not cfn in heat?16:46
zanebharset: yes16:46
harsetzaneb, thank you very much for the help!16:46
zanebboth APIs are independent and they talk to the same engine16:46
harsetI must read more carefull this http://docs.openstack.org/developer/heat/template_guide/hot_guide.html :|16:47
shardyactually bug #1318599 has a better description of the problem16:47
uvirtbotLaunchpad bug 1318599 in heat "heat engine doesn't create new instance when ceilometer alarm is in "alarm" status using autoscaling feature "OS::Heat::AutoScalingGroup"" [High,In progress] https://launchpad.net/bugs/131859916:47
*** e0ne has quit IRC16:48
shardyYou need to remove the "ec2tokens" from the ec2authtoken auth_uri in heat.conf, as packstack currently misconfigures it16:48
zanebthanks shardy, I would have completely forgotten about that bug16:48
shardyrelatedly, anyone fancy reviewing this fix?16:48
shardyhttps://review.openstack.org/#/c/98827/16:48
shardywe probably need the puppet stuff fixed too, but that makes heat a bit more tolerant16:49
*** gokrokve has quit IRC16:49
*** jmckind has quit IRC16:50
zanebshardy: approved16:51
shardythanks16:51
harsetshardy, thanks for the hint!16:52
*** zhiyan is now known as zhiyan_16:54
*** nati_ueno has quit IRC16:59
*** nati_ueno has joined #heat17:02
*** david-lyle has quit IRC17:03
*** david-lyle has joined #heat17:05
ramishra_Hello zaneb17:05
*** harlowja_away is now known as harlowja17:05
ramishra_zaneb: I am still seeing the same issue https://bugs.launchpad.net/heat/+bug/1328342 after your related patch. Am I missing something17:06
uvirtbotLaunchpad bug 1328342 in heat "stack-update with SoftwareDeployment 400 response from signal" [High,Fix committed]17:06
sjmc7ramishra_ - zane's working on a patch now17:07
zanebramishra_: yes, we've established the fix isn't working, see bug #133187217:07
uvirtbotLaunchpad bug 1331872 in heat "Changes in update_and_save not always committed" [High,Confirmed] https://launchpad.net/bugs/133187217:07
sjmc7the patch here fixes it if you want to try - http://paste.fedoraproject.org/111175/3193797117:07
ramishra_oh.. ok17:08
*** BillArnold has joined #heat17:10
*** blamar has joined #heat17:15
ramishra_sjmc7: yep.. it works.. thanks..17:16
*** gokrokve has joined #heat17:18
zanebnothing more frustrating than a patch that works for users, but the tests say it's broken. and vice-versa.17:20
sjmc7zaneb- which tests aren't working for you?17:20
*** blues-man has quit IRC17:23
*** proffalken has quit IRC17:24
*** e0ne has joined #heat17:29
*** kfox1111 has quit IRC17:33
*** kfox1111 has joined #heat17:33
*** ramishra_ has quit IRC17:34
harsetbye17:35
*** harset has quit IRC17:35
*** tspatzier has quit IRC17:37
*** jcoufal has quit IRC17:38
*** tspatzier has joined #heat17:45
*** saju_m has joined #heat17:46
*** tspatzier has quit IRC17:46
*** lipinski has joined #heat17:48
lipinskiQuestion about stack-update17:48
lipinskiI have a stack that was created with a environment and template file(s).17:48
lipinskiIf I perform a stack-update, do I have to pass the original environment file?  I'm trying to pass a slightly different one that is missing a particular, optional parameter.  Excluding this parameter is causing heat to detect a dependency change and rebuild VMs.17:49
*** sab has joined #heat17:50
*** proffalken has joined #heat17:51
*** randallburt has joined #heat17:52
zanebsjmc7: heat.tests.test_parser.StackTest.test_update_rollback heat.tests.test_parser.StackTest.test_update_by_reference_and_rollback_1 heat.tests.test_parser.StackTest.test_update_by_reference_and_rollback_217:52
sjmc7abc and smelly17:52
zanebsjmc7: so the problem is with rollback, evidently17:52
zanebyeah17:52
sjmc7yeah, i just got the same. it takes forever to run the tests on my system for some reason17:52
*** e0ne has quit IRC17:52
*** e0ne has joined #heat17:53
zaneblipinski: you don't have to pass the same env file, but if calculated parameter values change then that can cause replacement on update, depending on where they are used17:53
*** sab has quit IRC17:54
*** ericgoncz has joined #heat17:54
*** randallburt has quit IRC17:54
zanebsjmc7: btw the test_hot ones are just duplicates of the test_parser ones17:54
*** randallburt has joined #heat17:54
*** blamar has quit IRC17:57
*** e0ne has quit IRC17:57
sjmc7the stack will still be in the session, right?17:57
*** rbuilta has quit IRC17:57
zanebshould be, yeah17:58
sjmc7i hate ORMs sometimes :)17:59
*** BillArnold has quit IRC18:01
*** lskyw has joined #heat18:02
*** alexpilotti has joined #heat18:03
*** lskyw has quit IRC18:03
*** lskyw has joined #heat18:04
*** andreaf_ has joined #heat18:05
*** david-lyle has quit IRC18:08
*** dims has quit IRC18:08
*** andersonvom has quit IRC18:08
*** sballe has quit IRC18:08
*** john-n-seattle has quit IRC18:08
*** ccrouch1 has quit IRC18:08
*** zpatten has quit IRC18:08
*** gokrokve has quit IRC18:08
*** arbylee has quit IRC18:08
*** andreaf has quit IRC18:08
*** PhilK_ has quit IRC18:08
*** pas-ha has quit IRC18:08
*** mkerrin has quit IRC18:08
*** wpf has quit IRC18:08
*** jasond has quit IRC18:08
*** cody-somerville has quit IRC18:08
*** skraynev has quit IRC18:08
*** fbo_away has quit IRC18:08
*** trash has quit IRC18:08
*** lindsayk has quit IRC18:08
*** tnurlygayanov has quit IRC18:08
*** wendar has quit IRC18:08
*** harlowja has quit IRC18:08
*** sileht has quit IRC18:08
*** adam_g has quit IRC18:08
*** kgriffs has quit IRC18:08
*** gondoi has quit IRC18:08
*** stevebaker has quit IRC18:08
*** lvh has quit IRC18:08
*** FL1SK has quit IRC18:08
*** Adri2000 has quit IRC18:08
*** zigo has quit IRC18:08
*** radix has quit IRC18:08
*** zhiyan_ has quit IRC18:08
*** chmouel has quit IRC18:08
*** shufflebot has quit IRC18:08
*** uvirtbot has quit IRC18:08
*** abramley has quit IRC18:08
*** cdent has quit IRC18:08
*** gokrokve has joined #heat18:10
*** david-lyle has joined #heat18:10
*** dims has joined #heat18:10
*** arbylee has joined #heat18:10
*** andersonvom has joined #heat18:10
*** sballe has joined #heat18:10
*** john-n-seattle has joined #heat18:10
*** PhilK_ has joined #heat18:10
*** pas-ha has joined #heat18:10
*** mkerrin has joined #heat18:10
*** ccrouch1 has joined #heat18:10
*** wpf has joined #heat18:10
*** wendar has joined #heat18:10
*** zpatten has joined #heat18:10
*** harlowja has joined #heat18:10
*** jasond has joined #heat18:10
*** cody-somerville has joined #heat18:10
*** tnurlygayanov has joined #heat18:10
*** skraynev has joined #heat18:10
*** sileht has joined #heat18:10
*** fbo_away has joined #heat18:10
*** adam_g has joined #heat18:10
*** trash has joined #heat18:10
*** kgriffs has joined #heat18:10
*** lindsayk has joined #heat18:10
*** uvirtbot has joined #heat18:10
*** chmouel has joined #heat18:10
*** shufflebot has joined #heat18:10
*** radix has joined #heat18:10
*** zigo has joined #heat18:10
*** zhiyan_ has joined #heat18:10
*** FL1SK has joined #heat18:10
*** lvh has joined #heat18:10
*** Adri2000 has joined #heat18:10
*** abramley has joined #heat18:10
*** stevebaker has joined #heat18:10
*** gondoi has joined #heat18:10
*** kashyapk has joined #heat18:11
*** ericgoncz has quit IRC18:12
*** david-lyle has quit IRC18:14
*** dims has quit IRC18:14
*** andersonvom has quit IRC18:14
*** sballe has quit IRC18:14
*** john-n-seattle has quit IRC18:14
*** ccrouch1 has quit IRC18:14
*** zpatten has quit IRC18:14
*** gokrokve has quit IRC18:14
*** arbylee has quit IRC18:14
*** PhilK_ has quit IRC18:14
*** pas-ha has quit IRC18:14
*** mkerrin has quit IRC18:14
*** wpf has quit IRC18:14
*** jasond has quit IRC18:14
*** cody-somerville has quit IRC18:14
*** skraynev has quit IRC18:14
*** fbo_away has quit IRC18:14
*** trash has quit IRC18:14
*** lindsayk has quit IRC18:14
*** tnurlygayanov has quit IRC18:14
*** wendar has quit IRC18:14
*** harlowja has quit IRC18:14
*** sileht has quit IRC18:14
*** adam_g has quit IRC18:14
*** kgriffs has quit IRC18:14
*** gondoi has quit IRC18:14
*** stevebaker has quit IRC18:14
*** lvh has quit IRC18:14
*** FL1SK has quit IRC18:14
*** Adri2000 has quit IRC18:14
*** zigo has quit IRC18:14
*** radix has quit IRC18:14
*** zhiyan_ has quit IRC18:14
*** chmouel has quit IRC18:14
*** shufflebot has quit IRC18:14
*** uvirtbot has quit IRC18:14
*** abramley has quit IRC18:14
*** ericgoncz has joined #heat18:15
*** gokrokve has joined #heat18:16
*** david-lyle has joined #heat18:16
*** dims has joined #heat18:16
*** arbylee has joined #heat18:16
*** andersonvom has joined #heat18:16
*** sballe has joined #heat18:16
*** john-n-seattle has joined #heat18:16
*** PhilK_ has joined #heat18:16
*** pas-ha has joined #heat18:16
*** mkerrin has joined #heat18:16
*** ccrouch1 has joined #heat18:16
*** wpf has joined #heat18:16
*** wendar has joined #heat18:16
*** zpatten has joined #heat18:16
*** harlowja has joined #heat18:16
*** jasond has joined #heat18:16
*** cody-somerville has joined #heat18:16
*** tnurlygayanov has joined #heat18:16
*** skraynev has joined #heat18:16
*** sileht has joined #heat18:16
*** fbo_away has joined #heat18:16
*** adam_g has joined #heat18:16
*** trash has joined #heat18:16
*** kgriffs has joined #heat18:16
*** lindsayk has joined #heat18:16
*** uvirtbot has joined #heat18:16
*** chmouel has joined #heat18:16
*** shufflebot has joined #heat18:16
*** radix has joined #heat18:16
*** zigo has joined #heat18:16
*** zhiyan_ has joined #heat18:16
*** FL1SK has joined #heat18:16
*** lvh has joined #heat18:16
*** Adri2000 has joined #heat18:16
*** abramley has joined #heat18:16
*** stevebaker has joined #heat18:16
*** gondoi has joined #heat18:16
*** bmahalakshmi has joined #heat18:18
*** ericgoncz has quit IRC18:21
*** BillArnold has joined #heat18:28
*** kgriffs is now known as kgriffs|afk18:30
lipinskizaneb: sorry, stepped away for a bit.  Is there a way to prevent replacement?18:31
lipinskiWhat is happening is this.18:31
zaneblipinski: yes, don't change anything ;)18:31
lipinskiI have a parameter that is passed down through provider templates to a softwareconfig resource.  Essentially I am allowing a way to have a heat parameter end up as a file on a VM18:31
lipinskiI don't want to have to pass that parameter with the same contents each time.  So, I'd like to do a stack-update, but have the non-existence of the parameter not trigger a VM replacement.18:32
lipinskiIt seems to be thinking the paramter changed, so the SoftwareConfig changed, so it rebuilds the VM18:32
zanebsjmc7: W00t! fixed it18:32
*** dims has quit IRC18:33
lipinskiIt's actually not even rebuilding the VM.  But rather choking on some volume attachment thing - which I'm not sure how that even comes into play.18:33
sjmc7ooo! i've been poking at it off and on for the last hour18:33
*** zpatten has quit IRC18:33
sjmc7what caused it?18:33
*** sballe has quit IRC18:33
*** dims has joined #heat18:33
*** zpatten has joined #heat18:33
lipinskiIf there was some way to break the resource dependency on the parameter...18:33
*** andersonvom has quit IRC18:33
zaneblipinski: read the whiteboard on https://blueprints.launchpad.net/heat/+spec/troubleshooting-low-level-control ... I think that's what you want?18:33
zanebsjmc7: the backup stack was using the same raw_template in the DB as the main stack. it wasn't a problem before as raw_templates were immutable. but now both stacks are making opposing changes to the template, so rollback was not working correctly18:35
*** kashyapk has quit IRC18:35
sjmc7ah, that makes sense. thanks for spending the time on this18:36
*** kashyapk has joined #heat18:37
lipinskizaneb: thanks - will check that out18:38
zaneblipinski: to be clear, that's not implemented yet, but tango is looking at it now18:39
*** kgriffs|afk is now known as kgriffs18:43
tangolipinski: If you have some thought on what would help, please feel free to put your comment on the whiteboard and I would be glad to add the support18:50
zanebsjmc7: https://review.openstack.org/#/c/101288/ <- the fix18:51
sjmc7thanks, will check it18:52
*** david-lyle has quit IRC18:54
zanebI guess we lost our notification bot in the last freenode split-brain18:54
sjmc7zaneb - the behavior now is (in the database) that the original raw_template is updated correctly, and an additional raw_template row is created (i'm guessing the backup)18:59
sjmc7is that what you'd expect?18:59
zanebyep, that's correct19:00
sjmc7splendid19:00
zanebso the original one should change, and the backup one should hopefully not19:00
sjmc7yep, correct19:00
sjmc7+1ed, many thanks19:00
zanebwe should probably stop soft-deleting the backup stacks19:01
zanebthey really ought to be hard-deleted when we're done with them19:01
sjmc7yeah19:01
zanebI'll open a bug for that19:02
SpamapSzaneb: https://review.openstack.org/#/c/101288/ needs tests19:03
SpamapSzaneb: but very awesome that we tracked this down19:03
*** samuelmz_ has joined #heat19:03
*** sabeen has quit IRC19:05
*** sabeen has joined #heat19:05
*** sabeen has quit IRC19:06
*** sabeen has joined #heat19:06
samuelmz_Hi, when using a OS::Heat::InstanceGroup resource type in a Heat HOT template, we need to define a property called 'LaunchConfigurationName' of type String19:06
samuelmz_afaik, it should point to another resource into the template, for instance: LaunchConfigurationName : compute-node, where compute node is another resource19:07
samuelmz_what type of resource it should point to? I tried with OS::Nova::Server and it didnt work :/19:08
zanebsamuelmz: AWS::AutoScaling::LaunchConfiguration19:08
zanebhence the name ;)19:09
SpamapSthats not confusing _at all_. :-/19:09
samuelmz_is that not possible using an OS::?? resource?19:10
*** nati_uen_ has joined #heat19:10
zanebsamuelmz_: no19:11
samuelmz_I'd like to use a pure hot template19:11
samuelmz_I'm following the http://docs.openstack.org/developer/heat/template_guide/hot_spec.html19:11
zanebsamuelmz_: and btw, you should use get_resource, not just use the name (although both will work atm)19:11
*** kashyapk has quit IRC19:11
zanebsamuelmz_: I recommend using OS::Heat::AutoScalingGroup instead of InstanceGroup19:11
zanebthat will let you define the launch configuration inline, without using any AWS resources19:12
zanebalthough, for the record, use of AWS resources and the template format (HOT vs. CloudFormation) are completely orthogonal19:13
samuelmz_zaneb: ok then; but it looks weird to me to have OS::Heat::InstanceGroup pointing to something (LaunchConfiguration) specific of AWS .. and not having a pure OS::?? alternative19:13
zanebwell, the alternative is for OS::Heat::InstanceGroup to just not exist19:14
*** nati_ueno has quit IRC19:14
zanebit's a building block for our AWS compatibility resources that we exposed to users19:14
zanebthe way forward for the future is OS::Heat::AutoScalingGroup19:14
samuelmz_zaneb: dont you plan to have a resource like OS::Heat::LaunchConfiguration?19:14
zanebno, because we have OS::Heat::AutoScalingGroup19:15
samuelmz_zaneb: hmm, great... thats what I was looking for19:15
*** saju_m has quit IRC19:20
samuelmz_zaneb: I'm looking at OS::Heat::AutoScalingGroup .. it will fit better what I want .. thanks for your help19:21
SpamapSat one point we had wanted to have OS:: equivalents for everything19:24
*** noTHD has joined #heat19:25
*** julienvey has joined #heat19:27
zanebSpamapS: we do, but InstanceGroup is going to get deprecated19:28
*** e0ne has joined #heat19:28
zanebif OS::Heat::AutoScalingGroup required a launch config, we would want a native one19:29
zanebbut we decided it wouldn't use one at all19:29
kfox1111anyone familior with mock awake?19:29
SpamapSzaneb: right I like the inline config better. :)19:32
zanebI don't, but I didn't win the argument ;)19:32
*** bmahalakshmi has quit IRC19:35
*** e0ne has quit IRC19:35
*** e0ne has joined #heat19:36
SpamapSzaneb: well we need more generic composition methods19:37
SpamapSzaneb: only having group configs be reusable is not ideal.19:38
zanebI do agree with that19:38
*** e0ne has quit IRC19:38
zanebI am disappointed that the AWS compat resources will now be hacks forever though19:39
zanebbut I have moved on with my life ;)19:39
SpamapSI think the more we diverge the more compelling a CFN -> HOT translator will become.19:40
SpamapSwe'll get tired of maintaining parallel resource plugins19:40
zanebyou might be right19:41
*** e0ne has joined #heat19:41
SpamapSand now to fix this tempest test for os-collect-config19:41
*** m_22 has joined #heat19:48
*** e0ne has quit IRC19:49
*** e0ne has joined #heat19:49
*** akuznetsov has quit IRC19:49
*** sabeen has quit IRC19:51
*** sabeen has joined #heat19:51
*** e0ne has quit IRC19:52
*** akuznetsov has joined #heat19:55
*** e0ne has joined #heat19:57
*** akuznetsov has quit IRC19:59
*** e0ne has quit IRC20:02
*** david-lyle has joined #heat20:10
*** samuelmz_ has quit IRC20:20
*** nati_uen_ has quit IRC20:25
*** nati_ueno has joined #heat20:25
*** nati_ueno has quit IRC20:27
*** nati_ueno has joined #heat20:28
*** blamar has joined #heat20:28
*** akuznetsov has joined #heat20:34
*** alexpilotti has quit IRC20:35
shardyzaneb, SpamapS: when we solve the nesting limit issue, most CFN resources should become provider templates IMO20:36
*** m_22 has quit IRC20:36
zanebmost probably should20:36
zanebI think there are a bunch where that is not technically possible though20:37
zanebI know asalkeld had to jump through a lot of hoops to do that for alarms, for example20:37
zanebincluding creating multiple new intrinsic functions20:37
shardyYeah I guess there may be a few, but many of them would be simple wrappers20:39
zanebagree20:40
*** mestery has quit IRC20:42
*** mestery has joined #heat20:43
*** sorantis has joined #heat20:46
*** sabeen1 has joined #heat20:46
*** erecio has quit IRC20:47
*** m_22 has joined #heat20:47
*** mestery has quit IRC20:47
SpamapSshardy: What if we solved the "1:1 provider resource mappings shouldn't be a whole nested stack" issue instead?20:47
SpamapSor just solve the nesting problem by not giving each nested stack its own stack object... just namespace the root stack's resources/outputs/parameters.20:48
zanebhave we established that there is actually a "problem"?20:49
shardySpamapS: Yeah, I'm not sure what the optimal solution is, but we can't provider-resource all-the-things until a default nesting limit is either removed or increased a lot20:49
zanebother than the arbitrary limit being too low20:49
*** sabeen has quit IRC20:49
shardyzaneb: well if you use one nested stack containing a provider resource now, you're at the limit20:49
SpamapSzaneb: It's too low20:50
SpamapSI went conservative.20:50
SpamapSAnd I stand by that default20:50
zanebright, we can all agree that it's too low :)20:50
SpamapSuntil we can count all the resources in a nested chain reliably... it has to be low.20:50
shardySpamapS: IIRC you mentioned a recursive resource count instead, which I think would be much better20:50
zanebok, so the problem is that we can't count reliably20:50
SpamapSwhich is where I wonder.. why do we even have "nested" stack objects.. if we just had one stack, but the nested bits just got a namesace in the root stack.. the counting would be easier and the memory/database footprint would also be lower.20:51
*** akuznetsov has quit IRC20:51
zaneband that's because some templates could be fetched at runtime?20:51
SpamapSWe could just graft them on.20:51
*** jomara has quit IRC20:51
zanebbecause i know an easy solution to that ;)20:51
SpamapSThe counting problem, anyway, may just have been me being new, and not having enough clarity on the issue.20:52
*** packet has quit IRC20:52
zanebSpamapS: btw bgorski's new OS::Heat::Stack resource is always going to use python-heatclient to create nested stacks20:52
zanebso that also solves this problem, I think20:53
zanebwe'll only have to worry about layers of nesting for providers20:53
shardyzaneb: does it?  Or does it just circumvent the limit?20:53
zaneband let quotas on the API handle the rest20:53
*** pafuent has left #heat20:53
SpamapShm20:54
SpamapSzaneb: I like that.. sort of going the opposite direction and making the explicit nested stack _more_ separate.20:54
SpamapSso then we can loosen the providers up a bit20:54
shardyzaneb: there's a quota patch series up btw, although I stopped reviewing it because my comments were not being addressed20:54
SpamapScan users define their own providers?20:54
zanebshardy: it doesn't allow you to do anything the user couldn't already do themselves by just creating lots of stacks20:54
shardySpamapS: yes!20:55
zanebSpamapS: a provider is just a template, so yes20:55
zanebthat's the whole point ;)20:55
SpamapSOk so we still have to care20:55
shardyzaneb: Yeah, that's what I meant, so does the nested limit actually achieve anything20:55
SpamapSThat doesn't really solve the issue then.20:55
zanebdeeply nested providers are considerably more rare in the wild though20:55
SpamapSSo there are a few easy solutions.20:56
SpamapS1 is maintain a resource count per tenant.20:57
SpamapSWe can always determine that from the database + any template being parsed.20:57
SpamapSActually thats what I think may be necessary.20:57
*** asalkeld has joined #heat20:58
SpamapSBecause as we move intelligence away from parsing the whole stack.. that is the most scalable option.20:58
shardySpamapS: yeah per tenant quota for #stacks and #resources is what we need20:58
*** jdob has quit IRC20:58
SpamapSif you have #resources, that is implicitly #stacks20:58
shardybut the author of the quota patches wants per-user, which gets really hard with updates from multiple users20:58
shardySpamapS: stacks can have zero resources20:58
SpamapSshardy: yeah, just use the same limit I mean.20:59
SpamapSshardy: as a count, you do actually need it separate. curses.  I get what you meant now.20:59
SpamapSshardy: by user is fine too.20:59
SpamapSjust charge to stack owner21:00
shardySpamapS: So that ties in to the owner wrt trust impersonation - if user "A" creates a stack, then user "B" updates it, who should we impersonate when doing deferred operations?21:00
SpamapSshardy: oh so they're saying if userA updates stack "foo", owned by userB, to add resources, then userA gets a separate limit?21:00
shardyCurrently it's always "A", even if they no longer exist21:01
shardySpamapS: yes, that's what I think is being proposed21:01
SpamapSAh so we need a chown21:01
shardyI said lets get per-project quotas in and working first ;)21:01
shardyheat stack-update --owner21:01
SpamapSshardy: so we need to record user ID per resource instead of per stack. I think that's simple enough.21:01
shardyor something21:01
shardySpamapS: but then you split quota for a stack over multiple users?21:02
SpamapSbut not a trivial amount of work and testing for those quotas. :-P21:02
shardySeems really messy to me21:02
SpamapSOne might also argue that we should NOT do the quotas .. just the impersonation.. and let physical resource services do the quoats.21:02
shardyI'd rather you have one owner, and if you do a stack update, you use quota for the whole stack21:02
SpamapSquoats: Oats that quiver.21:02
shardythat is one owner per stack, which can change via an update21:03
SpamapSshardy: What if we just punt on our own fine grained quotas?21:03
SpamapSshardy: why do we need a quota when the resources will already take up a much more expensive resource than our "rows in the DB and RAM in the negine" .. VMs.. volumes.. etc.21:03
SpamapSmy typnig si amzaign tdoya21:04
shardySpamapS: there's a whole series of patches up where you can ask those questions :D21:04
SpamapSshardy: heat-specs?21:04
SpamapSor just "I'm going to run off and do this and not ask anybody if it is a good idea." patches?21:04
shardyhttps://review.openstack.org/#/q/status:open+project:openstack/heat+branch:master+topic:bp/add-quota-api-for-heat,n,z21:05
shardyIIRC it was discussed pre-heat-specs21:05
shardyTo be fair the author has been regularly asking for review feedback in here21:06
SpamapSspeaking of specs... convergence needs approval soon so I can start work on it.21:06
SpamapSshardy: mmk21:06
* shardy needs to revisit the convergence specs21:06
SpamapSI poked at some observer work but haven't wanted to go too far in case we found some gotchyas.21:07
stevebakergood morning!21:08
stevebakershardy: can we have a quick chat about the keystone client plugin?21:15
kfox1111good morning.21:15
*** PhilK_ is now known as Philk21:15
*** Philk is now known as PhilK21:15
*** blomquisg is now known as blomquisg_away21:16
lipinskiIs there a way to specify the replacement policy for a provider template?21:17
shardyhey stevebaker sure, I'm just about still awake ;)21:17
*** sgordon has quit IRC21:17
lipinskiFor example, I have some metadata associated with a Server.  This metadata can be modified without replacing the server/VM resource.21:18
lipinskiHowever, if I "encapsulate" that server in a provider template, now if I change the provider resource's metadata, it causes the entire provider to be rebuilt - including the server21:18
SpamapSshardy: thanks for the pointer. I went ahead and sent an email to the list about it, as I think it deserves a wide discussion.21:18
stevebakershardy: I'd like to move all the methods in KeystoneClientV3 into the keystone client plugin class https://review.openstack.org/#/c/97985/10/heat/engine/clients/os/keystone.py so that the keystone client is the *actual* keystone client. Apart from the merge pain are you OK with that?21:19
SpamapSlipinski: it should not be rebuilding the whole provider21:19
SpamapSlipinski: it should be doing a stack update on it.21:19
shardystevebaker: Yup sounds good, the main reason for heat_keystoneclient exisitng was that we didn't have what you're currently working on21:20
stevebakershardy: cool, thought as much21:20
lipinskiSpamapS: not sure.  I have a provider resource that simply is a Server.  I set some metadata for the provider resource.  If I change that metadata, it causes the underlying Server to be rebuilt.21:21
randallburtlipinski:  I'm about to look at a similar bug reported by one of our users about ResourceGroup (it uses a nested stack behind the scenes like provider resources)21:21
lipinskiLet me see if I can simplify my scenario to a very basic setup.21:21
randallburtlipinski:  points to something in StackResource looks like.21:21
stevebakerzaneb: similarly, most of the nova_utils methods can move to the nova client plugin. I'd like to delete nova_utils but would you prefer deprecation?21:22
*** david-lyle has quit IRC21:23
*** dims has quit IRC21:23
*** morganfainberg is now known as morganfainberg_Z21:24
*** dims has joined #heat21:24
lipinskirandallburt: how are you using ResourceGroups?  I have found them very limited since you cannot specify anything about each instance - e.g., must use same or random VM name, for example21:25
randallburtlipinski:  well, there's a patch in-flight that lets you use the resource index to customize the properties of the nested resource. To be fair, if you need a bunch of resources of the same types and different configuration, ResourceGroup isn't the thing to use.21:26
lipinskirandallburt: The thing I'm interested in is creating many resources of the same type, flavor, etc.  But, the ability to specify a few different pieces of data for each instance.21:29
lipinskiThe simpliest example I can think of is setting the name of each21:29
randallburtlipinski:  yeah, the in-flight patch kinda lets you do that.21:30
lipinskiIt allows for many more complicated examples when coupled with provider templates and parameters.21:30
lipinskiCan you point me to something with some info on that?21:30
randallburtsomething like name: "server-%index%"21:30
randallburtlipinski:  https://review.openstack.org/#/c/88636/21:30
lipinskiThanks.  I'll look at that.  I was hoping for something like this that I opened: https://blueprints.launchpad.net/heat/+spec/per-instance-properties-on-resourcegroups21:32
*** jprovazn_afk has quit IRC21:32
lipinskiI actually had code to do it that was fairly simple.  It just didn't handle when a resource parameter was a list.21:32
lipinskiI was thinking along the lines of say a resource group where I want the resources evenly distributed across two availability zones.  I pass a list of avail zones as a parameter to the resource group, and it alternates for each resoruce, passing a avail zone.21:33
randallburtlipinski:  cool. maybe convert that to a spec and maybe get more input.21:33
*** andersonvom has joined #heat21:34
*** openstackgerrit has joined #heat21:34
lipinskiI'm not a Heat developer.  Not to mention, my company beauracracy prevents me from becoming one and agreeing to the agreement because of intel-property, blah, blah.  (I'm not happy about it - would definitely want to contribute)21:35
lipinskiand my spelling stinks - you wouldn't want me ;)21:35
lipinskiHow do I convert a blueprint to a spec?21:36
*** rpothier has quit IRC21:43
SpamapSlipinski: did you, by any chance, reference the metadata in the underlying server's properties? Some properties cause replacement.21:44
*** m_22 has quit IRC21:45
randallburtlipinski:  you submit a review to the heat-specs repository21:45
*** DandyPandy has quit IRC21:45
randallburtSpamapS:  that could be it, but in my case, the user saw the same behavior when increasing the number of resources in a ResourceGroup. That re-generates the template, but should only result in a resource being added21:45
lipinskiSpamapS: I just retested this scenario21:46
randallburtlipinski:  so if you can't sign the contributor agreement, I don't think you can submit a patch. zaneb, is that a hard rule? Do we need to require the contributor agreement for specs?21:46
SpamapSlipinski: the thing that you want is very interesting. I believe Mike Spreitzer from IBM has been working on building up use cases for a holistic scheduler for OpenStack that fits that bill nicely.21:46
*** kfox1111 has quit IRC21:47
lipinskiI have a simple provider template that has a server.  I create a resoruce using that provider template, and assign that resource a piece of metadata.21:47
lipinskiI then change that metadata and the server within the provider resoruce gets recreated.21:47
lipinskiThe metadata is not referenced anywhere (I'm not sure how to reference metadata within a template...)21:47
randallburtlipinski:  metadata as in the key value pairs associated with a server or the metadata associated with the Resource?21:48
lipinskiSpamapS: The proposal gets really cool when you apply it to providers too.  Tons of flexibility.  I can set data lists that (pre-)specify things that will be used for future VMs, e.g., when the number of servers in a resource group changes,e tc.21:48
lipinskimetadata associated with the provider resource.21:48
lipinskiit's a simple string.  I set metadata: my_data: abc (where I define the provider resource)21:49
lipinskiI change that to xyz, and the Server within the provider gets rebuilt21:49
*** david-lyle has joined #heat21:50
SpamapSlipinski: rebuilt, or replaced? rebuild is a specific nova command that only is used to change images without detaching network ports or volumes.21:50
lipinskisorry - replaced.21:50
lipinskiI see a new instance appear, then the old disappears.  On the dashboard, for a very short time, I have two VMs.21:50
SpamapSlipinski: thanks. So that sounds really weird. Check event-list .. I suspect the whole provider is being replaced.21:50
SpamapSI wonder if nested stacks don't allow metadata updates21:52
lipinskiSpamapS: odd.  The event-list shows the provider resource goes to UPDATE_IN_PROGRESS then CREATE_IN_PROGRESS then CREATE_COMPLETE21:53
lipinskiI don't see a delete, per se.21:53
SpamapSweird indeed!21:53
lipinskithis is all a function of the heat-engine, right?21:54
SpamapSlipinski: yes21:54
zanebrandallburt, lipinski: I don't think anyone would actually care for specs, but for technical reasons it's not possible without signing the CLA afaik. so it's effectively a hard rule :(21:54
lipinskiI'm wondering if I find a environment with the official Icehouse.  I think my engine was pulled from github before the official release.21:54
randallburt:(21:55
*** morganfainberg_Z is now known as morganfainberg21:55
*** dims has quit IRC21:58
zanebstevebaker: I'd like to delete nova_utils... but I suspect we'd be safer with a deprecation period21:59
*** arbylee has quit IRC21:59
SpamapShmmmmmm21:59
stevebakerzaneb: the way it is strucutured, the deprecated nova_utils will be dead code. I suppose the unit tests can't be deleted either22:00
openstackgerritKevin Fox proposed a change to openstack/heat: Add a OS::Nova::ServerGroup resource.  https://review.openstack.org/10099522:00
SpamapSlipinski: any chance, on your engine, that you have "Resource xxxx does not implement metadata update" as a warning?22:00
zanebstevebaker: I'm ok with some dead code, as long as we still have tests22:00
*** achampion has joined #heat22:00
SpamapSlipinski: it kind of looks like only servers allow updating metadata.22:01
zanebSpamapS: as of last month, a metadata change should never trigger update replace22:01
stevebakerthe original nova_utils just had the user data building method. That should really move to a cloud_init_utils22:01
shardySpamapS: and WaitCOnditionHandle - I've been working on patches today to convert that to handle_signal22:01
SpamapSzaneb: right I kind of remember that.. but I can't find where that is implemented22:01
shardyIMO we should kill metadata_update and align everything with handle_signal, as they basically do the same thing22:02
shardyactually I'll post what I have as I'm on PTO tomorrow22:02
SpamapSlipinski: ok AFAICT, just changing the metadata should not affect anything22:04
zanebSpamapS: well, we used to implement a thing that triggered replacement if anything changed that was not in a list allowed by the resource. But now we don't. So you're looking for where something is not implemented. I'd say you already found it.22:04
SpamapSlipinski: it should just update the resource.22:04
openstackgerritSteven Hardy proposed a change to openstack/heat: Convert WaitConditionHandle to use handle_signal  https://review.openstack.org/10135122:04
openstackgerritSteven Hardy proposed a change to openstack/heat: Update waitcondition API to use signal RPC interface  https://review.openstack.org/10135222:04
openstackgerritSteven Hardy proposed a change to openstack/heat: Refactor waitcondition resources to allow easier subclassing  https://review.openstack.org/10135322:04
openstackgerritSteven Hardy proposed a change to openstack/heat: Add an OS::Heat::WaitCondition resource  https://review.openstack.org/10135422:04
SpamapSzaneb: thats what I was afraid of :)22:04
SpamapSlooking for a needle NOT in a haystack22:04
*** gondoi is now known as zz_gondoi22:05
lipinskiSpamapS: you mean it could be because I'm not running the latest Icehouse heat engine?22:05
zaneblooking for a non-needle in a haystack22:05
shardynative handle resource patch still in-progress but that hopefully somewhat illustrates why we should kill metadata_update22:05
SpamapSlipinski: s/Icehouse/juno.22:06
zaneblipinski: you'll need later than Icehouse even22:06
*** dims has joined #heat22:06
SpamapSlipinski: Icehouse is _SO_ last season.22:06
lipinskiahhh - darn22:06
SpamapScontinuous deployment FTW :)22:06
lipinskiWell, our customers have placed a requirement on use that things work with Icehouse.22:06
lipinskion us22:06
*** nati_ueno has quit IRC22:07
*** nati_ueno has joined #heat22:07
zaneblipinski, randallburt: re distributing autoscaled instance across availability zones... that specific thing is an outright missing feature of autoscaling IMO. it doesn't *necessarily* follow that we should generalise that to all kinds of properties22:08
SpamapSlipinski: this is where not being a dev really hurts you. The only way you can do that is to backport things into the stable branch one bug at a time. :_/22:08
randallburtzaneb:  not necessarily, no.22:09
lipinskizaneb: That was just an example.  The idea was more about passing data through to instances contained within a resource group22:09
randallburtbut there are other use cases where the ability to customize some aspect of the property makes sense too.22:09
lipinskiI could pre-name all my VMs that I expect to ever be created within a group, as another example.22:09
zaneblipinski: right, what I'm saying is that's not just an example. it's a specific special case that ought to be its own feature22:10
lipinskizaneb: oh, okay.  I thought you were saying there was already something that implemented that for avail zones.22:10
zanebyeah, there's not but there should be22:10
*** nati_ueno has quit IRC22:11
lipinskiMy ideal would be having a ResourceGroup of provider resources.  I could then have something auto-scaling my providers, I could pre-set data for "future" resources at the time I create the stack, etc.22:11
zanebwe can talk about the generic customisation thing too. I know randallburt's a big fan ;)22:11
zanebbut it is a different issue imo22:11
randallburtzaneb:  indeed :)22:12
*** nati_ueno has joined #heat22:12
openstackgerritSteve Baker proposed a change to openstack/python-heatclient: Do not set up logging handler in http  https://review.openstack.org/10135822:13
openstackgerritSteve Baker proposed a change to openstack/python-heatclient: Improve --debug logging output  https://review.openstack.org/10135922:13
*** sabeen1 has quit IRC22:14
shardySuper trivial review if someone has a moment:22:14
shardyhttps://review.openstack.org/#/c/99583/22:14
zanebshardy: done22:15
shardymtreinish, stevebaker: I think https://review.openstack.org/#/c/90143/ is ready now, reviews would be awesome as all the check jobs are now working22:15
lipinskiis there a way to reference metadata within a resource?  maybe soemthing like a get_metadata intrinsic funcion?22:16
zaneblipinski: fwiw OS::Heat::AutoscalingGroup scales provider resources22:16
lipinskiTrying to find a way to break my metadata off my provider resource and maybe into some pseudo resource that acts as a data-object store22:16
*** sabeen has joined #heat22:16
zanebno, there isn't22:17
shardypass the data as parameters then reference them22:17
lipinskizaneb: Thanks, but it seems to have the same shortcomings that prevent me from using ResourceGroups.22:17
zanebonly within a provider template, where you can get the facade resource's metadata22:17
stevebakershardy: should the test be tagged slow if it uses cirros?22:18
shardystevebaker: well it takes a couple of minutes even on bare metal22:18
shardyOr maybe like 90seconds22:18
shardyI wasn't really sure what the definition of "slow" was tbh22:19
stevebakerwe have plenty of time in heat-slow, lets just leave it there22:19
stevebakershardy: my definition of slow is "boots fedora"22:19
shardystevebaker: Ok, it also touches lots of stuff, hence my thinking it's more of a kitchen-sink test appropriate for -slow22:20
stevebakerfair enough22:20
shardyhappy to change the tag if needed though22:20
*** arbylee has joined #heat22:20
*** alexpilotti has joined #heat22:20
zaneblipinski: have you raised a bug for this metadata issue?22:21
lipinskizaneb: no.  Based on your discussion, I wasn't sure if it was one.22:22
zaneblipinski: I think it is, and I'd be prepared to fix it in Icehouse for you22:22
lipinskiYou mean the fact that changing metadata for a Provider resource causes it to be recreated (or whatever it is doing...)22:22
zanebcan't speak for reviewers ;)22:22
zanebyes, exactly22:22
lipinskiok - I'll open one.22:22
*** noTHD has quit IRC22:23
zanebthanks, ping me with the number22:23
*** ccrouch1 has quit IRC22:25
*** andreaf_ has quit IRC22:26
*** julienvey has quit IRC22:26
*** noTHD has joined #heat22:27
*** andersonvom has quit IRC22:27
lipinskizaneb: so this is different frmo https://bugs.launchpad.net/bugs/126256222:31
uvirtbotLaunchpad bug 1262562 in heat "AWS::AutoScaling::LaunchConfiguration metadata causes replacement on update" [Low,Confirmed]22:31
*** arbylee has quit IRC22:31
*** blamar has quit IRC22:34
*** kgriffs is now known as kgriffs|afk22:34
*** openstackgerrit has quit IRC22:34
*** 20WAAHXAJ has joined #heat22:35
*** ccrouch has joined #heat22:39
lipinskizaneb: https://bugs.launchpad.net/heat/+bug/133235422:39
uvirtbotLaunchpad bug 1332354 in heat "provider metadata updates cause recreation" [Undecided,New]22:39
lipinskiLet me know if you need more info.  I was a bit terse so that I could get it created quickly.22:39
*** andersonvom has joined #heat22:40
20WAAHXAJKevin Fox proposed a change to openstack/heat: Add a OS::Nova::ServerGroup resource.  https://review.openstack.org/10099522:48
*** openstack has joined #heat22:51
sjmc7what's a reasonable time for the heat unit tests to run? 'tox -epy27' takes a very long time for me, which makes me think I have a configuration problem22:54
sjmc7by long time i mean 20 minutes22:54
SpamapSsjmc7: on my 4CPU laptop, 90s22:55
sjmc7trade?22:55
SpamapSsjmc7: note that the first time you run it, you have to download "the world" from pypi22:55
sjmc7yeah, this is every time22:55
SpamapSsjmc7: can you paste your "slowest tests" output?22:55
SpamapSshould print as the last thing before success22:56
sjmc7will do. this run failed from testing a patch earlier, will rerun off master22:56
*** alexpilotti has quit IRC23:00
20WAAHXAJA change was merged to openstack/heat: parser.Stack add use_stored_context option  https://review.openstack.org/9973023:06
*** Demitar_ has quit IRC23:08
20WAAHXAJA change was merged to openstack/heat: Convert service.py to use_stored_context  https://review.openstack.org/9973123:08
20WAAHXAJA change was merged to openstack/heat: Remove test_autoscaling _stub_validate  https://review.openstack.org/10036523:08
20WAAHXAJA change was merged to openstack/heat: test_autoscaling refactor suspend/resume stubbing  https://review.openstack.org/10036623:09
*** wpf is now known as wpf-norman23:11
*** lipinski has quit IRC23:11
sjmc7SpamapS - this one was only 878 seconds23:14
sjmc7http://pastebin.com/X8DHwFd223:14
SpamapSsjmc7: ah see, already getting faster23:15
*** fbo_away has quit IRC23:15
sjmc7great success23:15
SpamapSsjmc7: yeah those are uncannily high23:15
SpamapSsjmc7: I'm on a dual-core i7 (so 4x because of HT, and python HT's pretty well) ..23:16
SpamapSsjmc7: but I'd expect on a single core w/o HT that it would just take 360s .. not 87823:16
sjmc7ah... i'm running in a VM23:16
sjmc7so if it's reliant on concurrency that might not be too surprising23:17
sjmc7the autoscaling ones are the slow ones, so that makes perfect sense23:17
SpamapSsjmc7: oh and the VM is going to suck for CPU23:17
*** randallburt has quit IRC23:18
SpamapSsjmc7: there's always tox -epypy ;)23:18
sjmc7:)23:18
*** sorantis has quit IRC23:20
*** sabeen has quit IRC23:26
*** 20WAAHXAJ has quit IRC23:29
*** openstackgerrit has joined #heat23:30
*** julienvey has joined #heat23:37
zanebugh, is there really no way to report a bug against a released version in Launchpad?23:38
*** julienvey has quit IRC23:41
zanebok, found it23:41
zanebSpamapS: 90s? I have quad core + HT and it's >2 minutes23:44
zanebsjmc7: but yeah, autoscaling tests are really slow. you want at least 2 cores ideally23:44
sjmc7ok. i'll give the vm some more grunt23:46
openstackgerritKevin Fox proposed a change to openstack/heat: Add a OS::Nova::ServerGroup resource.  https://review.openstack.org/10099523:46
*** mestery has joined #heat23:46
*** daneyon has joined #heat23:52
*** daneyon has quit IRC23:52
*** daneyon has joined #heat23:53
*** nati_ueno has quit IRC23:54
*** nati_ueno has joined #heat23:55
*** nati_ueno has quit IRC23:55
*** david-lyle has quit IRC23:56
*** nati_ueno has joined #heat23:56
*** dims has quit IRC23:58

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