Skip to content
This repository was archived by the owner on Aug 16, 2021. It is now read-only.

Commit ee98987

Browse files
authored
Merge pull request #21 from startupturbo/ecr_login
Call `aws ecr login ...`. Remove "--clean-run-only" option
2 parents 348907f + aacbbff commit ee98987

File tree

3 files changed

+1
-27
lines changed

3 files changed

+1
-27
lines changed

nancy_run.sh

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,6 @@ while true; do
4242
--target-ddl-undo )
4343
#s3 url|filename +
4444
TARGET_DDL_UNDO="$2"; shift 2 ;;
45-
--clean-run-only )
46-
CLEAN_RUN_ONLY=1; shift 1 ;;
4745
--target-config )
4846
#s3 url|filename +
4947
TARGET_CONFIG="$2"; shift 2 ;;
@@ -82,7 +80,6 @@ then
8280
echo "workload_replay_speed: $WORKLOAD_REPLAY_SPEED"
8381
echo "target_ddl_do: $TARGET_DDL_DO"
8482
echo "target_ddl_undo: $TARGET_DDL_UNDO"
85-
echo "clean_run_only: $CLEAN_RUN_ONLY"
8683
echo "target_config: $TARGET_CONFIG"
8784
echo "artifacts_destination: $ARTIFACTS_DESTINATION"
8885
echo "aws-key-pair: $AWS_KEY_PAIR"
@@ -197,12 +194,6 @@ function checkParams() {
197194
exit 1
198195
fi
199196

200-
if (([ -v TARGET_DDL_DO ] || [ -v TARGET_CONFIG ]) && [ -v CLEAN_RUN_ONLY ])
201-
then
202-
>&2 echo "ERROR: Cannot be execute 'target run' and 'clean run' at the same time."
203-
exit 1;
204-
fi
205-
206197
if (([ ! -v TARGET_DDL_UNDO ] && [ -v TARGET_DDL_DO ]) || ([ ! -v TARGET_DDL_DO ] && [ -v TARGET_DDL_UNDO ]))
207198
then
208199
>&2 echo "ERROR: DDL code must have do and undo part."
@@ -314,6 +305,7 @@ fi
314305

315306
echo "Docker $DOCKER_MACHINE is running."
316307

308+
`aws ecr get-login --no-include-email`
317309
containerHash=$(docker `docker-machine config $DOCKER_MACHINE` run --name="pg_nancy" \
318310
-v /home/ubuntu:/machine_home -dit "950603059350.dkr.ecr.us-east-1.amazonaws.com/nancy:postgres${PG_VERSION}")
319311
dockerConfig=$(docker-machine config $DOCKER_MACHINE)

tests/nancy_run_no_options.sh

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,3 @@ if [[ $output =~ "ERROR: AWS keys not given" ]]; then
77
else
88
>&2 echo -e "\e[31mFAILED\e[39m"
99
fi
10-
11-
12-
#
13-
#if [[ $(./nancy_run.sh --aws-key-path ~/.ssh/awskey.pem --aws-key-pair awskey --pg-version 9.6 --artifacts-destination s3://postgres-misc/tmp --workload-custom-sql 'select pg_sleep(1);' --db-dump-path s3://postgres-misc/db.dumps/postila_ru.dump.sql.bz2 --clean-run-only YEEES --target-ddl-do 'select 1;' 2>&1 | grep "ERROR: 2 or more targets given." | wc -l) == 1 ]]; then
14-
# echo "OK!";
15-
#else
16-
# >&2 echo "FAILED";
17-
#fi

tests/nancy_run_options_multi_targets.sh

Lines changed: 0 additions & 10 deletions
This file was deleted.

0 commit comments

Comments
 (0)