feat/Broker status - version/image extraction improvements

[azun]

Description

Performance improvements: parallel scrape for broker version and image.

Type of Change

• Bug Fix
• New Feature
• Breaking Change
• Refactor
• Documentation
• Other (please describe)

Checklist

• I have read the contributing guidelines
• Existing issues have been referenced (where applicable)
• I have verified this change is not present in other open pull requests
• Functionality is documented
• All code style checks pass
• New code contribution is covered by automated tests
• All new and existing tests pass

[dobrerazvan]

How did you find that slow function (updateStatusWithDockerImageAndVersion)?

[azun]

How did you find that slow function (updateStatusWithDockerImageAndVersion)?

operator was logging status updates for each broker every 5-15s

[dobrerazvan]

Suggested change

	- interval: {{ .Values.prometheusMetrics.interval }}
	- interval: {{ .Values.prometheusMetrics.podMonitor.interval }}

[dobrerazvan]

Suggested change

	app: kafka-operator
	component: operator
	app.kubernetes.io/name: {{ include "kafka-operator.name" . }}
	app.kubernetes.io/instance: {{ .Release.Name }}
	app.kubernetes.io/component: operator

matchLabels in the spec:selector:matchLabels: is immutable and we already have the necessary labels to identify the operator pod.

[dobrerazvan]

Suggested change

	app: kafka-operator
	component: operator

[dobrerazvan]

Suggested change

	app: kafka-operator
	component: operator

[dobrerazvan]

Goroutines have no cancellation path. If the controller shuts down mid-reconcile, these goroutines run to completion (or until JMX times out). Worth passing a context.Context from the reconcile call if JMX extraction can be long-running.

[dobrerazvan]

Drain the hole channel first then update the broker status so all or none are updated.