You've already forked pedini.dev
feat(deploy): improve environment variable handling and deployment workflow
Some checks failed
Deploy website on production server when committing on main / test (push) Failing after 12s
Some checks failed
Deploy website on production server when committing on main / test (push) Failing after 12s
- Update .vars to properly export deployment configuration variables - Modify deploy.sh to conditionally source .vars only in interactive mode - Remove include directive from makefile to prevent automatic variable loading - Enhance deployment script reliability by ensuring proper environment setup
This commit is contained in:
4
.vars
4
.vars
@@ -1,4 +1,4 @@
|
|||||||
#/usr/bin/env bash
|
#/usr/bin/env bash
|
||||||
|
|
||||||
"${DEPLOYMENT_HOST:=docker.infra.bjphoster.cloud}"
|
export DEPLOYMENT_HOST=docker.infra.bjphoster.cloud
|
||||||
"${DEPLOYMENT_PATH:=/opt/pedini.dev}"
|
export DEPLOYMENT_PATH=/opt/pedini.dev
|
||||||
|
|||||||
@@ -12,6 +12,10 @@ set -e
|
|||||||
# then remove everything in the data path, untar the tarball and reload the server
|
# then remove everything in the data path, untar the tarball and reload the server
|
||||||
# finally remove the tarball, both from the remote host and locally (cleanup)
|
# finally remove the tarball, both from the remote host and locally (cleanup)
|
||||||
|
|
||||||
|
if [ -t 0 ]; then # Interactive: prompt user
|
||||||
|
source .vars
|
||||||
|
fi
|
||||||
|
|
||||||
# Check if the username variable is set
|
# Check if the username variable is set
|
||||||
if [ ! -z "${SSH_USERNAME}" ]; then
|
if [ ! -z "${SSH_USERNAME}" ]; then
|
||||||
SSH_USERNAME="${SSH_USERNAME}@"
|
SSH_USERNAME="${SSH_USERNAME}@"
|
||||||
|
|||||||
Reference in New Issue
Block a user