Skip to content

fix(operator): remove CRD shortNames that collide with the resource's own names - #6689

Merged
ChrisJBurns merged 1 commit into
stacklok:mainfrom
lightsabit:fix/crd-shortname-collisions
Sep 21, 2026
Merged

ChrisJBurns merged 1 commit into
stacklok:mainfrom
lightsabit:fix/crd-shortname-collisions

Conversation

@lightsabit

Copy link
Copy Markdown
Contributor

The 0.49.0 mcpservers CRD ships shortNames [mcpserver, mcpservers], both equal to the CRD's own singular/plural, so it installs with NamesAccepted=False and helm --wait hangs on it (#6685).

Sweeping the other CRDs found the same problem on mcpgroups, which ships shortName mcpgroup -- its own singular. This drops the colliding entries (keeping mcpg) and regenerates via task operator-manifests, so both CRDs install clean.

Hit this on a live v0.49.0 upgrade; worked around it with a post-renderer until now.

… own names

The API server rejects a shortName equal to the CRD's singular or plural,
so the CRD installs with NamesAccepted=False and helm --wait never returns.

mcpservers shipped shortName=mcpserver;mcpservers, and mcpgroups
shortName=mcpgroup -- both their own singulars. Regenerated the CRDs.

Fixes stacklok#6685

Signed-off-by: lights <115397533+lightsabit@users.noreply.github.com>
@codecov

codecov Bot commented Sep 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.17%. Comparing base (5a4854a) to head (49226d4).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6689      +/-   ##
==========================================
- Coverage   79.18%   79.17%   -0.01%     
==========================================
  Files         790      791       +1     
  Lines       79121    79518     +397     
==========================================
+ Hits        62649    62959     +310     
- Misses      16467    16554      +87     
  Partials        5        5              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ChrisJBurns

Copy link
Copy Markdown
Collaborator

Many thanks for this @lightsabit !!

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.

2 participants