Skip to content

fix(secrets): Adding graylog-root-sha2 to automatic password generation#104

Draft
alix-graylog wants to merge 1 commit into
mainfrom
fix/rotating-secret
Draft

fix(secrets): Adding graylog-root-sha2 to automatic password generation#104
alix-graylog wants to merge 1 commit into
mainfrom
fix/rotating-secret

Conversation

@alix-graylog

Copy link
Copy Markdown
Collaborator

Summary

This PR addresses a bug with the management of auto generated Graylog application secret where GRAYLOG_ROOT_PASSWORD_SHA2 was not sourced when pulling credentials from a backup

Details

  • List of meaningful technical changes

Linked issues

#102

PR Checklist

Please check the items that apply to your change.

  • Tests added/updated
  • Documentation updated
  • This PR includes a new feature
  • This PR includes a bugfix
  • This PR includes a refactor

Testing Checklist

Static Validation

  • Linter check passes: helm lint ./charts/graylog
  • Helm renders local template sucessfully: helm template graylog ./charts/graylog --validate

Installation

  • Fresh installation completes successfully: helm install graylog ./charts/graylog
  • All pods reach Running state: kubectl rollout status statefulset/graylog
  • Helm tests pass: helm test graylog

Functional (if applicable)

  • Web UI accessible and login works
  • DataNodes visible in System > Cluster Configuration
  • Inputs can be created and receive data

Upgrade (if applicable)

  • Upgrade from previous release succeeds
  • Scaling up/down works correctly
  • Configuration changes apply correctly

Specific to this PR

  • describe what was specifically tested

Notes for reviewers

  • Verify all applicable tests above pass
  • Validate that the linked issues are no longer reproducible, if applicable
  • Sync up with the author before merging
  • The commit history should be preserved - use rebase-merge or standard merge options when applicable

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