Skip to content

HBASE-25591 Upgrade opentelemetry to 0.17.1 - #2971

Merged
Apache9 merged 1 commit into
apache:HBASE-22120from
Apache9:HBASE-25591
Feb 26, 2021
Merged

HBASE-25591 Upgrade opentelemetry to 0.17.1#2971
Apache9 merged 1 commit into
apache:HBASE-22120from
Apache9:HBASE-25591

Conversation

@Apache9

Copy link
Copy Markdown
Contributor

No description provided.

@Apache9Apache9 self-assigned this Feb 21, 2021
@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 30sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 32sMaven dependency ordering for branch
+1 💚mvninstall3m 46sHBASE-22120 passed
+1 💚compile8m 9sHBASE-22120 passed
+1 💚checkstyle1m 54sHBASE-22120 passed
-1 ❌spotbugs8m 57sroot in HBASE-22120 has 1 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 23sMaven dependency ordering for patch
+1 💚mvninstall3m 32sthe patch passed
+1 💚compile8m 6sthe patch passed
+1 💚javac8m 6sthe patch passed
+1 💚checkstyle1m 57sroot: The patch generated 0 new + 25 unchanged - 1 fixed = 25 total (was 26)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 3sThe patch has no ill-formed XML file.
+1 💚hadoopcheck17m 25sPatch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.
+1 💚spotbugs0m 51shbase-common in the patch passed.
+1 💚spotbugs1m 7shbase-client in the patch passed.
+1 💚spotbugs2m 9shbase-server in the patch passed.
+1 💚spotbugs0m 41shbase-it in the patch passed.
+1 💚spotbugs9m 6sroot generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
_ Other Tests _
+1 💚asflicense1m 6sThe patch does not generate ASF License warnings.
82m 48s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#2971
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile xml
unameLinux 853e11a60fc8 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / b765a0b
Default JavaAdoptOpenJDK-1.8.0_232-b09
spotbugshttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/artifact/yetus-general-check/output/branch-spotbugs-root-warnings.html
Max. process+thread count141 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-it . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=3.1.12
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec6m 23sDocker mode activated.
-0 ⚠️yetus0m 4sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 29sMaven dependency ordering for branch
+1 💚mvninstall4m 43sHBASE-22120 passed
+1 💚compile3m 8sHBASE-22120 passed
+1 💚shadedjars7m 52sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc4m 47sHBASE-22120 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 20sMaven dependency ordering for patch
+1 💚mvninstall4m 30sthe patch passed
+1 💚compile3m 10sthe patch passed
+1 💚javac3m 10sthe patch passed
+1 💚shadedjars7m 50spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc4m 48sthe patch passed
_ Other Tests _
-1 ❌unit215m 51sroot in the patch failed.
266m 19s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#2971
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 24e88d9d9b94 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / b765a0b
Default JavaAdoptOpenJDK-11.0.6+10
unithttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/testReport/
Max. process+thread count2957 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-it . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 26sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 29sMaven dependency ordering for branch
+1 💚mvninstall4m 7sHBASE-22120 passed
+1 💚compile2m 34sHBASE-22120 passed
+1 💚shadedjars7m 47sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc3m 45sHBASE-22120 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 21sMaven dependency ordering for patch
+1 💚mvninstall3m 53sthe patch passed
+1 💚compile2m 39sthe patch passed
+1 💚javac2m 39sthe patch passed
+1 💚shadedjars7m 42spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc3m 50sthe patch passed
_ Other Tests _
-1 ❌unit334m 19sroot in the patch failed.
375m 14s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#2971
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 1937d3ed7cb2 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / b765a0b
Default JavaAdoptOpenJDK-1.8.0_232-b09
unithttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/testReport/
Max. process+thread count2905 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-it . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/1/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Copy link
Copy Markdown
ContributorAuthor

The spotbugs warning is not introduced by this patch, it is a bug in the master branch.

Let me see the failed UTs. Seems opentelemetry has changed the way of initialization.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 30sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 33sMaven dependency ordering for branch
+1 💚mvninstall3m 48sHBASE-22120 passed
+1 💚compile8m 4sHBASE-22120 passed
+1 💚checkstyle1m 57sHBASE-22120 passed
-1 ❌spotbugs8m 54sroot in HBASE-22120 has 1 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 22sMaven dependency ordering for patch
+1 💚mvninstall3m 30sthe patch passed
+1 💚compile8m 11sthe patch passed
+1 💚javac8m 11sthe patch passed
+1 💚checkstyle1m 55sroot: The patch generated 0 new + 25 unchanged - 1 fixed = 25 total (was 26)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 3sThe patch has no ill-formed XML file.
+1 💚hadoopcheck17m 32sPatch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.
+1 💚spotbugs0m 50shbase-common in the patch passed.
+1 💚spotbugs1m 7shbase-client in the patch passed.
+1 💚spotbugs2m 10shbase-server in the patch passed.
+1 💚spotbugs0m 43shbase-it in the patch passed.
+1 💚spotbugs9m 8sroot generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
_ Other Tests _
+1 💚asflicense1m 6sThe patch does not generate ASF License warnings.
82m 57s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#2971
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile xml
unameLinux dcbdec40e491 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / b765a0b
Default JavaAdoptOpenJDK-1.8.0_232-b09
spotbugshttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/2/artifact/yetus-general-check/output/branch-spotbugs-root-warnings.html
Max. process+thread count141 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-it . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/2/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=3.1.12
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 4sDocker mode activated.
-0 ⚠️yetus0m 4sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 31sMaven dependency ordering for branch
+1 💚mvninstall4m 37sHBASE-22120 passed
+1 💚compile3m 8sHBASE-22120 passed
+1 💚shadedjars7m 48sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc4m 48sHBASE-22120 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 20sMaven dependency ordering for patch
+1 💚mvninstall4m 57sthe patch passed
+1 💚compile3m 37sthe patch passed
+1 💚javac3m 37sthe patch passed
+1 💚shadedjars8m 26spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc5m 3sthe patch passed
_ Other Tests _
-1 ❌unit255m 3sroot in the patch failed.
302m 2s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#2971
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 8133142dfd30 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / b765a0b
Default JavaAdoptOpenJDK-11.0.6+10
unithttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/2/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/2/testReport/
Max. process+thread count4992 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-it . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/2/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Copy link
Copy Markdown
ContributorAuthor

The failed shell UTs is not introduced by the patch here. Filed

https://issues.apache.org/jira/browse/HBASE-25602

for addressing them.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 30sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 33sMaven dependency ordering for branch
+1 💚mvninstall3m 46sHBASE-22120 passed
+1 💚compile8m 6sHBASE-22120 passed
+1 💚checkstyle1m 59sHBASE-22120 passed
-1 ❌spotbugs8m 55sroot in HBASE-22120 has 1 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 23sMaven dependency ordering for patch
+1 💚mvninstall3m 27sthe patch passed
+1 💚compile8m 4sthe patch passed
+1 💚javac8m 4sthe patch passed
+1 💚checkstyle1m 56sroot: The patch generated 0 new + 25 unchanged - 1 fixed = 25 total (was 26)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 3sThe patch has no ill-formed XML file.
+1 💚hadoopcheck17m 11sPatch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.
+1 💚spotbugs0m 51shbase-common in the patch passed.
+1 💚spotbugs1m 8shbase-client in the patch passed.
+1 💚spotbugs2m 8shbase-server in the patch passed.
+1 💚spotbugs0m 43shbase-it in the patch passed.
+1 💚spotbugs9m 4sroot generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
_ Other Tests _
+1 💚asflicense1m 6sThe patch does not generate ASF License warnings.
82m 45s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#2971
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile xml
unameLinux 6e84699fa629 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / fd160ac
Default JavaAdoptOpenJDK-1.8.0_232-b09
spotbugshttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/3/artifact/yetus-general-check/output/branch-spotbugs-root-warnings.html
Max. process+thread count141 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-it . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/3/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=3.1.12
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 9sDocker mode activated.
-0 ⚠️yetus0m 4sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 29sMaven dependency ordering for branch
+1 💚mvninstall4m 49sHBASE-22120 passed
+1 💚compile3m 11sHBASE-22120 passed
+1 💚shadedjars7m 49sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc4m 49sHBASE-22120 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 21sMaven dependency ordering for patch
+1 💚mvninstall4m 39sthe patch passed
+1 💚compile3m 12sthe patch passed
+1 💚javac3m 12sthe patch passed
+1 💚shadedjars7m 45spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc4m 51sthe patch passed
_ Other Tests _
-1 ❌unit210m 29sroot in the patch failed.
256m 18s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#2971
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 1fff8ca54012 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / fd160ac
Default JavaAdoptOpenJDK-11.0.6+10
unithttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/3/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/3/testReport/
Max. process+thread count3468 (vs. ulimit of 30000)
modulesC: hbase-common hbase-client hbase-server hbase-it . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2971/3/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@infraioinfraio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@Apache9
Apache9 merged commit f4a4eda into apache:HBASE-22120Feb 26, 2021
Apache9 added a commit that referenced this pull request Feb 27, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Mar 2, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Mar 8, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Mar 10, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Mar 24, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Mar 28, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Apr 2, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Apr 3, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Apr 8, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Apr 9, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Apr 15, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Apr 24, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Apache9 added a commit that referenced this pull request Apr 25, 2021
Signed-off-by: Guanghao Zhang <zghao@apache.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Apache9@Apache-HBase@infraio