Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix quoting of commit message #367

Merged
merged 3 commits into from
Aug 19, 2020
Merged

Fix quoting of commit message #367

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
fa089f2
Fix quoting of commit message in 'ws app publish chart'
kierenevans Aug 19, 2020
1638e46
Quote variables
kierenevans Aug 19, 2020
693b687
Update src/_base/harness/config/pipeline.yml
kierenevans Aug 19, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions src/_base/harness/config/pipeline.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ command('app build'):
exec: |
#!bash(workspace:/)|@
ws external-images pull

# dependency ordered build
passthru docker-compose build console
passthru docker-compose build php-fpm nginx
Expand Down Expand Up @@ -42,15 +42,15 @@ command('app publish chart <release> <message>'):

export GIT_SSH_COMMAND='ssh -i ./id_rsa -o "IdentitiesOnly yes" -F /dev/null -o StrictHostKeyChecking=no'

run git clone $REPOSITORY ./build-artifacts-repository
run git clone "$REPOSITORY" ./build-artifacts-repository
run git -C ./build-artifacts-repository config user.email "${USER_EMAIL}"

run mkdir -p $ARTIFACTS_PATH
run rsync --exclude='*.twig' --exclude='_twig' --delete -a .my127ws/helm/app/ ${ARTIFACTS_PATH}/
run rsync --exclude='*.twig' --exclude='_twig' --delete -a .my127ws/helm/app/ "${ARTIFACTS_PATH}/"

export GIT_SSH_COMMAND='ssh -i ../id_rsa -o "IdentitiesOnly yes" -F /dev/null -o StrictHostKeyChecking=no'
run git -C ./build-artifacts-repository add .
run git -C ./build-artifacts-repository commit --allow-empty -m "${MESSAGE}"
run git -C ./build-artifacts-repository commit --allow-empty -m "'${MESSAGE}'"
kierenevans marked this conversation as resolved.
Show resolved Hide resolved
run git -C ./build-artifacts-repository push origin master

command('app deploy <environment>'):
Expand All @@ -63,12 +63,12 @@ command('app deploy <environment>'):
#!bash(harness:/helm)|=
set -o pipefail
cd "${ENVIRONMENT}"
doctl -t $DO_ACCESS_TOKEN kubernetes cluster kubeconfig show $CLUSTER > kubectl.config.yaml
doctl -t "$DO_ACCESS_TOKEN" kubernetes cluster kubeconfig show "$CLUSTER" > kubectl.config.yaml
if helm version --short --client | grep '^Client: v2' >/dev/null 2>&1; then
passthru helm init --client-only
fi
passthru helm dependency build
passthru helm --kubeconfig=$PWD/kubectl.config.yaml upgrade --wait --atomic --install --timeout "${TIMEOUT}" --namespace "${NAMESPACE}" "${NAMESPACE}" ./
passthru helm --kubeconfig="${PWD}/kubectl.config.yaml" upgrade --wait --atomic --install --timeout "${TIMEOUT}" --namespace "${NAMESPACE}" "${NAMESPACE}" ./

command('helm template <chart-path>'):
env:
Expand Down