Skip to content

Conversation

nandakumar131
Copy link
Contributor

No description provided.

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

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]

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 43 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
0 shelldocs 0 Shelldocs was not available.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 1 new or modified test files.
_ trunk Compile Tests _
0 mvndep 33 Maven dependency ordering for branch
+1 mvninstall 590 trunk passed
+1 mvnsite 0 trunk passed
-1 pylint 2 Error running pylint. Please check pylint stderr files.
+1 shadedclient 801 branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
0 mvndep 25 Maven dependency ordering for patch
+1 mvninstall 553 the patch passed
+1 mvnsite 0 the patch passed
-1 pylint 3 Error running pylint. Please check pylint stderr files.
+1 pylint 3 There were no new pylint issues.
-1 shellcheck 0 The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 701 patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 unit 99 hadoop-hdds in the patch passed.
+1 unit 279 hadoop-ozone in the patch passed.
+1 asflicense 44 The patch does not generate ASF License warnings.
3366
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/1/artifact/out/Dockerfile
GITHUB PR #1310
Optional Tests dupname asflicense mvnsite unit shellcheck shelldocs pylint
uname Linux 2b84cf533e11 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / d873ddd
pylint https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/1/artifact/out/branch-pylint-stderr.txt
pylint https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/1/artifact/out/patch-pylint-stderr.txt
shellcheck https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/1/artifact/out/diff-patch-shellcheck.txt
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/1/testReport/
Max. process+thread count 415 (vs. ulimit of 5500)
modules C: hadoop-ozone hadoop-ozone/fault-injection-test/network-tests U: hadoop-ozone
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/1/console
versions git=2.7.4 maven=3.3.9 shellcheck=0.4.6 pylint=1.9.2
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

Copy link
Contributor

@mukul1987 mukul1987 left a 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
Copy link
Contributor

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 ?

Copy link
Contributor Author

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

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]

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 98 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
0 shelldocs 0 Shelldocs was not available.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 1 new or modified test files.
_ trunk Compile Tests _
0 mvndep 29 Maven dependency ordering for branch
+1 mvninstall 646 trunk passed
+1 mvnsite 0 trunk passed
-1 pylint 1 Error running pylint. Please check pylint stderr files.
+1 shadedclient 874 branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
0 mvndep 26 Maven dependency ordering for patch
+1 mvninstall 595 the patch passed
+1 mvnsite 0 the patch passed
-1 pylint 1 Error running pylint. Please check pylint stderr files.
+1 pylint 2 There were no new pylint issues.
-1 shellcheck 0 The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 796 patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 unit 114 hadoop-hdds in the patch passed.
+1 unit 295 hadoop-ozone in the patch passed.
+1 asflicense 41 The patch does not generate ASF License warnings.
3711
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/2/artifact/out/Dockerfile
GITHUB PR #1310
Optional Tests dupname asflicense mvnsite unit shellcheck shelldocs pylint
uname Linux 68d68c560062 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 6244502
pylint https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/2/artifact/out/branch-pylint-stderr.txt
pylint https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/2/artifact/out/patch-pylint-stderr.txt
shellcheck https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/2/artifact/out/diff-patch-shellcheck.txt
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/2/testReport/
Max. process+thread count 306 (vs. ulimit of 5500)
modules C: hadoop-ozone hadoop-ozone/fault-injection-test/network-tests U: hadoop-ozone
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/2/console
versions git=2.7.4 maven=3.3.9 shellcheck=0.4.6 pylint=1.9.2
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

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

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]

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 158 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
0 shelldocs 1 Shelldocs was not available.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 1 new or modified test files.
_ trunk Compile Tests _
0 mvndep 36 Maven dependency ordering for branch
+1 mvninstall 811 trunk passed
+1 mvnsite 0 trunk passed
-1 pylint 5 Error running pylint. Please check pylint stderr files.
+1 shadedclient 1069 branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
0 mvndep 34 Maven dependency ordering for patch
+1 mvninstall 717 the patch passed
+1 mvnsite 0 the patch passed
-1 pylint 6 Error running pylint. Please check pylint stderr files.
+1 pylint 7 There were no new pylint issues.
-1 shellcheck 1 The patch generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 882 patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 unit 123 hadoop-hdds in the patch passed.
+1 unit 359 hadoop-ozone in the patch passed.
+1 asflicense 57 The patch does not generate ASF License warnings.
4511
Subsystem Report/Notes
Docker Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/3/artifact/out/Dockerfile
GITHUB PR #1310
Optional Tests dupname asflicense mvnsite unit shellcheck shelldocs pylint
uname Linux cfbaf6f553dc 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / e684b17
pylint https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/3/artifact/out/branch-pylint-stderr.txt
pylint https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/3/artifact/out/patch-pylint-stderr.txt
shellcheck https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/3/artifact/out/diff-patch-shellcheck.txt
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/3/testReport/
Max. process+thread count 306 (vs. ulimit of 5500)
modules C: hadoop-ozone hadoop-ozone/fault-injection-test/network-tests U: hadoop-ozone
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/3/console
versions git=2.7.4 maven=3.3.9 shellcheck=0.4.6 pylint=1.9.2
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@elek
Copy link
Member

elek commented Aug 21, 2019

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

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]

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 82 Docker mode activated.
_ Prechecks _
+1 dupname 0 No case conflicting files found.
0 shelldocs 1 Shelldocs was not available.
+1 @author 0 The patch does not contain any @author tags.
+1 test4tests 0 The patch appears to include 1 new or modified test files.
_ trunk Compile Tests _
0 mvndep 16 Maven dependency ordering for branch
+1 mvninstall 662 trunk passed
+1 mvnsite 0 trunk passed
-1 pylint 1 Error running pylint. Please check pylint stderr files.
+1 shadedclient 888 branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
0 mvndep 28 Maven dependency ordering for patch
+1 mvninstall 594 the patch passed
+1 mvnsite 0 the patch passed
-1 pylint 1 Error running pylint. Please check pylint stderr files.
+1 pylint 2 There were no new pylint issues.
-1 shellcheck 0 The patch generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 767 patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 unit 124 hadoop-hdds in the patch passed.
+1 unit 317 hadoop-ozone in the patch passed.
+1 asflicense 39 The patch does not generate ASF License warnings.
3715
Subsystem Report/Notes
Docker Client=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/4/artifact/out/Dockerfile
GITHUB PR #1310
Optional Tests dupname asflicense mvnsite unit shellcheck shelldocs pylint
uname Linux 13f1213cfe28 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / ee7c261
pylint https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/4/artifact/out/branch-pylint-stderr.txt
pylint https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/4/artifact/out/patch-pylint-stderr.txt
shellcheck https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/4/artifact/out/diff-patch-shellcheck.txt
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/4/testReport/
Max. process+thread count 305 (vs. ulimit of 5500)
modules C: hadoop-ozone hadoop-ozone/fault-injection-test/network-tests U: hadoop-ozone
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-1310/4/console
versions git=2.7.4 maven=3.3.9 shellcheck=0.4.6 pylint=1.9.2
Powered by Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@nandakumar131 nandakumar131 merged commit 20064b6 into apache:trunk Aug 23, 2019
@nandakumar131 nandakumar131 deleted the HDDS-1978 branch August 23, 2019 16:56
amahussein pushed a commit to amahussein/hadoop that referenced this pull request Oct 29, 2019
RogPodge pushed a commit to RogPodge/hadoop that referenced this pull request Mar 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants