Skip to content

Fix deprecated mc call, add more logging on CI failure #555

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

Merged
merged 8 commits into from
Jun 1, 2025

Conversation

AetherUnbound
Copy link
Collaborator

@AetherUnbound AetherUnbound commented May 26, 2025

Description of Changes

We've had some recent failures in the CI that looked like they were the result of Minio-related issues. This PR adds some additional debug tooling for the future, as well as addresses the underlying issue which is replacing mc config host add with mc alias set.

See: minio/minio#21327 (comment)

Notes for Deployment

Screenshots (if appropriate)

Testing instructions

  • Run just pull and then just up && just test on main, and you should see the issue occur
  • Running the same on this branch should let all tests pass!

Checks

  • I have rebased my changes on main

  • just lint passes

  • just test passes

@AetherUnbound AetherUnbound changed the title Tear down existing containers in CI & wait for buckets Fix deprecated mc call, add more logging on CI failure May 28, 2025
@AetherUnbound AetherUnbound merged commit e66b87a into main Jun 1, 2025
2 checks passed
@AetherUnbound AetherUnbound deleted the fix/ci-issues branch June 1, 2025 01:43
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Orca Collective Task Tracker Jun 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants