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

packaging: fix for keycloak httpd conf.d restore #405

Merged
merged 1 commit into from
Jun 1, 2022

Conversation

arso
Copy link
Contributor

@arso arso commented Jun 1, 2022

This patch includes keycloak related httpd configuration files in backup
paths so that, after restoring from the backup, these files were available
without the need to regenerate them.
Additionally grafana httpd proxy configuration has been added as well
for the same reason.

Copy link
Member

@mwperina mwperina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@arso arso marked this pull request as draft June 1, 2022 08:14
@arso
Copy link
Contributor Author

arso commented Jun 1, 2022

I found that keycloak db is not backed up at all. I need to fix it before merging.
[UPDATE] it is because the ENABLE variable was moved to ovirt engine and changed its key. Fix in a moment.

@arso arso marked this pull request as ready for review June 1, 2022 08:24
@arso
Copy link
Contributor Author

arso commented Jun 1, 2022

Backup / restore verified manually. Waiting now for the engine build to produce artifacts for OST.

@arso
Copy link
Contributor Author

arso commented Jun 1, 2022

/ost

@arso
Copy link
Contributor Author

arso commented Jun 1, 2022

/ost

1 similar comment
@arso
Copy link
Contributor Author

arso commented Jun 1, 2022

/ost

This patch includes keycloak related httpd configuration files in backup
paths so that, after restoring from the backup, these files were available
without the need to regenerate them.
Additionally grafana httpd proxy configuration has been added as well
for the same reason.
@arso
Copy link
Contributor Author

arso commented Jun 1, 2022

/ost

@mwperina mwperina merged commit a16c21b into oVirt:master Jun 1, 2022
@arso arso deleted the keycloak_restore_fix branch June 1, 2022 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants