Sam Leavens
3ee70291c8
ref(pkg/chartutil): Capitalize YAML
...
Capitalize "yaml" in error message. Remove empty var from test error message.
8 years ago
Sam Leavens
51b59497ee
ref(pkg/chartutil): decrease map lookups
...
Decrease map lookups in `chartutil.coalesceValues` and move comment to unit
test.
Closes #2663
8 years ago
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.
8 years ago
Scott Rigby
6dc31f8e96
Test that the coalesced key is properly removed for all YAML null syntax options, but not for empty string
8 years ago
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.
8 years ago
Matt Butcher
883da97396
Merge pull request #2607 from sushilkm/issues/2599
...
Adds charts in "charts\" directory to dependencies
8 years ago
Ilya Semenov
13b15f5662
Remove service.type mention from ingress docs
8 years ago
Justin Scott
af4c243ee3
Correct punctuation and capitalization for user facing strings.
8 years ago
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
8 years ago
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
8 years ago
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
8 years ago
Reinhard Nägele
76e6e15a0c
Add comment on resources
8 years ago
Reinhard Nägele
51923e54b1
Improve 'helm create'
8 years ago
Taylor Thomas
e3250e3319
Merge pull request #2534 from sushilkm/issues/2508
...
First alias would be dependency rename
8 years ago
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>
```
8 years ago
Sushil Kumar
541d052202
First alias would be dependency rename
...
Fixes https://github.com/kubernetes/helm/issues/2508
8 years ago
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
8 years ago
Sushil Kumar
42aa81e7ca
Added tests for alias(es) for chart dependencies
8 years ago
Sushil Kumar
34f9c67e04
Shallow copy existing chart details
8 years ago
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
```
8 years ago
lead4good
bb4be33331
fixed fromYaml | toJson
8 years ago
Adnan Abdulhussein
f8b2c5eb57
fix(create): incorrect URL in default NOTES.txt
8 years ago
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
8 years ago
Matthias Thubauville
ce505f16e6
Fix indent for defaultDeployment
8 years ago
Sushil Kumar
99065aa331
Validates chart-dir before running dependency list command
...
Fixes issues/2276: helm dependency list hangs if run on large directory
8 years ago
Taylor Thomas
d03311b45a
Merge pull request #2188 from scottrigby/chartutil-ingress-example
...
chartutil create ingress example
8 years ago
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
8 years ago
Adam Reese
85c41ef11a
ref(*): kubernetes v1.6
...
Rebase on kubernetes v1.6
8 years ago
Scott Rigby
5bdcbee9af
Change secret name to something more standard
8 years ago
Scott Rigby
2be4547d13
Change to hosts array, and parameterize serviceName and servicePort
8 years ago
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
8 years ago
Justin Scott
75ea566413
Correct indention of YAML field in subchartB
8 years ago
Justin Scott
31e57d8921
Remove commented code
8 years ago
Justin Scott
3bf143f052
Fix codefences and nits in charts.md. Correct whitespace in charts. Add clarity to description of ImportValues requirements field.
8 years ago
Justin Scott
7ea4d8c7c4
Refactor so parent's values win
8 years ago
Justin Scott
4a5721fb36
Fixup style and errors
8 years ago
Justin Scott
007bb9dbae
Implement 'exports' convetion for simple list items
8 years ago
Justin Scott
2bd4d1d003
Cleanup old todo, unused log and value for ImportValues feature
8 years ago
Justin Scott
d1424f6c08
Handle missed error and make error messages unique
8 years ago
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
8 years ago
Scott Rigby
22cec7cdc1
Fix spacing
8 years ago
Scott Rigby
37fd7df627
chartutil create ingress example
8 years ago
Matt Butcher
df827ab738
Merge pull request #2050 from jackzampolin/jz-to-toml
...
Add toToml template function
8 years ago
Jack Zampolin
fc60d51be9
Add toToml function
8 years ago
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.
8 years ago
Maxim Ivanov
edd4e56112
fix(helm): resolve symlinks when loading chart
...
Possible fix for #1639
8 years ago
YuviPanda
cd7275069e
Use backtick quotes to avoid excess escaping
8 years ago
YuviPanda
97a655d225
Add tests for ToJson
8 years ago
YuviPanda
5c52fe6ad6
Add ToJson and FromJson template functions
8 years ago
Justin Scott
17c6ae61da
feat(helm): re-enable log warnings for tags and conditions
8 years ago
Justin Scott
fe864ad699
feat(helm): remove logging of tags/condition warnings to pass ci
8 years ago
Justin Scott
58c8aca1cc
feat(helm): fixup if/ele,remove extra string casts, add comments
8 years ago
Justin Scott
004c5bcc88
feat(helm): fix condition when no requirements.yaml exists during tag/condition processing
8 years ago
Justin Scott
faae1f68b6
feat(helm): Check len of correct slice
8 years ago
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
8 years ago
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.
8 years ago
Lachlan Evenson
4d5cf66308
removed Version from create tests as output schema and values aren't tested
8 years ago
Lachlan Evenson
2e0f949d49
bugfix(tiller): replace + with - in Chart version to support Kubernetes constraint
8 years ago
Matt Butcher
cb0a6c7e07
feat(tiller): add {{.Capabilities}} object
...
This adds the {{.Capabilities}} object to the template variables so that
chart authors can write charts that are aware of teh Kubernetes
capabilities of the current cluster.
Closes #1608
8 years ago
Matt Butcher
090e00ddf8
Merge pull request #1686 from kubernetes/feat/1604-fromyaml
...
feat(tiller): add fromYaml to template functions
8 years ago
Euan Kemp
25f74989a5
fix(chartutil): improve 'missing name' error
8 years ago
Matt Butcher
b3d812b346
Merge pull request #1692 from technosophos/fix/allow-global-map-merges
...
fix(tller): allow deep merge of global maps
8 years ago
Matt Butcher
6faf4675ac
fix(tller): allow deep merge of global maps
...
This reverts a previous decision to only do shallow merges of globals.
It allows globals to be nested maps.
8 years ago
Matt Butcher
b2d762af48
feat(tiller): add .Release.IsInstall
8 years ago
Matt Butcher
de7c5e5a38
feat(tiller): Add .Release.Revision and .IsUpgrade
...
New properties are available to template developers.
Closes #1554
8 years ago
Matt Butcher
7f4ea91028
Merge pull request #1645 from rodcloutier/Windows_archive_backslash_support
...
fix(helm): added support for compressed charts containing \ in file paths
8 years ago
Matt Butcher
d8540d78f1
feat(tiller): add fromYaml to template functions
...
This adds a fromYaml template function.
Closes #1604
8 years ago
Andrew Stuart
1df13df88d
Include lines functions
8 years ago
Andrew Stuart
6760aa1588
Add Lines method, remove path utils (Sprig pr)
8 years ago
Andrew Stuart
0474bcbcf1
Document ToYaml function
8 years ago
Andrew Stuart
9771973888
Add AsSecrets, AsConfig methods for Files object. Move ToYaml to chartutil
8 years ago
Adam Reese
0c6b6d1c62
fix(*): correct file permissions on source files
8 years ago
Rodrigue Cloutier
37920761a4
fix(helm): added support for compressed charts containing \ in file paths
8 years ago
Matt Butcher
69e50abc6c
Merge pull request #1628 from rodcloutier/Windows_chart_creation_support
...
test(*): fix Windows tests
8 years ago
Rodrigue Cloutier
f3b205c6ef
fix(*): fixed tests for Windows
8 years ago
John Hofman
c38fb120bf
fix(helm): add trimSuffix to helper functions
8 years ago
Matt Butcher
e26d9f9633
Merge pull request #1616 from rollulus/no-exec-yaml
...
fix(helm): write yaml without exec permission
8 years ago
Andrew Stuart
ec020a9e40
Improve formatting of godoc
8 years ago
Andrew Stuart
2de1728f05
feat(helm): Add Files.Glob method to permit file organization
8 years ago
Rollulus
e8eed3cc3e
fix(helm): write yaml with non-exec permission
8 years ago
Matt Butcher
5517d00a48
Merge pull request #1547 from technosophos/feat/packs
...
feat(helm): support 'helm create --starter=mypack'
8 years ago
Matt Butcher
784a339627
feat(helm): support 'helm create --pack=mypack'
...
This adds support for packs, pre-configured chart patterns that can be
used to quickly create a custom layout for your new chart.
8 years ago
Adam Reese
62df1ad358
Merge pull request #1549 from adamreese/fix/1519-bubbly-yaml-errors
...
fix(tiller): return yaml parsing errors
8 years ago
Adam Reese
e2ab407209
fix(tiller): return yaml parsing errors
...
closes : #1519
8 years ago
Matt Butcher
0a163a2a3a
fix(helm): helm create will not overwrite existing files
...
It will ignore files if they exist, and create files if they were
missing.
Closes #1515
8 years ago
Adam Reese
d59f66fc9e
fix(*): fix new linter errors
8 years ago
Sebastian Mandrean
b3d45aef7a
fix(1516): use correct value reference in boilerplate NOTES.txt
8 years ago
Vic Iglesias
4179282760
Dont use privileged port in port-forward of default chart
8 years ago
Vic Iglesias
5af08fa5b5
Fix labels in NOTES.txt
8 years ago
Vic Iglesias
8527f2f96c
Use chart name for container name
8 years ago
Vic Iglesias
156d706dd1
Use internal port for health check
8 years ago
Jim
7a054e0e6b
fix(pkg/chartutil): add error if chart yaml not in directory
...
For archived files the Chart.yaml file should be contained in a base
directory. This commit adds an error when the Chart.yaml file is found
in the root directory.
Fixes #1171
8 years ago
Vic Iglesias
665615f30f
Remove **** from default notes
8 years ago
Vic Iglesias
2b2e1d59c8
Simplify final logic block in create
8 years ago
Vic Iglesias
06d9ccd038
Fix formatting
8 years ago
Vic Iglesias
cd34492b5a
Add tests for new files in create
8 years ago
Vic Iglesias
8d867657bb
Add resources and probes
8 years ago
Vic Iglesias
4120e67240
Add deployment, service and NOTES to create
8 years ago
Matt Butcher
2388e71528
fix(helm): ignore dotfiles in charts/ directories
...
This causes 'helm dep [up|install]' to ignore files in charts/ that
start with either a dot or an underscore. It also changes the
chartloader to ignore those files.
Also, if a 'helm dep up' does not find a charts/ directory, it creates
one.
Closes #1342
8 years ago
Adam Reese
f71230ccd3
fix(*): resolve go linter issues
8 years ago
Matt Butcher
3e3312cea0
feat(tiller): re-use values during upgrade
...
When `helm install -f foo.yaml bar` is called, and then the release is
upgraded with `helm upgrade happy-panda bar`, this will now re-use the
values that were submitted with `-f foo.yaml`. The same is true for
values specified with `--set`.
Closes #1227
8 years ago
Matt Butcher
e0227c7510
fix(chart): Chart.yaml now has an apiVersion field.
...
This is to future-proof charts, and also be consistent with repo and
index YAML files.
Closes #1264
8 years ago
Matt Butcher
68dd4c9a66
fix(proto): remove unused fields
...
Some fields were introduced to Chart metadata after Alpha.4, but are not
going to be used ever. So we made the decision to remove them before we
get stuck with ugliness.
This should not break compatibility, since these fields were not used.
8 years ago
Matt Butcher
593718d749
feat(helm): add 'helm dependency' commands
...
This also refactors significant portions of the CLI, moving much of the
shared code into a library.
Also in this release, a testing repository server has been added.
8 years ago
Matt Butcher
a5921faf99
feat(chartutils): add support for requirements.yaml
8 years ago
vaikas-google
6d26024a27
first cut of version on client and server
8 years ago
Matt Butcher
34577d1ebc
feat(charts): add 'dependencies:' to Chart.yaml
...
This feature adds a dependencies section to a chart file. It is a
prerequisite for adding automated chart management tooling as described
in #874 .
8 years ago
joe2far
96de0e35aa
feat(helm): ignore by default all dotfiles in templates/
8 years ago
Matt Butcher
a46a033de4
feat(chart): support 'image:URL' in charts.
...
Closes #1034
8 years ago
Matt Butcher
2e95230b30
fix(helm): fix helmignore evaluation of dirs
...
This adds a few extra settings to the default .helmignore file. In
doing this, I found a bug that some directory patterns are not
evaluated correctly. Fixed that and added tests.
Closes #989
Closes #1027
9 years ago
Matt Butcher
b7945d05c4
ref(templates): change GetString to Get
...
This changes "pkg/chartutil".Files.Get to return a string, removes
"pkg/chartutil".Files.GetString, and adds
"pkg/chartutil".Files.GetBytes.
Closes #1020
9 years ago
Matt Butcher
033dbfe75e
feat(tiller): add Files map to templates
...
Templates can now access the non-template files in a chart by using
the '{{.Files}}' map inside of a template.
Relates to #950
9 years ago
Matt Butcher
95eeba3805
fix(tiller): merge -f values correctly
...
This fixes a bug in which passed-in values files were not correctly
merged into the chart's default values YAML data. I believe it also
fixes some other prioritization bugs in values merging.
The existing unit test was wrong (see TestCoalesceValues). It is
fixed now. Also added more tests to simulate issue #971 .
In the course of writing this, I removed some vestigial code as
mentioned in #920 .
Closes #971
Closes #920
9 years ago
Adam Reese
a14e76a65c
chore(*): add canonical import path annotation
...
https://golang.org/doc/go1.4#canonicalimports
9 years ago
Matt Butcher
310ef9bbcb
fix(helm): fix linter test panic
...
Handle a previously unhandled error in the linter. This simply bails out
if a chart's values files do not parse.
Also, changed the implementation of CoalesceValues to return a map even
on error.
9 years ago
Miguel Ángel Martínez Triviño
81fd9edf4d
Merge pull request #899 from migmartri/897-values-refactorign
...
Support Linter for Values namespace
9 years ago
Adnan Abdulhussein
9b257077d5
Merge pull request #915 from prydonius/fix/912-nil-map-readvalues
...
fix(chartutil): Ensure ReadValues doesn't return a nil map
9 years ago
Adnan Abdulhussein
4ef61a7136
fix(chartutil): Ensure ReadValues doesn't return a nil map
9 years ago
Matt Butcher
cd75b6a781
Merge pull request #902 from technosophos/feat/894-template-info-in-templates
...
feat(tiller): add .Template object in templates
9 years ago
Miguel Martinez
12aa72f121
Replacing options interface argument
9 years ago
Matt Butcher
532f03ec78
feat(tiller): add .Template object in templates
...
This allows templates to access information about the template file.
Right now, the template can only access the .Template.Name, which is the
chart-relative path to the current template.
Closes #894
9 years ago
Miguel Martinez
7bb4893cad
Support Linter for Values
9 years ago
Michelle Noorali
bcffe8a3f2
fix(chartutil): fix Expand function
...
This ensures that all necessary directories
get created when expanding a chart in the
chartutil.Expand function. fixes #892
9 years ago
Matt Butcher
90c46e896d
fix(chartutil): fix Table() method to test Values
...
This makes the Table() method more flexible than the original version.
It allows either a map[string]interface{} or a chartutil.Values to be
treated as a table.
9 years ago
Matt Butcher
22ac61469f
feat(*): add Values namespace to templates
...
This adds the .Values namespace qualifier to all values
9 years ago
Miguel Martinez
03d27779d3
Templates Lint rules
...
Template rules
Adding chart errors
Added function that checks the existence of all the values in the templates
Adding chartfile unit tests
Testing runLinterRule
Fixing out of range
Fixing out of range
Improving quote detector
9 years ago
Matt Butcher
e757b24aed
fix(*): add license header
9 years ago
Matt Butcher
23ce6142f9
fix(chartutil): remove a wayward println
9 years ago
Matt Butcher
f30ff915f2
feat(helm): add 'helm get values --all' for all values
...
Add support for displaying computed configuration values. This is
useful for debugging.
9 years ago
Matt Butcher
57a32f1df8
Merge pull request #832 from technosophos/feat/helmignore
...
feat(pkg/ignore): add helmignore library
9 years ago
Matt Butcher
713020359b
feat(pkg/ignore): add helmignore library
...
This adds support for .helmignore files. These files roughly follow
the conventions established for .gitignore files:
https://git-scm.com/docs/gitignore
Closes #748
9 years ago
Matt Butcher
2b2a5788f3
Merge pull request #824 from technosophos/feat/global-vars
...
feat(chartutil): support global variables
9 years ago
Matt Butcher
88622a20da
fix(helm): read values.yaml instead of values.toml
...
Closes #822
9 years ago
Matt Butcher
60f5341b91
feat(chartutil): support global variables
...
This provides support for "global" variables. It does this by
declaring "global" to be a special namespace. It then copies this
namespace into every subchart, coalescing it into any "global"
namespace found there.
The net result is that if "global.foo" is set in the YAML file, it
will be available to every chart/subchart as ".global.foo" regardless of
where that chart is in the subchart tree.
9 years ago
Matt Butcher
490cef784c
fix(tiller): refactor template render to use chartutil.
9 years ago
Matt Butcher
e8109048a9
fix(chartutil): move values coalescing into chartutil
9 years ago
Adam Reese
d95a144563
fix(chartutil): fix nil error on values
...
Fixes: https://github.com/kubernetes/helm/issues/803
9 years ago
Matt Butcher
377a43385e
fix(*): fixed import paths
9 years ago
Matt Butcher
72be00c6fc
fix(chartutil): update 'fetch' and 'package' to use chartutil.
9 years ago
Matt Butcher
ebd81152dd
fix(helm): update 'helm create' to use YAML.
9 years ago
Matt Butcher
9ca8c27e16
fix(*): change TOML to YAML
9 years ago
Matt Butcher
7aa4ffa4d4
feat(chartutil): switch TOML to YAML
9 years ago
Adam Reese
818c878700
chore(*): use k8s.io as the import path
9 years ago
Matt Butcher
15a1cb4816
feat(chartutil): add Save function
...
This replicates the previous Save function, which can save an
in-memory Chart proto to a tgz.
9 years ago
Matt Butcher
2802fcbd87
feat(chartutil): optimize for re-use
...
This is a refactor of the loader to use in-memory buffers instead of
trying to optimize for memory usage by delaying reads until the last
possible moment. Since charts tend to stay well below 1M in size, this
makes more sense, and is easier to work with.
9 years ago
Matt Butcher
25053e6ada
feat(pkg/chartutil): add chartutil package
9 years ago