*** DandyPandy has quit IRC | 00:01 | |
*** ericgoncz has quit IRC | 00:13 | |
openstackgerrit | Kevin Fox proposed a change to openstack/heat: Add a OS::Nova::ServerGroup resource. https://review.openstack.org/100995 | 00:16 |
---|---|---|
*** andersonvom has joined #heat | 00:20 | |
*** matsuhashi has joined #heat | 00:20 | |
*** tango has quit IRC | 00:21 | |
*** Qiming has joined #heat | 00:25 | |
*** randallburt has joined #heat | 00:26 | |
*** nati_ueno has quit IRC | 00:26 | |
*** randallburt has quit IRC | 00:26 | |
*** randallburt has joined #heat | 00:26 | |
kfox1111 | hi Randall | 00:27 |
randallburt | hey | 00:27 |
*** andersonvom has quit IRC | 00:28 | |
*** ramishra has joined #heat | 00:28 | |
kfox1111 | thanks for the review. | 00:28 |
randallburt | np | 00:28 |
kfox1111 | I think I have everything done now. | 00:28 |
kfox1111 | Way more work then I would have hoped. :/ | 00:29 |
kfox1111 | A good learning experience though. | 00:29 |
randallburt | it always is. (work, that is ;) | 00:29 |
randallburt | can you shoot me a link? I lost the email. | 00:29 |
kfox1111 | https://review.openstack.org/#/c/100995/ | 00:30 |
randallburt | looking | 00:30 |
*** zaneb has quit IRC | 00:31 | |
*** ramishra has quit IRC | 00:32 | |
sjmc7 | anyone still here? | 00:42 |
sjmc7 | :) | 00:42 |
sjmc7 | stevebaker, you got a sec? | 00:42 |
*** nosnos has joined #heat | 00:43 | |
randallburt | kfox1111: a couple more minor nits, but LGTM. sorry about the older tests as examples. | 00:44 |
*** zaneb has joined #heat | 00:44 | |
sjmc7 | zaneb - sorry to bug you again | 00:45 |
randallburt | its like you summoned him sjmc7 ;) | 00:45 |
sjmc7 | but 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 properly | 00:45 |
sjmc7 | randallburt :) i fear i have outstayed my welcome today | 00:46 |
*** dims_ has quit IRC | 00:46 | |
randallburt | :D | 00:46 |
sjmc7 | i'll file a bug for this | 00:46 |
randallburt | sjmc7: k. | 00:46 |
sjmc7 | and once more swear at ORMs in general | 00:47 |
*** dims_ has joined #heat | 00:48 | |
*** kfox1111 has quit IRC | 00:48 | |
*** DandyPandy has joined #heat | 00:50 | |
*** DandyPandy has quit IRC | 00:51 | |
openstackgerrit | Randall Burt proposed a change to openstack/heat: Update to latest version of pyrax and add Swift support https://review.openstack.org/94658 | 00:55 |
openstackgerrit | Randall Burt proposed a change to openstack/heat: Account for differences in Rackspace Cloud Glance https://review.openstack.org/96337 | 00:55 |
*** sjmc7 has quit IRC | 00:57 | |
*** TravT has quit IRC | 01:01 | |
*** dims_ has quit IRC | 01:05 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements https://review.openstack.org/96507 | 01:07 |
*** m_22 has joined #heat | 01:08 | |
*** spzala has quit IRC | 01:10 | |
*** nanjj`` has joined #heat | 01:11 | |
*** samstav has joined #heat | 01:13 | |
*** nanjj`` has quit IRC | 01:18 | |
*** tiantian has joined #heat | 01:19 | |
tiantian | morning all :) | 01:19 |
*** RockKuo_Office has joined #heat | 01:21 | |
*** Qiming has quit IRC | 01:23 | |
*** m_22 has quit IRC | 01:23 | |
*** gokrokve has quit IRC | 01:25 | |
*** gokrokve has joined #heat | 01:25 | |
liusheng | morning | 01:28 |
*** ramishra has joined #heat | 01:29 | |
*** mestery has joined #heat | 01:29 | |
*** gokrokve has quit IRC | 01:30 | |
*** ramishra has quit IRC | 01:34 | |
*** mestery has quit IRC | 01:34 | |
*** bgorski has quit IRC | 01:39 | |
SpamapS | stevebaker: FYI: https://review.openstack.org/101073 | 01:46 |
*** ericgoncz has joined #heat | 01:46 | |
SpamapS | stevebaker: testing using os-collect-config locally.. we'll see how it goes. :) | 01:46 |
* SpamapS calls it a day | 01:47 | |
*** mestery has joined #heat | 01:54 | |
*** DandyPandy has joined #heat | 01:57 | |
*** gokrokve has joined #heat | 01:59 | |
*** Tross has quit IRC | 02:00 | |
*** zhangyang has quit IRC | 02:03 | |
*** tiantian has quit IRC | 02:04 | |
*** liusheng has quit IRC | 02:04 | |
*** liusheng has joined #heat | 02:04 | |
*** zhangyang has joined #heat | 02:04 | |
*** tiantian has joined #heat | 02:04 | |
elynn | morning guys :) | 02:07 |
*** matsuhashi has quit IRC | 02:07 | |
*** matsuhashi has joined #heat | 02:08 | |
*** matsuhashi has joined #heat | 02:08 | |
*** arbylee has joined #heat | 02:12 | |
*** mestery_ has joined #heat | 02:20 | |
*** mestery_ has quit IRC | 02:21 | |
zhangyang | stevebaker:this patch needs your review, https://review.openstack.org/#/c/100830/ | 02:21 |
*** mestery_ has joined #heat | 02:22 | |
*** mestery has quit IRC | 02:23 | |
stevebaker | zhangyang: does that change stop all the deprecation warnings in a test run? | 02:24 |
stevebaker | zhangyang: I suspect the barbican resources landed at the same time, which also do the old way | 02:25 |
*** DandyPandy has quit IRC | 02:26 | |
*** ramishra has joined #heat | 02:29 | |
*** bnemec has quit IRC | 02:30 | |
*** samstav has quit IRC | 02:31 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Fix count for stack-list while show deleted https://review.openstack.org/101078 | 02:31 |
*** achampion has quit IRC | 02:32 | |
tiantian | stevebaker: would you please to review the patch: https://review.openstack.org/#/c/100804/ thanks | 02:33 |
*** ramishra has quit IRC | 02:35 | |
*** ericgoncz_ has joined #heat | 02:36 | |
*** ericgoncz has quit IRC | 02:37 | |
*** ericgoncz_ is now known as ericgoncz | 02:37 | |
*** ramishra has joined #heat | 02: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 |
stevebaker | zhangyang: if you've fixed all the deprecation warnings then ignore me | 02:44 |
zhangyang | alright, tks. | 02:45 |
*** ericgoncz has quit IRC | 02:45 | |
*** mestery_ is now known as mestery | 02:48 | |
*** ramishra has quit IRC | 02:51 | |
*** rpothier has joined #heat | 02:53 | |
*** ramishra_ has joined #heat | 02:56 | |
*** achampion has joined #heat | 02:58 | |
elynn | hi, anyone can review these two patches? https://review.openstack.org/#/c/100191/ https://review.openstack.org/#/c/100178/ | 02:59 |
*** ramishra has joined #heat | 03:00 | |
*** gokrokve has quit IRC | 03:00 | |
*** ramishra_ has quit IRC | 03:01 | |
*** achampion has quit IRC | 03:03 | |
*** samstav has joined #heat | 03:13 | |
*** cmyster has joined #heat | 03:27 | |
*** rwsu has quit IRC | 03:27 | |
cmyster | morning | 03:28 |
cmyster | stevebaker: evening, here ? | 03:28 |
stevebaker | cmyster: afternoon | 03:29 |
cmyster | hey, 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 #heat | 03:33 | |
*** ramishra has quit IRC | 03:33 | |
*** zhiyan_ is now known as zhiyan | 03:34 | |
*** gokrokve_ has joined #heat | 03:36 | |
stevebaker | cmyster: looks good | 03:36 |
* SpamapS realizes he has not run devstack in over a year. :-P | 03:37 | |
*** blamar has joined #heat | 03:38 | |
*** blamar has quit IRC | 03:38 | |
*** matsuhashi has quit IRC | 03:38 | |
cmyster | thanks stevebaker, | 03:39 |
*** matsuhashi has joined #heat | 03:39 | |
cmyster | SpamapS: its easy, stack.sh, wait for it to break, fix it locally, unstack, stack, repeat ;) | 03:39 |
*** gokrokve has quit IRC | 03:39 | |
cmyster | J/K its working rather well the last few attempts I had with it over the last few weeks | 03:40 |
SpamapS | cmyster: I refuse to subject my laptop to stack.sh | 03:41 |
SpamapS | cmyster: I'll run it in hpcloud instances. :) | 03:41 |
* cmyster does it on bare metal | 03:41 | |
*** akuznetsov has joined #heat | 03:43 | |
*** matsuhashi has quit IRC | 03:45 | |
*** matsuhashi has joined #heat | 03:45 | |
*** matsuhashi has quit IRC | 03:46 | |
*** jyoti_ranjan has joined #heat | 03:47 | |
*** nosnos has quit IRC | 04:02 | |
*** mestery has quit IRC | 04:06 | |
*** mestery has joined #heat | 04:07 | |
*** sab has joined #heat | 04:09 | |
*** mestery has quit IRC | 04:11 | |
*** zhangyang has quit IRC | 04:24 | |
*** zhangyang has joined #heat | 04:24 | |
*** randallburt has quit IRC | 04:26 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Fix count for stack-list while show deleted https://review.openstack.org/101078 | 04:30 |
*** samstav has quit IRC | 04:35 | |
*** matsuhashi has joined #heat | 04:37 | |
*** kopparam has joined #heat | 04:42 | |
*** kirankv has joined #heat | 04:42 | |
*** nosnos has joined #heat | 04:43 | |
*** akuznetsov has quit IRC | 04:53 | |
*** nkhare has joined #heat | 04:56 | |
*** matsuhashi has quit IRC | 04:56 | |
*** matsuhashi has joined #heat | 04:57 | |
*** akuznetsov has joined #heat | 04:58 | |
*** achampion has joined #heat | 05:05 | |
*** kopparam has quit IRC | 05:11 | |
*** kopparam has joined #heat | 05:11 | |
*** achampion has quit IRC | 05:12 | |
*** alexpilotti has joined #heat | 05:12 | |
*** nkhare has quit IRC | 05:21 | |
*** nati_ueno has joined #heat | 05:23 | |
skraynev | good morning | 05:27 |
elynn | Hi 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 |
elynn | Need your feedback :) | 05:29 |
skraynev | elynn: eyah, I planned to do it after review "client-as-plugins" patches. IMO, looks good, but I wanted to test it else ;) | 05:31 |
elynn | thx skraynev :) | 05:31 |
elynn | review codes is a hard work. | 05:33 |
*** tiantian has quit IRC | 05:34 | |
*** rakesh_hs2 has joined #heat | 05:35 | |
*** bmahalakshmi has joined #heat | 05:37 | |
*** liusheng has quit IRC | 05:43 | |
*** akuznetsov has quit IRC | 05:43 | |
*** liusheng has joined #heat | 05:43 | |
*** Demitar has joined #heat | 05:50 | |
*** harlowja is now known as harlowja_away | 05:54 | |
*** nkhare has joined #heat | 05:55 | |
*** Demitar has quit IRC | 05:55 | |
*** morganfainberg has quit IRC | 05:57 | |
*** morganfainberg has joined #heat | 05:58 | |
*** lazy_prince has joined #heat | 06:00 | |
*** kopparam has quit IRC | 06:01 | |
*** alexpilotti has quit IRC | 06:01 | |
*** alexpilotti has joined #heat | 06:01 | |
*** kopparam has joined #heat | 06:01 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/89750 | 06:02 |
*** akuznetsov has joined #heat | 06:09 | |
skraynev | elynn: +1. and very important :) | 06:12 |
*** rakesh_hs2 has quit IRC | 06:15 | |
*** rakesh_hs2 has joined #heat | 06:16 | |
elynn | thx skraynev :) | 06:22 |
*** tiantian has joined #heat | 06:22 | |
*** andreaf has joined #heat | 06:22 | |
*** jprovazn has joined #heat | 06:32 | |
*** kopparam has quit IRC | 06:42 | |
*** e0ne has joined #heat | 06:45 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat: Fix error count for stack-list while show deleted https://review.openstack.org/101078 | 06:59 |
*** elynn has quit IRC | 07:02 | |
*** andreaf has quit IRC | 07:03 | |
*** achampion has joined #heat | 07:03 | |
*** gokrokve_ has quit IRC | 07:05 | |
*** PhilK has quit IRC | 07:05 | |
*** e0ne has quit IRC | 07:06 | |
*** PhilK_ has joined #heat | 07:07 | |
*** shardy_afk is now known as shardy | 07:08 | |
shardy | morning all | 07:08 |
*** jcoufal has joined #heat | 07:10 | |
openstackgerrit | Zhang Yang proposed a change to openstack/heat: Hide deprecate warnings for metadata tests https://review.openstack.org/100830 | 07:11 |
therve | Good morning! | 07:11 |
cmyster | morning | 07:11 |
*** Michalik- has joined #heat | 07:12 | |
*** gokrokve has joined #heat | 07:15 | |
*** ajc_ has joined #heat | 07:17 | |
*** mcwoods_ has quit IRC | 07:17 | |
*** zhangyang has quit IRC | 07:19 | |
*** zhangyang has joined #heat | 07:19 | |
*** gokrokve has quit IRC | 07:19 | |
*** kragniz_ has joined #heat | 07:24 | |
*** baffle_ has joined #heat | 07:24 | |
*** kragniz has quit IRC | 07:27 | |
*** baffle has quit IRC | 07:27 | |
*** dteselkin has quit IRC | 07:27 | |
*** dteselkin has joined #heat | 07:28 | |
*** achampion has quit IRC | 07:30 | |
*** gokrokve has joined #heat | 07:35 | |
*** mcwoods has joined #heat | 07:36 | |
pas-ha | morning all | 07:40 |
*** gokrokve has quit IRC | 07:40 | |
openstackgerrit | huangtianhua proposed a change to openstack/python-heatclient: Show physical_resource_id in resource-list https://review.openstack.org/101125 | 07:44 |
*** kopparam has joined #heat | 07:52 | |
cmyster | shardy: mind checking https://review.openstack.org/#/c/89790/ ? jenkins is in a rare small working window | 07:59 |
*** alexpilotti has quit IRC | 08:00 | |
*** kopparam has quit IRC | 08:01 | |
*** nati_ueno has quit IRC | 08:01 | |
*** kopparam has joined #heat | 08:01 | |
*** chandan_kumar has quit IRC | 08:01 | |
*** alexpilotti has joined #heat | 08:02 | |
*** e0ne has joined #heat | 08:05 | |
openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Adding validation algorithm for get attr functions https://review.openstack.org/82488 | 08:06 |
*** e0ne has quit IRC | 08:13 | |
*** jyoti_ranjan has quit IRC | 08:15 | |
*** e0ne has joined #heat | 08:23 | |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::NodeGroupTemplate resource https://review.openstack.org/100288 | 08:27 |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement sahara client plugin https://review.openstack.org/100588 | 08:27 |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::Cluster resource https://review.openstack.org/72336 | 08:27 |
*** gokrokve has joined #heat | 08:27 | |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::NodeGroupTemplate resource https://review.openstack.org/100288 | 08:29 |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement sahara client plugin https://review.openstack.org/100588 | 08:29 |
openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::Cluster resource https://review.openstack.org/72336 | 08:29 |
*** gokrokve has quit IRC | 08:32 | |
*** elynn has joined #heat | 08:33 | |
sab | I am getting an error "AttributeError: 'Module_six_moves_urllib_parse' object has no attribute 'SplitResult'", can anyone help me? | 08:35 |
*** andreaf has joined #heat | 08:37 | |
elynn | sab, maybe you can paste your log here :) | 08:37 |
elynn | and how you get this error. | 08:38 |
*** jyoti_ranjan has joined #heat | 08:39 | |
sab | elynn, http://paste.openstack.org/show/84466/ | 08:39 |
*** maishsk has joined #heat | 08:40 | |
sab | this is while just starting up using devstack, | 08:40 |
* maishsk says good morning ! | 08:40 | |
maishsk | anyone here able to help with a question about docker and heat? | 08:40 |
*** Qiming has joined #heat | 08:41 | |
maishsk | keep on getting this message | 08:42 |
maishsk | ERROR: Unknown resource Type : DockerInc::Docker::Container | 08:43 |
elynn | sab, how do you get this error? | 08:43 |
elynn | create a stack? | 08:43 |
openstackgerrit | huangtianhua proposed a change to openstack/heat-specs: Events pagination and sorting Specification https://review.openstack.org/101133 | 08:43 |
skraynev | maishsk: looks like your have not such resource | 08:43 |
elynn | sab, what version of six library do you use? | 08:44 |
skraynev | maishsk: AFAIK, all resources in contrib directory are not avaliable by default | 08:44 |
*** nosnos has quit IRC | 08:44 | |
shardy | cmyster: done, lgtm! | 08:44 |
maishsk | skraynev: I dont need to have docker installed on the controller - if I remember correctly | 08:44 |
maishsk | skraynev: I followed instructions from here | 08:45 |
maishsk | https://github.com/openstack/heat/tree/master/contrib/docker/docker | 08:45 |
maishsk | skraynev: my heat.conf has this plugin_dirs=/root/heat/contrib/docker/docker | 08:46 |
*** saju_m has joined #heat | 08:46 | |
maishsk | the github is not correct | 08:46 |
cmyster | woot | 08:46 |
*** nosnos has joined #heat | 08:46 | |
cmyster | thanks shardy, | 08:46 |
skraynev | maishsk: no, it's correct | 08:47 |
maishsk | there is no plugin folder | 08:47 |
cmyster | now I need someone from tempest to +1 it as well ? | 08:47 |
skraynev | maishsk: you may check that this plugin was loaded. look in heat-engine.log for this information. | 08:48 |
sab | elynn, its 1.5.2 | 08:48 |
shardy | cmyster: You need two tempest-cores to review it, I've just added them all as reviewers | 08:49 |
sab | elynn, heat-eng doesn't even start :) | 08:49 |
elynn | sab, I think you hit this bug, https://bugs.launchpad.net/devstack/+bug/1316328 | 08:50 |
uvirtbot | Launchpad bug 1316328 in devstack "Stack trace when stack.sh hits "cinder-manage db sync"" [Undecided,Fix released] | 08:50 |
maishsk | skraynev: I dont understand - are the instructions on Github correct? | 08:50 |
maishsk | skraynev: there is no plugin folder | 08:51 |
maishsk | skraynev: or should it be set that way - regardless? | 08:51 |
skraynev | maishsk: instructions on Github is correct. It's right way | 08:51 |
elynn | sab, you can try to upgrade six to 1.6.0 and see if this error still exist | 08:51 |
shardy | maishsk: is /root accessible to the stack user? | 08:52 |
sab | elynn, this is after I did a git pull yesterday or today morning. Yes, I will try updating | 08:52 |
shardy | I would assume not, so shouldn't that be /opt/stack/heat/contrib... | 08:52 |
maishsk | skraynev: Even though there is plugin folder? | 08:52 |
maishsk | skraynev: I am not using devstack | 08:53 |
skraynev | maishsk: Hm. I think, that it should work for you. | 08:54 |
*** rakesh_hs2 has quit IRC | 08:54 | |
skraynev | maishsk: my suggestion is to check that plugin was registered. | 08:54 |
*** julienvey has joined #heat | 08:54 | |
maishsk | I dont think it was | 08:54 |
maishsk | skraynev: the folder is not there - so the instructions will not work | 08:55 |
maishsk | [root@icontrol ~(keystone_veeas)]# cd heat/contrib/docker-plugin/plugin | 08:55 |
maishsk | -su: cd: heat/contrib/docker-plugin/plugin: No such file or directory | 08:55 |
maishsk | [root@icontrol ~(keystone_veeas)]# cd heat/contrib/docker/docker/ | 08:56 |
openstackgerrit | huangtianhua proposed a change to openstack/heat-specs: Events pagination and sorting Specification https://review.openstack.org/101133 | 08:56 |
maishsk | skraynev: that works | 08:56 |
skraynev | it's normal | 08:57 |
skraynev | have you any log files? In this case log for heat-engine process. | 08:57 |
*** rakesh_hs2 has joined #heat | 08:58 | |
maishsk | i do | 08:58 |
openstackgerrit | huangtianhua proposed a change to openstack/heat-specs: Events pagination and sorting Specification https://review.openstack.org/101133 | 08:59 |
tiantian | shardy: would you review https://review.openstack.org/#/c/101078/ ths | 08:59 |
maishsk | skraynev: see http://paste.openstack.org/show/84467/ - just restarted the service | 09:00 |
maishsk | skraynev: I have removed the IP's from the logs... | 09:01 |
maishsk | skraynev: I don't see anything there regarding docker whil restarting heat-engine | 09:02 |
*** piyush has joined #heat | 09:02 | |
skraynev | maishsk: ok, let me look.. | 09:03 |
sab | elynn, great, thanks . its works :) | 09:04 |
elynn | sab, np :) | 09:04 |
maishsk | skraynev: Thanks! | 09:05 |
skraynev | maishsk: hm . really looks, like docker plugin was not registered. | 09:05 |
maishsk | skraynev: which takes me back to the question - how can I point in the heat.conf file to a folder that does not exist | 09:06 |
maishsk | skraynev: which is why I think that the README is not correct | 09:07 |
*** Demitar has joined #heat | 09:07 | |
openstackgerrit | huangtianhua proposed a change to openstack/heat-specs: Events pagination and sorting Specification https://review.openstack.org/101133 | 09:09 |
skraynev | maishsk: Oh. Now I understand you fully | 09:09 |
skraynev | maishsk: so it's bug in documentation. | 09:09 |
maishsk | skraynev: OK | 09:10 |
maishsk | :) | 09:10 |
skraynev | maishsk: possibly it was missed, when it was renamed :) | 09:10 |
maishsk | skranyev: so it should be heat/contrib/docker/docker/ | 09:10 |
maishsk | ? | 09:10 |
skraynev | maishsk: I think so | 09:11 |
maishsk | OK | 09:11 |
maishsk | should I will put it a code change for the documentation? | 09:11 |
*** e0ne_ has joined #heat | 09:11 | |
skraynev | maishsk: Try to do it and if it works correct for you, please upload a fix | 09:11 |
maishsk | skraynev: so that is the problem - it does not | 09:12 |
maishsk | skraynev: as you can see from the log... | 09:12 |
tiantian | shardy: 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-spec | 09:12 |
skraynev | maishsk: I mean test, that docker/docker works for you :) | 09:13 |
*** e0ne has quit IRC | 09:13 | |
maishsk | skraynev: It does not | 09:13 |
*** openstackgerrit has quit IRC | 09:14 | |
*** elynn has quit IRC | 09:14 | |
maishsk | skraynev: requirements are met - see http://paste.openstack.org/show/84469/ | 09:14 |
*** piyush has quit IRC | 09:15 | |
*** Fayablazer has joined #heat | 09:17 | |
*** Demitar_ has joined #heat | 09:17 | |
skraynev | maishsk: do you think that problem is not only in directory name? | 09:19 |
*** Demitar has quit IRC | 09:19 | |
maishsk | skraynev: I dont know - I cannot get it to work... | 09:20 |
skraynev | maishsk: ok, as I understood you right you have docker plugin for this path | 09:20 |
skraynev | /root/heat/contrib/docker/docker | 09:21 |
skraynev | right? | 09:21 |
maishsk | skraynev: correct | 09:21 |
skraynev | and same path is provided in config file? | 09:21 |
skraynev | plugin_dirs=/root/heat/contrib/docker/docker | 09:22 |
maishsk | skraynev: in heat.conf ---> plugin_dirs=/root/heat/contrib/docker/docker | 09:22 |
skraynev | maishsk: and I hope, that it's not single line in heat.conf ? | 09:24 |
maishsk | skraynev: no it is not the only line in heat.conf | 09:24 |
skraynev | fooh.. good news :) | 09:24 |
cmyster | thanks shardy, I noted the names, I'll remember to add them to other patches as well | 09:25 |
skraynev | next. have heat access to this file ? | 09:25 |
shardy | cmyster: You can just add tempest-core in the add reviewer box | 09:25 |
shardy | same for all projects | 09:25 |
shardy | I'd suggest not doing it unless a patch is ready and has been posted for some time though | 09:26 |
shardy | as it'll spam everyone with notifications via email | 09:26 |
cmyster | kk | 09:27 |
skraynev | maishsk: in my case I have such log http://paste.openstack.org/show/84470/ , after adding plugin_dirs=../contrib/docker/docker in heat.conf | 09:27 |
skraynev | maishsk: line 17 means, that it was registered | 09:28 |
maishsk | skraynev: looking to see if permissions are correct. | 09:28 |
maishsk | one sec | 09:28 |
*** gokrokve has joined #heat | 09:28 | |
*** Demitar_ has quit IRC | 09:28 | |
skraynev | maishsk: It's my last guess about your problem :( | 09:29 |
shardy | maishsk: is the heat-engine actually running as root though? | 09:30 |
maishsk | no it's not | 09:30 |
maishsk | skraynev: heat 14399 1.5 0.0 291340 72340 ? Ss 08:58 0:30 /usr/bin/python /usr/bin/heat-engine | 09:31 |
shardy | then it's not going to be able to access a plugin in /root then is it? | 09:31 |
maishsk | shardy: Nope.... ;) | 09:31 |
*** gokrokve has quit IRC | 09:33 | |
skraynev | maishsk: I do not use root, because my repo is placed in not root directory:) | 09:33 |
*** Demitar has joined #heat | 09:35 | |
*** cmyster has quit IRC | 09:35 | |
*** e0ne_ has quit IRC | 09:38 | |
skraynev | shardy: about readme for contrib resources. How do you think: should we also fix rewriting whole heat.conf file ? | 09:38 |
*** e0ne has joined #heat | 09:39 | |
skraynev | shardy: https://review.openstack.org/#/c/101144/2/contrib/docker/docker/README.md | 09:39 |
shardy | skraynev: lol, yes ;) | 09:40 |
skraynev | shardy: IMO, using ">" for writing plugin_dirs in heat.conf looks wrong | 09:40 |
*** nosnos has quit IRC | 09:40 | |
skraynev | shardy: I copy-pasted these commands and got awesome result with short heat.conf :) | 09:41 |
*** jprovazn has quit IRC | 09:42 | |
*** e0ne has quit IRC | 09:43 | |
*** matsuhashi has quit IRC | 09:44 | |
*** matsuhashi has joined #heat | 09:44 | |
*** matsuhashi has quit IRC | 09:46 | |
*** matsuhashi has joined #heat | 09:47 | |
*** akuznetsov has quit IRC | 09:52 | |
maishsk | :) | 09:54 |
*** Qiming has quit IRC | 09:58 | |
maishsk | skraynev: added that to the change. | 10:00 |
maishsk | skraynev: ok - some progress - but now have a new error. | 10:03 |
maishsk | skraynev: http://paste.openstack.org/show/84475/ | 10:04 |
*** cdent has joined #heat | 10:07 | |
skraynev | maishsk: wait a minute, I will look | 10:09 |
skraynev | maishsk: 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 #heat | 10:14 | |
maishsk | skraynev: that could be - how can I verify? | 10:14 |
skraynev | maishsk: make git log in both repositories that you use. and compare date of last commit | 10:16 |
skraynev | maishsk: cd /root/heat; git log | 10:16 |
skraynev | and cd <directory with heat, that installed in you system>; git log | 10:17 |
tiantian | bye all:) | 10:17 |
skraynev | in case with devstack it's /opt/stack/heat | 10:17 |
skraynev | tiantian: bye | 10:17 |
maishsk | skraynev: the installation is from a package | 10:18 |
maishsk | Metadata-Version: 1.1 | 10:18 |
maishsk | Name: heat | 10:18 |
maishsk | Version: 2014.1 | 10:18 |
maishsk | Summary: OpenStack Orchestration | 10:18 |
maishsk | Home-page: http://www.openstack.org/ | 10:18 |
maishsk | Author: OpenStack | 10:18 |
maishsk | Author-email: openstack-dev@lists.openstack.org | 10:18 |
maishsk | License: UNKNOWN | 10:18 |
maishsk | Description: ==== | 10:18 |
skraynev | maishsk: yes, this version has not new attribute schema | 10:20 |
maishsk | skraynev: joy... :( | 10:20 |
skraynev | maishsk: so you have two ways: use new package or downgrade /root/heat repo | 10:21 |
skraynev | I prefer second | 10:21 |
maishsk | will try | 10:21 |
skraynev | cd /root/heat; git checkout -t aa6fa5e975606576c759ed14d1e4d67186e2a377 | 10:22 |
*** Demitar_ has joined #heat | 10:25 | |
*** Demitar has quit IRC | 10:28 | |
*** julienvey has quit IRC | 10:28 | |
*** gokrokve has joined #heat | 10:29 | |
*** gokrokve_ has joined #heat | 10:31 | |
*** Demitar has joined #heat | 10:32 | |
*** gokrokve has quit IRC | 10:33 | |
*** Demitar_ has quit IRC | 10:34 | |
*** gokrokve_ has quit IRC | 10:35 | |
*** Qiming has joined #heat | 10:51 | |
*** Demitar_ has joined #heat | 10:53 | |
*** Demitar has quit IRC | 10:57 | |
*** zhiyan is now known as zhiyan_ | 10:57 | |
*** Demitar has joined #heat | 10:59 | |
*** Demitar_ has quit IRC | 10:59 | |
*** matsuhashi has quit IRC | 11:00 | |
*** matsuhashi has joined #heat | 11:01 | |
*** e0ne has joined #heat | 11:05 | |
*** matsuhashi has quit IRC | 11:05 | |
*** sorantis has joined #heat | 11:06 | |
*** dmakogon_ has quit IRC | 11:15 | |
*** dmakogon_ has joined #heat | 11:15 | |
*** matsuhashi has joined #heat | 11:16 | |
*** Demitar_ has joined #heat | 11:18 | |
*** Demitar has quit IRC | 11:21 | |
*** maishsk has quit IRC | 11:23 | |
*** tiantian has quit IRC | 11:24 | |
*** Demitar has joined #heat | 11:30 | |
*** gokrokve has joined #heat | 11:32 | |
*** Demitar_ has quit IRC | 11:33 | |
*** RockKuo_Office has quit IRC | 11:35 | |
*** gokrokve has quit IRC | 11:37 | |
*** akuznetsov has joined #heat | 11:39 | |
*** dmakogon_ is now known as denis_makogon | 11:43 | |
*** Rajalakshmi has quit IRC | 11:45 | |
*** Demitar_ has joined #heat | 11:48 | |
*** Demitar has quit IRC | 11:49 | |
*** rpothier has quit IRC | 11:49 | |
*** ajc_ has quit IRC | 11:50 | |
*** Demitar has joined #heat | 11:50 | |
*** ajc_ has joined #heat | 11:50 | |
*** Demitar_ has quit IRC | 11:53 | |
*** sab has quit IRC | 11:53 | |
*** bmahalakshmi has quit IRC | 11:54 | |
*** julienvey has joined #heat | 11:54 | |
*** ajc_ has quit IRC | 11:55 | |
*** BillArnold has quit IRC | 11:56 | |
*** Demitar_ has joined #heat | 11:58 | |
*** kopparam has quit IRC | 11:59 | |
*** Demitar has quit IRC | 12:01 | |
*** mkollaro has quit IRC | 12:03 | |
*** nkhare has quit IRC | 12:04 | |
*** mkollaro has joined #heat | 12:05 | |
afaranha | Hi 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 IRC | 12:13 | |
*** matsuhashi has joined #heat | 12:14 | |
*** Demitar has joined #heat | 12:15 | |
*** Demitar_ has quit IRC | 12:17 | |
*** matsuhas_ has joined #heat | 12:18 | |
*** matsuhashi has quit IRC | 12:18 | |
*** rbuilta has joined #heat | 12:23 | |
*** kashyapk has joined #heat | 12:25 | |
*** sorantis has quit IRC | 12:25 | |
*** gokrokve has joined #heat | 12:32 | |
*** kashyapk has quit IRC | 12:34 | |
*** kashyapk has joined #heat | 12:35 | |
*** sorantis has joined #heat | 12:35 | |
shardy | afaranha: Here's one way to do it: | 12:35 |
shardy | http://paste.fedoraproject.org/111104/40318132 | 12:35 |
*** DandyPandy has joined #heat | 12:36 | |
shardy | You can pass cloud-config data which tells cloud-init to setup the password | 12:36 |
*** DandyPandy has quit IRC | 12:36 | |
*** DandyPandy has joined #heat | 12:37 | |
*** gokrokve has quit IRC | 12:37 | |
*** asalkeld has quit IRC | 12:37 | |
*** kashyapk has quit IRC | 12:39 | |
*** Demitar_ has joined #heat | 12:39 | |
*** john-n-seattle has joined #heat | 12:42 | |
*** Demitar has quit IRC | 12:42 | |
*** rpothier has joined #heat | 12:43 | |
*** erecio has joined #heat | 12:44 | |
*** andreaf has quit IRC | 12:46 | |
*** mkollaro1 has joined #heat | 12:48 | |
*** mkollaro has quit IRC | 12:48 | |
afaranha | shardy: Thanks! | 12:53 |
*** jmckind has joined #heat | 12:56 | |
*** jcoufal has quit IRC | 12:56 | |
*** Demitar has joined #heat | 12:59 | |
*** Demitar_ has quit IRC | 13:01 | |
*** mestery has joined #heat | 13:09 | |
*** mestery has quit IRC | 13:09 | |
*** mestery has joined #heat | 13:10 | |
*** Demitar_ has joined #heat | 13:10 | |
*** Demitar has quit IRC | 13:12 | |
*** pafuent has joined #heat | 13:18 | |
*** blomquisg has joined #heat | 13:19 | |
*** Demitar has joined #heat | 13:21 | |
*** Demitar_ has quit IRC | 13:22 | |
*** jcoufal has joined #heat | 13:23 | |
*** andreaf has joined #heat | 13:24 | |
*** kashyapk has joined #heat | 13:25 | |
*** Demitar_ has joined #heat | 13:28 | |
*** Demitar has quit IRC | 13:30 | |
*** gokrokve has joined #heat | 13:33 | |
*** blues-man has joined #heat | 13:35 | |
*** Demitar has joined #heat | 13:35 | |
blues-man | hello | 13:35 |
*** julienvey has quit IRC | 13:35 | |
*** norman is now known as wpf | 13:36 | |
*** DandyPandy has quit IRC | 13:37 | |
*** gokrokve has quit IRC | 13:37 | |
*** DandyPandy has joined #heat | 13:38 | |
*** arbylee has quit IRC | 13:38 | |
*** Demitar_ has quit IRC | 13:39 | |
*** kashyapk has quit IRC | 13:40 | |
*** sgordon has joined #heat | 13:40 | |
*** sgordon has quit IRC | 13:40 | |
*** sgordon has joined #heat | 13:40 | |
*** kashyapk has joined #heat | 13:40 | |
*** m_22 has joined #heat | 13:44 | |
*** kashyapk has quit IRC | 13:44 | |
*** jyoti_ranjan has quit IRC | 13:45 | |
*** jergerber has joined #heat | 13:45 | |
*** sballe has joined #heat | 13:47 | |
*** anteaya has quit IRC | 13:51 | |
*** anteaya has joined #heat | 13:53 | |
*** zhiyan_ is now known as zhiyan | 13:54 | |
*** DandyPandy has quit IRC | 13:57 | |
afaranha | shardy: 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 resource | 13:58 |
*** andersonvom has joined #heat | 13:59 | |
*** TravT has joined #heat | 13:59 | |
*** julienvey has joined #heat | 14:00 | |
afaranha | this 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 |
shardy | afaranha: It waits until you send a signal from the instance | 14:03 |
shardy | so if the time until you send that signal is 1400s, you're right, you need >1400s or it will fail | 14:03 |
*** sorantis has quit IRC | 14:04 | |
*** arbylee has joined #heat | 14:04 | |
afaranha | Understand, thank you! I was just checking because is quite expensive to just try if it works | 14:04 |
*** gokrokve has joined #heat | 14:04 | |
*** arbylee has quit IRC | 14:08 | |
*** arbylee has joined #heat | 14:09 | |
*** e0ne has quit IRC | 14:10 | |
*** kashyapk has joined #heat | 14:11 | |
*** bnemec has joined #heat | 14:14 | |
*** sjmc7 has joined #heat | 14:14 | |
*** zz_gondoi is now known as gondoi | 14:15 | |
*** kashyapk has quit IRC | 14:15 | |
*** e0ne has joined #heat | 14:17 | |
*** matsuhas_ has quit IRC | 14:17 | |
jasond | hm, gerrit stopped resetting the votes on new patchsets | 14:18 |
*** mcwoods has quit IRC | 14:19 | |
*** Demitar_ has joined #heat | 14:20 | |
*** Demitar has quit IRC | 14:22 | |
*** tango has joined #heat | 14:22 | |
*** matsuhashi has joined #heat | 14:23 | |
*** tspatzier has joined #heat | 14:24 | |
*** sabeen has joined #heat | 14:24 | |
*** jdob has joined #heat | 14:33 | |
*** rakesh_hs2 has quit IRC | 14:33 | |
*** matsuhashi has quit IRC | 14:33 | |
*** matsuhashi has joined #heat | 14:33 | |
*** rwsu has joined #heat | 14:34 | |
*** DandyPandy has joined #heat | 14:34 | |
*** kirankv has quit IRC | 14:37 | |
*** dims has joined #heat | 14:37 | |
*** matsuhashi has quit IRC | 14:38 | |
*** matsuhashi has joined #heat | 14:40 | |
*** noTHD has quit IRC | 14:41 | |
*** jergerber has quit IRC | 14:41 | |
pas-ha | jasond, there's something more strange than that | 14:42 |
pas-ha | more like if it was a conflict-less rebase that it does not reset. but that was yesterday... | 14:43 |
jasond | hopefully it's just a bug | 14:46 |
*** noTHD has joined #heat | 14:55 | |
shardy | pas-ha: That's expected, there's trivial rebase detection which re-applies previous reviews (but not approval) if a simple rebase has been performed | 14:56 |
pas-ha | yes, but I think I saw some inconsistencies when it is a series of dependent patches and something changes down the line | 14:57 |
*** e0ne has quit IRC | 14:59 | |
*** e0ne has joined #heat | 15:00 | |
zaneb | I think the trivial rebase detector script has gone away, and that functionality is now built in to Gerrit | 15:00 |
*** david-lyle has joined #heat | 15:02 | |
jasond | apparently, it doesn't always do the right thing https://review.openstack.org/#/c/94957/ | 15:02 |
*** noTHD has quit IRC | 15:02 | |
*** david-lyle has quit IRC | 15:03 | |
zaneb | jasond: those two patchsets are the same... how could it do anything else? | 15:04 |
*** david-lyle has joined #heat | 15:04 | |
jasond | zaneb: i added the correct dependency in patchset 2 | 15:04 |
*** e0ne has quit IRC | 15:04 | |
*** jprovazn is now known as jprovazn_afk | 15:05 | |
zaneb | right, but the heuristic is that if two patch sets are the same, they keep their votes | 15:05 |
jasond | is that the new behavior or did it always work that way? | 15:06 |
zaneb | the only alternatives are (a) never keep the votes, even for trivial rebases, or (b) build an artificial intelligence system that can interpret comments | 15:06 |
zaneb | it's new to gerrit, but the trivial rebase script always worked that way | 15:06 |
zaneb | the only difference is the script used to leave a comment to say what it was doing | 15:07 |
sjmc7 | i vote b) | 15:07 |
zaneb | rofl | 15:07 |
*** matsuhashi has quit IRC | 15:07 | |
*** m_22 has quit IRC | 15:07 | |
*** matsuhashi has joined #heat | 15:07 | |
jasond | oh ok. for some reason i thought it only kept the votes if the commit message changed | 15:08 |
sjmc7 | zaneb - 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 situations | 15:08 |
zaneb | sjmc7: ooooh, interesting | 15:08 |
sjmc7 | i filed https://bugs.launchpad.net/heat/+bug/1331872, but it means this was broken for over a month | 15:08 |
uvirtbot | Launchpad bug 1331872 in heat "Changes in update_and_save not always committed" [Undecided,New] | 15:08 |
zaneb | tell me more | 15:08 |
sjmc7 | so i'm surprised nobody found it | 15:08 |
zaneb | how do you know it was broken for over a month? | 15:09 |
sjmc7 | the update-and-save code was added may 5 | 15:09 |
zaneb | sjmc7: of 2012 | 15:10 |
sjmc7 | ah :) | 15:11 |
sjmc7 | i'll blame that on 8pm | 15:11 |
sjmc7 | it would only affect situations where the session wasn't already attached to the object | 15:11 |
zaneb | lol | 15:11 |
sjmc7 | so it may just be that this happens to be triggering it | 15:11 |
*** matsuhashi has quit IRC | 15:11 | |
zaneb | we *should* be using one common session for everything in a request | 15:11 |
*** kashyapk has joined #heat | 15:12 | |
zaneb | it's stored in the context | 15:12 |
therve | You don't really want long-lived transations though | 15:12 |
sjmc7 | it's calling get_session rather than using one already attached to the model | 15:12 |
*** blamar has joined #heat | 15:12 | |
zaneb | oh, I am not passing the context from parser.Stack | 15:13 |
zaneb | my fault | 15:13 |
zaneb | easily fixed | 15:13 |
* zaneb mutters about optional parameters | 15:14 | |
zaneb | going afk for a few minutes, but will post a patch when I get back | 15:14 |
sjmc7 | no worries | 15:15 |
*** kashyapk has quit IRC | 15:16 | |
*** Demitar has joined #heat | 15:18 | |
*** kgriffs|afk is now known as kgriffs | 15:18 | |
*** Demitar_ has quit IRC | 15:19 | |
*** jcoufal has quit IRC | 15:21 | |
*** TravT has quit IRC | 15:26 | |
*** Demitar has quit IRC | 15:30 | |
*** denis_makogon has quit IRC | 15:32 | |
*** dmakogon_ has joined #heat | 15:33 | |
*** TravT has joined #heat | 15:35 | |
*** mkollaro1 has quit IRC | 15:35 | |
*** Demitar has joined #heat | 15:36 | |
zaneb | back | 15:37 |
*** ramishra has joined #heat | 15:38 | |
*** harset has joined #heat | 15:39 | |
harset | hi | 15:39 |
*** mkollaro has joined #heat | 15:39 | |
*** ramishra_ has joined #heat | 15:41 | |
harset | hi 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 |
sjmc7 | can you repaste the template, harset? | 15:42 |
zaneb | sjmc7: remove the comma and it works | 15:43 |
sjmc7 | 8pm! | 15:43 |
*** ramishra has quit IRC | 15:43 | |
harset | sjmc7, why? | 15:43 |
zaneb | harset: your problem is that you have an unholy mix of HOT and cfn syntax | 15:44 |
zaneb | harset: change Ref to get_resource or get_param, as appropriate | 15:44 |
zaneb | and Fn::GetAtt to get_attr | 15:45 |
*** saju_m has quit IRC | 15:45 | |
harset | zaneb, so eg i must change LaunchConfigurationName: {Ref: launch} to LaunchConfigurationName: {get_resource: launch} ?! | 15:46 |
*** mkollaro has quit IRC | 15:46 | |
zaneb | harset: yes, you should. although looking at the code, it appears we do allow Ref | 15:47 |
zaneb | so it must be another problem | 15:47 |
zaneb | Fn::GetAtt is the problem. use get_attr instead | 15:49 |
zaneb | on lines 66 & 82 | 15:49 |
harset | zaneb, wow this fix it :) thank you really much! | 15:51 |
zaneb | np | 15:52 |
zaneb | we should probably do a bit more validation there, to give a better error message | 15:52 |
harset | zaneb, yes :) | 15:53 |
*** alexpilotti has quit IRC | 15:55 | |
*** alexpilotti has joined #heat | 15:55 | |
zaneb | harset: if you file a bug, I'll fix it right now. deal? | 15:56 |
sjmc7 | zaneb, my other fires are dying out, i'll fix the stack update thing | 15:56 |
zaneb | sjmc7: testing a patch right now, but there are unit test failures | 15:57 |
*** e0ne has joined #heat | 15:57 | |
sjmc7 | ah, ok, great (not the failures). will test when you're done | 15:57 |
zaneb | :( | 15:57 |
zaneb | it'a like the unit tests only work without context and the real world only works with it. sigh | 15:59 |
harset | zaneb, absolutely not, I am grateful for the work you do. I simply agree with you about the cryptic error message :| | 15:59 |
zaneb | harset: I'm offering to fix the cryptic error message if you file a bug report about it | 16:00 |
sjmc7 | :) | 16:00 |
*** lazy_prince has quit IRC | 16:00 | |
*** alexpilotti has quit IRC | 16:01 | |
harset | zaneb, ah ok,now I understood :) .. where is the right place to file a bug report? https://launchpad.net/heat here?! | 16:01 |
sjmc7 | yep, harset | 16:01 |
zaneb | yep | 16:01 |
*** e0ne has quit IRC | 16:01 | |
sjmc7 | which tests are failing, zaneb? | 16:02 |
harset | ok I'll do after the work in office :) | 16:02 |
zaneb | sjmc7: basically all of the stack update ones | 16:02 |
*** e0ne has joined #heat | 16:03 | |
zaneb | sjmc7: http://paste.fedoraproject.org/111175/31937971 <- there is the patch if you want to try it | 16:03 |
harset | mmm... 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 |
sjmc7 | the last three days have been like the software twilight zone. nothing surprises me any more | 16:03 |
*** Qiming has quit IRC | 16:04 | |
*** david-lyle has quit IRC | 16:04 | |
*** jcoufal has joined #heat | 16:06 | |
*** david-lyle has joined #heat | 16:07 | |
*** packet has joined #heat | 16:09 | |
*** Demitar_ has joined #heat | 16:15 | |
zaneb | harset: check the heat-engine log. most likely you are encountering the same issue that sjmc7 and others were having yesterday | 16:17 |
zaneb | we think that was due to https://bugs.launchpad.net/tripleo/+bug/1331445 | 16:17 |
uvirtbot | Launchpad bug 1331445 in tripleo "Refusing to bootstrap mysql cluster until role is known. (missing heat metadata) (dup-of: 1331720)" [Critical,Triaged] | 16:17 |
uvirtbot | Launchpad bug 1331720 in heat "missing TripleO heat metadata" [High,In progress] | 16:17 |
zaneb | and the offending commit there has been reverted, so you might want to update your code and try again | 16:17 |
sjmc7 | zaneb- that patch fixes the update issue | 16:17 |
*** blamar has quit IRC | 16:18 | |
*** Demitar has quit IRC | 16:18 | |
zaneb | sjmc7: cool! so I just have to make the unit tests work :/ | 16:19 |
*** bgorski has joined #heat | 16:20 | |
sjmc7 | yeah.. i'm puzzled that this change breaks them | 16:22 |
*** nati_ueno has joined #heat | 16:23 | |
*** Fayablazer has quit IRC | 16:23 | |
zaneb | sjmc7: 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 context | 16:23 |
harset | zaneb, I can't spot any error in the heat-engine.log http://pastebin.com/r2KBFqnA | 16:25 |
zaneb | harset: is there nothing after that? looks like the alarm is not being triggered at all | 16:30 |
*** cdent_ has joined #heat | 16:31 | |
*** julienvey has quit IRC | 16:32 | |
*** cdent has quit IRC | 16:32 | |
*** cdent_ is now known as cdent | 16:32 | |
harset | zaneb, nothing :( ..but the alarm is triggered http://pastebin.com/PkDaQL1M (state==alarm) | 16:33 |
harset | here http://pastebin.com/vDtYwdXS the resource list | 16:34 |
zaneb | yeah, so somewhere between Ceilometer and heat-engine it is getting lost | 16:34 |
zaneb | I'd check the ceilometer and heat-api-cfn logs to see how far it is getting | 16:35 |
*** kfox1111 has joined #heat | 16:35 | |
harset | zaneb, heat-api (I haven't an heat-api-cfn log) shows only | 16:36 |
harset | INFO urllib3.connectionpool [-] Starting new HTTP connection (1): 10.0.0.98 | 16:36 |
zaneb | iirc the URL returned from AWS::AutoScaling::ScalingPolicy will be to heat-api-cfn, so the fact you're not running that is probably the issue | 16:38 |
harset | zaneb, here http://pastebin.com/gbfcmtxZ the resource show for CPUAlarmHigh | 16:39 |
harset | zaneb, how can I check if I'm running all the necessary services? | 16:40 |
zaneb | ok, I'm wrong. port 8004 is heat-api, not heat-api-cfn | 16:41 |
zaneb | oh, sorry, no | 16:41 |
*** jcoufal has quit IRC | 16:41 | |
zaneb | that is just the resource link | 16:41 |
zaneb | we can't see the alarm url | 16:41 |
*** jcoufal has joined #heat | 16:42 | |
zaneb | harset: are you using devstack? | 16:42 |
harset | zaneb, no i'm using packstack on a centos | 16:42 |
zaneb | ah, ok | 16:42 |
harset | packstack Icehouse 2014.1.1dev1068 | 16:42 |
*** e0ne has quit IRC | 16:43 | |
zaneb | telnet localhost 8000 | 16:43 |
zaneb | should give you a fair idea if heat-api-cfn is running | 16:43 |
*** e0ne has joined #heat | 16:43 | |
zaneb | although the fact that there's no log file for it is pretty conclusive already | 16:43 |
harset | zaneb, netstat -natp |grep 8000 show me nothing | 16:43 |
shardy | harset: did you specify --os-heat-cfn-install=y when running packstack? | 16:45 |
harset | zaneb, CONFIG_HEAT_CFN_INSTALL=n .. no :( | 16:45 |
zaneb | it seems pretty clear that you haven't got heat-api-cfn installed/running | 16:45 |
zaneb | harset: ok, so edit your packstack answer file and re-run packstack | 16:45 |
shardy | harset: You can re-run packstack after setting that in the answerfile | 16:45 |
shardy | Then you'll need to make a minor fix to the heat.conf: | 16:46 |
shardy | https://bugzilla.redhat.com/show_bug.cgi?id=1106394 | 16:46 |
uvirtbot | shardy: Error: Could not parse XML returned by bugzilla.redhat.com: HTTP Error 404: Not Found | 16:46 |
harset | can I use both cfn and not cfn in heat? | 16:46 |
zaneb | harset: yes | 16:46 |
harset | zaneb, thank you very much for the help! | 16:46 |
zaneb | both APIs are independent and they talk to the same engine | 16:46 |
harset | I must read more carefull this http://docs.openstack.org/developer/heat/template_guide/hot_guide.html :| | 16:47 |
shardy | actually bug #1318599 has a better description of the problem | 16:47 |
uvirtbot | Launchpad 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/1318599 | 16:47 |
*** e0ne has quit IRC | 16:48 | |
shardy | You need to remove the "ec2tokens" from the ec2authtoken auth_uri in heat.conf, as packstack currently misconfigures it | 16:48 |
zaneb | thanks shardy, I would have completely forgotten about that bug | 16:48 |
shardy | relatedly, anyone fancy reviewing this fix? | 16:48 |
shardy | https://review.openstack.org/#/c/98827/ | 16:48 |
shardy | we probably need the puppet stuff fixed too, but that makes heat a bit more tolerant | 16:49 |
*** gokrokve has quit IRC | 16:49 | |
*** jmckind has quit IRC | 16:50 | |
zaneb | shardy: approved | 16:51 |
shardy | thanks | 16:51 |
harset | shardy, thanks for the hint! | 16:52 |
*** zhiyan is now known as zhiyan_ | 16:54 | |
*** nati_ueno has quit IRC | 16:59 | |
*** nati_ueno has joined #heat | 17:02 | |
*** david-lyle has quit IRC | 17:03 | |
*** david-lyle has joined #heat | 17:05 | |
ramishra_ | Hello zaneb | 17:05 |
*** harlowja_away is now known as harlowja | 17:05 | |
ramishra_ | zaneb: I am still seeing the same issue https://bugs.launchpad.net/heat/+bug/1328342 after your related patch. Am I missing something | 17:06 |
uvirtbot | Launchpad bug 1328342 in heat "stack-update with SoftwareDeployment 400 response from signal" [High,Fix committed] | 17:06 |
sjmc7 | ramishra_ - zane's working on a patch now | 17:07 |
zaneb | ramishra_: yes, we've established the fix isn't working, see bug #1331872 | 17:07 |
uvirtbot | Launchpad bug 1331872 in heat "Changes in update_and_save not always committed" [High,Confirmed] https://launchpad.net/bugs/1331872 | 17:07 |
sjmc7 | the patch here fixes it if you want to try - http://paste.fedoraproject.org/111175/31937971 | 17:07 |
ramishra_ | oh.. ok | 17:08 |
*** BillArnold has joined #heat | 17:10 | |
*** blamar has joined #heat | 17:15 | |
ramishra_ | sjmc7: yep.. it works.. thanks.. | 17:16 |
*** gokrokve has joined #heat | 17:18 | |
zaneb | nothing more frustrating than a patch that works for users, but the tests say it's broken. and vice-versa. | 17:20 |
sjmc7 | zaneb- which tests aren't working for you? | 17:20 |
*** blues-man has quit IRC | 17:23 | |
*** proffalken has quit IRC | 17:24 | |
*** e0ne has joined #heat | 17:29 | |
*** kfox1111 has quit IRC | 17:33 | |
*** kfox1111 has joined #heat | 17:33 | |
*** ramishra_ has quit IRC | 17:34 | |
harset | bye | 17:35 |
*** harset has quit IRC | 17:35 | |
*** tspatzier has quit IRC | 17:37 | |
*** jcoufal has quit IRC | 17:38 | |
*** tspatzier has joined #heat | 17:45 | |
*** saju_m has joined #heat | 17:46 | |
*** tspatzier has quit IRC | 17:46 | |
*** lipinski has joined #heat | 17:48 | |
lipinski | Question about stack-update | 17:48 |
lipinski | I have a stack that was created with a environment and template file(s). | 17:48 |
lipinski | If 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 #heat | 17:50 | |
*** proffalken has joined #heat | 17:51 | |
*** randallburt has joined #heat | 17:52 | |
zaneb | sjmc7: 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_2 | 17:52 |
sjmc7 | abc and smelly | 17:52 |
zaneb | sjmc7: so the problem is with rollback, evidently | 17:52 |
zaneb | yeah | 17:52 |
sjmc7 | yeah, i just got the same. it takes forever to run the tests on my system for some reason | 17:52 |
*** e0ne has quit IRC | 17:52 | |
*** e0ne has joined #heat | 17:53 | |
zaneb | lipinski: 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 used | 17:53 |
*** sab has quit IRC | 17:54 | |
*** ericgoncz has joined #heat | 17:54 | |
*** randallburt has quit IRC | 17:54 | |
zaneb | sjmc7: btw the test_hot ones are just duplicates of the test_parser ones | 17:54 |
*** randallburt has joined #heat | 17:54 | |
*** blamar has quit IRC | 17:57 | |
*** e0ne has quit IRC | 17:57 | |
sjmc7 | the stack will still be in the session, right? | 17:57 |
*** rbuilta has quit IRC | 17:57 | |
zaneb | should be, yeah | 17:58 |
sjmc7 | i hate ORMs sometimes :) | 17:59 |
*** BillArnold has quit IRC | 18:01 | |
*** lskyw has joined #heat | 18:02 | |
*** alexpilotti has joined #heat | 18:03 | |
*** lskyw has quit IRC | 18:03 | |
*** lskyw has joined #heat | 18:04 | |
*** andreaf_ has joined #heat | 18:05 | |
*** david-lyle has quit IRC | 18:08 | |
*** dims has quit IRC | 18:08 | |
*** andersonvom has quit IRC | 18:08 | |
*** sballe has quit IRC | 18:08 | |
*** john-n-seattle has quit IRC | 18:08 | |
*** ccrouch1 has quit IRC | 18:08 | |
*** zpatten has quit IRC | 18:08 | |
*** gokrokve has quit IRC | 18:08 | |
*** arbylee has quit IRC | 18:08 | |
*** andreaf has quit IRC | 18:08 | |
*** PhilK_ has quit IRC | 18:08 | |
*** pas-ha has quit IRC | 18:08 | |
*** mkerrin has quit IRC | 18:08 | |
*** wpf has quit IRC | 18:08 | |
*** jasond has quit IRC | 18:08 | |
*** cody-somerville has quit IRC | 18:08 | |
*** skraynev has quit IRC | 18:08 | |
*** fbo_away has quit IRC | 18:08 | |
*** trash has quit IRC | 18:08 | |
*** lindsayk has quit IRC | 18:08 | |
*** tnurlygayanov has quit IRC | 18:08 | |
*** wendar has quit IRC | 18:08 | |
*** harlowja has quit IRC | 18:08 | |
*** sileht has quit IRC | 18:08 | |
*** adam_g has quit IRC | 18:08 | |
*** kgriffs has quit IRC | 18:08 | |
*** gondoi has quit IRC | 18:08 | |
*** stevebaker has quit IRC | 18:08 | |
*** lvh has quit IRC | 18:08 | |
*** FL1SK has quit IRC | 18:08 | |
*** Adri2000 has quit IRC | 18:08 | |
*** zigo has quit IRC | 18:08 | |
*** radix has quit IRC | 18:08 | |
*** zhiyan_ has quit IRC | 18:08 | |
*** chmouel has quit IRC | 18:08 | |
*** shufflebot has quit IRC | 18:08 | |
*** uvirtbot has quit IRC | 18:08 | |
*** abramley has quit IRC | 18:08 | |
*** cdent has quit IRC | 18:08 | |
*** gokrokve has joined #heat | 18:10 | |
*** david-lyle has joined #heat | 18:10 | |
*** dims has joined #heat | 18:10 | |
*** arbylee has joined #heat | 18:10 | |
*** andersonvom has joined #heat | 18:10 | |
*** sballe has joined #heat | 18:10 | |
*** john-n-seattle has joined #heat | 18:10 | |
*** PhilK_ has joined #heat | 18:10 | |
*** pas-ha has joined #heat | 18:10 | |
*** mkerrin has joined #heat | 18:10 | |
*** ccrouch1 has joined #heat | 18:10 | |
*** wpf has joined #heat | 18:10 | |
*** wendar has joined #heat | 18:10 | |
*** zpatten has joined #heat | 18:10 | |
*** harlowja has joined #heat | 18:10 | |
*** jasond has joined #heat | 18:10 | |
*** cody-somerville has joined #heat | 18:10 | |
*** tnurlygayanov has joined #heat | 18:10 | |
*** skraynev has joined #heat | 18:10 | |
*** sileht has joined #heat | 18:10 | |
*** fbo_away has joined #heat | 18:10 | |
*** adam_g has joined #heat | 18:10 | |
*** trash has joined #heat | 18:10 | |
*** kgriffs has joined #heat | 18:10 | |
*** lindsayk has joined #heat | 18:10 | |
*** uvirtbot has joined #heat | 18:10 | |
*** chmouel has joined #heat | 18:10 | |
*** shufflebot has joined #heat | 18:10 | |
*** radix has joined #heat | 18:10 | |
*** zigo has joined #heat | 18:10 | |
*** zhiyan_ has joined #heat | 18:10 | |
*** FL1SK has joined #heat | 18:10 | |
*** lvh has joined #heat | 18:10 | |
*** Adri2000 has joined #heat | 18:10 | |
*** abramley has joined #heat | 18:10 | |
*** stevebaker has joined #heat | 18:10 | |
*** gondoi has joined #heat | 18:10 | |
*** kashyapk has joined #heat | 18:11 | |
*** ericgoncz has quit IRC | 18:12 | |
*** david-lyle has quit IRC | 18:14 | |
*** dims has quit IRC | 18:14 | |
*** andersonvom has quit IRC | 18:14 | |
*** sballe has quit IRC | 18:14 | |
*** john-n-seattle has quit IRC | 18:14 | |
*** ccrouch1 has quit IRC | 18:14 | |
*** zpatten has quit IRC | 18:14 | |
*** gokrokve has quit IRC | 18:14 | |
*** arbylee has quit IRC | 18:14 | |
*** PhilK_ has quit IRC | 18:14 | |
*** pas-ha has quit IRC | 18:14 | |
*** mkerrin has quit IRC | 18:14 | |
*** wpf has quit IRC | 18:14 | |
*** jasond has quit IRC | 18:14 | |
*** cody-somerville has quit IRC | 18:14 | |
*** skraynev has quit IRC | 18:14 | |
*** fbo_away has quit IRC | 18:14 | |
*** trash has quit IRC | 18:14 | |
*** lindsayk has quit IRC | 18:14 | |
*** tnurlygayanov has quit IRC | 18:14 | |
*** wendar has quit IRC | 18:14 | |
*** harlowja has quit IRC | 18:14 | |
*** sileht has quit IRC | 18:14 | |
*** adam_g has quit IRC | 18:14 | |
*** kgriffs has quit IRC | 18:14 | |
*** gondoi has quit IRC | 18:14 | |
*** stevebaker has quit IRC | 18:14 | |
*** lvh has quit IRC | 18:14 | |
*** FL1SK has quit IRC | 18:14 | |
*** Adri2000 has quit IRC | 18:14 | |
*** zigo has quit IRC | 18:14 | |
*** radix has quit IRC | 18:14 | |
*** zhiyan_ has quit IRC | 18:14 | |
*** chmouel has quit IRC | 18:14 | |
*** shufflebot has quit IRC | 18:14 | |
*** uvirtbot has quit IRC | 18:14 | |
*** abramley has quit IRC | 18:14 | |
*** ericgoncz has joined #heat | 18:15 | |
*** gokrokve has joined #heat | 18:16 | |
*** david-lyle has joined #heat | 18:16 | |
*** dims has joined #heat | 18:16 | |
*** arbylee has joined #heat | 18:16 | |
*** andersonvom has joined #heat | 18:16 | |
*** sballe has joined #heat | 18:16 | |
*** john-n-seattle has joined #heat | 18:16 | |
*** PhilK_ has joined #heat | 18:16 | |
*** pas-ha has joined #heat | 18:16 | |
*** mkerrin has joined #heat | 18:16 | |
*** ccrouch1 has joined #heat | 18:16 | |
*** wpf has joined #heat | 18:16 | |
*** wendar has joined #heat | 18:16 | |
*** zpatten has joined #heat | 18:16 | |
*** harlowja has joined #heat | 18:16 | |
*** jasond has joined #heat | 18:16 | |
*** cody-somerville has joined #heat | 18:16 | |
*** tnurlygayanov has joined #heat | 18:16 | |
*** skraynev has joined #heat | 18:16 | |
*** sileht has joined #heat | 18:16 | |
*** fbo_away has joined #heat | 18:16 | |
*** adam_g has joined #heat | 18:16 | |
*** trash has joined #heat | 18:16 | |
*** kgriffs has joined #heat | 18:16 | |
*** lindsayk has joined #heat | 18:16 | |
*** uvirtbot has joined #heat | 18:16 | |
*** chmouel has joined #heat | 18:16 | |
*** shufflebot has joined #heat | 18:16 | |
*** radix has joined #heat | 18:16 | |
*** zigo has joined #heat | 18:16 | |
*** zhiyan_ has joined #heat | 18:16 | |
*** FL1SK has joined #heat | 18:16 | |
*** lvh has joined #heat | 18:16 | |
*** Adri2000 has joined #heat | 18:16 | |
*** abramley has joined #heat | 18:16 | |
*** stevebaker has joined #heat | 18:16 | |
*** gondoi has joined #heat | 18:16 | |
*** bmahalakshmi has joined #heat | 18:18 | |
*** ericgoncz has quit IRC | 18:21 | |
*** BillArnold has joined #heat | 18:28 | |
*** kgriffs is now known as kgriffs|afk | 18:30 | |
lipinski | zaneb: sorry, stepped away for a bit. Is there a way to prevent replacement? | 18:31 |
lipinski | What is happening is this. | 18:31 |
zaneb | lipinski: yes, don't change anything ;) | 18:31 |
lipinski | I 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 VM | 18:31 |
lipinski | I 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 |
lipinski | It seems to be thinking the paramter changed, so the SoftwareConfig changed, so it rebuilds the VM | 18:32 |
zaneb | sjmc7: W00t! fixed it | 18:32 |
*** dims has quit IRC | 18:33 | |
lipinski | It'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 |
sjmc7 | ooo! i've been poking at it off and on for the last hour | 18:33 |
*** zpatten has quit IRC | 18:33 | |
sjmc7 | what caused it? | 18:33 |
*** sballe has quit IRC | 18:33 | |
*** dims has joined #heat | 18:33 | |
*** zpatten has joined #heat | 18:33 | |
lipinski | If there was some way to break the resource dependency on the parameter... | 18:33 |
*** andersonvom has quit IRC | 18:33 | |
zaneb | lipinski: read the whiteboard on https://blueprints.launchpad.net/heat/+spec/troubleshooting-low-level-control ... I think that's what you want? | 18:33 |
zaneb | sjmc7: 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 correctly | 18:35 |
*** kashyapk has quit IRC | 18:35 | |
sjmc7 | ah, that makes sense. thanks for spending the time on this | 18:36 |
*** kashyapk has joined #heat | 18:37 | |
lipinski | zaneb: thanks - will check that out | 18:38 |
zaneb | lipinski: to be clear, that's not implemented yet, but tango is looking at it now | 18:39 |
*** kgriffs|afk is now known as kgriffs | 18:43 | |
tango | lipinski: 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 support | 18:50 |
zaneb | sjmc7: https://review.openstack.org/#/c/101288/ <- the fix | 18:51 |
sjmc7 | thanks, will check it | 18:52 |
*** david-lyle has quit IRC | 18:54 | |
zaneb | I guess we lost our notification bot in the last freenode split-brain | 18:54 |
sjmc7 | zaneb - 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 |
sjmc7 | is that what you'd expect? | 18:59 |
zaneb | yep, that's correct | 19:00 |
sjmc7 | splendid | 19:00 |
zaneb | so the original one should change, and the backup one should hopefully not | 19:00 |
sjmc7 | yep, correct | 19:00 |
sjmc7 | +1ed, many thanks | 19:00 |
zaneb | we should probably stop soft-deleting the backup stacks | 19:01 |
zaneb | they really ought to be hard-deleted when we're done with them | 19:01 |
sjmc7 | yeah | 19:01 |
zaneb | I'll open a bug for that | 19:02 |
SpamapS | zaneb: https://review.openstack.org/#/c/101288/ needs tests | 19:03 |
SpamapS | zaneb: but very awesome that we tracked this down | 19:03 |
*** samuelmz_ has joined #heat | 19:03 | |
*** sabeen has quit IRC | 19:05 | |
*** sabeen has joined #heat | 19:05 | |
*** sabeen has quit IRC | 19:06 | |
*** sabeen has joined #heat | 19: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 String | 19:06 |
samuelmz_ | afaik, it should point to another resource into the template, for instance: LaunchConfigurationName : compute-node, where compute node is another resource | 19:07 |
samuelmz_ | what type of resource it should point to? I tried with OS::Nova::Server and it didnt work :/ | 19:08 |
zaneb | samuelmz: AWS::AutoScaling::LaunchConfiguration | 19:08 |
zaneb | hence the name ;) | 19:09 |
SpamapS | thats not confusing _at all_. :-/ | 19:09 |
samuelmz_ | is that not possible using an OS::?? resource? | 19:10 |
*** nati_uen_ has joined #heat | 19:10 | |
zaneb | samuelmz_: no | 19:11 |
samuelmz_ | I'd like to use a pure hot template | 19:11 |
samuelmz_ | I'm following the http://docs.openstack.org/developer/heat/template_guide/hot_spec.html | 19:11 |
zaneb | samuelmz_: and btw, you should use get_resource, not just use the name (although both will work atm) | 19:11 |
*** kashyapk has quit IRC | 19:11 | |
zaneb | samuelmz_: I recommend using OS::Heat::AutoScalingGroup instead of InstanceGroup | 19:11 |
zaneb | that will let you define the launch configuration inline, without using any AWS resources | 19:12 |
zaneb | although, for the record, use of AWS resources and the template format (HOT vs. CloudFormation) are completely orthogonal | 19: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::?? alternative | 19:13 |
zaneb | well, the alternative is for OS::Heat::InstanceGroup to just not exist | 19:14 |
*** nati_ueno has quit IRC | 19:14 | |
zaneb | it's a building block for our AWS compatibility resources that we exposed to users | 19:14 |
zaneb | the way forward for the future is OS::Heat::AutoScalingGroup | 19:14 |
samuelmz_ | zaneb: dont you plan to have a resource like OS::Heat::LaunchConfiguration? | 19:14 |
zaneb | no, because we have OS::Heat::AutoScalingGroup | 19:15 |
samuelmz_ | zaneb: hmm, great... thats what I was looking for | 19:15 |
*** saju_m has quit IRC | 19:20 | |
samuelmz_ | zaneb: I'm looking at OS::Heat::AutoScalingGroup .. it will fit better what I want .. thanks for your help | 19:21 |
SpamapS | at one point we had wanted to have OS:: equivalents for everything | 19:24 |
*** noTHD has joined #heat | 19:25 | |
*** julienvey has joined #heat | 19:27 | |
zaneb | SpamapS: we do, but InstanceGroup is going to get deprecated | 19:28 |
*** e0ne has joined #heat | 19:28 | |
zaneb | if OS::Heat::AutoScalingGroup required a launch config, we would want a native one | 19:29 |
zaneb | but we decided it wouldn't use one at all | 19:29 |
kfox1111 | anyone familior with mock awake? | 19:29 |
SpamapS | zaneb: right I like the inline config better. :) | 19:32 |
zaneb | I don't, but I didn't win the argument ;) | 19:32 |
*** bmahalakshmi has quit IRC | 19:35 | |
*** e0ne has quit IRC | 19:35 | |
*** e0ne has joined #heat | 19:36 | |
SpamapS | zaneb: well we need more generic composition methods | 19:37 |
SpamapS | zaneb: only having group configs be reusable is not ideal. | 19:38 |
zaneb | I do agree with that | 19:38 |
*** e0ne has quit IRC | 19:38 | |
zaneb | I am disappointed that the AWS compat resources will now be hacks forever though | 19:39 |
zaneb | but I have moved on with my life ;) | 19:39 |
SpamapS | I think the more we diverge the more compelling a CFN -> HOT translator will become. | 19:40 |
SpamapS | we'll get tired of maintaining parallel resource plugins | 19:40 |
zaneb | you might be right | 19:41 |
*** e0ne has joined #heat | 19:41 | |
SpamapS | and now to fix this tempest test for os-collect-config | 19:41 |
*** m_22 has joined #heat | 19:48 | |
*** e0ne has quit IRC | 19:49 | |
*** e0ne has joined #heat | 19:49 | |
*** akuznetsov has quit IRC | 19:49 | |
*** sabeen has quit IRC | 19:51 | |
*** sabeen has joined #heat | 19:51 | |
*** e0ne has quit IRC | 19:52 | |
*** akuznetsov has joined #heat | 19:55 | |
*** e0ne has joined #heat | 19:57 | |
*** akuznetsov has quit IRC | 19:59 | |
*** e0ne has quit IRC | 20:02 | |
*** david-lyle has joined #heat | 20:10 | |
*** samuelmz_ has quit IRC | 20:20 | |
*** nati_uen_ has quit IRC | 20:25 | |
*** nati_ueno has joined #heat | 20:25 | |
*** nati_ueno has quit IRC | 20:27 | |
*** nati_ueno has joined #heat | 20:28 | |
*** blamar has joined #heat | 20:28 | |
*** akuznetsov has joined #heat | 20:34 | |
*** alexpilotti has quit IRC | 20:35 | |
shardy | zaneb, SpamapS: when we solve the nesting limit issue, most CFN resources should become provider templates IMO | 20:36 |
*** m_22 has quit IRC | 20:36 | |
zaneb | most probably should | 20:36 |
zaneb | I think there are a bunch where that is not technically possible though | 20:37 |
zaneb | I know asalkeld had to jump through a lot of hoops to do that for alarms, for example | 20:37 |
zaneb | including creating multiple new intrinsic functions | 20:37 |
shardy | Yeah I guess there may be a few, but many of them would be simple wrappers | 20:39 |
zaneb | agree | 20:40 |
*** mestery has quit IRC | 20:42 | |
*** mestery has joined #heat | 20:43 | |
*** sorantis has joined #heat | 20:46 | |
*** sabeen1 has joined #heat | 20:46 | |
*** erecio has quit IRC | 20:47 | |
*** m_22 has joined #heat | 20:47 | |
*** mestery has quit IRC | 20:47 | |
SpamapS | shardy: What if we solved the "1:1 provider resource mappings shouldn't be a whole nested stack" issue instead? | 20:47 |
SpamapS | or 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 |
zaneb | have we established that there is actually a "problem"? | 20:49 |
shardy | SpamapS: 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 lot | 20:49 |
zaneb | other than the arbitrary limit being too low | 20:49 |
*** sabeen has quit IRC | 20:49 | |
shardy | zaneb: well if you use one nested stack containing a provider resource now, you're at the limit | 20:49 |
SpamapS | zaneb: It's too low | 20:50 |
SpamapS | I went conservative. | 20:50 |
SpamapS | And I stand by that default | 20:50 |
zaneb | right, we can all agree that it's too low :) | 20:50 |
SpamapS | until we can count all the resources in a nested chain reliably... it has to be low. | 20:50 |
shardy | SpamapS: IIRC you mentioned a recursive resource count instead, which I think would be much better | 20:50 |
zaneb | ok, so the problem is that we can't count reliably | 20:50 |
SpamapS | which 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 IRC | 20:51 | |
zaneb | and that's because some templates could be fetched at runtime? | 20:51 |
SpamapS | We could just graft them on. | 20:51 |
*** jomara has quit IRC | 20:51 | |
zaneb | because i know an easy solution to that ;) | 20:51 |
SpamapS | The counting problem, anyway, may just have been me being new, and not having enough clarity on the issue. | 20:52 |
*** packet has quit IRC | 20:52 | |
zaneb | SpamapS: btw bgorski's new OS::Heat::Stack resource is always going to use python-heatclient to create nested stacks | 20:52 |
zaneb | so that also solves this problem, I think | 20:53 |
zaneb | we'll only have to worry about layers of nesting for providers | 20:53 |
shardy | zaneb: does it? Or does it just circumvent the limit? | 20:53 |
zaneb | and let quotas on the API handle the rest | 20:53 |
*** pafuent has left #heat | 20:53 | |
SpamapS | hm | 20:54 |
SpamapS | zaneb: I like that.. sort of going the opposite direction and making the explicit nested stack _more_ separate. | 20:54 |
SpamapS | so then we can loosen the providers up a bit | 20:54 |
shardy | zaneb: there's a quota patch series up btw, although I stopped reviewing it because my comments were not being addressed | 20:54 |
SpamapS | can users define their own providers? | 20:54 |
zaneb | shardy: it doesn't allow you to do anything the user couldn't already do themselves by just creating lots of stacks | 20:54 |
shardy | SpamapS: yes! | 20:55 |
zaneb | SpamapS: a provider is just a template, so yes | 20:55 |
zaneb | that's the whole point ;) | 20:55 |
SpamapS | Ok so we still have to care | 20:55 |
shardy | zaneb: Yeah, that's what I meant, so does the nested limit actually achieve anything | 20:55 |
SpamapS | That doesn't really solve the issue then. | 20:55 |
zaneb | deeply nested providers are considerably more rare in the wild though | 20:55 |
SpamapS | So there are a few easy solutions. | 20:56 |
SpamapS | 1 is maintain a resource count per tenant. | 20:57 |
SpamapS | We can always determine that from the database + any template being parsed. | 20:57 |
SpamapS | Actually thats what I think may be necessary. | 20:57 |
*** asalkeld has joined #heat | 20:58 | |
SpamapS | Because as we move intelligence away from parsing the whole stack.. that is the most scalable option. | 20:58 |
shardy | SpamapS: yeah per tenant quota for #stacks and #resources is what we need | 20:58 |
*** jdob has quit IRC | 20:58 | |
SpamapS | if you have #resources, that is implicitly #stacks | 20:58 |
shardy | but the author of the quota patches wants per-user, which gets really hard with updates from multiple users | 20:58 |
shardy | SpamapS: stacks can have zero resources | 20:58 |
SpamapS | shardy: yeah, just use the same limit I mean. | 20:59 |
SpamapS | shardy: as a count, you do actually need it separate. curses. I get what you meant now. | 20:59 |
SpamapS | shardy: by user is fine too. | 20:59 |
SpamapS | just charge to stack owner | 21:00 |
shardy | SpamapS: 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 |
SpamapS | shardy: oh so they're saying if userA updates stack "foo", owned by userB, to add resources, then userA gets a separate limit? | 21:00 |
shardy | Currently it's always "A", even if they no longer exist | 21:01 |
shardy | SpamapS: yes, that's what I think is being proposed | 21:01 |
SpamapS | Ah so we need a chown | 21:01 |
shardy | I said lets get per-project quotas in and working first ;) | 21:01 |
shardy | heat stack-update --owner | 21:01 |
SpamapS | shardy: so we need to record user ID per resource instead of per stack. I think that's simple enough. | 21:01 |
shardy | or something | 21:01 |
shardy | SpamapS: but then you split quota for a stack over multiple users? | 21:02 |
SpamapS | but not a trivial amount of work and testing for those quotas. :-P | 21:02 |
shardy | Seems really messy to me | 21:02 |
SpamapS | One might also argue that we should NOT do the quotas .. just the impersonation.. and let physical resource services do the quoats. | 21:02 |
shardy | I'd rather you have one owner, and if you do a stack update, you use quota for the whole stack | 21:02 |
SpamapS | quoats: Oats that quiver. | 21:02 |
shardy | that is one owner per stack, which can change via an update | 21:03 |
SpamapS | shardy: What if we just punt on our own fine grained quotas? | 21:03 |
SpamapS | shardy: 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 |
SpamapS | my typnig si amzaign tdoya | 21:04 |
shardy | SpamapS: there's a whole series of patches up where you can ask those questions :D | 21:04 |
SpamapS | shardy: heat-specs? | 21:04 |
SpamapS | or just "I'm going to run off and do this and not ask anybody if it is a good idea." patches? | 21:04 |
shardy | https://review.openstack.org/#/q/status:open+project:openstack/heat+branch:master+topic:bp/add-quota-api-for-heat,n,z | 21:05 |
shardy | IIRC it was discussed pre-heat-specs | 21:05 |
shardy | To be fair the author has been regularly asking for review feedback in here | 21:06 |
SpamapS | speaking of specs... convergence needs approval soon so I can start work on it. | 21:06 |
SpamapS | shardy: mmk | 21:06 |
* shardy needs to revisit the convergence specs | 21:06 | |
SpamapS | I poked at some observer work but haven't wanted to go too far in case we found some gotchyas. | 21:07 |
stevebaker | good morning! | 21:08 |
stevebaker | shardy: can we have a quick chat about the keystone client plugin? | 21:15 |
kfox1111 | good morning. | 21:15 |
*** PhilK_ is now known as Philk | 21:15 | |
*** Philk is now known as PhilK | 21:15 | |
*** blomquisg is now known as blomquisg_away | 21:16 | |
lipinski | Is there a way to specify the replacement policy for a provider template? | 21:17 |
shardy | hey stevebaker sure, I'm just about still awake ;) | 21:17 |
*** sgordon has quit IRC | 21:17 | |
lipinski | For example, I have some metadata associated with a Server. This metadata can be modified without replacing the server/VM resource. | 21:18 |
lipinski | However, 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 server | 21:18 |
SpamapS | shardy: 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 |
stevebaker | shardy: 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 |
SpamapS | lipinski: it should not be rebuilding the whole provider | 21:19 |
SpamapS | lipinski: it should be doing a stack update on it. | 21:19 |
shardy | stevebaker: Yup sounds good, the main reason for heat_keystoneclient exisitng was that we didn't have what you're currently working on | 21:20 |
stevebaker | shardy: cool, thought as much | 21:20 |
lipinski | SpamapS: 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 |
randallburt | lipinski: 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 |
lipinski | Let me see if I can simplify my scenario to a very basic setup. | 21:21 |
randallburt | lipinski: points to something in StackResource looks like. | 21:21 |
stevebaker | zaneb: 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 IRC | 21:23 | |
*** dims has quit IRC | 21:23 | |
*** morganfainberg is now known as morganfainberg_Z | 21:24 | |
*** dims has joined #heat | 21:24 | |
lipinski | randallburt: 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 example | 21:25 |
randallburt | lipinski: 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 |
lipinski | randallburt: 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 |
lipinski | The simpliest example I can think of is setting the name of each | 21:29 |
randallburt | lipinski: yeah, the in-flight patch kinda lets you do that. | 21:30 |
lipinski | It allows for many more complicated examples when coupled with provider templates and parameters. | 21:30 |
lipinski | Can you point me to something with some info on that? | 21:30 |
randallburt | something like name: "server-%index%" | 21:30 |
randallburt | lipinski: https://review.openstack.org/#/c/88636/ | 21:30 |
lipinski | Thanks. 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-resourcegroups | 21:32 |
*** jprovazn_afk has quit IRC | 21:32 | |
lipinski | I actually had code to do it that was fairly simple. It just didn't handle when a resource parameter was a list. | 21:32 |
lipinski | I 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 |
randallburt | lipinski: cool. maybe convert that to a spec and maybe get more input. | 21:33 |
*** andersonvom has joined #heat | 21:34 | |
*** openstackgerrit has joined #heat | 21:34 | |
lipinski | I'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 |
lipinski | and my spelling stinks - you wouldn't want me ;) | 21:35 |
lipinski | How do I convert a blueprint to a spec? | 21:36 |
*** rpothier has quit IRC | 21:43 | |
SpamapS | lipinski: did you, by any chance, reference the metadata in the underlying server's properties? Some properties cause replacement. | 21:44 |
*** m_22 has quit IRC | 21:45 | |
randallburt | lipinski: you submit a review to the heat-specs repository | 21:45 |
*** DandyPandy has quit IRC | 21:45 | |
randallburt | SpamapS: 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 added | 21:45 |
lipinski | SpamapS: I just retested this scenario | 21:46 |
randallburt | lipinski: 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 |
SpamapS | lipinski: 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 IRC | 21:47 | |
lipinski | I 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 |
lipinski | I then change that metadata and the server within the provider resoruce gets recreated. | 21:47 |
lipinski | The metadata is not referenced anywhere (I'm not sure how to reference metadata within a template...) | 21:47 |
randallburt | lipinski: metadata as in the key value pairs associated with a server or the metadata associated with the Resource? | 21:48 |
lipinski | SpamapS: 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 |
lipinski | metadata associated with the provider resource. | 21:48 |
lipinski | it's a simple string. I set metadata: my_data: abc (where I define the provider resource) | 21:49 |
lipinski | I change that to xyz, and the Server within the provider gets rebuilt | 21:49 |
*** david-lyle has joined #heat | 21:50 | |
SpamapS | lipinski: rebuilt, or replaced? rebuild is a specific nova command that only is used to change images without detaching network ports or volumes. | 21:50 |
lipinski | sorry - replaced. | 21:50 |
lipinski | I see a new instance appear, then the old disappears. On the dashboard, for a very short time, I have two VMs. | 21:50 |
SpamapS | lipinski: thanks. So that sounds really weird. Check event-list .. I suspect the whole provider is being replaced. | 21:50 |
SpamapS | I wonder if nested stacks don't allow metadata updates | 21:52 |
lipinski | SpamapS: odd. The event-list shows the provider resource goes to UPDATE_IN_PROGRESS then CREATE_IN_PROGRESS then CREATE_COMPLETE | 21:53 |
lipinski | I don't see a delete, per se. | 21:53 |
SpamapS | weird indeed! | 21:53 |
lipinski | this is all a function of the heat-engine, right? | 21:54 |
SpamapS | lipinski: yes | 21:54 |
zaneb | randallburt, 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 |
lipinski | I'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 morganfainberg | 21:55 | |
*** dims has quit IRC | 21:58 | |
zaneb | stevebaker: I'd like to delete nova_utils... but I suspect we'd be safer with a deprecation period | 21:59 |
*** arbylee has quit IRC | 21:59 | |
SpamapS | hmmmmmm | 21:59 |
stevebaker | zaneb: the way it is strucutured, the deprecated nova_utils will be dead code. I suppose the unit tests can't be deleted either | 22:00 |
openstackgerrit | Kevin Fox proposed a change to openstack/heat: Add a OS::Nova::ServerGroup resource. https://review.openstack.org/100995 | 22:00 |
SpamapS | lipinski: any chance, on your engine, that you have "Resource xxxx does not implement metadata update" as a warning? | 22:00 |
zaneb | stevebaker: I'm ok with some dead code, as long as we still have tests | 22:00 |
*** achampion has joined #heat | 22:00 | |
SpamapS | lipinski: it kind of looks like only servers allow updating metadata. | 22:01 |
zaneb | SpamapS: as of last month, a metadata change should never trigger update replace | 22:01 |
stevebaker | the original nova_utils just had the user data building method. That should really move to a cloud_init_utils | 22:01 |
shardy | SpamapS: and WaitCOnditionHandle - I've been working on patches today to convert that to handle_signal | 22:01 |
SpamapS | zaneb: right I kind of remember that.. but I can't find where that is implemented | 22:01 |
shardy | IMO we should kill metadata_update and align everything with handle_signal, as they basically do the same thing | 22:02 |
shardy | actually I'll post what I have as I'm on PTO tomorrow | 22:02 |
SpamapS | lipinski: ok AFAICT, just changing the metadata should not affect anything | 22:04 |
zaneb | SpamapS: 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 |
SpamapS | lipinski: it should just update the resource. | 22:04 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Convert WaitConditionHandle to use handle_signal https://review.openstack.org/101351 | 22:04 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Update waitcondition API to use signal RPC interface https://review.openstack.org/101352 | 22:04 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Refactor waitcondition resources to allow easier subclassing https://review.openstack.org/101353 | 22:04 |
openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add an OS::Heat::WaitCondition resource https://review.openstack.org/101354 | 22:04 |
SpamapS | zaneb: thats what I was afraid of :) | 22:04 |
SpamapS | looking for a needle NOT in a haystack | 22:04 |
*** gondoi is now known as zz_gondoi | 22:05 | |
lipinski | SpamapS: you mean it could be because I'm not running the latest Icehouse heat engine? | 22:05 |
zaneb | looking for a non-needle in a haystack | 22:05 |
shardy | native handle resource patch still in-progress but that hopefully somewhat illustrates why we should kill metadata_update | 22:05 |
SpamapS | lipinski: s/Icehouse/juno. | 22:06 |
zaneb | lipinski: you'll need later than Icehouse even | 22:06 |
*** dims has joined #heat | 22:06 | |
SpamapS | lipinski: Icehouse is _SO_ last season. | 22:06 |
lipinski | ahhh - darn | 22:06 |
SpamapS | continuous deployment FTW :) | 22:06 |
lipinski | Well, our customers have placed a requirement on use that things work with Icehouse. | 22:06 |
lipinski | on us | 22:06 |
*** nati_ueno has quit IRC | 22:07 | |
*** nati_ueno has joined #heat | 22:07 | |
zaneb | lipinski, 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 properties | 22:08 |
SpamapS | lipinski: 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 |
randallburt | zaneb: not necessarily, no. | 22:09 |
lipinski | zaneb: That was just an example. The idea was more about passing data through to instances contained within a resource group | 22:09 |
randallburt | but there are other use cases where the ability to customize some aspect of the property makes sense too. | 22:09 |
lipinski | I could pre-name all my VMs that I expect to ever be created within a group, as another example. | 22:09 |
zaneb | lipinski: right, what I'm saying is that's not just an example. it's a specific special case that ought to be its own feature | 22:10 |
lipinski | zaneb: oh, okay. I thought you were saying there was already something that implemented that for avail zones. | 22:10 |
zaneb | yeah, there's not but there should be | 22:10 |
*** nati_ueno has quit IRC | 22:11 | |
lipinski | My 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 |
zaneb | we can talk about the generic customisation thing too. I know randallburt's a big fan ;) | 22:11 |
zaneb | but it is a different issue imo | 22:11 |
randallburt | zaneb: indeed :) | 22:12 |
*** nati_ueno has joined #heat | 22:12 | |
openstackgerrit | Steve Baker proposed a change to openstack/python-heatclient: Do not set up logging handler in http https://review.openstack.org/101358 | 22:13 |
openstackgerrit | Steve Baker proposed a change to openstack/python-heatclient: Improve --debug logging output https://review.openstack.org/101359 | 22:13 |
*** sabeen1 has quit IRC | 22:14 | |
shardy | Super trivial review if someone has a moment: | 22:14 |
shardy | https://review.openstack.org/#/c/99583/ | 22:14 |
zaneb | shardy: done | 22:15 |
shardy | mtreinish, stevebaker: I think https://review.openstack.org/#/c/90143/ is ready now, reviews would be awesome as all the check jobs are now working | 22:15 |
lipinski | is there a way to reference metadata within a resource? maybe soemthing like a get_metadata intrinsic funcion? | 22:16 |
zaneb | lipinski: fwiw OS::Heat::AutoscalingGroup scales provider resources | 22:16 |
lipinski | Trying to find a way to break my metadata off my provider resource and maybe into some pseudo resource that acts as a data-object store | 22:16 |
*** sabeen has joined #heat | 22:16 | |
zaneb | no, there isn't | 22:17 |
shardy | pass the data as parameters then reference them | 22:17 |
lipinski | zaneb: Thanks, but it seems to have the same shortcomings that prevent me from using ResourceGroups. | 22:17 |
zaneb | only within a provider template, where you can get the facade resource's metadata | 22:17 |
stevebaker | shardy: should the test be tagged slow if it uses cirros? | 22:18 |
shardy | stevebaker: well it takes a couple of minutes even on bare metal | 22:18 |
shardy | Or maybe like 90seconds | 22:18 |
shardy | I wasn't really sure what the definition of "slow" was tbh | 22:19 |
stevebaker | we have plenty of time in heat-slow, lets just leave it there | 22:19 |
stevebaker | shardy: my definition of slow is "boots fedora" | 22:19 |
shardy | stevebaker: Ok, it also touches lots of stuff, hence my thinking it's more of a kitchen-sink test appropriate for -slow | 22:20 |
stevebaker | fair enough | 22:20 |
shardy | happy to change the tag if needed though | 22:20 |
*** arbylee has joined #heat | 22:20 | |
*** alexpilotti has joined #heat | 22:20 | |
zaneb | lipinski: have you raised a bug for this metadata issue? | 22:21 |
lipinski | zaneb: no. Based on your discussion, I wasn't sure if it was one. | 22:22 |
zaneb | lipinski: I think it is, and I'd be prepared to fix it in Icehouse for you | 22:22 |
lipinski | You mean the fact that changing metadata for a Provider resource causes it to be recreated (or whatever it is doing...) | 22:22 |
zaneb | can't speak for reviewers ;) | 22:22 |
zaneb | yes, exactly | 22:22 |
lipinski | ok - I'll open one. | 22:22 |
*** noTHD has quit IRC | 22:23 | |
zaneb | thanks, ping me with the number | 22:23 |
*** ccrouch1 has quit IRC | 22:25 | |
*** andreaf_ has quit IRC | 22:26 | |
*** julienvey has quit IRC | 22:26 | |
*** noTHD has joined #heat | 22:27 | |
*** andersonvom has quit IRC | 22:27 | |
lipinski | zaneb: so this is different frmo https://bugs.launchpad.net/bugs/1262562 | 22:31 |
uvirtbot | Launchpad bug 1262562 in heat "AWS::AutoScaling::LaunchConfiguration metadata causes replacement on update" [Low,Confirmed] | 22:31 |
*** arbylee has quit IRC | 22:31 | |
*** blamar has quit IRC | 22:34 | |
*** kgriffs is now known as kgriffs|afk | 22:34 | |
*** openstackgerrit has quit IRC | 22:34 | |
*** 20WAAHXAJ has joined #heat | 22:35 | |
*** ccrouch has joined #heat | 22:39 | |
lipinski | zaneb: https://bugs.launchpad.net/heat/+bug/1332354 | 22:39 |
uvirtbot | Launchpad bug 1332354 in heat "provider metadata updates cause recreation" [Undecided,New] | 22:39 |
lipinski | Let 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 #heat | 22:40 | |
20WAAHXAJ | Kevin Fox proposed a change to openstack/heat: Add a OS::Nova::ServerGroup resource. https://review.openstack.org/100995 | 22:48 |
*** openstack has joined #heat | 22:51 | |
sjmc7 | what'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 problem | 22:54 |
sjmc7 | by long time i mean 20 minutes | 22:54 |
SpamapS | sjmc7: on my 4CPU laptop, 90s | 22:55 |
sjmc7 | trade? | 22:55 |
SpamapS | sjmc7: note that the first time you run it, you have to download "the world" from pypi | 22:55 |
sjmc7 | yeah, this is every time | 22:55 |
SpamapS | sjmc7: can you paste your "slowest tests" output? | 22:55 |
SpamapS | should print as the last thing before success | 22:56 |
sjmc7 | will do. this run failed from testing a patch earlier, will rerun off master | 22:56 |
*** alexpilotti has quit IRC | 23:00 | |
20WAAHXAJ | A change was merged to openstack/heat: parser.Stack add use_stored_context option https://review.openstack.org/99730 | 23:06 |
*** Demitar_ has quit IRC | 23:08 | |
20WAAHXAJ | A change was merged to openstack/heat: Convert service.py to use_stored_context https://review.openstack.org/99731 | 23:08 |
20WAAHXAJ | A change was merged to openstack/heat: Remove test_autoscaling _stub_validate https://review.openstack.org/100365 | 23:08 |
20WAAHXAJ | A change was merged to openstack/heat: test_autoscaling refactor suspend/resume stubbing https://review.openstack.org/100366 | 23:09 |
*** wpf is now known as wpf-norman | 23:11 | |
*** lipinski has quit IRC | 23:11 | |
sjmc7 | SpamapS - this one was only 878 seconds | 23:14 |
sjmc7 | http://pastebin.com/X8DHwFd2 | 23:14 |
SpamapS | sjmc7: ah see, already getting faster | 23:15 |
*** fbo_away has quit IRC | 23:15 | |
sjmc7 | great success | 23:15 |
SpamapS | sjmc7: yeah those are uncannily high | 23:15 |
SpamapS | sjmc7: I'm on a dual-core i7 (so 4x because of HT, and python HT's pretty well) .. | 23:16 |
SpamapS | sjmc7: but I'd expect on a single core w/o HT that it would just take 360s .. not 878 | 23:16 |
sjmc7 | ah... i'm running in a VM | 23:16 |
sjmc7 | so if it's reliant on concurrency that might not be too surprising | 23:17 |
sjmc7 | the autoscaling ones are the slow ones, so that makes perfect sense | 23:17 |
SpamapS | sjmc7: oh and the VM is going to suck for CPU | 23:17 |
*** randallburt has quit IRC | 23:18 | |
SpamapS | sjmc7: there's always tox -epypy ;) | 23:18 |
sjmc7 | :) | 23:18 |
*** sorantis has quit IRC | 23:20 | |
*** sabeen has quit IRC | 23:26 | |
*** 20WAAHXAJ has quit IRC | 23:29 | |
*** openstackgerrit has joined #heat | 23:30 | |
*** julienvey has joined #heat | 23:37 | |
zaneb | ugh, is there really no way to report a bug against a released version in Launchpad? | 23:38 |
*** julienvey has quit IRC | 23:41 | |
zaneb | ok, found it | 23:41 |
zaneb | SpamapS: 90s? I have quad core + HT and it's >2 minutes | 23:44 |
zaneb | sjmc7: but yeah, autoscaling tests are really slow. you want at least 2 cores ideally | 23:44 |
sjmc7 | ok. i'll give the vm some more grunt | 23:46 |
openstackgerrit | Kevin Fox proposed a change to openstack/heat: Add a OS::Nova::ServerGroup resource. https://review.openstack.org/100995 | 23:46 |
*** mestery has joined #heat | 23:46 | |
*** daneyon has joined #heat | 23:52 | |
*** daneyon has quit IRC | 23:52 | |
*** daneyon has joined #heat | 23:53 | |
*** nati_ueno has quit IRC | 23:54 | |
*** nati_ueno has joined #heat | 23:55 | |
*** nati_ueno has quit IRC | 23:55 | |
*** david-lyle has quit IRC | 23:56 | |
*** nati_ueno has joined #heat | 23:56 | |
*** dims has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!