-
Notifications
You must be signed in to change notification settings - Fork 56
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
jobs/build: add support for reading variant from stream/pipeline config #787
Conversation
Let's not mention any specific variant name in this PR. Something like "Add support for reading variant from the stream/pipecfg config" |
Can you add documentation for the new knob in |
I added a note in the docs about this in the most recent push |
jobs/cloud-replicate.Jenkinsfile
Outdated
@@ -35,6 +35,8 @@ properties([ | |||
|
|||
def build_description = "[${params.STREAM}]" | |||
|
|||
def stream_info = pipecfg.streams[params.STREAM] |
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 already defined lower down.
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.
Fixed
jobs/kola-openstack.Jenkinsfile
Outdated
@@ -41,6 +41,8 @@ properties([ | |||
|
|||
currentBuild.description = "[${params.STREAM}][${params.ARCH}] - ${params.VERSION}" | |||
|
|||
def variant = stream_info.variant ? "--variant ${stream_info.variant}" : "" |
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.
I suppose you meant to define stream_info
here?
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.
Fixed
jobs/release.Jenkinsfile
Outdated
@@ -42,6 +42,8 @@ properties([ | |||
|
|||
def build_description = "[${params.STREAM}]" | |||
|
|||
def stream_info = pipecfg.streams[params.STREAM] |
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 already defined lower down.
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.
Fixed
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! (Just pushed a small update to avoid some unnecessary whitespace changes.)
🎉 |
Add support in
build.jenkinsfile
andbuild-arch.jenkinsfile
for reading variant information from the pipeline config/stream and passing it intocosa init
.xref: https://gitlab.cee.redhat.com/coreos/rhcos-devel-pipecfg/-/merge_requests/3 & https://gitlab.cee.redhat.com/coreos/rhcos-art-pipecfg/-/merge_requests/9
See COS-1925