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

Add kafkastore.topic.skip.validation configurations option #1733

Merged
merged 3 commits into from Jan 12, 2021
Merged

Add kafkastore.topic.skip.validation configurations option #1733

merged 3 commits into from Jan 12, 2021

Conversation

CGA1123
Copy link
Contributor

@CGA1123 CGA1123 commented Jan 10, 2021

Adds the kafkastore.topic.skip.validation option (defaulting to false to
preserve current behaviour). This options controls whether
schema-registry performs validation and auto-creation of the schema
topics on boot.

Enabling this option is required in environments where clients do not
have the DescribeConfigs or where topic auto creation is not
supported.

For example, this is required in order to run the schema-registry with
Heroku's managed Apache Kafka service (See Kafka on Heroku ACLs).

Signed-off-by: Christian Gregg christian@bissy.io

@ghost
Copy link

ghost commented Jan 10, 2021

It looks like @CGA1123 hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement here.

Once you've signed reply with [clabot:check] to prove it.

Appreciation of efforts,

clabot

@CGA1123 CGA1123 changed the title Add kafkastore.topic.verify configurations options Add kafkastore.topic.verify configurations option Jan 10, 2021
@CGA1123
Copy link
Contributor Author

CGA1123 commented Jan 10, 2021

[clabot:check]

@ghost
Copy link

ghost commented Jan 10, 2021

@confluentinc It looks like @CGA1123 just signed our Contributor License Agreement. 👍

Always at your service,

clabot

Adds the `kafkastore.topic.verify` option (defaulting to `true` to
preserve current behaviour). This options controls whether
schema-registry performs verification and auto-creation of topics on
boot.

Disabling this functionality is required in environments where clients
do not have the `DescribeConfigs` or where topic auto creation is not
supported.

For example, this is required in order to run the schema-registry
with Heroku's managed Apache Kafka service (See [Kafka on Heroku ACLs]).

[Kafka on Heroku ACLs]: https://devcenter.heroku.com/articles/kafka-on-heroku#understanding-acls

Signed-off-by: Christian Gregg <christian@bissy.io>
Copy link
Member

@rayokota rayokota left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @CGA1123 ! Please rename the property to kafkastore.topic.skip.validation, with default false and importance MEDIUM

@CGA1123 CGA1123 changed the title Add kafkastore.topic.verify configurations option Add kafkastore.topic.skip.validation configurations option Jan 11, 2021
Adds the `kafkastore.topic.skip.validation` option (defaulting to `false` to
preserve current behaviour). This options controls whether
schema-registry performs validation and auto-creation of the schema
topics on boot.

Enabling this option is required in environments where clients do not
have the `DescribeConfigs` or where topic auto creation is not
supported.

For example, this is required in order to run the schema-registry with
Heroku's managed Apache Kafka service (See [Kafka on Heroku ACLs]).

[Kafka on Heroku ACLs]:
https://devcenter.heroku.com/articles/kafka-on-heroku#understanding-acls

Signed-off-by: Christian Gregg <christian@bissy.io>
@CGA1123
Copy link
Contributor Author

CGA1123 commented Jan 11, 2021

@rayokota Updated!

@rayokota
Copy link
Member

rayokota commented Jan 12, 2021

Thanks @CGA1123 ! Here are some Checkstyle issues to fix:

2021-01-11T22:43:03.368Z] [ERROR] /home/jenkins/workspace/ributors_schema-registry_PR-1733/core/src/main/java/io/confluent/kafka/schemaregistry/storage/KafkaStore.java:107: Line is longer than 100 characters (found 117). [LineLength]
[2021-01-11T22:43:03.368Z] [ERROR] /home/jenkins/workspace/ributors_schema-registry_PR-1733/core/src/main/java/io/confluent/kafka/schemaregistry/rest/SchemaRegistryConfig.java:80: Line is longer than 100 characters (found 106). [LineLength]
[2021-01-11T22:43:03.368Z] [ERROR] /home/jenkins/workspace/ributors_schema-registry_PR-1733/core/src/main/java/io/confluent/kafka/schemaregistry/rest/SchemaRegistryConfig.java:416: Line is longer than 100 characters (found 118). [LineLength]

@CGA1123
Copy link
Contributor Author

CGA1123 commented Jan 12, 2021

Thanks @rayokota -- I was having some issues accessing Jenkins yesterday, was 404ing!

Copy link
Member

@rayokota rayokota left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again for the PR @CGA1123 ! LGTM

@rayokota rayokota merged commit 40dc952 into confluentinc:master Jan 12, 2021
@CGA1123 CGA1123 deleted the very-schema-topic branch January 12, 2021 17:48
@CGA1123 CGA1123 mentioned this pull request Jan 13, 2021
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.

None yet

2 participants