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

create keycloak #2

Open
wants to merge 27 commits into
base: main
Choose a base branch
from
Open

create keycloak #2

wants to merge 27 commits into from

Conversation

jamal68
Copy link

@jamal68 jamal68 commented May 25, 2021

No description provided.

@akastav akastav added the enhancement New feature or request label May 27, 2021
keycloak/main.tf Outdated Show resolved Hide resolved
keycloak/main.tf Outdated
chart = "keyclok"
chart_version = var.chart_version
conf = merge(local.conf_defaults, var.conf)
conf_defaults = merge({
Copy link
Contributor

Choose a reason for hiding this comment

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

What about another param? Like an admin password or ingress? Just install helm chart not enough for us

@gfrntz
Copy link

gfrntz commented Jun 22, 2021

@jamal68 pls squash git history. Too many similar commit messages.


## Example how add with module
```
module "keycloak" {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you show full example with google auth values and depend-on?

locals {
name = "kube-keycloak"
repository = "https://github.com/bitnami/charts/tree/master/bitnami/keycloak/"
chart = "kube-keycloak"
Copy link
Contributor

Choose a reason for hiding this comment

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

chart name is keycloak (without kube- prefix)


locals {
name = "kube-keycloak"
repository = "https://github.com/bitnami/charts/tree/master/bitnami/keycloak/"
Copy link
Contributor

Choose a reason for hiding this comment

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

repository incorrect, this is Github, needs helm chart repo https://charts.bitnami.com/bitnami

conf = merge(local.conf_defaults, var.conf)
password = var.keycloak_password == "" ? random_password.keycloak_password.result : var.keycloak_password
conf_defaults = {
"keycloak.enabled" = true
Copy link
Contributor

Choose a reason for hiding this comment

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

description = "A name of namespace for creating"
}

variable "module_depends_on" {
Copy link
Contributor

Choose a reason for hiding this comment

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

after Terraform update 0.14 this variable deprecated - use depends_on in module level

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants