SteNicholas commented on a change in pull request #33: URL: https://github.com/apache/flink-kubernetes-operator/pull/33#discussion_r817841360
########## File path: README.md ########## @@ -83,3 +83,62 @@ Considering the cost of running the builds, the stability, and the maintainabili All the unit tests, integration tests, and the end-to-end tests will be triggered for each PR. Note: Please make sure the CI passed before merging. + +## Operator Metrics + +The operator extends the [Flink Metric System](https://nightlies.apache.org/flink/flink-docs-master/docs/ops/metrics/) that allows gathering and exposing metrics to centralized monitoring solutions. The well known [Metric Reporters](https://nightlies.apache.org/flink/flink-docs-master/docs/deployment/metric_reporters) are shipped in the operator image and are ready to use. + +### Slf4j +The default metrics reporter in the operator is Slf4j. It does not require any external monitoring systems, and it is enabled in the operator Helm chart by default, mainly for demonstrating purposes. +```properties +metrics.reporter.slf4j.factory.class: org.apache.flink.metrics.slf4j.Slf4jReporterFactory +metrics.reporter.slf4j.interval: 1 MINUTE +``` Review comment: It's better to add the template reference link(https://github.com/apache/flink-kubernetes-operator/blob/main/helm/flink-operator/templates/flink-operator.yaml) at the end of this introduction. cc @gyfora -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
