KafkaConnect - Restart Connector KIP-745

[rlanhellas]

Related to #11484

Kafka KIP-745 adds two optional query parameters to restart a connector:

includeTasks and onlyFailed . Today in our current annotation we have the following options:

strimzi.io/restart
strimzi.io/restart-task

The strimzi.io/restart calls exactly the API mentioned on KIP-745, but always set false to the new optional arguments, as you can see here.

If you take a look on other very famous products like nginx, they allow most of configs using annotations, check this out. Then my proposal is to do the same here, and we should have the following:

strimzi.io/restart
strimzi.io/restart-include-tasks
strimzi.io/restart-only-failed
strimzi.io/restart-task

The two new annotations strimzi.io/restart-include-tasks and strimzi.io/restart-only-failed will have default to false and will be ignored if strimzi.io/restart is not set.

[ppatierno]

@rlanhellas thanks for the proposal but you should use the Strimzi proposals repository in order to do so and not the Github discussions.
If you go here https://github.com/strimzi/proposals you can open a PR with your proposal. There is a template to start with or anyway you can use any other proposal as an example about how to structure it.

[rlanhellas]

Let me move this . Please close this discussion

[katheris]

Closing since the proposal can now be found here: strimzi/proposals#163