Skip to content

.agents/skills: add cl2-artifacts-reference guide - #4100

Open
p0lyn0mial wants to merge 1 commit into
kubernetes:masterfrom
p0lyn0mial:cl2-artifacts-reference-v1
Open

p0lyn0mial wants to merge 1 commit into
kubernetes:masterfrom
p0lyn0mial:cl2-artifacts-reference-v1

Conversation

@p0lyn0mial

Copy link
Copy Markdown
Contributor

What type of PR is this?

/kind documentation

What this PR does / why we need it:

Add a reference guide for ClusterLoader2 CI artifacts as a skill.
Note that is not a skill per se. It is rather documentation that describes the artifact directory structure.

Starts with two sections but more sections can be added in the future PRs.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 11, 2026
@p0lyn0mial

Copy link
Copy Markdown
Contributor Author

@serathius @dims @Jefftree please let me know your thoughts on this PR.

@@ -0,0 +1,22 @@
---
name: cl2-artifacts-reference
description: Reference guide for ClusterLoader2 CI artifacts and their GCS paths.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could be the place where the domain knowledge is defined.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's doesn't sound like skill rather reference

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Use this reference to understand what each file is and where it lives
in the artifact tree.

# Reference

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should be more specific about what files can be found in the directory structure.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, I feel that a lot of these descriptions are evident from the name. The specific nuanced things are the worthwhile things that an agent can't easily derive from directory name alone.

@k8s-ci-robot k8s-ci-robot added the kind/documentation Categorizes issue or PR as related to documentation. label Jun 11, 2026
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: p0lyn0mial
Once this PR has been reviewed and has the lgtm label, please assign mengqiy 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:

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

@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 11, 2026
@serathius

Copy link
Copy Markdown
Member

/cc @seans3

@k8s-ci-robot
k8s-ci-robot requested a review from seans3 June 11, 2026 18:40
## 1. Artifact Directory Structure
- **Root**: `artifacts/` under the CI job run's output path.
- **Subdirectories**:
- `control-plane-*/` — control-plane node logs.

@Jefftree Jefftree Jun 11, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Realistically, would an agent really need these kinds of descriptions to navigate? (this is a question, not asking you to change)

Use this reference to understand what each file is and where it lives
in the artifact tree.

# Reference

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, I feel that a lot of these descriptions are evident from the name. The specific nuanced things are the worthwhile things that an agent can't easily derive from directory name alone.

@@ -0,0 +1,22 @@
---
name: cl2-artifacts-reference
description: Reference guide for ClusterLoader2 CI artifacts and their GCS paths.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/documentation Categorizes issue or PR as related to documentation. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants