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

Enable Kubernetes aggregation by default #279

Merged
merged 1 commit into from
Oct 6, 2021
Merged

Conversation

dghubble
Copy link
Member

@dghubble dghubble commented Oct 6, 2021

  • 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 (I'd personally recommend you avoid them like the plague)
  • https://kubernetes.io/docs/concepts/extend-kubernetes/api-extension/apiserver-aggregation/

* 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
* https://kubernetes.io/docs/concepts/extend-kubernetes/api-extension/apiserver-aggregation/
@dghubble dghubble merged commit fadb5bb into master Oct 6, 2021
@dghubble dghubble deleted the enable-aggregation branch October 6, 2021 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant