Skip to content

CSPL-4324 Emitting events for passwords, secrets, app fw, CM, scaling and smart store#1689

Merged
Igor-splunk merged 10 commits intodevelopfrom
feature/CSPL-4338-password-sync-events
Feb 12, 2026
Merged

CSPL-4324 Emitting events for passwords, secrets, app fw, CM, scaling and smart store#1689
Igor-splunk merged 10 commits intodevelopfrom
feature/CSPL-4338-password-sync-events

Conversation

@kasiakoziol
Copy link
Collaborator

@kasiakoziol kasiakoziol commented Feb 6, 2026

Description

It emits events for passwords, secrets, app fw, CM, scaling and smart store

Key Changes

Events:

  • PasswordSyncCompleted (Normal)
  • ClusterQuorumRestored (Normal)
  • ClusterInitialized (Normal)
  • ScaledUp (Normal)
  • ScaledDown (Normal)
  • UpgradeBlockedVersionMismatch (Warning)
  • PasswordSyncFailed (Warning)
  • AppRepositoryConnectionFailed (Warning)
  • SecretMissing (Warning)
  • SecretInvalid (Warning)
  • ClusterQuorumLost (Warning)
  • ScalingBlockedRF (Warning)

Testing and Verification

  • Automated tests
  • Manual tests

Related Issues

Epic: https://splunk.atlassian.net/browse/CSPL-4324

PR Checklist

  • Code changes adhere to the project's coding standards.
  • Relevant unit and integration tests are included.
  • Documentation has been updated accordingly.
  • All tests pass locally.
  • The PR description follows the project's guidelines.

@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 21753278296

Details

  • 128 of 179 (71.51%) changed or added relevant lines in 9 files are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.07%) to 86.144%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/splunk/enterprise/searchheadcluster.go 16 21 76.19%
pkg/splunk/enterprise/upgrade.go 2 7 28.57%
pkg/splunk/enterprise/standalone.go 7 15 46.67%
pkg/splunk/enterprise/searchheadclusterpodmanager.go 11 21 52.38%
pkg/splunk/enterprise/indexercluster.go 56 79 70.89%
Files with Coverage Reduction New Missed Lines %
pkg/splunk/enterprise/events.go 2 88.1%
Totals Coverage Status
Change from base Build 21647487450: -0.07%
Covered Lines: 10874
Relevant Lines: 12623

💛 - Coveralls

@coveralls
Copy link
Collaborator

coveralls commented Feb 6, 2026

Pull Request Test Coverage Report for Build 21901475741

Details

  • 141 of 179 (78.77%) changed or added relevant lines in 9 files are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.2%) to 86.382%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/splunk/enterprise/searchheadcluster.go 16 21 76.19%
pkg/splunk/enterprise/standalone.go 7 15 46.67%
pkg/splunk/enterprise/searchheadclusterpodmanager.go 11 21 52.38%
pkg/splunk/enterprise/indexercluster.go 64 79 81.01%
Files with Coverage Reduction New Missed Lines %
pkg/splunk/enterprise/events.go 2 88.1%
Totals Coverage Status
Change from base Build 21647487450: 0.2%
Covered Lines: 10904
Relevant Lines: 12623

💛 - Coveralls

@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


2 out of 3 committers have signed the CLA.
@kasiakoziol
@Igor-splunk
❌ @igor.grzankowski
igor.grzankowski seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request

@Igor-splunk Igor-splunk marked this pull request as ready for review February 12, 2026 09:02

// Reset recorder and prepare second call with indexing_ready=false
recorder.events = []mockEvent{}
quorumLostInfo := `{"entry":[{"content":{"initialized_flag":true,"indexing_ready_flag":false,"service_ready_flag":true,"maintenance_mode":false,"rolling_restart_flag":false,"label":"splunk-manager1-cluster-manager-0","active_bundle":{"bundle_path":"/opt/splunk/var/run/splunk/cluster/remote-bundle/506c58d5aeda1dd6017889e3186e7571-1583870198.bundle","checksum":"ABC123","timestamp":1583870198},"latest_bundle":{"bundle_path":"/opt/splunk/var/run/splunk/cluster/remote-bundle/506c58d5aeda1dd6017889e3186e7571-1583870198.bundle","checksum":"ABC123","timestamp":1583870198},"multisite":"false","replication_factor":3,"site_replication_factor":"origin:2,total:3"}}]}`
Copy link
Collaborator

Choose a reason for hiding this comment

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

Such fixture looks like hard to manage. Maybe we can came up with something more readable - even multiline json would be more readable

Copy link
Collaborator

Choose a reason for hiding this comment

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

[optional]

@Igor-splunk Igor-splunk merged commit b879d5f into develop Feb 12, 2026
57 of 61 checks passed
@Igor-splunk Igor-splunk deleted the feature/CSPL-4338-password-sync-events branch February 12, 2026 13:14
@github-actions github-actions bot locked and limited conversation to collaborators Feb 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants