-
Notifications
You must be signed in to change notification settings - Fork 56
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
RHBK: Major Upgrade fails due to missing conf/
, providers/
and themes/
data from prev. installation
#216
Comments
conf/
, providers/
and themes/
data from prev. installation
Hello; I am actively on this (on the downstream tests for the rhbk collections which are not visible publicly , sorry) |
Fact is, ideally there shouldn't be anything in conf/ or providers/ which is not configured by the collection itself; themes is a different beast. I am not sure how to handle the case of "manually" added content. |
Actually, I don't a reason for not placing the upgrade molecule upstream too |
Thinking about it, you are actually right; it's just that the role is missing the possibilities to add policies (black/white lists); moreover, the ability to download providers (the functionality you've recently added via |
Mate, the idea is to catch up feature-wise, and then keep up, with rhbk for the time being :) |
Closing in favor of #229, #226, #222. It's only really the
But to be honest, from a production point of view the whole thing should be put in providers. |
As per 3. of https://access.redhat.com/documentation/en-us/red_hat_build_of_keycloak/24.0/html-single/upgrading_guide/index#install_new_version:
we should consider copying
conf/
,providers/
andthemes/
whenever the version changes and copy them over since otherwise the systemd service doesn't come up (assuming one has password blacklists etc. installed).I haven't experienced issues on any minor update so far, but I think it wouldn't hurt either.
Should we persist the previous version (similar to bootstrapping) as a local fact and then run this task conditionally, wdyt?
Alternatively we could extract the version by
stat
-ing the installation directory, but the former seems way cleaner to me tbhThe text was updated successfully, but these errors were encountered: