]>
review.fuel-infra Code Review - openstack-build/heat-build.git/log
Jenkins [Fri, 31 Aug 2012 15:24:36 +0000 (15:24 +0000)]
Merge "Connect to heat client, add accessor method, and fix ip check"
Jenkins [Fri, 31 Aug 2012 15:23:30 +0000 (15:23 +0000)]
Merge "Add new methods to get access to nova/glance clients"
Jenkins [Fri, 31 Aug 2012 15:22:08 +0000 (15:22 +0000)]
Merge "Make sure the functional test always cleans up"
Steven Hardy [Thu, 30 Aug 2012 12:56:47 +0000 (13:56 +0100)]
heat engine : Add set_watch_state engine RPC action
Add set_watch_state which allows a watch/alarm state
to be temporarily overridden, simplified asynchronous
version following review/discussion
Change-Id: I9f1424007fc16d1cec2f7dc504600455fe5ab3bb
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Wed, 29 Aug 2012 14:11:06 +0000 (15:11 +0100)]
heat engine : Expose valid watch states via engine.api
Expose valid watch states via engine.api
Change-Id: I26a1d8939ce75a0be88e646858f3f3b4feb5e8d0
Signed-off-by: Steven Hardy <shardy@redhat.com>
Jeff Peeler [Fri, 31 Aug 2012 05:19:39 +0000 (01:19 -0400)]
Connect to heat client, add accessor method, and fix ip check
Search server list for IP based on the PhysicalResourceId, not an arbitrary
server name. Wait until stack create event before obtaining value.
Change-Id: Ie9e3df5369132dbf668f8b42cd10a7f2faec1922
Signed-off-by: Jeff Peeler <jpeeler@redhat.com>
Jeff Peeler [Thu, 30 Aug 2012 15:38:09 +0000 (11:38 -0400)]
Add new methods to get access to nova/glance clients
Change-Id: I5be8d7d57d021635422035742f59acdaeb8b409a
Signed-off-by: Jeff Peeler <jpeeler@redhat.com>
Tomas Sedovic [Thu, 30 Aug 2012 13:44:41 +0000 (15:44 +0200)]
Make sure the functional test always cleans up
Change-Id: I740f676452905d3aceb3d2df37e4c4cdb6b81d7e
Signed-off-by: Tomas Sedovic <tomas@sedovic.cz>
Steven Hardy [Tue, 28 Aug 2012 18:25:28 +0000 (19:25 +0100)]
heat API : add get_param_value to API utils
Add get_param_value helper to common aws API utils
Change-Id: I69ab3ed1e34f8691457f3d986b4af06b1d381039
Signed-off-by: Steven Hardy <shardy@redhat.com>
Jeff Peeler [Wed, 29 Aug 2012 17:30:47 +0000 (13:30 -0400)]
Add new arguments to run_tests
Execute unit tests and pep8 by default. Allow tests to be chosen in a more
additive fashion. Cleaned up unnecessary let statements and variable
defines.
Change-Id: I40676c5d0deda7bd3abc4bdadfd404f5909c166e
Signed-off-by: Jeff Peeler <jpeeler@redhat.com>
Zane Bitter [Wed, 29 Aug 2012 17:12:48 +0000 (19:12 +0200)]
Refactor reformat_dict_keys()
This way is a little more Pythonic.
Change-Id: Ib48d0e9b7e8a4ed6e220c645a29510c893e97419
Signed-off-by: Zane Bitter <zbitter@redhat.com>
Zane Bitter [Wed, 29 Aug 2012 17:35:12 +0000 (19:35 +0200)]
Refactor extract_param_pairs()
Implement it in terms of the more general extract_param_list().
Change-Id: Ifbf0dd8205da87e707610c0b8cb0ca616d7b2e22
Signed-off-by: Zane Bitter <zbitter@redhat.com>
Zane Bitter [Wed, 29 Aug 2012 14:10:31 +0000 (16:10 +0200)]
Refactor extract_param_list()
Change-Id: Ia338c7c4a17af90885b1c83297d32a61d2177ecd
Signed-off-by: Zane Bitter <zbitter@redhat.com>
Jeff Peeler [Tue, 28 Aug 2012 19:23:44 +0000 (15:23 -0400)]
Add link to RPM repository in getting started
Change-Id: I79fcc10b2594b977097d7d91022cbd50adcc9167
Signed-off-by: Jeff Peeler <jpeeler@redhat.com>
Steven Hardy [Thu, 23 Aug 2012 17:21:07 +0000 (18:21 +0100)]
heat API : Add api.aws.utils.extract_param_list
Add new common helper function which extracts
AWS parameter lists
Change-Id: Ic77ff6014a4f39d4fb5d59a1391f87cdd94f0f7f
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Tue, 28 Aug 2012 08:35:20 +0000 (09:35 +0100)]
heat tests : rename test_stacks
Rename test_stacks.py to test_engine_manager.py
as this better reflects what is being tested
Change-Id: I612351286fbb008b2be3db7519f0a24ab9d2d8e2
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Tue, 28 Aug 2012 08:33:01 +0000 (09:33 +0100)]
heat tests : add test for new manager show_watch_metric method
Add unit tests for new manager show_watch_metric action
Change-Id: Ie1b7c0a0e1eb5c7e5fad74674b1a9669bf7c9a1c
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Fri, 24 Aug 2012 15:33:19 +0000 (16:33 +0100)]
heat tests : add tests for new manager show_watch method
Add unit test for new show_watch engine action
Change-Id: I00153f258687e02837b901c399df51085bb34303
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Fri, 24 Aug 2012 15:30:39 +0000 (16:30 +0100)]
heat engine : bugfix show_watch, fix single-watch mode
When called with watch_name!=None we need to wrap the
watch_rule_get response in a list
Change-Id: I83bbe816758767afd3eb49b090b0b652036cad60
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Fri, 24 Aug 2012 12:22:16 +0000 (13:22 +0100)]
heat tests : add new engine rpcapi unit tests
Add tests for new watch engine methods
Change-Id: Icbfc900d7578d7d0f8cd9024d8a5b57a9695c0a4
Signed-off-by: Steven Hardy <shardy@redhat.com>
Jeff Peeler [Mon, 27 Aug 2012 20:33:59 +0000 (16:33 -0400)]
Add test-requires in RPM package list form
This is to allow automated installation of dependences since functional
tests are not going to be run with packages from pip.
Change-Id: Ia9f74bc353bab79d7c78cb086210d8628d701387
Signed-off-by: Jeff Peeler <jpeeler@redhat.com>
Jenkins [Mon, 27 Aug 2012 14:46:51 +0000 (14:46 +0000)]
Merge "Functional test improvements"
Jenkins [Mon, 27 Aug 2012 09:54:20 +0000 (09:54 +0000)]
Merge "Fix the Timeout/Interval check in the LB."
Jenkins [Mon, 27 Aug 2012 08:35:23 +0000 (08:35 +0000)]
Merge "Set the max url length in eventlet 0.9.17"
Tomas Sedovic [Wed, 22 Aug 2012 14:45:27 +0000 (16:45 +0200)]
Set the max url length in eventlet 0.9.17
The version with configurable url limit was released and packaged for
Fedora 17.
The monkey-patch does not work on new eventlet.
ref #55
fixes #197
Change-Id: I728cc3b0851261667bb9b07220c855bced1344a4
Signed-off-by: Tomas Sedovic <tomas@sedovic.cz>
Jeff Peeler [Mon, 27 Aug 2012 06:18:53 +0000 (02:18 -0400)]
Functional test improvements
-Move everything out of init so unit tests are not affected.
-Make JEOS creation part of utils class, now tests can choose
which JEOS to use.
-Only attempt to create JEOS if a suitable one is not found in glance.
Change-Id: I162e7447ae1fed17855bdca8eba16ac7f952db01
Signed-off-by: Jeff Peeler <jpeeler@redhat.com>
Angus Salkeld [Mon, 27 Aug 2012 05:09:06 +0000 (15:09 +1000)]
Fix the Timeout/Interval check in the LB.
http://docs.amazonwebservices.com/ElasticLoadBalancing/latest/APIReference/API_HealthCheck.html
Fixes: #198
Change-Id: I3ec3e5a946e32695f3f9b0930b922837ab80ea66
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Angus Salkeld [Mon, 27 Aug 2012 04:02:26 +0000 (14:02 +1000)]
Fix the way self.properties is checked for a value.
The following only checks if the key is in the properties, and
all schema keys are, so it is not the way to check if a value
has been set.
if 'DesiredCapacity' in self.properties:
change to:
if self.properties:
Fixes: #199
Change-Id: I6bcb3e74420031532dc249aafe85d5a428d0a80e
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Steven Dake [Mon, 27 Aug 2012 01:09:43 +0000 (18:09 -0700)]
Make RDS (mysql) wait until setup is finished before proceeding
It is possible a service dependent on RDS may issue commands to
the RDS service before mysql has finished installing. Added
a wait condition to force synchronization.
Change-Id: I6abf2d62e92d19945c587deeb61605df2c97ec85
Signed-off-by: Steven Dake <sdake@redhat.com>
Steven Dake [Mon, 27 Aug 2012 00:00:12 +0000 (17:00 -0700)]
Put selinux in permissive mode by default via a runcmd in cloudconfig
Change-Id: I7db02a967de9a3e85876901163f7611196a9429f
Signed-off-by: Steven Dake <sdake@redhat.com>
Steven Hardy [Thu, 23 Aug 2012 15:14:43 +0000 (16:14 +0100)]
heat API : make extract_user_params more generic
Make api.aws.utils.extract_user_params into a more
generic aws-format-list-extraction function
Change-Id: If6811ea1a138ddff96542f983397852dfbd483fd
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Wed, 22 Aug 2012 13:11:52 +0000 (14:11 +0100)]
heat engine : Add show_watch_metric RPC action
Adds show_watch_metric, which allow alarm/watch
metric data to be retrieved from the engine
Change-Id: Ie69691dbf781016e6262bbbabbe57f9c3396a5d5
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Tue, 21 Aug 2012 16:33:49 +0000 (17:33 +0100)]
heat engine : add show_watch RPC method
Add new show_watch RPC method, to allow retrieval
of information related to watches for cloudwatch
Change-Id: Ib434c80cf6cfab7f2cba972d5316f0ed2bbcbf30
Signed-off-by: Steven Hardy <shardy@redhat.com>
Jenkins [Thu, 23 Aug 2012 23:09:22 +0000 (23:09 +0000)]
Merge "Split functional test into standalone and utility class"
Jeff Peeler [Tue, 21 Aug 2012 20:26:41 +0000 (16:26 -0400)]
Split functional test into standalone and utility class
This allows functional tests to be written for each individual template.
See the test_Wordpress_Single_Instance for an example, but the basic
form to follow is:
import util
def test_template():
func_utils = util.FuncUtils()
func_utils.create_stack(template, 'F17')
func_utils.check_cfntools()
func_utils.wait_for_provisioning()
func_utils.check_user_data(template)
ssh = func_utils.get_ssh_client()
# test stuff here
func_utils.cleanup()
Functional test changes:
Changed jeos creation to use new tool.
Added distribution to heat stack creation.
Improved IP parsing from glance.
Improve SSH connection exception handling.
Changed cfntools SHA check to match installed copy instead of locally.
Fixed stack parsing to check correct file and handle user-data injection.
Clean up run_tests a little to make it clear default execution is on host.
Add a short option for unit test running only (may be important due to not
wanting to run functional tests locally).
Also added paramiko to test-requires.
Change-Id: Ib2ae1be32801cb0ee3d1937c4f82ab9f574b4591
Signed-off-by: Jeff Peeler <jpeeler@redhat.com>
Steven Hardy [Wed, 22 Aug 2012 19:14:11 +0000 (20:14 +0100)]
heat DB : make watch_data_get_all work with no watch ID
Align watch_data_get_all with other "all" methods, and
return all watch data without requiring an ID. We may
also need by_id and by_name variants but this is more
consistent and makes things easier for cloudwatch.
Change-Id: I09c90278349efc769ab7c719f686a1fd854b425a
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Wed, 22 Aug 2012 08:54:08 +0000 (09:54 +0100)]
heat API : Add HeatAPINotImplementedError exception
Add heat-specific HeatAPINotImplementedError, which
allows us to return a sensible error to requests for
API actions which have not yet been implemented
Change-Id: Ibaee8312e7e563d29801bd325d7638bca9efa622
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Tue, 21 Aug 2012 10:52:24 +0000 (11:52 +0100)]
heat API : Move aws api common code into aws/utils.py
Move heat-api AWS common utility functions into a new
utils.py, so these functions can be reused by cloudwatch
Change-Id: I030d796b1048ffc4e7c40f7c8760121ab2854733
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Tue, 21 Aug 2012 10:14:18 +0000 (11:14 +0100)]
heat API : move remote_error to common exception.py
Move API exception-mapping function to the common exception.py
Change-Id: If64511be0d77d9c9f9e610bbb0564d57f4530499
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Tue, 21 Aug 2012 10:01:25 +0000 (11:01 +0100)]
heat API : move aws common files from api/v1 to api/aws
Move aws api common files to common directory
(so they can be more easily reused by cloudwatch)
Change-Id: I1a455ef11226dd960503bac5d79fa5c28607a1f6
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Tue, 21 Aug 2012 09:48:33 +0000 (10:48 +0100)]
heat engine : Fix engine.api variable shadowing
Fix weird-looking/confusing variable shadowing in
the engine api
Change-Id: Ic7aca18ef877149bbdb49ee33abc62241706a3f3
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Mon, 20 Aug 2012 12:01:23 +0000 (13:01 +0100)]
heat tools : glance-jeos-add-from-github avoid duplicates
Avoid adding images to glance when they already exist
Change-Id: Ia914cc3015fcdcc0c12e3773985298fce6872751
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Mon, 20 Aug 2012 11:46:23 +0000 (12:46 +0100)]
heat tools : glance-jeos-add-from-github scrape image names
Scrape image names from github index page to avoid hard-coded
names in the script
Change-Id: Ifca85b71a1c20d436486a80cc5e9347a7791e5fa
Signed-off-by: Steven Hardy <shardy@redhat.com>
Jeff Peeler [Fri, 17 Aug 2012 19:36:28 +0000 (15:36 -0400)]
Inject command into userdata
The injected command touches /var/lib/cloud/instance/provision-finished
so it is easily determinable if provision has been completed.
Change-Id: Ib5b93f7bd14f533af41182ffdab6a029a3b9e13a
Signed-off-by: Jeff Peeler <jpeeler@redhat.com>
Steven Dake [Fri, 17 Aug 2012 17:11:21 +0000 (10:11 -0700)]
Add a tool to register prebuilt JEOS images into glance from github
Change-Id: Idb02e892f390516557c9cce3ca0fbab7109d33e1
Signed-off-by: Steven Dake <sdake@redhat.com>
Steven Hardy [Wed, 15 Aug 2012 16:11:56 +0000 (17:11 +0100)]
heat template : Align two Openshift template varieties
Align the build-from-scratch Openshift.template with the
now-working OpenShift_Prebuilt_JEOS.template
Change-Id: I8d164b0837b4cb28a6ce4ad1573b19f8daf68b8a
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Dake [Thu, 16 Aug 2012 17:50:05 +0000 (10:50 -0700)]
Use cached keystoneclient
Change-Id: I787edd10bcc29746a08878fd371204f81f0400db
Signed-off-by: Steven Dake <sdake@redhat.com>
Steven Hardy [Wed, 15 Aug 2012 14:16:23 +0000 (15:16 +0100)]
heat cli : Workaround inconsistent boto return type
Work around the inconsistent boto return type for DescribeStackResource
action, upstream patch pending but not yet merged, so this provides
a simple workaround
Fixes #175
Change-Id: I026ec7b1845fb591a47a5fb12cfcb25705b33909
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Wed, 15 Aug 2012 13:09:54 +0000 (14:09 +0100)]
heat cli : Rework to separate cli tool from client-API wrappers
Rework to remove duplication between heat and heat-boto, and to
provide better separation between the CLI tool logic and the
underlying client API (should allow easier porting to new ReST API)
Ref #175 (partially fixes)
Fixes #192
Change-Id: Ib1f821667c40c78770a345204af923163daeffae
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Tue, 14 Aug 2012 16:27:16 +0000 (17:27 +0100)]
heat cli : remove some duplication from parameter formatting
Cleanup - bin/heat move some duplicated code into a function
Change-Id: Ib74077c557a2b8d7c640ee1686237bb5223b2a85
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Tue, 14 Aug 2012 16:15:08 +0000 (17:15 +0100)]
heat cli : remove unused paths from heat cli tool
Cleanup the the jeos and cfntools paths which are no longer needed
Change-Id: Idc7d37446a0072e38369939cbd3fd5fb374c15c9
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Dake [Tue, 14 Aug 2012 16:07:44 +0000 (09:07 -0700)]
Update Getting started to match change to heat-jeos
heat-jeos list will no longer list jeos in the image name. As a result,
we can't specify "-jeos" in the getting started guide.
Signed-off-by: Steven Dake <sdake@redhat.com>
Steven Hardy [Tue, 14 Aug 2012 14:56:13 +0000 (15:56 +0100)]
heat api : bugfix, save config correctly in EC2Token
Fix issue where EC2Token fails to save the config correctly,
which breaks AWS Authentication
Fixes #191
Change-Id: I77dfe182aac81c55b7d487d88073b40eac1faadd
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Mon, 13 Aug 2012 13:31:12 +0000 (14:31 +0100)]
heat templates : Add openshift template for pre-built jeos
Add new openshift template which uses a pre-built JEOS
also creates the example application allowing a fairly
quick full demo of a two-instance openshift setup
Fixes #186
Change-Id: I9bd886b0717c8b0d810eff036b76883b537c2421
Signed-off-by: Steven Hardy <shardy@redhat.com>
Zane Bitter [Mon, 13 Aug 2012 11:54:52 +0000 (13:54 +0200)]
Getting Started: Add configuration info for the metadata server
Change-Id: Ie99bfd1c5e7bcaedd7757872d7d618d2cf29b137
Signed-off-by: Zane Bitter <zbitter@redhat.com>
Zane Bitter [Fri, 10 Aug 2012 17:16:01 +0000 (19:16 +0200)]
Getting Started: Use install.sh script
Change-Id: I7f189f8d3e0557503eca23a5489a98589bfff4f3
Signed-off-by: Zane Bitter <zbitter@redhat.com>
Tomas Sedovic [Mon, 13 Aug 2012 13:59:49 +0000 (15:59 +0200)]
Make template params resolution standalone
This makes the two functions for resolving template data (Fn::FindInMap,
Ref, etc.) available outside of a Stack object.
Other programs can then import these functions and see how the template
parameters get resolved.
Change-Id: I92de937a3fb25f907404335cb71554036a807c50
Signed-off-by: Tomas Sedovic <tomas@sedovic.cz>
Jenkins [Fri, 10 Aug 2012 19:06:22 +0000 (19:06 +0000)]
Merge "change amd64 reference to x86_64 to match latest cfntools"
Steven Dake [Fri, 10 Aug 2012 18:39:31 +0000 (11:39 -0700)]
change amd64 reference to x86_64 to match latest cfntools
Change-Id: I11f4bd974326c7f1b4e287389b22c5ec3f20bdf9
Signed-off-by: Steven Dake <sdake@redhat.com>
Steven Dake [Wed, 8 Aug 2012 21:58:51 +0000 (14:58 -0700)]
Remove crankcase patch since fix was merged upstream
Added a patch to force yum to retry indefinately on crankcase builds
because the OpenShift repositories were too busy. I submitted this upstream
in:
https://github.com/openshift/crankcase/pull/336
and it was merged.
As a result, removing this workaround.
Change-Id: I26e0a162d5cfb4bc23843c93678318ff21610973
Signed-off-by: Steven Dake <sdake@redhat.com>
Angus Salkeld [Mon, 6 Aug 2012 07:17:31 +0000 (17:17 +1000)]
Add calculate_properties() to update() and restart_resource()
This is the same fix as
783b1112490ef07db05251c1258c04f8b1caa5a0
Change-Id: I17a30b0c670dee5b228b4af8dad06ad109471e5d
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Jenkins [Sat, 4 Aug 2012 03:14:00 +0000 (03:14 +0000)]
Merge "Log all startup operations to /var/log/heat-startup.log"
Steven Dake [Sat, 4 Aug 2012 02:31:22 +0000 (19:31 -0700)]
Force rpm to retry indefinately during crankcase build
If RPM is not forced, sometimes packages will fail to download within the
default 10 retries. This results in crankcase failing to build OpenShift.
Change-Id: I5174928fc2ac205dcd7818ecf1da6b8e32f48e08
Signed-off-by: Steven Dake <sdake@redhat.com>
Steven Dake [Fri, 3 Aug 2012 23:42:46 +0000 (16:42 -0700)]
Log all startup operations to /var/log/heat-startup.log
Change-Id: I0e7f415e7aee60c09d0b5b172742af7a5b313191
Signed-off-by: Steven Dake <sdake@redhat.com>
Steve Baker [Thu, 2 Aug 2012 05:32:07 +0000 (17:32 +1200)]
Port existing rpc calls to use the new RPC client API.
Patch 2/2, this implements github heat issue 181.
test_api_v1 was modified to handle the extra rpc arguments.
metadata_register_address hasn't been included in EngineAPI, and it
possibly doesn't deserve to. This means that bin/heat-metadata has the
only remaining raw rpc call.
Steve Baker [Fri, 27 Jul 2012 09:57:59 +0000 (21:57 +1200)]
Implements a client side engine RPC API.
Patch 1/2, this implements github heat issue 181.
The topic string for the rpc call is built from the cfg.CONF parameters 'host'
and 'engine_topic' defined in heat.common.config.engine_opts.
delete_stack seems to be the only method which returns nothing, so it can be
invoked as cast or call, with cast being the default.
The tests exercise each api method with the same approach found in
nova/nova/tests/compute/test_rpcapi.py
Change-Id: Ia20850821083d1236ad628172db00d01f073313c
Steven Hardy [Wed, 1 Aug 2012 13:27:55 +0000 (14:27 +0100)]
heat templates : update Openshift template to add node instance
Update openshift templat to add a second instance running
the node to the existing broker. Move to crankcase master
and attempt to work around build issues.
Ref #186
Change-Id: If7d1b39e1072d3934f37817be3d7773edd7785c3
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Thu, 2 Aug 2012 13:37:13 +0000 (14:37 +0100)]
heat : Make instance flavors consistent
Add additional instance flavors to align templates with
nova
Fixes #179
Change-Id: I586e57748c9b0a0c7594746399d5328da788efcd
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Thu, 2 Aug 2012 12:15:53 +0000 (13:15 +0100)]
heat engine : raise appropriate error for incorrect stack_name
Raise an AttributeError instead of returning a string when passed
an incorrect stack name, this then maps to the appropriate error
in the API.
Fixes #182
Change-Id: I7fe80496883bfbe515d9c235de812aaa14c41c12
Signed-off-by: Steven Hardy <shardy@redhat.com>
Steven Hardy [Fri, 27 Jul 2012 13:43:37 +0000 (14:43 +0100)]
heat API : Return correct AWS error response for invalid parameter
Passing template parameter values which are not in AllowedValues
makes the engine throw a ValueError exception, which should be
mapped back to HeatInvalidParameterValueError so that the API
error response is correctly formatted.
Fixes #180
Change-Id: I8615cb7fbbbfd21a403a4973fe0a63300c6cf852
Signed-off-by: Steven Hardy <shardy@redhat.com>
Tomas Sedovic [Tue, 31 Jul 2012 12:50:17 +0000 (14:50 +0200)]
Bump to v6
Heat version 5 is released, let's start working on 6.
Change-Id: Icd0db5c0a9317d17ae4c2f161d50da8b5c50c40e
Signed-off-by: Tomas Sedovic <tomas@sedovic.cz>
Steven Hardy [Mon, 30 Jul 2012 17:53:23 +0000 (18:53 +0100)]
heat engine : Avoid writing to class-scope parameters schema
Fixes issue where multiple instances of the same resource types
which have different parameters break, because the parser is
writing back to the class-scope properties_schema which should
be immutable. This patch fixes by making a per-instance copy.
Fixes #183
Change-Id: Ia29f67465acbcfaf8dfe511ddaa9075bc48157ad
Signed-off-by: Steven Hardy <shardy@redhat.com>
Tomas Sedovic [Mon, 30 Jul 2012 15:56:44 +0000 (17:56 +0200)]
Update getting started with credential crypto
We encrypt the sensitive information in the database now. This updates
the Getting Started guide to generate an encryption key.
Change-Id: I44a96302e736997a747f82c9d23006425396633f
Signed-off-by: Tomas Sedovic <tomas@sedovic.cz>
Ian Main [Fri, 27 Jul 2012 16:41:25 +0000 (09:41 -0700)]
Add tools directory to python package.
This adds the tools/ dir to the tarball for distribution.
Change-Id: I82a18df568bf2d7421a49e057e80a7bc236e2078
Signed-off-by: Ian Main <imain@redhat.com>
Tomas Sedovic [Fri, 27 Jul 2012 12:37:17 +0000 (14:37 +0200)]
Don't use uuidgen in install.sh
uuidgen is not present in Ubuntu. This simply reads 16 bytes from
/dev/random and stores their hex representation.
Change-Id: If1152242b024218f20bd4c787e03836f0d5f49cf
Signed-off-by: Tomas Sedovic <tomas@sedovic.cz>
Ian Main [Thu, 26 Jul 2012 19:57:57 +0000 (12:57 -0700)]
Add encryption support for authentication information in db.
This patch uses an encryption key generated in install.sh to perform
symmetrical encryption on sensitive authentication information stored
in the database for HA operations.
Change-Id: Ifd09f3f566ba3ebd941a6f453953576011b518b9
Signed-off-by: Ian Main <imain@redhat.com>
Ian Main [Wed, 25 Jul 2012 19:02:53 +0000 (12:02 -0700)]
Change to per-tenant stacks.
This patch switches from per-user stacks to per-tenant stacks.
Change-Id: Ia7ac65033560bcff1db83d4d673945029e06ea6a
Signed-off-by: Ian Main <imain@redhat.com>
Angus Salkeld [Wed, 25 Jul 2012 11:45:09 +0000 (21:45 +1000)]
Make a template that demonstrates instance monitoring and restarting.
We use cfn-push-stats to send a heartbeat and if we don't get
it within the set interval we restart the instance.
Other changes to make this work:
- clear the waitcondition when it is deleted.
- set the resource id to None when deleted, else it won't re-create
properly.
- don't run watch alarms if the stack is not completed.
Change-Id: I5dfc8b372f557cf43379c6c5b7436d1010f83e3c
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Angus Salkeld [Wed, 25 Jul 2012 02:36:35 +0000 (12:36 +1000)]
Call calulate_properties() before calling FnGetAtt()
The problem is that in the delete the order of resolution is reversed
so in delete loop we have not yet done a calulate_properties() on the
resource that we are about to call FnGetAtt() for.
Fixes #178
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Change-Id: I8af6f8a78086a7b1f2bcc280d57af0b4311f08cc
Angus Salkeld [Tue, 24 Jul 2012 09:04:07 +0000 (19:04 +1000)]
Log the error when a resouce fails to delete
Also bail out of handle_delete if instance_id is None
Change-Id: I2bd77345ac35860201006a23999a06dd81ac6fe0
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Steven Hardy [Tue, 24 Jul 2012 16:58:06 +0000 (17:58 +0100)]
heat API : Add more tests covering error paths
Add some more tests testing error path responses
Change-Id: I10f3b599f8a0f5daad84752042079a591968de09
Signed-off-by: Steven Hardy <shardy@redhat.com>
Angus Salkeld [Tue, 24 Jul 2012 06:48:55 +0000 (16:48 +1000)]
delete duplicate validate() method
probably a merge conflict.
Change-Id: Ibe8941ab471943b7385ceb4b4d4559719a7092b6
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Angus Salkeld [Tue, 24 Jul 2012 04:09:29 +0000 (14:09 +1000)]
loadbalancer: implement Interval and Timeout
Change-Id: I74abc742371cb23ddfe38f83812662a9d42f3d43
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Angus Salkeld [Tue, 24 Jul 2012 04:03:24 +0000 (14:03 +1000)]
autoscaling: implement DesiredCapacity
Change-Id: I1403f7857e03795aa4773f9a3f1fc8f082526477
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Jenkins [Tue, 24 Jul 2012 04:54:19 +0000 (04:54 +0000)]
Merge "Get rid of PyCrypto dependency"
Zane Bitter [Mon, 23 Jul 2012 23:39:41 +0000 (19:39 -0400)]
Update GettingStarted guide to use correct state name
This changed from IN_PROGRESS to CREATE_IN_PROGRESS in
0191587ac6a79509576e2043412020d9c4257b47 .
Change-Id: If1cb20f6a16e719ee423db937b591b575fd4a038
Signed-off-by: Zane Bitter <zbitter@redhat.com>
Jeff Peeler [Mon, 23 Jul 2012 22:15:49 +0000 (18:15 -0400)]
Get rid of PyCrypto dependency
The origin of this code is from Glance and is not used in Heat.
Change-Id: I4a22974e24fb277aea79212f0aee2f7e85a0ea76
Signed-off-by: Jeff Peeler <jpeeler@redhat.com>
Jenkins [Mon, 23 Jul 2012 17:47:48 +0000 (17:47 +0000)]
Merge "Add AWS::RDS::DBInstance"
Steven Hardy [Mon, 23 Jul 2012 16:43:39 +0000 (17:43 +0100)]
heat API : Add more unit tests
Add more unit tests to improve API test coverage
Change-Id: I5f3a8933a9a09acb6b904ccc76ec20ad210a5974
Signed-off-by: Steven Hardy <shardy@redhat.com>
Tomas Sedovic [Wed, 18 Jul 2012 13:39:37 +0000 (15:39 +0200)]
Add AWS::RDS::DBInstance
Fixes #163
This is an initial implementation of the DBInstance resource type and a
sample Wordpress template showing it off.
Change-Id: I5e156dc58eee563ae9de068664bafa1af1fb5ffe
Signed-off-by: Tomas Sedovic <tomas@sedovic.cz>
Angus Salkeld [Mon, 23 Jul 2012 10:12:05 +0000 (20:12 +1000)]
autoscaling: don't kill all instances when scaling down
only delete the difference between the old capacity and the new capacity
doh!
Change-Id: Idbb16b2fa93abcad2793d5b4365c18a01e588b99
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Angus Salkeld [Mon, 23 Jul 2012 04:58:11 +0000 (14:58 +1000)]
make the autoscaling template easier to debug
shorten the period and lower the thresholds
Change-Id: I3d621fadacc6e77002d100cf725d970671475106
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Angus Salkeld [Mon, 23 Jul 2012 04:48:29 +0000 (14:48 +1000)]
Return the name from the autoscaling group not the instance.
the default is to return an instance_id if it is not none.
Change-Id: Ide5dcb3da5d992683dd235f1b53b264455a9fd07
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Angus Salkeld [Mon, 23 Jul 2012 03:09:19 +0000 (13:09 +1000)]
Add python-psutil and fix cfn-push-stats options
Change-Id: Ic88e1e886c5f8ccbcbc5bc49fa612366c7faeae5
Signed-off-by: Angus Salkeld <asalkeld@redhat.com>
Steven Dake [Fri, 20 Jul 2012 17:40:22 +0000 (10:40 -0700)]
Add an EC2Token_filter_factory and use it as the default paste filter
Change-Id: I70989ff6be655832497e1e8a3019497438a739fb
Signed-off-by: Steven Dake <sdake@redhat.com>
Steven Hardy [Tue, 17 Jul 2012 14:46:49 +0000 (15:46 +0100)]
heat api/engine : Implement UpdateStack functionality
Implements initial support for UpdateStack, currently
all resources default to delete/create on update.
Ref #171
Change-Id: I3e6e63143d554c21ccdee19879c4dfb8b6e693d7
Jenkins [Fri, 20 Jul 2012 14:20:23 +0000 (14:20 +0000)]
Merge "heat db : fix per-object delete"
Steven Hardy [Fri, 20 Jul 2012 10:17:08 +0000 (11:17 +0100)]
heat db : fix per-object delete
Fixes problem where per-resource destroy() operations do not
delete the underlying database row because the model delete
method is broken
Change-Id: I3562c9048f8d4e54c1ea51090e74abe982ffb330
Signed-off-by: Steven Hardy <shardy@redhat.com>
Angus Salkeld [Fri, 20 Jul 2012 06:08:32 +0000 (16:08 +1000)]
Combined autoscaling and loadbalancer
Change-Id: Ib2cff867eb5d9fbee8cedb828e9c35a93436cbe5
Steven Dake [Fri, 20 Jul 2012 02:51:47 +0000 (19:51 -0700)]
Optimize filter context
Just use a direct filter factory for ContextMiddleware
Change-Id: Ie17bb88c331bdb4354d0abb24346ab80f13cd323
Signed-off-by: Steven Dake <sdake@redhat.com>