Skip to content
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

chore(tests): enable upgrade_tests again #18207

Closed
wants to merge 21 commits into from
Closed

chore(tests): enable upgrade_tests again #18207

wants to merge 21 commits into from

Conversation

GJZwiers
Copy link
Contributor

@GJZwiers GJZwiers commented Mar 15, 2023

Closes #7412

These tests would sometimes throw an ExecutableFileBusy error, most likely due to parallel std::fs::copy calls of the same file. This is an attempt to fix the issue by switching to std::fs::hard_link.

Copy link
Collaborator

@aapoalas aapoalas left a comment

Choose a reason for hiding this comment

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

LGTM. Let's hope this indeed does fix the flakiness.

@aapoalas aapoalas enabled auto-merge (squash) March 26, 2023 09:24
@GJZwiers
Copy link
Contributor Author

GJZwiers commented Mar 26, 2023

Looks like they are still flaky after all

@aapoalas
Copy link
Collaborator

Yup :(

@GJZwiers
Copy link
Contributor Author

@aapoalas @bartlomieju The upgrade_with_out_in_tmpdir test seems to have broken in 0742ea1 (which was not caught because it is ignored on main) rather than it being flaky, because locally it fails consistently for me from that commit onwards but not before that.

@bartlomieju
Copy link
Member

@GJZwiers good catch; should we revert that change or can it be fixed in-line in this PR?

@GJZwiers
Copy link
Contributor Author

@bartlomieju I think maybe it should be reverted, there was a regression in 0742ea1 because deno upgrade --output fails on the canary version same as the test:

cd /tmp
mkdir foo
deno upgrade --version 1.11.5 --output foo/deno
error: No such file or directory (os error 2)

But on 1.32.1 it works:

deno upgrade --version 1.11.5 --output foo/deno
Downloading https://github.com/denoland/deno/releases/download/v1.11.5/deno-x86_64-unknown-linux-gnu.zip
Deno is upgrading to version 1.11.5
Archive:  /tmp/.tmp2QmbJs/deno.zip
  inflating: deno                    
Upgraded successfully
Release notes: https://github.com/denoland/deno/releases/tag/v1.11.5
Blog post: https://deno.com/blog/v1.11

auto-merge was automatically disabled March 29, 2023 18:59

Head branch was pushed to by a user without write access

@GJZwiers
Copy link
Contributor Author

GJZwiers commented Apr 20, 2023

@bartlomieju I am getting test failures after pulling the latest main branch but only on Windows. It does not happen locally, was anything changed to the CI recently?

thread 'integration::upgrade::upgrade_with_version_in_tmpdir' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 17, kind: CrossesDevices, message: "The system cannot move the file to a different disk drive." }', cli\tests\integration\upgrade_tests.rs:51:56

@GJZwiers GJZwiers closed this Apr 26, 2023
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.

reenable upgrade tests
3 participants