Jeremy, Very nice patch - thanks for all your work and setting up the jobs in all the calls last week - this verify job is useful. Verified that a busted helm (on purpose) patch now gets caught during the helm-verify - before merge https://gerrit.onap.org/r/#/c/63641/ -{{ tpl (.Files.Glob +{{ tplx (.Files.Glob https://jenkins.onap.org/job/oom-master-verify-helm/209/console 00:18:20 [ERROR] templates/: parse error in "pomba/charts/pomba-search-data/templates/configmap.yaml": template: pomba/charts/pomba-search-data/templates/configmap.yaml:7: function "tplx" not defined
Verified - OK Issue closed /michael -----Original Message----- From: Jeremy Phelps via RT <onap-helpd...@rt.linuxfoundation.org> Sent: Wednesday, August 29, 2018 6:31 PM To: Michael O'Brien <frank.obr...@amdocs.com> Cc: onap-discuss@lists.onap.org Subject: [ONAP Helpdesk #60323] RE: [deploy] helm verify job failing - being worked on Hi All, Here is a patch that fixes the helm verify job to checkout the gerrit patch instead of tip of master. https://gerrit.onap.org/r/63627 Jeremy On Wed Aug 29 18:07:05 2018, frank.obr...@amdocs.com wrote: > Prudence Au, Mandeep and I triaged this. > > There were 2 issues > > 1. The oom-master-verify-helm job (new) does not cherry-pick changes > – it runs directly from master prior to merges > * (note that this helm verify job is new from last week – there > have not been a lot of merge testing – so this is part of verification > testing in a way) > * The effect of this is that a helm-verify job that should fail – > does not – later the master-merge-helm job fails > * Any attempt to fix the build can only be done in the small > window before the jobbuilder (verify-helm) runs and puts a -1 on the > review > > (Mandeep did one of these for us) > > 1. The copyright symbol caused issues in the make job inside of the > helm verify job > * Removing the symbol (in this case) worked locally but required > the quick slip under the jobbuilder to get a +1 > * Copying the CR symbol from another helm chart also worked > > > > Fix required: > > TODO: The verify-helm job needs to be adjusted so that it picks up the > review instead of running only on the merged master branch > > Thanks > /Mandeep, Prudence, Michael > > From: onap-discuss@lists.onap.org <onap-discuss@lists.onap.org> On > Behalf Of Michael O'Brien > Sent: Wednesday, August 29, 2018 4:22 PM > To: onap-discuss@lists.onap.org > Subject: [onap-discuss] [deploy] helm verify job failing - being > worked on > > OOM deployers, > There is currently an issue with one of the charts – the > effect of any broken chart breaks the make for all of onap – hence the > helm verify job. > We are looking at fixing this right now – usually this > “UTF-8 octet” means windows to unix DB char issues. > > The [make all] as part of the master-merge-helm correctly identified > this broken chart – thanks Jeremy of the LF. > > So the procedure for merging after talking with Mike and Mandeep now > included waiting for the helm verify jobs – and every developer that > gets their review merged should also check the post-merge helm job to > make sure the build is OK for the next developer. > > Let the jobbuilder kick in – it runs the helm verify – then merge the > code In this case patchset 3 was committed – helm verify [master > level] ran OK > > https://gerrit.onap.org/r/#/c/61111/3 > https://jenkins.onap.org/job/oom-master-verify-helm/182/ > > 14:20:59 [pomba] > > 14:20:59 make[1]: Entering directory '/w/workspace/oom-master-verify- > helm/kubernetes' > > 14:20:59 Hang tight while we grab the latest from your chart > repositories... > > 14:20:59 ...Successfully got an update from the "local" chart > repository > > 14:21:00 ...Successfully got an update from the "stable" chart > repository > > 14:21:00 Update Complete. ⎈Happy Helming!⎈ > > 14:21:00 Saving 1 charts > > 14:21:00 Downloading common from repo > http://127.0.0.1:8879<http://127.0.0.1:8879/> > > 14:21:01 Deleting outdated charts > > 14:21:01 ==> Linting pomba > > 14:21:01 [INFO] Chart.yaml: icon is recommended > > 14:21:01 > > 14:21:01 1 chart(s) linted, no failures > > 14:21:01 Successfully packaged chart and saved it to: > /w/workspace/oom-master-verify-helm/kubernetes/dist/packages/pomba- > 2.0.0.tgz > > 14:21:01 make[1]: Leaving directory '/w/workspace/oom-master-verify- > helm/kubernetes' > > 14:21:01 > > > The merge-helm was the issue - > https://jenkins.onap.org/job/oom-master-merge-helm/29/console > > 19:18:56 make[1]: Entering directory '/w/workspace/oom-master-merge- > helm/kubernetes' > 19:18:57 Error: error unpacking pomba-search-data in pomba: error > converting YAML to JSON: yaml: invalid leading UTF-8 octet > 19:18:57 Makefile:42: recipe for target 'dep-pomba' failed > > Looks like the issue was between the runs – something went in between > 14:21 and 19:18 – where the automated merge busted one of the yamls > > /michael > This message and the information contained herein is proprietary and > confidential and subject to the Amdocs policy statement, you may > review at https://www.amdocs.com/about/email-disclaimer > > This message and the information contained herein is proprietary and > confidential and subject to the Amdocs policy statement, > > you may review at https://www.amdocs.com/about/email-disclaimer > <https://www.amdocs.com/about/email-disclaimer> This message and the information contained herein is proprietary and confidential and subject to the Amdocs policy statement, you may review at https://www.amdocs.com/about/email-disclaimer <https://www.amdocs.com/about/email-disclaimer> -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#12134): https://lists.onap.org/g/onap-discuss/message/12134 Mute This Topic: https://lists.onap.org/mt/25088931/21656 Group Owner: onap-discuss+ow...@lists.onap.org Unsubscribe: https://lists.onap.org/g/onap-discuss/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-