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

BACKUP=BAREOS: fix bconsole CLI argument format #2901

Merged
merged 1 commit into from Jan 4, 2023

Conversation

hseagle2015
Copy link
Contributor

Use appropriate CLI argument format depending on the installed bconsole version.

Relax-and-Recover (ReaR) Pull Request Template

Please fill in the following items before submitting a new pull request:

Pull Request Details:
  • Type: Bug Fix / New Feature / Enhancement / Other?
    Bug Fix

  • Impact: Low / Normal / High / Critical / Urgent
    Normal

  • Reference to related issue (URL):
    Bareos 22 bconsole switch parsing breaking change #2900

  • How was this pull request tested?
    Manually, on multiple KVM hosts running Bareos 22 and Bareos 21 release.

  • Brief description of the changes in this pull request:
    Bareos 22 introduced breaking change in how the CLI tools (such as bconsole) parse the arguments. This pull request fixes the issue when calling the bconsole -xc command.

Before bconsole configuration export command is called, the script will check the bconsole version and will use the appropriate argument format later on.

If bconsole is called with incorrect argument format, the utility will terminate with 113 exit code, and rear mkrescue will fail as well.

Use appropriate CLI argument format depending on the installed bconsole version.
@gdha gdha added minor bug An alternative or workaround exists external tool The issue depends on other software e.g. third-party backup tools. labels Dec 28, 2022
@gdha gdha self-assigned this Dec 28, 2022
@gdha gdha added this to the ReaR v2.8 milestone Dec 28, 2022
@jsmeix jsmeix added enhancement Adaptions and new features and removed minor bug An alternative or workaround exists labels Jan 2, 2023
@jsmeix jsmeix changed the title fix bconsole CLI argument format BACKUP=BAREOS: fix bconsole CLI argument format Jan 2, 2023
@jsmeix jsmeix requested a review from a team January 2, 2023 11:25
@jsmeix
Copy link
Member

jsmeix commented Jan 2, 2023

@rear/contributors
when there are no objections
I would like to merge this pull request
tomorrow afternoon.

@jsmeix jsmeix merged commit f263572 into rear:master Jan 4, 2023
jsmeix added a commit that referenced this pull request Jan 5, 2023
Explicit comment that explains the Bareos 22 introduced breaking change
how its CLI tools (such as bconsole) parse the arguments ('-xc' vs. '--xc')
cf. #2901
antonvoznia pushed a commit to antonvoznia/rear that referenced this pull request Feb 21, 2023
Explicit comment that explains the Bareos 22 introduced breaking change
how its CLI tools (such as bconsole) parse the arguments ('-xc' vs. '--xc')
cf. rear#2901
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Adaptions and new features external tool The issue depends on other software e.g. third-party backup tools. fixed / solved / done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants