-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
[Backport release-24.05] keycloak: 24.0.5 -> 25.0.6 #323480
Conversation
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.
It's probably failing because the feature name is not valid. I also saw references to hostname:v1
in the docs, but it's seems the feature name is hostname-v1
.
Source: https://www.keycloak.org/server/features
Any particular reason for this backport? And is it free of breaking changes? |
|
Some additional observations that caught my eye while trying to debug this. I could not spot clues to a solution yet.
and
|
694cda0
to
815c4e3
Compare
In an unexpected turn of events, I found out why this was not working in #346439 |
Result of 3 tests built:
1 package built:
|
@ofborg test keycloak |
Let's add the commit raising test vm memory size, should help with the flaky tests. |
So they only support one major version: https://endoflife.date/keycloak We might as well leave it as is. Version 25 is no better than 24 in terms of support now. |
https://www.keycloak.org/2024/10/release-updates Just FYI, keycloak is changing the way they do releases so that it's less disruptive, and hopefully this will be easier to deal with. |
Diff: keycloak/keycloak@24.0.5...25.0.1 (cherry picked from commit 134a223)
(cherry picked from commit 5182f25)
(cherry picked from commit 84c49af)
(cherry picked from commit c6fa7e4)
Diff: keycloak/keycloak@None...25.0.6 (cherry picked from commit fcb30ff)
(cherry picked from commit 937d396)
Diff: aerogear/keycloak-metrics-spi@refs/tags/5.0.0...6.0.0 (cherry picked from commit fc92a20)
3fe41d2
to
990d052
Compare
Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.