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

schema: add tmpfs.mode field #325

Merged
merged 1 commit into from Nov 30, 2022
Merged

Conversation

milas
Copy link
Member

@milas milas commented Nov 30, 2022

Schema included size for tmpfs mounts but not mode. This appears to have been an oversight.

See https://github.com/compose-spec/compose-spec/blob/1eb16a20675324b35258b74b8375b48399893eab/spec.md?plain=1#L1868-L1870.

Fixes #302.

@milas milas added the bug Something isn't working label Nov 30, 2022
@milas milas self-assigned this Nov 30, 2022
Schema included `size` for `tmpfs` mounts but not `mode`. This
appears to have been an oversight.

Fixes compose-spec#302.

Signed-off-by: Milas Bowman <milas.bowman@docker.com>
milas added a commit to milas/compose that referenced this pull request Nov 30, 2022
See compose-spec/compose-go#325 for the acutal spec change. This
propagates it to the Engine API object and adds an E2E test via
Cucumber 🥒

Fixes docker#9873.
milas added a commit to milas/compose that referenced this pull request Nov 30, 2022
See compose-spec/compose-go#325 for the acutal spec change. This
propagates it to the Engine API object and adds an E2E test via
Cucumber 🥒

Fixes docker#9873.

Signed-off-by: Milas Bowman <milas.bowman@docker.com>
milas added a commit to milas/compose that referenced this pull request Nov 30, 2022
See compose-spec/compose-go#325 for the acutal spec change. This
propagates it to the Engine API object and adds an E2E test via
Cucumber 🥒

Fixes docker#9873.

Signed-off-by: Milas Bowman <milas.bowman@docker.com>
@ndeloof ndeloof merged commit afd9c33 into compose-spec:master Nov 30, 2022
milas added a commit to milas/compose that referenced this pull request Dec 2, 2022
See compose-spec/compose-go#325 for the acutal spec change. This
propagates it to the Engine API object and adds an E2E test via
Cucumber 🥒

Fixes docker#9873.

Signed-off-by: Milas Bowman <milas.bowman@docker.com>
milas added a commit to docker/compose that referenced this pull request Dec 2, 2022
See compose-spec/compose-go#325 for the acutal spec change. This
propagates it to the Engine API object and adds an E2E test via
Cucumber 🥒

Fixes #9873.

Signed-off-by: Milas Bowman <milas.bowman@docker.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing support for mode parameter of tmpfs type volumes
3 participants