-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Update deprecated APIs #144
Comments
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
@mumoshu please add the |
Any chance of updating admissionregistration.k8s.io/v1beta1 in the charts? We are getting "build" failures on lint. |
@jwalters-gpsw Hey! Thanks for reporting. As I have other high-priority items now, would you mind submitting a PR for that? I thought we've already migrated to newer controller-runtime and controller-gen in #629 and upgraded admissionregistration API used in the kustomize config. Perhaps you can make similar manual changes in the chart. |
Also note that CRDs have already been upgraded to the v1 API in #629 |
Apparently, we only migrated to the newer API for newly generated resources. There are still a few remaining resources to be upgraded. You can try removing |
This fixes the below error on installing the chart: ``` Error: UPGRADE FAILED: error validating "": error validating data: [ValidationError(MutatingWebhookConfiguration.webhooks[0]): missing required field "admissionReviewVersions" in io.k8s.api.admissionregistration.v1.MutatingWebhook, ValidationError(MutatingWebhookConfiguration.webhooks[1]): missing required field "admissionReviewVersions" in io.k8s.api.admissionregistration.v1.MutatingWebhook, ValidationError(MutatingWebhookConfiguration.webhooks[2]): missing required field "admissionReviewVersions" in io.k8s.api.admissionregistration.v1.MutatingWebhook, ValidationError(MutatingWebhookConfiguration.webhooks[3]): missing required field "admissionReviewVersions" in io.k8s.api.admissionregistration.v1.MutatingWebhook] ``` Ref #144
This fixes the below error that occurs in `make release`: ``` kustomize build config/default > release/actions-runner-controller.yaml Error: accumulating resources: accumulation err='accumulating resources from '../webhook': '/home/mumoshu/p/actions-runner-controller/config/webhook' must resolve to a file': recursed accumulation of path '/home/mumoshu/p/actions-runner-controller/config/webhook': accumulating resources: accumulation err='accumulating resources from 'manifests.v1beta1.yaml': evalsymlink failure on '/home/mumoshu/p/actions-runner-controller/config/webhook/manifests.v1beta1.yaml' : lstat /home/mumoshu/p/actions-runner-controller/config/webhook/manifests.v1beta1.yaml: no such file or directory': evalsymlink failure on '/home/mumoshu/p/actions-runner-controller/config/webhook/manifests.v1beta1.yaml' : lstat /home/mumoshu/p/actions-runner-controller/config/webhook/manifests.v1beta1.yaml: no such file or directory make: *** [Makefile:156: release] Error 1 ``` Ref #144
At least CRD requires also fixing of tests and the default value handling of protocol.
The text was updated successfully, but these errors were encountered: