fix: replace context.TODO with timeout context in config dump

[jaffarkeikei]

Replaces context.TODO() with context.WithTimeout() to enable proper cancellation and timeouts.

Before:

pods, err := listPods(context.TODO(), client, cd.Namespace, ...)

After:

ctx, cancel := context.WithTimeout(context.Background(), 30*time.Second)
defer cancel()
pods, err := listPods(ctx, client, cd.Namespace, ...)

Benefits:

• Prevents indefinite hangs when Kubernetes API is slow
• Enables proper cancellation
• Follows Go best practices for context propagation

Fixes #8121

[netlify]

✅ Deploy Preview for cerulean-figolla-1f9435 ready!

Name	Link
🔨 Latest commit	0cf2ac3
🔍 Latest deploy log	https://app.netlify.com/projects/cerulean-figolla-1f9435/deploys/697b73b7d982b90008c3b9a6
😎 Deploy Preview	https://deploy-preview-8122--cerulean-figolla-1f9435.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[zirain]

I would prefer this(30s) be configurable with a default value of 30s.

[jaffarkeikei]

Good point! I've updated the implementation to make the timeout configurable:

• Added a Timeout field to the ConfigDump struct
• Created a DefaultConfigDumpTimeout constant set to 30s
• Added a getTimeout() helper that returns the configured timeout or falls back to the default

This way, users can customize the timeout when needed, while maintaining the sensible 30s default. Thanks for the review!

[zirain]

Good point! I've updated the implementation to make the timeout configurable:

• Added a Timeout field to the ConfigDump struct
• Created a DefaultConfigDumpTimeout constant set to 30s
• Added a getTimeout() helper that returns the configured timeout or falls back to the default

This way, users can customize the timeout when needed, while maintaining the sensible 30s default. Thanks for the review!

Thanks for your patient, the next step is making it configurable as part of the args in egctl x collect.

[zirain]

BTW, please fix DCO.

[jaffarkeikei]

DCO fixed! Added sign-off to all commits. ✅

[jaffarkeikei]

Hi @zirain - Both commits have the Signed-off-by trailer:

Commit 1 (5bab0f7):

Signed-off-by: jaffar <keikei.jaffar@mail.utoronto.ca>

Commit 2 (b88453f):

Signed-off-by: jaffar <keikei.jaffar@mail.utoronto.ca>

You can verify by checking the commits directly:

• 5bab0f7
• b88453f

The DCO should pass now. Perhaps the DCO bot needs to re-check? ✅

[codecov]

Codecov Report

❌ Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.65%. Comparing base (42522ff) to head (0cf2ac3).
⚠️ Report is 23 commits behind head on main.

Files with missing lines	Patch %	Lines
internal/troubleshoot/collect/config_dump.go	57.14%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8122      +/-   ##
==========================================
- Coverage   73.65%   73.65%   -0.01%     
==========================================
  Files         239      240       +1     
  Lines       36311    36467     +156     
==========================================
+ Hits        26745    26859     +114     
- Misses       7670     7703      +33     
- Partials     1896     1905       +9     

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

[zirain]

This's good enough for first contributor.