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

kola: skip Docker torcx tests in case of Edge channel #61

Merged
merged 1 commit into from
Mar 13, 2020

Conversation

dongsupark
Copy link
Member

Since the docker 1.12 profile was removed in Edge, many kola tests started to fail to run, because of the missing docker 1.12 profile. Ideally we should be able to make a different set of profiles for each channel. Unfortunately the current Jenkins pipeline is not able to do that. It takes into account only flatcar-master branch.

To work around the issue, read directly a version from /etc/os-release to detect if the image is Edge. If the semver's minor version is 99, then it means Edge, so skip the tests.

Note, the code is only a quick-and-dirty approach. In the future, we should create a generic way to make channel names available for individual tests.

Since the docker 1.12 profile was removed in Edge, many kola tests
started to fail to run, because of the missing docker 1.12 profile.
Ideally we should be able to make a different set of profiles for each
channel. Unfortunately the current Jenkins pipeline is not able to do
that. It takes into account only flatcar-master branch.

To work around the issue, read directly a version from `/etc/os-release`
to detect if the image is Edge. If the semver's minor version is 99,
then it means Edge, so skip the tests.

Note, the code is only a quick-and-dirty approach. In the future, we
should create a generic way to make channel names available for
individual tests.
@t-lo
Copy link
Member

t-lo commented Mar 13, 2020

Testing done:

  • ran docker.torcx-flag-file.cloud-config and docker.torcx-flag-file on Edgel Alpha, Beta, and Stable. (only on Azure, but results should be repeatable on all other platforms)
  • Tests passed for Alpha, Beta, and Stable. Tests were skipped for Edge.

LGTM!

@dongsupark dongsupark merged commit 397a61f into flatcar-master Mar 13, 2020
@dongsupark dongsupark deleted the dongsu/edge-skip-docker-torcx-flags branch March 13, 2020 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working edge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants