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
handle save_y2logs when content modified when creating tarball #1145
Conversation
scripts/save_y2logs
Outdated
err=$? | ||
|
||
rm -rf $TMPDIR | ||
|
||
# tar returns 1 when content of tarball is older then current system. Happen when something still writting to logs or memsampler (bsc#1182710#c2) | ||
[ $err == 1 ] && echo "Warning: logs modified during save. Yast is probably running. Content of tarball can be older then current content." >&2 && err=0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
older than
(user message, so not just NP)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks, will fix
scripts/save_y2logs
Outdated
err=$? | ||
|
||
rm -rf $TMPDIR | ||
|
||
# tar returns 1 when content of tarball is older then current system. Happen when something still writting to logs or memsampler (bsc#1182710#c2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NP: older than the current system
e51a767
to
5d9267a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
✔️ Public Jenkins job #289 successfully finished |
✔️ Internal Jenkins job #149 successfully finished |
How it looks now ( easy to reproduce with Y2DEBUG=1 and slide show ):