[OpAMP] enable OpAMP endpoint by default by ycombinator · Pull Request #6657 · elastic/fleet-server

ycombinator · 2026-03-24T21:31:05Z

Summary

Previously, the OpAMP endpoint was disabled by default. Users had to explicitly enable it by adding the following to their Fleet Server configuration:

server:
  feature_flags:
    enable_opamp: true

With this change, the OpAMP endpoint is enabled by default. Users who need to disable it can do so by setting server.feature_flags.enable_opamp: false in the Fleet Server input's configuration.

Changes in this PR

Set EnableOpAMP to true in InitDefaults() so the OpAMP endpoint is enabled out of the box
Update config tests to use defaultServer() with overrides instead of manually built structs, removing unused test helpers
Update reference config, setup docs, and e2e test template to reflect the new default

🤖 Generated with Claude Code

Set EnableOpAMP to true in InitDefaults() so the OpAMP endpoint is enabled out of the box without requiring explicit configuration. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Replace the manually built Server struct with defaultServer() plus overrides so it picks up the new EnableOpAMP default. Remove unused helpers (defaultServerBulk, defaultServerGC, defaultPBKDF2). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Rename test cases to "explicitly disabled/enabled" to reflect that the flag now defaults to true. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

The feature flag is now enabled by default, so the docs now explain how to disable it rather than how to enable it. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

The flag now defaults to true, so explicitly setting it is unnecessary. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

mergify · 2026-03-24T21:31:45Z

This pull request does not have a backport label. Could you fix it @ycombinator? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

backport-./d./d is the label to automatically backport to the 8./d branch. /d is the digit
backport-active-all is the label that automatically backports to all active branches.
backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

ebeahan

LGTM.

changelog/fragments/1774388797-opamp-ff-default-enabled.yaml

github-actions · 2026-03-25T16:29:50Z

@Mergifyio backport 9.2 9.3

mergify · 2026-03-25T16:36:24Z

backport 9.2 9.3

✅ Backports have been created

Details

#6661 [9.2](backport #6657) [OpAMP] enable OpAMP endpoint by default has been created for branch 9.2
#6662 [9.3](backport #6657) [OpAMP] enable OpAMP endpoint by default has been created for branch 9.3

* feat: enable OpAMP feature flag by default Set EnableOpAMP to true in InitDefaults() so the OpAMP endpoint is enabled out of the box without requiring explicit configuration. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * test: update config test to use defaultServer() for input-config case Replace the manually built Server struct with defaultServer() plus overrides so it picks up the new EnableOpAMP default. Remove unused helpers (defaultServerBulk, defaultServerGC, defaultPBKDF2). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * test: clarify feature flag test case names Rename test cases to "explicitly disabled/enabled" to reflect that the flag now defaults to true. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * docs: update reference config to reflect OpAMP enabled by default Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * docs: update OpAMP setup guide for default-enabled flag The feature flag is now enabled by default, so the docs now explain how to disable it rather than how to enable it. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * e2e: remove redundant enable_opamp flag from OpAMP test template The flag now defaults to true, so explicitly setting it is unnecessary. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * changelog: add fragment for OpAMP feature flag default change Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: fix gofmt formatting in config_test.go Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> (cherry picked from commit fee63f0)

* feat: enable OpAMP feature flag by default Set EnableOpAMP to true in InitDefaults() so the OpAMP endpoint is enabled out of the box without requiring explicit configuration. * test: update config test to use defaultServer() for input-config case Replace the manually built Server struct with defaultServer() plus overrides so it picks up the new EnableOpAMP default. Remove unused helpers (defaultServerBulk, defaultServerGC, defaultPBKDF2). * test: clarify feature flag test case names Rename test cases to "explicitly disabled/enabled" to reflect that the flag now defaults to true. * docs: update reference config to reflect OpAMP enabled by default * docs: update OpAMP setup guide for default-enabled flag The feature flag is now enabled by default, so the docs now explain how to disable it rather than how to enable it. * e2e: remove redundant enable_opamp flag from OpAMP test template The flag now defaults to true, so explicitly setting it is unnecessary. * changelog: add fragment for OpAMP feature flag default change * chore: fix gofmt formatting in config_test.go --------- (cherry picked from commit fee63f0) Co-authored-by: Shaunak Kashyap <ycombinator@gmail.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

ycombinator and others added 6 commits March 24, 2026 14:30

feat: enable OpAMP feature flag by default

8f1aeac

Set EnableOpAMP to true in InitDefaults() so the OpAMP endpoint is enabled out of the box without requiring explicit configuration. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

test: clarify feature flag test case names

97bc476

Rename test cases to "explicitly disabled/enabled" to reflect that the flag now defaults to true. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

docs: update reference config to reflect OpAMP enabled by default

396abc5

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

docs: update OpAMP setup guide for default-enabled flag

6ea764b

The feature flag is now enabled by default, so the docs now explain how to disable it rather than how to enable it. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

e2e: remove redundant enable_opamp flag from OpAMP test template

7acf220

The flag now defaults to true, so explicitly setting it is unnecessary. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

ycombinator changed the title ~~feat: enable OpAMP feature flag by default~~ [OpAMP] enable OpAMP endpoint by default Mar 24, 2026

ycombinator added Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team backport-active-9 Automated backport with mergify to all the active 9.[0-9]+ branches labels Mar 24, 2026

changelog: add fragment for OpAMP feature flag default change

e80f028

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

ycombinator requested a review from a team as a code owner March 24, 2026 21:47

ycombinator requested review from blakerouse and swiatekm March 24, 2026 21:47

chore: fix gofmt formatting in config_test.go

6ae4c73

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

ycombinator requested a review from michel-laterman March 24, 2026 22:41

ebeahan approved these changes Mar 25, 2026

View reviewed changes

ebeahan reviewed Mar 25, 2026

View reviewed changes

changelog/fragments/1774388797-opamp-ff-default-enabled.yaml Show resolved Hide resolved

michel-laterman approved these changes Mar 25, 2026

View reviewed changes

ycombinator merged commit fee63f0 into elastic:main Mar 25, 2026
11 checks passed

ycombinator deleted the opamp-ff-default-enabled branch March 25, 2026 16:30

mergify bot mentioned this pull request Mar 25, 2026

[9.2](backport #6657) [OpAMP] enable OpAMP endpoint by default #6661

Merged

mergify bot mentioned this pull request Mar 25, 2026

[9.3](backport #6657) [OpAMP] enable OpAMP endpoint by default #6662

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[OpAMP] enable OpAMP endpoint by default#6657

[OpAMP] enable OpAMP endpoint by default#6657
ycombinator merged 8 commits intoelastic:mainfrom
ycombinator:opamp-ff-default-enabled

ycombinator commented Mar 24, 2026 •

edited

Loading

Uh oh!

mergify bot commented Mar 24, 2026

Uh oh!

ebeahan left a comment

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Mar 25, 2026

Uh oh!

mergify bot commented Mar 25, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

ycombinator commented Mar 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes in this PR

Uh oh!

mergify bot commented Mar 24, 2026

Uh oh!

ebeahan left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Mar 25, 2026

Uh oh!

mergify bot commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Backports have been created

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

ycombinator commented Mar 24, 2026 •

edited

Loading

mergify bot commented Mar 25, 2026 •

edited

Loading