Skip to content

Transport profiles do not inherit default transport settings #86048

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ppf2
Copy link
Member

@ppf2 ppf2 commented Apr 20, 2022

Transport profiles do not inherit default transport settings. This PR clarifies the documentation. Please back port accordingly.

Related: #39960

@ppf2 ppf2 added the >docs General docs changes label Apr 20, 2022
@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label Apr 20, 2022
@elasticmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

@ppf2 please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation.

@elasticsearchmachine elasticsearchmachine added v8.3.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Apr 20, 2022
@elasticsearchmachine elasticsearchmachine changed the base branch from master to main July 22, 2022 23:07
@mark-vieira mark-vieira added v8.5.0 and removed v8.4.0 labels Jul 27, 2022
@csoulios csoulios added v8.6.0 and removed v8.5.0 labels Sep 21, 2022
@kingherc kingherc added v8.7.0 and removed v8.6.0 labels Nov 16, 2022
@rjernst rjernst added v8.8.0 and removed v8.7.0 labels Feb 8, 2023
@stu-elastic stu-elastic added the :Security/TLS SSL/TLS, Certificates label Mar 29, 2023
@elasticsearchmachine elasticsearchmachine added the Team:Security Meta label for security team label Mar 29, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

@abdonpijpelink
Copy link
Contributor

@elasticmachine retest this please

Comment on lines +2348 to +2352
NOTE: Transport profiles do not inherit TLS/SSL settings
from the default transport.

The same settings that are available for the <<transport-tls-ssl-settings, default transport>>
are also available for each transport profile. By default, the settings for a
transport profile will be the same as the default transport unless they
are specified.
are also available for each transport profile.
Copy link
Member

Choose a reason for hiding this comment

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

The xpack.security.transport.ssl.enabled setting is an exception:

  1. It controls whether SSL is enabled for both the default transport and any transport profiles.
  2. Transport profiles do not have a corresponding setting for it.

@bpintea bpintea removed the v8.8.1 label Jun 6, 2023
@bpintea bpintea added the v8.8.2 label Jun 6, 2023
@joegallo joegallo added v8.8.3 and removed v8.8.2 labels Jun 23, 2023
@quux00 quux00 added v8.11.0 and removed v8.10.0 labels Aug 16, 2023
@mattc58 mattc58 added v8.12.0 and removed v8.11.0 labels Oct 4, 2023
@mark-vieira mark-vieira added auto-backport Automatically create backport pull requests when merged and removed auto-backport-and-merge labels Oct 4, 2024
@leemthompo
Copy link
Contributor

Important

Elastic documentation is migrating to Markdown for version 9.0+. See the migration guide for details.

ℹ️ What's happening?

  • Starting January 29, we will start closing all unmerged documentation PRs targeting main/master
  • We're migrating from AsciiDoc to Markdown for 9.0+
  • 9.0 docs will be frozen from January 29 until February 20 2024
  • NOTE: PRs that include both code and documentation changes will remain open

What do I need to do?

For <=8.x docs:

  1. Rebase your PR to target the relevant 8.x branch instead
  2. The content can remain in AsciiDoc format

For 9.0+ docs:

Option 1:

  • Draft docs in Markdown
  • Once migration freeze ends, find the relevant page in the new docs system and use the edit options to submit your changes

Option 2:

💡 Need help?

  1. For Elasticians: Ask in #docs Slack channel
  2. For external contributors: Open an issue in elastic/docs-content

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged >docs General docs changes external-contributor Pull request authored by a developer outside the Elasticsearch team :Security/TLS SSL/TLS, Certificates Team:Docs Meta label for docs team Team:Security Meta label for security team v8.7.2 v8.8.3 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.