[COE-54] COE Gerrits should include a build job that verifies the Go code Created: 16/Jan/19  Updated: 19/Jan/19

Status: Open
Project: coe
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Medium
Reporter: Michael Vorburger Assignee: Prem Sankar
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Blocks
blocks COE-10 COE Deployment and Packaging for Kube... Open

 Description   

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.



 Comments   
Comment by Michael Vorburger [ 19/Jan/19 ]

It occurred to me that CSIT jobs today probably already build the Go code (and with COE-55 certainly will), so the point of having such a new job, or even adding this to the existing verify job, would just be to catch anything broken in the Go code (and the container builder) faster - which would be cool & useful IMHO (but it's not super critical).

Generated at Wed Feb 07 19:15:30 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.