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: [email protected] <[email protected]> On Behalf Of
Michael O'Brien
Sent: Wednesday, August 29, 2018 4:22 PM
To: [email protected]
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>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#12133): https://lists.onap.org/g/onap-discuss/message/12133
Mute This Topic: https://lists.onap.org/mt/25077611/21656
Group Owner: [email protected]
Unsubscribe: https://lists.onap.org/g/onap-discuss/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
