ricardozanini commented on PR #367: URL: https://github.com/apache/incubator-kie-kogito-serverless-operator/pull/367#issuecomment-1924378712
@wmedvede I'd rather keep all managed properties in the CM. One place to look for the managed properties by the operator can facilitate admin tasks. Regarding the bug you've found, this is intentional to override the ones by the user. Otherwise, Quarkus will try to read them and parse. Since `kubernetes:` is not a valid schema, it will fail. Actually, by chance, @dmartinol found a way to remove our `microprofile-config` add-on. We don't need it anymore since it's already doing the expansion on the operator side. So in your example, only these will do: > application-dev.properties (Managed) ```properties user.property3=<discovered URL> user.property4=<discovered URL> ``` > application.properties (User) ```properties user.property3=<desired Kubernetes endpoint> user.property4=<desired Kubernetes endpoint> ``` We can ditch the jar and live happily. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
