Thursday, 2023-11-16

@jreed:matrix.org> <@fungicide:matrix.org> close. i left a review comment indicating what you'll want to adjust18:40
@fungi The version of helm that is used is OLD. It's got to be updated.
2023-11-16 18:39:33.694094 | debian-bullseye | Processing chart _pathType dir...
2023-11-16 18:39:33.694120 | debian-bullseye | Linting Helm Chart: ./tests/unit/resources/adminer/adminer
2023-11-16 18:39:33.694146 | debian-bullseye | Chart Directory: /home/zuul/src/opendev.org/starlingx/app-gen-tool/tests/unit/TEST_OUTPUT/app-adminer/FluxCD/charts
2023-11-16 18:39:33.694171 | debian-bullseye | ==> Linting ./tests/unit/resources/adminer/adminer
2023-11-16 18:39:33.694197 | debian-bullseye | [ERROR] Chart.yaml: apiVersion 'v2' is not valid. The value must be "v1"
2023-11-16 18:39:33.694223 | debian-bullseye | [ERROR] templates/: apiVersion 'v2' is not valid. The value must be "v1"
2023-11-16 18:39:33.694252 | debian-bullseye |
2023-11-16 18:39:33.694278 | debian-bullseye |
2023-11-16 18:39:33.694303 | debian-bullseye | Error: 1 chart(s) linted, 1 chart(s) failed
2023-11-16 18:39:33.694328 | debian-bullseye |
2023-11-16 18:39:33.694353 | debian-bullseye | Generating tarball for helm chart: adminer error!
2023-11-16 18:39:33.694391 | debian-bullseye | F
@jreed:matrix.org> <@jreed:matrix.org> @fungi  The version of helm that is used is OLD.  It's got to be updated.18:40
>
> 2023-11-16 18:39:33.694094 | debian-bullseye | Processing chart _pathType dir...
> 2023-11-16 18:39:33.694120 | debian-bullseye | Linting Helm Chart: ./tests/unit/resources/adminer/adminer
> 2023-11-16 18:39:33.694146 | debian-bullseye | Chart Directory: /home/zuul/src/opendev.org/starlingx/app-gen-tool/tests/unit/TEST_OUTPUT/app-adminer/FluxCD/charts
> 2023-11-16 18:39:33.694171 | debian-bullseye | ==> Linting ./tests/unit/resources/adminer/adminer
> 2023-11-16 18:39:33.694197 | debian-bullseye | [ERROR] Chart.yaml: apiVersion 'v2' is not valid. The value must be "v1"
> 2023-11-16 18:39:33.694223 | debian-bullseye | [ERROR] templates/: apiVersion 'v2' is not valid. The value must be "v1"
> 2023-11-16 18:39:33.694252 | debian-bullseye |
> 2023-11-16 18:39:33.694278 | debian-bullseye |
> 2023-11-16 18:39:33.694303 | debian-bullseye | Error: 1 chart(s) linted, 1 chart(s) failed
> 2023-11-16 18:39:33.694328 | debian-bullseye |
> 2023-11-16 18:39:33.694353 | debian-bullseye | Generating tarball for helm chart: adminer error!
> 2023-11-16 18:39:33.694391 | debian-bullseye | F
Otherwise its working.
@fungicide:matrix.orgjreed: that's apparently a configurable default (which seems advantageous to update): https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/ensure-helm/defaults/main.yaml#L218:46
@fungicide:matrix.orgyou can still override it like this: https://opendev.org/openstack/openstack-helm-infra/src/branch/master/playbooks/build-chart.yaml#L1718:46
@jreed:matrix.orgJust stick that in my playbooks folder... okay18:47
@fungicide:matrix.orgyeah, it's just a string variable18:47
@jreed:matrix.orgIt works! :) TY -  I was able to get an end-to-end coverage test rolling, so new contributors cannot let code coverage/pytest slide.  18:58

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!