Use uint64 for resources to keep consistency with runtime-spec

[filbranden]

This is a backport of opencontainers/runc#1375 and it fixes kubernetes/kubernetes#61474 (comment) but please note there were some follow ups to that commit in upstream runc, not sure if those should be considered here as well...

Signed-off-by: Qiang Huang [email protected]

(cherry-picked from commit 8430cc4)
Signed-off-by: Filipe Brandenburger [email protected]

[filbranden]

/cc @adelton

[vfreex]

Because of this issue, OpenShift/Kubernetes nodes can't be run on latest Fedora.
This is definitely a high priority bug. I am surprised this issue doesn't get much attention.

[dustymabe]

FYI: related BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1584909#c4

[runcom]

@mrunalp PTAL

[filbranden]

So this was fixed with 18eb957 which reverts the backports.

I think you might want to consider adding back the changes around StartTransientUnit, since there seems to be a race condition there. That's opencontainers/runc#1772 and two or three other PRs that precede it. See opencontainers/runc#1683 for the first fix in the series, which discusses the race condition a bit...

This one is no longer relevant, so closing. Thanks!

[runcom]

So this was fixed with 18eb957 which reverts the backports.

I think you might want to consider adding back the changes around StartTransientUnit, since there seems to be a race condition there. That's opencontainers/runc#1772 and two or three other PRs that precede it. See opencontainers/runc#1683 for the first fix in the series, which discusses the race condition a bit...

This one is no longer relevant, so closing. Thanks!

thanks for the pointer, do you have an ordered list of the fixes that must go in? I can't seem to follow every fix that went in 😕

[runcom]

@filbranden ^^

[filbranden]

@runcom If you want to fix the race condition, then you need:

• Fix race against systemd opencontainers/runc#1683
• Add timeout while waiting for StartTransinetUnit completion signal opencontainers/runc#1754
• Fix systemd.Apply() to check for DBus error before waiting on a channel. opencontainers/runc#1772
• Make channel for StartTransientUnit buffered opencontainers/runc#1781

But see also this comment, since it seems the two first PRs were merged here (or in another branch) and later reverted. They introduced bugs that were later fixed by the last two PRs.

You should check with @sjenning, @mrunalp, @vikaschoudhary16 and @derekwaynecarr to see whether these PRs should be backported here or not.

This is about the race with systemd and libcontainer trying to mkdir the directories in the cgroup tree. That's all this patch set fixes.

Cheers,
Filipe

[runcom]

Thanks a lot!!!

[dustymabe]

Because of this issue, OpenShift/Kubernetes nodes can't be run on latest Fedora.
This is definitely a high priority bug. I am surprised this issue doesn't get much attention.

@vfreex - can I get you to test/karma https://bodhi.fedoraproject.org/updates/FEDORA-2018-c2e93d5623 ?