Skip to content

Add debug-ci skill for investigating E2E test failures#840

Open
mandre wants to merge 1 commit into
k-orc:mainfrom
shiftstack:add-debug-ci-skill
Open

Add debug-ci skill for investigating E2E test failures#840
mandre wants to merge 1 commit into
k-orc:mainfrom
shiftstack:add-debug-ci-skill

Conversation

@mandre

@mandre mandre commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot added the semver:patch No API change label Jul 7, 2026
@mandre
mandre force-pushed the add-debug-ci-skill branch from 99f7727 to 57d5b32 Compare July 8, 2026 06:06

@eshulman2 eshulman2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Added some minor improvement suggestions overall looks good

Comment thread .agents/skills/debug-ci/SKILL.md Outdated

### 3a. Understand the Test

KUTTL tests live in `internal/controllers/<resource>/tests/<test-name>/`. Each step has:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

it usually also have a readme.md with it describing it with additional context maybe it would be nice to add it here as well to provide the agent with more context

| **OpenStack bug** | Neutron/Nova/etc. returned success but resource is broken | File upstream or document as known flake |
| **ORC bug** | Controller logic error, race condition, or incorrect error handling | File ORC issue with fix plan |
| **Test bug** | Wrong credentials, wrong assertions, missing dependencies | Fix the test |
| **Infrastructure flake** | Timeout, resource exhaustion, network issue | Document and consider retry logic |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

from personal experience agents tend to over chose this options when unsure, I would suggest some reflection instructions here to make sure the model have sufficient evidence to conclude this and that other hypothesis proven false

@github-actions github-actions Bot added semver:minor Backwards-compatible change and removed semver:patch No API change labels Jul 21, 2026
@mandre
mandre force-pushed the add-debug-ci-skill branch from 04be18d to bf8c7c9 Compare July 21, 2026 06:02
@github-actions github-actions Bot added semver:patch No API change and removed semver:minor Backwards-compatible change labels Jul 21, 2026
@mandre
mandre force-pushed the add-debug-ci-skill branch from bf8c7c9 to afaf45a Compare July 21, 2026 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:patch No API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants