Skip to content
This repository has been archived by the owner on Sep 4, 2020. It is now read-only.

The resources field should be required #364

Closed
technosophos opened this issue Oct 15, 2019 · 1 comment · Fixed by #435
Closed

The resources field should be required #364

technosophos opened this issue Oct 15, 2019 · 1 comment · Fixed by #435
Labels
Type: Bug Something isn't working
Milestone

Comments

@technosophos
Copy link
Contributor

According to the current version of the spec:

  • the container.resources field is required
  • the container.resources.cpu field is required
  • the container.resources.memory field is required

I think this is bad, but... it is what the spec says, so we should fix that in Rudr.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant