The Wayback Machine - https://web.archive.org/web/20251014210147/https://github.com/docker/cli/pull/73
Skip to content

Conversation

vdemeester
Copy link
Collaborator

The read_only key in a composefile is not taken into account right
now. Now that services support --read-only, so should stack deploy

Ref: moby/moby#32994

🦁

Signed-off-by: Vincent Demeester vincent@sbr.pm

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

apparenty it was already in the compose-file schema 😅

Copy link
Contributor

@dnephin dnephin left a comment

Choose a reason for hiding this comment

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

You'll need to remove it from UnsupportedProperties in cli/compose/types/types.go as well. I guess the tests are still passing because it's just printing a warning.

The `read_only` key in a composefile is not taken into account right
now. Now that services support `--read-only`, so should `stack deploy`

Ref: moby/moby#32994

Signed-off-by: Vincent Demeester <vincent@sbr.pm>
@vdemeester vdemeester force-pushed the stack-deploy-support-readonly branch from e5fac26 to 0ab760a Compare May 11, 2017 15:33
@vdemeester
Copy link
Collaborator Author

@dnephin updated 👼

Copy link
Contributor

@dnephin dnephin left a comment

Choose a reason for hiding this comment

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

LGTM when green

@diogomonica
Copy link
Contributor

LGTM

@dnephin dnephin merged commit c0cbb65 into docker:master May 11, 2017
@vdemeester vdemeester deleted the stack-deploy-support-readonly branch May 11, 2017 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants