mehdid93 opened a new pull request, #645: URL: https://github.com/apache/flink-kubernetes-operator/pull/645
## What is the purpose of the change Fixing bug in Flink SQL Runner example preventing the user to provide a PEM certificate. ## Brief change log Changes in the sample code to avoid removing the BEGIN/END from PEM certificates. ## Verifying this change This change can be verified as follows (for Kafka) when having ssl enabled in Kafka. - In https://github.com/apache/flink-kubernetes-operator/blob/main/examples/flink-sql-runner-example/sql-scripts/statement-set.sql you should have the property `properties.ssl.truststore.certificates` **Example** ``` WITH ( 'connector' = 'kafka', ... 'properties.ssl.truststore.certificates' = '-----BEGIN CERTIFICATE----- ... -----END CERTIFICATE----- ' ); ``` - Deploy and check that there is no error to create the Kafka producer/consumer. ## Does this pull request potentially affect one of the following parts: - Dependencies (does it add or upgrade a dependency): (yes / no) **no** - The public API, i.e., is any changes to the `CustomResourceDescriptors`: (yes / no) **no** - Core observer or reconciler logic that is regularly executed: (yes / no) **no** ## Documentation - Does this pull request introduce a new feature? (yes / no) **no** - If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented) **not applicable but it may be worth mentioning it in the release notes of the future operator version** -- 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]
