Skip to content

ci: skip stress suites per PR#28

Merged
hev merged 2 commits into
mainfrom
ci/skip-stress-per-pr
Jul 8, 2026
Merged

ci: skip stress suites per PR#28
hev merged 2 commits into
mainfrom
ci/skip-stress-per-pr

Conversation

@hevbot

@hevbot hevbot commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Renames all repeat-count concurrent-write suites in lance_concurrent_writes.rs to durable _stress_ names.
  • Changes per-PR MinIO integration CI from --skip _100_runs_ to --skip _stress_, covering the old 50/100/200-run suites.
  • Adds a scheduled/manual Stress workflow that runs only MinIO stress tests with _stress_minio.

Closes #27

Evidence

Before rename, repeat-count suites were:

concurrent_writers_100_runs_minio
concurrent_writers_100_runs_aws
concurrent_writers_100_runs_r2
concurrent_writers_100_runs_tigris
concurrent_writers_100_runs_b2
concurrent_writers_100_runs_spaces
concurrent_writers_100_runs_gcs
concurrent_writers_100_runs_gcs_native
concurrent_writes_during_compaction_50_runs_minio
concurrent_writes_during_compaction_aggressive_200_runs_minio

After rename/list:

concurrent_writers_stress_aws
concurrent_writers_stress_b2
concurrent_writers_stress_gcs
concurrent_writers_stress_gcs_native
concurrent_writers_stress_minio
concurrent_writers_stress_r2
concurrent_writers_stress_spaces
concurrent_writers_stress_tigris
concurrent_writes_during_compaction_aggressive_stress_minio
concurrent_writes_during_compaction_stress_minio

Nightly MinIO stress selector proof:

./scripts/cargo test -p hevsearch-core --test lance_concurrent_writes _stress_minio -- --list
concurrent_writers_stress_minio: test
concurrent_writes_during_compaction_aggressive_stress_minio: test
concurrent_writes_during_compaction_stress_minio: test

Focused PR-lane filter proof:

./scripts/cargo test -p hevsearch-core --test lance_concurrent_writes -- --ignored --skip _aws --skip _stress_
running 9 tests
test result: ok. 9 passed; 0 failed; 0 ignored; 0 measured; 13 filtered out; finished in 2.64s
real 188.85

Full workspace integration command attempted twice locally against localhost MinIO using the new skip:

./scripts/cargo test --workspace --no-fail-fast -- --ignored --skip _aws --skip _stress_

Both local attempts failed before test execution in the linker with collect2: fatal error: ld terminated with signal 9 [Killed] while linking object_cache_s3_smoke, including a retry with CARGO_BUILD_JOBS=1 and stripped debug. The focused lance_concurrent_writes command above validates the selection behavior without linking every workspace integration binary.

YAML parse check:

ruby -e 'require "yaml"; ARGV.each { |f| YAML.load_file(f); puts "ok #{f}" }' .github/workflows/ci.yml .github/workflows/stress.yml
ok .github/workflows/ci.yml
ok .github/workflows/stress.yml

Depot runners

Skipped runner-label changes. Repo-local evidence only shows Depot build usage in docker-publish.yml (depot/setup-action and depot/build-push-action with project t4vlld595v); there is no depot.json or checked-in org/repo runner registration proving runs-on: depot-ubuntu-latest is available. Using Depot GitHub Actions runners still needs org/repo registration by hev before runs-on can safely change.

@hevbot hevbot added the gate:impl Implementation PR awaiting CI + spec-review + merge (agentic-gates §2) label Jul 8, 2026
@hevbot

hevbot commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator Author

Factory verified: durable _stress_ rename across all repeat-count suites (50/100/200 × all backends), per-PR lane skips them, dedicated scheduled Stress workflow keeps coverage on MinIO. Files scoped to the two workflows + the one test file. Merge-ready — every PR lane after this merges drops from ~35m to ~8m.

@hev hev merged commit b5ef9be into main Jul 8, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gate:impl Implementation PR awaiting CI + spec-review + merge (agentic-gates §2)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI takes ~35 min: _200_runs_ stress test evades the _100_runs_ skip filter

2 participants