samba-in-kubernetes / samba-operator

An operator for a Samba as a service on PVCs in kubernetes
Apache License 2.0
117 stars 24 forks source link

integration tests: add new suite to test that things get cleaned up #128

Closed phlogistonjohn closed 2 years ago

phlogistonjohn commented 2 years ago

This adds integration tests to verify that after SmbShares are created the resources used to support the smb instances get cleaned up when the SmbShare is deleted.

The patches add cases for single server shares, single server shares in a different namespace, and clustered shares. Getting the clustered shares to work took additional infrastructure changes to handle cases where the pods were in the terminating state much longer than was reasonable for the test to wait. (details in the commit msgs)

Some of the lessons learned here could probably be applied to the existing tests as well, but that's a job for another day.