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

Add additional Helm documentation for Fleet Server, and Agent #6154

Merged
merged 8 commits into from
Dec 29, 2022

Conversation

naemono
Copy link
Contributor

@naemono naemono commented Nov 8, 2022

This adds additional documentation for installing/managing Fleet Server, and managed Elastic Agents using the Helm charts.

While writing this, I noticed that the referenced examples in the documentation for Elasticsearch/Kibana were missing, and this also adds them where they need to be.

Add missing Elasticsearch/Kibana examples
Move Fleet example to folder.
@naemono naemono added the >docs Documentation label Nov 8, 2022
Copy link
Collaborator

@pebrc pebrc left a comment

Choose a reason for hiding this comment

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

LGTM but I would make the paths branch version specific. While testing this I realised that our Helm Chart release did not work properly. The Elastic Stack chart I get is still the one prior to the 2.5 release, without the Agent charts. We did not bump the Chart version of the stack chart when we added new dependencies which is a mistake, which I think we can only fix through a new release.

@naemono naemono added the v2.6.0 label Nov 30, 2022
Copy link
Contributor

@thbkrkr thbkrkr left a comment

Choose a reason for hiding this comment

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

Looks good, one last little thing: use current instead of master or 2.2 in the ECK doc URLs referenced in deploy/eck-stack/examples/elasticsearch/hot-warm-cold.yaml for consistency.

@thbkrkr
Copy link
Contributor

thbkrkr commented Dec 19, 2022

I resolved the conflict with main. Sorry for the late comments but this makes me ask 2 little questions.

Make documentation more clear concerning xpack fleet namespace settings.
Be consistent with namespace choice in readmes
@naemono naemono merged commit ea459ba into elastic:main Dec 29, 2022
naemono added a commit to naemono/cloud-on-k8s that referenced this pull request Dec 29, 2022
…c#6154)

* Add additional helm documentation for Fleet Server, and Agent
Add missing Elasticsearch/Kibana examples
Move Fleet example to folder.

* Make values links raw content, and using specific target tag.

* Use current instead of master in links.

Co-authored-by: Thibault Richard <[email protected]>

* Use current in links.

Co-authored-by: Thibault Richard <[email protected]>

* Update all doc links to point to current, not master, or a branch.

* Remove version from fleet example.
Make documentation more clear concerning xpack fleet namespace settings.
Be consistent with namespace choice in readmes

* Remove additional version setting.

Co-authored-by: Thibault Richard <[email protected]>
Co-authored-by: Thibault Richard <[email protected]>
(cherry picked from commit ea459ba)

# Conflicts:
#	deploy/eck-stack/examples/agent/fleet-agents.yaml
@naemono
Copy link
Contributor Author

naemono commented Dec 29, 2022

💚 All backports created successfully

Status Branch Result
2.6

Questions ?

Please refer to the Backport tool documentation

naemono added a commit that referenced this pull request Jan 3, 2023
…#6263)

* Add additional helm documentation for Fleet Server, and Agent
Add missing Elasticsearch/Kibana examples
Move Fleet example to folder.

* Make values links raw content, and using specific target tag.

* Use current instead of master in links.

Co-authored-by: Thibault Richard <[email protected]>

* Use current in links.

Co-authored-by: Thibault Richard <[email protected]>

* Update all doc links to point to current, not master, or a branch.

* Remove version from fleet example.
Make documentation more clear concerning xpack fleet namespace settings.
Be consistent with namespace choice in readmes

* Remove additional version setting.

Co-authored-by: Thibault Richard <[email protected]>
Co-authored-by: Thibault Richard <[email protected]>
(cherry picked from commit ea459ba)

# Conflicts:
#	deploy/eck-stack/examples/agent/fleet-agents.yaml
@thbkrkr thbkrkr changed the title Add additional helm documentation for Fleet Server, and Agent Add additional Helm documentation for Fleet Server, and Agent Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs Documentation v2.6.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants