@jreed:matrix.org | > <@fungicide:matrix.org> close. i left a review comment indicating what you'll want to adjust | 18: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.org | jreed: 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#L2 | 18:46 |
@fungicide:matrix.org | you can still override it like this: https://opendev.org/openstack/openstack-helm-infra/src/branch/master/playbooks/build-chart.yaml#L17 | 18:46 |
@jreed:matrix.org | Just stick that in my playbooks folder... okay | 18:47 |
@fungicide:matrix.org | yeah, it's just a string variable | 18:47 |
@jreed:matrix.org | It 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/!