1
0
Fork 0
mirror of https://github.com/fog/fog.git synced 2022-11-09 13:51:43 -05:00
Commit graph

27 commits

Author SHA1 Message Date
James Bence
f5dfebec84 Use specific error classes, not generic Fog::Compute::AWS::Error 2013-07-03 10:00:07 -07:00
James Bence
7fb0e3753e Refactor error handling
Match against the error object in both the old (against error.message)
and the new (against error.response.body); return a hash from this
method.

In the rescue block, try hard to raise an exception that includes the
code and message extracted from the error.
2013-07-02 19:39:23 -07:00
James Bence
b7b8367770 Inspect error.response.body, not error.message
Recent versions of excon have a middleware component,
Excon::Middleware::Expects and fog requests mostly record an expectation
of a 200 status code. Some calls to AWS return status other than 200 and
in some cases the error handling obscures the underlying error.

Current handling parsed error.message; this instance of error is
constructed by excon and includes the response as an attribute. The
message is always something like 'Expected(200) <=> Actual(404 Not Found)'
and so the parsing never succeeds.

Instead we now attempt to parse error.response.body which should allow
extraction of the underlying AWS Code value, which in turn will produce
an exception that points to the actual underlying cause.
2013-07-01 16:47:29 -07:00
Michael Hale
cc12069f5c [AWS | Autoscaling] dry up expected options 2013-06-10 12:44:45 -04:00
VirtualStaticVoid
5e3ee29e58 fix for AWS error message parsing 2013-03-22 11:45:05 +02:00
Amy Woodward
fcf829839d Add support for AWS Australia (ap-southeast-2). 2012-11-13 18:32:07 -08:00
Nick Osborn
f2dbf7efb6 [aws|auto_scaling] tags support
- implement create_or_update_tags
- implement delete_tags
- implement describe_tags
- don't capitalise keys in create_auto_scaling_group
2012-09-23 21:55:05 +01:00
Nick Osborn
d60ec0bff5 [aws|auto_scaling] improve notification configurations
- implement describe_notification_configurations
- implement mocks for (delete|put)_notification_configurations
- tests for *_notification_configurations
- delete notification configurations in delete_auto_scaling_group mock
2012-09-23 19:24:16 +01:00
Nick Osborn
58f572b56d [aws|auto_scaling] expose delete_notification_configuration request 2012-09-23 12:40:32 +01:00
Nick Osborn
a5701e9e19 [aws|auto_scaling] DRY out ARNs in mocks. 2012-09-22 18:51:49 +01:00
Nick Osborn
545c52a989 [aws|auto_scaling] update display_*_types
- implement describe_auto_scaling_notification_types
- implement describe_termination_policy_types
- make existing display_*_types parsers more robust
- split out display_*_types tests
2012-09-22 14:55:12 +01:00
Michael Hale
4d2b2a4d26 [aws|auto_scaling] Add instrumentation support. 2012-09-11 10:03:17 -04:00
Trotter Cashion
d0f947f07f Merge branch 'master' into aws-launch-configuration-parameters 2012-06-25 13:39:29 -07:00
Frederick Cheung
38c75c85ab start an instance with an IAM profile and access the credentials 2012-06-20 23:17:46 +01:00
Trotter Cashion
eab5d489ed Add better ScalingPolicy support 2012-06-15 11:10:04 -07:00
James Herdman
97b16d08d2 Remove absolute path when requiring 2012-05-26 11:27:28 -04:00
James Herdman
d7539ea27a Remove unused variable 2012-05-26 11:27:27 -04:00
Zuhaib M Siddique
813eaecdbb [AWS|Auto Scale] Add support for put_notification_configuration and
change AWS API to use 01-01-2011 Spec
2012-04-21 00:09:21 -07:00
Pavel Repin
d717e8af69 Nix hardcoded regions: Autoscaling. 2012-02-02 15:38:23 -08:00
Nick Osborn
a01e149085 [aws|autoscaling] allow sa-east-1 region in mocks 2011-12-17 12:47:44 +00:00
thattommyhall
e239c9e5ad added sa-east-1 region 2011-12-15 12:37:31 +00:00
geemus
cb3942d633 [aws] add us-west-2 region 2011-11-09 11:57:48 -06:00
geemus
ffd01e51dd [core] pass connection_options through service init
closes #411
2011-09-12 10:01:56 -05:00
Jens Braeuer
d06ddbe7f3 [compute|aws] Fix NameError.
Exception handling seems to be been forgotten, when AWS moved in Fog::Compute namespace.
2011-09-09 16:50:04 +02:00
geemus
b2c8ff0388 [core] work toward separate requires
Should facilitate the following:

    require 'fog/aws'
    require 'fog/compute'
    require 'fog/aws/compute'

I think once you `require 'rubygems'` those should work.

further cleanup should reduce common requires (ie nokogiri)
2011-08-31 15:52:53 -05:00
Nick Osborn
36601e796d [aws|compute] improve model and tests 2011-06-21 22:15:33 +01:00
Nick Osborn
9afb8c8460 [aws|auto_scaling] implement 2010-08-01 API 2011-06-12 21:07:42 +01:00