search

rackspace-cookbooks / rackspace_cloud_monitoring

DEPRECATED, use https://github.com/rackspace-cookbooks/rackspace_monitoring instead
Other
1 stars 1 forks source link

Circle ci #13

Closed jujugrrr closed 9 years ago

jujugrrr commented 9 years ago

Adding dependencies so CircleCi can pass.

Somehow something change in Chef 12.1.0 preventing the test about raising exception to pass. which is why I've done 1e15658.

I tried to reproduce the issue on a clean repo but it doesn't work, I'm wondering if somehow now Chef intercepts exception from LWRP and return an error instead ( I was not using LWRP when trying to reproduce the issue) :

with Chef 12.0.3 : 

[2015-03-17T11:29:39+00:00] DEBUG: Platform ubuntu version 14.04 found
[2015-03-17T11:29:39+00:00] INFO: Running queued delayed notifications before re-raising exception

With Chef 12.1.0 : 

[2015-03-17T11:22:53+00:00] INFO: Running queued delayed notifications before re-raising exception
[2015-03-17T11:22:53+00:00] ERROR: Converge failed with error message rackspace_cloud_monitoring_service[default] (rackspace_cloud_monitoring_service_test::default line 3) had an error: RuntimeError: Cloud credential api_key missing, cannot setup cloud-monitoring (please set :cloud_credentials_api_key)

@martinb3 Maybe you are aware of something ?

martinb3 commented 9 years ago

I diagnosed the exception not being raised down to a change in chef, then found these: https://github.com/chef/chef/issues/3078 https://github.com/sethvargo/chefspec/pull/572