Skip to content

Add behavioral test to serving-llms-on-instinct - #99

Merged
danielholanda merged 9 commits into
mainfrom
dholanda/instinct_test_t2
Aug 7, 2026
Merged

Add behavioral test to serving-llms-on-instinct#99
danielholanda merged 9 commits into
mainfrom
dholanda/instinct_test_t2

Conversation

@danielholanda

@danielholanda danielholanda commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Enable tests to run on MI300 machine

@danielholanda
danielholanda temporarily deployed to behavioral-instinct August 4, 2026 23:18 — with GitHub Actions Inactive
@danielholanda danielholanda changed the title serving-llms-on-instinct test [DRAFT] serving-llms-on-instinct test Aug 5, 2026
@danielholanda
danielholanda marked this pull request as ready for review August 5, 2026 23:28
@Mahdi-CV
Mahdi-CV requested a review from archakamk August 6, 2026 22:34

@archakamk archakamk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Couple things to consider:

  • This test makes 1 agent run + 5 LLM-judge calls per run all on Opus. Can we run the behavioral test on Sonnet? A weaker model could be a better test of whether the skill instructions are unambiguous. Or at minimum, can the LLM judge use Sonnet?

  • _cleanup_test_containers runs in the test's finally, which might not fire if the 45-minute timeout fires and GitHub cancels the job. It could potentially leak a container. Could maybe add an if: always() cleanup step.

@archakamk archakamk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approving so teams can start working on tests. Will create PR to handle issues previously raised.

@danielholanda
danielholanda deployed to behavioral-instinct August 7, 2026 04:57 — with GitHub Actions Active
@danielholanda danielholanda changed the title serving-llms-on-instinct test Add behavioral test to serving-llms-on-instinct Aug 7, 2026
@danielholanda
danielholanda merged commit 8d5332c into main Aug 7, 2026
21 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants