Commit graph

168 commits

Author SHA1 Message Date
Dhilip
67af4998de helm create all defined template names should be namespaced … (#2845)
* helm create all defined template names should be namespaced https://docs.helm.sh/chart_best_practices/#names-of-defined-templates

* Update Doc

* Update example
2017-10-06 09:04:53 -06:00
Matthew Fisher
e3e1116eaa Merge pull request #2857 from llsheldon/fix-dependency-alias
fix(helm):Fix dependency aliaes not working
2017-10-03 10:59:11 -07:00
Matthew Fisher
f278675f98 remove references to facebookgo/symwalk 2017-09-26 15:21:40 -07:00
Matthew Fisher
211f5f6f33 Revert "fix(helm): resolve symlinks when loading chart"
This reverts commit edd4e56112.
2017-09-26 15:02:33 -07:00
Matthew Fisher
cf3fc21e81 Merge pull request #2898 from zlabjp/kube-version
feat(helm): add --kube-version flag to helm template
2017-09-25 15:19:05 -07:00
carlory
ad2d8e434a fix fromjson comment 2017-09-18 15:46:36 +08:00
carlory
b9d3974df0 fix func comment 2017-09-18 15:38:35 +08:00
Kazuki Suda
fa3ee5aecf feat(helm): add --kube-version flag to helm template
This commit adds --kube-version flag to helm template. It allows you to
override the Kubernetes version used as Capabilities.KubeVersion.Major/Minor (e.g. 1.7).
2017-09-11 11:22:00 +09:00
Kazuki Suda
3508cebbf6 Use the same defaults as done in helm lint for Capabilities 2017-09-11 10:09:42 +09:00
Michelle Noorali
84bb77f183 Merge pull request #1997 from AmandaCameron/issue/1401
Take a crack at Issue #1401
2017-09-08 07:58:37 -06:00
Amanda Cameron
415e52bf55 Add support for sub-directory scanning as in issue #1401 2017-09-08 09:33:44 -04:00
llsheldon
0440b54bbf fix(helm):Fix dependency aliaes not working
The alias functionality only works when a hardcoded version
is used. Any use of semver logic causes unexpected behavior.
I use version.IsCompatibleRange to check the dependency version.

Closes #2794
2017-09-07 19:46:13 +08:00
Adnan Abdulhussein
8fd8a7c324 fix(create): port-forward should use internal port
cc @james-w
2017-09-07 10:43:53 +01:00
Justin Scott
4a02a71f1e WIP feat(helm): add template command
This adds the functionality from the helm-template plugin to allow the
rendering of templates without Tiller.

Closes #2755
2017-08-28 18:04:56 -07:00
Kira
4c1a47480e feat(2450): add annotations to chart metadata (#2671)
* feat(2450): add options to chart metadata
2017-08-16 18:27:13 -04:00
Maxime Guyot
bbf2d6b7e8 Fix the default NOTES.txt 2017-08-09 13:29:51 +02:00
Miouge1
6cfed9d635 Updating default values.yaml comments
Following the instructions to simply uncomment the following lines results in a invalid yaml output
2017-07-27 10:53:52 +02:00
Steven E. Harris
e8d80729ac Merge pull request #2703 from rbwsam/ref/simplify_error
ref(pkg/chartutil): replace use of 'fmt.Efforf'
2017-07-25 10:14:32 -04:00
Sam Leavens
098df8b186 ref(pkg/chartutil): improve comment spacing
Reduce comment spacing to single tab.
2017-07-20 14:21:40 -07:00
Sam Leavens
f7ca52dfc8 ref(pkg/chartutil): improve comment for PathValue
Make comment for PathValue more explanatory.
2017-07-20 13:59:28 -07:00
Sam Leavens
cf09e344d1 fix(pkg/chartutil): Fix test, improve message
Fix inverted equaility check and update error message.
2017-07-20 13:19:48 -07:00
Sam Leavens
8df52b6501 ref(pkg/chartutil): move comment
Move comment out of test. Move ref location in test.
2017-07-20 13:10:59 -07:00
Sam Leavens
3ee70291c8 ref(pkg/chartutil): Capitalize YAML
Capitalize "yaml" in error message. Remove empty var from test error message.
2017-07-20 11:52:49 -07:00
Sam Leavens
51b59497ee ref(pkg/chartutil): decrease map lookups
Decrease map lookups in `chartutil.coalesceValues` and move comment to unit
test.

Closes #2663
2017-07-19 17:55:11 -07:00
Sam Leavens
9b8928edf4 ref(pkg/chartutil): replace use of 'fmt.Efforf'
'fmt.Errorf' is unnecessary when checking for 0 length path in Values.PathValue
due to the lack of formatting. Add test covering changes.
2017-07-19 15:53:12 -07:00
Scott Rigby
6dc31f8e96 Test that the coalesced key is properly removed for all YAML null syntax options, but not for empty string 2017-07-07 14:32:56 -04:00
Scott Rigby
645f01eb22 New Helm functionality to delete a coalesced YAML key when the value is nil.
- Note that this covers all YAML null syntax options:
  ref: http://yaml.org/type/null.html
- Note that we do a nil comparison because the encoding/yaml package parses
  YAML properly and any variation of null, Null, NULL, or ~ is converted to nil
  by the time we get here.
2017-07-07 14:32:56 -04:00
Matt Butcher
883da97396 Merge pull request #2607 from sushilkm/issues/2599
Adds charts in "charts\" directory to dependencies
2017-07-06 21:27:37 -06:00
Ilya Semenov
13b15f5662 Remove service.type mention from ingress docs 2017-07-02 16:04:50 +07:00
Justin Scott
af4c243ee3 Correct punctuation and capitalization for user facing strings. 2017-06-26 10:23:26 -07:00
Sushil Kumar
82da721c0a 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-06-22 10:44:20 -07:00
Sushil Kumar
7ce5c7925b 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-06-21 15:40:37 -07:00
Sushil Kumar
bfd74c35e9 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-06-21 11:18:27 -07: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