| *** sabeen3 has quit IRC | 00:04 | |
| *** Qiming has joined #heat | 00:07 | |
| *** piyush has quit IRC | 00:11 | |
| *** tomek_adamczewsk has quit IRC | 00:13 | |
| *** tomek_adamczewsk has joined #heat | 00:13 | |
| *** tomek_adamczewsk has quit IRC | 00:21 | |
| *** ccrouch has joined #heat | 00:22 | |
| *** tomek_adamczewsk has joined #heat | 00:22 | |
| *** IlyaE has quit IRC | 00:23 | |
| *** tomek_adamczewsk has quit IRC | 00:25 | |
| *** tomek_adamczewsk has joined #heat | 00:25 | |
| *** Yanyan has joined #heat | 00:25 | |
| *** piyush has joined #heat | 00:26 | |
| *** gokrokve has quit IRC | 00:28 | |
| *** gokrokve has joined #heat | 00:29 | |
| *** IlyaE has joined #heat | 00:29 | |
| *** sarob has quit IRC | 00:30 | |
| *** tango has quit IRC | 00:30 | |
| *** gokrokve has quit IRC | 00:33 | |
| *** tomek_adamczewsk has quit IRC | 00:34 | |
| *** Yanyan has quit IRC | 00:34 | |
| *** tomek_adamczewsk has joined #heat | 00:35 | |
| *** bandarji has joined #heat | 00:42 | |
| *** tomek_adamczewsk has quit IRC | 00:46 | |
| *** Qiming has quit IRC | 00:48 | |
| *** asalkeld has joined #heat | 00:48 | |
| *** IlyaE has quit IRC | 00:50 | |
| *** mohits has quit IRC | 00:53 | |
| *** Yanyan has joined #heat | 00:59 | |
| *** liusheng has quit IRC | 01:02 | |
| *** DandyPandy has joined #heat | 01:06 | |
| *** DandyPandy has joined #heat | 01:07 | |
| *** shakamunyi has quit IRC | 01:08 | |
| *** sgordon has quit IRC | 01:16 | |
| *** tiantian has joined #heat | 01:23 | |
| tiantian | morning:) | 01:25 |
|---|---|---|
| *** asalkeld has quit IRC | 01:26 | |
| *** liusheng has joined #heat | 01:32 | |
| openstackgerrit | Haiyang DING proposed a change to openstack/heat: Change Volume metadata attribute to raw dict. https://review.openstack.org/95705 | 01:32 |
| *** asalkeld has joined #heat | 01:37 | |
| *** shakamunyi has joined #heat | 01:41 | |
| *** shakamunyi has quit IRC | 01:42 | |
| *** nosnos has joined #heat | 01:42 | |
| *** shakamunyi has joined #heat | 01:51 | |
| *** shakamunyi has quit IRC | 01:51 | |
| *** shakamunyi has joined #heat | 01:52 | |
| *** shakamunyi has quit IRC | 01:52 | |
| *** shakamunyi has joined #heat | 01:53 | |
| *** shakamunyi has quit IRC | 01:53 | |
| *** DandyPandy has quit IRC | 01:57 | |
| *** piyush has quit IRC | 01:59 | |
| *** DandyPandy has joined #heat | 01:59 | |
| *** Qiming has joined #heat | 02:01 | |
| *** shakamunyi has joined #heat | 02:02 | |
| *** killer_prince is now known as lazy_prince | 02:03 | |
| *** zhiyan_ is now known as zhiyan | 02:08 | |
| *** ericgoncz has joined #heat | 02:17 | |
| *** ericgoncz_ has joined #heat | 02:18 | |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Support loading resources from stevedore extensions https://review.openstack.org/103044 | 02:20 |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Change functions to stevedore extensions https://review.openstack.org/103484 | 02:20 |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Use setuptools to install contrib plugins https://review.openstack.org/105334 | 02:20 |
| *** ericgoncz has quit IRC | 02:21 | |
| *** ericgoncz_ is now known as ericgoncz | 02:21 | |
| *** lazy_prince is now known as killer_prince | 02:25 | |
| *** robsparker has quit IRC | 02:29 | |
| *** asalkeld has quit IRC | 02:32 | |
| *** asalkeld has joined #heat | 02:32 | |
| elynn | morning all :) | 02:33 |
| *** asalkeld has quit IRC | 02:36 | |
| *** ericgoncz has quit IRC | 02:37 | |
| openstackgerrit | Ethan Lynn proposed a change to openstack/heat-specs: Implement neutron custom constraints for subnet/port/router https://review.openstack.org/103714 | 02:42 |
| *** robsparker has joined #heat | 02:45 | |
| *** gokrokve has joined #heat | 02:45 | |
| *** liusheng has quit IRC | 02:51 | |
| *** tiantian has quit IRC | 02:51 | |
| *** zhangyang has quit IRC | 02:51 | |
| *** zhangyang has joined #heat | 02:52 | |
| *** ramishra has joined #heat | 02:52 | |
| *** tiantian has joined #heat | 02:53 | |
| *** liusheng has joined #heat | 02:53 | |
| *** robsparker has quit IRC | 02:56 | |
| ramishra | stevebaker: Hi | 02:57 |
| stevebaker | ramishra: | 02:57 |
| stevebaker | ramishra: higgins | 02:57 |
| stevebaker | arg, how many times do I need to do that? | 02:57 |
| ramishra | stevebaker: lol..do you have 2 mins for a question on software config and the new spec Tom has been proposing? | 02:58 |
| stevebaker | ramishra: yes | 02:58 |
| lifeless | stevebaker: BOSON | 02:58 |
| stevebaker | snigger | 02:59 |
| ramishra | stevebaker: First one.. Is there a way we can order softwareconfigs as they and their application to a resource? | 02:59 |
| stevebaker | ramishra: currently the execution order is determined by the sorted names of the deployment resources | 03:00 |
| ramishra | stevebaker: that would help..Can we have something to explicitly specify that? | 03:02 |
| stevebaker | ramishra: you mean document it? http://docs.openstack.org/developer/heat/template_guide/openstack.html#OS::Heat::SoftwareDeployment -> name | 03:02 |
| Qiming | :( | 03:02 |
| Qiming | InvalidTemplateVersion: The template version is invalid: Template version was not provided | 03:03 |
| *** gokrokve has quit IRC | 03:05 | |
| Qiming | mostly related to '/opt/stack/heat/contrib/rackspace/rackspace/tests/test_xxx.py' | 03:06 |
| stevebaker | Qiming: the root cause may be something to do with constraints plugins. Try pip install -e ./ | 03:07 |
| ramishra | stevebaker: I didn't mean the documentation.. but wondering would it be a good idea to explicitly give something like order_id or set some dependancy. I assume this name based ordering would work with provider templates too. | 03:07 |
| Qiming | stevebaker: thanks, will try | 03:07 |
| *** robsparker has joined #heat | 03:07 | |
| ramishra | stevebaker: I was not aware of name based ordering. I'll give it a go | 03:08 |
| stevebaker | ramishra: resource dependencies is orthogonal to execution order, so we can't do that. I'd like name ordering to *be* the explicit mechanism, but we need to figure out how to do that with SoftwareComponent. The different action configs will need different orders but we'll only have one deployment resource | 03:09 |
| ramishra | stevebaker: the other question I've is how useful would be action aware 'SoftwareComponent' that is proposed wrt config management tools. What would be the granularity of SoftwareComponet? | 03:11 |
| *** asalkeld has joined #heat | 03:12 | |
| stevebaker | ramishra: good question, ask Thomas ;) I would think a component would define a microservice or an application | 03:12 |
| ramishra | stevebaker: IMO, a rollback script for every config explicitly specified would be some sort of starting point. Rollback is a nightmare for many tools | 03:13 |
| stevebaker | ramishra: you could do that, but that would be somewhat missing the point of cloud | 03:14 |
| *** IlyaE has joined #heat | 03:15 | |
| *** flyemsafe has joined #heat | 03:16 | |
| ramishra | stevebaker: for example, If a script has to 'adduser' as CREATE action and the user exists. How the DELETE action would know not to remove it? This becomes much more complicated with other things like package management. | 03:16 |
| ramishra | stevebaker: sure..I'll speak to Thomas on these. thanks.. | 03:17 |
| stevebaker | ramishra: yes, you get to the point quickly where it becomes complicated to the point of impossible. You would be better off bringing up a fresh server and configuring that. Or building a custom golden image then not configuring anything ;) | 03:18 |
| *** gokrokve has joined #heat | 03:18 | |
| *** chandan_kumar has joined #heat | 03:18 | |
| tiantian | <stevebaker>: do you have time for approve my bps in lauchpad which the specs have been merged, https://blueprints.launchpad.net/heat/+spec/implement-ec2instance-bdm https://blueprints.launchpad.net/heat/+spec/events-pagination thanks very much | 03:20 |
| stevebaker | tiantian: remember you don't need to wait for approval to start developing | 03:20 |
| *** nosnos has quit IRC | 03:22 | |
| tiantian | <stevebaker> yes, you are right:) the code patches are in reviewing... I mean hope you to approve in lauchpad if you have time to do it:) | 03:22 |
| stevebaker | tiantian: ah, that is zaneb's job ;) | 03:22 |
| tiantian | <stevebaker> ok, got it, thanks | 03:23 |
| *** dims has quit IRC | 03:24 | |
| *** gokrokve_ has joined #heat | 03:25 | |
| *** gokrokve has quit IRC | 03:28 | |
| Qiming | stevebaker, tried 'pip install -e ./', now seeing failures like this: ClientException: (500, 'ClientException: The server has either erred or is incapable of performing the requested operation.') | 03:30 |
| Qiming | stevebaker, reported from heat.tests.test_nova_utils.NovaUtilsRefreshServerTests.test_500_error | 03:30 |
| stevebaker | Qiming: how do you run your tests? | 03:31 |
| Qiming | ./run_tests.sh | 03:31 |
| stevebaker | Qiming: It may be time for you to learn the ways of the tox ;) | 03:31 |
| Qiming | heat/engine/resources/nova_utils.py line 54 | 03:32 |
| stevebaker | Qiming: I've just had to stop using run_tests.sh due to the recent stevedore changes - its only practical to run our tests in a properly setup virtualenv now, which tox does | 03:32 |
| *** gokrokve has joined #heat | 03:36 | |
| *** chandan_kumar has quit IRC | 03:36 | |
| *** chandan_kumar has joined #heat | 03:36 | |
| *** saju_m has joined #heat | 03:37 | |
| *** gokrokve_ has quit IRC | 03:39 | |
| *** flyemsafe has quit IRC | 03:42 | |
| Qiming | stevebaker, okay. tryied this: pip install -e ./; rm -fr * .tox; tox -epy26 | 03:46 |
| Qiming | now I'm getting 'py26: commands succeeded congratulations :)' | 03:46 |
| stevebaker | Qiming: tox will do the pip install for you | 03:46 |
| Qiming | so, just clear the old cache in .tox, then 'tox -epy26' would be okay? | 03:47 |
| elynn | Or you can just use "tox --recreate -e py26" | 03:49 |
| *** higgins has quit IRC | 03:55 | |
| *** higgins has joined #heat | 03:55 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Updated from global requirements https://review.openstack.org/105166 | 03:59 |
| openstackgerrit | Yanyan Hu proposed a change to openstack/heat: Default to keystone V3 for software deployment https://review.openstack.org/104754 | 04:04 |
| *** gokrokve_ has joined #heat | 04:07 | |
| *** alexheneveld has joined #heat | 04:09 | |
| openstackgerrit | Yanyan Hu proposed a change to openstack/heat: Default to keystone V3 for software deployment https://review.openstack.org/104754 | 04:10 |
| *** gokrokve has quit IRC | 04:11 | |
| *** killer_prince is now known as lazy_prince | 04:12 | |
| openstackgerrit | huangtianhua proposed a change to openstack/heat: Implement BlockDeviceMappings for AWS::EC2::Instance https://review.openstack.org/103725 | 04:13 |
| *** gokrokve_ has quit IRC | 04:14 | |
| *** gokrokve has joined #heat | 04:15 | |
| *** asalkeld has quit IRC | 04:16 | |
| *** gokrokve has quit IRC | 04:20 | |
| *** IlyaE has quit IRC | 04:21 | |
| *** ajc_ has joined #heat | 04:25 | |
| *** nosnos has joined #heat | 04:25 | |
| *** gokrokve has joined #heat | 04:30 | |
| *** gokrokve_ has joined #heat | 04:37 | |
| *** rushiagr has joined #heat | 04:37 | |
| *** gokrokve has quit IRC | 04:39 | |
| *** DandyPandy has quit IRC | 04:40 | |
| *** rakesh_hs has joined #heat | 04:45 | |
| *** samstav has quit IRC | 04:49 | |
| *** samstav has joined #heat | 04:49 | |
| *** gokrokve has joined #heat | 04:50 | |
| *** saju_m has quit IRC | 04:50 | |
| *** gokrokve_ has quit IRC | 04:52 | |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Use setuptools to install contrib plugins https://review.openstack.org/103441 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Only support client plugins https://review.openstack.org/98688 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Glance, swift use ignore_not_found() https://review.openstack.org/100424 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement keystone client plugin https://review.openstack.org/97985 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement heat client plugin https://review.openstack.org/97984 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Client plugin exception handling methods https://review.openstack.org/100421 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Move network constraint to neutron plugin https://review.openstack.org/100420 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Neutron use ignore_not_found() https://review.openstack.org/100423 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Nova use ignore_not_found() https://review.openstack.org/100422 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Move nova_utils functions to nova client plugin https://review.openstack.org/101782 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Store clients in context https://review.openstack.org/98935 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Move glance_utils functions to glance client plugin https://review.openstack.org/100419 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement barbican client plugin https://review.openstack.org/98684 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Use config region_name_for_services for rackspace clients https://review.openstack.org/98686 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Port rackspace clients to client plugins https://review.openstack.org/98687 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Remove function get_software_config https://review.openstack.org/101396 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Cinder, trove, ceilo use ignore_not_found() https://review.openstack.org/101397 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement marconi client plugin https://review.openstack.org/98683 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement nova client plugin https://review.openstack.org/97977 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement swift client plugin https://review.openstack.org/97978 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement glance client plugin https://review.openstack.org/97979 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement neutron client plugin https://review.openstack.org/97980 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement cinder client plugin https://review.openstack.org/97981 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement trove client plugin https://review.openstack.org/97982 | 04:53 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement ceilometer client plugin https://review.openstack.org/97983 | 04:53 |
| *** lazy_prince has quit IRC | 04:53 | |
| *** samstav has quit IRC | 04:54 | |
| *** gokrokve_ has joined #heat | 04:56 | |
| *** gokrokve has quit IRC | 04:57 | |
| *** dims has joined #heat | 04:58 | |
| *** chandan_kumar has quit IRC | 04:58 | |
| *** lazy_prince has joined #heat | 05:01 | |
| *** dims has quit IRC | 05:02 | |
| *** tspatzier has joined #heat | 05:02 | |
| *** chandan_kumar has joined #heat | 05:04 | |
| *** bandarji has quit IRC | 05:04 | |
| *** chandan_kumar has quit IRC | 05:04 | |
| *** gokrokve has joined #heat | 05:04 | |
| *** chandankumar has joined #heat | 05:04 | |
| *** gokrokve has quit IRC | 05:05 | |
| *** gokrokve_ has quit IRC | 05:05 | |
| *** tango has joined #heat | 05:08 | |
| *** ckmvishnu has joined #heat | 05:08 | |
| *** shakamunyi has quit IRC | 05:09 | |
| *** Yanyan has quit IRC | 05:09 | |
| skraynev | Morning :) | 05:14 |
| *** nkhare has joined #heat | 05:15 | |
| *** bvivek has joined #heat | 05:19 | |
| *** killer_prince has joined #heat | 05:19 | |
| *** tiantian has quit IRC | 05:20 | |
| *** gokrokve has joined #heat | 05:25 | |
| *** Dev_Jin has joined #heat | 05:25 | |
| *** lazy_prince has quit IRC | 05:26 | |
| *** killer_prince is now known as lazy_prince | 05:26 | |
| *** gokrokve_ has joined #heat | 05:27 | |
| *** gokrokve has quit IRC | 05:30 | |
| *** arbylee has quit IRC | 05:35 | |
| *** DandyPandy has joined #heat | 05:37 | |
| *** harlowja is now known as harlowja_away | 05:42 | |
| skraynev | elynn: are you here? | 05:43 |
| *** gokrokve has joined #heat | 05:46 | |
| *** unmeshg has joined #heat | 05:47 | |
| *** gokrokve_ has quit IRC | 05:50 | |
| elynn | Hi skraynev | 05:52 |
| *** gokrokve_ has joined #heat | 05:53 | |
| *** ramishra has quit IRC | 05:55 | |
| *** gokrokve has quit IRC | 05:56 | |
| *** Ng_ has joined #heat | 05:56 | |
| skraynev | elynn: about https://review.openstack.org/#/c/103714, did you see html output ? | 05:56 |
| *** IlyaE has joined #heat | 05:57 | |
| skraynev | elynn: I just worry that result still is not good for reading in section http://docs-draft.openstack.org/14/103714/2/check/gate-heat-specs-docs/4943609/doc/build/html/specs/neutron-custom-constraint.html#proposed-change | 05:58 |
| *** dims has joined #heat | 05:58 | |
| elynn | Oh... | 05:59 |
| skraynev | elynn: will be really cool to have each number (constraint) on separate line ;) | 05:59 |
| elynn | I'm not very familiar with rst to html convert. | 06:00 |
| elynn | Will modify my patch in next patchset | 06:00 |
| elynn | BTW, where can I find the link you mention above? | 06:01 |
| skraynev | elynn: np, just suggestion. I want to add +2, but can not do it in current situation | 06:02 |
| elynn | :) | 06:02 |
| skraynev | Last comment from Jenkins | 06:02 |
| skraynev | F.e. gate-heat-specs-docs SUCCESS in 1m 09s | 06:02 |
| *** dims has quit IRC | 06:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/heat: Imported Translations from Transifex https://review.openstack.org/104975 | 06:03 |
| elynn | Oh... I got it! | 06:03 |
| skraynev | use link in job name | 06:03 |
| skraynev | ;) | 06:03 |
| elynn | :D | 06:04 |
| skraynev | yeah it's very useful. | 06:04 |
| *** saju_m has joined #heat | 06:05 | |
| *** ramishra has joined #heat | 06:05 | |
| *** sgordon has joined #heat | 06:10 | |
| *** gokrokve_ has quit IRC | 06:11 | |
| *** gokrokve has joined #heat | 06:12 | |
| *** piyush has joined #heat | 06:12 | |
| *** sarob has joined #heat | 06:15 | |
| *** gokrokve has quit IRC | 06:16 | |
| *** asalkeld has joined #heat | 06:17 | |
| *** cmyster has joined #heat | 06:18 | |
| *** nkhare has quit IRC | 06:18 | |
| cmyster | morning | 06:19 |
| *** alexheneveld has quit IRC | 06:20 | |
| *** alexheneveld has joined #heat | 06:21 | |
| *** alexheneveld has quit IRC | 06:22 | |
| openstackgerrit | Ethan Lynn proposed a change to openstack/heat-specs: Implement neutron custom constraints for subnet/port/router https://review.openstack.org/103714 | 06:24 |
| *** alexheneveld has joined #heat | 06:24 | |
| *** tiantian has joined #heat | 06:25 | |
| *** zigo has quit IRC | 06:27 | |
| *** alexheneveld has quit IRC | 06:29 | |
| *** saju_m has quit IRC | 06:30 | |
| *** ishant has joined #heat | 06:31 | |
| *** zigo has joined #heat | 06:31 | |
| *** lazy_prince has quit IRC | 06:32 | |
| therve | Good morning! | 06:32 |
| *** nkhare has joined #heat | 06:35 | |
| *** jstrachan has joined #heat | 06:42 | |
| *** sarob has quit IRC | 06:49 | |
| *** jprovazn has joined #heat | 06:49 | |
| *** lazy_prince has joined #heat | 06:50 | |
| cmyster | morning therve | 06:52 |
| *** sy01 has joined #heat | 06:54 | |
| skraynev | stevebaker, could you please fix imports in unittests for swiftclient too ? | 06:55 |
| sy01 | Hello, I am testing nested stack, I am able to created a stack when the "TemplateURL" is a web url but stack creation is failing in case of local refrence of the template, | 06:55 |
| *** gokrokve has joined #heat | 06:56 | |
| sy01 | Resource Create Failed: Valueerror: Could Not Fetch Remote Template 'File:///Tmp/Test.Yaml': Invalid Url Scheme File | 06:56 |
| *** sgordon has quit IRC | 06:56 | |
| sy01 | It does not look like invalid url scheme | 06:56 |
| *** tomek_adamczewsk has joined #heat | 06:59 | |
| *** dims_ has joined #heat | 06:59 | |
| *** piyush has quit IRC | 07:00 | |
| *** gokrokve has quit IRC | 07:01 | |
| *** jcoufal has joined #heat | 07:01 | |
| *** IlyaE has quit IRC | 07:03 | |
| *** cmyster has quit IRC | 07:03 | |
| *** dims_ has quit IRC | 07:04 | |
| therve | sy01, You can't specify file:// URLs in TemplateURL | 07:05 |
| sy01 | therve: Ohh, then how should i reference a local template | 07:06 |
| sy01 | therve: is it not possible right now ? | 07:06 |
| therve | sy01, Not with AWS::CloudFormation::Stack | 07:06 |
| therve | It depends what you mean by local too | 07:07 |
| sy01 | therve: Any work around, so that i can use nested stack feature with templates present in my local folder | 07:07 |
| therve | sy01, You mean local to your client machine then? | 07:08 |
| *** cmyster has joined #heat | 07:08 | |
| *** cmyster has joined #heat | 07:08 | |
| sy01 | therve: Yes, on the machine where python-heat-client is running | 07:08 |
| therve | sy01, The support mechanism in icehouse is to use type: file.yaml | 07:08 |
| therve | In your template | 07:09 |
| sy01 | i tried that, but it does n't work. | 07:09 |
| *** tomek_adamczewsk has quit IRC | 07:09 | |
| therve | Sure it does | 07:10 |
| sy01 | therve: "ERROR: Could not fetch remote template 'test.yaml': Invalid URL scheme" | 07:11 |
| sy01 | therve: type: test.yaml | 07:11 |
| therve | sy01, What's the verson the heatclient you're using? | 07:11 |
| *** tomek_adamczewsk has joined #heat | 07:12 | |
| sy01 | therve: python-heatclient 0.2.8-0ubuntu1 | 07:12 |
| *** pasquier-s has joined #heat | 07:13 | |
| therve | That may be a bit old | 07:13 |
| *** tspatzier has quit IRC | 07:14 | |
| *** saju_m has joined #heat | 07:14 | |
| *** tango has quit IRC | 07:16 | |
| *** iyozhikov is now known as IgorYozhikov | 07:20 | |
| openstackgerrit | A change was merged to openstack/heat: Implement nova client plugin https://review.openstack.org/97977 | 07:20 |
| *** shakamunyi has joined #heat | 07:22 | |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Moving network_id_from_subnet_id to neutron_utils https://review.openstack.org/105382 | 07:22 |
| *** nkhare has quit IRC | 07:25 | |
| *** IlyaE has joined #heat | 07:28 | |
| *** pasquier-s has quit IRC | 07:30 | |
| openstackgerrit | Thomas Herve proposed a change to openstack/heat: Fix serialization of InvalidSchemaError https://review.openstack.org/104189 | 07:32 |
| *** nkhare has joined #heat | 07:33 | |
| *** IlyaE has quit IRC | 07:33 | |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Remove unused parameter name in validate_value https://review.openstack.org/97765 | 07:35 |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Using default FnGetRefId behavior for LaunchConfig https://review.openstack.org/97543 | 07:35 |
| *** nati_ueno has quit IRC | 07:37 | |
| *** tomek_adamczewsk has quit IRC | 07:39 | |
| *** shakamunyi has quit IRC | 07:46 | |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Adding own child_template for AutoScalingGroup https://review.openstack.org/95142 | 07:47 |
| sy01 | therve: Ubuntu package for 0.2.10 is not there and while installing from ubuntu packages | 07:49 |
| sy01 | therve: if i try to do install it via pip, it does install the 0.2.10 | 07:50 |
| sy01 | therve: So i am stuck with 0.2.8 | 07:50 |
| openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Implement PrivateIpAddress property for EC2::Instance https://review.openstack.org/98580 | 07:50 |
| therve | sy01, Why can't you use the pip version? | 07:50 |
| *** akuznetsov has quit IRC | 07:54 | |
| *** akuznetsov has joined #heat | 07:55 | |
| *** gokrokve has joined #heat | 07:56 | |
| *** jistr has joined #heat | 07:56 | |
| *** sergmelikyan has quit IRC | 07:56 | |
| *** rushiagr_ has joined #heat | 07:58 | |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Support loading resources from stevedore extensions https://review.openstack.org/103044 | 07:58 |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Use setuptools to install contrib plugins https://review.openstack.org/105334 | 07:58 |
| openstackgerrit | Angus Salkeld proposed a change to openstack/heat: Change functions to stevedore extensions https://review.openstack.org/103484 | 07:58 |
| *** asalkeld has quit IRC | 07:59 | |
| *** pas-ha has joined #heat | 07:59 | |
| *** pasquier-s has joined #heat | 08:00 | |
| *** rushiagr has quit IRC | 08:00 | |
| *** gokrokve has quit IRC | 08:00 | |
| *** rushiagr_ has quit IRC | 08:02 | |
| *** killer_prince has joined #heat | 08:04 | |
| *** ryansb has quit IRC | 08:06 | |
| *** sjmc7 has quit IRC | 08:08 | |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Providing reference on resource during preview https://review.openstack.org/97544 | 08:09 |
| *** IgorYozhikov is now known as iyozhikov | 08:09 | |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Adding validation algorithm for get attr functions https://review.openstack.org/82488 | 08:09 |
| *** sjmc7 has joined #heat | 08:10 | |
| openstackgerrit | huangtianhua proposed a change to openstack/python-heatclient: Implement events pagination, sorting and filtering https://review.openstack.org/105110 | 08:10 |
| *** TonyBurn_ has joined #heat | 08:13 | |
| *** pasquier-s has quit IRC | 08:13 | |
| *** pasquier-s has joined #heat | 08:14 | |
| shardy | morning all | 08:14 |
| cmyster | morning | 08:15 |
| *** derekh_ has joined #heat | 08:16 | |
| *** pasquier-s has quit IRC | 08:16 | |
| *** pasquier-s has joined #heat | 08:17 | |
| *** tspatzier has joined #heat | 08:20 | |
| *** sy01 has quit IRC | 08:20 | |
| *** pasquier-s has quit IRC | 08:21 | |
| *** pasquier-s has joined #heat | 08:22 | |
| *** pasquier-s has quit IRC | 08:25 | |
| *** pasquier-s has joined #heat | 08:26 | |
| *** lazy_prince has quit IRC | 08:26 | |
| *** killer_prince is now known as lazy_prince | 08:26 | |
| openstackgerrit | A change was merged to openstack/heat-specs: Implement neutron custom constraints for subnet/port/router https://review.openstack.org/103714 | 08:26 |
| *** julienvey has joined #heat | 08:26 | |
| shardy | therve: hey, around? | 08:27 |
| therve | shardy, Hi! | 08:27 |
| *** sergmelikyan has joined #heat | 08:27 | |
| shardy | therve: hi! Just wanted a quick chat re https://review.openstack.org/#/c/104189/ | 08:27 |
| therve | shardy, Yeah thanks. What do you think? | 08:28 |
| *** dims has joined #heat | 08:28 | |
| shardy | So the issue is, folks often add a new exception, which if it can get raised and passed back via RPC, results in a CFN API 500 error | 08:28 |
| shardy | So I'm confused by your comment about it being different/big patch etc - it's one line more? | 08:28 |
| shardy | Or are you saying the exception can never get raised via CFN templates? | 08:28 |
| shardy | I'm just trying to avoid yet-another "cfn API raises 500" bug later ;) | 08:29 |
| therve | No just that it's a different issue than the one reported | 08:29 |
| therve | IE almost completely orthogonal to the fix I'm making | 08:29 |
| shardy | but your patch adds a new exception type? | 08:30 |
| therve | Not really | 08:30 |
| *** radez_g0n3 is now known as radez | 08:30 | |
| shardy | The CFN API maps every exception not explicitly mapped to 500 | 08:30 |
| therve | The exception was already here, I just moved it | 08:30 |
| pas-ha | morning all | 08:30 |
| shardy | therve: hmm, oh yes, I see | 08:31 |
| shardy | So the CFN API is probably broken already then :( | 08:31 |
| therve | Yes :) | 08:31 |
| * shardy sighs | 08:31 | |
| shardy | therve: Ok, fair enough, thanks for the clarification | 08:32 |
| therve | shardy, That reminds me, WaitCondition errors inherits from Error so are likely to have the same problem | 08:32 |
| therve | shardy, I don't know how to test it though | 08:32 |
| shardy | therve: WaitCondition errors aren't ever propagated via RPC, so it's not the same problem | 08:32 |
| *** dims has quit IRC | 08:33 | |
| shardy | the issue is exceptions in the path of synchronous calls | 08:33 |
| therve | OK I was wondering | 08:33 |
| shardy | Stuff caught while IN_PROGRESS is reported via the stack status not the API call | 08:33 |
| shardy | it's not always obvious which is which, but I think most validation exceptions fall into the API response category | 08:34 |
| openstackgerrit | huangtianhua proposed a change to openstack/heat: Implement BlockDeviceMappings for AWS::EC2::Instance https://review.openstack.org/103725 | 08:35 |
| openstackgerrit | huangtianhua proposed a change to openstack/heat: Implement events pagination, sorting and filtering https://review.openstack.org/104439 | 08:35 |
| *** pasquier-s_ has joined #heat | 08:37 | |
| *** pasquier-s has quit IRC | 08:38 | |
| *** andreaf has quit IRC | 08:40 | |
| *** andreaf_ is now known as andreaf | 08:40 | |
| *** andreaf_ has joined #heat | 08:40 | |
| openstackgerrit | Ethan Lynn proposed a change to openstack/heat: Implement PrivateIpAddress property for EC2::Instance https://review.openstack.org/98580 | 08:40 |
| *** pasquier-s__ has joined #heat | 08:41 | |
| *** ramishra has quit IRC | 08:41 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::NodeGroupTemplate resource https://review.openstack.org/100288 | 08:41 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement sahara client plugin https://review.openstack.org/100588 | 08:41 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::Cluster resource https://review.openstack.org/72336 | 08:41 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement OS::Sahara::ClusterTemplate resource https://review.openstack.org/101509 | 08:41 |
| *** pasquier-s_ has quit IRC | 08:43 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement CFN CancelUpdateStack action https://review.openstack.org/102852 | 08:48 |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Implement cancelling of stack update https://review.openstack.org/102851 | 08:48 |
| *** sarob has joined #heat | 08:50 | |
| *** pasquier-s has joined #heat | 08:50 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed a change to openstack/heat: Fix test for stack delete on same active engine https://review.openstack.org/104525 | 08:51 |
| *** pasquier-s__ has quit IRC | 08:51 | |
| *** rushiagr_ has joined #heat | 08:54 | |
| *** sarob has quit IRC | 08:54 | |
| *** gokrokve has joined #heat | 08:56 | |
| *** gokrokve has quit IRC | 09:00 | |
| tiantian | stevedore.extension DistributionNotFound: python-neutronclient>=2.3.5,<3 ?? | 09:00 |
| tiantian | the python-neutronclient is up-to-date | 09:01 |
| *** Ng has quit IRC | 09:02 | |
| *** Ng_ is now known as Ng | 09:02 | |
| tiantian | someone who can help? | 09:02 |
| shardy | tiantian: If you're running devstack, try doing python setup.py install | 09:03 |
| shardy | I had issues yesterday until I did that | 09:03 |
| shardy | (thanks again to therve and skraynev for the help ;) | 09:03 |
| tiantian | shardy: I have run the command :pip install -e ./ | 09:03 |
| tiantian | or I missing something | 09:04 |
| therve | There are many ways this can be raised | 09:04 |
| therve | Do you have python-neutronclient system package installed? | 09:05 |
| *** bvandenh has joined #heat | 09:06 | |
| gilliard | Hi - w.r.t bug 1217194, I can put up a patch using fnmatch. Any objections? | 09:06 |
| uvirtbot | Launchpad bug 1217194 in heat "AllowedPattern is using python re module with user input" [High,Triaged] https://launchpad.net/bugs/1217194 | 09:06 |
| gilliard | I don't see any alternative regex library that fits our needs, so I think greghaynes suggestion is OK | 09:07 |
| tiantian | <therve>: you mean I have to delete the system package in /usr/local/lib/python2.7/dist-packages? | 09:07 |
| therve | tiantian, Not in usr/local no | 09:08 |
| therve | Maybe update it though | 09:08 |
| therve | gilliard, That's worth trying yes | 09:10 |
| tiantian | <therve>:pip install python-neutronclient -U and 'Requirement already up-to-date: python-neutronclient in /usr/local/lib/python2.7/dist-packages' returns, how can I do? | 09:11 |
| therve | tiantian, I don't know | 09:12 |
| skraynev | tiantian: when does it happen? | 09:13 |
| *** jstrachan has quit IRC | 09:13 | |
| skraynev | tiantian: for tests or you try to start heat-engine ? | 09:13 |
| tiantian | <therve>: start the engine thread using ' python /opt/stack/heat/bin/heat-engine --config-file=/etc/heat/heat.conf' | 09:13 |
| tiantian | <skraynev>: hi | 09:14 |
| tiantian | <skraynev>: and when create a stack, an error raised:'ERROR: The template version is invalid: Template version was not provided' | 09:15 |
| skraynev | tiantian: hm. strange I use last master and have not seen such problem... | 09:15 |
| skraynev | tiantian: oh :) I know this one | 09:15 |
| shardy | tiantian: I had that issue yesterday, setup.py install solved it | 09:16 |
| skraynev | shardy is right, also ... | 09:16 |
| tiantian | <skraynev>: I use last master too | 09:17 |
| skraynev | tiantian: I remember one more thing. Possibly you have some *.pyc files | 09:17 |
| tiantian | wait... I will delete them | 09:17 |
| skraynev | I periodically clean these files from heat repo (find . -name *.pyc -delete) | 09:18 |
| tiantian | :( doesn't work | 09:18 |
| skraynev | also it may be the cause of error about import for neutron-client | 09:18 |
| skraynev | I did it for /usr/lib/python2.7/dist-packages/ | 09:19 |
| *** cmyster has quit IRC | 09:19 | |
| skraynev | so python setup.py install and cleaning *.pyc files wfm | 09:19 |
| *** ajc_ has quit IRC | 09:20 | |
| tiantian | <skraynev>: you mean to delete the *.pyc in /usr/lib/python2.7/dist-packages/?? | 09:20 |
| therve | You don't need to do that. Don't randomly delete stuff, try to understand what's going on | 09:21 |
| skraynev | tiantian: I know, that it's not the best idea, but yes, I did it, when had problem with six f.e. | 09:21 |
| skraynev | p.s. did it for six | 09:21 |
| skraynev | listen therve, I am risky man :( | 09:22 |
| tiantian | :) | 09:24 |
| *** rushiagr_ has quit IRC | 09:24 | |
| *** rushiagr_ has joined #heat | 09:25 | |
| tiantian | thanks all:) I will try to understand what's going on, hope there is a result before back home | 09:26 |
| *** dims has joined #heat | 09:29 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient add get_user_token https://review.openstack.org/102885 | 09:29 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: stack user add _user_token https://review.openstack.org/102887 | 09:29 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: clients make heat_url public https://review.openstack.org/102886 | 09:29 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add an OS::Heat::WaitCondition resource https://review.openstack.org/101354 | 09:29 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add native WaitConditionHandle resource https://review.openstack.org/102888 | 09:29 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Refactor waitcondition resources to allow easier subclassing https://review.openstack.org/105402 | 09:29 |
| *** lazy_prince has quit IRC | 09:30 | |
| skraynev | tiantian: my guess for your problem is some old files :) | 09:31 |
| openstackgerrit | huangtianhua proposed a change to openstack/python-heatclient: Implement events pagination, sorting and filtering https://review.openstack.org/105110 | 09:31 |
| tiantian | <skraynev>: so try to git clone heat project? | 09:32 |
| *** dims has quit IRC | 09:33 | |
| skraynev | skraynev: this good solution if these files are in heat repo. If problems are in dependencies (other packages), it will not solve the problem. | 09:34 |
| shardy | derekh_: Hey, around? | 09:34 |
| derekh_ | shadower: yup | 09:34 |
| derekh_ | opps shardy | 09:34 |
| shardy | derekh_: Hi! Have a minute to discuss bug #1337772? | 09:35 |
| uvirtbot | Launchpad bug 1337772 in heat "Heat: cfn API not returning Metadata. tripleo-ci: Failing to connect to overcloud controller" [High,In progress] https://launchpad.net/bugs/1337772 | 09:35 |
| shardy | I'm having issues reproducing and could use some tripleo knowledge :) | 09:35 |
| shardy | I re-posted the patch series which broke things, but with the API change removed | 09:36 |
| *** lazy_prince has joined #heat | 09:36 | |
| *** pasquier-s_ has joined #heat | 09:36 | |
| shardy | This is the first patch which got reverted, and I'm not sure where to look in the check-experimental job logs to see if the problem is happening: | 09:36 |
| shardy | https://review.openstack.org/#/c/105219/ | 09:36 |
| derekh_ | shardy: ok, looking | 09:36 |
| shardy | I've dug a bit, and AFAICS it's not, but I'm not sure why check-tripleo-ironic-seed-precise failed | 09:37 |
| *** pasquier-s has quit IRC | 09:37 | |
| shardy | derekh_: Since I can't reproduce locally, my current assumption is that https://review.openstack.org/#/c/101352/ is actually the patch which broke things, because it changed the behavior of the waitcondition API (the PUT no longer returns a body) | 09:38 |
| therve | shardy, I'm pretty sure it's the change in EngineService | 09:39 |
| openstackgerrit | Sergey Kraynev proposed a change to openstack/heat: Adding validation algorithm for get attr functions https://review.openstack.org/82488 | 09:39 |
| shardy | therve: I'm sure that's possible, I just can't reproduce the problem yet | 09:39 |
| therve | shardy, It called metadata_update on all resources, which reset metadata on server resources | 09:40 |
| therve | AFAIU | 09:40 |
| shardy | I've tried via tests and templates with metadata in various formats, including updating metadata referencing WaitCondition Data | 09:40 |
| shardy | therve: metadata_update already does the exact same thing | 09:40 |
| shardy | it's a no-op unless you pass data | 09:40 |
| shardy | All it does is forces the instance resources to re-resolve the metadata section | 09:41 |
| therve | shardy, Not on Server resources? | 09:41 |
| derekh_ | shardy: I recon the ironic seed failure is probly unrelated, | 09:41 |
| therve | It's no-op unless you don't pass data | 09:41 |
| derekh_ | shardy: looking at the overcloud os-collect-config logs, the problem doesn't seem to be presenthttp://logs.openstack.org/19/105219/1/experimental-tripleo/check-tripleo-overcloud-precise/59cea90/logs/overcloud-controller0-ih6kcth47rxx_logs/os-collect-config.txt.gz | 09:42 |
| shardy | therve: https://github.com/openstack/heat/blob/master/heat/engine/service.py#L1032 | 09:42 |
| therve | Well anyway the patch that just reverted that part fixed the issue in the gate | 09:42 |
| skraynev | shardy when you have a minute could you please look https://review.openstack.org/#/c/98580 in light of AWS documentation. | 09:42 |
| shardy | therve: Yeah, I'm not arguing it didn't, I just don't understand why yet :) | 09:43 |
| therve | shardy, Did you change it to do in the handle_signal case, though? | 09:43 |
| derekh_ | shardy: I think the overcloud log is the one you need to be looking at, basically look out for an oc-collect-config error "expected string or buffer" | 09:43 |
| shardy | therve: Yes, so the refresh happens when a wait condition notification happens via handle_signal | 09:44 |
| shardy | but it's a cut/paste from metadata_update, so it should work the same | 09:44 |
| therve | shardy, Yeah but it also happens if you use signal for something else? | 09:44 |
| shardy | therve: Yes, but it's not passing any data in the loop? | 09:44 |
| shardy | Any signal could contain data which affects the attributes of any resource, so it's probably wrong that we don't do the update now | 09:45 |
| shardy | It just so happens that the main time you need to do that atm (WaitConditions) goes via the metadata_update path atm | 09:45 |
| openstackgerrit | huangtianhua proposed a change to openstack/heat: Implement BlockDeviceMappings for AWS::EC2::Instance https://review.openstack.org/103725 | 09:46 |
| therve | Ah possibly, but you're changing behavior, so maybe that's the problem | 09:46 |
| shardy | If e.g resource_signal takes data from say ceilometer which is made available via an attribute, the same problem would exist | 09:46 |
| shardy | therve: Well evidently I did *something* which caused the problem ;) | 09:47 |
| shardy | but atm the only think I can see is the API change, which I've now removed from the series | 09:47 |
| shardy | s/think/thing | 09:47 |
| therve | I don't quite understand what Server.metadata_update does to be honest | 09:47 |
| *** jcoufal has quit IRC | 09:48 | |
| therve | shardy, I was under the impression that tripleo isn't using wait conditions, so I don't think the API change could have been the problem | 09:49 |
| shardy | therve: apparently they are, SpamapS said so yesterday, but only one for the whole deployment I think | 09:49 |
| therve | Ah he said the opposite on Saturday :) | 09:50 |
| shardy | therve: https://github.com/openstack/heat/blob/master/heat/engine/rsrc_defn.py#L168 | 09:51 |
| pas-ha | BTW, AWS waitcondition does not support any updates. In light of recent ML discussion and since we are going to have a native WaitCondition, should we deprecate updates on AWS one? | 09:51 |
| *** cdent has joined #heat | 09:51 | |
| shardy | therve: the instance/server metadata_update is a bit weird, its supposed to re-resolve the metadata from the template, so we get an update of e.g any attributes referenced in that section | 09:51 |
| therve | shardy, So here's my thought. _populate_deployments_metadata adds stuff to the metadata, but doesn't change the template | 09:53 |
| shardy | pas-ha: probably, yes, but IIRC SpamapS implemented UpdateWaitConditionHandle because he needed to do updates, so we'll need to ensure the new resource works for his use-case first | 09:53 |
| therve | shardy, If you call metadata_update afterwards, it would override what's has been set previously | 09:53 |
| shardy | therve: Ah! | 09:53 |
| *** pasquier-s_ has quit IRC | 09:54 | |
| shardy | https://github.com/openstack/heat/blob/master/heat/engine/rsrc_defn.py#L168 | 09:54 |
| shardy | That's where we re-resolve from the definition | 09:54 |
| *** dmakogon_ is now known as denis_makogon | 09:54 | |
| shardy | therve: So I'm wondering why the WaitCondition doesn't trigger the same problem already | 09:55 |
| shardy | maybe it just wipes the metadata at the very end of the deployment | 09:55 |
| therve | Possibly | 09:55 |
| shardy | but then SoftwareDeployment polling would break | 09:55 |
| therve | Maybe it's in a different template? | 09:56 |
| *** shakamunyi has joined #heat | 09:56 | |
| *** gokrokve has joined #heat | 09:56 | |
| shardy | therve: Yeah maybe, thanks that's a really helpful observation, I'll write a test and try to work out what's going on | 09:56 |
| *** ryansb has joined #heat | 09:57 | |
| * shardy goes to make a very large coffee before starting ;) | 09:57 | |
| therve | :) | 09:58 |
| Qiming | lol, 'large' coffee, wondering how large is it | 09:58 |
| openstackgerrit | Ishant Tyagi proposed a change to openstack/heat: Add subnet attribute to OS::Neutron::Port resource https://review.openstack.org/105408 | 09:58 |
| shardy | Qiming: I have a special mug for days like this, it holds about 0.5litres :D | 09:59 |
| Qiming | shardy, that's something else I can learn | 10:00 |
| skraynev | Qiming: there are some types of coffee size: small, medium, large, very large, size to open third eye and find all problems in WaitCondition ;) | 10:00 |
| *** alexpilotti has joined #heat | 10:00 | |
| *** alexpilotti has quit IRC | 10:00 | |
| *** gokrokve has quit IRC | 10:01 | |
| Qiming | shardy, seems like a flavor for bugs | 10:01 |
| shardy | LOL :) | 10:02 |
| skraynev | table mug-flavor-list !? | 10:03 |
| *** kopparam has joined #heat | 10:04 | |
| *** sorantis has joined #heat | 10:07 | |
| *** flyemsafe has joined #heat | 10:10 | |
| *** pasquier-s has joined #heat | 10:12 | |
| *** piyush has joined #heat | 10:13 | |
| SpamapS | shardy: FYI, we don't use wait conditions at all anymore. We use software deployments now. | 10:18 |
| *** shakamunyi has quit IRC | 10:20 | |
| shardy | SpamapS: interesting, but with the CFN transport? | 10:21 |
| shardy | If therve's analysis is correct (and it looks like it probably is) then it's the metadata refresh causing the problem, and the same problem would happen now if you used WaitConditions | 10:22 |
| *** Qiming has quit IRC | 10:23 | |
| therve | shardy, I think this a bug in Server. Heat shouldn't set metadata on behalf of the user. | 10:24 |
| therve | Oh although I guess that's how it's served to them | 10:25 |
| therve | Sigh | 10:25 |
| shardy | therve: Yeah, it's a problem as we're reusing resource metadata for communicating to the agents when it can also be specified in the template | 10:26 |
| shardy | For now we just need a way to refresh/re-resolve the references of the current metadata without overwriting it | 10:26 |
| shardy | Maybe we should expose resource data via the API as well as resource metadata | 10:27 |
| shardy | then the resource data can be the stuff modified by heat, and the metadata is always directly derived from the template | 10:28 |
| therve | Possibly. They are somewhat exposed via abandon | 10:28 |
| therve | shardy, That sounds nice | 10:28 |
| shardy | therve: Yeah, I'll look into it - it's pretty much how we work internally anyway so probably won't be too hard | 10:29 |
| therve | metadata management needs to be improved. It's weird that not all resources support it | 10:29 |
| shardy | the main issue is we save lots of sensitive information in the resource_data, so I'm not sure if it's a great idea to make it available to everything in the stack | 10:30 |
| shardy | therve: it's an unfortunate cfn hangover, it's a shame we supported it for the server resource really | 10:31 |
| *** sorantis_ has joined #heat | 10:32 | |
| therve | shardy, Well being to set key/value on resource and allowing them to retrieve it is nice | 10:33 |
| therve | It's the whole signal/update thing which is strange | 10:33 |
| *** tomek_adamczewsk has joined #heat | 10:33 | |
| shardy | therve: Yeah, but e.g server already has a property for metadata key/value storage, which nova makes available to the instance | 10:35 |
| *** sorantis has quit IRC | 10:35 | |
| *** sorantis_ is now known as sorantis | 10:35 | |
| therve | shardy, True. Non modifiable though | 10:35 |
| shardy | it's odd that we then have another resource-level metadata store, just because that's what CFN do | 10:35 |
| shardy | therve: Oh really? I thought user-data was fixed and meta-data could be updated, but I've not looked into it deeply | 10:36 |
| *** alexpilotti has joined #heat | 10:37 | |
| therve | shardy, You can change them, I'm not sure it's reflected post boot on the metadata service | 10:38 |
| *** tspatzier has quit IRC | 10:38 | |
| *** tspatzier has joined #heat | 10:39 | |
| *** rushiagr_ has quit IRC | 10:39 | |
| *** rushiagr_ has joined #heat | 10:40 | |
| openstackgerrit | A change was merged to openstack/heat: Improve OS::Trove::Instance resource https://review.openstack.org/97460 | 10:41 |
| therve | shardy, You're right it works | 10:44 |
| *** zhiyan is now known as zhiyan_ | 10:46 | |
| *** zhiyan_ is now known as zhiyan | 10:47 | |
| *** tspatzier has quit IRC | 10:51 | |
| *** julienvey has quit IRC | 10:53 | |
| *** tiantian has quit IRC | 10:54 | |
| *** jcoufal has joined #heat | 10:56 | |
| *** gokrokve has joined #heat | 10:56 | |
| *** achampion has quit IRC | 11:01 | |
| *** gokrokve has quit IRC | 11:01 | |
| *** dims_ has joined #heat | 11:02 | |
| *** alexheneveld has joined #heat | 11:02 | |
| *** pasquier-s has quit IRC | 11:06 | |
| *** alexheneveld has quit IRC | 11:07 | |
| *** mkerrin has quit IRC | 11:07 | |
| *** Qiming has joined #heat | 11:12 | |
| *** nosnos has quit IRC | 11:23 | |
| saju_m | crypt.encrypt(value.encode('utf-8')) ValueError: AES key must be either 16, 24, or 32 bytes long | 11:26 |
| saju_m | I am getting ValueError: AES key must be either 16, 24, or 32 bytes long | 11:27 |
| saju_m | while creating a stack | 11:27 |
| saju_m | Complete log :http://dpaste.com/1VRG24S | 11:28 |
| *** flyemsafe has quit IRC | 11:29 | |
| saju_m | posing command and error http://dpaste.com/1D6VYZC | 11:30 |
| *** alexheneveld has joined #heat | 11:33 | |
| *** bvandenh has quit IRC | 11:35 | |
| *** alexheneveld has quit IRC | 11:39 | |
| *** julienvey has joined #heat | 11:39 | |
| *** swygue has joined #heat | 11:40 | |
| therve | saju_m, I believe that's because your auth_encryption_key in the config is not of the proper lenght | 11:40 |
| saju_m | therve, yes, i just find that | 11:40 |
| saju_m | therve, thanks | 11:41 |
| saju_m | that should be 32 | 11:41 |
| *** Yanyan has joined #heat | 11:41 | |
| *** julienvey has quit IRC | 11:43 | |
| *** bvivek has quit IRC | 11:44 | |
| *** julienvey has joined #heat | 11:48 | |
| *** radez is now known as radez_g0n3 | 11:49 | |
| ckmvishnu | shoud there be a capability to load a stack using both integer "id" and a varchar "uuid"? | 11:54 |
| ckmvishnu | If thats the case I will have to alter stack_get function to accept both argument. | 11:55 |
| *** gokrokve has joined #heat | 11:56 | |
| ckmvishnu | this will be part of https://bugs.launchpad.net/heat/+bug/1263787 | 11:56 |
| uvirtbot | Launchpad bug 1263787 in heat "stack table's uuid primary key wastes resources in other tables" [High,Triaged] | 11:56 |
| *** bvandenh has joined #heat | 11:57 | |
| *** jdob has joined #heat | 12:01 | |
| *** gokrokve has quit IRC | 12:01 | |
| therve | ckmvishnu, Instinctively I'd say no but it's probably something you need to figure out while fixing the issue | 12:06 |
| *** pasquier-s has joined #heat | 12:14 | |
| ckmvishnu | therve, thanks. i'll leave it untouched for now. | 12:17 |
| *** pasquier-s has quit IRC | 12:20 | |
| *** che-arne has joined #heat | 12:21 | |
| *** DandyPandy has quit IRC | 12:22 | |
| *** kopparam has quit IRC | 12:23 | |
| *** kopparam has joined #heat | 12:25 | |
| *** dims_ has quit IRC | 12:26 | |
| *** pasquier-s has joined #heat | 12:26 | |
| *** dims_ has joined #heat | 12:26 | |
| *** dims_ has quit IRC | 12:27 | |
| *** dims_ has joined #heat | 12:28 | |
| *** pasquier-s_ has joined #heat | 12:34 | |
| *** cmyster has joined #heat | 12:36 | |
| *** shakamunyi has joined #heat | 12:37 | |
| *** kopparam has quit IRC | 12:37 | |
| *** rushiagr_ has quit IRC | 12:38 | |
| *** pasquier-s has quit IRC | 12:38 | |
| *** tspatzier has joined #heat | 12:41 | |
| *** DandyPandy has joined #heat | 12:42 | |
| *** Yanyan has quit IRC | 12:42 | |
| *** rbuilta has joined #heat | 12:46 | |
| *** morganfainberg is now known as morganfainberg_Z | 12:46 | |
| *** nkhare has quit IRC | 12:47 | |
| *** jmckind has joined #heat | 12:53 | |
| *** che-arne has quit IRC | 12:53 | |
| *** gokrokve has joined #heat | 12:56 | |
| *** sgordon has joined #heat | 12:56 | |
| *** rpothier has joined #heat | 12:59 | |
| *** cmyster has quit IRC | 13:01 | |
| *** gokrokve has quit IRC | 13:01 | |
| *** shakamunyi has quit IRC | 13:02 | |
| *** killer_prince has joined #heat | 13:03 | |
| *** jistr has quit IRC | 13:04 | |
| *** shakamunyi has joined #heat | 13:05 | |
| *** shakamunyi has quit IRC | 13:05 | |
| *** cmyster has joined #heat | 13:05 | |
| *** cmyster has joined #heat | 13:05 | |
| *** jistr has joined #heat | 13:06 | |
| *** shakamunyi has joined #heat | 13:06 | |
| *** unmeshg has left #heat | 13:11 | |
| *** jmckind has quit IRC | 13:14 | |
| *** ericgoncz has joined #heat | 13:17 | |
| *** lazy_prince has quit IRC | 13:21 | |
| *** killer_prince is now known as lazy_prince | 13:21 | |
| *** ericgoncz_ has joined #heat | 13:22 | |
| cmyster | distro hopped again... | 13:23 |
| *** piyush has quit IRC | 13:23 | |
| *** ericgoncz has quit IRC | 13:24 | |
| *** ericgoncz_ is now known as ericgoncz | 13:24 | |
| skraynev | cmyster: ? | 13:24 |
| *** ericgoncz has quit IRC | 13:24 | |
| cmyster | a general ? | 13:24 |
| *** samstav has joined #heat | 13:25 | |
| cmyster | what would you like me to explain here skraynev :) | 13:25 |
| *** samstav has quit IRC | 13:25 | |
| *** samstav has joined #heat | 13:26 | |
| skraynev | cmyster: just wanted to know, what distro hopped ? | 13:26 |
| cmyster | distro hopping is when you are fed up with a distro and hop to another | 13:26 |
| *** aweiteka has joined #heat | 13:27 | |
| *** Adri2000_ is now known as Adri2000 | 13:27 | |
| skraynev | cmyster: oh. got it :) thx for explanation | 13:27 |
| *** andreaf has quit IRC | 13:28 | |
| *** Tross has quit IRC | 13:28 | |
| *** robsparker has quit IRC | 13:29 | |
| cmyster | sure thing | 13:30 |
| cmyster | ummm | 13:30 |
| *** ishant has quit IRC | 13:30 | |
| cmyster | there was a page somewhere with a list of tempest blueprints and which member is assigned to which blueprint... | 13:33 |
| cmyster | I think that shardy shared it a while back ? | 13:33 |
| cmyster | searching LP only gets me to the large catalog made by sdake and this is a bit outdated | 13:33 |
| *** ramishra has joined #heat | 13:34 | |
| pas-ha | cmyster, I think those were bugs, not blueprints, with tags wishlist and tempest | 13:34 |
| cmyster | oooh thanks leeme search that | 13:35 |
| shardy | cmyster: https://bugs.launchpad.net/heat/+bugs?field.tag=tempest | 13:35 |
| shardy | That is probably not super up-to-date either tbh | 13:36 |
| shardy | but that is where we decided to track the new tests | 13:36 |
| *** alexheneveld has joined #heat | 13:36 | |
| cmyster | ok, I was referring to something else but if this is how we decided to track things... | 13:38 |
| cmyster | hmmm, | 13:39 |
| cmyster | brb | 13:39 |
| *** sorantis has quit IRC | 13:39 | |
| *** cmyster has quit IRC | 13:39 | |
| *** piyush has joined #heat | 13:40 | |
| *** piyush has quit IRC | 13:40 | |
| *** alexheneveld has quit IRC | 13:41 | |
| *** cmyster has joined #heat | 13:42 | |
| *** cmyster has joined #heat | 13:42 | |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Unit tests: Check for lost resource after failed rollback https://review.openstack.org/103236 | 13:43 |
| openstackgerrit | Thomas Herve proposed a change to openstack/heat: Migrate to oslo.db https://review.openstack.org/104566 | 13:43 |
| *** robsparker has joined #heat | 13:45 | |
| *** cmyster has quit IRC | 13:53 | |
| *** IlyaE has joined #heat | 13:56 | |
| *** gokrokve has joined #heat | 13:56 | |
| *** ericgoncz has joined #heat | 13:57 | |
| *** gokrokve has quit IRC | 14:01 | |
| *** shakamunyi has quit IRC | 14:01 | |
| *** radez_g0n3 is now known as radez | 14:03 | |
| zaneb | am I the only one who thinks that stevedore is worse in every way than what we have? | 14:09 |
| *** gokrokve has joined #heat | 14:10 | |
| *** pafuent has joined #heat | 14:11 | |
| *** gokrokve_ has joined #heat | 14:11 | |
| *** DandyPandy has quit IRC | 14:11 | |
| skraynev | zaneb: one bonus - it is in global-requirements... | 14:12 |
| *** DandyPandy has joined #heat | 14:12 | |
| *** gokrokve has quit IRC | 14:14 | |
| zaneb | skraynev: well, what we are using now was like 50 lines of code built in to heat... so it didn't _need_ to be in global requirements | 14:14 |
| *** piyush has joined #heat | 14:16 | |
| *** gokrokve_ has quit IRC | 14:17 | |
| *** gokrokve has joined #heat | 14:18 | |
| *** jmckind has joined #heat | 14:18 | |
| *** piyush1 has joined #heat | 14:19 | |
| *** david-lyle has joined #heat | 14:19 | |
| *** ramishra has quit IRC | 14:20 | |
| *** ckmvishnu has quit IRC | 14:20 | |
| *** piyush has quit IRC | 14:20 | |
| *** gokrokve has quit IRC | 14:22 | |
| skraynev | zaneb: agree, I do not mind old solution, I just try to find some positive goals of using existing feature (stevedore) | 14:24 |
| *** arbylee has joined #heat | 14:27 | |
| *** tango has joined #heat | 14:28 | |
| *** pasquier-s_ has quit IRC | 14:29 | |
| *** pasquier-s_ has joined #heat | 14:31 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: remove bogus comments re stack_delete_after in tests https://review.openstack.org/105469 | 14:34 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Don't overwrite deployments meadata in Server resource https://review.openstack.org/105470 | 14:35 |
| shardy | therve: ^^ Thanks for the discussion earlier, you were correct | 14:35 |
| *** rushiagr_ has joined #heat | 14:36 | |
| *** thomas_ustudio has joined #heat | 14:36 | |
| *** saju_m has quit IRC | 14:37 | |
| *** gokrokve has joined #heat | 14:37 | |
| *** ericgoncz has quit IRC | 14:38 | |
| therve | shardy, Cool, looking | 14:38 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Don't overwrite deployments metadata in Server resource https://review.openstack.org/105470 | 14:41 |
| *** rakesh_hs has quit IRC | 14:43 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient add get_user_token https://review.openstack.org/102885 | 14:44 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: stack user add _user_token https://review.openstack.org/102887 | 14:44 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: clients make heat_url public https://review.openstack.org/102886 | 14:44 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add an OS::Heat::WaitCondition resource https://review.openstack.org/101354 | 14:44 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add native WaitConditionHandle resource https://review.openstack.org/102888 | 14:44 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Convert WaitConditionHandle to use handle_signal https://review.openstack.org/105219 | 14:44 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Refactor waitcondition resources to allow easier subclassing https://review.openstack.org/105402 | 14:44 |
| *** tonisbones has joined #heat | 14:44 | |
| *** ishant has joined #heat | 14:48 | |
| *** tonisbones has quit IRC | 14:48 | |
| Qiming | coffee worked, :) | 14:49 |
| *** gokrokve has quit IRC | 14:50 | |
| *** ramishra has joined #heat | 14:51 | |
| *** ericgoncz has joined #heat | 14:51 | |
| *** tonisbones has joined #heat | 14:52 | |
| *** alexpilotti has quit IRC | 14:55 | |
| *** ramishra has quit IRC | 14:55 | |
| *** swygue has quit IRC | 15:00 | |
| *** zhiyan is now known as zhiyan_ | 15:00 | |
| *** jmckind has quit IRC | 15:01 | |
| *** DandyPandy has quit IRC | 15:01 | |
| SpamapS | shardy: "In future, we could consider exposing the resource_data via a separate API | 15:02 |
| SpamapS | call, so the resource-metadata could revert to being a pure representation | 15:02 |
| SpamapS | of the template-defined metadata, with resource_data being the data created | 15:02 |
| *** arbylee1 has joined #heat | 15:02 | |
| *** nati_ueno has joined #heat | 15:03 | |
| *** DandyPandy has joined #heat | 15:03 | |
| SpamapS | by the resource itself." .. I explicitly -1'd that during the software deployments implementation. The way it works now is _really_ simple from a user standpoint. Having two things to query and merge and discover is a large burden and seems to almost always be what is desired. | 15:03 |
| *** ericgoncz_ has joined #heat | 15:04 | |
| *** ericgoncz has quit IRC | 15:05 | |
| *** ericgoncz_ is now known as ericgoncz | 15:05 | |
| *** jmckind has joined #heat | 15:05 | |
| *** arbylee1 has quit IRC | 15:06 | |
| *** arbylee1 has joined #heat | 15:06 | |
| *** sgordon has quit IRC | 15:06 | |
| *** arbylee has quit IRC | 15:07 | |
| *** chandankumar has quit IRC | 15:07 | |
| *** jmckind has quit IRC | 15:08 | |
| *** gokrokve has joined #heat | 15:10 | |
| *** david-lyle has quit IRC | 15:10 | |
| *** david-lyle has joined #heat | 15:11 | |
| *** jistr has quit IRC | 15:12 | |
| *** jistr has joined #heat | 15:13 | |
| *** jmckind has joined #heat | 15:13 | |
| *** gokrokve_ has joined #heat | 15:15 | |
| *** nati_ueno has quit IRC | 15:16 | |
| *** gokrokve has quit IRC | 15:17 | |
| *** nati_ueno has joined #heat | 15:18 | |
| *** alexheneveld has joined #heat | 15:20 | |
| *** nati_ueno has quit IRC | 15:22 | |
| *** samstav_ has joined #heat | 15:23 | |
| *** daneyon has joined #heat | 15:23 | |
| *** nati_uen_ has joined #heat | 15:24 | |
| *** sabeen1 has joined #heat | 15:24 | |
| *** kebray has joined #heat | 15:26 | |
| *** samstav has quit IRC | 15:26 | |
| *** kebray has quit IRC | 15:27 | |
| *** zz_gondoi is now known as gondoi | 15:27 | |
| *** kebray has joined #heat | 15:27 | |
| *** samstav_ has quit IRC | 15:27 | |
| shardy | SpamapS: it's really simple until you accidentally add some key to the template which is then silently overwritten at create time by the resource | 15:28 |
| shardy | SpamapS: I'm happy to remove the comment, but IMO at the least we should add a validation step which e.g rejects a template with a metadata key of 'deployments' if the resource properties indicate it will be overwritten | 15:29 |
| *** jprovazn has quit IRC | 15:30 | |
| *** jmckind has quit IRC | 15:30 | |
| *** TonyBurn_ has quit IRC | 15:32 | |
| *** samstav has joined #heat | 15:34 | |
| *** samstav has quit IRC | 15:35 | |
| SpamapS | shardy: no the comment is good, just making the past case known. | 15:35 |
| *** samstav has joined #heat | 15:35 | |
| SpamapS | shardy: with re-use, that is a valid use-case | 15:36 |
| shardy | SpamapS: np, I'll remove the comment anyway, if that discussion already happened | 15:36 |
| SpamapS | shardy: you may have a statically defined deployment that can be overridden by deployments in another template. | 15:36 |
| shardy | poking data into metadta from two data sources seems a bit wrong, but whatever works I guess :) | 15:37 |
| *** jmckind has joined #heat | 15:37 | |
| SpamapS | shardy: As long as it is well documented I think we're o-k. Another thing might be to make the key configurable. | 15:37 |
| SpamapS | sort of the same way we allow changing the name of 'get_input' | 15:37 |
| shardy | SpamapS: I may propose a patch which adds some validation, so at least then we can detect and explicitly reject any metadata key conflicts | 15:37 |
| *** tomek_adamczewsk has quit IRC | 15:37 | |
| shardy | It's probably unlikely, but this is a bit hidden and magical, so I'd prefer an explicit check which fails early | 15:38 |
| *** Dev_Jin has quit IRC | 15:38 | |
| zaneb | shardy: is there a review up for the native signal stuff you are working on? | 15:39 |
| *** sarob has joined #heat | 15:39 | |
| shardy | zaneb: Yes, several - the patch I'm discussing with SpamapS is https://review.openstack.org/#/c/105470/ which addreses the bug in OS::Nova::Server exposed by my earlier patches (which were reverted on Friday) | 15:40 |
| *** gokrokve has joined #heat | 15:41 | |
| *** Tross has joined #heat | 15:41 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient add get_user_token https://review.openstack.org/102885 | 15:41 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: stack user add _user_token https://review.openstack.org/102887 | 15:41 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: clients make heat_url public https://review.openstack.org/102886 | 15:41 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add an OS::Heat::WaitCondition resource https://review.openstack.org/101354 | 15:41 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add native WaitConditionHandle resource https://review.openstack.org/102888 | 15:41 |
| zaneb | shardy: ok, it isn't a separate resource type, just a native reimplementation of the existing one | 15:41 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Convert WaitConditionHandle to use handle_signal https://review.openstack.org/105219 | 15:41 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Refactor waitcondition resources to allow easier subclassing https://review.openstack.org/105402 | 15:41 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Don't overwrite deployments metadata in Server resource https://review.openstack.org/105470 | 15:41 |
| *** nati_uen_ has quit IRC | 15:42 | |
| shardy | zaneb: It adds native WaitCondition and WaitConditionHandle resources, which can be signalled via the native API | 15:42 |
| *** shakamunyi has joined #heat | 15:42 | |
| *** shakamunyi has quit IRC | 15:42 | |
| zaneb | shardy: ah, found the patch. thanks | 15:43 |
| *** gokrokve_ has quit IRC | 15:43 | |
| openstackgerrit | Andrea Rosa proposed a change to openstack/heat: Add validation constraints on config inputs https://review.openstack.org/105496 | 15:43 |
| shardy | zaneb: The idea is these can be used for simple "one shot" signalling for simple deployments, and SoftwareDeployments will be used where you care about subsequent updates during the lifecycle of the stack | 15:44 |
| zaneb | shardy: yep, just writing an email that mentions jasond's OS::Swift::Signal resource, and how we need a better strategy for picking an implementation | 15:45 |
| zaneb | that is to say, for the operator to pick an implementation | 15:45 |
| shardy | zaneb: My understanding was that we'd make the resource properties compatible, so you could map them via the environment | 15:45 |
| zaneb | right, and that is a terrible solution ;) | 15:46 |
| shardy | zaneb: how so? | 15:46 |
| *** pas-ha has quit IRC | 15:46 | |
| zaneb | because it's not the user's problem | 15:46 |
| shardy | zaneb: The deployer controls the global environment :\ | 15:46 |
| zaneb | or shouldn't be, but is once you call that a solution | 15:46 |
| shardy | Or I suppose we can add yet another config option | 15:47 |
| * shardy shrugs | 15:47 | |
| shardy | either way the properties need to be compatible | 15:47 |
| *** jmckind has quit IRC | 15:47 | |
| zaneb | I think we should have a pluggable implementation, and then a native resource, and AWS resource and the software deployment resource (and any custom resources requiring signalling functionality) can all use it | 15:48 |
| zaneb | and the operator selects which implementation will actually work on their cloud | 15:48 |
| therve | Isn't the global env the way to control that? | 15:49 |
| shardy | zaneb: well that's pretty much the way I've been headed with my patches, abstracting the common bits into base-classes | 15:49 |
| zaneb | therve: it's abstracting at the wrong level imo | 15:49 |
| zaneb | shardy: ++ | 15:49 |
| zaneb | shardy: I'm not suggesting you throw any work away :) | 15:50 |
| shardy | zaneb: I thought selecting mappings from aliases to concrete implementations was precisely what the global environment was for | 15:50 |
| therve | zaneb, So how do you suggest that the operator selects the implementation? | 15:51 |
| shardy | but honestly, I don't really care which file we make deployers edit all that much ;) | 15:51 |
| *** robsparker has quit IRC | 15:51 | |
| zaneb | therve: with stevedore plugins | 15:51 |
| *** ramishra has joined #heat | 15:51 | |
| * zaneb is writing an email to this effect as we speak | 15:52 | |
| therve | Mokay | 15:52 |
| therve | I'm not sure why we should have a different mechanism to say "register this resource to this name" | 15:53 |
| shardy | therve: I agree, if we go this way, we should probably deprecate the global environment | 15:54 |
| zaneb | therve: I'm saying that there should be only one resource plugin, not two | 15:54 |
| zaneb | or more | 15:54 |
| shardy | but personally I think that the global environment is a much clearer way to specify these sorts of mappings | 15:54 |
| zaneb | because e.g. software deployments also use signalling | 15:54 |
| zaneb | they should use the same implementation | 15:55 |
| zaneb | we don't want an OS::Swift::SoftwareDeployment resource | 15:55 |
| shardy | zaneb: So you want a pluggable SignalResponder implementation, not the actual resources | 15:55 |
| zaneb | shardy: exactly, yeah | 15:55 |
| shardy | zaneb: Ok, cool, yeah that makes sense | 15:55 |
| shardy | I was just thinking of the two mostly-compatible WaitCondition solutions | 15:56 |
| therve | So maybe... software deployments should use resources? Is that possible? | 15:56 |
| zaneb | therve: here be dragons | 15:56 |
| *** ramishra has quit IRC | 15:56 | |
| *** samstav has quit IRC | 15:57 | |
| zaneb | shardy: the problem with the global environment is that it leaks info to the user - e.g. both names for the resource type are accessible, so things can break on different clouds | 15:57 |
| shardy | zaneb: Yeah, we'd need to add a simple way to blacklist the resources you don't want | 15:58 |
| therve | That part exists already | 15:58 |
| *** Tross has quit IRC | 15:59 | |
| zaneb | it's still abstraction at the wrong level IMO, this is an implementation that needs abstracting, not a resource. it should have an API, not separate plugins | 15:59 |
| *** gondoi is now known as zz_gondoi | 16:00 | |
| shardy | zaneb: Yup, I'm pretty much in agreement now I understand you want a pluggable base-class not selection between to mostly different resource implementations with a similar interface | 16:01 |
| *** samstav has joined #heat | 16:03 | |
| *** Qiming has quit IRC | 16:03 | |
| therve | A pluggable base-class doesn't sound like a implementation plan, though | 16:04 |
| therve | It'd be better to have an interface rather than use inheritance, IMHO | 16:04 |
| zaneb | therve: agree, I don't think the base class would be pluggable. there would a be a pluggable api and the base class would call the api | 16:05 |
| therve | zaneb, Cool, sounds reasonable | 16:05 |
| *** tomek_adamczewsk has joined #heat | 16:07 | |
| *** tspatzier has quit IRC | 16:10 | |
| *** alexheneveld has quit IRC | 16:12 | |
| *** andersonvom has joined #heat | 16:14 | |
| *** chandankumar has joined #heat | 16:17 | |
| *** lazy_prince is now known as killer_prince | 16:19 | |
| shardy | Anyone aware of any current issues with VolumeAttachments not working? | 16:19 |
| *** tonisbones has quit IRC | 16:19 | |
| *** che-arne has joined #heat | 16:21 | |
| *** pasquier-s_ has quit IRC | 16:22 | |
| *** DandyPandy has quit IRC | 16:23 | |
| *** DandyPandy has joined #heat | 16:24 | |
| *** alexpilotti has joined #heat | 16:26 | |
| *** sgordon has joined #heat | 16:26 | |
| *** sgordon has quit IRC | 16:26 | |
| *** sgordon has joined #heat | 16:26 | |
| *** arbylee1 has quit IRC | 16:29 | |
| *** tomek_adamczewsk has quit IRC | 16:30 | |
| *** nati_ueno has joined #heat | 16:31 | |
| *** alexpilotti has quit IRC | 16:32 | |
| *** bandarji has joined #heat | 16:34 | |
| openstackgerrit | A change was merged to openstack/heat: remove bogus comments re stack_delete_after in tests https://review.openstack.org/105469 | 16:39 |
| *** jistr has quit IRC | 16:40 | |
| *** nati_ueno has quit IRC | 16:41 | |
| *** nati_ueno has joined #heat | 16:42 | |
| *** IlyaE has quit IRC | 16:42 | |
| *** jcoufal has quit IRC | 16:43 | |
| *** denis_makogon has quit IRC | 16:48 | |
| *** sarob has quit IRC | 16:50 | |
| *** sarob has joined #heat | 16:51 | |
| openstackgerrit | Ishant Tyagi proposed a change to openstack/heat: Add subnet attribute to OS::Neutron::Port resource https://review.openstack.org/105408 | 16:53 |
| *** denis_makogon has joined #heat | 16:53 | |
| *** sarob has quit IRC | 16:55 | |
| *** derekh_ has quit IRC | 16:58 | |
| *** che-arne|2 has joined #heat | 17:00 | |
| *** denis_makogon has quit IRC | 17:01 | |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Add stack-check action to the API https://review.openstack.org/89905 | 17:01 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Implement check for Instance resource https://review.openstack.org/89906 | 17:01 |
| openstackgerrit | Anderson Mesquita proposed a change to openstack/heat: Check resources are in expected states https://review.openstack.org/89908 | 17:01 |
| andersonvom | SpamapS: I just rebased and fixed the docstrings and log messages you mentioned. | 17:02 |
| andersonvom | other folks, if you could take a look at them, that would be great! =D | 17:03 |
| *** che-arne has quit IRC | 17:03 | |
| SpamapS | andersonvom: great! :) | 17:06 |
| SpamapS | andersonvom: will do a round of reviews soon. | 17:06 |
| andersonvom | SpamapS: thanks | 17:06 |
| *** achampion has joined #heat | 17:09 | |
| *** arbylee has joined #heat | 17:09 | |
| *** arbylee has quit IRC | 17:10 | |
| *** tango has quit IRC | 17:10 | |
| *** arbylee has joined #heat | 17:10 | |
| *** chandankumar has quit IRC | 17:13 | |
| openstackgerrit | Jason Dunsmore proposed a change to openstack/heat: Fix OS::Trove::Instance restore_point property https://review.openstack.org/105210 | 17:14 |
| *** cdent_ has joined #heat | 17:15 | |
| *** gokrokve has quit IRC | 17:15 | |
| *** gokrokve has joined #heat | 17:15 | |
| *** cdent has quit IRC | 17:16 | |
| *** cdent_ is now known as cdent | 17:16 | |
| *** sballe has quit IRC | 17:18 | |
| *** gokrokve has quit IRC | 17:19 | |
| sjmc7 | hi guys.. did the heat template version change recently? | 17:24 |
| therve | Nope | 17:25 |
| *** cdent has quit IRC | 17:26 | |
| sjmc7 | i'm getting "ERROR: The template version is invalid: Template version was not provided | 17:26 |
| sjmc7 | " from previously-working templates | 17:26 |
| sjmc7 | ok, will check my installation | 17:27 |
| *** harlowja_away is now known as harlowja | 17:29 | |
| therve | sjmc7, pip install -e . | 17:29 |
| sjmc7 | this is a devstack install - do that in /opt/stack/heat ? | 17:30 |
| therve | Yeah | 17:30 |
| therve | Or restack | 17:31 |
| sjmc7 | pip install -e didn't help. did something specific change, or is this a 'turn it off and on' kind of thing? | 17:34 |
| *** piyush1 has quit IRC | 17:35 | |
| therve | sjmc7, Did you restart the heat engine? | 17:35 |
| therve | It started using stevedore, and it needs to run setup again | 17:36 |
| sjmc7 | ah.. getting a stevedore exception from glance now | 17:36 |
| sjmc7 | ok, restack it is | 17:37 |
| *** gokrokve has joined #heat | 17:37 | |
| *** piyush has joined #heat | 17:39 | |
| *** piyush1 has joined #heat | 17:41 | |
| *** dims_ has quit IRC | 17:41 | |
| *** jprovazn has joined #heat | 17:42 | |
| *** piyush has quit IRC | 17:43 | |
| *** ishant has quit IRC | 17:43 | |
| *** tomek_adamczewsk has joined #heat | 17:45 | |
| *** daneyon has quit IRC | 17:48 | |
| *** daneyon has joined #heat | 17:49 | |
| *** tomek_adamczewsk has quit IRC | 17:50 | |
| *** tango has joined #heat | 17:52 | |
| *** IlyaE has joined #heat | 17:57 | |
| *** tomek_adamczewsk has joined #heat | 17:57 | |
| *** [1]maishsk has joined #heat | 17:59 | |
| *** tomek_adamczewsk has quit IRC | 18:00 | |
| *** tomek_adamczewsk has joined #heat | 18:00 | |
| *** nati_ueno has quit IRC | 18:03 | |
| *** Tross has joined #heat | 18:05 | |
| *** ckmvishnu has joined #heat | 18:05 | |
| *** che-arne|2 has quit IRC | 18:06 | |
| *** tomek_adamczewsk has quit IRC | 18:06 | |
| *** rushiagr_ has quit IRC | 18:07 | |
| *** lindsayk1 has joined #heat | 18:08 | |
| *** dims_ has joined #heat | 18:08 | |
| *** tomek_adamczewsk has joined #heat | 18:08 | |
| *** jcoufal has joined #heat | 18:12 | |
| *** dims_ has quit IRC | 18:12 | |
| *** tomek_adamczewsk has quit IRC | 18:15 | |
| *** dims_ has joined #heat | 18:16 | |
| *** tomek_adamczewsk has joined #heat | 18:18 | |
| *** alexpilotti has joined #heat | 18:18 | |
| *** ckmvishnu has quit IRC | 18:19 | |
| *** aweiteka has quit IRC | 18:20 | |
| *** zz_gondoi is now known as gondoi | 18:22 | |
| openstackgerrit | Vijendar Komalla proposed a change to openstack/heat: Fix early resource property value validation https://review.openstack.org/93080 | 18:24 |
| *** vijendar has joined #heat | 18:24 | |
| *** tomek_adamczewsk has quit IRC | 18:27 | |
| *** tomek_adamczewsk has joined #heat | 18:28 | |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: heat_keystoneclient add get_user_token https://review.openstack.org/102885 | 18:29 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: stack user add _user_token https://review.openstack.org/102887 | 18:29 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: clients make heat_url public https://review.openstack.org/102886 | 18:29 |
| openstackgerrit | Steven Hardy proposed a change to openstack/heat: Add native WaitConditionHandle resource https://review.openstack.org/102888 | 18:29 |
| *** sarob has joined #heat | 18:30 | |
| [1]maishsk | shardy - you around? | 18:30 |
| shardy | [1]maishsk: I'm just going to grab some dinner, is it a quick question? | 18:32 |
| *** asalkeld has joined #heat | 18:32 | |
| *** aweiteka has joined #heat | 18:33 | |
| [1]maishsk | shardy: enjoy! | 18:33 |
| [1]maishsk | catch you later | 18:34 |
| shardy | [1]maishsk: Ok, I'll probably be around again later (couple of hours) | 18:34 |
| *** shardy is now known as shardy_afk | 18:34 | |
| *** tomek_adamczewsk has quit IRC | 18:39 | |
| *** tomek_adamczewsk has joined #heat | 18:40 | |
| *** kevinbenton has quit IRC | 18:40 | |
| *** [1]maishsk has quit IRC | 18:50 | |
| *** tomek_adamczewsk has quit IRC | 18:51 | |
| *** IlyaE has quit IRC | 18:52 | |
| *** daneyon has quit IRC | 18:52 | |
| *** daneyon has joined #heat | 18:53 | |
| *** kevinbenton has joined #heat | 18:53 | |
| *** radez is now known as radez_g0n3 | 18:54 | |
| *** julienvey has quit IRC | 18:54 | |
| *** dsneddon is now known as dsneddon_lunch | 18:55 | |
| *** nati_ueno has joined #heat | 18:56 | |
| *** radez_g0n3 is now known as radez | 18:57 | |
| *** nati_ueno has quit IRC | 19:00 | |
| *** asalkeld has quit IRC | 19:00 | |
| *** nati_ueno has joined #heat | 19:00 | |
| *** nati_ueno has quit IRC | 19:03 | |
| *** nati_ueno has joined #heat | 19:03 | |
| *** IlyaE has joined #heat | 19:04 | |
| openstackgerrit | Zane Bitter proposed a change to openstack/heat: Create a new revision of HOT for Juno https://review.openstack.org/105558 | 19:09 |
| *** asalkeld has joined #heat | 19:13 | |
| *** andersonvom has quit IRC | 19:13 | |
| *** IlyaE has quit IRC | 19:17 | |
| *** piyush1 has quit IRC | 19:24 | |
| *** gokrokve has quit IRC | 19:28 | |
| *** maishsk has joined #heat | 19:28 | |
| *** gokrokve has joined #heat | 19:28 | |
| *** julienvey has joined #heat | 19:28 | |
| *** gokrokve has quit IRC | 19:33 | |
| *** piyush has joined #heat | 19:33 | |
| *** piyush has quit IRC | 19:33 | |
| *** rbuilta has quit IRC | 19:37 | |
| openstackgerrit | Matthew Printz proposed a change to openstack/heat: Fix typo; Closes-Bug: 1339269 https://review.openstack.org/105566 | 19:38 |
| *** ericgoncz has quit IRC | 19:44 | |
| *** pafuent has left #heat | 19:48 | |
| *** tomek_adamczewsk has joined #heat | 19:52 | |
| *** ericgoncz has joined #heat | 19:54 | |
| *** sarob has quit IRC | 19:55 | |
| *** sarob has joined #heat | 19:56 | |
| *** julienvey has quit IRC | 19:58 | |
| *** tomek_adamczewsk has quit IRC | 19:58 | |
| *** asalkeld has quit IRC | 19:58 | |
| *** sarob has quit IRC | 20:01 | |
| *** tomek_adamczewsk has joined #heat | 20:01 | |
| *** lindsayk1 has quit IRC | 20:01 | |
| openstackgerrit | Lance Chen proposed a change to openstack/heat: Fix docker plugin related path inside README.md https://review.openstack.org/105574 | 20:02 |
| *** dsneddon_lunch is now known as dsneddon | 20:04 | |
| *** ericgoncz_ has joined #heat | 20:04 | |
| *** lindsayk1 has joined #heat | 20:04 | |
| *** nati_ueno has quit IRC | 20:05 | |
| *** shardy_afk is now known as shardy | 20:05 | |
| *** nati_ueno has joined #heat | 20:05 | |
| *** ericgoncz has quit IRC | 20:06 | |
| *** ericgoncz_ is now known as ericgoncz | 20:06 | |
| *** radez is now known as radez_g0n3 | 20:07 | |
| *** nati_ueno has quit IRC | 20:07 | |
| *** tomek_adamczewsk has quit IRC | 20:07 | |
| *** cyen has joined #heat | 20:08 | |
| *** tomek_adamczewsk has joined #heat | 20:08 | |
| openstackgerrit | Matthew Printz proposed a change to openstack/heat: Fix typo in HOT template guide https://review.openstack.org/105566 | 20:09 |
| *** nati_ueno has joined #heat | 20:11 | |
| *** lindsayk2 has joined #heat | 20:12 | |
| *** lindsayk1 has quit IRC | 20:15 | |
| *** sarob has joined #heat | 20:18 | |
| *** asalkeld has joined #heat | 20:18 | |
| *** IlyaE has joined #heat | 20:19 | |
| *** tomek_adamczewsk has quit IRC | 20:19 | |
| openstackgerrit | Matthew Printz proposed a change to openstack/heat: Fix typo in HOT template guide https://review.openstack.org/105566 | 20:23 |
| *** aweiteka has quit IRC | 20:28 | |
| *** tomek_adamczewsk has joined #heat | 20:39 | |
| *** aweiteka has joined #heat | 20:41 | |
| *** nati_ueno has quit IRC | 20:46 | |
| *** nati_ueno has joined #heat | 20:46 | |
| *** tomek_adamczewsk has quit IRC | 20:47 | |
| SpamapS | stevebaker: \o/ heat native will be the default collection method in the next release of os-collect-config. :-D | 20:48 |
| *** tomek_adamczewsk has joined #heat | 20:48 | |
| SpamapS | stevebaker: and cfn will be turned off | 20:48 |
| *** piyush has joined #heat | 20:48 | |
| openstackgerrit | Matthew Printz proposed a change to openstack/heat: Fix typo in HOT template guide https://review.openstack.org/105566 | 20:50 |
| SpamapS | actually hrm, I wonder if we should consider that a breaking change in os-collect-config | 20:50 |
| *** tomek_adamczewsk has quit IRC | 20:52 | |
| stevebaker | SpamapS: oooo, are you re-using the token? | 20:52 |
| SpamapS | stevebaker: aye | 20:53 |
| stevebaker | super | 20:53 |
| SpamapS | stevebaker: https://git.openstack.org/cgit/openstack/os-collect-config/tree/os_collect_config/keystone.py#n58 | 20:53 |
| *** Qiming has joined #heat | 20:54 | |
| stevebaker | SpamapS: can you take a look at my HOT port changes at some point? | 20:55 |
| Qiming | unblieveable ... | 20:56 |
| *** achampion has quit IRC | 20:56 | |
| SpamapS | stevebaker: sure. | 20:57 |
| stevebaker | Qiming: brazil? | 21:00 |
| *** shakamunyi has joined #heat | 21:00 | |
| Qiming | stevebaker, yes, sir! | 21:00 |
| stevebaker | its painful | 21:00 |
| *** piyush has quit IRC | 21:01 | |
| *** jdob has quit IRC | 21:01 | |
| Qiming | let's see what will happen during the 2nd half | 21:02 |
| *** ericgoncz has quit IRC | 21:02 | |
| *** shakamunyi has quit IRC | 21:03 | |
| *** maishsk has quit IRC | 21:05 | |
| *** shakamunyi has joined #heat | 21:06 | |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Use setuptools to install contrib plugins https://review.openstack.org/103441 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Only support client plugins https://review.openstack.org/98688 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Glance, swift use ignore_not_found() https://review.openstack.org/100424 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement keystone client plugin https://review.openstack.org/97985 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement heat client plugin https://review.openstack.org/97984 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Client plugin exception handling methods https://review.openstack.org/100421 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Move network constraint to neutron plugin https://review.openstack.org/100420 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Neutron use ignore_not_found() https://review.openstack.org/100423 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Nova use ignore_not_found() https://review.openstack.org/100422 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Move nova_utils functions to nova client plugin https://review.openstack.org/101782 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Store clients in context https://review.openstack.org/98935 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Move glance_utils functions to glance client plugin https://review.openstack.org/100419 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement barbican client plugin https://review.openstack.org/98684 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Use config region_name_for_services for rackspace clients https://review.openstack.org/98686 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Port rackspace clients to client plugins https://review.openstack.org/98687 | 21:09 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Remove function get_software_config https://review.openstack.org/101396 | 21:10 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Cinder, trove, ceilo use ignore_not_found() https://review.openstack.org/101397 | 21:10 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement marconi client plugin https://review.openstack.org/98683 | 21:10 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement swift client plugin https://review.openstack.org/97978 | 21:10 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement glance client plugin https://review.openstack.org/97979 | 21:10 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement neutron client plugin https://review.openstack.org/97980 | 21:10 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement cinder client plugin https://review.openstack.org/97981 | 21:10 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement trove client plugin https://review.openstack.org/97982 | 21:10 |
| openstackgerrit | Steve Baker proposed a change to openstack/heat: Implement ceilometer client plugin https://review.openstack.org/97983 | 21:10 |
| *** samstav has quit IRC | 21:10 | |
| stevebaker | I'm on holiday now. If there are nits to fix in ^ then feel free to overlook them or fix them yourselves | 21:11 |
| *** asalkeld_ has joined #heat | 21:15 | |
| *** asalkeld has quit IRC | 21:15 | |
| *** rpothier has quit IRC | 21:19 | |
| zaneb | stevebaker: enjoy! | 21:19 |
| *** jprovazn has quit IRC | 21:20 | |
| *** asalkeld_ has quit IRC | 21:21 | |
| *** asalkeld_ has joined #heat | 21:21 | |
| *** sarob has quit IRC | 21:22 | |
| *** DandyPandy has quit IRC | 21:24 | |
| *** DandyPandy has joined #heat | 21:25 | |
| asalkeld_ | stevebaker, have a good break | 21:26 |
| stevebaker | asalkeld_: will do. Heading to Gold Coast but we'll be too busy to visit | 21:26 |
| asalkeld_ | ok | 21:27 |
| asalkeld_ | well I am in hawaii until the weekend | 21:27 |
| asalkeld_ | watching the germany/brazil ass kicking at the pool bar :-) | 21:28 |
| *** zaneb has quit IRC | 21:28 | |
| asalkeld_ | whilst repling to zane's "don't kill the pluginmanager" email | 21:28 |
| asalkeld_ | :) | 21:29 |
| asalkeld_ | zane is getting plugin manager withdrawal | 21:29 |
| asalkeld_ | brb | 21:31 |
| *** asalkeld_ has quit IRC | 21:31 | |
| *** aweiteka has quit IRC | 21:31 | |
| *** asalkeld has joined #heat | 21:36 | |
| *** jprovazn has joined #heat | 21:39 | |
| *** jprovazn has quit IRC | 21:40 | |
| *** IlyaE has quit IRC | 21:41 | |
| *** zaneb has joined #heat | 21:42 | |
| asalkeld | yo zaneb | 21:42 |
| zaneb | hey | 21:42 |
| asalkeld | responded to your email | 21:42 |
| zaneb | saw that, thanks | 21:43 |
| zaneb | I have to dig more deeply into some parts of it | 21:43 |
| asalkeld | as i wrote those plugins i liked stevedore more and more | 21:43 |
| *** ccrouch has quit IRC | 21:43 | |
| *** ccrouch has joined #heat | 21:44 | |
| asalkeld | but totally agree re: functions | 21:44 |
| asalkeld | that is messy | 21:44 |
| asalkeld | there is a balance between pluggablity and cross cloud portablity | 21:45 |
| zaneb | cool, we can definitely agree on that one then :) | 21:45 |
| *** shakamunyi has quit IRC | 21:45 | |
| asalkeld | zaneb, we do the the environment (resource registry) | 21:46 |
| asalkeld | that should really control what is exposed | 21:46 |
| zaneb | parse error | 21:46 |
| *** shakamunyi has joined #heat | 21:46 | |
| openstackgerrit | A change was merged to openstack/heat: Add datastore properties to OS::Trove::Instance https://review.openstack.org/60532 | 21:46 |
| asalkeld | the mechanism of pulling the plugins should not be important | 21:46 |
| asalkeld | the mechanism of loading the plugins should not be important | 21:47 |
| asalkeld | one issue is the resource registry loads everything | 21:47 |
| asalkeld | and only allows disabling | 21:47 |
| asalkeld | and renaming | 21:47 |
| *** ccrouch has quit IRC | 21:48 | |
| asalkeld | dang 7-1 | 21:49 |
| asalkeld | well done germany | 21:49 |
| *** zaneb has quit IRC | 21:52 | |
| *** zaneb has joined #heat | 21:54 | |
| *** vijendar has quit IRC | 21:54 | |
| zaneb | ugh, internet is flaky this afternoon | 21:54 |
| *** dims__ has joined #heat | 21:55 | |
| zaneb | asalkeld: sorry, connection dropped right after "<zaneb> asalkeld: it's not clear to me why we couldn't use stevedore but still keep the resource_map() functions as the entry points for the extensions?" | 21:55 |
| *** ccrouch has joined #heat | 21:56 | |
| asalkeld | zaneb, that's how stevedore works? | 21:56 |
| asalkeld | zaneb, it looks in the namespace you specify | 21:56 |
| *** dims_ has quit IRC | 21:56 | |
| asalkeld | so one option is the name you give to the resource is bogus | 21:57 |
| asalkeld | and we then find the plugin and assiciate that name | 21:57 |
| zaneb | asalkeld: right, but it's looking for particular classes and mapping those to resource type names in the setup file, isn't it? | 21:57 |
| asalkeld | yeah | 21:57 |
| zaneb | whereas it could just look for the resource_map() functions | 21:58 |
| zaneb | that way we could keep the mapping between names and classes in the code | 21:58 |
| asalkeld | it would have to move to a class static | 21:58 |
| zaneb | near where they are defined | 21:58 |
| zaneb | oh, stevedore only accepts classes as plugins? | 21:58 |
| asalkeld | zaneb, that is weird to igore the name you give in setup.cfg | 21:58 |
| asalkeld | zaneb, yes | 21:59 |
| *** gondoi is now known as zz_gondoi | 21:59 | |
| asalkeld | so we have : | 21:59 |
| asalkeld | heat.resources = | 21:59 |
| asalkeld | <name> = class | 21:59 |
| zaneb | asalkeld: maybe it's more of a hook rather than an extension pattern if we do it that way? | 21:59 |
| asalkeld | we would have to ignore <name> | 22:00 |
| asalkeld | let me check the def. of hook | 22:00 |
| asalkeld | http://stevedore.readthedocs.org/en/latest/patterns_loading.html#hooks-single-name-many-entry-points | 22:01 |
| *** maishsk has joined #heat | 22:02 | |
| *** Qiming has quit IRC | 22:03 | |
| asalkeld | it doesn't seem to have a check_func | 22:03 |
| *** jcoufal has quit IRC | 22:03 | |
| *** lindsayk has quit IRC | 22:03 | |
| zaneb | that's unfortunate | 22:05 |
| zaneb | but we could use something similar to the existing pattern | 22:06 |
| asalkeld | zaneb, yeah | 22:06 |
| zaneb | maybe prettier, if we are breaking backwards compat anyway | 22:06 |
| asalkeld | we could still have our own enable_func | 22:06 |
| *** IlyaE has joined #heat | 22:06 | |
| *** lindsayk2 has quit IRC | 22:06 | |
| asalkeld | and then not add it to the env | 22:07 |
| asalkeld | so loaded as far as stevedore thinks | 22:07 |
| asalkeld | but not visible to users | 22:07 |
| zaneb | +1 | 22:07 |
| * zaneb -> food | 22:07 | |
| zaneb | bbiab, sorry | 22:07 |
| asalkeld | no worries | 22:08 |
| *** piyush has joined #heat | 22:10 | |
| *** lindsayk has joined #heat | 22:13 | |
| *** thomas_ustudio has quit IRC | 22:13 | |
| *** shakamunyi has quit IRC | 22:20 | |
| *** bandarji has quit IRC | 22:20 | |
| *** shakamunyi has joined #heat | 22:20 | |
| *** shakamunyi has quit IRC | 22:21 | |
| *** sarob has joined #heat | 22:23 | |
| zaneb | back | 22:24 |
| *** derekh_ has joined #heat | 22:24 | |
| *** sarob_ has joined #heat | 22:25 | |
| jogo | shadower: ping | 22:26 |
| *** sarob has quit IRC | 22:28 | |
| *** shakamunyi has joined #heat | 22:28 | |
| *** shakamunyi has quit IRC | 22:28 | |
| *** shakamunyi has joined #heat | 22:29 | |
| *** arbylee has quit IRC | 22:30 | |
| *** arbylee has joined #heat | 22:31 | |
| *** IlyaE has quit IRC | 22:32 | |
| *** IlyaE has joined #heat | 22:32 | |
| *** sarob_ is now known as sarob | 22:36 | |
| *** arbylee has quit IRC | 22:42 | |
| *** arbylee has joined #heat | 22:44 | |
| *** mescanef has joined #heat | 23:01 | |
| *** blamar_ has joined #heat | 23:02 | |
| *** blamar has quit IRC | 23:04 | |
| *** blamar_ is now known as blamar | 23:04 | |
| *** achampion has joined #heat | 23:08 | |
| *** shakamunyi has quit IRC | 23:10 | |
| *** piyush has quit IRC | 23:13 | |
| *** amrita has joined #heat | 23:19 | |
| *** david-lyle has quit IRC | 23:19 | |
| *** kebray has quit IRC | 23:21 | |
| amrita | Hello - Right in the middle of creating a bunch of virtaul machines using heat - the metadata server on my cluster (intermitenttly) dies down 0r goes in to the unresponding state | 23:21 |
| amrita | this cause the whole user-data-plugin to fail its execution | 23:21 |
| amrita | anybody here seen something similar ? | 23:21 |
| amrita | i am on icehouse | 23:22 |
| *** IlyaE has quit IRC | 23:25 | |
| *** piyush has joined #heat | 23:26 | |
| *** IlyaE has joined #heat | 23:27 | |
| *** alexpilotti has quit IRC | 23:35 | |
| *** tango has quit IRC | 23:38 | |
| bodepd | how do you convert the refs output from a resource_group to a list of ips (I didn't see anything that looked like it woudl work like a map intrinsic function) | 23:40 |
| *** derekh_ has quit IRC | 23:41 | |
| *** IlyaE has quit IRC | 23:48 | |
| *** IlyaE has joined #heat | 23:51 | |
| *** dsneddon has quit IRC | 23:54 | |
| *** m_22 has joined #heat | 23:55 | |
| *** ccrouch has quit IRC | 23:56 | |
| *** lindsayk has quit IRC | 23:58 | |
| *** lindsayk has joined #heat | 23:58 | |
| *** daneyon has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!