Commit Graph
138 Commits
Author SHA1 Message Date
Sushil Kumar ebbf1b0022 Added tests for different combinations of subcharts & requirements.yaml
Added more tests to test following comibnations of subcharts and requirements.yaml
- [ ] subcharts not specified in requirements.yaml
- [ ] some subcharts specified in requirements.yaml while others should be picked directly from "charts\"
- [ ] all subcharts specified in requirements.yaml without alias
- [ ] subcharts specified in requirements.yaml with alias is already tested
2017-07-12 13:04:09 -06:00
Sushil Kumar 39525ce7cc Adds charts in "charts\" directory to dependencies
Dependencies should be processed even if they are not added to requirements.yaml
Fixes https://github.com/kubernetes/helm/issues/2599
2017-07-12 13:03:52 -06:00
Sushil Kumar f4a1018512 Added omitempty to Requirements struct
This was needed to get correct sha for requirements.yaml and requirements.lock
Fixes https://github.com/kubernetes/helm/issues/2598
2017-07-12 12:21:07 -06:00
Reinhard Nägele 76e6e15a0c Add comment on resources 2017-06-18 14:42:36 +02:00
Reinhard Nägele 51923e54b1 Improve 'helm create' 2017-06-16 09:41:38 +02:00
Taylor Thomas e3250e3319 Merge pull request #2534 from sushilkm/issues/2508
First alias would be dependency rename
2017-06-06 22:33:51 -07:00
Sushil Kumar 716be14ad3 Updated code to read requirements.yaml as per following structure
```
- name: <dependency-chart-name>
  alias: <alias-name-to-be-used>
  version: <dependency-chart-version>
  repository: <dependency-chart-version>
```
2017-06-05 14:35:44 -07:00
Sushil Kumar 541d052202 First alias would be dependency rename
Fixes https://github.com/kubernetes/helm/issues/2508
2017-06-05 14:23:05 -07:00
Justin Scott 40052d3e8f fix(helm): remove unnecessary values merge in processImportValues
The function chartutil.processImportValues was merging values unnecessarily and incorrectly merging --set values.

Closes #2331
2017-06-01 22:17:14 -07:00
Sushil Kumar 42aa81e7ca Added tests for alias(es) for chart dependencies 2017-05-25 14:18:32 -07:00
Sushil Kumar 34f9c67e04 Shallow copy existing chart details 2017-05-25 10:45:18 -07:00
Sushil Kumar 156d48bc3b Adds alias for dependencies
Fixes https://github.com/kubernetes/helm/issues/2383

Sample `requirements.yaml` I used for test purpose was for wordpress chart

```dependencies:
- name: mariadb
  version: 0.5.10
  repository: https://kubernetes-charts.storage.googleapis.com/
  alias:
  - db1
  - db2
  - db3
```
2017-05-23 15:41:08 -07:00
lead4good bb4be33331 fixed fromYaml | toJson 2017-05-18 12:26:53 +02:00
Adnan Abdulhussein f8b2c5eb57 fix(create): incorrect URL in default NOTES.txt 2017-05-02 19:15:18 +01:00
Matt Butcher 46035c35c4 fix(tiller): fix TOML panic
This fixes a TOML panic by replacing one parser library with another.
The older library did not gracefully handle reflection issues, and so
was prone to panic. The new one is not great, but it doesn't seem to
panic.

Closes #2271
2017-05-01 11:25:51 -06:00
Matthias Thubauville ce505f16e6 Fix indent for defaultDeployment 2017-04-19 09:50:11 +02:00
Sushil Kumar 99065aa331 Validates chart-dir before running dependency list command
Fixes issues/2276: helm dependency list hangs if run on large directory
2017-04-17 14:23:04 -07:00
Taylor Thomas d03311b45a Merge pull request #2188 from scottrigby/chartutil-ingress-example
chartutil create ingress example
2017-04-13 09:21:24 -07:00
Matt Butcher 19b111b123 fix(helm): fix nil pointer in requirements.go
This fixes a segfault that was the result of assigning to a property of
a nil pointer.

Closes #2244
2017-04-12 12:06:33 -06:00
Adam Reese 85c41ef11a ref(*): kubernetes v1.6
Rebase on kubernetes v1.6
2017-04-11 15:42:23 -07:00
Scott Rigby 5bdcbee9af Change secret name to something more standard 2017-04-08 23:17:51 -04:00
Scott Rigby 2be4547d13 Change to hosts array, and parameterize serviceName and servicePort 2017-04-08 23:17:23 -04:00
Justin Scott 8c720ee2c2 fix (pkg/chartutil): correctly parse input values for ProcessRequirementsEnabled
Fixes a bug where tags and conditions specified in values.yaml were not being respeceted, causing incorrect loading of subcharts.

Closes #2139
2017-04-04 13:40:13 -07:00
Justin Scott 75ea566413 Correct indention of YAML field in subchartB 2017-03-31 10:46:50 -07:00
Justin Scott 31e57d8921 Remove commented code 2017-03-31 10:25:59 -07:00
Justin Scott 3bf143f052 Fix codefences and nits in charts.md. Correct whitespace in charts. Add clarity to description of ImportValues requirements field. 2017-03-31 10:19:08 -07:00
Justin Scott 7ea4d8c7c4 Refactor so parent's values win 2017-03-31 10:19:08 -07:00
Justin Scott 4a5721fb36 Fixup style and errors 2017-03-31 10:19:08 -07:00
Justin Scott 007bb9dbae Implement 'exports' convetion for simple list items 2017-03-31 10:19:08 -07:00
Justin Scott 2bd4d1d003 Cleanup old todo, unused log and value for ImportValues feature 2017-03-31 10:19:08 -07:00
Justin Scott d1424f6c08 Handle missed error and make error messages unique 2017-03-31 10:19:08 -07:00
Justin Scott 0e81899f5f WIP feat(helm): import child values to parent
Implements a mechanism in requirements.yaml to allow the import
and re-parenting of value table from child chart.

Closes #1995
2017-03-31 10:19:08 -07:00
Scott Rigby 22cec7cdc1 Fix spacing 2017-03-25 20:01:08 -04:00
Scott Rigby 37fd7df627 chartutil create ingress example 2017-03-25 17:42:37 -04:00
Matt Butcher df827ab738 Merge pull request #2050 from jackzampolin/jz-to-toml
Add toToml template function
2017-03-06 14:37:55 -07:00
Jack Zampolin fc60d51be9 Add toToml function 2017-03-01 10:19:21 -08:00
Matthew Fisher de6fb23e7a expose LoadFiles as a usable function
In third-party libraries, charts can be embedded in memory rather than in files, directories or
tarballs. Exposing LoadFiles allows a third-party library the ability to load static templates
in and spit out a *chart.Chart.
2017-02-28 15:47:40 -08:00
Maxim Ivanov edd4e56112 fix(helm): resolve symlinks when loading chart
Possible fix for #1639
2017-02-23 23:09:57 +00:00
YuviPanda cd7275069e Use backtick quotes to avoid excess escaping 2017-02-21 12:59:16 -08:00
YuviPanda 97a655d225 Add tests for ToJson 2017-02-21 01:08:18 -08:00
YuviPanda 5c52fe6ad6 Add ToJson and FromJson template functions 2017-02-21 00:34:17 -08:00
Justin Scott 17c6ae61da feat(helm): re-enable log warnings for tags and conditions 2017-02-11 13:39:15 -08:00
Justin Scott fe864ad699 feat(helm): remove logging of tags/condition warnings to pass ci 2017-02-11 12:56:43 -08:00
Justin Scott 58c8aca1cc feat(helm): fixup if/ele,remove extra string casts, add comments 2017-02-11 12:56:43 -08:00
Justin Scott 004c5bcc88 feat(helm): fix condition when no requirements.yaml exists during tag/condition processing 2017-02-11 12:56:43 -08:00
Justin Scott faae1f68b6 feat(helm): Check len of correct slice 2017-02-11 12:56:43 -08:00
Justin Scott 8ef733ca7d feat(helm): add conditions and tags
This feature adds the ability to selectively control the loading of charts using entries in top chart's values.
When 'helm install --set tags.mytag=true', charts with that tag will be enabled unless disabled in parent by condition.
When 'helm install --set mychart.enabled=true', charts with that yaml path specified will be enabled.

Closes #1837
2017-02-11 12:56:43 -08:00
z00358552 b0d4cacd54 fix(helm) change name length limit to 63 for helm create
since kubernetes 1.4, the upper limit for resource name is 63 characters,
so it is better to change the template when running `helm create` accordingly.
2017-01-17 15:51:31 +08:00
Lachlan Evenson 4d5cf66308 removed Version from create tests as output schema and values aren't tested 2017-01-11 12:22:21 -08:00
Lachlan Evenson 2e0f949d49 bugfix(tiller): replace + with - in Chart version to support Kubernetes constraint 2017-01-11 11:21:14 -08:00