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

tests: test rclone #307

Draft
wants to merge 1 commit into
base: zfs-Windows-2.2.0-release
Choose a base branch
from

Conversation

EchterAgo
Copy link

I'm only going to make this a draft for now, but testing like this produces all kinds of results. Crashes my VM in a very short time consistently.

@EchterAgo EchterAgo force-pushed the tests_rclone branch 2 times, most recently from 6d44720 to ce135a9 Compare October 24, 2023 01:20
@andrewc12
Copy link

rclone sync %WINDIR% Z:\copy.bin

How does this work exactly? copy.bin is a folder?

@EchterAgo
Copy link
Author

You're right, I believe I forgot to change this, but rclone seems to create this? I'll have to check tomorrow, but it did copy and work for me.

@EchterAgo
Copy link
Author

When I first created it I tried with just a single file copy but it worked just fine. Maybe things only start to go wrong when rclone copies things in parallel.

@lundman
Copy link

lundman commented Oct 24, 2023

I can confirm this brings the VM to a halt surprisingly quickly.

@EchterAgo
Copy link
Author

I can confirm this brings the VM to a halt surprisingly quickly.

Good to know it is not just my machine. Would adding a test like this make sense? We could use chocolatey to install rclone on the runner machine and make it copy from a set of files made by the script instead of WINDIR.

@andrewc12
Copy link

I don't think this is a good test to run automatically since BSODs don't play nice with github runners.
(Unless you have a self hosted runner I guess.)

But we could turn it into a script to test regressions, or even just to run manually on a dev VM.

Signed-off-by: Axel Gembe <[email protected]>
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