Standardize proxy port field in MCPRemoteProxy

[slyt3]

Description:

This PR aligns MCPRemoteProxy with MCPServer by standardizing the proxy port field naming.

Summary

• Added proxyPort to MCPRemoteProxySpec
• Deprecated the legacy port field
• Preserved backward compatibility during the transition

Details

• GetProxyPort() now resolves ports in the following order:
  1. proxyPort
  2. port
  3. default (8080)

• Updated controller and deployment tests

• Added test coverage to ensure proxyPort takes precedence over port

• Verified existing resources using port continue to work

This improves API consistency across MCP resources while maintaining a smooth migration path for existing users.

Fixes #3102

[codecov]

Codecov Report

❌ Patch coverage is 50.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.95%. Comparing base (5d6121e) to head (52cc3df).
⚠️ Report is 5 commits behind head on main.

Files with missing lines	Patch %	Lines
.../thv-operator/api/v1alpha1/mcpremoteproxy_types.go	50.00%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3257      +/-   ##
==========================================
+ Coverage   65.94%   65.95%   +0.01%     
==========================================
  Files         413      413              
  Lines       41070    41075       +5     
==========================================
+ Hits        27082    27092      +10     
+ Misses      11892    11888       -4     
+ Partials     2096     2095       -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.

[dmjb]

This looks like a worthwhile change to me. @ChrisJBurns @JAORMX any concerns?

[dmjb]

@slyt3 You'll need to regenerate the CRDs. From the base of the project, run:

cd cmd/thv-operator/
task crdref-gen

[slyt3]

@slyt3 You'll need to regenerate the CRDs. From the base of the project, run:

cd cmd/thv-operator/

task crdref-gen

Done

[dmjb]

@slyt3 Looks good, but you'll also need to bump the helm chart under ./deploy/charts/operator-crds/Chart.yaml

After doing that, you may need to run task docs to update helm chart documentation.

[ChrisJBurns]

@dmjb I remember being in favour of this but I think @JAORMX had some comments on it in a previous PR.

[slyt3]

@slyt3 Looks good, but you'll also need to bump the helm chart under ./deploy/charts/operator-crds/Chart.yaml

After doing that, you may need to run task docs to update helm chart documentation.

done :)

[slyt3]

Uh, I had multiple Helm validation failures and CI compatibility issues,

Its really IMPORTANT, because this is significant stabilization changes. Its started with minor update, but it evolved to kinda huge.

its important because Pipeline unblocked after I upgraded cosign and CI caching to fix persistent release and CI failures. also removed hardcodede UIDs from helm charts. soo it will allow project to run on restrcited platforms like OpenShift.

[dmjb]

@slyt3 It looks like you included some changes to the github actions, and I see your comment about running into various problems. We haven't noticed any issues with the CI in our repo, were you trying to run them in your repo?

[slyt3]

@dmjb I will move the caching/CI changes to the separte branch and will open new PR for those, I hope i rebased it correctly

[dmjb]

@slyt3 Thanks for splitting the PR, I'll merge this in once the build is green.