Skip to content

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