-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HDDS-1978. Create helper script to run blockade tests. #1310
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.
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
Conversation
OZONE_VERSION=$(grep "<ozone.version>" "$DIR/../../pom.xml" | sed 's/<[^>]*>//g'| sed 's/^[ \t]*//') | ||
cd "$DIR/../../dist/target/ozone-$OZONE_VERSION/tests" || exit 1 | ||
|
||
source ../compose/ozoneblockade/.env |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shellcheck:1: note: Not following: ../compose/ozoneblockade/.env: openBinaryFile: does not exist (No such file or directory) [SC1091]
💔 -1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this. I am +1 after fixing the checkstyle issues.
cd "$DIR/../../dist/target/ozone-$OZONE_VERSION/tests" || exit 1 | ||
|
||
source ../compose/ozoneblockade/.env | ||
export HADOOP_RUNNER_VERSION |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How are these variables populated ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The variables are set by source ../compose/ozoneblockade/.env
The .env
file will contains proper values.
OZONE_VERSION=$(grep "<ozone.version>" "$DIR/../../pom.xml" | sed 's/<[^>]*>//g'| sed 's/^[ \t]*//') | ||
cd "$DIR/../../dist/target/ozone-$OZONE_VERSION/tests" || exit 1 | ||
|
||
source ../compose/ozoneblockade/.env |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shellcheck:1: note: Not following: ../compose/ozoneblockade/.env: openBinaryFile: does not exist (No such file or directory) [SC1091]
💔 -1 overall
This message was automatically generated. |
OZONE_VERSION=$(grep "<ozone.version>" "$DIR/../../pom.xml" | sed 's/<[^>]*>//g'| sed 's/^[ \t]*//') | ||
cd "$DIR/../../dist/target/ozone-$OZONE_VERSION/tests" || exit 1 | ||
|
||
source ${DIR}/../../dist/target/ozone-${OZONE_VERSION}/compose/ozoneblockade/.env |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shellcheck:1: warning: Can't follow non-constant source. Use a directive to specify location. [SC1090]
shellcheck:39: note: Double quote to prevent globbing and word splitting. [SC2086]
shellcheck:8: note: Double quote to prevent globbing and word splitting. [SC2086]
💔 -1 overall
This message was automatically generated. |
Looks good to me. Let me test it with starting the blockade test on the CI server. |
OZONE_VERSION=$(grep "<ozone.version>" "$DIR/../../pom.xml" | sed 's/<[^>]*>//g'| sed 's/^[ \t]*//') | ||
cd "$DIR/../../dist/target/ozone-$OZONE_VERSION/tests" || exit 1 | ||
|
||
source ${DIR}/../../dist/target/ozone-${OZONE_VERSION}/compose/ozoneblockade/.env |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shellcheck:1: warning: Can't follow non-constant source. Use a directive to specify location. [SC1090]
shellcheck:39: note: Double quote to prevent globbing and word splitting. [SC2086]
shellcheck:8: note: Double quote to prevent globbing and word splitting. [SC2086]
💔 -1 overall
This message was automatically generated. |
No description provided.