Skip to content

HBASE-25424 Find a way to config OpenTelemetry tracing without direct… - #2808

Merged
Apache9 merged 2 commits into
apache:HBASE-22120from
Apache9:HBASE-25424
Dec 24, 2020
Merged

HBASE-25424 Find a way to config OpenTelemetry tracing without direct…#2808
Apache9 merged 2 commits into
apache:HBASE-22120from
Apache9:HBASE-25424

Conversation

@Apache9

Copy link
Copy Markdown
Contributor

…ly depending on opentelemetry-sdk

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 30sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 30sMaven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for patch
-0 ⚠️shellcheck0m 2sThe patch generated 3 new + 34 unchanged - 0 fixed = 37 total (was 34)
-0 ⚠️whitespace0m 0sThe patch has 2 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply
+1 💚xml0m 4sThe 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.
_ Other Tests _
+1 💚asflicense0m 29sThe patch does not generate ASF License warnings.
27m 18s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#2808
Optional Testsdupname asflicense shellcheck shelldocs hadoopcheck xml
unameLinux 58b822d742a9 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / bd745da
shellcheckhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/1/artifact/yetus-general-check/output/diff-patch-shellcheck.txt
whitespacehttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/1/artifact/yetus-general-check/output/whitespace-eol.txt
Max. process+thread count76 (vs. ulimit of 30000)
modulesC: . hbase-assembly U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/1/console
versionsgit=2.17.1 maven=3.6.3 shellcheck=0.4.6
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 🆗reexec0m 28sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed 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 12sHBASE-22120 passed
+1 💚compile2m 48sHBASE-22120 passed
+1 💚shadedjars7m 9sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc3m 1sHBASE-22120 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 22sMaven dependency ordering for patch
+1 💚mvninstall4m 2sthe patch passed
+1 💚compile2m 56sthe patch passed
+1 💚javac2m 56sthe patch passed
+1 💚shadedjars7m 0spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc2m 58sthe patch passed
_ Other Tests _
-1 ❌unit144m 1sroot in the patch failed.
182m 6s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#2808
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 4a0a4da538ad 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / bd745da
Default JavaAdoptOpenJDK-11.0.6+10
unithttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/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-2808/1/testReport/
Max. process+thread count4147 (vs. ulimit of 30000)
modulesC: hbase-assembly . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/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 🆗reexec9m 22sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗shelldocs0m 1sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 30sMaven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
+1 💚shellcheck0m 2sThere were no new shellcheck issues.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 5sThe patch has no ill-formed XML file.
+1 💚hadoopcheck20m 36sPatch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.
_ Other Tests _
+1 💚asflicense0m 28sThe patch does not generate ASF License warnings.
39m 12s
SubsystemReport/Notes
DockerClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#2808
Optional Testsdupname asflicense shellcheck shelldocs hadoopcheck xml
unameLinux 997d5111a4ed 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / bd745da
Max. process+thread count63 (vs. ulimit of 30000)
modulesC: . hbase-assembly U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/2/console
versionsgit=2.17.1 maven=3.6.3 shellcheck=0.4.6
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 14sDocker 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 34sMaven dependency ordering for branch
+1 💚mvninstall4m 19sHBASE-22120 passed
+1 💚compile2m 50sHBASE-22120 passed
+1 💚shadedjars7m 3sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc2m 57sHBASE-22120 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 22sMaven dependency ordering for patch
+1 💚mvninstall3m 59sthe patch passed
+1 💚compile2m 51sthe patch passed
+1 💚javac2m 51sthe patch passed
+1 💚shadedjars6m 58spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc2m 56sthe patch passed
_ Other Tests _
-1 ❌unit149m 47sroot in the patch failed.
188m 28s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#2808
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 8055092030e8 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 / bd745da
Default JavaAdoptOpenJDK-11.0.6+10
unithttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/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-2808/2/testReport/
Max. process+thread count3857 (vs. ulimit of 30000)
modulesC: hbase-assembly . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/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.

Comment threadconf/hbase-env.sh

# Uncomment to enable trace, you can change the options to use other exporters such as jaeger or
# zipkin
# export HBASE_TRACE_OPTS="-Dotel.config.sampler.probability=0.1 -Dotel.exporter=logging"

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.

Can we add a url to show that how to config otel.exporter?

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 28sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 35sMaven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚shellcheck0m 2sThere were no new shellcheck issues.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 3sThe patch has no ill-formed XML file.
+1 💚hadoopcheck17m 48sPatch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.
_ Other Tests _
+1 💚asflicense0m 30sThe patch does not generate ASF License warnings.
26m 30s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#2808
Optional Testsdupname asflicense shellcheck shelldocs hadoopcheck xml
unameLinux 0023de2a58ee 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / bd745da
Max. process+thread count76 (vs. ulimit of 30000)
modulesC: . hbase-assembly U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/3/console
versionsgit=2.17.1 maven=3.6.3 shellcheck=0.4.6
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

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 29sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗shelldocs0m 0sShelldocs was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ HBASE-22120 Compile Tests _
+0 🆗mvndep0m 19sMaven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗mvndep0m 19sMaven dependency ordering for patch
+1 💚shellcheck0m 1sThere were no new shellcheck issues.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 4sThe patch has no ill-formed XML file.
+1 💚hadoopcheck17m 8sPatch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.
_ Other Tests _
+1 💚asflicense0m 27sThe patch does not generate ASF License warnings.
25m 40s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#2808
Optional Testsdupname asflicense shellcheck shelldocs hadoopcheck xml
unameLinux def44d78dba3 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / bd745da
Max. process+thread count76 (vs. ulimit of 30000)
modulesC: hbase-assembly . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/4/console
versionsgit=2.17.1 maven=3.6.3 shellcheck=0.4.6
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 1sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed 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 19sMaven dependency ordering for branch
+1 💚mvninstall4m 0sHBASE-22120 passed
+1 💚compile2m 51sHBASE-22120 passed
+1 💚shadedjars7m 0sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc2m 57sHBASE-22120 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 19sMaven dependency ordering for patch
+1 💚mvninstall4m 3sthe patch passed
+1 💚compile2m 50sthe patch passed
+1 💚javac2m 50sthe patch passed
+1 💚shadedjars7m 0spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc2m 53sthe patch passed
_ Other Tests _
-1 ❌unit150m 59sroot in the patch failed.
188m 24s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/4/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#2808
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 552a83048cbd 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 / bd745da
Default JavaAdoptOpenJDK-11.0.6+10
unithttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/4/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-2808/4/testReport/
Max. process+thread count3890 (vs. ulimit of 30000)
modulesC: hbase-assembly . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/4/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 13sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed 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 17sMaven dependency ordering for branch
+1 💚mvninstall3m 25sHBASE-22120 passed
+1 💚compile2m 26sHBASE-22120 passed
+1 💚shadedjars6m 58sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc2m 21sHBASE-22120 passed
_ Patch Compile Tests _
+0 🆗mvndep0m 19sMaven dependency ordering for patch
+1 💚mvninstall3m 26sthe patch passed
+1 💚compile2m 19sthe patch passed
+1 💚javac2m 19sthe patch passed
+1 💚shadedjars6m 50spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc2m 19sthe patch passed
_ Other Tests _
-1 ❌unit317m 32sroot in the patch failed.
351m 56s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/4/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#2808
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 5f20a1483d07 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-22120 / bd745da
Default JavaAdoptOpenJDK-1.8.0_232-b09
unithttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/4/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-2808/4/testReport/
Max. process+thread count4278 (vs. ulimit of 30000)
modulesC: hbase-assembly . U: .
Console outputhttps://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2808/4/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
Apache9 merged commit 91f8aaf into apache:HBASE-22120Dec 24, 2020
asfgit pushed a commit that referenced this pull request Dec 26, 2020
asfgit pushed a commit that referenced this pull request Dec 28, 2020
asfgit pushed a commit that referenced this pull request Dec 29, 2020
asfgit pushed a commit that referenced this pull request Dec 30, 2020
asfgit pushed a commit that referenced this pull request Dec 31, 2020
Apache9 added a commit that referenced this pull request Jan 1, 2021
Apache9 added a commit that referenced this pull request Jan 5, 2021
Apache9 added a commit that referenced this pull request Jan 7, 2021
asfgit pushed a commit that referenced this pull request Jan 8, 2021
asfgit pushed a commit that referenced this pull request Jan 8, 2021
Apache9 added a commit that referenced this pull request Jan 11, 2021
Apache9 added a commit that referenced this pull request Jan 12, 2021
Apache9 added a commit that referenced this pull request Jan 13, 2021
Apache9 added a commit that referenced this pull request Jan 18, 2021
Apache9 added a commit that referenced this pull request Jan 21, 2021
Apache9 added a commit that referenced this pull request Jan 23, 2021
Apache9 added a commit that referenced this pull request Jan 28, 2021
Apache9 added a commit that referenced this pull request Jan 28, 2021
Apache9 added a commit that referenced this pull request Feb 18, 2021
Apache9 added a commit that referenced this pull request Feb 21, 2021
Apache9 added a commit that referenced this pull request Feb 25, 2021
Apache9 added a commit that referenced this pull request Feb 27, 2021
Apache9 added a commit that referenced this pull request Mar 2, 2021
Apache9 added a commit that referenced this pull request Mar 8, 2021
Apache9 added a commit that referenced this pull request Mar 10, 2021
Apache9 added a commit that referenced this pull request Mar 24, 2021
Apache9 added a commit that referenced this pull request Mar 28, 2021
Apache9 added a commit that referenced this pull request Apr 2, 2021
Apache9 added a commit that referenced this pull request Apr 3, 2021
Apache9 added a commit that referenced this pull request Apr 8, 2021
Apache9 added a commit that referenced this pull request Apr 9, 2021
Apache9 added a commit that referenced this pull request Apr 15, 2021
Apache9 added a commit that referenced this pull request Apr 24, 2021
Apache9 added a commit that referenced this pull request Apr 25, 2021
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