Skip to content

kola/tests: move OCP tests to RequiredTag#4122

Merged
jlebon merged 1 commit into
coreos:mainfrom
jlebon:pr/required-openshift
Jun 3, 2025
Merged

kola/tests: move OCP tests to RequiredTag#4122
jlebon merged 1 commit into
coreos:mainfrom
jlebon:pr/required-openshift

Conversation

@jlebon
Copy link
Copy Markdown
Member

@jlebon jlebon commented Jun 2, 2025

Now that the layered CoreOS work has landed, the variants built by cosa no longer have OCP components. So let's default to not running OpenShift tests at the kola level by changing them to use RequiredTag.

This will allow us to drop the --tag !openshift in various places. Running OpenShift tests will require passing --tag openshift.

Now that the layered CoreOS work has landed, the variants built by cosa
no longer have OCP components. So let's default to not running OpenShift
tests at the kola level by changing them to use `RequiredTag`.

This will allow us to drop the `--tag !openshift` in various places.
Running OpenShift tests will require passing `--tag openshift`.
jlebon added a commit to jlebon/rhel-coreos-config that referenced this pull request Jun 2, 2025
This is no longer required now that we don't run OpenShift tests by
default.

Requires: coreos/coreos-assembler#4122
jlebon added a commit to jlebon/rhel-coreos-config that referenced this pull request Jun 2, 2025
This is no longer required now that we don't run OpenShift tests by
default.

Requires: coreos/coreos-assembler#4122
Copy link
Copy Markdown
Member

@dustymabe dustymabe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jlebon jlebon merged commit ec20f34 into coreos:main Jun 3, 2025
5 checks passed
@jlebon jlebon deleted the pr/required-openshift branch June 3, 2025 19:13
dustymabe pushed a commit to jlebon/rhel-coreos-config that referenced this pull request Jun 20, 2025
This is no longer required now that we don't run OpenShift tests by
default.

Requires: coreos/coreos-assembler#4122
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants