debiswal commented on PR #672: URL: https://github.com/apache/knox/pull/672#issuecomment-1318650215
Hi @smolnar82 Thank you for review would like to inform SSL_ENABLED is defined at the role level in CM, so it should be defined on the role level in the model generator as well & earlier it was set in ServiceProperty which is not correct & now it is set to RoleProperty .we have changed where we get the property but we have not changed it where we store the property. The current patch is tested as well in knox cluster. -- 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]
