Skip to content

Add support for disabling encryption and authentication - Documentation - #13041

Open
scholzj wants to merge 4 commits into
scholzj/add-none-encryption-and-authentication-kafka-exporterfrom
scholzj/add-none-encryption-and-authentication-docs
Open

Add support for disabling encryption and authentication - Documentation#13041
scholzj wants to merge 4 commits into
scholzj/add-none-encryption-and-authentication-kafka-exporterfrom
scholzj/add-none-encryption-and-authentication-docs

Conversation

@scholzj

@scholzj scholzj commented Aug 8, 2026

Copy link
Copy Markdown
Member

Type of change

  • Documentation

Description

This PR continues the work on disabling TLS encryption and mTLS authentication in the Kafka clusters and adds the related documentation and CHANGELOG.md changes.

Checklist

  • Update documentation
  • Update CHANGELOG.md (if present)
  • AI assistance was used to create this PR (see the Strimzi AI policy)

@scholzj scholzj added this to the 1.3.0 milestone Aug 8, 2026
@scholzj
scholzj requested a review from a team August 8, 2026 16:01
@snyk-io

snyk-io Bot commented Aug 8, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codecov

codecov Bot commented Aug 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.56%. Comparing base (f581ca9) to head (f972bad).

Additional details and impacted files
@@                                         Coverage Diff                                         @@
##             scholzj/add-none-encryption-and-authentication-kafka-exporter   #13041      +/-   ##
===================================================================================================
- Coverage                                                            80.58%   80.56%   -0.03%     
+ Complexity                                                            6581     6578       -3     
===================================================================================================
  Files                                                                  344      344              
  Lines                                                                22776    22776              
  Branches                                                              3111     3111              
===================================================================================================
- Hits                                                                 18355    18349       -6     
- Misses                                                                3205     3206       +1     
- Partials                                                              1216     1221       +5     

see 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@scholzj
scholzj force-pushed the scholzj/add-none-encryption-and-authentication-docs branch 2 times, most recently from 775c2ad to 8ed2323 Compare August 8, 2026 21:56
@scholzj
scholzj force-pushed the scholzj/add-none-encryption-and-authentication-docs branch from 8ed2323 to b0d5579 Compare August 9, 2026 14:42
@scholzj
scholzj force-pushed the scholzj/add-none-encryption-and-authentication-docs branch from b0d5579 to 1d64cac Compare August 11, 2026 09:41
@scholzj
scholzj force-pushed the scholzj/add-none-encryption-and-authentication-docs branch 2 times, most recently from fb0a1e4 to 6ec1a10 Compare August 11, 2026 13:15
If the `strimzi.io/internal-cluster-security` annotation is not present, Strimzi uses `strimzi-tls` encryption with `strimzi-mtls` authentication.
This behavior is unchanged from previous releases.

The following explicit configuration is equivalent to omitting the annotation:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What's the reason for including this snippet? Presumably we want to encourage people not to add this config if they don't need it, especially since the API is expected to change

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@katheris I won't lie ... the AI included it there and I did not see any reason why not to include it. 🤷 Do you want me to remove it?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think I would prefer to remove it, just because to me it adds unnecessary length to the docs that people have to scroll past

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ok, removed. Please have a look if it is ok now.

Comment thread documentation/assemblies/security/assembly-internal-cluster-security.adoc Outdated
Comment thread documentation/assemblies/security/assembly-internal-cluster-security.adoc Outdated
Comment thread documentation/modules/security/proc-changing-internal-cluster-security.adoc Outdated
Comment thread documentation/modules/security/con-internal-cluster-security.adoc Outdated
Comment thread documentation/modules/security/proc-changing-internal-cluster-security.adoc Outdated
@scholzj
scholzj force-pushed the scholzj/add-none-encryption-and-authentication-docs branch from 6ec1a10 to 7014520 Compare August 13, 2026 13:48
@scholzj
scholzj force-pushed the scholzj/add-none-encryption-and-authentication-docs branch from 7eca1bf to 23eb7aa Compare August 13, 2026 15:36
@scholzj

scholzj commented Aug 15, 2026

Copy link
Copy Markdown
Member Author

@katheris @ppatierno Anything more on this? Thanks.

@scholzj
scholzj force-pushed the scholzj/add-none-encryption-and-authentication-docs branch from 23eb7aa to 633fa49 Compare August 16, 2026 09:25
Signed-off-by: Jakub Scholz <[email protected]>

@katheris katheris left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @scholzj

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants