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 getting started guide to include Fleet Server #602

Merged
merged 2 commits into from
May 5, 2021

Conversation

dedemorton
Copy link
Contributor

@dedemorton dedemorton commented Apr 29, 2021

Fixes some of the items listed in #522

Please respond to inline questions.

Previews:

NOTE: This PR is for quick starts only. I'll update the other installation topics in a follow-up PR.

@apmmachine
Copy link
Contributor

apmmachine commented Apr 29, 2021

A documentation preview will be available soon:

@dedemorton dedemorton marked this pull request as ready for review May 4, 2021 01:20
@dedemorton dedemorton requested a review from a team as a code owner May 4, 2021 01:20
@dedemorton dedemorton requested a review from mostlyjason May 4, 2021 01:20
@dedemorton dedemorton self-assigned this May 4, 2021
@dedemorton dedemorton added v7.13.0 Team:Fleet Label for the Fleet team labels May 4, 2021
Copy link
Member

@bmorelli25 bmorelli25 left a comment

Choose a reason for hiding this comment

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

LGTM!

Comment on lines 68 to 69
//QUESTION: The build of cloud I'm using doesn't show a default policy. Is that
//a bug, or expected?
Copy link
Member

Choose a reason for hiding this comment

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

I think this question is more for Jason than me, but where are you seeing this? I'm not able to replicate on ESS staging.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I saw it on cloud staging yesterday, but it sounds like there was a problem. I'll just remove this comment because I think we should test the quick starts again when everything is working for real.

@dedemorton
Copy link
Contributor Author

I'm going to merge and backport so we can get these updates into the build for testers. We can add more changes in a follow-on PR, if necessary.

@dedemorton dedemorton merged commit 99c4ef7 into elastic:master May 5, 2021
@dedemorton dedemorton deleted the issue#522 branch May 5, 2021 18:42
dedemorton added a commit to dedemorton/observability-docs that referenced this pull request May 5, 2021
* Update getting started guide to include Fleet Server

* Remove inline question
dedemorton added a commit to dedemorton/observability-docs that referenced this pull request May 5, 2021
* Update getting started guide to include Fleet Server

* Remove inline question
dedemorton added a commit that referenced this pull request May 7, 2021
* Update getting started guide to include Fleet Server

* Remove inline question
dedemorton added a commit that referenced this pull request May 7, 2021
* Update getting started guide to include Fleet Server

* Remove inline question
bmorelli25 pushed a commit to bmorelli25/observability-docs that referenced this pull request Apr 11, 2023
* Update getting started guide to include Fleet Server

* Remove inline question
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Fleet Label for the Fleet team v7.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants