search

Stouts / Stouts.backup

Ansible role to backup files, databases
MIT License
150 stars 40 forks source link

quick fix for default backup_postgres_port #11

Closed viatoriche closed 9 years ago

viatoriche commented 9 years ago

quick fix for default backup_postgres_port

TASK: [Stouts.backup | backup-configure | Configure pre scripts] ** fatal: [default] => {'msg': "TypeError: cannot concatenate 'str' and 'int' objects", 'failed': True} fatal: [default] => {'msg': 'One or more items failed.', 'failed': True, 'changed': False, 'results': [{'msg': "TypeError: cannot concatenate 'str' and 'int' objects", 'failed': True}]}

FATAL: all hosts have already failed -- aborting

klen commented 9 years ago

I see the problem, but disagree with the PR. It could confuse users. I we have the backup_postgres_port option we should support integer values.

klen commented 9 years ago

Quick fix: https://github.com/Stouts/Stouts.backup/commit/b403fa0682ca1d584ad48143e111300549ea444c

viatoriche commented 9 years ago

sorry for this bug :c