splunk / splunk-operator / 22049735322

85%

main: 87%

Build Type

Pull #1699

circleci

Committed by vivekr-splunk

Commit Message

Fix smoke test timeouts by ensuring legacy secret exists before SDK resolution

Root Cause:
- SDK's ResolveSecret() expects legacy secret splunk-{namespace}-secret to exist
- In smoke tests (and new deployments), this secret doesn't exist initially
- SDK returned "secret not ready" error, causing reconciliation to fail and retry
- Reconciliation loop continued indefinitely until 4-hour GitHub Actions timeout

Solution:
- Call ApplyNamespaceScopedSecretObject() BEFORE SDK secret resolution
- This ensures the legacy source secret exists for SDK to resolve and version
- Maintains backward compatibility with existing deployments
- SDK can now create versioned secrets (v1, v2, v3) from the legacy source

Changes:
- Added call to splutil.ApplyNamespaceScopedSecretObject() at standalone.go:309
- This mirrors legacy behavior which always creates the secret if missing
- Updated step numbering in comments for clarity
- All unit tests pass (137 tests in pkg/splunk/enterprise)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Pull Request Pull Request #1699: [SPIKE] Platform SDK Integration for Standalone Controller

Run Details

152 of 377 new or added lines in 4 files covered. (40.32%)

1 existing line in 1 file now uncovered.

11087 of 13061 relevant lines covered (84.89%)

28.59 hits per line