-
Notifications
You must be signed in to change notification settings - Fork 212
jenkins: Add initial jenkinsfile #189
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.
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
Conversation
0c09537 to
a47b025
Compare
Jenkinsfile
Outdated
| } | ||
| } | ||
| } | ||
| stage("make intel-fpga-admissionwebhook") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is practically useless. stage("make"){} already has built everything.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes made, thanks for the review
Jenkinsfile
Outdated
| } | ||
| } | ||
| } | ||
| stage("make images with buildah") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
images without buildah are not built.
Also, use parallel stages for building something in variations (e.g. images with docker and images with buildah)
https://jenkins.io/blog/2017/09/25/declarative-1/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes made, thanks for the review
b7e6cd1 to
8e870a4
Compare
|
Can you please evaluate if several of stages can be done in parallel? |
Jenkinsfile
Outdated
| } | ||
| stage('make plugins') { | ||
| parallel { | ||
| stage("make intel-fpga-admissionwebhook") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, in practice, those "make intel-*" targets are parts of the "make images" target, and in practice we are building all images at least twice with this pipeline. Can we improve that please?
4e905e6 to
748b873
Compare
|
@chuyd why it's crashing? no memory/disk space in the VM or something else? |
I was suspecting the same, I doubled the memory/disk resources in the last run a few hours ago and it failed on other different make target of the same stage |
0a528e2 to
836501b
Compare
Codecov Report
@@ Coverage Diff @@
## master #189 +/- ##
=======================================
Coverage 79.18% 79.18%
=======================================
Files 11 11
Lines 1494 1494
=======================================
Hits 1183 1183
Misses 279 279
Partials 32 32Continue to review full report at Codecov.
|
This Jenkinsfile provides basic CI based on the .travis.yml which ports the travis behaviour to Jenkins by checking the following make targets functionallity: - make format - make lint - make vet - make cyclomatic-check - make test - make images - make images BUILDER=buildah - make demos - make demos BUILDER=buildah The main intention of this commit is to provide an starting point where anybody can improve the coverage at anytime and get PR cheks on Jenkins.
kad
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
…c+gRPC-plugin ci-build: add rules to install protoc and gRPC plugin.
This Jenkinsfile provides basic CI based on the .travis.yml
which ports the travis behaviour to Jenkins by checking the
following make targets:
The main intention of this commit is to provide an starting point
where anybody can improve the coverage at anytime and get PR cheks
on Jenkins.