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.

Sign up for GitHub

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

Jump to bottom

Feature/105 add ability to configure cas OIDC authenticator with terraform #106

Merged

alexander-dammeier merged 35 commits into develop from feature/105-add-ability-to-configure-cas-oidc-authonticator-with-terraform

Feb 5, 2025

Contributor

meiserloh commented Jan 31, 2025

Resolves #105

meiserloh and others added 17 commits

January 16, 2025 15:21


          #105 add ability to configure cas oidc authonticator with terraform

e75b44f


          #105 resolve terraform errors

357ba54


          #105 resolve terraform errors

9063c46


          #105 join scopes list

3b4e3f0


          #105 join other lists

04eaf61


          #105 Adding defaults to all cas oidc variables

f6f832a


          #105 Create keycloak module

cdc26ec


          #105 Output client id and secret in keycloak module

57c4126


          #105 Create example for CES with Keycloak and GKE

6513d18


          #105 Extend variable template

06de8b7


          #105 Document variables

0de6bb9


          #105 Remove provider to migrate from legacy module

e761a8d


          #105 fix typo

115ea7b


          #105 changed default value of scopes to reflect the standard naming

ac66576


          #105 add instructions to keycloak module how to set up the keycloak

f881d5b


          #105 Simplify configuration and improve docs + rename keycloak module

89d9d24


          #105 Add lifecycle blocks to ignore changes for randomly generated va…

8890e6a

…lues

alexander-dammeier changed the title ~~Feature/105 add ability to configure cas OIDC authonticator with terraform~~ Feature/105 add ability to configure cas OIDC authenticator with terraform


          #105 Move docs to new directory

9ac0b51

alexander-dammeier requested changes

View reviewed changes

Contributor

alexander-dammeier left a comment

Please reduce the complexity of the example. The intention of this is, that customers can copy the code and can modify it for there needs. Only the modules need to be configurable and ready for production.

terraform/ces-module/main.tf Outdated Show resolved Hide resolved

terraform/ces-module/setup.json.tftpl Outdated Show resolved Hide resolved

terraform/ces-module/variables.tf Outdated Show resolved Hide resolved

terraform/ces-module/variables.tf Outdated Show resolved Hide resolved

terraform/ces-module/variables.tf Outdated Show resolved Hide resolved

terraform/examples/ces_keycloak_gke/ces_cluster/variables_gke.tf Outdated Show resolved Hide resolved

terraform/examples/ces_keycloak_gke/google_bucket/README.md Outdated Show resolved Hide resolved

terraform/examples/ces_keycloak_gke/google_bucket/main.tf Outdated Show resolved Hide resolved

terraform/keycloak-client-module/README.md Outdated Show resolved Hide resolved

terraform/keycloak-client-module/ces-service-account-client.json Outdated Show resolved Hide resolved


          #105 Update changelog

412da88

alexander-dammeier reviewed

View reviewed changes

terraform/keycloak-client-module/main.tf Outdated Show resolved Hide resolved

alexander-dammeier reviewed

View reviewed changes

terraform/keycloak-client-module/main.tf Outdated Show resolved Hide resolved

alexander-dammeier reviewed

View reviewed changes

terraform/keycloak-client-module/main.tf Show resolved Hide resolved

alexander-dammeier reviewed

View reviewed changes

terraform/keycloak-client-module/main.tf Outdated Show resolved Hide resolved

alexander-dammeier reviewed

View reviewed changes

terraform/keycloak-client-module/main.tf Outdated Show resolved Hide resolved

alexander-dammeier reviewed

View reviewed changes

terraform/keycloak-client-module/main.tf Outdated Show resolved Hide resolved

alexander-dammeier reviewed

View reviewed changes

terraform/keycloak-client-module/variables.tf Outdated Show resolved Hide resolved

alexander-dammeier reviewed

View reviewed changes

terraform/examples/ces_keycloak_gke/ces_cluster/variables_ces.tf Outdated Show resolved Hide resolved


          #105 Simplify keycloak GKE example

6d78ee5

jelemux and others added 7 commits

January 31, 2025 14:56


          #105 Use latest setup version for example


          #105 use an object for cas oidc configuration

856b7e7


          #105 Refactor keycloak-client module

20df39b


          Merge remote-tracking branch 'origin/feature/105-add-ability-to-confi…

92369eb

…gure-cas-oidc-authonticator-with-terraform' into feature/105-add-ability-to-configure-cas-oidc-authonticator-with-terraform


          #105 use cas oidc config object in keycloak example

fdd80cb


          #105 Improve default for description

a3b4856


          #105 remove cas oidc client secret from config object to put it in se…

2a25b52

…cret

meiserloh force-pushed the feature/105-add-ability-to-configure-cas-oidc-authonticator-with-terraform branch from a000548 to 2a25b52 Compare

January 31, 2025 17:59

meiserloh added 8 commits

February 3, 2025 11:28


          #105 remove quotes from setup.json object template

24209d3


          #105 change cas scopes to string

d73c017


          #105 change cas allowedGroups and initialAdminUsernames to string

610326e


          #105 change cas enabled and optional to string

69a0022

trying to fix following error:
failed to write registry config data to registry: failed to write entries in registry config: failed to write oidc in registry config: unexpected type bool for oidc/enabled


          #105 revert change cas enabled and optional to string, where unnecessary

44b8b58


          #105 introduce local variable to join lists in ces module

43a1c9c


          #105 Fix typo

7a2b2e7


          #105 Add Readme description

c0b9bdf

alexander-dammeier approved these changes

View reviewed changes

Contributor

alexander-dammeier left a comment

LGTM!

alexander-dammeier merged commit bbbc171 into develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet