Skip to content

fix(operator): Repair moco/mysql rename regressions#184

Merged
jthakkar04 merged 2 commits into
v2from
jthakkar/moco-fixes
May 19, 2026
Merged

fix(operator): Repair moco/mysql rename regressions#184
jthakkar04 merged 2 commits into
v2from
jthakkar/moco-fixes

Conversation

@jthakkar04

Copy link
Copy Markdown

Summary

Repairs the chain of moco/mysql string mismatches that blocked on-prem installs from getting past MySQL setup. This PR Restores the end-to-end install: MOCO comes up Healthy, the W&B application pods deploy, and OTLP traffic flows from the gorilla services to the Victoria backends.

Testing

We spun up a cluster using:

./hack/scripts/setup_kind.sh ; tilt up

The before/after screenshots show the MySQL infrastructure stand-up succeeding after this PR.

Before:

We can notice the WeightsAndBiases CR's MYSQL column is empty and kubectl get mysqlcluster returns no resources — the operator never successfully created the cluster.
Screenshot 2026-05-19 at 1 37 09 PM

After:

We can notice the MYSQL reads Healthy and the wandb-moco MySQLCluster shows AVAILABLE=True, HEALTHY=True — which unblocks the rest of the install downstream.
Screenshot 2026-05-19 at 1 49 25 PM

@coderabbitai

coderabbitai Bot commented May 19, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ed10ac9f-050b-4a51-ad16-8807b55a3592

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jthakkar/moco-fixes

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jthakkar04 jthakkar04 force-pushed the jthakkar/moco-fixes branch from a037776 to 53e5fbc Compare May 19, 2026 19:32
@jthakkar04 jthakkar04 changed the title fix(operator): repair moco/mysql rename regressions fix(operator): Repair moco/mysql rename regressions May 19, 2026
@jthakkar04 jthakkar04 requested a review from collinol May 19, 2026 19:50
Comment thread internal/controller/reconciler/pods.go Outdated
Restores on-prem install by repairing the chain of moco/mysql string
mismatches introduced by #182 that blocked MySQL setup end-to-end.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@jthakkar04 jthakkar04 force-pushed the jthakkar/moco-fixes branch from 53e5fbc to 8c4d5bb Compare May 19, 2026 20:22
@jthakkar04 jthakkar04 requested a review from collinol May 19, 2026 20:34
@jthakkar04 jthakkar04 merged commit 3c72e62 into v2 May 19, 2026
5 checks passed
@jthakkar04 jthakkar04 deleted the jthakkar/moco-fixes branch May 19, 2026 21:24
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.

3 participants