Skip to content

Commit

Permalink
Issue 3104: Clean up gradle properties (pravega#3530)
Browse files Browse the repository at this point in the history
* Making the evaluation of tasks conditional to setting parameters.

Signed-off-by: Flavio Junqueira (fpj) <[email protected]>
  • Loading branch information
fpj authored and shrids committed Mar 29, 2019
1 parent 911df70 commit 5277ddf
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 27 deletions.
8 changes: 5 additions & 3 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -829,7 +829,7 @@ project('test:system') {
if(ext.getProperty('awsExecution').toString().equals("true")) {
commandLine './aws/preTestScript.sh'
args "$aws_access_key", "$aws_secret_key", "$aws_region", "$aws_key_name", "$cred_path", "$config_path", "$pravega_org", "$pravega_branch"
} else {
} else if (project.hasProperty('CLUSTER_NAME')) {
commandLine "./preTestScript.sh"
args "$CLUSTER_NAME", "$MASTER", "$NUM_SLAVES"
}
Expand Down Expand Up @@ -904,8 +904,10 @@ project('test:system') {
}

task collectSystemTestLogsFromAws(type: Exec) {
commandLine './aws/postTestScript.sh'
args "$aws_access_key", "$aws_secret_key", "$aws_region", "$aws_key_name", "$cred_path", "$config_path", "$pravega_org", "$pravega_branch", "$travis_commit"
if (project.hasProperty('aws_secret_key')) {
commandLine './aws/postTestScript.sh'
args "$aws_access_key", "$aws_secret_key", "$aws_region", "$aws_key_name", "$cred_path", "$config_path", "$pravega_org", "$pravega_branch", "$travis_commit"
}
}
}

Expand Down
25 changes: 1 addition & 24 deletions gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -58,29 +58,6 @@ pravegaBaseTag=pravega/pravega
bookkeeperBaseTag=pravega/bookkeeper

# Pravega Signing Key
signing.keyId=05949AF6
signing.keyId=
# This will be defaulted to ~/.gnupg/secring.gpg if not provided as a command line property
signing.secretKeyRingFile=

#Default values for cluster name, master IP and number of slaves in the cluster.
#To override this use -PCLUSTER_NAME=<> -PMASTER=<> -PNUM_SLAVES=<> while running gradle task
CLUSTER_NAME=ant-man
MASTER=127.0.0.1
NUM_SLAVES=3

#Default host,user path values for pravega.io server
#To override this use -Phostname=<> -PMASTER=<> -Pusername=<> while running gradle task
hostname=0.0.0.0
username=root

#Default config values for system test execution on AWS
#To override this use -Paws_access_key=<> and so on... while running gradle task `startSystemTestsWithDocker`
aws_access_key=null
aws_secret_key=null
aws_region=us-east-2
aws_key_name=aws-key-pair
cred_path=/home/ubuntu
config_path=/home/ubuntu
pravega_org=pravega
pravega_branch=master
travis_commit=0

0 comments on commit 5277ddf

Please sign in to comment.