[COE-54] COE Gerrits should include a build job that verifies the Go code

Blocks — Sat, 19 Jan 2019 01:03:43 +0000

While working on ~~COE-30~~, it occurred to me that it's a great basis for adding non-regression to the Go build to all of our Gerrits... AFAIK for the moment the COE verify jobs only build the Java part.

Based on https://git.opendaylight.org/gerrit/#/c/79589/ (and it's parent pre-requisites) it may not be hard now to add a new job (via JJB ?) which just does make (or even directly docker build -t odlovs-cni .) in coe/odlCNIPlugin/odlovs-cni (and after COE-32 ditto for the watcher) ...

All we need is a Build VM which has Docker (or even better because much safer and more secure not Docker nor podman or podman-docker but just buildah).

NB: The build node VM does NOT, should not, need to have go and dep installed - that's (now, ~~COE-30~~) all self contained in the multi stage container build!

This issue is NOT the same as ~~COE-30~~, the point here is not the make container images available in a remote container registry (~~COE-30~~ will do that), but just the build on Gerrit changes.

I'm unlikely to have the spare cycles to work on this myself, but can answer Qs anyone who would like to take this on may have on how we could best go about this.

OpenDaylight JIRA

[COE-54] COE Gerrits should include a build job that verifies the Go code