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

[7.15] Add integration links metricbeat docs (#28110) #28136

Merged
merged 2 commits into from
Oct 6, 2021

Conversation

dedemorton
Copy link
Contributor

@dedemorton dedemorton commented Sep 27, 2021

Backports #28110 to 7.15 branch.

(Required manual cherrypick to run make update before opening PR.)

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Sep 27, 2021
@dedemorton dedemorton changed the title Backport 28110 7.15 [7.15] Add integration links metricbeat docs (#28110) Sep 27, 2021
@elasticmachine
Copy link
Collaborator

elasticmachine commented Sep 27, 2021

💔 Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-10-06T01:00:52.179+0000

  • Duration: 17 min 14 sec

  • Commit: d88b9e7

Steps errors 5

Expand to view the steps failures

metricbeat-lint - make -C metricbeat check;make -C metricbeat update;make -C x-pack/metricbeat che
  • Took 3 min 44 sec . View more details here
  • Description: make -C metricbeat check;make -C metricbeat update;make -C x-pack/metricbeat check;make -C x-pack/metricbeat update;make check-no-changes;
List files to upload
  • Took 0 min 0 sec . View more details here
  • Description: ls -l src/github.com/elastic/beats/build/system-tests-*.tar.gz
x-pack/metricbeat-lint - make -C x-pack/metricbeat check;make -C x-pack/metricbeat update;make -C
  • Took 3 min 57 sec . View more details here
  • Description: make -C x-pack/metricbeat check;make -C x-pack/metricbeat update;make -C metricbeat check;make -C metricbeat update;make check-no-changes;
List files to upload
  • Took 0 min 0 sec . View more details here
  • Description: ls -l src/github.com/elastic/beats/build/system-tests-*.tar.gz
Error signal
  • Took 0 min 0 sec . View more details here
  • Description: Error 'hudson.AbortException: script returned exit code 2'

❕ Flaky test report

No test was executed to be analysed.

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

@dedemorton dedemorton added the Team:Docs Label for the Observability docs team label Sep 27, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/obs-docs (Team:Docs)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Sep 27, 2021
@dedemorton
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/docs

@dedemorton
Copy link
Contributor Author

jenkins run the tests please

@dedemorton
Copy link
Contributor Author

/test

@dedemorton dedemorton self-assigned this Oct 5, 2021
* Link to integration docs from Metricbeat modules

* Fix broken links in Filebeat module docs
@dedemorton
Copy link
Contributor Author

Doc build passes. There are no errors related to this docs-only PR, so I am merging it.

@dedemorton dedemorton merged commit e0bb5b7 into elastic:7.15 Oct 6, 2021
@dedemorton dedemorton deleted the backport_28110_7.15 branch October 6, 2021 01:22
@v1v
Copy link
Member

v1v commented Oct 6, 2021

There are some linted errors in this PR for the metricbeats:

metricbeat-lint - make -C metricbeat check;make -C metricbeat update;make -C x-pack/metricbeat che

https://beats-ci.elastic.co//blue/rest/organizations/jenkins/pipelines/Beats/pipelines/beats/pipelines/PR-28136/runs/4/steps/1572/log/?start=0

x-pack/metricbeat-lint - make -C x-pack/metricbeat check;make -C x-pack/metricbeat update;make -C

https://beats-ci.elastic.co//blue/rest/organizations/jenkins/pipelines/Beats/pipelines/beats/pipelines/PR-28136/runs/4/steps/1580/log/?start=0

@v1v
Copy link
Member

v1v commented Oct 6, 2021

I wonder if the comment #28136 (comment) is too verbose or hard to understand.

Any suggestions to make this easier from your point of view?

@narph narph mentioned this pull request Oct 6, 2021
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport docs Team:Docs Label for the Observability docs team v7.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants