Skip to content

CSPL-4513 Events for Index & Ingestion separation#1706

Open
kasiakoziol wants to merge 125 commits intofeature/slog-as-default-loggerfrom
feature/cspl-4513-events-for-index-ingest-sep
Open

CSPL-4513 Events for Index & Ingestion separation#1706
kasiakoziol wants to merge 125 commits intofeature/slog-as-default-loggerfrom
feature/cspl-4513-events-for-index-ingest-sep

Conversation

@kasiakoziol
Copy link
Collaborator

@kasiakoziol kasiakoziol commented Feb 18, 2026 •
edited
Loading

Description

It adds events and enhances logs for IngestorCluster and enhances events for IndexerCluster.

Key Changes

  • Refactoring existing event messages for IngestorCluster and IndexerCluster
  • Introducing new events for IngestorCluster: ScaledUp, ScaledDown, QueueConfigUpdated and
    IngestorsRestarted
  • Introducing new events for IndexerCluster: QueueConfigUpdated and
    IngestorsRestarted
  • Introducing slog
  • Automated tests for the above

Testing and Verification

Automated and manual tests

Related Issues

Jira: https://splunk.atlassian.net/browse/CSPL-4513

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.

kasiakoziol and others added 30 commits July 24, 2025 13:49
@kasiakoziol
CSPL-3551 Init IngestorCluster CR implementation
0e36568
@kasiakoziol
CSPL-3551 Enhancing Ingestor inputs
2cca0d7
@kasiakoziol
Merge branch 'develop' into CSPL-3551-ingestion-cr
f747ab4
@kasiakoziol
fix
9a7503c
@kasiakoziol
fix
7134350
@kasiakoziol
CSPL-3551 Adding tests
44349fa
@kasiakoziol
CSPL-3551 Mound defaults and update them with no Splunk restart
e71ea9c
@kasiakoziol
Merge branch 'develop' into CSPL-3551-ingestion-cr
14ccef4
@kasiakoziol
CSPL-3551 Fixing code after tests
8876d0e
@Igor-splunk @kasiakoziol
CSPL-3895-indexercluster (#1562)
5c31cab 
* test

* test

* test

* test

* Use endpoint to update conf file

* CSPL-3895 Matching changes from Ingestor to Indexer

---------

Co-authored-by: Kasia Koziol <kkoziol@splunk.com>
Co-authored-by: igor.grzankowski <@splunk.com>
@kasiakoziol
CSPL-3551 Applying fixes do Indexer integration
0c9e908
@kasiakoziol
Merge branch 'develop' into CSPL-3551-ingestion-cr
89f9eb4
@kasiakoziol
CSPL-3551 Fixes
d885a73
@kasiakoziol
Merge branch 'develop' into CSPL-3551-ingestion-cr
56aa780
@kasiakoziol
CSPL-3551 Fixes
1c77634
@kasiakoziol
CSPL-3560 Initial docs for I&I separation
fb4e87f
@kasiakoziol
Merge branch 'develop' into CSPL-3551-ingestion-cr
3e376d8
@kasiakoziol
CSPL-3551 Update documentation to reflect on Grafana
e62f381
@kasiakoziol
CSPL-3556 Unit tests
84d8491
@kasiakoziol
CSPL-3972 Addressing TODOs
c3b3aa6
@kasiakoziol
CSPL-3972 Addressing TODOs
2af9061
@kasiakoziol
CSPL-3551 Moving default from types to controller code
053352d
@kasiakoziol
CSPL-3551 Applying previous changes to IndexerCluster
e844286
@kasiakoziol
CSPL-3551 Moving validations to separate function and adding validati…
500cdfa 
…on for empty values
@kasiakoziol
CSPL-3551 Making sure all inputs are put into status
4b064a6
@kasiakoziol
CSPL-3551 Update of conf files when queue name or type change
9c6cd69
@kasiakoziol
CSPL-3551 Unit tests
759b09b
@kasiakoziol
CSPL-4003 Helm charts support for bus and pipeline configs
20d9d6c
@kasiakoziol
CSPL-4003 Docs update
f9fb45a
@kasiakoziol
CSPL-4003 Addressing comments
c5e4c2b
@kasiakoziol
CSPL-4002 Update Ansible role to ingestor (from standalone) for Inges…
7bef9e3 
…torCluster (#1606)

* CSPL-4002 Update role for ingestor to ingestor from standalone

* CSPL-4212 Address changes from unit tests from develop branch

* Addressing comments

* Run tests against Splunk 10.2

* Fix

* Address changes from develop

* CSPL-4002 Revert 10.2 changes
@kasiakoziol
Merge branch 'CSPL-3551-ingestion-cr' into feature/cspl-4513-events-f…
7b5154f 
…or-index-ingest-sep
Base automatically changed from CSPL-3551-ingestion-cr to develop February 20, 2026 15:29
vivekr-splunk
vivekr-splunk reviewed Feb 23, 2026
View reviewed changes
vivekr-splunk
vivekr-splunk reviewed Feb 23, 2026
View reviewed changes
vivekr-splunk
vivekr-splunk reviewed Feb 23, 2026
View reviewed changes
pkg/splunk/enterprise/indexercluster.go Outdated
Copy link
Collaborator

@vivekr-splunk vivekr-splunk Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we pass new parameter here for logger ?

pkg/splunk/enterprise/indexercluster.go
return result, err
}

eventPublisher.Normal(ctx, "QueueConfigUpdated",
Copy link
Collaborator

@vivekr-splunk vivekr-splunk Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

queue configured message is updated before restart

Copy link
Collaborator Author

@kasiakoziol kasiakoziol Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because it is updated in the .conf file before the restart. Restart just makes it work.

@kasiakoziol kasiakoziol force-pushed the feature/cspl-4513-events-for-index-ingest-sep branch from d540e31 to 14d99b5 Compare February 23, 2026 15:00
@kasiakoziol kasiakoziol changed the base branch from develop to feature/slog-as-default-logger February 26, 2026 07:07
@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026 •
edited
Loading

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

@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

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

5 out of 6 committers have signed the CLA.
✅ @kasiakoziol
✅ @Igor-splunk
✅ @gabrielm-splunk
✅ @kubabuczak
✅ @minjieqiu
❌ @Nguyen Duc Trung
NGUYEN Duc Trung seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request

@kasiakoziol kasiakoziol force-pushed the feature/cspl-4513-events-for-index-ingest-sep branch from 2aa9a8e to 30c2546 Compare February 27, 2026 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rlieberman-splunk rlieberman-splunk rlieberman-splunk approved these changes

@Igor-splunk Igor-splunk Igor-splunk approved these changes

@patrykw-splunk patrykw-splunk Awaiting requested review from patrykw-splunk

@kubabuczak kubabuczak Awaiting requested review from kubabuczak

@gabrielm-splunk gabrielm-splunk Awaiting requested review from gabrielm-splunk

@minjieqiu minjieqiu Awaiting requested review from minjieqiu

@qingw-splunk qingw-splunk Awaiting requested review from qingw-splunk

@vivekr-splunk vivekr-splunk Awaiting requested review from vivekr-splunk

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kasiakoziol @coveralls @vivekr-splunk @rlieberman-splunk @Igor-splunk