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

Observability Platform - Add the modernisation-platform aws account as a cloudwatch data source #8692

Closed
2 tasks done
Tracked by #8678 ...
mikereiddigital opened this issue Dec 5, 2024 · 5 comments · Fixed by ministryofjustice/modernisation-platform-environments#9237
Assignees
Labels
firebreak Mod Platform skunk works

Comments

@mikereiddigital
Copy link
Contributor

mikereiddigital commented Dec 5, 2024

User Story

As a Modernisation Platform Engineer
I need to have access to modernisation platform aws account cloudwatch data for dashboard reports.
Currently this account is not a data source.

Value / Purpose

Gives us access to modernisation platform account cloudwatch data via grafana & so avoids the need to create & run custom scripts.

Context / Background

The modernisation platform account does not have a cloudwatch data source in the Observability Platform.

Useful Contacts

No response

Additional Information

No response

Definition of Done

  • Add the modernisation platform account as a cloudwatch data source
  • Test & confirm working.
@ASTRobinson
Copy link
Contributor

Step 1 completed - #8721

but have run into an issue with step 2 ministryofjustice/modernisation-platform-environments#9042 as the modernisation platform isn't listed within the account_ids block in the environment_management secret.

@ASTRobinson
Copy link
Contributor

Following on from the investigation into adding the Modernisation Platform (MP) account to the environment_management secret, which turned out to be not feasible due to potential conflicts with other workflows, I have been working on an alternative approach. The goal was to merge the MP account details with the secret into a new variable, merged_account_ids, and use this variable in the tenant_configuration and relevant modules.

The plan seemed straightforward, but the Terraform plan output is showing unexpected changes. Here's a breakdown of the current status:

Steps Taken:

  1. Merging MP Account into merged_account_ids:
  • I attempted to merge the MP account details into a new variable called merged_account_ids, as defined in the platform_locals.tf.
  • This variable is intended to hold all relevant account IDs, including the MP account, to be passed to the relevant modules.
  1. Passing merged_account_ids to Tenant Configuration:
  • The merged variable was then passed to the tenant_configuration module as the environment_management argument.
  • The variable is also referenced in relevant modules to ensure the MP account is included in the configuration.

Outstanding Issues:

The integration of the MP account with the other accounts via the merged_account_ids variable still requires further investigation. We need to confirm that the merging process works as expected and that the variable is correctly passed through to avoid unnecessary changes in the plan.
The unexpected changes in the Terraform plan will require further debugging and might involve adjusting the way the merged_account_ids variable is merged or handled

@ASTRobinson ASTRobinson moved this from In Progress to To Do in Modernisation Platform Dec 20, 2024
@ASTRobinson
Copy link
Contributor

Branch i've been working on ministryofjustice/modernisation-platform-environments#9042

@ASTRobinson
Copy link
Contributor

ASTRobinson commented Jan 7, 2025

moving to review as the Modernisation-Platform has been successfully onboarded and the cloudwatch datasource is now available in Grafana:
Image

@ASTRobinson ASTRobinson moved this from To Do to For Review in Modernisation Platform Jan 7, 2025
@markgov
Copy link
Contributor

markgov commented Jan 10, 2025

all tasks complete

@markgov markgov closed this as completed Jan 10, 2025
@github-project-automation github-project-automation bot moved this from For Review to Done in Modernisation Platform Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
firebreak Mod Platform skunk works
Projects
Status: Done
4 participants