Skip to content

allow tmpfs.size to be set in unit bytes#192

Merged
ulyssessouza merged 1 commit intocompose-spec:masterfrom
ndeloof:tmpfs_size
Sep 29, 2021
Merged

allow tmpfs.size to be set in unit bytes#192
ulyssessouza merged 1 commit intocompose-spec:masterfrom
ndeloof:tmpfs_size

Conversation

@ndeloof
Copy link
Copy Markdown
Collaborator

@ndeloof ndeloof commented Sep 29, 2021

Comment on lines +605 to +610
"capabilities": {"$ref": "#/definitions/list_of_strings"},
"count": {"type": ["string", "integer"]},
"device_ids": {"$ref": "#/definitions/list_of_strings"},
"driver":{"type": "string"},
"options":{"$ref": "#/definitions/list_or_dict"}
},
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentation seems to be 2 characters in the rest of the file, was this change intended?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I copied the exact content from compose-spec - we haven't yet found a way to directly refer to the reference repository for this file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a JSON linter and the same approach as we use on V1, which is to integrate the download of the spec file in the build.

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Copy link
Copy Markdown
Contributor

@ulyssessouza ulyssessouza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ulyssessouza ulyssessouza merged commit 162cc24 into compose-spec:master Sep 29, 2021
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.

Compose V2 tmpfs.size must be a integer

3 participants