From fccdedc43fef589f30fcc8ac288b6306323d0f28 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E1=84=8B=E1=85=A2=E1=84=91=E1=85=B3=E1=86=AF=20=E1=84=82?= =?UTF-8?q?=E1=85=A9=E1=84=90=E1=85=B3=E1=84=87=E1=85=AE=E1=86=A8?= Date: Fri, 3 May 2019 19:09:53 +0900 Subject: [PATCH] Fix: subprocess.call argument to array --- .../chocomae/backup_unpaid_maestro_log.py | 38 +++++++++++++++---- 1 file changed, 30 insertions(+), 8 deletions(-) diff --git a/src/web/python/chocomae/backup_unpaid_maestro_log.py b/src/web/python/chocomae/backup_unpaid_maestro_log.py index 14fefe9..ad1d21d 100644 --- a/src/web/python/chocomae/backup_unpaid_maestro_log.py +++ b/src/web/python/chocomae/backup_unpaid_maestro_log.py @@ -18,6 +18,7 @@ def print_items(list): def send_unpaid_maestro_mysqldump_to_test_server(): + ''' space = ' ' command = 'mysqldump' + space user = '-u root' + space @@ -26,22 +27,43 @@ def send_unpaid_maestro_mysqldump_to_test_server(): db = 'chocomae' + space table = 'maestro' + space where_clause = '--where="DATE(NOW()) - INTERVAL 7 DAY > AcceptClausesDateTime AND ActivateStatus = 0"' + space + ''' output_directory = '/home/chocomae/unregister-user-backup/' if mode == 'mac': output_directory = '' today = datetime.today().strftime('%Y%m%d-%H%M%S') - output_filename = '> ' + output_directory + today + '.sql' + output_filename = today + '.sql' + output_fullpath = output_directory + output_filename - command_db_backup = command + user + password + transaction + db + table + where_clause + output_filename + # command_db_backup = command + user + password + transaction + db + table + where_clause + output_filename # 'mysqldump -u "${DBUSER}" -p"${DBPASS}" --single-transaction "${DBNAME}" "${filename}" # --where="DATE(${record_date})=CURDATE()" > "${filename}.dump.bydate.sql"' + + command_mysqldump = [] + command_mysqldump.append('mysql') + command_mysqldump.append('-u') + command_mysqldump.append('root') + command_mysqldump.append('-p52napark!') + command_mysqldump.append('--single-transaction') + command_mysqldump.append('chocomae') + command_mysqldump.append('maestro') + command_mysqldump.append('--where="DATE(NOW()) - INTERVAL 7 DAY > AcceptClausesDateTime AND ActivateStatus = 0"') + command_mysqldump.append('>') + command_mysqldump.append(output_fullpath) + print(command_mysqldump) + + command_scp = [] + command_scp.append('scp') + command_scp.append(output_fullpath) + backup_server_address = 'chocomae@test.chocomae.com:' + command_scp.append(backup_server_address + output_directory) + print(command_scp) + if mode == 'mac': - print(command_db_backup) - else: - subprocess.call(command_db_backup) - backup_server_address = 'chocomae@test.chocomae.com:' - command_scp = 'scp' + space + output_filename + space + backup_server_address + output_directory - subprocess.call(command_scp) + return + + subprocess.call(command_mysqldump) + subprocess.call(command_scp) # backup unpaid for 7 days maestro list