Open iwilltry42 opened 3 years ago
@iwilltry42 Any plans to merge this?
@hemals24 , sure, but I'm not a maintainer, so waiting for review + merge :man_shrugging:
@byxorna The injector fails on the newer versions of k8s. This PR would fix the issue. Do you have plans to merge this?
@iwilltry42 Add people as reviewers, that should help speed things up
Unfortunately, that option is disabled and I cannot assign reviewers :thinking:
Strange !!! I used your code and modified it a little bit to pass the build. https://github.com/tumblr/k8s-sidecar-injector/pull/60
What and why?
Testing Steps
Please provide adequate testing steps (including screenshots if necessary). Include any test fixtures or sample configurations in your commit.
https://github.com/kubernetes/kubernetes/blob/v1.10.0-beta.1/staging/src/k8s.io/api/admissionregistration/v1beta1/types.go#L218
note: k8s is smart enough to use 443 or the only exposed port on the service
note: this requires the service to serve TLS directly (not thru ingress)
service: name: "k8s-sidecar-injector" namespace: "{{ .Release.Namespace }}" path: "/mutate" # what /url/slug to send requests at
See README.md for how this was generated!
caBundle: {{ required $caCertRequiredWarning (.Files.Get $caCert) | b64enc }} admissionReviewVersions: ["v1", "v1beta1"] sideEffects: None
Reviewers
Required reviewers: @byxorna Request reviews from other people you want to review this PR in the "Reviewers" section on the right.