Skip to content

HBASE-22924 GitHUB PR job should use when clause to filter to just PRs - #4915

Merged
Apache9 merged 1 commit into
apache:masterfrom
shankybnl:HBASE-22924
Dec 8, 2022
Merged

HBASE-22924 GitHUB PR job should use when clause to filter to just PRs#4915
Apache9 merged 1 commit into
apache:masterfrom
shankybnl:HBASE-22924

Conversation

@shankybnl

Copy link
Copy Markdown
Contributor
  1. Added changerequest() condition to create builds for PRs
  2. Removed the current shell script clause that skips non-PR builds

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 5sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗mvndep0m 4sMaven dependency ordering for patch
_ Other Tests _
2m 15s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4915/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#4915
Optional Tests
unameLinux 6437dfeeb29e 5.4.0-131-generic #147-Ubuntu SMP Fri Oct 14 17:07:22 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / febe511
Max. process+thread count39 (vs. ulimit of 30000)
modulesC: U:
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4915/1/console
versionsgit=2.34.1 maven=3.8.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 34sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗mvndep0m 4sMaven dependency ordering for patch
_ Other Tests _
2m 37s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4915/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#4915
Optional Tests
unameLinux 49c08953523a 5.4.0-1083-aws #90~18.04.1-Ubuntu SMP Fri Aug 5 08:12:44 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / febe511
Max. process+thread count32 (vs. ulimit of 30000)
modulesC: U:
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4915/1/console
versionsgit=2.34.1 maven=3.8.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 0sDocker 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.
_ master Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚spotless0m 40sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 4sMaven dependency ordering for patch
+1 💚shellcheck0m 1sThere were no new shellcheck issues.
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚spotless0m 38spatch has no errors when running spotless:check.
_ Other Tests _
+0 🆗asflicense0m 1sASF License check generated no output?
3m 35s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4915/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4915
Optional Testsdupname asflicense spotless shellcheck shelldocs
unameLinux 6b2cf93408f6 5.4.0-124-generic #140-Ubuntu SMP Thu Aug 4 02:23:37 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / febe511
Max. process+thread count43 (vs. ulimit of 30000)
modulesC: U:
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4915/1/console
versionsgit=2.34.1 maven=3.8.6 shellcheck=0.8.0
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Copy link
Copy Markdown
Contributor

@busbey I'm not sure how to test this change?

@Apache9

Copy link
Copy Markdown
Contributor

Let me push this to a feature branch...

@Apache9

Copy link
Copy Markdown
Contributor

@Apache9
Apache9 merged commit e1ad781 into apache:masterDec 8, 2022
Apache9 pushed a commit that referenced this pull request Dec 8, 2022
#4915)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit e1ad781)
Apache9 pushed a commit that referenced this pull request Dec 8, 2022
#4915)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit e1ad781)
Apache9 pushed a commit that referenced this pull request Dec 8, 2022
#4915)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit e1ad781)
vinayakphegde pushed a commit to vinayakphegde/hbase that referenced this pull request Apr 4, 2024
apache#4915)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit e1ad781)
(cherry picked from commit cc86844)
Change-Id: I3f023d0b24e97795810ae2fff74dd18390a52c0b
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

@shankybnl@Apache-HBase@Apache9