Skip to content
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

cmd-kola: add support for kola-blacklist.yaml #866

Merged
merged 1 commit into from
Oct 22, 2019

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Oct 22, 2019

Rather than encoding the blacklist in three different places (the
fedora-coreos-config CI, the coreos-assembler CI, and the pipeline),
let's just teach cosa kola to auto-detect a kola-blacklist.yaml from
the src config and automatically blacklisting them when executing kola.

I proposed making this baked in kola itself with slightly different
semantics in: https://github.com/coreos/mantle/issues/1103

Though teaching this to cosa should still make things easier to maintain
for now at least.

jlebon added a commit to jlebon/fedora-coreos-config that referenced this pull request Oct 22, 2019
This will contain a documented list of tests known to fail right now.
For details, see the corresponding cosa PR which learns how to read
this, and the mantle RFE:

coreos/coreos-assembler#866
https://github.com/coreos/mantle/issues/1103
Rather than encoding the blacklist in three different places (the
fedora-coreos-config CI, the coreos-assembler CI, and the pipeline),
let's just teach `cosa kola` to auto-detect a `kola-blacklist.yaml` from
the src config and automatically blacklisting them when executing kola.

I proposed making this baked in kola itself with slightly different
semantics in: https://github.com/coreos/mantle/issues/1103

Though teaching this to cosa should still make things easier to maintain
for now at least.
@jlebon jlebon force-pushed the pr/kola-blacklist branch from 2ed535a to 6467811 Compare October 22, 2019 14:34
@jlebon
Copy link
Member Author

jlebon commented Oct 22, 2019

This and coreos/fedora-coreos-config#203 should unblock the pipeline.

Copy link
Member

@cgwalters cgwalters left a comment

Choose a reason for hiding this comment

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

Nice.

@jlebon
Copy link
Member Author

jlebon commented Oct 22, 2019

CI failure is from fcos.python, which will be fixed by coreos/fedora-coreos-config#203, which requires this PR. So merging!

@jlebon jlebon merged commit 81dce18 into coreos:master Oct 22, 2019
@jlebon jlebon deleted the pr/kola-blacklist branch October 22, 2019 15:21
jlebon added a commit to coreos/fedora-coreos-config that referenced this pull request Oct 22, 2019
This will contain a documented list of tests known to fail right now.
For details, see the corresponding cosa PR which learns how to read
this, and the mantle RFE:

coreos/coreos-assembler#866
https://github.com/coreos/mantle/issues/1103
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.

3 participants