-
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 stream level skip_kola_tags
knob
#1008
Conversation
Introduce a kola parameter to allow skipping a list of kola tags. This is particularly useful when adding new streams to the pipeline. xref: coreos/fedora-coreos-pipeline#1008 Reference: coreos/fedora-coreos-pipeline#1002
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.
Small nit, but LGTM overall!
One thing I'll mention is that another way we could have done this is to extend test metadata to specify the variants against which it can run (similarly to what we have today for distros
, or maybe even as an extension to that, e.g. distros: [fcos, rhcos:my-variant]
).
OTOH, this is straightforward and it's not clear whether we'll need this functionality again once we've fully moved over to the new variants. At which point, we could also revert this PR and the one in coreos-ci-lib.
jobs/kola-azure.Jenkinsfile
Outdated
@@ -138,7 +138,8 @@ cosaPod(memory: "${cosa_memory_request_mb}Mi", kvm: false, | |||
platformArgs: """-p=azure \ | |||
--azure-credentials \${AZURE_KOLA_TESTS_CONFIG} \ | |||
--azure-location $region \ | |||
--azure-disk-uri /subscriptions/${azure_subscription}/resourceGroups/${azure_testing_resource_group}/providers/Microsoft.Compute/images/${azure_image_name}""") | |||
--azure-disk-uri /subscriptions/${azure_subscription}/resourceGroups/${azure_testing_resource_group}/providers/Microsoft.Compute/images/${azure_image_name}""", | |||
skipKolaTags: stream_info.skip_kola_tags) |
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.
Indentation here is off.
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!
Introduce a kola parameter to allow skipping a list of kola tags. This is particularly useful when adding new streams to the pipeline. xref: coreos/fedora-coreos-pipeline#1008 Reference: coreos/fedora-coreos-pipeline#1002
Kola can now support skipping kola tags per stream[1]. Add support for that here by modifying each Jenkins job to pass the list of kola tags to skip to the kola function. [1]: coreos/coreos-ci-lib#159 See: coreos#1002
Introduce a kola parameter to allow skipping a list of kola tags. This is particularly useful when adding new streams to the pipeline. xref: coreos/fedora-coreos-pipeline#1008 Reference: coreos/fedora-coreos-pipeline#1002
Introduce a kola parameter to allow skipping a list of kola tags. This is particularly useful when adding new streams to the pipeline. xref: coreos/fedora-coreos-pipeline#1008 Reference: coreos/fedora-coreos-pipeline#1002
Kola can now support skipping kola tags per stream [1]. Add support for that here by modifying each Jenkins job to pass the list of kola tags to skip to the kola function.
[1]: coreos/coreos-ci-lib#159
See: #1002