Closed markdryan closed 6 years ago
@egernst, @mcastelino, @devimc Here's the proposed update to the clear-containers3-x workload. Let me know if you think this will cause any issues. It doesn't seem to be used much at the moment, so hopefully it won't.
@markdryan did you test CC 3.0 with golang 1.10 ? did you run all the tests ?
@devimc It turns out go vet 1.10 reports errors on the cc code. So I've downgraded the workload to use 1.9.2. Thanks for catching this.
lgtm
Commands in the MOTD don't seem to be quite correct:
rob@alarmed-arthur:~$ go get -d github.com/clearcontainers/runtime...
package github.com/clearcontainers/runtime...: cannot expand ... in "github.com/clearcontainers/runtime..."
rob@alarmed-arthur:~/go/src/github.com/clearcontainers/runtime$ sudo -E PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin make install-cc-system
make: go: Command not found
Makefile:38: arch/-options.mk: No such file or directory
make: *** No rule to make target 'arch/-options.mk'. Stop.
The directory where go is installed also needs to be in PATH
Might also be worth doing s/cor/cc-runtime/ in the docker systemd file as that seems to be what upstream does.
This commit makes a number of changes to the clearcontainers-3.x workload.