Skip to content

Avoid monitoring console restarts for peer updates#1827

Open
vivekr-splunk wants to merge 2 commits intodevelopfrom
fix/mc-no-restart-runtime-sync
Open

Avoid monitoring console restarts for peer updates#1827
vivekr-splunk wants to merge 2 commits intodevelopfrom
fix/mc-no-restart-runtime-sync

Conversation

@vivekr-splunk
Copy link
Copy Markdown
Collaborator

Summary

  • update monitoring console reconciliation to sync peer changes at runtime instead of forcing pod restarts on MC config changes
  • keep MC integration checks polling on actual config convergence and add explicit no-reset assertions for monitoring console pods
  • extend the shared no-pod-reset helper so appframework suites also catch unexpected monitoring console pod resets

Testing

  • env GOCACHE=/tmp/splunk-operator-go-cache go test ./pkg/splunk/client
  • env GOCACHE=/tmp/splunk-operator-go-cache go test ./pkg/splunk/enterprise -run 'TestApplyMonitoringConsole|TestGetMonitoringConsoleDesiredPeers|TestApplyMonitoringConsoleEnvConfigMap|TestGetMonitoringConsoleStatefulSet|TestAddMonitoringConsolePeer|TestRemoveMonitoringConsolePeer'
  • env GOCACHE=/tmp/splunk-operator-go-cache go test -run '^$' ./test/...

Notes

  • a one-time MC StatefulSet rollout may still happen when this operator version is first deployed because the pod template stops carrying the old configmap-resourceVersion churn annotation; subsequent peer changes should not restart MC

@vivekr-splunk vivekr-splunk marked this pull request as draft April 8, 2026 13:41
@vivekr-splunk vivekr-splunk marked this pull request as ready for review April 8, 2026 13:47
@rlieberman-splunk
Copy link
Copy Markdown
Collaborator

It looks like the unit tests are failing currently with these changes.

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