Skip to content

updated resource limit for MCPRemoteProxy#3133

Closed
chandru-mck-2002 wants to merge 2 commits intostacklok:mainfrom
chandru-mck-2002:main
Closed

updated resource limit for MCPRemoteProxy#3133
chandru-mck-2002 wants to merge 2 commits intostacklok:mainfrom
chandru-mck-2002:main

Conversation

@chandru-mck-2002
Copy link
Contributor

No description provided.

@chandru-mck-2002 chandru-mck-2002 marked this pull request as draft December 22, 2025 09:44
@github-actions github-actions bot added the size/S Small PR: 100-299 lines changed label Dec 22, 2025
@chandru-mck-2002 chandru-mck-2002 marked this pull request as ready for review December 22, 2025 09:45
@dmjb
Copy link
Contributor

dmjb commented Jan 13, 2026

@chandru-mck-2002 Thank you for your PR. Could you add a description of why you are changing the limits, and how you selected the new limits in your PR? Did you any testing of these limits?

@codecov
Copy link

codecov bot commented Jan 13, 2026

Codecov Report

❌ Patch coverage is 89.18919% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.67%. Comparing base (bc0d112) to head (945b8e1).
⚠️ Report is 630 commits behind head on main.

Files with missing lines Patch % Lines
cmd/thv-operator/pkg/controllerutil/resources.go 87.09% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3133      +/-   ##
==========================================
+ Coverage   63.64%   63.67%   +0.03%     
==========================================
  Files         361      361              
  Lines       35449    35484      +35     
==========================================
+ Hits        22560    22596      +36     
+ Misses      11082    11075       -7     
- Partials     1807     1813       +6     

☔ 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.

@JAORMX JAORMX closed this Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Small PR: 100-299 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants