Skip to content
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

Fix golang 1.10 build on Ubuntu with snap packages #4620

Merged
merged 1 commit into from
Jun 22, 2018

Conversation

joestringer
Copy link
Member

@joestringer joestringer commented Jun 22, 2018

Using -i in the "go build" arguments with Golang 1.10 on Ubuntu using
a snap package results in this error:

$ make -C daemon
  CHECK contrib/scripts/bindata.sh
  GO    daemon/cilium-agent
go build runtime/cgo: open /snap/go/2130/pkg/linux_amd64/runtime/cgo.a: read-only file system

According to the upstream issue, this is because it's attempting to rebuild
the core Golang packages on the system, and snap is preventing this via a
read-only file system to ensure consistency for the base Golang version.

The workaround suggested in the following issue is to remove -i.

golang/go#24674

For more information, see the Golang 1.10 release notes:
https://golang.org/doc/go1.10#build


This change is Reviewable

Using `-i` in the "go build" arguments with Golang 1.10 on Ubuntu using
a snap package results in this error:

    $ make -C daemon
      CHECK contrib/scripts/bindata.sh
      GO    daemon/cilium-agent
    go build runtime/cgo: open /snap/go/2130/pkg/linux_amd64/runtime/cgo.a: read-only file system

According to the upstream issue, this is because it's attempting to rebuild
the core Golang packages on the system, and snap is preventing this via a
read-only file system to ensure consistency for the base Golang version.

The workaround suggested in the following issue is to remove `-i`.

golang/go#24674

Signed-off-by: Joe Stringer <joe@covalent.io>
@joestringer joestringer added pending-review area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. labels Jun 22, 2018
@joestringer joestringer added this to the 1.2 milestone Jun 22, 2018
@joestringer joestringer requested a review from a team June 22, 2018 18:13
@joestringer joestringer requested review from a team as code owners June 22, 2018 18:13
@joestringer joestringer requested a review from a team June 22, 2018 18:13
@joestringer joestringer requested a review from a team as a code owner June 22, 2018 18:13
@joestringer
Copy link
Member Author

test-me-please

@aanm
Copy link
Member

aanm commented Jun 22, 2018

probably needs backport as will backport go 1.10?

@tgraf
Copy link
Member

tgraf commented Jun 22, 2018

Does this only affect developer builds or also the container image builds?

@tgraf tgraf merged commit e542412 into cilium:master Jun 22, 2018
@joestringer joestringer deleted the submit/build-i branch June 22, 2018 23:43
@joestringer
Copy link
Member Author

joestringer commented Jun 22, 2018

@tgraf Just developer builds on Ubuntu when using snaps for golang installation (not other methods).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/misc Impacts miscellaneous areas of the code not otherwise owned by another area.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants