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 agent_config sample configuration #5366

Merged

Conversation

stuartnelson3
Copy link
Contributor

Motivation/summary

Since this is predominantly existing for use by
elastic-agent/fleet, exposing this as manual
configuration in the .yml file is not necessary.

Checklist

  • Documentation has been updated

How to test these changes

Non-functional change

Since this is predominantly existing for use by
elastic-agent/fleet, exposing this as manual
configuration in the .yml file is not necessary.
@stuartnelson3 stuartnelson3 requested a review from a team May 28, 2021 09:27
@stuartnelson3 stuartnelson3 mentioned this pull request May 28, 2021
1 task
@apmmachine
Copy link
Contributor

💚 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 #5366 opened

  • Start Time: 2021-05-28T09:27:09.802+0000

  • Duration: 48 min 45 sec

  • Commit: 80ded05

Test stats 🧪

Test Results
Failed 0
Passed 6146
Skipped 120
Total 6266

Trends 🧪

Image of Build Times

Image of Tests

@stuartnelson3 stuartnelson3 merged commit 39b703c into elastic:master May 28, 2021
@stuartnelson3 stuartnelson3 deleted the remove-agent-configs-example branch May 28, 2021 11:24
mergify bot pushed a commit that referenced this pull request May 28, 2021
Since this is predominantly existing for use by
elastic-agent/fleet, exposing this as manual
configuration in the .yml file is not necessary.

(cherry picked from commit 39b703c)
stuartnelson3 added a commit that referenced this pull request May 28, 2021
Since this is predominantly existing for use by
elastic-agent/fleet, exposing this as manual
configuration in the .yml file is not necessary.

(cherry picked from commit 39b703c)

Co-authored-by: stuart nelson <[email protected]>
stuartnelson3 added a commit to stuartnelson3/apm-server that referenced this pull request May 31, 2021
Since this is predominantly existing for use by
elastic-agent/fleet, exposing this as manual
configuration in the .yml file is not necessary.
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