search

projectatomic / atomic-host-tests

A collection of single-host tests for Atomic Host
GNU General Public License v3.0
18 stars 21 forks source link

stop using workarounds for `atomic` command #431

Closed miabbott closed 6 years ago

miabbott commented 6 years ago

The atomic command is fixed across all the streams, so we can stop using the workarounds.

mike-nguyen commented 6 years ago

centos/7/atomic still too old?

miabbott commented 6 years ago

The version in OpenStack is, but the latest released version should be fine.

Once we get the image updated, I'll re-run the CI for that.

miabbott commented 6 years ago

@mike-nguyen all green! :smile:

mike-nguyen commented 6 years ago

@miabbott Nice work! I think I also added -n cockpit to the atomic install cockpit command so the uninstall workaround would work. I'm okay with leaving it as is but if you feel like changing it, let me know and I'll hold off on merging.

miabbott commented 6 years ago

I left that in because we get coverage on the use of -n during atomic install and with it omitted later on.

mike-nguyen commented 6 years ago

LGTM then