Skip to content

Added support for Bridge internal configuration#12598

Merged
ppatierno merged 5 commits intostrimzi:mainfrom
ppatierno:add-bridge-configuration
Apr 9, 2026
Merged

Added support for Bridge internal configuration#12598
ppatierno merged 5 commits intostrimzi:mainfrom
ppatierno:add-bridge-configuration

Conversation

@ppatierno
Copy link
Copy Markdown
Member

This PR implements SIP 134 by adding support for user-provided configuration to the KafkaBridge custom resource via a the new spec.config field.
This allows users to specify additional bridge-specific properties while filtering out forbidden properties that could conflict with operator-managed settings.
Currently the only supported configuration parameters are related to the internal executor service: bridge.executor.pool.size and bridge.executor.queue.size.

Signed-off-by: Paolo Patierno <ppatierno@live.com>
@ppatierno ppatierno added this to the 1.0.0 milestone Apr 7, 2026
@ppatierno ppatierno requested review from a team and PaulRMellor April 7, 2026 09:41
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

⏳ System test verification started: link
Waiting for build to finish...

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

❌ Patch coverage is 92.85714% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 75.02%. Comparing base (263027f) to head (d684a04).
⚠️ Report is 9 commits behind head on main.

Files with missing lines Patch % Lines
...mzi/operator/cluster/model/KafkaBridgeCluster.java 75.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #12598      +/-   ##
============================================
+ Coverage     75.01%   75.02%   +0.01%     
- Complexity     6444     6450       +6     
============================================
  Files           374      375       +1     
  Lines         24924    24937      +13     
  Branches       3213     3214       +1     
============================================
+ Hits          18697    18710      +13     
- Misses         4901     4902       +1     
+ Partials       1326     1325       -1     
Files with missing lines Coverage Δ
...erator/cluster/model/KafkaBridgeConfiguration.java 100.00% <100.00%> (ø)
...cluster/model/KafkaBridgeConfigurationBuilder.java 91.12% <100.00%> (+0.27%) ⬆️
...mzi/operator/cluster/model/KafkaBridgeCluster.java 89.33% <75.00%> (-0.31%) ⬇️

... and 2 files with indirect coverage changes

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

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

System test validation failed

The provided pipeline/profile names did not match any configured tests.

Available pipelines:
acceptance, acceptance-helm, regression, regression-fg, regression-rbac, smoke, upgrade

Available profiles:
acceptance, azp_connect_mirrormaker, azp_dynconfig_listeners_tracing_watcher, azp_kafka_oauth, azp_kafka_upgrade, azp_kraft_upgrade, azp_operators, azp_rbac_remaining, azp_rolling_update_bridge, azp_security, brokers-and-security, operands, operators, smoke

Please check your command for typos and try again.

@ppatierno
Copy link
Copy Markdown
Member Author

/gha run profile=azp_bridge

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

⏳ System test verification started: link
Waiting for build to finish...

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

System test validation failed

The provided pipeline/profile names did not match any configured tests.

Available pipelines:
acceptance, acceptance-helm, regression, regression-fg, regression-rbac, smoke, upgrade

Available profiles:
acceptance, azp_connect_mirrormaker, azp_dynconfig_listeners_tracing_watcher, azp_kafka_oauth, azp_kafka_upgrade, azp_kraft_upgrade, azp_operators, azp_rbac_remaining, azp_rolling_update_bridge, azp_security, brokers-and-security, operands, operators, smoke

Please check your command for typos and try again.

@ppatierno
Copy link
Copy Markdown
Member Author

/gha run groups=bridge

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

⏳ System test verification started: link

The following 1 job(s) will be executed:

  • custom-all-amd64 (ubuntu-latest)

Tests will start after successful build completion.

@scholzj scholzj self-requested a review April 7, 2026 12:56
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

❌ System test verification failed: link

@ppatierno
Copy link
Copy Markdown
Member Author

@im-konge AFAIU, you mentioned that the failing OLM tests are not related to this PR but because of kind setup.

Signed-off-by: Paolo Patierno <ppatierno@live.com>
Signed-off-by: Paolo Patierno <ppatierno@live.com>
Signed-off-by: Paolo Patierno <ppatierno@live.com>
Copy link
Copy Markdown
Member

@scholzj scholzj left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

Copy link
Copy Markdown
Contributor

@PaulRMellor PaulRMellor left a comment

Choose a reason for hiding this comment

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

Thanks. I left a few suggestions.

Signed-off-by: Paolo Patierno <ppatierno@live.com>
@ppatierno
Copy link
Copy Markdown
Member Author

/gha run pipeline=regression

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

⏳ System test verification started: link

The following 6 job(s) will be executed:

  • regression-brokers-and-security-amd64 (oracle-vm-8cpu-32gb-x86-64)
  • regression-operators-amd64 (oracle-vm-8cpu-32gb-x86-64)
  • regression-operands-amd64 (oracle-vm-8cpu-32gb-x86-64)
  • regression-brokers-and-security-arm64 (oracle-vm-8cpu-32gb-arm64)
  • regression-operators-arm64 (oracle-vm-8cpu-32gb-arm64)
  • regression-operands-arm64 (oracle-vm-8cpu-32gb-arm64)

Tests will start after successful build completion.

@ppatierno
Copy link
Copy Markdown
Member Author

@im-konge do you have anything else to add to this PR?

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🎉 System test verification passed: link

@ppatierno ppatierno merged commit 04fc3db into strimzi:main Apr 9, 2026
32 checks passed
@ppatierno ppatierno deleted the add-bridge-configuration branch April 9, 2026 04:00
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.

4 participants