Skip to content

Update libcontainer to 1597c68f7b941fd97881155d7f077852e2914e7b#9795

Merged
crosbymichael merged 1 commit intomoby:masterfrom
LK4D4:update_libcontainer
Dec 23, 2014
Merged

Update libcontainer to 1597c68f7b941fd97881155d7f077852e2914e7b#9795
crosbymichael merged 1 commit intomoby:masterfrom
LK4D4:update_libcontainer

Conversation

@LK4D4
Copy link
Copy Markdown
Contributor

@LK4D4 LK4D4 commented Dec 23, 2014

@LK4D4
Update libcontainer to 1597c68f7b941fd97881155d7f077852e2914e7b
50905a6 
This commit contains changes for docker:
* user.GetGroupFile to user.GetGroupPath docker-archive/libcontainer#301
* Add systemd support for OOM docker-archive/libcontainer#307
* Support for custom namespaces docker-archive/libcontainer#279, docker-archive/libcontainer#312
* Fixes #9699 docker-archive/libcontainer#308

Signed-off-by: Alexander Morozov <lk4d4@docker.com>
@tianon
Copy link
Copy Markdown
Member

tianon commented Dec 23, 2014

LGTM

@tianon
Copy link
Copy Markdown
Member

tianon commented Dec 23, 2014

Drone is sad...

@jessfraz
Copy link
Copy Markdown
Contributor

jessfraz commented Dec 23, 2014

I restarted, idk something in integration, all of integration-cli passed

On Tue, Dec 23, 2014 at 12:34 PM, Tianon Gravi notifications@github.com
wrote:

Drone is sad...

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

@LK4D4
Copy link
Copy Markdown
Contributor Author

LK4D4 commented Dec 23, 2014

Hah, passed

@jessfraz
Copy link
Copy Markdown
Contributor

jessfraz commented Dec 23, 2014

yayyyy LGTM

@crosbymichael
Copy link
Copy Markdown
Contributor

crosbymichael commented Dec 23, 2014

LGTM

crosbymichael added a commit that referenced this pull request Dec 23, 2014
@crosbymichael
Merge pull request #9795 from LK4D4/update_libcontainer
305c898 
Update libcontainer to 1597c68f7b941fd97881155d7f077852e2914e7b
@crosbymichael crosbymichael merged commit 305c898 into moby:master Dec 23, 2014
This was referenced Dec 27, 2014
Closed
Closed
@unclejack
Copy link
Copy Markdown
Contributor

unclejack commented Jan 5, 2015

Do we need this for 1.4.2?

@LK4D4
Copy link
Copy Markdown
Contributor Author

LK4D4 commented Jan 5, 2015

@unclejack I don't know, this fixing bad bug with exec, but exec is not critical part of docker.

@ggrandes
Copy link
Copy Markdown

ggrandes commented Jan 5, 2015

+1 for 1.4.2 :-)

@SvenDowideit SvenDowideit mentioned this pull request Jan 27, 2015
Merged
@fcamblor
Copy link
Copy Markdown

fcamblor commented Jan 27, 2015

Is there any workaround to be able to stop the running instance ? (specifying a timeout to docker stop doesn't help)

@LK4D4
Copy link
Copy Markdown
Contributor Author

LK4D4 commented Jan 27, 2015

@fcamblor You need to find cgroups for this container and remove them.
I think something like:

find /sys/fs/cgroup *<container-id>

should help find them, but I don't tried.

@fcamblor
Copy link
Copy Markdown

fcamblor commented Jan 27, 2015

Thanks, in fact, I were using boot2docker (mac install) and stopping/restarting boot2docker removed the running instance.

@chenhanxiao chenhanxiao mentioned this pull request Feb 3, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Cannot stop container XXX: no such process w/ 1.4.0

7 participants

@LK4D4 @tianon @jessfraz @crosbymichael @unclejack @ggrandes @fcamblor