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

Debug c8s oom integration tests #14

Closed
wants to merge 7 commits into from

Conversation

pcahyna
Copy link
Owner

@pcahyna pcahyna commented Dec 18, 2023

No description provided.

pcahyna and others added 7 commits December 11, 2023 15:03
Remove a forgotten mkdir -p $BUILD_DIR/outputfs/$NETFS_PREFIX command in
restore/NETFS/default/400_restore_backup.sh.

Should have been deleted in 7dda23d
with other code that touched the backup location.

For the YUM method, the same change was already done in 6b9d8d8
OUTPUT=USB: add a check that OUTPUT_URL is mounted
Do not touch $BUILD_DIR/outputfs (the backup location) during restore
In output/USB/Linux-i386/850_make_USB_bootable.sh replaced the
BugError "Filesystem ... on $RAW_USB_DEVICE could not be found"
with explanatory LogPrintError plus Error what the actual reason is,
similar as in the related rear#3102
because in most cases this is not a bug in ReaR but an error
like a user configuration error for OUTPUT=USB e.g.
see rear#1571
and rear#3098
Use first LogPrintError for an additional info message
and then Error for the actual error message.
In those LogPrintError plus Error message pairs
show both $BUILD_DIR/outputfs and $RAW_USB_DEVICE
because $BUILD_DIR/outputfs is a meaningless directory
of the form /var/tmp/rear.XXXXXXXXXXXXXXX/outputfs
so $RAW_USB_DEVICE provides some context to the user
that those two messages are about his USB or disk device.
In [skel/default]/etc/scripts/boot set
dmesg -n 5
to limit console logging for 'dmesg' messages to level 5
so that kernel error and warning messages appear
(intermixed with ReaR messages) on the console
so that the user can notice when things go wrong
in kernel area which helps to understand problems,
see the related issue
rear#3107
Copy link

Stale pull request message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants