Skip to content

HW21-DeployScript#742

Open
ubunvova wants to merge 2 commits into
vsilkin/mainfrom
vsilkin/hw21
Open

HW21-DeployScript#742
ubunvova wants to merge 2 commits into
vsilkin/mainfrom
vsilkin/hw21

Conversation

@ubunvova

Copy link
Copy Markdown

No description provided.

Comment thread hw-21/.github/workflows/deploy.yml Outdated
username: ${{ secrets.SSH_USER }}
port: ${{ secrets.SSH_PORT }}
key: ${{ secrets.SSH_PRIVATE_KEY }}
source: "."

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так скопируется абсолютно всё, даже то, что не должно попадать в деплойную директорию и в прод

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

добавил только app

Comment thread hw-21/.github/workflows/deploy.yml Outdated
script: |
cd /var/www/app
docker compose down || true
docker compose up -d --build

@coders-lair-dev coders-lair-dev Apr 20, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Здесь будет downtime на время сборки-старта контейнеров. Так же нет проверки "точно ли всё хорошо с контейнерами или там рестарт каждые 5 секунд".
Для реализации "без downtime" нужен хотя бы минимальный blue/green

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

сделал

branches: [ "main" ]

jobs:
deploy:

@coders-lair-dev coders-lair-dev Apr 20, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так же нет демонстрации rollback'а

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

добавил в отдельный файл

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.

2 participants