mcp: adds labels and envs config tools and resources #2931

KapilSareen · 2025-07-09T19:20:16Z

Changes

This PR adds the remainder of config commands (labels and envs) to the MCP server

Related Epic #2806

knative-prow · 2025-07-09T19:20:26Z

Hi @KapilSareen. Thanks for your PR.

I'm waiting for a knative member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

codecov · 2025-07-09T19:28:12Z

Codecov Report

Attention: Patch coverage is 0% with 159 lines in your changes missing coverage. Please review.

Project coverage is 60.15%. Comparing base (f096936) to head (10f33f4).
Report is 7 commits behind head on main.

Files with missing lines	Patch %	Lines
pkg/mcp/tools.go	0.00%	80 Missing ⚠️
pkg/mcp/mcp.go	0.00%	74 Missing ⚠️
pkg/mcp/help.go	0.00%	5 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2931      +/-   ##
==========================================
+ Coverage   59.97%   60.15%   +0.18%     
==========================================
  Files         132      132              
  Lines       16626    16782     +156     
==========================================
+ Hits         9971    10096     +125     
- Misses       5728     5730       +2     
- Partials      927      956      +29

Flag	Coverage Δ
e2e-tests	`41.35% <0.00%> (+0.97%)`	⬆️
integration-tests	`54.87% <0.00%> (+1.20%)`	⬆️
unit-tests	`47.03% <0.00%> (-0.45%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

lkingland · 2025-07-10T00:34:11Z

Looks good, remove from draft status to trigger auto-merge

Signed-off-by: kapil <[email protected]>

lkingland · 2025-07-14T10:30:05Z

/approve 🤞tests pass

knative-prow · 2025-07-14T10:30:11Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: KapilSareen, lkingland

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~OWNERS~~ [lkingland]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

lkingland · 2025-07-24T01:10:14Z

/lgtm

Signed-off-by: kapil <[email protected]>

* mcp: adds func mcp command (knative#2836) Signed-off-by: KapilSareen <[email protected]> * mcp: adds create and deploy tools (knative#2859) Signed-off-by: kapil <[email protected]> * mcp: adds list tool (knative#2861) Signed-off-by: kapil <[email protected]> * mcp: adds build tool (knative#2865) Signed-off-by: kapil <[email protected]> * mcp: adds resource provider and prompt for root cmd docs (knative#2875) Signed-off-by: kapil <[email protected]> * mcp: adds delete tool and adds remote flag to deploy tool (knative#2863) * rebases * adds remote flag to deploy tool Signed-off-by: kapil <[email protected]> --------- Signed-off-by: kapil <[email protected]> * mcp: extends flags support for existing tools (knative#2889) Signed-off-by: kapil <[email protected]> * mcp: adds resources and prompts for func subcommands (knative#2890) Signed-off-by: kapil <[email protected]> * mcp: adds tool and resource for config volumes (knative#2925) Signed-off-by: kapil <[email protected]> * mcp: refactors pkg for easy maintainability (knative#2928) Signed-off-by: kapil <[email protected]> * mcp: adds labels and envs config tools and resources (knative#2931) Signed-off-by: kapil <[email protected]> * mcp: adds remote template support (knative#2951) Signed-off-by: kapil <[email protected]> --------- Signed-off-by: KapilSareen <[email protected]> Signed-off-by: kapil <[email protected]> Co-authored-by: Kapil Sareen <[email protected]>

knative-prow bot requested review from jrangelramos and nainaz July 9, 2025 19:20

knative-prow bot added needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Jul 9, 2025

KapilSareen marked this pull request as draft July 9, 2025 19:20

knative-prow bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 9, 2025

KapilSareen force-pushed the mcp-10 branch from c508729 to 016917a Compare July 9, 2025 19:21

lkingland approved these changes Jul 10, 2025

View reviewed changes

knative-prow bot assigned lkingland Jul 10, 2025

knative-prow bot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jul 10, 2025

mcp: adds labels and envs config tools and resources

10f33f4

Signed-off-by: kapil <[email protected]>

KapilSareen force-pushed the mcp-10 branch from 016917a to 10f33f4 Compare July 14, 2025 09:44

knative-prow bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Jul 14, 2025

KapilSareen marked this pull request as ready for review July 14, 2025 09:45

knative-prow bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 14, 2025

knative-prow bot requested a review from dsimansk July 14, 2025 09:45

knative-prow bot added the lgtm Indicates that a PR is ready to be merged. label Jul 24, 2025

knative-prow bot merged commit 0157893 into knative:main Jul 24, 2025
52 of 56 checks passed

lkingland pushed a commit to lkingland/func that referenced this pull request Jul 24, 2025

mcp: adds labels and envs config tools and resources (knative#2931)

50e4a61

Signed-off-by: kapil <[email protected]>

matejvasek pushed a commit to openshift-knative/kn-plugin-func that referenced this pull request Oct 6, 2025

mcp: adds labels and envs config tools and resources (knative#2931)

9ca89fe

Signed-off-by: kapil <[email protected]>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

mcp: adds labels and envs config tools and resources #2931

mcp: adds labels and envs config tools and resources #2931

Uh oh!

KapilSareen commented Jul 9, 2025

Uh oh!

knative-prow bot commented Jul 9, 2025

Uh oh!

codecov bot commented Jul 9, 2025 •

edited

Loading

Uh oh!

lkingland commented Jul 10, 2025

Uh oh!

lkingland commented Jul 14, 2025

Uh oh!

knative-prow bot commented Jul 14, 2025

Uh oh!

lkingland commented Jul 24, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

mcp: adds labels and envs config tools and resources #2931

mcp: adds labels and envs config tools and resources #2931

Uh oh!

Conversation

KapilSareen commented Jul 9, 2025

Changes

Uh oh!

knative-prow bot commented Jul 9, 2025

Uh oh!

codecov bot commented Jul 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

lkingland commented Jul 10, 2025

Uh oh!

lkingland commented Jul 14, 2025

Uh oh!

knative-prow bot commented Jul 14, 2025

Uh oh!

lkingland commented Jul 24, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

codecov bot commented Jul 9, 2025 •

edited

Loading