Skip to content

Commit

Permalink
Update chart-testing schema to not require repository field in chart …
Browse files Browse the repository at this point in the history
…dependencies (#83)

* Update chart-testing schema to not require repository field in chart dependencies

Update our derived chart-testing schema ahead of an actual chart-testing
release. Upstream PR: helm/chart-testing#300

* patch original chart_schema yaml with upstream changes
  • Loading branch information
ubergesundheit authored Mar 25, 2021
1 parent 397eedc commit cf7cc81
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 1 deletion.
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,13 @@ Based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), following [Se

## [Unreleased]

### Changed

- Update our derived chart schema for chart-testing ahead of a chart-testing release to fix issues
with local subcharts requiring a `repository` field.

## [0.2.1] - 2021-03-23

### Added

- Support for `requirements.lock` file updating
Expand Down
5 changes: 5 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,11 @@ RUN apk add --no-cache ca-certificates curl \
tar -C /binaries --strip-components 1 -xvzf - docker/docker \
&& curl -SL https://github.com/kubernetes-sigs/kind/releases/download/v${KIND_VER}/kind-linux-amd64 -o /binaries/kind

# patch the ct chart_schema.yaml file ahead of next release to fix
# issue https://github.com/helm/chart-testing/issues/324
# upstream PR https://github.com/helm/chart-testing/pull/300
RUN sed -i 's| repository: str()| repository: str(required=False)|' /etc/ct/chart_schema.yaml

COPY container-entrypoint.sh /binaries

RUN chmod +x /binaries/*
Expand Down
2 changes: 1 addition & 1 deletion resources/ct_schemas/gs_metadata_chart_schema.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ maintainer:
dependency:
name: str()
version: str()
repository: str()
repository: str(required=False)
condition: str(required=False)
tags: list(str(), required=False)
enabled: bool(required=False)
Expand Down

0 comments on commit cf7cc81

Please sign in to comment.