Closed grdryn closed 2 years ago
I think eventually we'll need to put these configs into the Strimzi bundle's configmap so they only apply to the specific bundle versions.
@MikeEdgar I'm not sure about this part. The intention of the duplication at the moment is to allow for this to be released independent of the version of the plugin in the broker. Then at some point in the future when there are no instances of the older plugin version remaining, the old config can be removed.
I'm probably misunderstanding what you're saying...could you point me in the direction of the configmap that you're referring to? That might make it clearer to me :slightly_smiling_face:
In the broker plugins project, we're updating the config properties so that they don't have the strimzi prefix.
This change updates the config set by this operator so that both the old and new properties are set. Once the version of the operator using the new properties is rolled out, we can then remove the old properties from being set here again in another change.
Given that this is intended to be a temporary duplication, I tried to keep it as simple as possible, so that it will be simple to remove when we get to that stage also.