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

Issues regarding system containers not using runc #42

Open
yuqi-zhang opened this issue Mar 10, 2017 · 5 comments
Open

Issues regarding system containers not using runc #42

yuqi-zhang opened this issue Mar 10, 2017 · 5 comments

Comments

@yuqi-zhang
Copy link
Contributor

Some considerations came up from the azure container, which uses systemd but not runc. If we run this container from the atomic CLI, there are a few issues I noticed:

  1. We generate a default config.json for a container image missing a config.json.template. For containers that don't use runc, should we somehow check that and not generate that file?

  2. In follow up to 1, I recall that we use the file at /run/runc/container/state.json for some things before, would that be affected?

  3. In the info file, we still have:
    "EXEC_START": "/bin/runc run 'azure'"
    "EXEC_STOP": "/bin/runc kill 'azure'"
    which doesn't match up to the service file

Perhaps I'm misunderstanding the method of running said containers and we shouldn't be using the atomic CLI. What do you think @giuseppe

@giuseppe
Copy link
Collaborator

  1. I think it is still useful to generate a default configuration OCI file, even if not present, so that users can manually edit it, I find it quite handly to test out with busybox. I've added a warning recently when the config.json.template file is missing so that it is clear the image won't automatically work as a system container.

2, yes, that is generated by atomic install --system but the containers are not forced to use it. For example we could have a oneshot system container that uses only EXEC_START

@rhatdan
Copy link
Member

rhatdan commented Mar 10, 2017

I would prefer if the info file would not talk about runc, or there was a way for the container image developer to indicate if he is using runc or not.

@ashcrow
Copy link
Collaborator

ashcrow commented Dec 11, 2017

@giuseppe is this still an ongoing conversation?

@giuseppe
Copy link
Collaborator

we have the possibility to specify another runtime now, so atomic is less bound to runc and bwrap-oci, if that helps for this issue

@ashcrow
Copy link
Collaborator

ashcrow commented Dec 11, 2017

@yuqi-zhang please advise

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

No branches or pull requests

4 participants