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

Edited backup script file paths to work in development environment & fixed engine restore bug #932

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Jul 1, 2024

  1. Fix file backup in dev mode & fatal restore error

    When we were restoring the engine from a previous backup where there was
    no active DWH, Cinderlib installation or Keycloak database, the restore
    always resulted in a FATAL error:
    
    -----
    Rewriting /home/build/ovirt//etc/ovirt-engine/engine.conf.d/10-setup-database.conf
    /home/build/ovirt/share/ovirt-engine/bin/engine-backup.sh: line 1419: [: syntax error: `-' unexpected
    FATAL: Can not find /home/build/ovirt//etc/ovirt-engine/engine.conf.d/10-setup-cinderlib-database.conf
    -----
    
    This is because of a check that is done when restoring the engine. When
    only restoring the DB, the value ${CHANGE_DB_CREDENTIALS} is true and
    because of the OR condition, the false flag of for example
    ${CHANGE_DWH_CREDENTIALS} does not have any effect anymore. By removing
    the CHANGE_DB_CREDENTIALS variable, the condition is dependent on only
    the flag of the type of DB itself and the DB_USER.
    
    There were also issues with running a backup in development mode.
    When installing a development environment, a path is specified with the
    PREFIX value. However, the backup script does not change the paths that
    it is backing up relative to this PREFIX value. This has also been
    changed to that it does backup the correct paths.
    
    Signed-off-by: Brooklyn Dewolf <contact@brooklyn.gent>
    BrooklynDewolf authored and sandrobonazzola committed Jul 1, 2024
    Configuration menu
    Copy the full SHA
    9cf397d View commit details
    Browse the repository at this point in the history