Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add minio/enterprise #6

Closed
wants to merge 105 commits into from
Closed

add minio/enterprise #6

wants to merge 105 commits into from

Conversation

stavros-k
Copy link
Contributor

No description provided.

@stavros-k
Copy link
Contributor Author

stavros-k commented May 10, 2024

That was a journey but,

  • removed the need of the additional go tool for checking container's health.
  • now for each test file in each changed app, a new job will spin. that some benefits:
    • parallel runs, (no more waiting an hour or even more for apps like minecraft or immich)
    • clean workspace for each test (so we can avoid parsing compose files to identify host paths used to clean them before the next run)

There are 2 python scripts that are probably hideous to look at. But I figured python would be easier for others to chime in and improve things later.
I've placed a max of 10 jobs in parallel to avoid hammering github :D.

@stavros-k stavros-k mentioned this pull request May 14, 2024
@stavros-k
Copy link
Contributor Author

See #8

@stavros-k stavros-k closed this May 14, 2024
@stavros-k stavros-k deleted the minio-enterprise branch August 1, 2024 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant