-
Notifications
You must be signed in to change notification settings - Fork 518
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
Updates #409
Updates #409
Conversation
AzureLinux AKS 1.29 Traefik 2.11 Other small bug fixes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nicely done @skabou, please find some feedback below.
@@ -10,7 +10,7 @@ GitOps allows a team to author Kubernetes manifest files, persist them in their | |||
- Cluster-wide configuration of Azure Monitor for Containers | |||
- The workload's namespace named `a0008` | |||
|
|||
1. Install `kubectl` 1.27 or newer. (`kubectl` supports ±1 Kubernetes version.) | |||
1. Install `kubectl` 1.28 or newer. (`kubectl` supports ±1 Kubernetes version.) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are we maintaining the client version n-1 for a reason? Any desire to move over 1.29 instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was the guidance I received from @ckittel on a previous PR.
Also, it appears to be official policy from Kubernetes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, while Im aware of how it gets supported, I was wondering why we are prescribing that. Just wanted to learn from this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
w00t @skabou, LGTM 🚢
Image cleaner no longer in preview, now GA
AKS 1.29
Traefik 2.11
AzureLinux
bugfix: container image was missing curl
bugfix: kustomization patchesJson6902->patches