-
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
apm-server: Remove data_streams.enabled
flag
#29168
apm-server: Remove data_streams.enabled
flag
#29168
Conversation
Removes the `"apm-server.data_streams.enabled` flag in the `apm-server` spec, since it's been marked as deprecated in `8.0`. Signed-off-by: Marc Lopez Rubio <[email protected]>
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
💚 Flaky test reportTests succeeded. 🤖 GitHub commentsTo re-run your PR in the CI, just comment with:
|
Signed-off-by: Marc Lopez Rubio <[email protected]>
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.
thanks!
/test |
* apm-server: Remove `data_streams.enabled` flag Removes the `"apm-server.data_streams.enabled` flag in the `apm-server` spec, since it's been marked as deprecated in `8.0`. Signed-off-by: Marc Lopez Rubio <[email protected]> (cherry picked from commit 6794652)
Removes the `"apm-server.data_streams.enabled` flag in the `apm-server` spec, since it's been marked as deprecated in `8.0`. Signed-off-by: Marc Lopez Rubio <[email protected]> (cherry picked from commit 6794652)
What does this PR do?
Removes the
"apm-server.data_streams.enabled
flag in theapm-server
spec, since it's been marked as deprecated in
8.0
.Why is it important?
To avoid
apm-server
logging deprecation messages.Checklist
- [ ] My code follows the style guidelines of this project- [ ] I have commented my code, particularly in hard-to-understand areas- [ ] I have made corresponding changes to the documentation- [ ] I have added tests that prove my fix is effective or that my feature works- [ ] I have added an entry inCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Related issues
elastic/apm-server#6606