Enable Kubernetes aggregation by default

* Change `enable_aggregation` default from false to true
* These days, Kubernetes control plane components emit annoying
messages related to assumptions baked into the Kubernetes API
Aggregation Layer if you don't enable it. Further the conformance
tests force you to remember to enable it if you care about passing
those
* This change is motivated by eliminating annoyances, rather than
any enthusiasm for Kubernetes' aggregation features

Rel: https://kubernetes.io/docs/concepts/extend-kubernetes/api-extension/apiserver-aggregation/
This commit is contained in:
Dalton Hubble 2021-12-09 17:25:35 -08:00
parent 39da5b53f5
commit e97c1cc9e5
11 changed files with 21 additions and 21 deletions

View File

@ -7,6 +7,7 @@ Notable changes between versions.
* Kubernetes [v1.23.0](https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.23.md#v1230) * Kubernetes [v1.23.0](https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.23.md#v1230)
* Normalize CA certs mounts in static Pods and kube-proxy * Normalize CA certs mounts in static Pods and kube-proxy
* With Calico, add missing `caliconodestatuses` CRD ([#289](https://github.com/poseidon/terraform-render-bootstrap/pull/289)) * With Calico, add missing `caliconodestatuses` CRD ([#289](https://github.com/poseidon/terraform-render-bootstrap/pull/289))
* Change `enable_aggregation` default to true ([#279](https://github.com/poseidon/terraform-render-bootstrap/pull/279))
### AWS ### AWS
@ -50,7 +51,6 @@ Notable changes between versions.
* Update Calico from v3.20.1 to [v3.20.2](https://github.com/projectcalico/calico/releases/tag/v3.20.2) * Update Calico from v3.20.1 to [v3.20.2](https://github.com/projectcalico/calico/releases/tag/v3.20.2)
* Use Calico's iptables legacy vs nft auto-detection * Use Calico's iptables legacy vs nft auto-detection
* Update flannel from v0.13.0 to v0.14.0 * Update flannel from v0.13.0 to v0.14.0
* Change `enable_aggregation` default to true ([#279](https://github.com/poseidon/terraform-render-bootstrap/pull/279))
### Bare-Metal ### Bare-Metal

View File

@ -142,8 +142,8 @@ variable "enable_reporting" {
variable "enable_aggregation" { variable "enable_aggregation" {
type = bool type = bool
description = "Enable the Kubernetes Aggregation Layer (defaults to false)" description = "Enable the Kubernetes Aggregation Layer"
default = false default = true
} }
variable "worker_node_labels" { variable "worker_node_labels" {

View File

@ -142,8 +142,8 @@ variable "enable_reporting" {
variable "enable_aggregation" { variable "enable_aggregation" {
type = bool type = bool
description = "Enable the Kubernetes Aggregation Layer (defaults to false)" description = "Enable the Kubernetes Aggregation Layer"
default = false default = true
} }
variable "worker_node_labels" { variable "worker_node_labels" {

View File

@ -117,8 +117,8 @@ variable "enable_reporting" {
variable "enable_aggregation" { variable "enable_aggregation" {
type = bool type = bool
description = "Enable the Kubernetes Aggregation Layer (defaults to false)" description = "Enable the Kubernetes Aggregation Layer"
default = false default = true
} }
variable "worker_node_labels" { variable "worker_node_labels" {

View File

@ -123,8 +123,8 @@ variable "enable_reporting" {
variable "enable_aggregation" { variable "enable_aggregation" {
type = bool type = bool
description = "Enable the Kubernetes Aggregation Layer (defaults to false)" description = "Enable the Kubernetes Aggregation Layer"
default = false default = true
} }
variable "worker_node_labels" { variable "worker_node_labels" {

View File

@ -146,8 +146,8 @@ variable "enable_reporting" {
variable "enable_aggregation" { variable "enable_aggregation" {
type = bool type = bool
description = "Enable the Kubernetes Aggregation Layer (defaults to false)" description = "Enable the Kubernetes Aggregation Layer"
default = false default = true
} }
# unofficial, undocumented, unsupported # unofficial, undocumented, unsupported

View File

@ -151,8 +151,8 @@ variable "enable_reporting" {
variable "enable_aggregation" { variable "enable_aggregation" {
type = bool type = bool
description = "Enable the Kubernetes Aggregation Layer (defaults to false)" description = "Enable the Kubernetes Aggregation Layer"
default = false default = true
} }
# unofficial, undocumented, unsupported # unofficial, undocumented, unsupported

View File

@ -94,8 +94,8 @@ variable "enable_reporting" {
variable "enable_aggregation" { variable "enable_aggregation" {
type = bool type = bool
description = "Enable the Kubernetes Aggregation Layer (defaults to false)" description = "Enable the Kubernetes Aggregation Layer"
default = false default = true
} }
# unofficial, undocumented, unsupported # unofficial, undocumented, unsupported

View File

@ -94,8 +94,8 @@ variable "enable_reporting" {
variable "enable_aggregation" { variable "enable_aggregation" {
type = bool type = bool
description = "Enable the Kubernetes Aggregation Layer (defaults to false)" description = "Enable the Kubernetes Aggregation Layer"
default = false default = true
} }
# unofficial, undocumented, unsupported # unofficial, undocumented, unsupported

View File

@ -118,8 +118,8 @@ variable "enable_reporting" {
variable "enable_aggregation" { variable "enable_aggregation" {
type = bool type = bool
description = "Enable the Kubernetes Aggregation Layer (defaults to false)" description = "Enable the Kubernetes Aggregation Layer"
default = false default = true
} }
variable "worker_node_labels" { variable "worker_node_labels" {

View File

@ -112,8 +112,8 @@ variable "enable_reporting" {
variable "enable_aggregation" { variable "enable_aggregation" {
type = bool type = bool
description = "Enable the Kubernetes Aggregation Layer (defaults to false)" description = "Enable the Kubernetes Aggregation Layer"
default = false default = true
} }
variable "worker_node_labels" { variable "worker_node_labels" {