splunk / splunk-operator / 22020349453

85%

main: 87%

Build Type

Pull #1699

circleci

Committed by vivekr-splunk

Commit Message

Make Platform SDK initialization optional with graceful fallback

This commit fixes smoke test timeouts by making Platform SDK
initialization optional. If SDK runtime initialization fails
(e.g., PlatformConfig CRDs not installed), the controller now
falls back to legacy mode instead of failing to start.

Changes:
- internal/controller/standalone_controller.go: Modified
  SetupWithManager to handle SDK initialization failures gracefully
  by setting SDKRuntime to nil (triggers legacy mode)

This ensures backward compatibility with environments that don't
have Platform SDK CRDs installed, such as existing smoke tests.

Previously, the smoke tests were timing out after 4 hours because:
1. SDK runtime initialization would fail if CRDs weren't present
2. The controller setup would fail entirely
3. Pods couldn't start, tests waited forever for Ready state

Now the controller will:
1. Attempt to initialize SDK runtime
2. If it fails, log a warning and use legacy mode (sdkRuntime=nil)
3. Fall back to legacy StatefulSet creation path
4. Allow smoke tests to proceed normally

Fixes timeout issues in:
- smoke-tests (managermc)
- smoke-tests (basic)
- smoke-tests (appframeworksS1)
- smoke-tests (managersecret)

🤖 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

148 of 366 new or added lines in 4 files covered. (40.44%)

1 existing line in 1 file now uncovered.

11083 of 13050 relevant lines covered (84.93%)

28.62 hits per line