Skip to content

Fix sidecar server config loading (7.0)#26206

Open
xd4rker wants to merge 1 commit into
7.0from
backport-7.0/fix/sidecar-config-loading
Open

Fix sidecar server config loading (7.0)#26206
xd4rker wants to merge 1 commit into
7.0from
backport-7.0/fix/sidecar-config-loading

Conversation

@xd4rker
Copy link
Copy Markdown
Member

@xd4rker xd4rker commented Jun 3, 2026

Note: This is a backport of #26200 to 7.0. (manually created due to conflicts)

Description

This fixes an issue with SidecarPluginConfiguration not being registered as a server config bean. Its sidecar_* settings in graylog.conf were ignored and always fell back to default values.

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have requested a documentation update.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

* Fix sidecar server config loading

* Add changelog

(cherry picked from commit 7ed809a)
@xd4rker xd4rker requested a review from patrickmann June 3, 2026 12:06
Copy link
Copy Markdown
Contributor

@patrickmann patrickmann left a comment

Choose a reason for hiding this comment

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

Looks good - minimal fix, no change from 7.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants