-
Notifications
You must be signed in to change notification settings - Fork 22
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
feat: Configure target ECS cluster ARN #15
feat: Configure target ECS cluster ARN #15
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution!
We don't spend a lot of time maintaining this or checking for notifications, but this looks like a sensible improvement so I'll try to help you get it in.
It'd be useful if you were to also update the README with the new option. I'm aware that's a little bit messy right now so don't worry about placement too much.
I've made the changes to the best of my knowledge. I'll test these in my own setup and fix any issues that come up. Otherwise, I'd also appreciate a second review anytime :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another small change to ensure the current behaviour is the same but looks good
Merged, thanks again! |
Allows configuring a specific ECS cluster to monitor, using the
--cluster=ARN
CLI argument.Fixes #13
If someone could point out how to adjust the log line during startup to include this setting, I'd be happy to include it.