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

validate: Increase the valid value of devices.type #459

Closed
wants to merge 1 commit into from

Conversation

zhouhao3
Copy link

@zhouhao3 zhouhao3 commented Sep 1, 2017

According to the spec.

Signed-off-by: zhouhao zhouhao@cn.fujitsu.com

Signed-off-by: zhouhao <zhouhao@cn.fujitsu.com>
@Mashimiao
Copy link

type validation is just value check, I prefer to put it into schema validation. We will remove it after #197 landed.

@Mashimiao
Copy link

And "" is not a valid value for devices.type. From the spec, unset value means it's nil not ""

@Mashimiao Mashimiao added this to the v0.2.0 milestone Sep 15, 2017
@wking
Copy link
Contributor

wking commented Sep 15, 2017

We can drop the whole r.Devices[index].Type check if/when opencontainers/runtime-spec#690 lands, after which 1.0.x JSON Schema checks will handle that for us.

@Mashimiao
Copy link

As #197 landed, let's close this PR. We will remove all duplicate validation items with schema in the future.

@Mashimiao Mashimiao closed this Sep 25, 2017
@zhouhao3 zhouhao3 deleted the devices-fix branch October 18, 2017 08:07
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.

3 participants