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

Reconcile duplicated attributes #10956

Merged
merged 11 commits into from
Mar 1, 2019
Merged

Conversation

dedemorton
Copy link
Contributor

Fixes issue #6144 by removing attributes that duplicate attributes defined here.

Note that I've changed all paths to use branch instead of doc-branch to be consistent with the shared attributes file. However, I have not removed doc-branch from version.asciidoc because the attribute is used by make docs.

I've also removed attributes that are not used anywhere.

@bmorelli25 Please confirm that APM does not use any of the attributes that I've removed.

@dedemorton dedemorton added docs review needs_backport PR is waiting to be backported to other branches. labels Feb 27, 2019
@dedemorton dedemorton requested review from bmorelli25 and a team February 27, 2019 04:36
@dedemorton dedemorton requested review from a team as code owners February 27, 2019 04:36
Copy link
Member

@bmorelli25 bmorelli25 left a comment

Choose a reason for hiding this comment

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

Looks like we use {libbeat}, {elasticsearch}, and {doc-branch} in the APM docs. I'll pull this PR and make the necessary changes in the APM repo.

Copy link
Contributor

@cachedout cachedout left a comment

Choose a reason for hiding this comment

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

No problems from the stack monitoring side of things. 👍

@dedemorton dedemorton merged commit 2e2b2a9 into elastic:master Mar 1, 2019
@dedemorton dedemorton deleted the issue#6144 branch March 1, 2019 17:17
dedemorton added a commit to dedemorton/beats that referenced this pull request Mar 22, 2019
* Replace libbeat with beats-ref

* Replace metricbeat with metricbeat-ref

* Replace filebeat with filebeat-ref

* Replace auditbeat with auditbeat-ref

* Replace logstashdoc with logstash-ref

* Replace elasticsearch with ref

* Replace elasticsearch-plugins with plugins

* Replace securitydoc with stack-ov

* Replace monitoringdoc with stack-ov

* Remove unused attributes

* Replace doc-branch with branch
dedemorton added a commit that referenced this pull request Mar 22, 2019
* Reconcile duplicated attributes (#10956)

* Replace libbeat with beats-ref

* Replace metricbeat with metricbeat-ref

* Replace filebeat with filebeat-ref

* Replace auditbeat with auditbeat-ref

* Replace logstashdoc with logstash-ref

* Replace elasticsearch with ref

* Replace elasticsearch-plugins with plugins

* Replace securitydoc with stack-ov

* Replace monitoringdoc with stack-ov

* Remove unused attributes

* Replace doc-branch with branch

* Replace remaining attributes

* Run make update
dedemorton added a commit to dedemorton/beats that referenced this pull request Mar 22, 2019
* Replace libbeat with beats-ref

* Replace metricbeat with metricbeat-ref

* Replace filebeat with filebeat-ref

* Replace auditbeat with auditbeat-ref

* Replace logstashdoc with logstash-ref

* Replace elasticsearch with ref

* Replace elasticsearch-plugins with plugins

* Replace securitydoc with stack-ov

* Replace monitoringdoc with stack-ov

* Remove unused attributes

* Replace doc-branch with branch
dedemorton added a commit that referenced this pull request Mar 22, 2019
* Reconcile duplicated attributes (#10956)

* Replace libbeat with beats-ref

* Replace metricbeat with metricbeat-ref

* Replace filebeat with filebeat-ref

* Replace auditbeat with auditbeat-ref

* Replace logstashdoc with logstash-ref

* Replace elasticsearch with ref

* Replace elasticsearch-plugins with plugins

* Replace securitydoc with stack-ov

* Replace monitoringdoc with stack-ov

* Remove unused attributes

* Replace doc-branch with branch

* Replace remaining attributes
dedemorton added a commit to dedemorton/beats that referenced this pull request Mar 22, 2019
* Replace libbeat with beats-ref

* Replace metricbeat with metricbeat-ref

* Replace filebeat with filebeat-ref

* Replace auditbeat with auditbeat-ref

* Replace logstashdoc with logstash-ref

* Replace elasticsearch with ref

* Replace elasticsearch-plugins with plugins

* Replace securitydoc with stack-ov

* Replace monitoringdoc with stack-ov

* Remove unused attributes

* Replace doc-branch with branch
@dedemorton dedemorton removed the needs_backport PR is waiting to be backported to other branches. label Mar 22, 2019
dedemorton added a commit that referenced this pull request Mar 22, 2019
* Replace libbeat with beats-ref

* Replace metricbeat with metricbeat-ref

* Replace filebeat with filebeat-ref

* Replace auditbeat with auditbeat-ref

* Replace logstashdoc with logstash-ref

* Replace elasticsearch with ref

* Replace elasticsearch-plugins with plugins

* Replace securitydoc with stack-ov

* Replace monitoringdoc with stack-ov

* Remove unused attributes

* Replace doc-branch with branch
DStape pushed a commit to DStape/beats that referenced this pull request Aug 20, 2019
* Replace libbeat with beats-ref

* Replace metricbeat with metricbeat-ref

* Replace filebeat with filebeat-ref

* Replace auditbeat with auditbeat-ref

* Replace logstashdoc with logstash-ref

* Replace elasticsearch with ref

* Replace elasticsearch-plugins with plugins

* Replace securitydoc with stack-ov

* Replace monitoringdoc with stack-ov

* Remove unused attributes

* Replace doc-branch with branch
DStape pushed a commit to DStape/beats that referenced this pull request Aug 20, 2019
* Reconcile duplicated attributes (elastic#10956)

* Replace libbeat with beats-ref

* Replace metricbeat with metricbeat-ref

* Replace filebeat with filebeat-ref

* Replace auditbeat with auditbeat-ref

* Replace logstashdoc with logstash-ref

* Replace elasticsearch with ref

* Replace elasticsearch-plugins with plugins

* Replace securitydoc with stack-ov

* Replace monitoringdoc with stack-ov

* Remove unused attributes

* Replace doc-branch with branch

* Replace remaining attributes
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
* Reconcile duplicated attributes (elastic#10956)

* Replace libbeat with beats-ref

* Replace metricbeat with metricbeat-ref

* Replace filebeat with filebeat-ref

* Replace auditbeat with auditbeat-ref

* Replace logstashdoc with logstash-ref

* Replace elasticsearch with ref

* Replace elasticsearch-plugins with plugins

* Replace securitydoc with stack-ov

* Replace monitoringdoc with stack-ov

* Remove unused attributes

* Replace doc-branch with branch

* Replace remaining attributes

* Run make update
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