-
Notifications
You must be signed in to change notification settings - Fork 554
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
runtime: create state directory and structure #41
Comments
I think it makes sense to embed the config into the state as is. |
Also I think we should allow implementation to store state in any desired form. It can be registry key for windows for example. But hooks will always consume text version of state, so info about storage method will never leak outside implementation. |
What does "cgroup/resource information" mean? |
/cc @vmarmol |
@crosbymichael I think all the items in your lists make sense as state that could be passed to the hooks. |
On Tue, Jun 30, 2015 at 04:59:34PM -0700, Michael Crosby wrote:
I think embedding the initial config is less useful than embedding the |
We spoke about this in the meeting today. We don't think it makes sense to have here config where the source of truth is the kernel (e.g.: resource limits). We should have some immutable state which with some pre-defined operation can provide the current state. Otherwise keeping these in sync will be expensive and difficult to do. |
On Wed, Jul 22, 2015 at 09:58:45PM -0700, Victor Marmol wrote:
The kernel is enforcing those resource limits, so why not make it the
Do you have an example workflow where a container's initial resource
I'm not sure about difficult, but I'm not terribly worried about the |
@crosbymichael My concern with having device node, sysctl, rlmit, etc state parameters is that we are simply serializing Kernel features and state. Why not tell consumers to read from the source of truth instead? |
Gah, sorry for piling on I had this tab with a half finished response and the stuff from Victor hadn't loaded yet. |
Ya, in my PR for the state it is much more conservative and gives people just the information required to look in the correct locations. |
For supporting things like hooks and updates we need a way to define runtime state of the container. This can be used to pass information to hooks so that they know the runtime state of the container and also used to dynamically make changes to resource allocations by running an update command for the runtime and have it reconcile the changes against the container's current state and the update requested.
For the hooks usecase the state should contain information such as:
For supporting updates the state should container information such as:
Some of this information would be duplicated from the initial container's config so it maybe worth it to look into embedding the original config into the state structure.
The text was updated successfully, but these errors were encountered: