Skip to content

Commit

Permalink
fix: remove legacy ingress annotations (#33)
Browse files Browse the repository at this point in the history
* fix: remove legacy ingress annotations

The SSL redirection is no longer defined by these annotations, I think this is a leftover from ancient code. The HTTP -> HTTPS redirection is handled natively by the Traefik module and is enabled by default (a variable is available to deactivate it if necessary).

* docs(terraform-docs): generate docs and write to README.adoc

---------

Co-authored-by: lentidas <lentidas@users.noreply.github.com>
  • Loading branch information
lentidas and lentidas authored Mar 1, 2024
1 parent 2fd51e8 commit ede4ddb
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 7 deletions.
4 changes: 2 additions & 2 deletions README.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ Description: Override of target revision of the application chart.

Type: `string`

Default: `"v3.0.0"`
Default: `"v3.1.0"`

==== [[input_cluster_issuer]] <<input_cluster_issuer,cluster_issuer>>

Expand Down Expand Up @@ -351,7 +351,7 @@ Description: Credentials for the administrator user of the master realm created
|[[input_target_revision]] <<input_target_revision,target_revision>>
|Override of target revision of the application chart.
|`string`
|`"v3.0.0"`
|`"v3.1.0"`
|no
|[[input_cluster_issuer]] <<input_cluster_issuer,cluster_issuer>>
Expand Down
2 changes: 0 additions & 2 deletions locals.tf
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,6 @@ locals {
"cert-manager.io/cluster-issuer" = "${var.cluster_issuer}"
"traefik.ingress.kubernetes.io/router.entrypoints" = "websecure"
"traefik.ingress.kubernetes.io/router.tls" = "true"
"ingress.kubernetes.io/ssl-redirect" = "true"
"kubernetes.io/ingress.allow-http" = "false"
}
hosts = [
{
Expand Down
6 changes: 3 additions & 3 deletions oidc_bootstrap/README.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -161,11 +161,11 @@ The following requirements are needed by this module:

The following providers are used by this module:

- [[provider_random]] <<provider_random,random>> (>= 3)
- [[provider_null]] <<provider_null,null>> (>= 3)

- [[provider_keycloak]] <<provider_keycloak,keycloak>> (>= 4)

- [[provider_null]] <<provider_null,null>> (>= 3)
- [[provider_random]] <<provider_random,random>> (>= 3)

=== Resources

Expand Down Expand Up @@ -310,9 +310,9 @@ Description: Map containing the credentials of each created user.
[cols="a,a",options="header,autowidth"]
|===
|Name |Version
|[[provider_null]] <<provider_null,null>> |>= 3
|[[provider_keycloak]] <<provider_keycloak,keycloak>> |>= 4
|[[provider_random]] <<provider_random,random>> |>= 3
|[[provider_null]] <<provider_null,null>> |>= 3
|===
= Resources
Expand Down

0 comments on commit ede4ddb

Please sign in to comment.