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
Switch to Jenkinsfile for jenkins job configuration #108
Conversation
Jenkinsfile
Outdated
def build(label, toxenv) { | ||
try { | ||
checkout scm | ||
if (label.startsWith("windows")) { |
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.
All of our builders appear to be windows for now?
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.
(FreeBSD comes next to match the existing bcrypt PR builder)
Jenkinsfile
Outdated
|
||
def build(label, toxenv) { | ||
try { | ||
checkout scm |
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.
Should this be a separate stage?
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.
Yeah probably so
cafe67f
to
8bd48fc
Compare
This is now ready for review. One caveat: the ansiColor build wrapper is currently of limited utility and doesn't work except in the combined console output view. |
Also, the separation of scm checkout into a different stage makes the non-blue ocean view pretty unreadable. |
Jenkinsfile
Outdated
} | ||
|
||
def builders = [:] | ||
for (x in configs) { |
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.
Real variable name.
Jenkinsfile
Outdated
def label = x["label"] | ||
def toxenvs = x["toxenvs"] | ||
|
||
for (y in toxenvs) { |
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.
Real vairabla nae
Jenkinsfile
Outdated
} else { | ||
ansiColor('xterm') { | ||
sh """ | ||
tox -r -e $toxenv -- --color=yes |
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.
Can this just be a one liner?
Besides those comments, is there anything blocking landing thsi? |
This is WIP until FreeBSD 11 is added (and it actually works)