Skip to content

Conversation

@thearchitector
Copy link
Contributor

@thearchitector thearchitector commented Jan 30, 2026

Add tf-generated password to our automated service account health check user.

Supports this charts PR: https://github.com/IndicoDataSolutions/charts/pull/1979


Note

Low Risk
Low risk: adds a new random_password and passes it into insights-health-check Helm values; primary risk is a one-time password rotation causing drift or requiring coordinated rollout.

Overview
Adds a new Terraform random_password (ins-svc-admin-password) when insights_enabled is true and wires its value into the insights_values Helm overrides under insights-health-check.user.password.

Applies the same change in both application.tf (AWS) and azure/application.tf so the Insights health check uses an autogenerated service-account password instead of an unset value.

Written by Cursor Bugbot for commit 277560c. This will update automatically on new commits. Configure here.

@emuzykin1
Copy link
Contributor

@ltellesfl - please review when you get a chance

Copy link
Contributor

@ltellesfl ltellesfl left a comment

Choose a reason for hiding this comment

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

lgtm

@emuzykin1 emuzykin1 merged commit 72ef9a5 into main Feb 2, 2026
24 checks passed
@emuzykin1 emuzykin1 deleted the QA-2495 branch February 2, 2026 21:45
emuzykin1 added a commit that referenced this pull request Feb 5, 2026
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