Skip to content
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

Merged
merged 3 commits into from
Dec 21, 2018

Conversation

jsoriano
Copy link
Member

I'd say we want to keep Jolokia autodiscover as is, so remove the experimental tag for 7.0.

@jsoriano jsoriano added review libbeat v7.0.0 Team:Integrations Label for the Integrations team labels Dec 20, 2018
@jsoriano jsoriano self-assigned this Dec 20, 2018
@jsoriano jsoriano requested a review from a team as a code owner December 20, 2018 11:13
CHANGELOG.asciidoc Outdated Show resolved Hide resolved
@jsoriano
Copy link
Member Author

jenkins, test this again please

@@ -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]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rebase issue?

Copy link
Member Author

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.

@ruflin
Copy link
Member

ruflin commented Dec 21, 2018

@jsoriano Should we backport this to 6.x?

@jsoriano
Copy link
Member Author

@ruflin I was thinking on doing this for 7.0, but I could backport it for 6.7, as you prefer

@ruflin
Copy link
Member

ruflin commented Dec 21, 2018

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?

@jsoriano
Copy link
Member Author

Ok, I will backport it till 6.6 🙂

@jsoriano jsoriano added needs_backport PR is waiting to be backported to other branches. v6.6.0 v6.7.0 labels Dec 21, 2018
@jsoriano jsoriano merged commit d286b62 into elastic:master Dec 21, 2018
@jsoriano jsoriano deleted the jolokia-autodiscover-ga branch December 21, 2018 21:11
@jsoriano jsoriano removed the needs_backport PR is waiting to be backported to other branches. label Dec 21, 2018
jsoriano added a commit to jsoriano/beats that referenced this pull request Dec 21, 2018
jsoriano added a commit to jsoriano/beats that referenced this pull request Dec 21, 2018
jsoriano added a commit that referenced this pull request Dec 22, 2018
jsoriano added a commit that referenced this pull request Dec 22, 2018
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants