mansehajsingh commented on PR #4: URL: https://github.com/apache/polaris-tools/pull/4#issuecomment-2800164798
Updates: I have added migration of principals and their assignments to principal roles to the tool. I have gated these behind a flag, `--sync-principals` so that users have to opt-in to migrate these entities. The reason for this is that the client credentials for these principals get reset on the target instance. Since these are only available at the time of creation, I've made it so that the tool logs the target credentials to stdout. eg. ``` WARN - Principal migration will reset credentials on the target Polaris instance. Principal migration will log the new target Principal credentials to stdout. ... INFO - Overwrote principal principal-2 on target. Target credentials: <client-id>:<client-secret> - 1/1 ``` I have added tests for this functionality. Improved the javadocs and made some minor changes. I have updated the design spec with these changes as well. -- 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]
