-
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
Remove the experimental tag from Jolokia autodiscover provider #9706
Conversation
03c097a
to
e04bad3
Compare
jenkins, test this again please |
CHANGELOG.asciidoc
Outdated
@@ -200,6 +200,12 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha1...v7.0.0-alpha2[Check the | |||
- Unify dashboard exporter tools. {pull}9097[9097] | |||
- Add cache.ttl to add_host_metadata. {pull}9359[9359] | |||
- Add support for index lifecycle management (beta). {pull}7963[7963] | |||
- Always include Pod UID as part of Pod metadata. {pull]9517[9517] |
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.
Rebase issue?
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.
Yes, there was something wrong there, fixed.
068caaf
to
ced27e7
Compare
@jsoriano Should we backport this to 6.x? |
@ruflin I was thinking on doing this for 7.0, but I could backport it for 6.7, as you prefer |
I assume there are no code changes in this for 6.6 / 6.7 or master? If yes, let's backport. I'm even thinking about updating 6.6? |
Ok, I will backport it till 6.6 🙂 |
(cherry picked from commit d286b62)
(cherry picked from commit d286b62)
) (cherry picked from commit 7aa51f1)
I'd say we want to keep Jolokia autodiscover as is, so remove the experimental tag for 7.0.