charmed-kubernetes / pytest-operator

Apache License 2.0
7 stars 13 forks source link

Skip machine logging for K8s #9

Closed chris-sanders closed 3 years ago

chris-sanders commented 3 years ago

Testing this for a Container charm the operator fails on the log line because there is no Machine for container units. I have tests this to work on a new charm and can deploy and remove the model as expected. I'm not sure if this is the only place, but I think the other instances are protected by loops which will simply not happen with no machine.

I ran black and import ordering was rearranged. I can probably revert that if it matters.

chris-sanders commented 3 years ago

Oh right, and anyone testing this it will look like the model isn't removed. It does remove it takes a really long time though. Give it 5 minutes and see if it goes away.