Skip to content

Conversation

@mrunalp
Copy link
Contributor

@mrunalp mrunalp commented Apr 13, 2016

Signed-off-by: Mrunal Patel [email protected]

@mrunalp
Copy link
Contributor Author

mrunalp commented Apr 13, 2016

@liangchenye @rhatdan PTAL

@liangchenye
Copy link
Member

LGTM

@rhatdan
Copy link
Contributor

rhatdan commented Apr 14, 2016

Why not make the default template use this method and ship it separate from the runc implementation.

@mrunalp
Copy link
Contributor Author

mrunalp commented Apr 14, 2016

@rhatdan That would mean going elsewhere to get the first template. With this approach one can start by creating a template and then use it as the base for other templates.

@rhatdan
Copy link
Contributor

rhatdan commented Apr 14, 2016

How do I see the default template other then reading the code? Can I easily generate the default template so I can edit it.

@mrunalp
Copy link
Contributor Author

mrunalp commented Apr 14, 2016

@rhatdan Just running ocitools generate will give you the default template.

@rhatdan
Copy link
Contributor

rhatdan commented Apr 14, 2016

Ok I guess we should document this, If you want to customize the default template, run ostree generate. Edit config.json, or run ocitools to create a config.json, which you want as your customer "template."

@mrunalp
Copy link
Contributor Author

mrunalp commented Apr 14, 2016

Yeah, I will follow up with a PR with updated documentation.

@mrunalp mrunalp merged commit 0c6ba09 into opencontainers:master Apr 14, 2016
wking added a commit to wking/ocitools-v2 that referenced this pull request Apr 28, 2016
Which landed in a937f7a (Add support for using a base template,
2016-04-13, opencontainers#35) and still needs docs [1].

[1]: opencontainers#35 (comment)

Signed-off-by: W. Trevor King <[email protected]>
wking added a commit to wking/ocitools-v2 that referenced this pull request Apr 28, 2016
Since a937f7a (Add support for using a base template, 2016-04-13,
opencontainers#35), users have been able to supply their own templates.  If
--mount-cgroups is not 'no', that will lead to additional cgroup
entries in the mount array (even if there was already a cgroup entry
in the mount array).  Instead of writing logic to find and clobber
existing cgroup entries, just make the default the no-op setting.

Signed-off-by: W. Trevor King <[email protected]>
wking added a commit to wking/ocitools-v2 that referenced this pull request Apr 28, 2016
Which landed in a937f7a (Add support for using a base template,
2016-04-13, opencontainers#35) and still needs docs [1].

[1]: opencontainers#35 (comment)

Signed-off-by: W. Trevor King <[email protected]>
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.

3 participants