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

[DOCS] Add 'secrets' to Fleet settings page #172637

Merged
merged 4 commits into from
Dec 14, 2023

Conversation

kilfoyle
Copy link
Contributor

@kilfoyle kilfoyle commented Dec 5, 2023

This updates the list of available Fleet settings in the Kibana docs, as follows:

  • Moves the config and proxy_id settings from the Required to the Optional section of xpack.fleet.outputs. CC @nchaulet since the settings were added to the docs via [Docs] Add new Fleet preconfiguration settings #158771). [See orange highlight]
  • Adds the ssl setting with the certificate property (this setting is already available but doesn't appear to have been documented). [See red highlight]
  • Adds the new secrets setting, with SSL key as a property. [See blue highlight]
  • Adds an example output configuration. [See green highlight]

Rel: elastic/ingest-docs#692
See docs preview


Screenshot 2023-12-05 at 4 22 42 PM

@kilfoyle kilfoyle added Team:Docs backport:skip This commit does not require backporting labels Dec 5, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs (Team:Docs)

Copy link
Contributor

github-actions bot commented Dec 5, 2023

Documentation preview:

@apmmachine
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • /oblt-deploy-serverless : Deploy a serverless Kibana instance using the Observability test environments.
  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@kilfoyle kilfoyle added the release_note:skip Skip the PR/issue when compiling release notes label Dec 5, 2023
@kilfoyle kilfoyle added backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) and removed backport:skip This commit does not require backporting labels Dec 6, 2023
Copy link
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@jillguyonnet
Copy link
Contributor

jillguyonnet commented Dec 13, 2023

Approved with a small nit (typo).

I have couple of questions that could be for future iterations:

  1. There doesn't seem to be a mention of settings required by certain types of outputs, for example a Kafka output would require a topics settings. There are optional settings as well that are not mentioned. Is this a gap we'd like to fill?
    Edit: these are documented under Fleet UI settings, perhaps a link to that page for the various types of outputs would be helpful?
  2. Will the remote ES output type be documented separately?

@kilfoyle
Copy link
Contributor Author

kilfoyle commented Dec 13, 2023

There doesn't seem to be a mention of settings required by certain types of outputs, for example a Kafka output would require a topics settings. There are optional settings as well that are not mentioned. Is this a gap we'd like to fill?
Edit: these are documented under Fleet UI settings, perhaps a link to that page for the various types of outputs would be helpful?

Definitely! We should fill that gap. For now, I've added a link at the top of the page pointing to the Fleet UI settings page. However, I agree that this Kibana "Fleet settings" page really should document all available settings. I've opened a separate issue for that. If you can give me any guidance about where I can find a full list of those settings I'd appreciate it.

Will the remote ES output type be documented separately?

The remote ES output is documented here. In a separate PR I'll update that page to note that the "Service token" field can be stored as a secret setting. If you think it should also go on this "Fleet settings in Kibana" docs page, I'm happy to add it here too.

@kilfoyle kilfoyle merged commit c7fabda into elastic:main Dec 14, 2023
6 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 14, 2023
This updates the list of available Fleet settings in the Kibana docs, as
follows:

- Moves the `config` and `proxy_id` settings from the Required to the
Optional section of `xpack.fleet.outputs`. CC @nchaulet since the
settings were added to the docs via
elastic#158771). [See orange highlight]
- Adds the `ssl` setting with the `certificate` property (this setting
is already available but doesn't appear to have been documented). [See
red highlight]
- Adds the new `secrets` setting, with SSL `key` as a property. [See
blue highlight]
 - Adds an example output configuration. [See green highlight]

Rel: elastic/ingest-docs#692
See [docs
preview](https://kibana_172637.docs-preview.app.elstc.co/guide/en/kibana/master/fleet-settings-kb.html)

---

![Screenshot 2023-12-05 at 4 22 42
PM](https://github.com/elastic/kibana/assets/41695641/a547bad7-57aa-4470-8328-30b80f257973)

(cherry picked from commit c7fabda)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.12

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Dec 14, 2023
# Backport

This will backport the following commits from `main` to `8.12`:
- [[DOCS] Add 'secrets' to Fleet settings page
(#172637)](#172637)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"David
Kilfoyle","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-12-14T16:01:26Z","message":"[DOCS]
Add 'secrets' to Fleet settings page (#172637)\n\nThis updates the list
of available Fleet settings in the Kibana docs, as\r\nfollows:\r\n\r\n-
Moves the `config` and `proxy_id` settings from the Required to
the\r\nOptional section of `xpack.fleet.outputs`. CC @nchaulet since
the\r\nsettings were added to the docs
via\r\nhttps://github.com//pull/158771). [See orange
highlight]\r\n- Adds the `ssl` setting with the `certificate` property
(this setting\r\nis already available but doesn't appear to have been
documented). [See\r\nred highlight]\r\n- Adds the new `secrets` setting,
with SSL `key` as a property. [See\r\nblue highlight]\r\n - Adds an
example output configuration. [See green highlight]\r\n\r\nRel:
https://github.com/elastic/ingest-docs/issues/692\r\nSee
[docs\r\npreview](https://kibana_172637.docs-preview.app.elstc.co/guide/en/kibana/master/fleet-settings-kb.html)\r\n\r\n---\r\n\r\n![Screenshot
2023-12-05 at 4 22
42\r\nPM](https://github.com/elastic/kibana/assets/41695641/a547bad7-57aa-4470-8328-30b80f257973)","sha":"c7fabda4c59789facbbc6127589e66d162148669","branchLabelMapping":{"^v8.13.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Docs","release_note:skip","backport:prev-minor","v8.13.0"],"number":172637,"url":"https://github.com/elastic/kibana/pull/172637","mergeCommit":{"message":"[DOCS]
Add 'secrets' to Fleet settings page (#172637)\n\nThis updates the list
of available Fleet settings in the Kibana docs, as\r\nfollows:\r\n\r\n-
Moves the `config` and `proxy_id` settings from the Required to
the\r\nOptional section of `xpack.fleet.outputs`. CC @nchaulet since
the\r\nsettings were added to the docs
via\r\nhttps://github.com//pull/158771). [See orange
highlight]\r\n- Adds the `ssl` setting with the `certificate` property
(this setting\r\nis already available but doesn't appear to have been
documented). [See\r\nred highlight]\r\n- Adds the new `secrets` setting,
with SSL `key` as a property. [See\r\nblue highlight]\r\n - Adds an
example output configuration. [See green highlight]\r\n\r\nRel:
https://github.com/elastic/ingest-docs/issues/692\r\nSee
[docs\r\npreview](https://kibana_172637.docs-preview.app.elstc.co/guide/en/kibana/master/fleet-settings-kb.html)\r\n\r\n---\r\n\r\n![Screenshot
2023-12-05 at 4 22
42\r\nPM](https://github.com/elastic/kibana/assets/41695641/a547bad7-57aa-4470-8328-30b80f257973)","sha":"c7fabda4c59789facbbc6127589e66d162148669"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.13.0","labelRegex":"^v8.13.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/172637","number":172637,"mergeCommit":{"message":"[DOCS]
Add 'secrets' to Fleet settings page (#172637)\n\nThis updates the list
of available Fleet settings in the Kibana docs, as\r\nfollows:\r\n\r\n-
Moves the `config` and `proxy_id` settings from the Required to
the\r\nOptional section of `xpack.fleet.outputs`. CC @nchaulet since
the\r\nsettings were added to the docs
via\r\nhttps://github.com//pull/158771). [See orange
highlight]\r\n- Adds the `ssl` setting with the `certificate` property
(this setting\r\nis already available but doesn't appear to have been
documented). [See\r\nred highlight]\r\n- Adds the new `secrets` setting,
with SSL `key` as a property. [See\r\nblue highlight]\r\n - Adds an
example output configuration. [See green highlight]\r\n\r\nRel:
https://github.com/elastic/ingest-docs/issues/692\r\nSee
[docs\r\npreview](https://kibana_172637.docs-preview.app.elstc.co/guide/en/kibana/master/fleet-settings-kb.html)\r\n\r\n---\r\n\r\n![Screenshot
2023-12-05 at 4 22
42\r\nPM](https://github.com/elastic/kibana/assets/41695641/a547bad7-57aa-4470-8328-30b80f257973)","sha":"c7fabda4c59789facbbc6127589e66d162148669"}}]}]
BACKPORT-->

Co-authored-by: David Kilfoyle <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:Docs v8.12.0 v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants