Uploaded image for project: 'coe'
  1. coe
  2. COE-54

COE Gerrits should include a build job that verifies the Go code

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Medium Medium
    • None
    • None
    • None
    • None

      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.

            prem.sankar.g@ericsson.com Prem Sankar
            vorburger Michael Vorburger
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated: