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.

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

defaults_tests: verify defaults when converting to an API object #6235

Merged
merged 1 commit into from Mar 31, 2015

Conversation

yujuhong
Copy link
Contributor

This change verifies that defaults are applied after decoding an versioned
object into an API object.

This addresses the test issues in #6210

This change verifies that defaults are applied after decoding an versioned
object into an API object.
@yujuhong
Copy link
Contributor Author

/cc @thockin

@thockin
Copy link
Member

thockin commented Mar 31, 2015

Thanks! But you seem to have a change in there that is unrelated - please look at the diff.

@yujuhong
Copy link
Contributor Author

Nope, the lack of HostNetwork in api.ContainerManifest is causing the tests to fail. Note that HostNetwork is there in all versioned ContainerManifests.

thockin added a commit that referenced this pull request Mar 31, 2015
defaults_tests: verify defaults when converting to an API object
@thockin thockin merged commit 0a8e2e0 into kubernetes:master Mar 31, 2015
@yujuhong yujuhong deleted the defaults_tests branch May 8, 2015 17:38
atwamahmoud added a commit to atwamahmoud/kubernetes that referenced this pull request Nov 8, 2023
atwamahmoud added a commit to atwamahmoud/kubernetes that referenced this pull request Jan 10, 2024
atwamahmoud added a commit to atwamahmoud/kubernetes that referenced this pull request Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants