-
Notifications
You must be signed in to change notification settings - Fork 259
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
stored: enable labeling of tapes in devices even when autoselect = no
#1021
stored: enable labeling of tapes in devices even when autoselect = no
#1021
Conversation
release
or label
when autoselect = no
release
and label
on devices where autoselect = no
release
and label
on devices where autoselect = no
autoselect = no
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.
Excellent work!
It might make sense to refactor this test to have setup and teardown, and to have two autochanger tests (the original one and the new one testing the label functionality)
bd336ae
to
c11ef92
Compare
c11ef92
to
6acc04f
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.
This looks like a good idea, I just wouldn't return a device with autoselect = no
unless the caller explicitly asks for that one.
48f81f6
to
917a9b9
Compare
systemtests/tests/autochanger/testrunner-label-release-autoselect
Outdated
Show resolved
Hide resolved
9da8672
to
dccbf0c
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.
bareos-check-sources is not happy yet.
Other than that, it looks great and can be merged.
added the following checks to the `autochanger` systemtest: -Checking `label` command doesn't skip device when `autoselect = no` -Checking `release` command doesn't skip device when `autoselect = no` -Checking backups are not affected by newly added change -Checking tape backups do not use devices that have `autoselect = no` systemtest: updated systemtest
dccbf0c
to
399a349
Compare
Description:
When devices are set with the option
autoselect=no
, labeling and releasing is not possible as the device is skipped in the search. This PR handles this issue by not accounting for theautoselect
attribute when searching devices. This has the same effect on themount
,unmount
, as well asreadlabel
andautochanger
commands. Backups are not affected.Please check
If you have any questions or problems, please give a comment in the PR.
Helpful documentation and best practices
Checklist for the reviewer of the PR (will be processed by the Bareos team)
General
Source code quality
bareos-check-sources --since-merge
does not report any problemsgit status
should not report modifications in the source tree after building and testingTests