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

Update installation tasks and remove persistent install steps #22290

Merged
merged 4 commits into from
Nov 9, 2020

Conversation

dedemorton
Copy link
Contributor

@dedemorton dedemorton commented Oct 30, 2020

The PR updates the existing procedures about managing Elastic Agent. There's a separate PR open to update the command reference and quick start docs, so please review these topics (it'll be easier than trying to make sense of what's in the widgets):

https://beats_22290.docs-preview.app.elstc.co/guide/en/ingest-management/master/elastic-agent-installation.html

https://beats_22290.docs-preview.app.elstc.co/guide/en/ingest-management/master/upgrade-elastic-agent.html

https://beats_22290.docs-preview.app.elstc.co/guide/en/ingest-management/master/uninstall-elastic-agent.html

https://beats_22290.docs-preview.app.elstc.co/guide/en/ingest-management/master/run-elastic-agent-standalone.html

https://beats_22290.docs-preview.app.elstc.co/guide/en/ingest-management/master/elastic-agent-configuration.html

Also please respond to questions flagged for //REVIEWERS in the source files.

TODO:

  • Fix bug in in topic about uninstall to fix display problem with nav.
  • After merging, fix cleanup todo items after the fleet guide is merged in observability-docs
  • Add info about directory structure and upgrade to the installation topic.
  • After 7.10 ships, move all how-to topics to the repo with other how-to info. Having content in different repos is very challenging when there are interdependencies.
  • Add topic about uninstalling agent.

@dedemorton dedemorton self-assigned this Oct 30, 2020
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Oct 30, 2020
@dedemorton dedemorton removed the needs_team Indicates that the issue/PR needs a Team:* label label Oct 30, 2020
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Oct 30, 2020
@elasticmachine
Copy link
Collaborator

elasticmachine commented Oct 30, 2020

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 1396
Skipped 4
Total 1400

@elasticmachine
Copy link
Collaborator

elasticmachine commented Oct 30, 2020

💚 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 #22290 updated]

  • Start Time: 2020-11-09T20:03:01.202+0000

  • Duration: 23 min 11 sec

Test stats 🧪

Test Results
Failed 0
Passed 1396
Skipped 4
Total 1400

@elasticmachine
Copy link
Collaborator

Pinging @elastic/ingest-management (Team:Ingest Management)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Oct 31, 2020
@dedemorton dedemorton requested review from ph and EricDavisX October 31, 2020 04:36
@dedemorton dedemorton force-pushed the update_agent_install_docs branch from 3d8513e to 05f8d76 Compare November 3, 2020 04:43
@dedemorton
Copy link
Contributor Author

@EricDavisX I've added some rough content about upgrading and uninstalling. Feel free to push commits to my fork if you want to add content (or add comments/suggestions to the review).

Copy link
Contributor

@ph ph left a comment

Choose a reason for hiding this comment

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

I've gone through the command and I think it look good.

@dedemorton
Copy link
Contributor Author

I'm going to merge this now to get the content building. I still have some to-do items and will address them in a follow-on PR. The docs also have some steps that still need to be tested (like deb/rpm) and are not 100% complete.

@dedemorton dedemorton merged commit 8d8d272 into elastic:master Nov 9, 2020
@dedemorton dedemorton deleted the update_agent_install_docs branch November 9, 2020 20:56
dedemorton added a commit to dedemorton/beats that referenced this pull request Nov 9, 2020
…c#22290)

* Update installation tasks and remove persistent install steps

* Fix bugs in widget code

* Add fixes from review and add to-do items
dedemorton added a commit to dedemorton/beats that referenced this pull request Nov 9, 2020
…c#22290)

* Update installation tasks and remove persistent install steps

* Fix bugs in widget code

* Add fixes from review and add to-do items
dedemorton added a commit that referenced this pull request Nov 9, 2020
#22504)

* Update installation tasks and remove persistent install steps

* Fix bugs in widget code

* Add fixes from review and add to-do items
dedemorton added a commit that referenced this pull request Nov 9, 2020
#22505)

* Update installation tasks and remove persistent install steps

* Fix bugs in widget code

* Add fixes from review and add to-do items
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