Skip to content
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.

Sign up for GitHub

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

Jump to bottom

HBASE-27935 Introduce Jenkins PR job for hbase-kustomize (addendum) #4

Merged
merged 1 commit into from Jun 19, 2023
Merged

HBASE-27935 Introduce Jenkins PR job for hbase-kustomize (addendum) #4

merged 1 commit into from Jun 19, 2023

Conversation

ndimiduk
Copy link
Member

Add executable permissions to the shell scripts.

@ndimiduk
Copy link
Member Author

PR build fails with

[2023-06-16T09:13:50.928Z] [INFO] Launching Yetus via /home/jenkins/jenkins-home/workspace/hbase-kustomize-github-pr_PR-3/yetus-precommit-check/src/dev-support/jenkins/jenkins_precommit_github_yetus.sh
[2023-06-16T09:13:50.928Z] /home/jenkins/jenkins-home/workspace/hbase-kustomize-github-pr_PR-3/yetus-precommit-check@tmp/durable-10c0e8a4/script.sh: line 4: /home/jenkins/jenkins-home/workspace/hbase-kustomize-github-pr_PR-3/yetus-precommit-check/src/dev-support/jenkins/jenkins_precommit_github_yetus.sh: Permission denied
script returned exit code 126

@ndimiduk
Copy link
Member Author

PR build fails with:

[2023-06-16T09:24:09.010Z] Starting docker build...
[2023-06-16T09:24:09.010Z] ERROR: BuildKit is enabled but the buildx component is missing or broken.
[2023-06-16T09:24:09.010Z]        Install the buildx component to build images with BuildKit:
[2023-06-16T09:24:09.010Z]        https://docs.docker.com/go/buildx/

@ndimiduk ndimiduk force-pushed the 27935-jenkins-pr-addendum branch 7 times, most recently from 44bfdd4 to 843e2ba Compare June 16, 2023 12:32
@Apache-HBase

This comment was marked as outdated.

Sign in to view
@ndimiduk ndimiduk requested a review from busbey June 16, 2023 12:40
@ndimiduk
HBASE-27935 Introduce Jenkins PR job for hbase-kustomize (addendum)
ae11ed7 
- add executable permissions to the shell scripts.
- disable docker buildkit in yetus.
- use Yetus's default dockerfile
- make yetus a little more forgiving
- correct pass-through of github credentials
- address yetus nits
- enable writing comments back to the PR (access doesn't permissions don't allow writing to Checks
  API ?)

Signed-off-by: Sean Busbey <busbey@apache.org>
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 3m 14s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ main Compile Tests _
_ Patch Compile Tests _
+1 💚 codespell 0m 0s No new issues.
+1 💚 detsecrets 0m 1s The patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 hadolint 0m 1s No new issues.
+1 💚 shellcheck 0m 0s No new issues.
+1 💚 shelldocs 0m 1s No new issues.
_ Other Tests _
+1 💚 asflicense 0m 0s The patch does not generate ASF License warnings.
3m 22s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43
GITHUB PR #4
Optional Tests dupname asflicense codespell detsecrets hadolint shellcheck shelldocs
uname Linux 12f64f2fa70f 5.4.0-1101-aws #109~18.04.1-Ubuntu SMP Mon Apr 24 20:40:49 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool nobuild
Personality /home/jenkins/jenkins-home/workspace/hbase-kustomize-github-pr_PR-4/yetus-precommit-check/src/.yetus/personality.sh
git revision main / ae11ed7
Max. process+thread count 8 (vs. ulimit of 5000)
modules C: . U: .
Console output https://ci-hbase.apache.org/job/hbase-kustomize-github-pr/job/PR-4/9/console
versions git=2.25.1 hadolint=2.10.0 codespell=2.2.1 detsecrets=1.2.0 shellcheck=0.8.0
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 26s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ main Compile Tests _
_ Patch Compile Tests _
+1 💚 codespell 0m 0s No new issues.
+1 💚 detsecrets 0m 1s The patch generated 0 new + 0 unchanged - 1 fixed = 0 total (was 1)
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 hadolint 0m 1s No new issues.
+1 💚 shellcheck 0m 0s No new issues.
+1 💚 shelldocs 0m 1s No new issues.
_ Other Tests _
+1 💚 asflicense 0m 0s The patch does not generate ASF License warnings.
0m 35s
Subsystem Report/Notes
Docker ClientAPI=1.43 ServerAPI=1.43
GITHUB PR #4
Optional Tests dupname asflicense codespell detsecrets hadolint shellcheck shelldocs
uname Linux c783bf341b49 5.4.0-1101-aws #109~18.04.1-Ubuntu SMP Mon Apr 24 20:40:49 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool nobuild
Personality /home/jenkins/jenkins-home/workspace/hbase-kustomize-github-pr_PR-4/yetus-precommit-check/src/.yetus/personality.sh
git revision main / ae11ed7
Max. process+thread count 9 (vs. ulimit of 5000)
modules C: . U: .
Console output https://ci-hbase.apache.org/job/hbase-kustomize-github-pr/job/PR-4/10/console
versions git=2.25.1 hadolint=2.10.0 codespell=2.2.1 detsecrets=1.2.0 shellcheck=0.8.0
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@ndimiduk ndimiduk merged commit dd6b2a1 into apache:main Jun 19, 2023
1 check passed
@ndimiduk ndimiduk deleted the 27935-jenkins-pr-addendum branch June 19, 2023 11:15
@ndimiduk
Copy link
Member Author

Thanks @busbey .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@busbey busbey busbey approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@ndimiduk @Apache-HBase @busbey