Как можно оптимизировать этот скрипт?

Question

[hckn]

Написал первый в жизни скрипт, для первоначального деплоя приложения (приложение состоит из двух отдельных - фронт и API).

1. полная настройка VPS, установка зависимостей, конфиг для сервера и тд
2. копирование ключей от гита, клонирование репозиториев, установка зависимостей и их запуск
3. перенос БД, перенос статики (картинки)

#!/bin/bash

USER=iamuser
HOST=88.88.88.88
DOMAIN=example.com

scp keys/* $USER@$HOST:/home/$USER/.ssh

Все работает на удивление без нареканий. Но поскольку это первый опыт и я совсем не изучал баш скриптинг (от слова вообще), подозреваю есть глупые моменты, и вообще можно как-то оптимизировать все это дело. Прошу помощи и критики более опытных разработчиков.

Answer 1

[Алексей Ступеньков]

Попробуйте ansible

Comments

[hckn]

Выглядит интересно. Но я так впечатлен возможностям баш, что хочу пока его познавать))

Не будет ли ansible перебором если мои проекты не подразумевают контейнирезацию, облачную инфраструктуру и тд?

[Алексей Ступеньков]

hckn, не будет

Answer 2

[AVKor]

Код не структурирован.

Нет обработки ошибок.

Нет комментариев.

Comments

[hckn]

Спасибо за замечания, все по делу.

Answer 3

[d'Ivan]

Неплохо для первого раза.
Согласен с AVKor насчёт обработки ошибок. Это нельзя игнорировать. Процесс может неудачно выполниться, связь с удалённым сервером может быть временно недоступна и т.д.

1. Как минимум, вот этот кусок надо проработать лучше:
   

   mongodump --db super-db
   tar -cvf dump.tar dump/
   scp dump.tar $USER@$HOST:/home/$USER
   rm dump.tar && rm -rf dump/

   
   Если mongodump закончится с ненулевым статусом, то дампа не будет и тогда останешься без резервной копии.
   
   
2. Архивы TAR лучше создавать в сочетании со сжатием. Ключами -j, -J или -z
   
3. Если нет необходимости именно в TAR, то можно синхронизировать файлы на уданённый сервер при помощи утилиты rsync
   

Comments

[hckn]

Да, пока это тестовый сервер я не делал бекапа базы. Замечание по делу. И rsync тоже стоит в задачах как раз для этого, спасибо за напоминание (пока с ним не работал).

Answer 4

[kadmin88]

Вот тут согласен с Романом.
mongodump --db super-db
tar -cvf dump.tar dump/
scp dump.tar $USER@$HOST:/home/$USER
rm dump.tar && rm -rf dump/

Можно добавить условие на проверку успешного выполнения, например так:
tar -cvf dump.tar dump/
if
[ $? == 0 ]
then
scp dump.tar $USER@$HOST:/home/$USER && rm -f ./dump.tar && rm -rf ./dump/
else
echo "error while dump"
fi
и, все-таки надежнее использовать полные пути.

Если не прав, поправьте меня.

Comments

[hckn]

Спасибо за пример, попробую.

Насчет путей, мне кажется будет не совсем удобно, ведь я могу перенести рабочую папку на другую машину и придется пути вручную менять. А так скрипт запускается просто из корня рабочей директории. Мне кажется так удобнее.