Skip to content

Fix: Updating GeoIP default image and secret handling - #168

Merged
alix-graylog merged 4 commits into
mainfrom
geoip-fix
Aug 5, 2026
Merged

Fix: Updating GeoIP default image and secret handling#168
alix-graylog merged 4 commits into
mainfrom
geoip-fix

Conversation

@alix-graylog

@alix-graylog alix-graylog commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Fixing an incorrect default image tag for the GeoIP side car, and moving the GeoIP secret details to it's own, externally provided secret preffered as to not conflict with auto generated secrets.

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

@alix-graylog
alix-graylog merged commit 64aed49 into main Aug 5, 2026
9 checks passed
@alix-graylog
alix-graylog deleted the geoip-fix branch August 5, 2026 15:52
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.

2 participants