Bug 11481 - --ignore-errors must be a separate option
Summary: --ignore-errors must be a separate option
Status: NEW
Alias: None
Product: rsync
Classification: Unclassified
Component: core (show other bugs)
Version: 3.1.2
Hardware: All All
: P5 enhancement (vote)
Target Milestone: ---
Assignee: Wayne Davison
QA Contact: Rsync QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-09-01 15:56 UTC by Muralidhar N
Modified: 2016-07-31 14:24 UTC (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Muralidhar N 2015-09-01 15:56:33 UTC
--ignore-errors must be a separate option since we can use this option even when --delete is not specified. Used to ignore various errors. Man page has to be updated to indicate the same.
Comment 1 Kevin Korb 2015-09-01 16:00:43 UTC
--force exists to force --delete to work even when there are errors.  That is the only behaviour that rsync changes based on errors.  So what would you expect --ignore-errors to change when there is no --delete?
Comment 2 Muralidhar N 2015-09-01 17:05:10 UTC
Yes. we must have --ignore-errors independent of --delete and also --force independent of --delete. sometimes we may not use --delete at all and want to continue with rsync even when errors are there and also sometimes rsync may refuse to run, so --force.
Comment 3 Kevin Korb 2015-09-01 17:08:20 UTC
Rsync does continue on errors unless those errors are fatal (like a failure to connect).  Sounds more like what you want is a retry option which is easy enough to script.
Comment 4 Wayne Davison 2015-09-13 23:53:29 UTC
I believe you're trying to make an enhancement suggestion to extend what --ignore-errors does.  If so, what more are you wanting it to do?
Comment 5 Paul N 2016-07-31 14:10:11 UTC
@Kevin, your understanding here is incorrect:  rsync bails entire transfers on things that are NOT fatal to the entire transfer, and --ignore-errors overrides this.  I ended up stumbling upon this bug as a result of trying to remediate the behaviour in the case of the following error:

rsync: write failed on "/mnt/PICO3/foo.tbz2": File too large (27)

--ignore-errors does indeed override the fail-through behaviour in this otherwise non-fatal instance.

(Hint: I'm rsyncing to a vfat filesystem ;), and this file is about 4.5G.  In addition to the fact that the --ignore-errors documentation clearly needs revising, this raises two additional, separate bugs pertaining to a) not caring what errno is, and b) the fact that in the spirit of saving bandwidth we should use fallocate(2) unless a --skip-[f]allocate switch (e.g. for compressed destination filesystems) is presented, so we're not transferring data that we could have predicted would get thrown out... and a third for c) --progress tells us the wrong thing when the file can no longer be grown, indicating that the transfer is actually at 100%!)
Comment 6 Paul N 2016-07-31 14:22:45 UTC
Slightly correction to my above comment, implying 1-2 more bugs:

- When --ignore-errors is not specified, all subsequent files fail with 'Operation not permitted (1)', which is incorrect, at least at the environment level (the receiver might be emitting this, but not for any earthly reason as it were).

- Using --ignore-errors also does The Wrong Thing:  Files after the problematic file (foo.tbz2 in my above example) are shown by --progress as transferred at increasingly fast speeds, in this case 150-200MB/sec, and the last couple approaching twice that... without stracing this these appear to be roughly the source filesystem read speeds?!

So, --ignore-errors being misdocumented and misimplemented appear to have come as a pair.
Comment 7 Paul N 2016-07-31 14:24:21 UTC
(Sorry, more...)

The greatest sin of how --ignore-errors misbehaves in the above case is that the only error REPORTED at the end of the transfer is the one for the oversized file that caused the problem... so a bunch of files are shown as transferred by --progress and yet never were, so perhaps this is --lie-about-errors :)