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

Removed Envoy entry on docs #8783

Merged
merged 3 commits into from
Nov 27, 2018

Conversation

sayden
Copy link
Contributor

@sayden sayden commented Oct 29, 2018

Related with this issue #8759

I have run make update and it seems that the docs have been updated but I'm still not sure if this is going to solve the conflicting document

@sayden sayden requested review from jsoriano and ruflin October 29, 2018 15:45
@ruflin ruflin added the needs_backport PR is waiting to be backported to other branches. label Oct 29, 2018
Copy link
Collaborator

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

Change LGTM, one minor comment for the changelog.

We should backport this to 6.5 and 6.x

@@ -421,6 +421,7 @@ https://github.com/elastic/beats/compare/v6.3.0...v6.3.1[View commits]
- Fix field mapping for the system process CPU ticks fields. {pull}7230[7230]
- Ensure canonical naming for JMX beans is disabled in Jolokia module. {pull}7047[7047]
- Fix Jolokia attribute mapping when using wildcards and MBean names with multiple properties. {pull}7321[7321]
- Remove duplicated envoy entry from docs {issue}8759[8759]
Copy link
Collaborator

Choose a reason for hiding this comment

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

We normally don't add changes about the docs to the changelog.

@ruflin
Copy link
Collaborator

ruflin commented Oct 29, 2018

I now realise we should potentially also backport it to 6.4 if the issue is also in the 6.4 docs.

@ruflin ruflin added the Team:Integrations Label for the Integrations team label Nov 23, 2018
@ruflin
Copy link
Collaborator

ruflin commented Nov 23, 2018

@sayden What is the status here? This probably only needs a rebase.

@sayden sayden force-pushed the remove_duplicated_envoy_entry_on_docs branch from b36dfd3 to dec0e91 Compare November 27, 2018 07:40
@sayden sayden merged commit 238e55d into elastic:master Nov 27, 2018
@sayden sayden added v6.6.0 and removed needs_backport PR is waiting to be backported to other branches. labels Nov 27, 2018
sayden added a commit to sayden/beats that referenced this pull request Nov 27, 2018
sayden added a commit that referenced this pull request Nov 27, 2018
@jsoriano
Copy link
Member

Don't forget to backport to 6.5 too 🙂 and maybe to 6.4.

sayden added a commit to sayden/beats that referenced this pull request Nov 27, 2018
@sayden sayden added the v6.5.2 label Nov 27, 2018
@sayden sayden added the v6.4.4 label Nov 27, 2018
sayden added a commit to sayden/beats that referenced this pull request Nov 27, 2018
sayden added a commit that referenced this pull request Nov 27, 2018
sayden added a commit that referenced this pull request Nov 27, 2018
@sayden sayden deleted the remove_duplicated_envoy_entry_on_docs branch August 25, 2022 12:50
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
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.

3 participants