yank1
76a40fc573
fix a typo in client
7 years ago
Thibaut Rousseau
a59d212512
docs: Fix FromJson comment
7 years ago
oilbeater
757db3a062
defer in loop may cause resource leak
7 years ago
Matthew Fisher
8009115d3a
Merge pull request #3465 from oilbeater/fix/defer-in-loop
...
defer in loop may cause resource leak
7 years ago
oilbeater
0764d95a03
defer in loop may cause resource leak
...
defer statement executes only when function return, and the resource still be hold during loop. Release the resource manually when not needed.
7 years ago
Matt Farina
6a1dd806ed
Merge pull request #3469 from mattfarina/fix/3322
...
fix(grpc): Fixes issue where message sending limited to 4mb
7 years ago
Matthew Fisher
3c39e72189
Merge pull request #3401 from bacongobbler/revert-kubeconfig-flag
...
Revert "feat(helm): adding kubeconfig flag"
7 years ago
Matt Farina
614cd9dfe7
fix(grpc): Fixes issue where message sending limited to 4mb
...
Between grpc 1.2.x and 1.7.x there was an API change. The
previous MaxMsgSize is now a wrapper around MaxRecvMsgSize. This
change now sets the MaxRecvMsgSize and MaxSendMsgSize which need
to be set independently.
7 years ago
Matt Farina
e6137ff05f
fix(api-machinery): Fixes patching for unstructured objects
...
CRDs and other objects seen as unstructured cannot use strategic
merge patching. It has never been supported on CRDs. Previously,
cases like unstructured objects could have caused an unregistered
error. This is no longer the case.
This change explicitly looks for unstructured objects and handles
those using json merge patching.
Closes #3382
7 years ago
Matthew Fisher
2724387506
Merge pull request #3373 from unguiculus/feature/fix-recreate-part-1
...
Fix pod recreation
7 years ago
Matt Farina
4b578f44ff
Merge pull request #3407 from paolomainardi/feature/3173_fix_wait_for_deployments
...
refs #3173 : add appsv1beta1 and appsv1beta2 apiVersion
7 years ago
Paolo Mainardi
b065ffe1fc
adding other missing apiVersions
7 years ago
Paolo Mainardi
2cee4c68fa
add appsv1beta1 and appsv1beta2 apiVersion
7 years ago
Reinhard Nägele
ea520afd3e
Fix 'getSelectorFromObject'
7 years ago
Matthew Fisher
748dbda29a
Merge pull request #3399 from bacongobbler/fix-release-checklist
...
Fix release checklist
7 years ago
Bin Liu
1e0906ed67
Fix type
7 years ago
Matthew Fisher
f13c4d28b5
Revert "feat(helm): adding kubeconfig flag"
...
This reverts commit 371ff8f26d
.
7 years ago
Matthew Fisher
8604e0b832
bump version to v2.8
...
also add it to the release checklist.
7 years ago
Reinhard Nägele
3d05da0109
Fix pod recreation
7 years ago
Matthew Fisher
14c7be1f92
Merge pull request #3288 from bradbeam/starter
...
feat Making starter templates more versatile
7 years ago
Matt Farina
c359ea66ed
Merge pull request #3350 from mattfarina/fix/3349
...
fix(capabilities): Adding GitVersion to default set
7 years ago
Alex Johnson
b29d25ef0b
Added a --wait flag for helm init which pings the Tiller server and ensures that it is ready to receive requests
...
Fixes #2114
Signed-off-by: Alex Johnson <ajohnson@bombora.com>
7 years ago
Adnan Abdulhussein
f432b8ec60
Merge pull request #3337 from unguiculus/feature/helm-create
...
Update initial chart experience
7 years ago
Matt Farina
d5a865b5f4
fix(capabilities): Adding GitVersion to default set
...
Prior to this, using the semver template functions with the full
version, which is represented in the GitVersion, was not possible
for helm template and lint commands because the property was not
populated by default. This update adds default handling.
Closes #3349
7 years ago
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.
7 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
7 years ago
Matt Butcher
4a7d47cd74
Merge pull request #2719 from nebril/release-module-nil
...
Handle nil responses returned from rudder funcs
7 years ago
Miouge1
6cfed9d635
Updating default values.yaml comments
...
Following the instructions to simply uncomment the following lines results in a invalid yaml output
7 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
7 years ago
Steven E. Harris
e8d80729ac
Merge pull request #2703 from rbwsam/ref/simplify_error
...
ref(pkg/chartutil): replace use of 'fmt.Efforf'
7 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.
7 years ago
Matt Butcher
88aa4def3e
Merge pull request #2701 from jascott1/i2560-hang
...
fix(tiller): remove locking system from storage and rely on backend
7 years ago
Steven E. Harris
98c8e3c138
Merge pull request #2704 from rbwsam/ref/2663
...
ref(pkg/chartutil): decrease map lookups
7 years ago
Nandor Kracser
9f963c84b4
Redefine the err variable in NewChartRepository, otherwise errors might slip through
7 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
7 years ago
Sam Leavens
098df8b186
ref(pkg/chartutil): improve comment spacing
...
Reduce comment spacing to single tab.
7 years ago
Sam Leavens
f7ca52dfc8
ref(pkg/chartutil): improve comment for PathValue
...
Make comment for PathValue more explanatory.
7 years ago
Sam Leavens
cf09e344d1
fix(pkg/chartutil): Fix test, improve message
...
Fix inverted equaility check and update error message.
7 years ago
Sam Leavens
8df52b6501
ref(pkg/chartutil): move comment
...
Move comment out of test. Move ref location in test.
7 years ago
Sam Leavens
3ee70291c8
ref(pkg/chartutil): Capitalize YAML
...
Capitalize "yaml" in error message. Remove empty var from test error message.
7 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
7 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
7 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.
7 years ago
Adam Reese
a29e610938
fix(helm): fix flag parsing once and for all
7 years ago
Aishwarya Thangappa
915e7023c4
Added more granular statuses
7 years ago
Aishwarya Thangappa
9d67d8b954
Added PENDING status to helm install
7 years ago
Aishwarya Thangappa
3d4361ca1f
Modified release_update and release_rollback
7 years ago
Aishwarya Thangappa
a9c16fc964
Added PENDING status to status.proto and regenerated the pb.go files
7 years ago
Adam Reese
9325d136d4
fix(tests): fix sorting hooks test flake
7 years ago
Taylor Thomas
0a20ed73be
Merge pull request #2644 from alonl/master
...
Fix a bug causing 'helm depndency update' to delete required charts
7 years ago
Steven E. Harris
2aaad8242e
Merge pull request #2666 from adamreese/ref/cleanup-tests
...
ref(tests): simplify kind sorter test
7 years ago
Steven E. Harris
e028be065d
Merge pull request #2650 from arschles/apiservice
...
Add the "APIService" kind to the installation sort order
7 years ago
Aaron Schlesinger
58a2d7b5c1
Adding APIService to the kind sorter tests
7 years ago
Adam Reese
0d1f6b981f
ref(tests): simplify kind sorter test
7 years ago
Scott Rigby
6dc31f8e96
Test that the coalesced key is properly removed for all YAML null syntax options, but not for empty string
7 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.
7 years ago
Adam Reese
845e990143
feat(*): use kubernetes 1.7
7 years ago
Matt Butcher
883da97396
Merge pull request #2607 from sushilkm/issues/2599
...
Adds charts in "charts\" directory to dependencies
7 years ago
Aaron Schlesinger
5ff75f84a6
Adding APIService to the sort order
...
This ensures that APIService resources are installed last and
uninstalled first
7 years ago
Alon Lavi
04a7e241e6
Fix a bug causing 'helm depndency update' to delete required charts
7 years ago
Ilya Semenov
13b15f5662
Remove service.type mention from ingress docs
7 years ago
Justin Scott
af4c243ee3
Correct punctuation and capitalization for user facing strings.
7 years ago
Michelle Noorali
29c4709b5e
Merge pull request #2550 from michelleN/export-fake-release-client
...
ref(helm): make FakeReleaseClient public
8 years ago
Nandor Kracser
5f96fb816c
CAFile is now optional, in that case the default RootCAs are used
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
Y.W
0f953403a2
give an uniform check for release process ( #2565 )
...
* give an uniform check for release process
* fixed as the review of adamreese: update the err message when releasename is empty and update the test units.
* fixed as the review of bacongobbler: add more detail information to return message. the regex rule is added to the return message.
8 years ago
Michelle Noorali
8a99a4aa6f
ref(helm): rename FakeReleaseClient -> FakeClient
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
Adam Reese
012cb0ac1a
chore(*): bump to v2.5.0
8 years ago
Sushil Kumar
f476f83e83
<chart>.tgz will now be downloaded to "$HELM_HOME/cache/archive" directory ( #2558 )
...
* <chart>.tgz will now be downloaded to "$HELM_HOME/cache/archive" directory
Fixes https://github.com/kubernetes/helm/issues/2142
* Added archive location to helm init
8 years ago
Reinhard Nägele
76e6e15a0c
Add comment on resources
8 years ago
Reinhard Nägele
51923e54b1
Improve 'helm create'
8 years ago
Adam Reese
72cd1464d2
Merge pull request #2570 from adamreese/ref/goimports
...
ref(tiller): cleanup goimports
8 years ago
Matt Butcher
dece57e0ba
Merge pull request #2350 from eicnix/master
...
Added "tpl" template function
8 years ago
Adam Reese
eccb041c0f
ref(tiller): cleanup goimports
8 years ago
Matt Butcher
ecef026b68
Merge pull request #2545 from technosophos/feat/set-list-index
...
feat(helm): support array index format for --set.
8 years ago
Steven E. Harris
9f9b3e8729
Use versioned API types from the client-go library ( #2524 )
...
* Use versioned API types from the client-go library
Wherever possible, use the k8s.io/client-go/kubernetes.Interface type
in favor of the client-related types from package
k8s.io/kubernetes/pkg/client/clientset_generated/internalclientset
The latter are still required by the kubectl "reaper" types used in
the "installer" and "kube" packages.
* Accept the default deployment replica count of one
Don't bother requesting a single replica explicitly.
8 years ago
Matt Butcher
c01c7318ab
feat(helm): support array index format for --set.
...
This adds support for specifying list position with an array index using
`--set`. For example, this now works: `--set servers[0].port=8080`
8 years ago
Lukas Eichler
39db9ec6e8
Merge branch 'master' into master
8 years ago
Michelle Noorali
15c14194c4
Merge pull request #2457 from michelleN/hooks-bug
...
fix(tiller): track hooks in multi-def manifests
8 years ago
Michelle Noorali
70520efac4
ref(helm): make FakeReleaseClient public
...
so it can be used by other projects that
use the helm client
8 years ago
Taylor Thomas
e3250e3319
Merge pull request #2534 from sushilkm/issues/2508
...
First alias would be dependency rename
8 years ago
Matt Butcher
d2a4c40fa5
Merge pull request #2541 from technosophos/fix/2525-helm-get-broken-release
...
fix(tiller): make GetReleaseContent return any release
8 years ago
Matt Butcher
c55577ace6
Merge pull request #2535 from jascott1/i2331
...
fix(helm): remove unnecessary values merge in processImportValues
8 years ago
Michelle Noorali
6bfb08a760
ref(tiller): refactor sortManifests()
8 years ago
Michelle Noorali
83c69a8e10
fix(tiller): track hooks in multi-def manifests
...
resolves #2290
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
Matt Butcher
c913911337
fix(tiller): make GetReleaseContent return any release
...
For some reason, GetReleaseContent was configured to return the latest
release only if it is in state DEPLOYED. But a release with a version is
returned regardless of release. This made it really hard to debug failed
releases, even though we have the data to show.
Closes #2525
8 years ago
Adam Reese
8272360681
Merge pull request #2476 from adamreese/fix/flag-envars
...
fix(helm): fix race conditions in flag parsing
8 years ago
Matt Butcher
3647e60786
Merge pull request #2512 from technosophos/fix/2510-missing-url-prefix
...
fix(helm): prepend repo URL to packages missing scheme
8 years ago
Matt Butcher
8f3c2d5e18
Merge pull request #2511 from technosophos/fix/2452-predictable-template-order
...
fix(2452): sort templates before parse
8 years ago
devinyan
d3b14db325
adjust the check parameter order for uninstall
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
Seth Goings
da383b35e5
Merge pull request #2518 from sgoings/fixup-plugin-install-grammar
...
fix(plugins): exists --> exist
8 years ago
Taylor Thomas
d9c0a8b434
fix(tiller): Adds missing import back
...
PR #2513 was behind master when merged. Other commits were
added that required the use of the log package. This re-adds
that package.
8 years ago
Taylor Thomas
b71bd6a455
Merge pull request #2513 from thomastaylor312/feat/moar_logging
...
feat(tiller): Adds more logging
8 years ago
Taylor Thomas
119fa6537c
feat(tiller): Adds more logging
...
This builds on previous work and adds more logging to follow
the full process of installing, updating, deleting, and rolling back.
This also standardizes capitalization of logs and small formatting
fixes
8 years ago
Seth Goings
e7a51d5424
fix(plugins): exists --> exist
8 years ago
Adam Reese
d797acbd7b
fix(helm): fix race conditions in flag parsing
...
* fix a number of issues with flag parsing
* add support for `HELM_DEBUG`
* lazy expand flag default envars
8 years ago
Lukas Eichler
9cd9fc47df
removed debug file
8 years ago
Lukas Eichler
439f1b31d1
Added unit test for include in tpl function
8 years ago
Lukas Eichler
ce8e8d6778
Added storage of parsed templates in engine
8 years ago
Matt Butcher
8937c775a9
fix(2452): sort templates before parse
...
This sorts templates by depth before sending them to the template
parser. Deepest templates are parsed first, with umbrella templates
parsed last. Since template definition names are LIFO, that means that
the highest level templates will claim the namespace.
Or, to put it simply, you can predictably override a child's defined
template by re-defining it in a parent chart.
Closes #2452
8 years ago
Matt Butcher
4c6a7cf759
fix(helm): prepend repo URL to packages missing scheme
...
When a repository is generated without --url, the packages in that
repository do not have FQDNs. In this case, the URL prefix (FQDN plus
base path) should be derived from the repository's base URL. This seems
to be a regression that crept in around Helm 2.2.0. This is now fixed.
Closes #2315
Closes #2510
8 years ago
peay
a9fa9282e1
Add --force to upgrade and rollback (generated changes)
8 years ago
peay
0f26cc5522
Add --force to upgrade and rollback
8 years ago
Taylor Thomas
7a49e5c3e1
Merge pull request #2500 from sushilkm/issues/2488
...
Check existence of $HELM_HOME/plugins before installing plugin
8 years ago
Taylor Thomas
7dd6eb22f7
Merge pull request #2486 from sushilkm/issues/2383
...
Adds alias for dependencies
8 years ago
Sushil Kumar
a6556b4982
Check existence of $HELM_HOME/plugins before installing plugin
...
Fixes https://github.com/kubernetes/helm/issues/2488
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
tyrannasaurusbanks
b4fc1c7cb2
Add check to ensure helm doesnt 'wait' for external services to become 'ready'
8 years ago
fibonacci1729
04d0abfd7c
ref(tiller): refactor tests into logical files
...
TestInstallRelease_WithChartAndDependencyNotes -> tiller/release_install_test.go
TestInstallRelease_WrongTillerVersion -> tiller/release_install_test.go
TestInstallRelease_WithNotesRendered -> tiller/release_install_test.go
TestInstallRelease_TillerVersion -> tiller/release_install_test.go
TestInstallRelease_FailedHooks -> tiller/release_install_test.go
TestInstallRelease_ReuseName -> tiller/release_install_test.go
TestInstallRelease_WithNotes -> tiller/release_install_test.go
TestInstallRelease_NoHooks -> tiller/release_install_test.go
TestInstallRelease_DryRun -> tiller/release_install_test.go
TestInstallRelease -> tiller/release_install_test.go
8 years ago
fibonacci1729
22db973932
ref(tiller): refactor tests into logical files
...
TestListReleasesByStatus -> tiller/release_list_test.go
TestListReleasesFilter -> tiller/release_list_test.go
TestListReleasesSort -> tiller/release_list_test.go
TestListReleases -> tiller/release_list_test.go
TestUpdateRelease_ResetReuseValues -> tiller/release_update_test.go
TestUpdateRelease_ResetValues -> tiller/release_update_test.go
TestUpdateRelease_ReuseValues -> tiller/release_update_test.go
TestUpdateReleaseNoChanges -> tiller/release_update_test.go
TestUpdateReleaseFailure -> tiller/release_update_test.go
TestUpdateReleaseNoHooks -> tiller/release_update_test.go
TestUpdateRelease -> tiller/release_update_test.go
TestRollbackReleaesFailure -> tiller/release_rollback_test.go
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
fibonacci1729
172dbb22cf
ref(tiller): refactor tests in logical files
...
TestRunReleaseTest -> tiller/release_testing_test.go
TestGetReleaseContent -> tiller/release_content_test.go
TestGetReleaseStatusDeleted -> tiller/release_status_test.go
TestGetReleaseStatus -> tiller/release_status_test.go
TestRollbackReleaseFailure -> tiller/release_rollback_test.go
TestRollbackReleaseNoHooks -> tiller/release_rollback_test.go
TestRollbackRelease -> tiller/release_rollback_test.go
TestUninstallReleaseWithKeepPolicy -> tiller/release_uninstall_test.go
TestUninstallPurgeDeleteRelease -> tiller/release_uninstall_test.go
TestUninstallReleaseNoHooks -> tiller/release_uninstall_test.go
TestUninstallPurgeRelease -> tiller/release_uninstall_test.go
TestUninstallRelease -> tiller/release_uninstall_test.go
8 years ago
fibonacci1729
2ae276ae39
ref(tiller): refactor tiller APIs into logical files
...
*ReleaseServer.InstallRelease -> tiller/release_install.go
*ReleaseServer.UpdateRelease -> tiller/release_update.go
8 years ago