Default bucket fix

General MR

Summary

I noticed something when doing the last mattermost update: the minio bucket creation job would sometimes fail (basically the istio-proxy was dying before the workload could complete - I couldn’t figure out why that was happening). It’s not actually a showstopper because Flux eventually reconciles it and it seems like a toss up on whether the workload beats istio-proxy. But while debugging this I got pointed to another workaround for that whole job - and that’s to set the bucket in the minio chart.

Not sure if there was a reason for this to be a separate job (other than not knowing about this setting or likely pre-dating it), but this MR removes the job and adds it as a passthrough to minio if that job is in fact excessive.

Relevant logs/screenshots

n/a

added kindbug statusreview labels

requested review from @cwilliams68067, @snaq11092, and @dhilgaertner2

marked this merge request as draft

added 3 commits

• 0c6570cc...012ad875 - 2 commits from branch main
• af211c69 - Merge remote-tracking branch 'origin/main' into default-bucket-fix

Compare with previous version

assigned to @meganwolf

added statusdoing label and removed statusreview label

added 1 commit

• 97145dd3 - Updated chart version and docs

Compare with previous version

marked this merge request as ready

added statusreview label and removed statusdoing label

This was a good find. The default-bucket-creation job was originally done because minio did not add buckets to their upstream charts until we kpt upgraded to v4.4.10 in minio@cff7d4ad

This doesn't fix the Warnings we see, but it at least leaves one less file to analyze when trying to troubleshoot pipelines.

added 6 commits

• 97145dd3...93e20d20 - 2 commits from branch main
• d46df03f - Added istio annotation
• c3394608 - dang tabs
• f0bd3b7e - testing bucket setting via minio chart
• f1fcf84a - rebase

Compare with previous version

Looks good to me. Waiting for pipeline

Looks like the banner check within the mattermost cypress health tests. It appears cypress/chrome crashes.

approved this merge request

resolved all threads

approved this merge request

mentioned in commit b3c1884b

merged