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

Escape initial - when passing postgresql_backup_rsync_backup_opts #52

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

nsoranzo
Copy link
Member

@nsoranzo nsoranzo commented Dec 1, 2023

Fix #44 .

The above issue only arises in Python's argparse when postgresql_backup_rsync_backup_opts is "one word", which is why UseGalaxy.org doesn't see this ( https://github.com/galaxyproject/usegalaxy-playbook/blob/main/env/main/group_vars/dbservers/vars.yml#L8C1-L8C36 ).

…ts``

Fix #44

The above issue only arises in Python's argparse when `postgresql_backup_rsync_backup_opts` is "one word", which is why UseGalaxy.org doesn't see this ( https://github.com/galaxyproject/usegalaxy-playbook/blob/main/env/main/group_vars/dbservers/vars.yml#L8C1-L8C36 ).
@hexylena hexylena merged commit b1044d3 into main Dec 4, 2023
1 check passed
@hexylena
Copy link
Member

hexylena commented Dec 4, 2023

Thanks @nsoranzo, what a bizarre issue

@nsoranzo nsoranzo deleted the nsoranzo-patch-1 branch December 4, 2023 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Backup options have issues with -
2 participants