1
0
mirror of https://github.com/helm/chart-testing.git synced 2026-02-05 09:45:14 +01:00

8 Commits

Author SHA1 Message Date
Ben Lavery-Griffiths
5b4fb1a4cf Fix chart_schema.yaml import-values rule (#725)
The existing rule for validating dependency import-values worked well for either of these two cases:

**1. Importing exports**

```yaml
dependencies:
- name: dep1
  version: 0.14.1
  repository: oci://myrepo
  import-values:
    - data
```

**2. Importing child values into the parent**

```yaml
dependencies:
- name: dep1
  version: 0.14.1
  repository: oci://myrepo
  import-values:
    - child: default.data
      parent: myimports
```

However, it failed on the following valid usecase:

```yaml
dependencies:
- name: dep1
  version: 0.14.1
  repository: oci://myrepo
  import-values:
    - data
    - child: default.data
      parent: myimports
```

Giving the following error:

```
dependencies.0.import-values.1: '{'parent': 'myimports', 'child': 'default.data'}' is not a str.
dependencies.0.import-values.0 : 'data' is not a map
```

This is because the Yamale is validating on a list of strings *or* a list of `import-value`, where a list containing both is also valid.

This commit changes the rule to allow both methods to be in the `import-values` list.

Signed-off-by: Ben Lavery-Griffiths <ben@lavery-griffiths.com>
2025-06-02 10:40:03 +02:00
Kevin M Granger
f9ff9a054c Make description optional in chart_schema (#372)
`description` is optional:
https://helm.sh/docs/topics/charts/#the-chartyaml-file

Signed-off-by: Kevin M Granger <kgranger@redhat.com>
2022-01-07 12:55:31 +01:00
Ian Tewksbury
ba5aed29ec chart_schema.yaml - dependency.repository should not be required (#300)
when using a local chart there is no repository so this should be optional field.

Signed-off-by: Ian Tewksbury <itewk@redhat.com>
2021-03-12 13:27:43 +01:00
Minh-Danh
594c3b60d5 Make home URL optional and support dependencies (#260)
Signed-off-by: Reinhard Nägele <unguiculus@gmail.com>
2020-08-26 22:14:23 +02:00
Scott Leggett
72362eb645 Add type field to chart schema (#252)
Signed-off-by: Scott Leggett <scott@sl.id.au>
2020-08-09 16:23:11 +02:00
Jeff Billimek
67ec12d315 appVersion should be optional (#239) 2020-07-24 09:14:27 +02:00
Will
814526768c Adding the apiVersion as a required field in the Chart.yaml vali… (#222)^
Signed-off-by: Will Varney <github@willvrny.co.uk>
2020-03-31 16:53:13 +02:00
Reinhard Nägele
96d2ce7c0e Factor out testing code from charts (#1)
* [WIP] Factor out testing code from charts

* Minor various fixes

 Please enter the commit message for your changes. Lines starting

* Start readme

* Update readme and add GKE example

* Add CircleCI job for ShellCheck

* Install Helm and kubectl after other tools

Also: Hard-code version for kubectl.

* Fix readme

* Update maintainer validation

'chartlib::validate_maintainers' should not log an error if there's no maintainers
section. This is already covered by the schema validation.

* Add random suffix to namespaces

* Improve container logs output

* Improve logging

* Add --cleanup flag to 'helm test'

* Improve namespace cleanup

* Improve logging

* Add random suffixes to releases

* Update logging and Docker image version

* Update GKE example

* Fix Bash array declarations

* Remove user from Docker image

* Add summary

* Use kubectl rollout status for deployments

* Add some more fixes

* Build dependencies before linting

* Fix array initialization

* Increase sleep time for namespace polling

* Add missing colon

* Create v1.0.0

* Update Helm to v2.9.1

* Add missing local keywords

* Bump image version

* Update copyright headers
2018-06-06 09:42:53 -04:00