Skip to content

feat(operator): add Ready printer column to MCPRemoteProxy CRD#4562

Merged
ChrisJBurns merged 1 commit intostacklok:mainfrom
tmchow:fix/mcpremoteproxy-ready-column
Apr 5, 2026
Merged

feat(operator): add Ready printer column to MCPRemoteProxy CRD#4562
ChrisJBurns merged 1 commit intostacklok:mainfrom
tmchow:fix/mcpremoteproxy-ready-column

Conversation

@tmchow
Copy link
Copy Markdown
Contributor

@tmchow tmchow commented Apr 4, 2026

MCPRemoteProxy was missing a Ready column in kubectl get output while other CRDs (MCPServer, MCPToolConfig, MCPGroup, MCPExternalAuthConfig) all surface their condition status. The controller already sets the Ready condition on MCPRemoteProxy resources; this adds the printer column to make it visible.

Changes:

  • Added +kubebuilder:printcolumn marker for Ready status in mcpremoteproxy_types.go
  • Regenerated CRD manifests via task operator-manifests

Fixes #4541

MCPRemoteProxy was the only CRD missing a Ready column in kubectl get
output. The controller already sets the Ready condition, it just wasn't
surfaced in the printer columns. Other CRDs (MCPServer, MCPToolConfig,
MCPGroup, MCPExternalAuthConfig) all show their Ready status.

CRD manifests regenerated via task operator-manifests.

Fixes stacklok#4541
@github-actions github-actions bot added the size/XS Extra small PR: < 100 lines changed label Apr 5, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.87%. Comparing base (ffef243) to head (4e7bbfc).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4562      +/-   ##
==========================================
+ Coverage   68.86%   68.87%   +0.01%     
==========================================
  Files         505      505              
  Lines       52320    52378      +58     
==========================================
+ Hits        36028    36076      +48     
- Misses      13504    13513       +9     
- Partials     2788     2789       +1     

☔ View full report in Codecov by Sentry.
📢 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 ChrisJBurns merged commit e64fc96 into stacklok:main Apr 5, 2026
36 checks passed
@tmchow
Copy link
Copy Markdown
Contributor Author

tmchow commented Apr 5, 2026

Thanks for the approve and merge @ChrisJBurns !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Extra small PR: < 100 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Ready printer column to MCPRemoteProxy

2 participants