SUSE / rookcheck

Apache License 2.0
4 stars 11 forks source link

Structure ses/rook upstream the same way #132

Closed brunoleon closed 4 years ago

brunoleon commented 4 years ago

This adds build/preinstall/install steps to both implementation. Having a build() method is necessary in ses (though is does nothing usefull) so we can call it early in threaded tests that will opportunitely build rook if possible before deployment.

Move install code to base class as it is no upstream specific.

Move ceph features method (rbd, fs...) to base class too.

brunoleon commented 4 years ago

The test is failing on SES but that it not due to the code nor this PR, but because direct-mount.yaml file does not exist on SES6 distribution (at least not named like that).

We'll need to figure out a proper handling of this but I think this is not blocking this one.