Skip to content
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

Integrate metrics reporter main #11008

Conversation

OwenCorrigan76
Copy link
Contributor

Type of change

Select the type of your PR

  • Bugfix
  • Enhancement / new feature
  • Refactoring
  • Documentation

Description

Please describe your pull request

Checklist

Please go through this checklist and make sure all applicable tasks have been done

  • Write tests
  • Make sure all tests pass
  • Update documentation
  • Check RBAC rights for Kubernetes / OpenShift roles
  • Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
  • Reference relevant issue(s) and close them after merging
  • Update CHANGELOG.md
  • Supply screenshots for visual changes, such as Grafana dashboards

OwenCorrigan76 and others added 2 commits December 20, 2024 17:31
* add metrics reporter config and api

Signed-off-by: Owen <[email protected]>

* Address Fedes Review

Signed-off-by: Owen <[email protected]>

* Latest changes before checking out main

* Fix toString annotation and typo for test

* Fix additionalProperties

* Fix most of review comments: excluding KafkaCluster

* Refactor KafkaCluster logic

* More Refactor KafkaCluster logic

* More changes to KafkaCluster

* resolve Fede's comments

Signed-off-by: ocorriga <[email protected]>

* add KafkaBrokerConfigurationBuilder withStrimziMetricsReporter and address nits

Signed-off-by: ocorriga <[email protected]>

* remove required valueFrom in api

Signed-off-by: ocorriga <[email protected]>

* fix port and zk test

Signed-off-by: ocorriga <[email protected]>

* fix port

Signed-off-by: ocorriga <[email protected]>

* fix imports

Signed-off-by: ocorriga <[email protected]>

* fix appendix_crds

Signed-off-by: ocorriga <[email protected]>

* Add packaging and documentation

Signed-off-by: ocorriga <[email protected]>

---------

Signed-off-by: Owen <[email protected]>
Signed-off-by: ocorriga <[email protected]>
Co-authored-by: ocorriga <[email protected]>
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