-
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
Fix output.elasticsearch.proxy_disable flag #27671
Fix output.elasticsearch.proxy_disable flag #27671
Conversation
Rename the output.elasticsearch.proxy_disabled flag to proxy_disable as that is what the flag appears as under all other settings including fleet.
Pinging @elastic/agent (Team:Agent) |
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪💚 Flaky test reportTests succeeded. Expand to view the summary
Test stats 🧪
|
/test |
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.
Looks good.
/test |
Shouldn't we do it the other way around and rename what is used in fleet-server? |
@ruflin |
7.14.1 is shipped, so it will be included in 7.14.2 (if and when that ships) |
@michel-laterman SGTM |
* Fix output.elasticsearch.proxy_disable flag Rename the output.elasticsearch.proxy_disabled flag to proxy_disable as that is what the flag appears as under all other settings including fleet. * Add pr number to CHANGELOG (cherry picked from commit 0c9e194)
* Fix output.elasticsearch.proxy_disable flag Rename the output.elasticsearch.proxy_disabled flag to proxy_disable as that is what the flag appears as under all other settings including fleet. * Add pr number to CHANGELOG (cherry picked from commit 0c9e194)
* Fix output.elasticsearch.proxy_disable flag Rename the output.elasticsearch.proxy_disabled flag to proxy_disable as that is what the flag appears as under all other settings including fleet. * Add pr number to CHANGELOG (cherry picked from commit 0c9e194)
…27710) * Fix output.elasticsearch.proxy_disable flag (#27671) * Fix output.elasticsearch.proxy_disable flag Rename the output.elasticsearch.proxy_disabled flag to proxy_disable as that is what the flag appears as under all other settings including fleet. * Add pr number to CHANGELOG (cherry picked from commit 0c9e194) * Fix CHANGELOG Co-authored-by: Michel Laterman <82832767+michel-laterman@users.noreply.github.com> Co-authored-by: michel-laterman <michel.laterman@elastic.co>
…27711) * Fix output.elasticsearch.proxy_disable flag (#27671) * Fix output.elasticsearch.proxy_disable flag Rename the output.elasticsearch.proxy_disabled flag to proxy_disable as that is what the flag appears as under all other settings including fleet. * Add pr number to CHANGELOG (cherry picked from commit 0c9e194) * Fix CHANGELOG Co-authored-by: Michel Laterman <82832767+michel-laterman@users.noreply.github.com> Co-authored-by: michel-laterman <michel.laterman@elastic.co>
…27709) * Fix output.elasticsearch.proxy_disable flag (#27671) * Fix output.elasticsearch.proxy_disable flag Rename the output.elasticsearch.proxy_disabled flag to proxy_disable as that is what the flag appears as under all other settings including fleet. * Add pr number to CHANGELOG (cherry picked from commit 0c9e194) * Fix CHANGELOG Co-authored-by: Michel Laterman <82832767+michel-laterman@users.noreply.github.com> Co-authored-by: michel-laterman <michel.laterman@elastic.co>
* Fix output.elasticsearch.proxy_disable flag Rename the output.elasticsearch.proxy_disabled flag to proxy_disable as that is what the flag appears as under all other settings including fleet. * Add pr number to CHANGELOG
What does this PR do?
Rename the output.elasticsearch.proxy_disabled flag to proxy_disable as
that is what the flag appears as under all other settings including
fleet.
Why is it important?
fleet-server expects the setting to be named proxy_disable
Checklist
I have made corresponding change to the default configuration filesI have added tests that prove my fix is effective or that my feature worksCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Related issues