feat: shp buildrun gather subcommand to extract the details of the failed BuildRun

[kaizakin]

Changes

• added dynamic client support in params to fetch taskrun objects
• added buildrun gather command that collects buildrun taskrun pod Objects and individual pod logs
• --output flag determines the output location, default value is . currect directory.
• --archive flag determines the output to be compressed as .tar.gz

Edit:

• gather now supports PipelineRun executor as well
• added options helper for dynamicClient and extended WithClientset NewParamsForTest to support dynamicClient
• added unit tests

Fixes #250

---

Type of PR

/kind feature

Submitter Checklist

• Includes tests if functionality changed/was added
• Includes docs if changes are user-facing
• Kind label has been set
• Release notes block has been filled in, or marked NONE

TaskRun

2026-03-22.20-23-03.mp4

PipelineRun

2026-03-31.02-28-01.mp4

Release Notes

Add "shp buildrun gather" subcommand

[openshift-ci]

@kaizakin: The label(s) kind/your-label-here cannot be applied, because the repository doesn't have them.

Details

In response to this:

Changes

Fixes #250

Related Issue

Fixes #

Type of PR

/kind your-label-here

Submitter Checklist

• Includes tests if functionality changed/was added
• Includes docs if changes are user-facing
• Kind label has been set
• Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.

Release Notes

your release note here

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.

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign saschaschwarze0 for approval. For more information see the Code Review Process.

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

Details

Needs approval from an approver in each of these files:

• OWNERS

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

[Copilot]

Pull request overview

Adds a new shp buildrun gather subcommand to collect BuildRun diagnostics (BuildRun/TaskRun/Pod objects + pod logs) into a directory or .tar.gz archive, to help debug failed BuildRuns (Fixes #250).

Changes:

• Add a dynamic.Interface client to CLI params for fetching Tekton objects.
• Introduce shp buildrun gather subcommand with --output and --archive flags to export YAMLs and logs.
• Add generated CLI docs for the new subcommand and link it from shp buildrun docs.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
pkg/shp/params/params.go	Adds dynamic client support for retrieving Tekton resources (TaskRuns).
pkg/shp/cmd/buildrun/gather.go	Implements the new gather subcommand, YAML/log collection, and optional tar.gz archiving.
pkg/shp/cmd/buildrun/buildrun.go	Registers gather under the buildrun command group.
docs/shp_buildrun_gather.md	Adds generated documentation for shp buildrun gather.
docs/shp_buildrun.md	Links the new gather subcommand in the BuildRun command index.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[IrvingMg]

Thanks for the PR @kaizakin!

I have a few suggestions for the first pass:

1. Please address Copilot’s review comments.
2. Fix the linter issues reported by CI.
3. Feel free to squash the commits as well.

Regarding your questions:

1. IIUC, supporting PipelineRun would add more complexity. So, in my opinion, it would be better to keep this PR focused on BuildRun, but I’d like to hear others’ opinions.
2. It would be nice to have unit tests, at least for basic scenarios.

[kaizakin]

requesting review @IrvingMg

[kaizakin]

Thanks for the review @IrvingMg
suggested changes has been made 😁