chore: release 6.0.0 (#70)

* chore: release 6.0.0 * docs: add better breaking changes and warning to changelog * docs(terraform-docs): generate docs and write to README.adoc --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Gonçalo Heleno <[email protected]> Co-authored-by: lentidas <[email protected]>
camptocamp · Feb 23, 2024 · 629db8a · 629db8a
1 parent 435996c
commit 629db8a
Show file tree

Hide file tree

Showing 10 changed files with 44 additions and 16 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,33 @@
 # Changelog
 
+## [6.0.0](https://github.com/camptocamp/devops-stack-module-traefik/compare/v5.0.0...v6.0.0) (2024-02-23)
+
+
+### :warning: BREAKING CHANGES
+
+* move global variables to the variant that requires them
+
+  After the removal of the redirection middleware, we noticed that the variables `cluster_name` and `base_domain` were not used except in the AKS variant, as such, we relocated them specifically to that variant.
+
+  As such, this is a breaking change for all the variants, with the exception of the AKS variant.
+
+### :warning: WARNING
+
+* remove the middleware to allow configuration of the subdomain
+
+  The addition of the variable `subdomain` on the other DevOps Stack modules posed some issues with the redirection middleware added by this module when the variable was set as an empty string. Consequently, we pondered on the utility of said middleware and we decided it is best to remove it.
+
+  This is not a breaking change *per se*, but you need to make sure of 2 things:
+    - you callback URLs on your OIDC configuration now need to include both the domain with and without the cluster name, otherwise you will have authentication errors.
+    - if you any of your workload´s ingresses relied on the this middleware for redirection, make sure to adapt it to respond to both the domain with and without the cluster name. 
+
+
+### Bug Fixes
+
+* make subdomain variable non-nullable ([014bb29](https://github.com/camptocamp/devops-stack-module-traefik/commit/014bb2914f4ac81d67245c40900f5c83b30042b9))
+* move global variables to the variant that requires them ([657a829](https://github.com/camptocamp/devops-stack-module-traefik/commit/657a82912776103302c1887b61cdd97fa62cfcc0))
+* remove the middleware to allow configuration of the subdomain ([37d7f4d](https://github.com/camptocamp/devops-stack-module-traefik/commit/37d7f4d13f3b9eae94eefbce15b88c4801f9cf7c))
+
 ## [5.0.0](https://github.com/camptocamp/devops-stack-module-traefik/compare/v4.1.0...v5.0.0) (2024-01-15)
 
 

diff --git a/README.adoc b/README.adoc
@@ -65,7 +65,7 @@ Description: Override of target revision of the application chart.
 
 Type: `string`
 
-Default: `"v5.0.0"`
+Default: `"v6.0.0"`
 
 ==== [[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>
 
@@ -206,7 +206,7 @@ Description: ID to pass other modules in order to refer to this module as a depe
 |[[input_target_revision]] <<input_target_revision,target_revision>>
 |Override of target revision of the application chart.
 |`string`
-|`"v5.0.0"`
+|`"v6.0.0"`
 |no
 
 |[[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>

diff --git a/aks/README.adoc b/aks/README.adoc
@@ -69,7 +69,7 @@ Description: Override of target revision of the application chart.
 
 Type: `string`
 
-Default: `"v5.0.0"`
+Default: `"v6.0.0"`
 
 ==== [[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>
 
@@ -208,7 +208,7 @@ Description: ID to pass other modules in order to refer to this module as a depe
 |[[input_target_revision]] <<input_target_revision,target_revision>>
 |Override of target revision of the application chart.
 |`string`
-|`"v5.0.0"`
+|`"v6.0.0"`
 |no
 
 |[[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>

diff --git a/eks/README.adoc b/eks/README.adoc
@@ -53,7 +53,7 @@ Description: Override of target revision of the application chart.
 
 Type: `string`
 
-Default: `"v5.0.0"`
+Default: `"v6.0.0"`
 
 ==== [[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>
 
@@ -180,7 +180,7 @@ Description: ID to pass other modules in order to refer to this module as a depe
 |[[input_target_revision]] <<input_target_revision,target_revision>>
 |Override of target revision of the application chart.
 |`string`
-|`"v5.0.0"`
+|`"v6.0.0"`
 |no
 
 |[[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>

diff --git a/kind/README.adoc b/kind/README.adoc
@@ -65,7 +65,7 @@ Description: Override of target revision of the application chart.
 
 Type: `string`
 
-Default: `"v5.0.0"`
+Default: `"v6.0.0"`
 
 ==== [[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>
 
@@ -212,7 +212,7 @@ Description: External IP address of Traefik LB service.
 |[[input_target_revision]] <<input_target_revision,target_revision>>
 |Override of target revision of the application chart.
 |`string`
-|`"v5.0.0"`
+|`"v6.0.0"`
 |no
 
 |[[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>

diff --git a/nodeport/README.adoc b/nodeport/README.adoc
@@ -53,7 +53,7 @@ Description: Override of target revision of the application chart.
 
 Type: `string`
 
-Default: `"v5.0.0"`
+Default: `"v6.0.0"`
 
 ==== [[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>
 
@@ -180,7 +180,7 @@ Description: ID to pass other modules in order to refer to this module as a depe
 |[[input_target_revision]] <<input_target_revision,target_revision>>
 |Override of target revision of the application chart.
 |`string`
-|`"v5.0.0"`
+|`"v6.0.0"`
 |no
 
 |[[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>

diff --git a/scaleway/README.adoc b/scaleway/README.adoc
@@ -53,7 +53,7 @@ Description: Override of target revision of the application chart.
 
 Type: `string`
 
-Default: `"v5.0.0"`
+Default: `"v6.0.0"`
 
 ==== [[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>
 
@@ -180,7 +180,7 @@ Description: ID to pass other modules in order to refer to this module as a depe
 |[[input_target_revision]] <<input_target_revision,target_revision>>
 |Override of target revision of the application chart.
 |`string`
-|`"v5.0.0"`
+|`"v6.0.0"`
 |no
 
 |[[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>

diff --git a/sks/README.adoc b/sks/README.adoc
@@ -75,7 +75,7 @@ Description: Override of target revision of the application chart.
 
 Type: `string`
 
-Default: `"v5.0.0"`
+Default: `"v6.0.0"`
 
 ==== [[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>
 
@@ -220,7 +220,7 @@ Description: ID to pass other modules in order to refer to this module as a depe
 |[[input_target_revision]] <<input_target_revision,target_revision>>
 |Override of target revision of the application chart.
 |`string`
-|`"v5.0.0"`
+|`"v6.0.0"`
 |no
 
 |[[input_enable_service_monitor]] <<input_enable_service_monitor,enable_service_monitor>>

diff --git a/variables.tf b/variables.tf
@@ -23,7 +23,7 @@ variable "destination_cluster" {
 variable "target_revision" {
   description = "Override of target revision of the application chart."
   type        = string
-  default     = "v5.0.0" # x-release-please-version
+  default     = "v6.0.0" # x-release-please-version
 }
 
 variable "enable_service_monitor" {

diff --git a/version.txt b/version.txt
@@ -1 +1 @@
-5.0.0
+6.0.0