Skip to content

Remove Vert.x from Cluster Operator Kubernetes Resource Operators#12849

Draft
scholzj wants to merge 3 commits into
strimzi:mainfrom
scholzj:remove-vertx-from-CO-Kubernetes-Resource-Operators
Draft

Remove Vert.x from Cluster Operator Kubernetes Resource Operators#12849
scholzj wants to merge 3 commits into
strimzi:mainfrom
scholzj:remove-vertx-from-CO-Kubernetes-Resource-Operators

Conversation

@scholzj

@scholzj scholzj commented Jun 19, 2026

Copy link
Copy Markdown
Member

Type of change

  • Refactoring

Description

This PR refactors the Kubernetes Resource Operators in the Cluster Operator and removes Vert.x from them.

Checklist

  • Write tests
  • Make sure all tests pass
  • Try your changes inside a Kubernetes cluster, not just from unit tests
  • AI assistance was used to create this PR (see the Strimzi AI policy)

Signed-off-by: Jakub Scholz <www@scholzj.com>
@snyk-io

snyk-io Bot commented Jun 19, 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.

@scholzj

scholzj commented Jun 19, 2026

Copy link
Copy Markdown
Member Author

/gha run pipeline=upgrade,regression,regression-fg

@scholzj scholzj added this to the 1.2.0 milestone Jun 19, 2026
@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown

⏳ System test verification started: link

The following 16 job(s) will be executed:

  • regression-brokers-and-security-amd64 (cncf-ubuntu-8-32-x86)
  • regression-operators-amd64 (cncf-ubuntu-8-32-x86)
  • regression-operands-amd64 (cncf-ubuntu-8-32-x86)
  • regression-brokers-and-security-arm64 (cncf-ubuntu-8-32-arm)
  • regression-operators-arm64 (cncf-ubuntu-8-32-arm)
  • regression-operands-arm64 (cncf-ubuntu-8-32-arm)
  • upgrade-azp_kraft_upgrade-amd64 (cncf-ubuntu-4-16-x86)
  • upgrade-azp_kafka_upgrade-amd64 (cncf-ubuntu-4-16-x86)
  • upgrade-azp_kraft_upgrade-arm64 (cncf-ubuntu-4-16-arm)
  • upgrade-azp_kafka_upgrade-arm64 (cncf-ubuntu-4-16-arm)
  • regression-fg-brokers-and-security-amd64 (cncf-ubuntu-8-32-x86)
  • regression-fg-operators-amd64 (cncf-ubuntu-8-32-x86)
  • regression-fg-operands-amd64 (cncf-ubuntu-8-32-x86)
  • regression-fg-brokers-and-security-arm64 (cncf-ubuntu-8-32-arm)
  • regression-fg-operators-arm64 (cncf-ubuntu-8-32-arm)
  • regression-fg-operands-arm64 (cncf-ubuntu-8-32-arm)

Tests will start after successful build completion.

@codecov

codecov Bot commented Jun 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.58025% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.75%. Comparing base (ac788d4) to head (a38ec5c).
⚠️ Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
...er/operator/resource/ResourceOperatorSupplier.java 69.23% 0 Missing and 4 partials ⚠️
...ster/operator/resource/kubernetes/CrdOperator.java 89.18% 3 Missing and 1 partial ⚠️
...operator/assembly/KafkaBridgeAssemblyOperator.java 83.33% 2 Missing and 1 partial ⚠️
...ster/operator/resource/kubernetes/PodOperator.java 76.92% 0 Missing and 3 partials ⚠️
...luster/operator/assembly/ConnectBuildOperator.java 90.90% 0 Missing and 2 partials ⚠️
...er/operator/assembly/KafkaListenersReconciler.java 90.47% 2 Missing ⚠️
...o/strimzi/operator/cluster/operator/VertxUtil.java 87.50% 1 Missing ⚠️
...erator/cluster/operator/assembly/CaReconciler.java 93.33% 1 Missing ⚠️
...perator/assembly/KafkaConnectAssemblyOperator.java 92.30% 1 Missing ⚠️
...or/assembly/KafkaMirrorMaker2AssemblyOperator.java 87.50% 1 Missing ⚠️
... and 4 more
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #12849      +/-   ##
============================================
- Coverage     79.31%   78.75%   -0.57%     
+ Complexity     6377     6286      -91     
============================================
  Files           335      330       -5     
  Lines         22712    22561     -151     
  Branches       3070     3057      -13     
============================================
- Hits          18015    17768     -247     
- Misses         3482     3593     +111     
+ Partials       1215     1200      -15     
Files with missing lines Coverage Δ
...a/io/strimzi/operator/cluster/ClusterOperator.java 76.81% <ø> (-0.34%) ⬇️
...rc/main/java/io/strimzi/operator/cluster/Main.java 27.11% <100.00%> (+1.25%) ⬆️
...er/operator/assembly/AbstractAssemblyOperator.java 100.00% <ø> (ø)
...ter/operator/assembly/AbstractConnectOperator.java 87.76% <100.00%> (ø)
...or/cluster/operator/assembly/AbstractOperator.java 79.46% <100.00%> (-0.90%) ⬇️
...ter/operator/assembly/CruiseControlReconciler.java 96.26% <100.00%> (ø)
...er/operator/assembly/EntityOperatorReconciler.java 89.71% <100.00%> (ø)
...r/operator/assembly/KRaftVersionChangeCreator.java 97.61% <100.00%> (ø)
...uster/operator/assembly/KafkaAssemblyOperator.java 84.26% <100.00%> (ø)
...rator/assembly/KafkaAutoRebalancingReconciler.java 81.49% <100.00%> (ø)
... and 48 more

... 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

❌ System test verification failed: link

Signed-off-by: Jakub Scholz <www@scholzj.com>
@scholzj

scholzj commented Jun 19, 2026

Copy link
Copy Markdown
Member Author

/gha run pipeline=upgrade,regression,regression-fg

@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown

⏳ System test verification started: link

The following 16 job(s) will be executed:

  • regression-brokers-and-security-amd64 (cncf-ubuntu-8-32-x86)
  • regression-operators-amd64 (cncf-ubuntu-8-32-x86)
  • regression-operands-amd64 (cncf-ubuntu-8-32-x86)
  • regression-brokers-and-security-arm64 (cncf-ubuntu-8-32-arm)
  • regression-operators-arm64 (cncf-ubuntu-8-32-arm)
  • regression-operands-arm64 (cncf-ubuntu-8-32-arm)
  • upgrade-azp_kraft_upgrade-amd64 (cncf-ubuntu-4-16-x86)
  • upgrade-azp_kafka_upgrade-amd64 (cncf-ubuntu-4-16-x86)
  • upgrade-azp_kraft_upgrade-arm64 (cncf-ubuntu-4-16-arm)
  • upgrade-azp_kafka_upgrade-arm64 (cncf-ubuntu-4-16-arm)
  • regression-fg-brokers-and-security-amd64 (cncf-ubuntu-8-32-x86)
  • regression-fg-operators-amd64 (cncf-ubuntu-8-32-x86)
  • regression-fg-operands-amd64 (cncf-ubuntu-8-32-x86)
  • regression-fg-brokers-and-security-arm64 (cncf-ubuntu-8-32-arm)
  • regression-fg-operators-arm64 (cncf-ubuntu-8-32-arm)
  • regression-fg-operands-arm64 (cncf-ubuntu-8-32-arm)

Tests will start after successful build completion.

@github-actions

Copy link
Copy Markdown

❌ System test verification failed: link

Signed-off-by: Jakub Scholz <www@scholzj.com>
@scholzj

scholzj commented Jun 19, 2026

Copy link
Copy Markdown
Member Author

/gha run pipeline=regression

@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown

⏳ System test verification started: link

The following 6 job(s) will be executed:

  • regression-brokers-and-security-amd64 (cncf-ubuntu-8-32-x86)
  • regression-operators-amd64 (cncf-ubuntu-8-32-x86)
  • regression-operands-amd64 (cncf-ubuntu-8-32-x86)
  • regression-brokers-and-security-arm64 (cncf-ubuntu-8-32-arm)
  • regression-operators-arm64 (cncf-ubuntu-8-32-arm)
  • regression-operands-arm64 (cncf-ubuntu-8-32-arm)

Tests will start after successful build completion.

@scholzj

scholzj commented Jun 19, 2026

Copy link
Copy Markdown
Member Author

/azp run regression

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@github-actions

Copy link
Copy Markdown

❌ System test verification failed: link

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.

1 participant