sshpass e encontrar erro no script de backup do bash

Question

Estou usando este script para fazer backup de arquivos remotos com o rsync. Há um problema quando tento iniciar recebo essa saída com erros que não sei como manipular. Obrigado por conselhos, se houver algum.

#!/bin/bash
#VARIABLES DEFINE TO STORE COMMANDS AND VALUES.
TODAY_DATE=$(date +%Y%m%d) #Store current date
MKDIR=$(which mkdir)    #store mkdir command
RSYNC=$(which rsync)    #store rsync command
DEL=$(which rm)             #store rm command
TAR=$(which tar)             #store tar command
MAIL=$(which mail)         #store mail command
LOGFILE='/tmp/backup.log' #store path of log file & touch this file in /tmp directory

#ADD EMAIL ID IN MAILTO FOR GETTING NOTIFICATIONS IN MAIL
MAILTO='[email protected]'
BODY="nnFor more information please check $LOGFILE.nnnThanks,n$0"

#CREATE DIRECTORY TO STORE BACKUP FILES
PRODBACKDIR=$TODAY_DATE/var/production-backup/

#IF CONDITION CHECK DIRECTORY IS PRESENT OR NOT AND IF NOT PRESENT CREATE DIRECTORY WITH PROVIDED DIR STRUCTURE.
if [ ! -d "{$PRODBACKDIR}" ] && [ '$MKDIR -p "$PRODBACKDIR"' ];
then
echo -e "Directory Created for site";
fi

#if [ ! -d “{$BLOGBACKDIR}” ] && [ '$MKDIR -p “$BLOGBACKDIR”' ];
#then
#echo -e “Directory Created for Production Blog Codebase”;
#fi

#REMOVE BACKUP FILES 7 DAYS OLDER
find /var/production-backup/ -mtime +7 -exec rm -rf {} ;

#RSYNC COMMAND  TO SYNC CODE OR DATA FROM REMOTE SERVER TO LOCAL USING SSHPASS TO PASS PASSWORD OF REMOTE SERVER
sshpass -p 'password' rsync –progress -rPz -e ssh backups@remotehost:/var/www/website $PRODBACKDIR

#FOR LOOP TO CHECK BACKUP FILES AND COMPRESSED IT WITH .TAR.GZ FILE WHICH SAVES DISK SPACE ON LOCAL SYSTEM AND ALSO REMOVE NON COMPRESSED FILES.
for DOC in $PRODBACKDIR; do
echo "'date '+%Y-%m-%d'' : ${DOC} backup started";
if [ -d ${DOC} ]; then
          FILENAME=$(echo $DOC | awk -F '/' '{print $NF}');
          TARFILE="$FILENAME-$TODAY_DATE.tar.gz";
          echo "'date '+%Y-%m-%d'' : ${DOC} archiving started.";
          $TAR -cvzf $PRODBACKDIR/$TARFILE $PRODBACKDIR/$FILENAME;

#IF CONDITION CHECK ERRORS WHILE CREATING TAR.GZ AND REPORT IF TAR COMMAND FAILS.
if [ $? -eq 0 ]; then
                       echo "'date '+%Y-%m-%d'' : ${DOC} archiving done."
                       $DEL -rf $PRODBACKDIR/$FILENAME
                       echo "'date '+%Y-%m-%d'' : ${DOC} deleting of non-archive directory done."
               else
                       echo "Error while creating tar."
                       echo -e "Error while creating tar file.$BODY" | $MAIL -s 'Error : Creating tar file' $MAILTO
                       exit 1
               fi
       fi
       echo "'date '+%Y-%m-%d %H:%M:%S'' : ${DOC} backup completed"
done


#SEND THE NOTIFICATION EMAIL AFTER SUCCESSFUL COMPLETION OF BACKUP.
echo -e "Backups successfully done for Production codebase" | $MAIL -s "Completed : Backup finished." $MAILTO

Erro:

find: missing argument to '-exec'
Try 'find --help' for more information.
Unexpected remote arg: backups@remotehost:/var/www/website
rsync error: syntax or usage error (code 1) at main.c(1361) [sender=3.1.2]
2018-08-02 : 20180802/var/production-backup/ backup started
2018-08-02 : 20180802/var/production-backup/ archiving started.
20180802/var/production-backup/
2018-08-02 : 20180802/var/production-backup/ archiving done.
2018-08-02 : 20180802/var/production-backup/ deleting of non-archive directory d                                                                                                                               one.
2018-08-02 06:58:59 : 20180802/var/production-backup/ backup completed

    

Answer 1

O ; no final do comando executado por find a -exec deve ser citado para protegê-lo do shell. O shell o verá como o final do comando find .

Alterar

find /var/production-backup/ -mtime +7 -exec rm -rf {} ;

para

find /var/production-backup/ -mtime +7 -exec rm -rf {} ';'

ou para

find /var/production-backup/ -mtime +7 -exec rm -rf {} \;

ou apenas

find /var/production-backup/ -mtime +7 -delete

se o seu find for compatível.

Observe também que o comando find não faz distinção entre diretórios, arquivos ou outras coisas em /var/production-backup . Se, por exemplo, o diretório /var/production-backup em si não tiver sido modificado em mais de sete dias (porque nenhum arquivo ou diretório foi criado / excluído), toda a hierarquia de arquivos será excluída.

Se você pretende excluir apenas arquivos comuns, use

find /var/production-backup/ -type f -mtime +7 -delete

Relacionados:

• Entendendo a opção -exec de 'localizar'

Outras notas sobre o seu código:

Se você fizer

TAR=$(which tar)

e depois use

$TAR ...

para criar um arquivo tar , você também pode usar tar diretamente. Não há nenhum benefício em definir TAR para o caminho do tar binário com which , pois isso seria pesquisar o PATH da mesma forma que usar tar diretamente. O mesmo vale para os outros comandos que você armazena em variáveis.

O teste [ '$MKDIR -p "$PRODBACKDIR"' ] deve ser um teste diretamente no status de retorno de mkdir . O utilitário mkdir não produz saída que você possa testar. Você também não precisa testar a existência do diretório antes de usar mkdir -p :

if mkdir -p "$PRODBACKDIR"; then
    echo 'Directory Created for site'
fi

Observe também que você está usando {$PRODBACKDIR} , que adicionará { e } ao nome do caminho do diretório. Como é feito em um teste de diretório inútil, nenhum dano foi causado (o teste sempre falhou). Você também não precisa de echo -e aqui.

Você não precisa de {} em torno dos nomes das variáveis nas expansões de variáveis. O único local em que você precisa escrever ${variable} é quando a expansão faz parte de uma sequência em que o caractere imediatamente a seguir é um caractere válido em um nome de variável, como em "${variable}x" . Você do precisa dar o dobro de todas as expansões de variáveis.

A opção --progress para rsync é escrita com dois traços, não um. Além disso, -e ssh é, até onde eu sei, o padrão para rsync desde há muito tempo.

A linha

FILENAME=$(echo $DOC | awk -F '/' '{print $NF}');

poderia ser escrito usando uma substituição de parâmetro padrão como

FILENAME=${DOC##*/}

Você raramente precisa testar contra $? . Por exemplo, em vez de

tar ...
if [ $? -eq 0 ]; then ...

faça

if tar ...; then ...

Considere também usar printf para gerar dados variáveis.

Você está definindo TODAY_DATE como uma string que contém a data de hoje, mas você a usa apenas duas vezes. Há pelo menos três outras invocações de date com um formato ligeiramente diferente ... (também, %Y-%m-%d pode ser substituído por %F , consulte man strftime ).

Você não precisa de ; no final dos comandos que não são imediatamente seguidos por outros comandos na mesma linha. Newline funciona como um terminador de comando.

Etc., ...

Também gostaria de sugerir que você analisasse o software de backup existente. Eu poderia recomendar borgbackup e restic , por exemplo.

Relacionados:

• Implicações de segurança de esquecer de citar uma variável em shells bash / POSIX
• Por que meu script de shell se sufoca em espaços em branco ou outros caracteres especiais?
• Quando é necessário fazer uma cotação dupla?
• Por que o printf é melhor que o eco?