Skip to content

Exclude sheepdog in nextest runs#1631

Merged
blt merged 2 commits intomainfrom
blt/exclude_sheepdog_in_nextest_runs
Dec 5, 2025
Merged

Exclude sheepdog in nextest runs#1631
blt merged 2 commits intomainfrom
blt/exclude_sheepdog_in_nextest_runs

Conversation

@blt
Copy link
Collaborator

@blt blt commented Dec 4, 2025

What does this PR do?

In PR #1590 I attempted to move sheepdog to CI-only integration
tests. However I failed to realize that sheepdog tests would still run
as a part of nextest runs because of the way workspaces are arranged
in this project. This commit should make local ci/validate loops
much faster.

In PR #1590 I attempted to move sheepdog to CI-only integration
tests. However I failed to realize that sheepdog tests would still run
as a part of nextest runs because of the way workspaces are arranged
in this project. This commit should make local `ci/validate` loops
much faster.

Signed-off-by: Brian L. Troutwine <brian.troutwine@datadoghq.com>
@blt blt added the no-changelog label Dec 4, 2025 — with Graphite App
Copy link
Collaborator Author

blt commented Dec 4, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@blt blt marked this pull request as ready for review December 4, 2025 23:20
@blt blt requested a review from a team as a code owner December 4, 2025 23:20
Signed-off-by: Brian L. Troutwine <brian.troutwine@datadoghq.com>
@blt blt merged commit 122119b into main Dec 5, 2025
29 checks passed
Copy link
Collaborator Author

blt commented Dec 5, 2025

Merge activity

  • Dec 5, 1:55 PM UTC: @blt merged this pull request with Graphite.

@blt blt deleted the blt/exclude_sheepdog_in_nextest_runs branch December 5, 2025 13:55
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