Skip to content
This repository was archived by the owner on May 7, 2021. It is now read-only.

Conversation

@arithx
Copy link
Contributor

@arithx arithx commented Mar 11, 2019

Adds a new flag --oscontainer to kola which allows the specification
of an oscontainer image pull spec when running against the rhcos
distribution. When specified it will create the relevant files & the
systemd units required to perform a pivot operation before sshd
starts.

Closes #987

@arithx
Copy link
Contributor Author

arithx commented Mar 11, 2019

Notably: both NFS related tests (coreos.ignition.systemd.enable-service & linux.nfs.v4) that are enabled on rhcos are broken when doing an oscontainer pivot (openshift/os#379)

@arithx arithx force-pushed the oscontainer_pivot branch from 9dd75b5 to 0860ebe Compare March 11, 2019 22:04
@arithx arithx force-pushed the oscontainer_pivot branch 2 times, most recently from 26de310 to 971cec3 Compare March 12, 2019 15:18
@arithx arithx force-pushed the oscontainer_pivot branch 2 times, most recently from b3890c0 to 5dad743 Compare March 13, 2019 14:44
Adds a new flag `--oscontainer` to `kola` which allows the specification
of an oscontainer image pull spec when running against the `rhcos`
distribution. When specified it will create the relevant files & the
systemd units required to perform a `pivot` operation before `sshd`
starts.
@arithx arithx force-pushed the oscontainer_pivot branch from 5dad743 to 8ca2950 Compare March 13, 2019 16:35
Also updates all tests using Ignition v1 to only run against the CL
distribution. Neither FCOS nor RHCOS should ever expect to receive
Ignition v1 configs.
@arithx arithx force-pushed the oscontainer_pivot branch from 8ca2950 to db4df2f Compare March 13, 2019 19:05
@arithx
Copy link
Contributor Author

arithx commented Mar 13, 2019

Pushed an update with a few more test disables. This one converts all Ignition v1 tests to be CL specific as neither RHCOS nor FCOS should ever be receiving these configs.

Copy link
Member

@miabbott miabbott left a comment

Choose a reason for hiding this comment

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

Thanks for improving the ostree/rpm-ostree tests!

@arithx arithx merged commit d600161 into coreos:master Mar 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants