Skip to content

[RTE-381] Pin to nightly-2025-02-22 compiler for some CI tests #713

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 11, 2025

Conversation

raoulstrackx
Copy link
Contributor

Compiling async-usercalls triggers an upstream compiler bug in nightly-2025-02-23 or subsequent compilers. This PR pins some CI tests to the nightly-2025-02-22 compiler until the bug is resolved upstream. A minimal PoC still needs to be created and a bug report still needs to be filed.

@Taowyoo
Copy link
Collaborator

Taowyoo commented Feb 26, 2025

Hi @raoulstrackx when talking about CI, do you mean the CI in our internal jenkins sgx CI?
Otherwise, the problem is already solved by this PR: #708

@s-arash
Copy link
Contributor

s-arash commented Feb 26, 2025

Yeah, since the CI is unblocked for now, let's not pin the rust toolchain version in CI scripts.

@raoulstrackx
Copy link
Contributor Author

Oh I missed that PR. In general I think it's better to pin to a working compiler. In the macro that #708 changed, we really meant that it needs to be a type, not a token tree. Once the upstream compiler is fixed, we probably will forget to revert that PR. Also note that if a stable compiler is ever produced with this bug, other CI tests that use it, will start failing too (e.g., the enclave-runner). Well, we should get this fixed before it gets that far.

@Taowyoo

This comment was marked as outdated.

Taowyoo
Taowyoo previously approved these changes Mar 11, 2025
@Taowyoo Taowyoo enabled auto-merge March 11, 2025 06:27
@raoulstrackx raoulstrackx force-pushed the raoul/rte-381-pin_rust_compiler_for_ci branch from b4f09a8 to 8f184be Compare March 11, 2025 08:47
@Taowyoo Taowyoo added this pull request to the merge queue Mar 11, 2025
Merged via the queue into master with commit 76c0e9f Mar 11, 2025
2 checks passed
@Taowyoo Taowyoo deleted the raoul/rte-381-pin_rust_compiler_for_ci branch March 11, 2025 17:16
@javierron javierron mentioned this pull request Mar 24, 2025
6 tasks
@raoulstrackx
Copy link
Contributor Author

Filed issue #138975 upstream

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.

3 participants