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

feat: allow logging of S3Sync/mobius3 metrics to CloudWatch #61

Merged
merged 1 commit into from
Mar 15, 2024

Conversation

michalc
Copy link
Member

@michalc michalc commented Mar 14, 2024

This adds the permission for each user to log S3Sync metrics to Cloudwatch, specifically ones added in https://github.com/uktrade/mobius3/releases/tag/v0.0.42

It also tightens up permissions on the Cloudwatch VPC endpoints, making sure that only our AWS accounts can use them.

@michalc michalc requested a review from a team as a code owner March 14, 2024 17:01
@michalc michalc force-pushed the feat/allow-logging-to-cloudwatch branch 4 times, most recently from df10d88 to 52ddaf0 Compare March 15, 2024 15:57
@michalc michalc changed the title feat: allow logging to CloudWatch feat: allow logging of S3Sync/mobius3 metrics to CloudWatch Mar 15, 2024
This adds the permission for each user to log S3Sync metrics to Cloudwatch,
specifically ones added in
https://github.com/uktrade/mobius3/releases/tag/v0.0.42

It also tightens up permissions on the Cloudwatch VPC endpoints, making sure
that only our AWS accounts can use them.
@michalc michalc force-pushed the feat/allow-logging-to-cloudwatch branch from 52ddaf0 to 9155f19 Compare March 15, 2024 16:56
@michalc michalc merged commit ca4a0b4 into main Mar 15, 2024
1 check passed
@michalc michalc deleted the feat/allow-logging-to-cloudwatch branch March 15, 2024 17:13
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.

1 participant