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

Cannot build kata-runtime with go1.18 #4194

Closed
zvonkok opened this issue May 3, 2022 · 2 comments
Closed

Cannot build kata-runtime with go1.18 #4194

zvonkok opened this issue May 3, 2022 · 2 comments
Labels
bug Incorrect behaviour needs-review Needs to be assessed by the team.

Comments

@zvonkok
Copy link
Contributor

zvonkok commented May 3, 2022

When building with go1.18,

error obtaining VCS status: exit status 128
	Use -buildvcs=false to disable VCS stamping.

To resolve this one needs to set --buildvcs=false , related golang issue: golang/go#51748.

Are we going to include any version control information into the binaries build?

Related release notes: https://tip.golang.org/doc/go1.18

We may add GOFLAGS to the Makefile to turn off or turn on such features coming with new golang versions.

$ export GOFLAGS=-buildvcs=false
$ sudo -E PATH=$PATH make install  
...
	qemu hypervisor path (QEMUPATH) : /usr/bin/qemu-system-x86_64
	cloud-hypervisor hypervisor path (CLHPATH) : /usr/bin/cloud-hypervisor
	firecracker hypervisor path (FCPATH) : /usr/bin/firecracker
	acrn hypervisor path (ACRNPATH) : /usr/bin/acrn-dm
	assets path (PKGDATADIR) : /usr/share/kata-containers
	shim path (PKGLIBEXECDIR) : /usr/libexec/kata-containers

     INSTALL  install-scripts
     INSTALL  install-completions
     INSTALL  install-configs
     INSTALL  install-configs
     INSTALL  install-bin
     INSTALL  install-containerd-shim-v2
     INSTALL  install-monitor
@zvonkok zvonkok added bug Incorrect behaviour needs-review Needs to be assessed by the team. labels May 3, 2022
@liubin
Copy link
Member

liubin commented May 6, 2022

Hi @zvonkok thank you for this info.
Now we are using 1.17:

golang:
description: "Google's 'go' language"
notes: "'version' is the default minimum version used by this project."
issue: "https://github.com/golang/go/issues/20676"
uscan-url: >-
https://github.com/golang/go/tags .*/go?([\d\.]+)\.tar\.gz
version: "1.16.10"
meta:
description: |
'newest-version' is the latest version known to work when
building Kata
newest-version: "1.17.3"

When upgrading golang, the build flag should be changed too as you said.

@liubin
Copy link
Member

liubin commented Sep 21, 2022

Go version issue kata-containers/tests#5134

@zvonkok zvonkok closed this as completed May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Incorrect behaviour needs-review Needs to be assessed by the team.
Projects
Issue backlog
  
To do
Development

No branches or pull requests

2 participants