ddev / ddev-minio

MinIO addon for DDEV
Apache License 2.0
6 stars 1 forks source link

Please get tests going, and nightly tests #6

Closed rfay closed 1 year ago

rfay commented 1 year ago

For these add-ons to be successful, some decent automated tests (and run nightly) are critical. Please enable them and make sure the tests are good.

seebeen commented 1 year ago

Hello,

There are tests. Both for stable, and Edge and HEAD. I'm running stable on my custom runner which always has a stable version of ddev for my other projects, and HEAD and edge tests on the github runners.

Stable is run on my runner because it's much much faster than github runners

Feel free to check out the workflows.

rfay commented 1 year ago

I'm fine with local runners, but it would be better if they were exposed here I guess. I assume you're not using self-hosted github actions runners.

I guess maybe link to the test workflows on the README if that's the strategy?

Thanks!

seebeen commented 1 year ago

No, for stable I'm using a custom runner maintained by me. It's always on latest stable DDEV version. It saves a lot of time during testing. Entire test suite is complete in 1m (compared to 5m+ for github runners).
For bleeding Edge DDEV, I'm using github hosted runners.

In any case, the tests are there - and addon works.

rfay commented 1 year ago

The problem with local runners is the maintenance factor. I know because we have to use buildkite local runners with DDEV for macOS and Windows + WSL2. And I'm the one who has to reboot them when they fail. And there's the visibility factor - nobody else can contribute. That's where Github Actions (or CircleCI, etc.) shine: They're maintainable by a team, and their results are visible and repeatable.

In any case, the tests are there - and addon works.

The idea of open source and Ci is not just that "it works", but rather than a team or organization can verify that it works and be confident that it will work in 1 year or 5 years.

seebeen commented 1 year ago

OK, so if I move the e2e tests to an online runner - it's all good? :)

rfay commented 1 year ago

Well, it's your add-on and can be done any way you want until the point you want to promote it to an official add-on.

But yes, online runners mean more visibility and a wider community being able to maintain and improve it.

seebeen commented 1 year ago

No problems. I'll rework the test suite according to your recommendations.

And, yes - I'd love for the addon to be migrated to official space - with me as maintainer 😊

rfay commented 1 year ago

I note that the tests are failing consistently. Could you please get them working?

I recommend not using the :latest tag for minio/minio, because things can change underneath you.

douggreen commented 1 year ago

You can close this issue. It seems to have been a problem with my local setup. I did a bunch of cleanup (removed ~/.lima, ~/.colima, and ~/.ddev) and got it working.

rfay commented 1 year ago

No, this issue can't be closed. The tests are failing every night.

seebeen commented 1 year ago

Tests reworked and fully operational

rfay commented 1 year ago

Thanks!