Skip to content

Commit

Permalink
Fix YW yb_release using the wrong DEVOPS_HOME
Browse files Browse the repository at this point in the history
Summary:
If DEVOPS_HOME was set, managed/yb_release would use that. This could cause it to point
to an invalid devops repo and break the release. Making sure we reset it to the local in-repo
devops. We are importing ybops pieces inside the managed/yb_release.py.

Test Plan:
call top-level
`yb_release --destination /tmp/tmpjaui3M --force --yw`

Reviewers: wesley, ram, mikhail

Reviewed By: mikhail

Subscribers: jenkins-bot, yugaware

Differential Revision: https://phabricator.dev.yugabyte.com/D7837
  • Loading branch information
bmatican committed Jan 24, 2020
1 parent 8798852 commit 2d52e1d
Showing 1 changed file with 1 addition and 10 deletions.
11 changes: 1 addition & 10 deletions managed/yb_release
Original file line number Diff line number Diff line change
Expand Up @@ -2,16 +2,7 @@
# Copyright (c) YugaByte, Inc.
set -euo pipefail

# TODO: move this to common file.
if [[ -z ${DEVOPS_HOME:-} ]]; then
devops_home_candidate_dir="${BASH_SOURCE%/*}"/devops
if [[ -d $devops_home_candidate_dir ]]; then
export DEVOPS_HOME=$( cd "$devops_home_candidate_dir" && pwd )
else
echo "DEVOPS_HOME not set and the devops repo not found at $devops_home_candidate_dir" >&2
exit 1
fi
fi
export DEVOPS_HOME="${BASH_SOURCE%/*}"/devops

. "$DEVOPS_HOME/bin/"/common.sh

Expand Down

0 comments on commit 2d52e1d

Please sign in to comment.