-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Elastic Agent] Fix default configuration after enroll #18232
[Elastic Agent] Fix default configuration after enroll #18232
Conversation
Pinging @elastic/ingest-management (Project:fleet) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change LGTM. What I wonder is why it worked before? Did this just break recently?
related to configuration changes pushed in yesterday where we commented out all the options across codebase |
@michalpristas I assume in the end, no config is needed and we will have this default also in the code base? |
yes that would be nice and we would avoid situations like this during bulk editing |
💚 Build SucceededExpand to view the summary
Build stats
|
* fix enroll * changelog * changelog
What does this PR do?
Changes default configuration after enroll so agent is not in standalone mode
Why is it important?
To switch behavior from standalone to enroll
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.cc @ruflin @ph