rhtyd opened a new pull request #2241: CLOUDSTACK-10052: Simplify dynamic roles enable checking URL: https://github.com/apache/cloudstack/pull/2241 This fixes issue of enabling dynamic roles based on the global setting only. This also fixes application of the default role/permissions mapping on upgrade from 4.8 and previous versions to 4.9+. Previously, it would make additional check to ensure commands.properties is not in the classpath however this creates confusion for admins who may skip/skim through the rn/docs and assume that mere changing the global settings was not enough. Pinging for review @PaulAngus @nvazquez @borisstoyanov @wido @pdion891 and others @blueorangutan package ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
With regards, Apache Git Services
