refactor: Replace Telepresence with ko --debug by Arpit529Srivastava · Pull Request #8869 · knative/eventing

Arpit529Srivastava · 2026-02-01T21:36:46Z

Proposed Changes

🧹 Replace Telepresence debugging documentation with ko --debug approach
🧹 Add instructions for VSCode and IntelliJ/GoLand remote debugging via Delve
🧹 Add troubleshooting section for common debugging issues

Pre-review Checklist

At least 80% unit test coverage
E2E tests for any new behavior
Docs PR for any user-facing impact; docs update itself (DEVELOPMENT.md)
Spec PR for any new API feature
Conformance test for any change to the spec

Release Note

updated `DEVELOPMENT.md` to use ko’s built-in `--debug` flag for remote debugging instead of telepresence. this simplifies the debugging workflow by using delve directly and avoids introducing extra dependencies.

Docs
this pr updates the developer documentation (DEVELOPMENT.md) in this repository. no external documentation changes are needed since this only affects the development workflow and not any user-facing features.

Signed-off-by: arpit529srivastava <arpitsrivastava529@gmail.com>

knative-prow · 2026-02-01T21:36:56Z

Welcome @Arpit529Srivastava! It looks like this is your first PR to knative/eventing 🎉

knative-prow · 2026-02-01T21:36:57Z

Hi @Arpit529Srivastava. 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.

Arpit529Srivastava · 2026-02-01T21:38:55Z

/cc @creydr PTAL,
thanks.

creydr

Thanks a lot @Arpit529Srivastava for improving this.
I left two comments

creydr · 2026-02-02T10:40:35Z

 ```

-## Debugging Knative controllers and friends locally
+## Debugging Knative controllers and webhooks


Suggested change

## Debugging Knative controllers and webhooks

## Debugging Knative controllers and friends locally

I'd keep the initial version here, because everything what is build with ko in knative can be debugged

i see, made the changes :)

creydr · 2026-02-02T10:42:20Z

+{
+  "version": "0.2.0",
+  "configurations": [
+    {
+      "name": "Connect to Eventing Controller",
+      "type": "go",
+      "request": "attach",
+      "mode": "remote",
+      "port": 40000,
+      "host": "127.0.0.1",
+      "substitutePath": [
+        {
+          "from": "${workspaceFolder}",
+          "to": "/go/src/knative.dev/eventing"
+        }
+      ]
+    }
+  ]
+}


@twoGiants as you're working a lot with VSCode: is all of this needed to use ko debug?

the substitutePath is there to map the local workspace path to the go module path inside the container, so vscode can correctly resolve source locations for breakpoints. that said, i'll wait for @twoGiants's review.

I just checked my old config and I simply used

"cwd": "${workspaceFolder}"

instead of the substitutePath. That way we don't need the substitution I guess

(and "host": "0.0.0.0",)

made the requested changes 👍🏻

Arpit529Srivastava · 2026-02-02T16:10:48Z

Thanks a lot @Arpit529Srivastava for improving this. I left two comments

addressed, thanks for the review :)

Arpit529Srivastava · 2026-02-09T09:52:12Z

@creydr a gentle ping, thanks :)

Signed-off-by: arpit529srivastava <arpitsrivastava529@gmail.com>

Arpit529Srivastava · 2026-02-09T12:31:17Z

@creydr PTAL, when you get time, thanks

creydr · 2026-02-09T12:43:38Z

/ok-to-test

codecov · 2026-02-09T13:07:45Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.63%. Comparing base (0ef7bc1) to head (00d9ac0).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8869      +/-   ##
==========================================
- Coverage   51.11%   50.63%   -0.48%     
==========================================
  Files         409      409              
  Lines       21381    21658     +277     
==========================================
+ Hits        10928    10966      +38     
- Misses       9602     9835     +233     
- Partials      851      857       +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.

Arpit529Srivastava · 2026-02-09T13:46:49Z

@creydr should i update the branch to latest commit?

creydr

Thanks a lot for working on this @Arpit529Srivastava
/lgtm

knative-prow · 2026-02-09T14:00:40Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Arpit529Srivastava, creydr

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~~ [creydr]

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

Arpit529Srivastava · 2026-02-09T14:13:44Z

Thanks a lot for working on this @Arpit529Srivastava /lgtm

thanks for reviewing :)

refactor: Replace Telepresence with ko --debug

b6d2a11

Signed-off-by: arpit529srivastava <arpitsrivastava529@gmail.com>

knative-prow Bot added needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 1, 2026

knative-prow Bot requested review from Cali0707 and Leo6Leo February 1, 2026 21:37

creydr reviewed Feb 2, 2026

View reviewed changes

Arpit529Srivastava requested a review from creydr February 2, 2026 16:08

docs: rename debugging section

00d9ac0

Signed-off-by: arpit529srivastava <arpitsrivastava529@gmail.com>

Arpit529Srivastava force-pushed the debug-flag branch from e44c38c to 00d9ac0 Compare February 9, 2026 12:30

knative-prow Bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Feb 9, 2026

creydr approved these changes Feb 9, 2026

View reviewed changes

knative-prow Bot assigned creydr Feb 9, 2026

knative-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Feb 9, 2026

knative-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 9, 2026

knative-prow Bot merged commit 9e76da0 into knative:main Feb 9, 2026
35 of 36 checks passed

Arpit529Srivastava deleted the debug-flag branch February 9, 2026 14:13

	## Debugging Knative controllers and webhooks
	## Debugging Knative controllers and friends locally

Uh oh!

Conversation

Arpit529Srivastava commented Feb 1, 2026

Proposed Changes

Pre-review Checklist

Uh oh!

knative-prow Bot commented Feb 1, 2026

Uh oh!

knative-prow Bot commented Feb 1, 2026

Uh oh!

Arpit529Srivastava commented Feb 1, 2026

Uh oh!

creydr left a comment

Choose a reason for hiding this comment

Uh oh!

creydr Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Arpit529Srivastava Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

creydr Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Arpit529Srivastava Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

creydr Feb 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Arpit529Srivastava Feb 9, 2026

Choose a reason for hiding this comment

Uh oh!

Arpit529Srivastava commented Feb 2, 2026

Uh oh!

Arpit529Srivastava commented Feb 9, 2026

Uh oh!

Arpit529Srivastava commented Feb 9, 2026

Uh oh!

creydr commented Feb 9, 2026

Uh oh!

codecov Bot commented Feb 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Arpit529Srivastava commented Feb 9, 2026

Uh oh!

creydr left a comment

Choose a reason for hiding this comment

Uh oh!

knative-prow Bot commented Feb 9, 2026

Uh oh!

Uh oh!

Arpit529Srivastava commented Feb 9, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

creydr Feb 9, 2026 •

edited

Loading

codecov Bot commented Feb 9, 2026 •

edited

Loading