Skip to content

Conversation

@minjieqiu
Copy link

@minjieqiu minjieqiu commented Jan 29, 2026

Description.

This PR implement SOK Telemetry enhancement. ERD:
https://cisco-my.sharepoint.com/:w:/p/mqiu/IQBoVUuEEY1SR4rDjbja0iPuAeN5dxFG-K-ZPpvO6RoWJp0?e=n5R1Ow

What does this PR have in it?.

Periodically collect and send SOK telemetry which includes:

  1. SOK telemetry.
    a. SOK version.
    b. CPU/Memory settings (limit and request) of containers including standalone, searchheadcluster, indexercluster,
    clustermaster, clustermanager, licensemaster and licensemanager.
    c. LincenseInfo (Splunk license ID and license type).
  2. Other component's telemetry which are submitted to SOK by adding key/value to the new telemetry configmap splunk-operator-manager-telemetry

Key Changes.

  • Created a new configmap splunk-operator-manager-telemetry
  • Create a new controller which reconciles on the telemetry configmap
  • Renamed the telemetry app to app_tel_for_sok8s

Highlight the updates in specific files

Testing and Verification.

Tested on s1, c3 and m4.

How did you test these changes? What automated tests are added?.
Added telemetry verification to existing s1, c3 and m4 tests.

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.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 29, 2026

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

@minjieqiu
Copy link
Author

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

@minjieqiu
Copy link
Author

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

@coveralls
Copy link
Collaborator

coveralls commented Jan 29, 2026

Pull Request Test Coverage Report for Build 21532544509

Details

  • 263 of 380 (69.21%) changed or added relevant lines in 5 files are covered.
  • 362 unchanged lines in 18 files lost coverage.
  • Overall coverage decreased (-0.7%) to 85.675%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/splunk/client/enterprise.go 27 29 93.1%
pkg/splunk/enterprise/names.go 0 6 0.0%
internal/controller/telemetry_controller.go 21 42 50.0%
pkg/splunk/enterprise/telemetry.go 213 301 70.76%
Files with Coverage Reduction New Missed Lines %
pkg/splunk/enterprise/afwscheduler.go 3 92.51%
internal/controller/clustermanager_controller.go 5 93.1%
internal/controller/licensemanager_controller.go 5 92.41%
internal/controller/licensemaster_controller.go 5 92.41%
internal/controller/monitoringconsole_controller.go 5 93.48%
internal/controller/searchheadcluster_controller.go 5 92.31%
internal/controller/standalone_controller.go 5 92.41%
internal/controller/clustermaster_controller.go 7 90.0%
internal/controller/indexercluster_controller.go 8 90.53%
pkg/splunk/enterprise/events.go 11 73.81%
Totals Coverage Status
Change from base Build 21358635731: -0.7%
Covered Lines: 10999
Relevant Lines: 12838

💛 - Coveralls

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.

2 participants