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

[plugin-aws-ec2-ebs] add actions to README need to be allowed in the iam policy #1219

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

miztch
Copy link
Contributor

@miztch miztch commented Aug 28, 2024

I've been setting up mackerel agent with aws-ec2-ebs plugin according to README, and was faced with insufficient AWS IAM policy attachment.

I've recieved two error messages (several ids are masked) and now it turned out that two essential actions (ec2:DescribeInstanceTypes and ec2:DescribeInstances) are not mentioned on README for aws-ec2-ebs plugin. So this PR fixes it.

Aug 28 07:48:10 ip-10-0-0-100 mackerel-agent[409653]: 2024/08/28 07:48:10 INFO <metrics.plugin> command mackerel-plugin-aws-ec2-ebs outputted to STDERR: "2024/08/28 07:48:10 UnauthorizedOperation: You are not authorized to perform this operation. User: arn:aws:sts::123456789012:assumed-role/my-role/i-01234567890abcdef is not authorized to perform: ec2:DescribeInstanceTypes because no identity-based policy allows the ec2:DescribeInstanceTypes action\n\tstatus code: 403, request id: c129b45e-410a-4be8-998c-681a127db319\n"
Aug 28 07:42:00 ip-10-0-0-100 mackerel-agent[408819]: 2024/08/28 07:42:00 INFO <metrics.plugin> command mackerel-plugin-aws-ec2-ebs outputted to STDERR: "2024/08/28 07:42:00 UnauthorizedOperation: You are not authorized to perform this operation. User: arn:aws:sts::123456789012:assumed-role/my-role/i-01234567890abcdef is not authorized to perform: ec2:DescribeInstances because no identity-based policy allows the ec2:DescribeInstances action\n\tstatus code: 403, request id: 99962773-9682-4b1d-922f-79184f2252a5\n"

@onk onk requested a review from yohfee August 29, 2024 05:29
@yohfee
Copy link
Contributor

yohfee commented Sep 2, 2024

Thank you for your contribution!

We fixed CI failure by #1220. Would you mind trying with it?

@miztch
Copy link
Contributor Author

miztch commented Sep 2, 2024

Thank you, I merged the master branch with the CI fixes into this branch, but the Action did not run.

I'm sorry I don't have idea how to deal with this, is there anything I should do, or is it OK to leave it as it is?

Copy link
Contributor

@yohfee yohfee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are grateful for your assistance!

@yohfee yohfee merged commit 409a09a into mackerelio:master Sep 3, 2024
13 checks passed
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

Successfully merging this pull request may close these issues.

2 participants