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

[BUG] Sync wont start when there sync diff is zero but sync incompleted #109

Open
tyjtyj opened this issue Jun 29, 2024 · 7 comments
Open
Labels
enhancement New feature or request wait for feedback

Comments

@tyjtyj
Copy link

tyjtyj commented Jun 29, 2024

The system crashed last sync this the sync did not completed

### SnapRAID TOUCH [Sat Jun 29 10:07:01 AM +08 2024]
Checking for zero sub-second files.
No zero sub-second timestamp files found.
TOUCH finished [Sat Jun 29 10:07:07 AM +08 2024]

Sync scrub runs fine here

### SnapRAID DIFF [Sat Jun 29 10:07:07 AM +08 2024]
Loading state from /root/snapraid/snapraid.content...
Comparing...

  xxxxx equal
       0 added
       0 removed
       0 updated
       0 moved
       0 copied
       0 restored
No differences

SnapRAID Status

WARNING! The array is NOT fully synced.
You have a sync in progress at 97%.
The full array was scrubbed at least one time.
No file has a zero sub-second timestamp.
No rehash is in progress or needed.
No error detected.

Yes. i know the version is abit old but i check your latest code and its not fix
SnapRAID AIO Script version 3.2

When running snapsync status, error below should give some warning or start sync
"WARNING! The array is NOT fully synced."

When the sync is really completed. This is the logs
"SYNC - Everything OK"

@auanasgheps
Copy link
Owner

I'm trying to recap because I'm not sure I've understood correctly what you mean.

  • You ran a job but the sync didn't complete because the system crashed
  • snapraid status tells the sync was in progress and did not complete

What is your ask?

@auanasgheps auanasgheps added question Further information is requested wait for feedback labels Jul 1, 2024
@tyjtyj
Copy link
Author

tyjtyj commented Jul 1, 2024

I run the snapraid sync but did not complete due to system crashed
Thus the script run snapraid diff and found no change, thus it did NOT process to run snapraid sync
The script ended with "SYNC - Everything OK"

However when checked found
WARNING! The array is NOT fully synced.

Problem is the script only rely on snapraid diff to decided if it need to run snapraid sync
and claim everything is ok even snapraid status shows "WARNING! The array is NOT fully synced."

@auanasgheps
Copy link
Owner

Got it.

To resolve this problem, the script should also run a snapraid status command and behave based on the array status.

I'll look into this and update this issue accordingly.

@auanasgheps auanasgheps added enhancement New feature or request to do and removed question Further information is requested wait for feedback labels Jul 11, 2024
@auanasgheps
Copy link
Owner

Hey! The feature is now in the dev branch. The script runs the snapraid status in the pre-check stage and stops if there are issues. Let me know if you need more information.

@tyjtyj
Copy link
Author

tyjtyj commented Sep 10, 2024

Testing it .. will revert if found there is issue.. thanks for the patch.. appreciate it

@GerardPolloRebozado
Copy link

I had same error and when trying the new version the sync won't start saying: "[WARNING] - Previous SnapRAID sync did not complete correctly." setting the script config to always force the sync wont fix it

@auanasgheps
Copy link
Owner

At the moment that's intentional: I haven't included a remediation for this case. The script is not interactive so I don't think there is a nice way to override this problem. Let me know what you think.

To proceed, run snapraid sync on your system and let it complete the sync regularly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wait for feedback
Projects
None yet
Development

No branches or pull requests

3 participants