bernardodemarco commented on PR #10790: URL: https://github.com/apache/cloudstack/pull/10790#issuecomment-3480622524
> but we still have the same issue; the use of configuration keys is so ubiquitous that we might run into issues somewhere else in the system. @DaanHoogland, the current PR is only proposing to change the `updateConfiguration` API response when the `value` parameter is not specified (`update configuration name="alert.email.addresses"`, for instance). Currently, it returns a success message, with the configuration attributes. Instead of that, it is proposed to return a self-explanatory error message for the operators (see APIs calls examples in https://github.com/apache/cloudstack/pull/10790#issuecomment-3473608551). Thus, I do not see how we might run into issues somewhere else -- 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]
