Skip to content

Conversation

@kilfoyle
Copy link
Contributor

@kilfoyle kilfoyle commented May 16, 2023

This fixes up these pages:

Changes include:

  • Updated screen captures to match the current UI
  • Removed the separate Add Fleet Server Hosts section since that's now part of the Install Fleet Server wizard (and this was causing some difficulty).
  • In the on-prem and self-managed steps, added a note about --fleet-server-host and --fleet-server-port, per this request from Luca.
  • Tried to make the two pages more similar in flow.
  • Changed "see" to "refer to" per accessibility guidelines.
  • Miscellaneous small changes, mostly cosmetic.

For now, I've commented out some content rather than removing it, just in case at some point we need to trace back on the history of what was changed.

Closes: #99

@kilfoyle kilfoyle marked this pull request as draft May 16, 2023 21:51
@kilfoyle kilfoyle requested a review from karenzone May 16, 2023 22:18
@kilfoyle kilfoyle marked this pull request as ready for review May 16, 2023 22:18
Comment on lines +196 to +200
* If you've configured a non-default port for {fleet-server} in the
{fleet-server} integration, you need to include the `--fleet-server-host` and
`--fleet-server-port` options in the `elastic-agent install` command. Refer to the
{fleet-guide}/elastic-agent-cmd-options.html#elastic-agent-install-command[install command documentation]
for details.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@lucabelluccini, here's the note that I added to address your comment in the parent issue. Please let me know if I didn't get it right. :-)

Screenshot 2023-05-16 at 6 22 09 PM

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

Left comments for consideration. Otherwise, LGTM

kilfoyle and others added 2 commits May 19, 2023 10:09
Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
@kilfoyle kilfoyle merged commit 681bf27 into elastic:main May 23, 2023
kilfoyle added a commit to kilfoyle/ingest-docs that referenced this pull request May 23, 2023
* Update Fleet Server on-prem setup docs

* Remove tagged regions since Cloud and on-prem are too different

* touchup

* Update docs/en/ingest-management/fleet/add-fleet-server-on-prem.asciidoc

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>

* Update docs/en/ingest-management/fleet/add-fleet-server-on-prem.asciidoc

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>

---------

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
(cherry picked from commit 681bf27)
kilfoyle added a commit to kilfoyle/ingest-docs that referenced this pull request May 23, 2023
* Update Fleet Server on-prem setup docs

* Remove tagged regions since Cloud and on-prem are too different

* touchup

* Update docs/en/ingest-management/fleet/add-fleet-server-on-prem.asciidoc

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>

* Update docs/en/ingest-management/fleet/add-fleet-server-on-prem.asciidoc

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>

---------

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
(cherry picked from commit 681bf27)
@kilfoyle
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.8
8.7

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kilfoyle added a commit that referenced this pull request May 23, 2023
* Update Fleet Server on-prem setup docs

* Remove tagged regions since Cloud and on-prem are too different

* touchup

* Update docs/en/ingest-management/fleet/add-fleet-server-on-prem.asciidoc

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>

* Update docs/en/ingest-management/fleet/add-fleet-server-on-prem.asciidoc

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>

---------

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
(cherry picked from commit 681bf27)
kilfoyle added a commit that referenced this pull request May 23, 2023
* Update Fleet Server on-prem setup docs

* Remove tagged regions since Cloud and on-prem are too different

* touchup

* Update docs/en/ingest-management/fleet/add-fleet-server-on-prem.asciidoc

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>

* Update docs/en/ingest-management/fleet/add-fleet-server-on-prem.asciidoc

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>

---------

Co-authored-by: Karen Metts <35154725+karenzone@users.noreply.github.com>
(cherry picked from commit 681bf27)
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.

[REQUEST]: Update Fleet deployment instructions to match latest UI

2 participants