Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't seem to get AWS/EC2 template working #1

Closed
sacreman opened this issue Aug 6, 2014 · 4 comments
Closed

Can't seem to get AWS/EC2 template working #1

sacreman opened this issue Aug 6, 2014 · 4 comments

Comments

@sacreman
Copy link

sacreman commented Aug 6, 2014

Hi, I can get AWS/RDS namespace stuff working but I can't seem to get anything out of AWS/EC2. Here's the relevant bit of config:

[[Accounts.Templates]]
Namespace = "AWS/EC2"
Region = "eu-west-1"
Backfill = 300
Period = 60
Interval = 10
CustomKeyFormat = "custom.ec2.%0.%m"
Statistics = ["Sum"]
DimensionList = [
  ["InstanceId",""]]

I changed the DimensionList to InstanceId (with a lowercase d) to see if that would help as it lists it like that in the AWS docs.

With it set as InstanceID I get no metrics returned.

With it set as InstanceId I get no metrics returned plus an error:

Type: Sender, Code: InvalidParameterValue, Message: Invalid nextToken
@ProTip
Copy link
Owner

ProTip commented Aug 8, 2014

Hello sacreman! I apologize this slipped through my inbox, and this project is a little rough as you and I are possibly the only two people to ever use it :)

I have actually switched this back to using the crowdmob goamz project, so you might want to pull and build again to see if that resolves your issue.

I'm getting back all the EC2 information myself however I don't believe I have more than 500 total. Over that and a "nextToken" will be returned. The goamz package attempts to consume the extra result sets however there may be a bug in there. I'm going to see if I can trigger the error and fix it.

@ProTip
Copy link
Owner

ProTip commented Aug 8, 2014

I believe I have found the problem, these lines need to be moved into the for loop underneath them:
https://github.com/crowdmob/goamz/blob/98253d722c3fd15eb1b8bbe0fdbf1ec59fdfdc03/cloudwatch/cloudwatch.go#L286-L287

With that change I can list all the metrics on my account. I'll submit a pull request to crowdmob.

@ProTip
Copy link
Owner

ProTip commented Aug 8, 2014

When this pull request is merged it should resolve the problem: https://github.com/crowdmob/goamz/pull/203.

Let me know how you get along :)

@ProTip
Copy link
Owner

ProTip commented Aug 25, 2014

crowdmob have merged my pull request, closing this issue.

@ProTip ProTip closed this as completed Aug 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants