Skip to content
This repository was archived by the owner on Sep 17, 2024. It is now read-only.

docs: document integrations in the proper README file#1181

Merged
mdelapenya merged 1 commit intoelastic:masterfrom
mdelapenya:1105-document-integrations
May 18, 2021
Merged

docs: document integrations in the proper README file#1181
mdelapenya merged 1 commit intoelastic:masterfrom
mdelapenya:1105-document-integrations

Conversation

@mdelapenya
Copy link
Copy Markdown
Contributor

@mdelapenya mdelapenya commented May 18, 2021

What does this PR do?

It simply moves one explanation about how to install Beats' integrations from fleet test suite docs to the Metricbeat one.

Why is it important?

Fix the bug in docs

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the Unit tests for the CLI, and they are passing locally
  • I have run the End-2-End tests for the suite I'm working on, and they are passing locally
  • I have noticed new Go dependencies (run make notice in the proper directory)

Related issues

@mdelapenya mdelapenya self-assigned this May 18, 2021
@mdelapenya mdelapenya requested a review from a team May 18, 2021 17:35
@mdelapenya mdelapenya marked this pull request as ready for review May 18, 2021 17:35
@mdelapenya mdelapenya added v7.13.0 v7.14.0 area:documentation Anything related to the Observability Robots documentation bug Something isn't working labels May 18, 2021
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented May 18, 2021

💚 Build Succeeded

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

Expand to view the summary

Build stats

  • Build Cause: Pull request #1181 opened

  • Start Time: 2021-05-18T17:35:35.499+0000

  • Duration: 10 min 36 sec

  • Commit: 705152a

Test stats 🧪

Test Results
Failed 0
Passed 19
Skipped 0
Total 19

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 19
Skipped 0
Total 19

@mdelapenya mdelapenya merged commit 24273bd into elastic:master May 18, 2021
mergify bot pushed a commit that referenced this pull request May 18, 2021
@mdelapenya
docs: document integrations in the proper README file (#1181)
f5bfcaa 
(cherry picked from commit 24273bd)
@mergify mergify bot mentioned this pull request May 18, 2021
Merged
mergify bot pushed a commit that referenced this pull request May 18, 2021
@mdelapenya
docs: document integrations in the proper README file (#1181)
455ffab 
(cherry picked from commit 24273bd)
@mergify mergify bot mentioned this pull request May 18, 2021
Merged
adam-stokes pushed a commit that referenced this pull request May 18, 2021
@mergify @mdelapenya
docs: document integrations in the proper README file (#1181) (#1182)
1a44769 
(cherry picked from commit 24273bd)

Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
adam-stokes pushed a commit that referenced this pull request May 18, 2021
@mergify @mdelapenya
docs: document integrations in the proper README file (#1181) (#1183)
2ea8991 
(cherry picked from commit 24273bd)

Co-authored-by: Manuel de la Peña <mdelapenya@gmail.com>
@mdelapenya mdelapenya deleted the 1105-document-integrations branch May 19, 2021 05:45
mdelapenya added a commit to mdelapenya/e2e-testing that referenced this pull request May 19, 2021
@mdelapenya
Merge branch 'master' into arm-suppport
84b93b4 
* master:
  fix: run linux integration on CI (elastic#1186)
  docs: document integrations in the proper README file (elastic#1181)
  fix: resolve issues in k8s-autodiscover test suite (elastic#1171)
  feat: make a stronger verification of fleet-server being bootstrapped (elastic#1164)
@mdelapenya mdelapenya mentioned this pull request Oct 26, 2021
Merged
8 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@v1v v1v v1v approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@mdelapenya mdelapenya

Labels

area:documentation Anything related to the Observability Robots documentation bug Something isn't working v7.13.0 v7.14.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Documented integrations sync for metricbeat test suite

3 participants

@mdelapenya @elasticmachine @v1v