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

Build breaking on Linux #231

Closed
moderation opened this issue Oct 7, 2015 · 4 comments
Closed

Build breaking on Linux #231

moderation opened this issue Oct 7, 2015 · 4 comments

Comments

@moderation
Copy link

Build from source failing on Linux. Xubuntu 15.04

--> Installing build dependencies
--> Building nomad
# github.com/hashicorp/nomad/client/executor
client/executor/exec_linux.go:213: cannot use uint16(resources.IOPS) (type uint16) as type int64 in assignment
# github.com/hashicorp/scada-client
../scada-client/client.go:145: c.client.NumStreams undefined (type *yamux.Session has no field or method NumStreams)
Makefile:34: recipe for target 'test' failed
make: *** [test] Error 1

I'm wondering if this is due to the upstream runc change mentioned in #220

@ghost
Copy link

ghost commented Oct 8, 2015

This is still happening with current master:

$ make test
--> Installing build dependencies
--> Building nomad
# github.com/hashicorp/nomad/client/executor
client/executor/exec_linux.go:213: cannot use uint16(resources.IOPS) (type uint16) as type int64 in assignment
make: *** [test] Error 1
$ make bin
==> Getting dependencies...
# github.com/hashicorp/nomad/client/executor
client/executor/exec_linux.go:213: cannot use uint16(resources.IOPS) (type uint16) as type int64 in assignment
make: *** [bin] Error 2
$

It looks like:

e.groups.BlkioWeight = uint16(resources.IOPS)

Is expected to be:

e.groups.BlkioWeight = int64(resources.IOPS)

Making this change allows the build to complete.

@dadgar
Copy link
Contributor

dadgar commented Oct 8, 2015

I think you are on an old version of lib container: https://github.com/opencontainers/runc/blob/master/libcontainer/configs/cgroup.go#L61

You can update your dependencies when working with Nomad with: make updatedeps

@dadgar dadgar closed this as completed Oct 8, 2015
@ghost
Copy link

ghost commented Oct 9, 2015

Looks like that was it, make updatedeps should be added to the README.md as it's not in there.

@github-actions
Copy link

I'm going to lock this issue because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants