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
syscall: TestAmbientCapsUserns test failing with operation not permitted #34015
Comments
Also, please note that this is not a build failure (i.e.: the code does not compile). The package compiles just fine, this is just a test failing. You can run |
this is what we run...
|
interesting this fails also in pbuilder chroot configuration... |
I can provide information if needed! Such as kernel configuration (Ubuntu 18.04 amd64 default one), |
@LocutusOfBorg Yes please. It'll help with the investigation. Thanks. |
on a Debian/Ubuntu system (I'm using ubuntu 18.04 LTS amd64)
Of course, inside the chroot you can install/play with whatever you want, install git, download stuff and so on... |
inside an Ubuntu/Debian machine you probably just need to install "ubuntu-dev-tools" |
This is my default kernel configuration |
This also fails on launchpad (see https://launchpadlibrarian.net/440240117/buildlog_ubuntu-xenial-amd64.golang-1.13_1.13-1longsleep1+xenial_BUILDING.txt.gz for example) |
Launchpad has no support for userns and the tests in the Go source are insufficient to detect that it is not supported. This change disables those tests when building this package. Refrence: golang/go#34015
Change https://golang.org/cl/195738 mentions this issue: |
@LocutusOfBorg @longsleep Could you please check whether https://golang.org/cl/195738 fixes the test failure for you? |
Hello, looks like still failing...
|
In my case, the kernel is 5.3, so the patch is not skipping the test... FWIW we have new systemd 241, so maybe /tmp is not able anymore to fork processes? |
Thanks for the feedback @LocutusOfBorg. Reopening the issue for further investigation. |
Trying the instructions in #34015 (comment) I was able to reproduce this in a chroot. I think we need to check in go/src/syscall/exec_linux_test.go Lines 80 to 85 in 62fb079
@LocutusOfBorg do you only see the issue inside a chroot or also outside? |
Change https://golang.org/cl/196500 mentions this issue: |
TestAmbientCapsUserns also needs to be skipped, e.g. in case the test is run inside a chroot. Updates #34015 Change-Id: I53913432fe9408217edfe64619adbfd911a51a7a Reviewed-on: https://go-review.googlesource.com/c/go/+/196500 Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
@tklauser only inside chroots, Ubuntu builders can't build outside. In my machine, and local builds are good |
@LocutusOfBorg thanks for confirming. Thus considering this fixed by https://golang.org/cl/196500 which skips |
thanks! |
Hello, we had a few golang 1.13 releases since 20 September, but none including this patch... |
@LocutusOfBorg we generally don't backport fixes to tests to minor releases. |
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (
go env
)?Ubuntu devel, 19.10
What did you do?
trying to build the source code, I get
I suspect Ubuntu builders should ignore that test?
The text was updated successfully, but these errors were encountered: