Improve testing - Using text files that represent object might be able to be improved. about cockroach-operator HOT 5 CLOSED

@vladdy on a personal note, I very much appreciate your incite and feedback. You know this project a lot better than I do, and did a ton of work on it. It is in really good shape, and I appreciate what you have done with it. I am lucky to work on this now, and collaborate with you more in the future!

from cockroach-operator.

Thanks Chris. I would be happy to help with some work here.

from cockroach-operator.

You don't need to update a lot of text documents with the current implementation. If what you get is expected, the '-update' command line flag for test will do the needed update of fixtures.

The problem with the Diff on the exact object in integration/e2e tests is that you can miss objects you do not expect to be created and it is hard to understand what underlying object is expected or the entire setup of multiple objects looking at the test code and matching it to the API objects. It also does not solve the original cause of your issue - there is no predefined expected configuration of test environment to run against. I experimented with KIND working on this operator and it appears to be an appropriate option in my opinion.

Those are just my thoughts and not something what I insist on.

from cockroach-operator.

there is no predefined expected configuration of a test environment to run against

That is gonna get fixed. Great point and opening an issue. And I think that will help with part of this. I am a gke guy and rarely run k8s locally, so hence I ran into this. I opened this issue mostly to write and think about this, as we will probably have to test this against GKE and other on-prem systems to pass testing.

from cockroach-operator.

Closing for now, an reopen if we need it

from cockroach-operator.