Skip to content

Conversation

@laijs
Copy link
Contributor

@laijs laijs commented Sep 9, 2015

There are two RootfsPropagation fields, one is Linux.RootfsPropagation,
the other one is LinuxRuntime.RootfsPropagation. They are duplicated,
one of them should be removed.

The RootfsPropagation is definitely a runtime specific configuration,
so we remove the one of Linux.RootfsPropagation.

And the description of it is moved from config-linux.md to
runtime-config-linux.md.

Signed-off-by: Lai Jiangshan jiangshanlai@gmail.com

There are two RootfsPropagation fields, one is Linux.RootfsPropagation,
the other one is LinuxRuntime.RootfsPropagation. They are duplicated,
one of them should be removed.

The RootfsPropagation is definitely a runtime specific configuration,
so we remove the one of Linux.RootfsPropagation.

And the description of it is moved from config-linux.md to
runtime-config-linux.md.

Signed-off-by: Lai Jiangshan <jiangshanlai@gmail.com>
@mrunalp
Copy link
Contributor

mrunalp commented Sep 9, 2015

LGTM

@philips
Copy link
Contributor

philips commented Sep 9, 2015

lgtm

mrunalp pushed a commit that referenced this pull request Sep 9, 2015
Deduplicate the field of RootfsPropagation
@mrunalp mrunalp merged commit fd0ff56 into opencontainers:master Sep 9, 2015
@laijs
Copy link
Contributor Author

laijs commented Sep 11, 2015

This pr is missing due to the previous error @vbatts

@laijs
Copy link
Contributor Author

laijs commented Sep 11, 2015

sorry, it is included. weird.

@vbatts
Copy link
Member

vbatts commented Sep 11, 2015

Yeah. For a short time, master was missing this merge, but I've since fixed
it.
On Sep 10, 2015 21:21, "Lai Jiangshan" notifications@github.com wrote:

sorry, it is included. weird.


Reply to this email directly or view it on GitHub
#163 (comment).

@laijs laijs deleted the RootfsPropagation branch September 17, 2015 01:31
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.

4 participants