-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
Rename kubernetes config section to kubernetes_executor #26873
Rename kubernetes config section to kubernetes_executor #26873
Conversation
e07e646
to
31106ad
Compare
83c657f
to
f20784f
Compare
01789fe
to
ccae4f9
Compare
ccae4f9
to
19b978d
Compare
Hi, today I tried upgrading to 2.5.0 with helm-chart version 1.7.0 and found the following warning messages
I think we have to update this part: to change from |
Bad warning resolved here #28109 But yes there may be chart updates that need to happen Make pr? |
@dstandish |
Now that KPO does not consider any core k8s config params, this section truly is just about kubernetes executor. Renaming it reduces potential for confusion.