-
Notifications
You must be signed in to change notification settings - Fork 52
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
Preparation for k8s 1.22 update #512
Conversation
@@ -268,7 +268,7 @@ resource "kubernetes_service" "api" { | |||
} | |||
} | |||
|
|||
resource "kubernetes_ingress" "api" { | |||
resource "kubernetes_ingress_v1" "api" { |
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.
If we do this type of change while also updating to k8s 1.22, terraform will fail to delete the previous ingress resource bc the resource spec was removed in 1.22
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.
Clarification
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.
🔥
What is the feature/fix?
This will update some k8s deprecated ingress resources we use in terraform and also update the metrics server deployment.
I also made some tweaks on the nginx deployment as the current config has some known bugs.
Add screenshot or video (optional)
** Any screenshot or video capture using the feature **
Does it has a breaking change?
Nope
How to use/test it?
The current e2e tests should cover it, as it communicated with the API server.
Checklist