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

Remove Grafana monitoring service (#5750) #439

Merged
merged 1 commit into from
Jul 1, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,6 @@ Wellcome Collection common infrastructure.

- [cloudfront](cloudfront/README.md): Managing the infrastructure for Wellcome Collection's CloudFront distributions & DNS.

- [monitoring](monitoring/README.md): Grafana platform monitoring stack.

- **photography_backups**: Backup storage for photography (needs cleanup?)

## No longer in this repo
Expand Down
35 changes: 0 additions & 35 deletions images/dockerfiles/nginx/grafana.nginx.conf

This file was deleted.

13 changes: 0 additions & 13 deletions images/terraform/ecr.tf
Original file line number Diff line number Diff line change
Expand Up @@ -49,19 +49,6 @@ module "ecr_nginx_frontend_identity" {
}
}

module "ecr_nginx_grafana" {
source = "./repo_pair"

namespace = local.namespace
repo_name = "nginx_grafana"

description = "An nginx image for reverse proxying Grafana"

providers = {
aws.ecr_public = aws.ecr_public
}
}

Comment on lines -52 to -64
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not really sure what this was for but I didn't find any references to it and I suspect it wasn't used, so hopefully it's safe to remove.

Copy link
Contributor

Choose a reason for hiding this comment

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

This creates both a public and private ECR repo. I believe it is safe to remove.

module "ecr_nginx_apigw" {
source = "./repo_pair"

Expand Down
9 changes: 0 additions & 9 deletions monitoring/README.md
Original file line number Diff line number Diff line change
@@ -1,13 +1,4 @@
# Monitoring

## Grafana dashboard

We have a [Grafana][grafana] dashboard for monitoring load tests, queue sizes, and our AWS bill, among other things.

It can be viewed at <https://monitoring.wellcomecollection.org/> (note this is only accessible from within the Wellcome IP range).

[grafana]: https://grafana.com/

## Slack alarms

We have Slack alerts for certain failures across the platform, which are published into the [#wc-platform-alerts channel][slack].
Expand Down
51 changes: 0 additions & 51 deletions monitoring/terraform/efs.tf

This file was deleted.

20 changes: 0 additions & 20 deletions monitoring/terraform/main.tf

This file was deleted.

18 changes: 0 additions & 18 deletions monitoring/terraform/stack/dns.tf

This file was deleted.

3 changes: 0 additions & 3 deletions monitoring/terraform/stack/ecs.tf

This file was deleted.

28 changes: 0 additions & 28 deletions monitoring/terraform/stack/iam_role_policy.tf

This file was deleted.

65 changes: 0 additions & 65 deletions monitoring/terraform/stack/load_balancer.tf

This file was deleted.

8 changes: 0 additions & 8 deletions monitoring/terraform/stack/provider.tf

This file was deleted.

27 changes: 0 additions & 27 deletions monitoring/terraform/stack/secrets.tf

This file was deleted.

73 changes: 0 additions & 73 deletions monitoring/terraform/stack/security_groups.tf

This file was deleted.

Loading
Loading