Skip to content

Commit

Permalink
Upgrade Keycloak
Browse files Browse the repository at this point in the history
Keycloak has been upgraded elsewhere and we need to respond
* Updates the default Keycloak image
* Changes the database environment variable to be dev-mem based
* Changes the env var for initial admin credentials
* Changes the mount path for the initial realm import
  • Loading branch information
psav committed Nov 28, 2023
1 parent c1d5cc2 commit 5513b3b
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 12 deletions.
2 changes: 1 addition & 1 deletion controllers/cloud.redhat.com/providers/utils/utils.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ var DefaultImageCaddySideCar = "quay.io/cloudservices/crc-caddy-plugin:a988cd2"
var DefaultImageCaddyGateway = DefaultImageCaddySideCar
var DefaultImageMBOP = "quay.io/cloudservices/mbop:959d00d"
var DefaultImageMocktitlements = "quay.io/cloudservices/mocktitlements:e24820c"
var DefaultKeyCloakVersion = "15.0.2"
var DefaultKeyCloakVersion = "23.0.0"
var DefaultImageKeyCloak = fmt.Sprintf("quay.io/keycloak/keycloak:%s", DefaultKeyCloakVersion)

// MakeLocalDB populates the given deployment object with the local DB struct.
Expand Down
24 changes: 14 additions & 10 deletions controllers/cloud.redhat.com/providers/web/resources_keycloak.go
Original file line number Diff line number Diff line change
Expand Up @@ -140,15 +140,15 @@ func makeKeycloak(o obj.ClowdObject, objMap providers.ObjectMap, _ bool, nodePor

envVars := []core.EnvVar{
{
Name: "DB_VENDOR",
Value: "h2",
Name: "KC_DB",
Value: "dev-mem",
},
{
Name: "PROXY_ADDRESS_FORWARDING",
Value: "true",
},
{
Name: "KEYCLOAK_USER",
Name: "KEYCLOAK_ADMIN",
ValueFrom: &core.EnvVarSource{
SecretKeyRef: &core.SecretKeySelector{
LocalObjectReference: core.LocalObjectReference{
Expand All @@ -159,7 +159,7 @@ func makeKeycloak(o obj.ClowdObject, objMap providers.ObjectMap, _ bool, nodePor
},
},
{
Name: "KEYCLOAK_PASSWORD",
Name: "KEYCLOAK_ADMIN_PASSWORD",
ValueFrom: &core.EnvVarSource{
SecretKeyRef: &core.SecretKeySelector{
LocalObjectReference: core.LocalObjectReference{
Expand Down Expand Up @@ -194,15 +194,15 @@ func makeKeycloak(o obj.ClowdObject, objMap providers.ObjectMap, _ bool, nodePor

livenessProbe := core.Probe{
ProbeHandler: probeHandler,
InitialDelaySeconds: 10,
InitialDelaySeconds: 30,
TimeoutSeconds: 2,
PeriodSeconds: 10,
SuccessThreshold: 1,
FailureThreshold: 3,
}
readinessProbe := core.Probe{
ProbeHandler: probeHandler,
InitialDelaySeconds: 20,
InitialDelaySeconds: 30,
TimeoutSeconds: 2,
PeriodSeconds: 10,
SuccessThreshold: 1,
Expand All @@ -213,9 +213,13 @@ func makeKeycloak(o obj.ClowdObject, objMap providers.ObjectMap, _ bool, nodePor
image := provutils.GetKeycloakImage(env)

c := core.Container{
Name: nn.Name,
Image: image,
Env: envVars,
Name: nn.Name,
Image: image,
Env: envVars,
Args: []string{
"start-dev",
"--import-realm",
},
Ports: ports,
LivenessProbe: &livenessProbe,
ReadinessProbe: &readinessProbe,
Expand All @@ -235,7 +239,7 @@ func makeKeycloak(o obj.ClowdObject, objMap providers.ObjectMap, _ bool, nodePor
VolumeMounts: []core.VolumeMount{
{
Name: "realm-import",
MountPath: "/json",
MountPath: "/opt/keycloak/data/import/",
},
},
}
Expand Down
2 changes: 1 addition & 1 deletion tests/kuttl/test-ephemeral-gateway/01-assert.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ spec:
template:
spec:
containers:
- image: quay.io/keycloak/keycloak:15.0.2
- image: quay.io/keycloak/keycloak:23.0.0
---
apiVersion: apps/v1
kind: Deployment
Expand Down

0 comments on commit 5513b3b

Please sign in to comment.