add new configmap flag in proxy runner to read config from there

[yrobla]

Initially it will read the configmap and validate that it exists

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 42.08%. Comparing base (0392dcf) to head (6d70dd1).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1789      +/-   ##
==========================================
+ Coverage   41.78%   42.08%   +0.30%     
==========================================
  Files         184      184              
  Lines       21533    21649     +116     
==========================================
+ Hits         8997     9111     +114     
- Misses      11841    11844       +3     
+ Partials      695      694       -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.

[coveralls]

coverage: 39.61% (+0.6%) from 39.043%
when pulling 6d70dd1 on issue-1638-add-configmap-flag
into 7e74099 on main.

[Copilot]

Pull Request Overview

This PR introduces a new readFromConfigMap flag to the MCPServer CRD that enables the proxy runner to read configuration from a Kubernetes ConfigMap instead of individual command-line flags. The implementation adds validation to ensure the ConfigMap exists and contains valid configuration data.

Key changes:

• Adds readFromConfigMap boolean field to MCPServer spec with operator logic to use --from-configmap flag
• Implements ConfigMap validation in proxy runner to verify existence and parse runconfig.json content
• Updates RBAC permissions to allow ConfigMap access and adds comprehensive end-to-end tests

Reviewed Changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
cmd/thv-operator/api/v1alpha1/mcpserver_types.go	Adds readFromConfigMap field to MCPServer spec
cmd/thv-operator/controllers/mcpserver_controller.go	Implements conditional logic to use ConfigMap flag vs individual flags
cmd/thv-proxyrunner/app/run.go	Adds --from-configmap flag with validation and ConfigMap identification logic
cmd/thv-proxyrunner/app/run_test.go	Comprehensive test coverage for new ConfigMap functionality
deploy/charts/operator-crds/templates/toolhive.stacklok.dev_mcpservers.yaml	Updates CRD schema to include new field
test/e2e/chainsaw/operator/single-tenancy/test-scenarios/configmap-mode/	Complete e2e test suite for ConfigMap mode
test/e2e/chainsaw/operator/*/test-scenarios/common/proxyrunner-role.yaml	Adds ConfigMap RBAC permissions

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}