Matthew Fisher
077be881c4
Merge pull request #3283 from nicolerenee/concurrency
...
feat(pkg/helm): allow concurrent requests on a shared client
7 years ago
Reinhard Nägele
f47e1619b0
Update initial chart experience
...
* Add template for chart name
* Use named ports and simplify service configuration
* Add affinity
* Add tolerations
* Use 'with' statements where possible
* Enhance ingress
* Add fullnameOverride to fullname template
7 years ago
Matthew Fisher
f652f45398
Merge pull request #2784 from rocky-nupt/proposal-add-k8s-version
...
feat(tiller/proto):Allow charts to specify k8s compatibility
7 years ago
Justin Scott
fbe80437a4
fix(helm): fix importValues warnings from disabled charts
...
ImportValues incorrectly processes charts that should be disabled. This
patch excludes those charts.
Closes #2620
7 years ago
Matthew Fisher
e1027fae73
refactor symWalk() into pkg/sympath ( #3199 )
...
* refactor symWalk() into pkg/sympath
* add Go copyright
* remove dead code
7 years ago
xuhaigang
61623bfe2d
feat(tiller/proto):Allow charts to specify k8s compatibility in semver field
...
Add an optional field kubeVersion to Chart.yaml that allows the chart
developer to specify a semver compliant version.
Closes #2708 , #2706
7 years ago
Adam Reese
a31e3b8fe1
ref(*): kubernetes v1.9 support ( #3234 )
7 years ago
Brad Beam
8e0eface2c
Making starter templates more versatile
...
This adds support for changing '<CHARTNAME>'
occurances in starter chart to the destination
chart name
7 years ago
Tim Dumol
8fae16fbb2
feat(helm): add better error message to 'dep up'
...
When a user enters a non-URL (such as stable)
in requirements.yaml and tries to `helm dep up`,
they get a potentially confusing error message.
This tries to make the error message clearer.
Closes #2672
7 years ago
Matthew Fisher
78d6b930bd
Merge pull request #3061 from eicnix/feature/tpl-fix
...
fix(helm): Set template context inside tpl function to outer function.
7 years ago
Timofey Kirillov
13cb8d8244
Show kind in resource-not-found-in-release error
...
This error occures when resource is not found in helm release:
`Error: UPGRADE FAILED: no resource with the name "redis-cluster-sentinel" found`
Changed to:
`Error: UPGRADE FAILED: no ConfigMap with the name "redis-cluster-sentinel" found`
So now that resource can easily be found in cluster.
7 years ago
Tim Hockin
05d4b30730
Convert registry to k8s.gcr.io
7 years ago
Nicole Renée Hubbard
6ade38fb72
feat(pkg/helm): allow concurrent requests on a shared client
...
change to helm client to no longer save request options in the client
options. This resulted in options from a previous call still being
present in future calls.
Closes #3253
7 years ago
Andrey Klimentyev
cfb7dfa82c
Typo fix
7 years ago
Nauris Sadovskis
bd8178d50b
Fix package url
7 years ago
Matthew Fisher
51a6f3c326
Merge pull request #3247 from mgalgs/missing-repos-help
...
feat(repo): Improve error message for missing repositories.yaml
7 years ago
Julian
ca415ab8db
feat(helm): support namespace and additional values in lint ( #2972 )
...
* Add "--namespace" to helm lint
Keep lint syntax as close as possible to "helm install" resp. "helm
upgrade", so that one only needs to change the command.
See #2036
* Align lintCmd struct
* Add "--set" and "--values" to helm lint
Keep lint syntax as close as possible to "helm install" resp. "helm
upgrade", so that one only needs to change the command.
Closes #2495,#2036
* Reuse strict parameter, when rendering during lint
We want to see the rendering fail, if we missed a value, so we reuse
"--strict".
See #2495,#2036
* Fix lint unit test
See #2495,#2036
* Update docs
7 years ago
Mitchel Humpherys
6de70254fd
feat(repo): Add test for non-existent repo path
7 years ago
Mitchel Humpherys
7417f09293
feat(repo): Improve error message for missing repositories.yaml
...
New users to helm don't always run `helm init` (e.g. if their cluster
already has helm installed). The user's initial interaction with any of
helm's repository commands (e.g. `helm repo list`) will then be an error
message due to a missing repositories.yaml in their local config.
Give the user a little hint about how to fix the error without them having
to hunt through the man/help pages.
7 years ago
Matthew Fisher
1042f6118b
Merge pull request #3122 from cofyc/fix_charts_permission
...
Remove executable permission bit for charts downloaded from remote repos
7 years ago
Lukas Eichler
8bc7dede18
fix(helm): Apply PR comments for tpl template name fix
...
Modified existing unit test to verify the changed behavior.
Removed debug print.
7 years ago
Lukas Eichler
1cebc760a0
Fixed warning for missing formating parameter inside error message.
7 years ago
Lukas Eichler
2c338db1bd
fix(helm): Set template context inside tpl function to outer function.
...
docs(helm): Added documentation about tpl function
7 years ago
Matthew Fisher
618094ccd2
fix TLS default path
...
Without this change, running `helm list --tls` without setting $HELM_HOME causes helm to look for TLS certificates under /.
7 years ago
Taylor Thomas
04769b7c26
Merge pull request #3131 from fgimenez/fix-2745
...
check if resource is present before adding it to remaining or keep on deletion
7 years ago
Federico Gimenez
a607fe798d
iinitialize message only when at least one manifest is kept
7 years ago
Federico Gimenez
93bce130fb
fix signatures
7 years ago
Federico Gimenez
81eaf2f263
only check existence for the possibly kept manifests
7 years ago
Adam Reese
e93b77166d
Merge pull request #3025 from beeradb/fakeclient-releasestatus
...
Improvements to FakeClient: support all CRUD operations
7 years ago
Matthew Fisher
cad89240f3
Merge pull request #3187 from hoesler/fix/chart_downloader
...
fix(helm): resolve relative chart paths
7 years ago
Matthew Fisher
7c79d1c765
Merge pull request #3193 from burdiyan/master
...
ref(helm): improve fullname template in starter chart
7 years ago
Johnny Bergström
7cc610960f
Walk symbolic link directories
...
Continue walking symlinks to be able to use them
for sub-charts. Useful when not specifying requirements
but still want to share charts locally.
7 years ago
Alexandr Burdiyan
f432e07126
ref(helm): improve fullname template in starter chart
...
Closes #1186
7 years ago
Christoph Hösler
98e0bd2ecb
refactor: rename function and improve error messages
7 years ago
Christoph Hösler
af9190f956
feature: let FindChartInRepoURL return absolute chart URLs
7 years ago
Matthew Fisher
d655911088
Merge pull request #3041 from jlyheden/feature/plugin-http-installer
...
Add plugin installer from http archive
7 years ago
Helgi Þormar Þorbjörnsson
803f7c706e
add a keepalive of 30s to the client ( #3183 )
7 years ago
Johnny Bergström
d762a42168
Fix helmignore for .* ( #3114 )
...
When the first rule matching entry for top level dir
will be empty string, the .* rule will match.
Skip this entry as it's not needed for processing.
7 years ago
Hadrien Chauvin
838d780894
update google.golang.org/grpc to 1.7.2 ( #3101 )
7 years ago
Matthew Fisher
4c21a0e739
Merge pull request #3123 from Miouge1/fullnameoverride
...
Add fullnameOverride
7 years ago
Matt Cholick
fe3eeaf39d
fix(helm): Tunnel closing already closed channel ( #3157 )
...
k8s client-go closes the ready channel that's passed in (see https://github.com/kubernetes/client-go/blob/master/tools/portforward/portforward.go#L171 ) This means that Tunnel's Close will always panic, as the client-go library will have closed then channel. This isn't reproducible unless helm.Client is externally, as the helm cli runner doesn't actually invoke Close.
7 years ago
Kai Chen
bc45e90198
Remove unnecessary parentheses in switch statements ( #3175 )
7 years ago
Matt Farina
275fbd431c
feat(http): Add a Helm user-agent string to the http getter
...
Adding a user-agent to the http getter will enable servers to
distinguish between helm (including various versions) and other
tools connecting to the server.
7 years ago
Matt Butcher
e8e6ac5d77
Fix/missing ssl params ( #3152 )
...
* fix(helm): add TLS params back
During a recent refactor, several TLS flags stopped being processed for
a few of the commands. This fixes those commands, and documents how to
set up TLS.
* fix(tiller): add stricter certificate verification
The older version of Tiller allowed a weaker set of certificate checks
than we intended. This version requires a client certificate, and then
requires that that certificate be signed by a known CA. This works
around the situation where a user could provide a self-signed
certificate.
7 years ago
Federico Gimenez
e6d907ed28
check if resource is present before adding it to remaining or keep on deletion
7 years ago
Maxime Guyot
c864ba179e
Add fullnameOverride
7 years ago
Yecheng Fu
126f0249a7
Remove executable permission bit for chart files.
7 years ago
Pietro Menna
ece9486182
fix(helm): home env not set on Windows
...
When setting $HELM_HOME, only $HOME was considered. This variable is not always present on Windows.
7 years ago
Adam Reese
d6da76f3cf
Merge pull request #3097 from adamreese/fix/2437-upgrade-release
...
fix(tiller): upgrade last deployed release
7 years ago
Adam Reese
82ef751414
fix(tiller): upgrade last deployed release
...
Fixes #2437
Two bugs were causing this behavior
- Tiller was marking the previous release superseded when an upgrade
failed.
- Upgrade was diffing against failed releases
7 years ago
Matthew Fisher
1954b51186
Merge pull request #2621 from gardlt/feature/helm/adding-kubeconifg-flag
...
feat(helm): adding kubeconfig flag
7 years ago
Matthew Fisher
93e1c765df
Merge pull request #3065 from cjauvin/fix-for-relative-chart-path-support
...
Fix for relative chart path support in index.yaml
7 years ago
Matt Tucker
b8734a173e
fix(docs): fix code comment for ReuseValues()
7 years ago
gardlt
371ff8f26d
feat(helm): adding kubeconfig flag
7 years ago
Christian Jauvin
8775f632f2
Fix for relative chart path support in index.yaml
7 years ago
Matt Farina
22e5381251
feat(chart): Adding maintainer url
...
Each maintainer can have an optional URL. There are seceral use
cases for providing a URL instead of an email address:
- The case where a maintainer wants to hide their email address
behind a webform for personal security
- When the maintainer is an organization and feedback should
go to an issue queue
This change:
- Adds the proto field for url
- Updates the generated files for the chart proto
- Includes linting for the url
- Updates the docs to include the url
Closes #3056
7 years ago
Adam Reese
08c1144f5e
chore(*): bump version to v2.7.0
7 years ago
Johan Lyheden
4831089500
Add tests to HTTPInstaller, update plugin documentation
7 years ago
Johan Lyheden
f1a08adb41
Update to comply with linter rules and gofmt
7 years ago
Johan Lyheden
d43d5ab452
Add plugin installer from http archive
7 years ago
Adam Reese
13912b3cc0
Merge pull request #3008 from adamreese/ref/kube-1.8
...
ref(*): kubernetes v1.8 support
7 years ago
Taylor Thomas
500c9983b5
Merge pull request #3019 from thomastaylor312/fix/crd_ordering
...
fix(tiller): Adds CRD ordering
7 years ago
Brad Bowman
e974ab31d8
[fakeclient-releasestatus] Move ReleaseMock and associates into helm package and update tests to use them
7 years ago
Adam Reese
2b43ecd405
ref(*): kubernetes v1.8 support
7 years ago
Brad Bowman
8d5cf13f69
Add test case for deleting the only existing item
7 years ago
Brad Bowman
cab3057ee3
Add support for all CRUD operations + test coverage for delete and install
7 years ago
Brad Bowman
61f2b24dca
Update comment for accuracy
7 years ago
Brad Bowman
76b70ec70f
gofmt -s
7 years ago
Brad Bowman
3bc7c44f05
Make ReleaseContent and ReleaseStatus respect release names in the fake client
7 years ago
Taylor Thomas
29c3b5276f
fix(tiller): Adds CRD ordering
...
This adds CRD ordering into the explicit kind sorter list
Closes #2925
7 years ago
Remington Reackhof
9af1018bd3
Add secrets storage backend for releases
7 years ago
Taylor Thomas
d4ccef7ba7
Merge pull request #2999 from bonifaido/getterConstructor-err
...
fix(helm): invoking getterConstructor returns downstream error now
7 years ago
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
7 years ago
Michal Cwienczek
dad8c6f644
Fix #2937 - helm always appends /index.yaml at the end of URL ( #2988 )
...
* Closes #2937
Added required dependency to run make test in developer's guide
* Fixed base URL appending when chart address is not absolute
* Removed requirement from developers.md
* Fixed unnecessary line breaks
* Added tests for query string repo
* Returning URL along with error
7 years ago
Matthew Fisher
d7a81c99e1
Merge pull request #2976 from HotelsDotCom/respect-apiversion-change
...
Ignore Apiversion, use Kind and Name only to search for original resource on update/delete
7 years ago
Matthew Fisher
d4f5a133dd
Merge pull request #2960 from HotelsDotCom/upgrade-failure-record-release-as-update
...
Correctly persists Release upgrade failure
7 years ago
Taylor Thomas
f8e9741578
Merge pull request #2984 from thomastaylor312/feat/add_missing_test
...
fix(sorter): Adds missing unit test
7 years ago
Matthew Fisher
e3e1116eaa
Merge pull request #2857 from llsheldon/fix-dependency-alias
...
fix(helm):Fix dependency aliaes not working
7 years ago
Matthew Fisher
3799c477d1
Merge pull request #2938 from bacongobbler/remove-facebookgo
...
Remove facebookgo dependencies
7 years ago
Nandor Kracser
a28e5dd2b3
fix(helm): invoking getterConstructor returns downstream error now
7 years ago
Justin Scott
b642ba2087
Merge pull request #2831 from jascott1/sort_unknown_kind
...
bug(tiller): sort unknown but different kinds alphabetically based on kind name
7 years ago
Matthew Fisher
b600ea475e
Merge pull request #2982 from HotelsDotCom/pkg-lint-test-panics
...
Avoid panics if test is failing
7 years ago
Taylor Thomas
acd113ec21
Merge pull request #2978 from HotelsDotCom/detailed-wait-log
...
More helpful wait messages in tiller logs
7 years ago
Taylor Thomas
333f8dd354
fix(sorter): Adds missing unit test
...
Adds a unit test that was missed as part of #2961
7 years ago
Maxim Ivanov
a6872c124a
Avoid panics if test is failing
7 years ago
Taylor Thomas
8c2c9fcf6e
Merge pull request #2961 from NauxLiu/patch-1
...
Load StorageClass before PersistentVolume is loaded.
7 years ago
Taylor Thomas
49120bb436
Merge pull request #2849 from yan234280533/patch-pod
...
Add the status of pod when using "Helm status" command, as the issue #2318
7 years ago
Maxim Ivanov
cd8bae1df5
More helpful wait messages in tiller logs
7 years ago
Maxim Ivanov
bed80e482c
When looking for original resource on update/delete, ignore Apiversion, use Kind and Name only
...
Kind can change it's APIVersion with time, such as "Deployment" which was
"extensions/v1beta1" and then migrated to "apps/v1beta1" in Kube 1.6.
Name, Kind and optinally Namespace are used to find relevant Info
object for a original resource which is being upgraded, it safe enough,
as it is uniquely identifies an object.
7 years ago
Matthew Fisher
f278675f98
remove references to facebookgo/symwalk
7 years ago
Matthew Fisher
211f5f6f33
Revert "fix(helm): resolve symlinks when loading chart"
...
This reverts commit edd4e56112
.
7 years ago
Matthew Fisher
9d1db2851e
Revert "Write repo file using atomicfile"
...
This reverts commit 965cb7fd1c
.
7 years ago
Maxim Ivanov
ff5acc9bd8
Make Memory driver to store copy of releases to stop hiding storage errors during tests
7 years ago
Matthew Fisher
cf3fc21e81
Merge pull request #2898 from zlabjp/kube-version
...
feat(helm): add --kube-version flag to helm template
7 years ago
NauxLiu
c3f9120e02
Load StorageClass before PersistentVolume is loaded.
7 years ago
Maxim Ivanov
38c3f58dca
Correctly persists Release upgrade failure
...
When release upgrade fails, updatedRelease is already created
in a storage by *ReleaseServer.UpdateRelease, therefore we should
be updating it's status, not creating it again.
7 years ago
carlory
ad2d8e434a
fix fromjson comment
7 years ago
carlory
b9d3974df0
fix func comment
7 years ago
devinyan
3458ad8e6a
Add the status of pod when using "Helm status" command
7 years ago
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).
7 years ago
Kazuki Suda
3508cebbf6
Use the same defaults as done in helm lint for Capabilities
7 years ago
Michelle Noorali
84bb77f183
Merge pull request #1997 from AmandaCameron/issue/1401
...
Take a crack at Issue #1401
7 years ago
Amanda Cameron
415e52bf55
Add support for sub-directory scanning as in issue #1401
7 years ago
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
7 years ago
Adnan Abdulhussein
8fd8a7c324
fix(create): port-forward should use internal port
...
cc @james-w
7 years ago
Justin Scott
ed24b3199a
bug(tiller): sort unknown but different kinds alphabetically based on kind name
...
Adds additional manifest sorting logic so that different unknown kinds
are sorted alphabetically so that manifest order is more deterministic.
7 years ago
Matthew Fisher
1ac5cd78de
Merge pull request #2901 from bacongobbler/bump-lint-k8s-version
...
bump k8s version in helm lint to 1.7
7 years ago
Steven E. Harris
f588315b76
Remove spurious field in kind sorter test ( #2667 )
...
The test never reads the manifest type's "content" field, so there's
no reason to populate it explicitly with its zero value.
7 years ago
Matthew Fisher
8b512808df
bump k8s version in helm lint to 1.7
7 years ago
Matthew Fisher
bb5db61325
Merge pull request #2781 from jascott1/i2755-template
...
feat(helm): add `template` command
7 years ago
DoctorZK
608fd7318a
fix(tiller): fix a warning bug in hook annotation
7 years ago
Justin Scott
8befd50c12
Merge pull request #2850 from rocky-nupt/fix-update-delete-subcharts
...
Fix(helm): Fix the bug of dependency update deleting subcharts
7 years ago
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
7 years ago
xuhaigang
b79033a715
Merge branch 'fix-update-delete-subcharts' of https://github.com/rocky-nupt/helm into fix-update-delete-subcharts
7 years ago
xuhaigang
4ffb9dfb4b
Fix(helm): Fix the bug of dependency update deleting subcharts
...
In Helm 2.6.0, the new delete feature that deletes old versions of
charts is deleting subcharts that aren't in requirements.yaml.
In this patch, I judge the dependency whether it is included in the
requirements.yaml before deleting it.
Closes #2830
7 years ago
DockerZK
734b124577
feat(*) add key helm.sh/hook-delete-policy to hook annotation
...
When "helm.sh/hook-delete-policy: hook-succeeded" is provided in a hook's annotation, Tiller will automatically delete the hook after the hook is succeeded. When "helm.sh/hook-delete-policy: hook-failed" is provided in a hook's annotation, Tiller will automatically delete the hook after the hook is failed.
Closes #1769
7 years ago
Matt Butcher
a736c2a3fe
Merge pull request #2636 from technosophos/feat/2332-max-history
...
feat(tiller): limit number of versions stored per release
7 years ago
Kira
4c1a47480e
feat(2450): add annotations to chart metadata ( #2671 )
...
* feat(2450): add options to chart metadata
7 years ago
Adam Reese
5bc7c619f8
chore(*): bump to v2.6.0
7 years ago
Taylor Thomas
7a6dbfea49
Merge pull request #2790 from nebril/dependency-update-chart-deletion
...
Delete old deps after chart deps are updated
7 years ago
Justin Scott
f239050996
Update kind subsort to sort unknown kinds alphabetically.
...
Remove length check from subsort test. Add check for unknowns.
7 years ago
Justin Scott
87d2b7ab67
Add test for kind sorter sub sort
7 years ago
Justin Scott
66c4f7453e
feat(tiller): sort manifests alphabetically if they are same kind
...
Updates Tiller manifest sorting so that manifests of the same kind are
then sorted alphabetically.
Closes #1696
7 years ago
Matt Butcher
0853f4906a
feat(tiller): limit number of versions stored per release
...
This adds a new configuration option to Tiller to limit the number of
records stored per release.
Tiller stores historical release information (helm history, helm
rollback). This makes it possible to set a maximum number of versions
per release.
To enable this feature, use `helm init --history-max NNN`. Note that
because of the restrictions on Deployment objects, you will have to
re-install Tiller to add a limit.
Along the way, I found an unreported bug in the Memory storage driver.
This fixes that bug and adds substantially more tests to catch
regressions.
Closes #2332
7 years ago
Maciej Kwiek
9c4bb14673
Delete old deps after chart deps are updated
...
This change changes the order of operations in
pkg/downloader.Manager.downloadAll
Old charts are moved to tmp directory which is restored in case any
dependency update fails. Otherwise tmp dir is deleted.
7 years ago
Maxime Guyot
bbf2d6b7e8
Fix the default NOTES.txt
7 years ago
Taylor Thomas
6ab706ae6c
Merge pull request #2768 from JarreyZhou/master
...
feat(helm):use a variable for labels to get tiller pod
7 years ago
Taylor Thomas
8cf72adf52
Merge pull request #2711 from bonifaido/handle-tlspath-error
...
Redefine the err variable in NewChartRepository, otherwise errors migh slip through
7 years ago
Taylor Thomas
8eaed9d3a7
Merge pull request #2729 from rbwsam/fix/leading_zeros
...
fix(pkg/strvals): preserve leading zeros in vals
7 years ago
Taylor Thomas
e5ea075271
Merge pull request #2727 from Miouge1/master
...
Updating default values.yaml comments
7 years ago
Jerry Zhou
23905f1494
feat(helm):use a variable for labels to get tiller pod
7 years ago
Sam Leavens
a5dc546726
fix(pkg/strvals): use rune literal instead of ASCII
...
When checking that a value begins with zero use a rune literal instead of the
ASCII code for zero.
8 years ago
Brian
d5682ae4f5
Merge pull request #2615 from bonifaido/optional-ca-file
...
CAFile is now optional, in that case the default RootCAs are used
8 years ago
Matt Butcher
4a7d47cd74
Merge pull request #2719 from nebril/release-module-nil
...
Handle nil responses returned from rudder funcs
8 years ago
Miouge1
6cfed9d635
Updating default values.yaml comments
...
Following the instructions to simply uncomment the following lines results in a invalid yaml output
8 years ago
Sam Leavens
609e72b357
fix(pkg/strvals): preserve leading zeros in vals
...
When passing values with "helm install --set" values with leading zeros are
preserved and not parsed as ints.
Closes #2693
8 years ago
Steven E. Harris
e8d80729ac
Merge pull request #2703 from rbwsam/ref/simplify_error
...
ref(pkg/chartutil): replace use of 'fmt.Efforf'
8 years ago
Maciej Kwiek
6b16ee2fb7
Handle nil responses returned from rudder funcs
...
Tiller panicked on nil pointer dereference when it was unable to connect
to rudder.
8 years ago
Matt Butcher
88aa4def3e
Merge pull request #2701 from jascott1/i2560-hang
...
fix(tiller): remove locking system from storage and rely on backend
8 years ago
Steven E. Harris
98c8e3c138
Merge pull request #2704 from rbwsam/ref/2663
...
ref(pkg/chartutil): decrease map lookups
8 years ago
Nandor Kracser
9f963c84b4
Redefine the err variable in NewChartRepository, otherwise errors might slip through
8 years ago
Taylor Thomas
8fc88ab626
Merge pull request #2681 from at1012/at-helm-list
...
feat(2063): Better 'helm list' visibility into state of pending upgrades
8 years ago
Sam Leavens
098df8b186
ref(pkg/chartutil): improve comment spacing
...
Reduce comment spacing to single tab.
8 years ago
Sam Leavens
f7ca52dfc8
ref(pkg/chartutil): improve comment for PathValue
...
Make comment for PathValue more explanatory.
8 years ago
Sam Leavens
cf09e344d1
fix(pkg/chartutil): Fix test, improve message
...
Fix inverted equaility check and update error message.
8 years ago
Sam Leavens
8df52b6501
ref(pkg/chartutil): move comment
...
Move comment out of test. Move ref location in test.
8 years ago
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
Justin Scott
fa68a6e1db
fix(tiller): remove locking system from storage and rely on backend controls
...
Tiller currently hangs indefinitely when deadlocks arise from certain
concurrent operations. This commit removes the nested mutex locking
system from pkg/Storage and relies on resource contention controls in k8s.
Closes #2560
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