cds-snc / notification-planning-core

Project planning for GC Notify Core Team

0 stars 0 forks source link

Wire New Relic into our Terraform and Single Alarm as a POC #143

Open jimleroyer opened 1 year ago

jimleroyer commented 1 year ago

Description

As an ops lead, I want to have the NewRelic integration wired into Terraform, So that I can boot it up anytime I need it, and make maintenance easier overall.

WHY are we building?

Easier maintenance; BCP.

WHAT are we building?

Integrate NewRelic into our Terraform plan. Link our accounts to NewRelic Integration of Secrets Develop new alerts!

VALUE created by our solution

Replication, stability, maintenance.

Acceptance Criteria

[x] NewRelic is wired into our TF setup.
[x] NewRelic's alarms are defined in our TF definition files.

QA Steps

[x] Tested against our NewRelic instance.
[ ] Verify that the alert was created after code is merged to main

sastels commented 4 months ago

docs from New Relic https://docs.newrelic.com/docs/more-integrations/terraform/terraform-intro/

P0NDER0SA commented 3 months ago

Reading through the documentation and browsing our NR instance to get some understanding on the product. Will be looking to bootstrap something soon.

sastels commented 3 months ago

Successfully made a test alert with terragrunt. May need to refactor secret usage a bit and add logic for environments.

P0NDER0SA commented 3 months ago

fixed up a lot of formatting, ready for a PR soon. Need to revisit/refactor how we handle the secrets/Env vars for the New Relic Provider :( . Bootstrapped a well organized file structure for migration of our NR stuff into TF.

P0NDER0SA commented 3 months ago

Merged and awaiting QA

P0NDER0SA commented 3 months ago

last PR is: https://github.com/cds-snc/notification-terraform/pull/1412/files

jimleroyer commented 2 months ago

Jimmy to QA!

jimleroyer commented 2 months ago

Pond deleted the alarm and hence this work needs to be redone.. or we move it to done and wait for the new integration!