Skip to content

SPL-290976: KVService controller flow for deployment#1683

Merged
sgontla merged 2 commits intofeature/epic_SPL-290563from
story_SPL-290976
Feb 18, 2026
Merged

SPL-290976: KVService controller flow for deployment#1683
sgontla merged 2 commits intofeature/epic_SPL-290563from
story_SPL-290976

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 CLA Assistant Lite bot All contributors have signed the COC ✍️ ✅

@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
@sgontla sgontla force-pushed the story_SPL-290976 branch 2 times, most recently from 758dba9 to b847775 Compare February 11, 2026 23:01
@sgontla sgontla merged commit 81ef5cf into feature/epic_SPL-290563 Feb 18, 2026
25 checks passed
@sgontla sgontla deleted the story_SPL-290976 branch February 18, 2026 02:47
@github-actions github-actions bot locked and limited conversation to collaborators Feb 18, 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.

4 participants

Comments