Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Addition of a dry run flag #107

Open
ThisIsQasim opened this issue Dec 12, 2022 · 1 comment
Open

Addition of a dry run flag #107

ThisIsQasim opened this issue Dec 12, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@ThisIsQasim
Copy link

@ChrisPates glad to see this project finally getting some much needed attention

Is your feature request related to a problem? Please describe.
This tool has been broken for quite a while and some of us have been running it with our own set of patches. However, now that there is a renewed focus on this, I expect people would want to get back to using the upstream version.

Describe the solution you'd like
In order to minimise the risk when switching sync tool or after version updates a --dry-run option would be very helpful.

@ChrisPates ChrisPates added the enhancement New feature or request label Jan 20, 2023
@ChrisPates
Copy link
Contributor

This is an excellent feature, its going to take a little house keeping on the code. Currently transactions are executed against the AWS api at various points.

For a clean implementation It would be better to determine all the changes should be made and with the dry run set report them to the logs/console without making them.

jjheinon added a commit to jjheinon/ssosync that referenced this issue Sep 25, 2024
  -n, --dry-run                     Print the commands that would be executed, but do not execute them.

Removed mixed usage of tabs/spaces to fix indentation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants