Skip to content

[CI] Do not fail test test_upgrade_from_local_checkpoint if Rust cannot be built; it may be impossible to build with the previous version of the libraries#6706

Queued
mihaibudiu wants to merge 1 commit into
feldera:mainfrom
mihaibudiu:issue6705

Conversation

@mihaibudiu

Copy link
Copy Markdown
Contributor

Fixes #6705

  • Integration tests added/updated

@mihaibudiu
mihaibudiu enabled auto-merge July 23, 2026 00:58
@mihaibudiu
mihaibudiu requested review from blp and gz July 23, 2026 00:59
@mihaibudiu mihaibudiu changed the title [CI] Do not fail test test_upgrade_from_local_checkpoint if Rust cannot be built; it may be impossible to build with the previous version of the libraries [CI] Do not fail test test_upgrade_from_local_checkpoint if Rust cannot be built; it may be impossible to build with the previous version of the libraries Jul 23, 2026
@mihaibudiu
mihaibudiu added this pull request to the merge queue Jul 23, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 23, 2026
…ot be built; it may be impossible to build with the previous version of the libraries

Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
@mihaibudiu
mihaibudiu enabled auto-merge July 23, 2026 01:54
@mihaibudiu
mihaibudiu added this pull request to the merge queue Jul 23, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 23, 2026
@gz
gz added this pull request to the merge queue Jul 23, 2026

@mythical-fred mythical-fred left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM. Reasonable escape hatch — old runtime images can drop off the network / become unbuildable, and hard-failing the whole test suite over that would just create noise. warnings.warn keeps the skip visible in CI output. "RustError" string match is loose but consistent with how the rest of test_checkpoint_runtime_upgrade.py inspects RuntimeError text.

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.

[CI] Test test_checkpoint_runtime_upgrade.py will always fail if the software cannot be built with the previous version of Feldera

3 participants