Skip to content

Shard smoketests using nextest - #5705

Open
joshua-spacetime wants to merge 6 commits into
masterfrom
joshua/ci/shard-smoketests
Open

Shard smoketests using nextest#5705
joshua-spacetime wants to merge 6 commits into
masterfrom
joshua/ci/shard-smoketests

Conversation

@joshua-spacetime

Copy link
Copy Markdown
Collaborator

Description of Changes

Shards both the linux and windows smoketests using nextest. Each shard is executed independently from the others in its own job on its own runner. Shards can be re-run independently as well, and they don't have to rebuild anything. Not spacetimedb-standalone, not the smoketest modules, not even the tests themselves.

API and ABI breaking changes

N/A

Expected complexity level and risk

3

Testing

N/A

Comment thread crates/guard/src/lib.rs
}

/// Returns the workspace root directory.
// TODO: Should this use something like `git rev-parse --show-toplevel` to avoid being directory-relative? Or perhaps `CARGO_WORKSPACE_DIR` is set?

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.

?

Comment thread tools/ci/src/smoketest.rs
}

fn run_smoketest_archive(archive_file: &Path, args: Vec<String>) -> Result<()> {
let workspace_root = env::current_dir()?;

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.

nit: maybe we should call ensure_repo_root?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants