Skip to content

Fix musl error assertion#1653

Merged
ludfjig merged 1 commit into
hyperlight-dev:mainfrom
ludfjig:fix_musl_symbol_error
Jul 18, 2026
Merged

Fix musl error assertion#1653
ludfjig merged 1 commit into
hyperlight-dev:mainfrom
ludfjig:fix_musl_symbol_error

Conversation

@ludfjig

@ludfjig ludfjig commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

closes #1620

Copilot AI review requested due to automatic review settings July 17, 2026 21:24
@ludfjig
ludfjig force-pushed the fix_musl_symbol_error branch from eddcc9c to c111daa Compare July 17, 2026 21:25
@ludfjig ludfjig added kind/bugfix For PRs that fix bugs ready-for-review PR is ready for (re-)review labels Jul 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a flaky Linux musl nightly failure by making the snapshot symlink-rejection tests robust to libc-specific error wording differences.

Changes:

  • Update two unit tests to avoid depending on exact libc capitalization/pluralization in the symlink error message.

Comment thread src/hyperlight_host/src/sandbox/snapshot/file_tests.rs Outdated
Comment thread src/hyperlight_host/src/sandbox/snapshot/file_tests.rs Outdated
@ludfjig
ludfjig force-pushed the fix_musl_symbol_error branch from c111daa to 9856278 Compare July 17, 2026 21:35
Signed-off-by: Ludvig Liljenberg <4257730+ludfjig@users.noreply.github.com>
@ludfjig
ludfjig force-pushed the fix_musl_symbol_error branch from 9856278 to 7e820f3 Compare July 17, 2026 23:53
@ludfjig
ludfjig enabled auto-merge (squash) July 17, 2026 23:53
@ludfjig
ludfjig merged commit c96655a into hyperlight-dev:main Jul 18, 2026
41 checks passed
@github-actions github-actions Bot removed the ready-for-review PR is ready for (re-)review label Jul 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bugfix For PRs that fix bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Nightly musl Failure - 145

3 participants