-
Notifications
You must be signed in to change notification settings - Fork 83
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
Adding Module and Example for ECS cluster monitoring with ecs_observer #211
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.
examples/ecs-cluster-with-vpc
Normally we try to keep examples as lean as possible to simplify maintenance. Normally ecs-cluster-with-vpc
should be flat and not nested. We usually have an apply-able main.tf in the examples. Here's it's not clear what we are testing.
Additionally there's a quite popular module actively maintained, maybe we should just reuse it and focus on our added value? https://github.com/terraform-aws-modules/terraform-aws-ecs but maybe I'm missing something?
When we finish the example, that use the new module, we should add a little doc https://aws-observability.github.io/terraform-aws-observability-accelerator/ on ECS monitoring as well |
@ruchimo any updates on this? |
@ruchimo please let me know when I ca re-review |
@bonclay7 - Please review.... |
@ruchimo please check the broken tests https://github.com/aws-observability/terraform-aws-observability-accelerator/actions/runs/6048526849/job/16418900113?pr=211 Also please install and run pre-commit, you can find instructions here |
Checking @bonclay7 , I already have it installed... |
@bonclay7 - I have fixed the failures detected by pre-commit |
@bonclay7 I have checked the failures
|
What does this PR do?
Adding Module and Example for ECS cluster monitoring with ecs_observer
🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted. Consult the CONTRIBUTING guide for submitting pull-requests.
Motivation
Part of the virtual Hackathon on the AWS Observability Accelerator.
More
pre-commit run -a
with this PRNote: Not all the PRs required examples and docs.
For Moderators
Additional Notes