Skip to content

Conversation

@mrunalp
Copy link
Contributor

@mrunalp mrunalp commented Mar 8, 2016

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

@vbatts
Copy link
Member

vbatts commented Mar 8, 2016

Oh good. I thought we had already done this!
LGTM

@wking
Copy link
Contributor

wking commented Mar 8, 2016

We probably also want to remove [1](found by grepping for /run).

@mrunalp mrunalp force-pushed the remove_state_dir branch from e12b371 to 16c0995 Compare March 8, 2016 18:36
@mrunalp
Copy link
Contributor Author

mrunalp commented Mar 8, 2016

@wking Updated.

@crosbymichael
Copy link
Member

LGTM

crosbymichael added a commit that referenced this pull request Mar 8, 2016
Remove the state directory as we now have a state operation instead
@crosbymichael crosbymichael merged commit 3b7c15d into opencontainers:master Mar 8, 2016
@crosbymichael crosbymichael added this to the v0.4.0 milestone Mar 8, 2016
@wking
Copy link
Contributor

wking commented Mar 8, 2016

On Tue, Mar 08, 2016 at 10:36:40AM -0800, Mrunal Patel wrote:

@wking Updated.

Thanks :). 16c0995 looks good to me.

wking added a commit to wking/opencontainer-runtime-spec that referenced this pull request Mar 9, 2016
Through 3b7c15d (Merge pull request opencontainers#334 from
mrunalp/remove_state_dir, 2016-03-08).

Signed-off-by: W. Trevor King <[email protected]>
vbatts pushed a commit to vbatts/oci-runtime-spec that referenced this pull request Mar 9, 2016
Through 3b7c15d (Merge pull request opencontainers#334 from
mrunalp/remove_state_dir, 2016-03-08).

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.

4 participants