Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Move to corev1.PodSpec now that vN-1 supports the containers field. #4221

Merged
merged 1 commit into from
Jun 3, 2019
Merged

Move to corev1.PodSpec now that vN-1 supports the containers field. #4221

merged 1 commit into from
Jun 3, 2019

Commits on Jun 3, 2019

  1. Move to corev1.PodSpec now that vN-1 supports the containers field. 

    Previously we defined our own partial PodSpec because the corev1 version
lacks `omitempty` and appears as `containers: null` in requests from generated
clients, even if unspecified, which would have broken webhook validation.

Now that the field has been out for a release, we can switch to the common
PodSpec.
    @mattmoor
    mattmoor committed Jun 3, 2019
    Configuration menu
    Copy the full SHA
    4cc6571 View commit details
    Browse the repository at this point in the history