Skip to content
New issue

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

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

Already on GitHub? # to your account

Added flag to skip file errors during copy #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sunoce
Copy link

@sunoce sunoce commented Apr 26, 2022

So in case an error occurs for one file, the file is skipped. The behavior is triggered via the method parameter skipErrorFiles.
If the parameter is set to false the behaviour is the same as before.

I have never coded go but this fix allowed me to skip files that got deleted during the backup run.
If there are better ways feel free to adjust or comment.

NOTE:
This PR can only be merged after this PR got merged: maorfr/skbn#23

Signed-off-by: lhgloj1 jannic.lollert@liebherr.com

Signed-off-by: lhgloj1 <jannic.lollert@liebherr.com>
@maorfr
Copy link
Owner

maorfr commented Jun 15, 2022

thanks for contributing! i believe you will need to update skbn to 0.6.0.

@sunoce
Copy link
Author

sunoce commented Dec 6, 2022

Yes I will do so

@amerello
Copy link

@sunoce I think in the meantime maorfr/skbn#23 was merged. Could you complete this PR? I'm really missing this feature you implemented 👍

@sdave2
Copy link

sdave2 commented Sep 22, 2023

@maorfr I am running into this issue also.
Can you please merge this PR?

@amerello
Copy link

amerello commented Sep 22, 2023

@sdave2 this project seems to be unmaintained. I forked it and fixed it myself in the meantime here: https://github.com/amerello/kube-tasks. My fork is working for me at least, you may have a try :)

Edit: I forked because the nuvo fork is incomplete

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants