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

Add AWS CloudWatch to FJ [WIP] #209

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Conversation

ronardcaktus
Copy link
Member

This has been deployed and is collecting data for the cluster. To see data:

  • Login to AWS
  • Go to CloudWatch
  • Click "All Metrics"
  • Click "ContainerInsights"
  • Drill down as needed

@ronardcaktus ronardcaktus changed the title Add AWS CloudWatch to FJ Add AWS CloudWatch to FJ [WIP] Jun 15, 2023
Copy link
Member

@tobiasmcnulty tobiasmcnulty left a comment

Choose a reason for hiding this comment

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

Skimmed it briefly and it lgtm! Thanks for making this @ronardcaktus and thanks for taking a look @Louis-Cloud. If you can confirm it runs and the metrics are flowing to CloudWatch, I'm happy for it to be merged.

deploy/deploy-cluster.yml Outdated Show resolved Hide resolved
@Afani97
Copy link
Contributor

Afani97 commented Feb 23, 2024

@ronardcaktus Is this PR ok to merge?

@ronardcaktus
Copy link
Member Author

@ronardcaktus Is this PR ok to merge?

@Afani97 I just went to CloudWatch and I don't see the logs flowing anymore, so let's halt on the merge.
@tobiasmcnulty do you remember where we left this? Logs are no longer going to Cloudwatch - I think they did originally. We ran the play because I can see the namespace:

 ❯ kubectl -n amazon-cloudwatch get pod
NAME                           READY   STATUS    RESTARTS   AGE
aws-cloudwatch-metrics-69sjf   1/1     Running   0          69d
aws-cloudwatch-metrics-fxhbf   1/1     Running   0          69d
aws-cloudwatch-metrics-s298j   1/1     Running   0          69d

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.

3 participants