Merge lp://qastaging/~ursinha/landscape-charm/collect-logs-ignore-file-changed into lp://qastaging/~landscape/landscape-charm/tools

Proposed by Ursula Junque
Status: Merged
Approved by: Ursula Junque
Approved revision: 29
Merged at revision: 30
Proposed branch: lp://qastaging/~ursinha/landscape-charm/collect-logs-ignore-file-changed
Merge into: lp://qastaging/~landscape/landscape-charm/tools
Diff against target: 52 lines (+10/-7)
2 files modified
collect-logs (+8/-5)
test_collect-logs.py (+2/-2)
To merge this branch: bzr merge lp://qastaging/~ursinha/landscape-charm/collect-logs-ignore-file-changed
Reviewer Review Type Date Requested Status
Andreas Hasenack Approve
Review via email: mp+305766@code.qastaging.launchpad.net

Commit message

Fixes bug 1623678, by re-adding the mechanism to ignore when tar returns 1 because files changed while tarfile is being created.

Description of the change

This branch fixes bug 1623678, by re-adding the mechanism to ignore when tar returns 1 because files changed while tarfile is being created.

I've done two things:
- Removed the tar --warning=no-file-changed option as that was only masking the output, tar exit code was still 1 causing collect-logs to silently skip tarball creation for such cases. Removing it allows me to check the exception output for that specific message and ignore it.
- Added mechanism to check if tar failure was due to files changing and deliberately proceed, ignoring this error in particular.

These two together restore previous and expected behavior.

To post a comment you must log in.
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

+1!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: