Skip to content

Conversation

@sgontla
Copy link
Collaborator

@sgontla sgontla commented Feb 2, 2026

Description

Adding KVService controller flow. This is still work in progress. Will bring in more pieces as we keep working on the upcoming stories.

Key Changes

Add support for K8s Deployment and pod updates for KVService.

Testing and Verification

KVService deployment comes up, but pod fails at this time, which is expected, as we need to bring in other supporting changes like the kvservice config in single tenant mode, and also needs db connectivity for the probes to succeed:

sgontla@sgk8s:~/specs$
NAME                                     READY   STATUS             RESTARTS       AGE
splunk-demo-kvservice-57dd65566f-c27jw   0/1     CrashLoopBackOff   6 (110s ago)   7m33s
splunk-demo-kvservice-57dd65566f-sbtbn   0/1     CrashLoopBackOff   6 (117s ago)   7m33s
splunk-demo-kvservice-57dd65566f-xzcnh   0/1     CrashLoopBackOff   6 (116s ago)   7m33s
sgontla@sgk8s:~/specs$
sgontla@sgk8s:~/specs$ k logs -f splunk-demo-kvservice-57dd65566f-c27jw
Service type is set to:  KvsCsp
{"level":"INFO","time":"2026-02-03T06:40:34.243Z","location":"restapi/configure_kvstore.go:124","message":"Starting service","service":"kvstore","hostname":"splunk-demo-kvservice-57dd65566f-c27jw","component":"kvstore-startup","requestId":"69819862b3e9ff0b298d0a1f","branch":"master","commit":"0.12.578","buildTime":"2026-01-13T03:00:12Z"}
{"level":"INFO","time":"2026-02-03T06:40:34.243Z","location":"tracer/tracer.go:18","message":"No distributed tracer enabled","service":"kvstore","hostname":"splunk-demo-kvservice-57dd65566f-c27jw","component":"kvstore-startup","requestId":"69819862b3e9ff0b298d0a1f"}
{"level":"FATAL","time":"2026-02-03T06:40:34.243Z","location":"cfg/config.go:466","message":"Missing required environment variable","service":"kvstore","hostname":"splunk-demo-kvservice-57dd65566f-c27jw","envVariable":"KVSERVICE_DB_SIZECHECKINTERVAL_SECONDS","error":"configuration error","callstack":"cd.splunkdev.com/core-services/kvstore/kvstore/cfg.GetenvRequired\n\t/home/sgontla/git/KVService/kvstore/kvstore/cfg/config.go:466\ncd.splunkdev.com/core-services/kvstore/kvstore/cfg.GetenvRequiredInt64\n\t/home/sgontla/git/KVService/kvstore/kvstore/cfg/config.go:479\ncd.splunkdev.com/core-services/kvstore/kvstore/cfg.UnmarshalWithPostgresVersion\n\t/home/sgontla/git/KVService/kvstore/kvstore/cfg/unmarshal.go:79\ncd.splunkdev.com/core-services/kvstore/kvstore/cfg.Unmarshal\n\t/home/sgontla/git/KVService/kvstore/kvstore/cfg/unmarshal.go:20\ncd.splunkdev.com/core-services/kvstore/kvstore/cfg.LoadCoreConfig\n\t/home/sgontla/git/KVService/kvstore/kvstore/cfg/config.go:376\ncd.splunkdev.com/core-services/kvstore/restapi.configureAPI\n\t/home/sgontla/git/KVService/kvstore/restapi/configure_kvstore.go:164\ncd.splunkdev.com/core-services/kvstore/restapi.(*Server).ConfigureAPI\n\t/home/sgontla/git/KVService/kvstore/restapi/server.go:59\nmain.main\n\t/home/sgontla/git/KVService/kvstore/cmd/kvstore-server/main.go:51\nruntime.main\n\t/home/sgontla/myGoProjects/pkg/mod/golang.org/toolchain@v0.0.1-go1.24.4.linux-amd64/src/runtime/proc.go:283"}

Related Issues

Jira tickets, GitHub issues, Support tickets...

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.

@sgontla sgontla self-assigned this Feb 2, 2026
@sgontla sgontla marked this pull request as draft February 2, 2026 21:21
@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@sgontla
Copy link
Collaborator Author

sgontla commented Feb 3, 2026

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

@sgontla
Copy link
Collaborator Author

sgontla commented Feb 3, 2026

I have read the Code of Conduct and I hereby accept the Terms

@sgontla sgontla changed the title [Draft] SPL-290976: KVService controller flow for deployment SPL-290976: KVService controller flow for deployment Feb 9, 2026
@jmsplunk jmsplunk marked this pull request as ready for review February 9, 2026 19:07
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.

3 participants