Matthew Fisher
da83115510
Merge pull request #5539 from jan25/fix-list-sorting-5499
...
fix(helm): sort release list with flags --reverse and --date
6 years ago
Matthew Fisher
f1594e564a
Merge pull request #5824 from rokii/fix5792
...
fix issue that values are not applied while running upgrade cmd
6 years ago
rokii
897a79a57f
fix and add test cases
...
Signed-off-by: rokii <shangtaocn@gmail.com>
6 years ago
Torsten Walter
829d8ff4d6
Revert "do not write empty templates to disk"
...
This reverts commit da260ec15f
.
Signed-off-by: Torsten Walter <mail@torstenwalter.de>
6 years ago
Torsten Walter
0528c7bb19
add test for output-dir
...
Signed-off-by: Torsten Walter <mail@torstenwalter.de>
6 years ago
Torsten Walter
7347be61ee
Merge branch 'dev-v3' into output-dir
6 years ago
rokii
871b092f32
fix issue 5792
...
Signed-off-by: rokii <shangtaocn@gmail.com>
6 years ago
Adam Reese
77a1b7e0a2
ref(*): expose klog flags
...
Signed-off-by: Adam Reese <adam@reese.io>
6 years ago
Torsten Walter
da260ec15f
do not write empty templates to disk
...
Signed-off-by: Torsten Walter <mail@torstenwalter.de>
6 years ago
Torsten Walter
cb58035f90
use outputDir instead of hardcoded value
...
Signed-off-by: Torsten Walter <mail@torstenwalter.de>
6 years ago
Adam Reese
f1718462c9
Merge pull request #5761 from cpanato/GH-5741
...
fix missing package name
6 years ago
Adam Reese
b49db9e6e6
ref(pkg/chartutil): break up chartutil into logical files
...
Signed-off-by: Adam Reese <adam@reese.io>
6 years ago
Torsten Walter
24578a6411
support --output-dir option for helm3 template
...
Signed-off-by: Torsten Walter <mail@torstenwalter.de>
6 years ago
Carlos Panato
b5f83a08e3
fix missing package name
...
Signed-off-by: Carlos Panato <ctadeu@gmail.com>
6 years ago
Adam Reese
45f63628e1
ref(pkg/chart): remove unused chart.RawValues
...
Signed-off-by: Adam Reese <adam@reese.io>
6 years ago
Adam Reese
590bf10ab5
fix(pkg/action): add namespace to release options
...
ref: https://github.com/helm/helm/issues/5732
Signed-off-by: Adam Reese <adam@reese.io>
6 years ago
Adam Reese
9401295d2a
Merge pull request #5736 from adamreese/v3/requirements
...
feat(pkg/chart): support deprecated requirements.yaml
6 years ago
Adam Reese
634fcfb7ef
feat(pkg/chart): support deprecated requirements.yaml
...
Signed-off-by: Adam Reese <adam@reese.io>
6 years ago
Adam Reese
0338576fc5
ref(pkg/kube): cleanup kube client interface
...
* move the main interface to it's own file
* removed summarizeKeptManifests() which was the last place kube.Get()
was called
* when polling for hooks, use external types
* refactor out legacyschema
* refactor detecting selectors from object
* refactor creating test client
Signed-off-by: Adam Reese <adam@reese.io>
6 years ago
Matt Farina
c62a3a4896
Merge pull request #5730 from mattfarina/fix-lint-apiversion-v3
...
Fix lint apiversion v3
6 years ago
Matthew Fisher
31667b6050
Merge pull request #5717 from jdolitsky/feat-v3/oci-registry-loglevel
...
Helm 3: pass debug option to registry client
6 years ago
Matthew Fisher
757eb88d6c
Merge pull request #5441 from hickeyma/update-lib-charts
...
feat(*): Add capability for application charts to be used as library charts
6 years ago
Matt Farina
50e06b1447
Adding apiVersion guidance to the linting
...
Signed-off-by: Matt Farina <matt@mattfarina.com>
6 years ago
Matt Farina
069eec9e42
Updatin gthe apiVersion linting for Helm v3
...
Signed-off-by: Matt Farina <matt@mattfarina.com>
6 years ago
Matt Farina
29842e040f
Adding lint check for apiVersion which is a required field
...
Fixes #5727
Signed-off-by: Matt Farina <matt@mattfarina.com>
6 years ago
Matthew Fisher
5367c6c296
fix(loader): assume apiVersion is v1 when loading charts
...
In Helm 2, no chart validation was performed on v1 charts, so there are
a few charts out there that never added an apiVersion to the Chart.yaml.
To ensure those charts continue to work for Helm 3, we should assume
that charts loaded without an apiVersion set should be assumed as v1.
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
6 years ago
Josh Dolitsky
5f1128b5f7
pass debug option to registry client
...
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
6 years ago
Martin Hickey
22d0ba8b51
Print manifest output for dry-run option
...
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
6 years ago
Adam Reese
8d1590016c
Merge pull request #5646 from adamreese/v3/kube-wait
...
ref(pkg/kube): extract wait logic from install/update
6 years ago
Adam Reese
b7a14de590
Merge pull request #5601 from adamreese/v3/capabilities
...
ref(pkg/chartutil): remove k8s version object dependency
6 years ago
Martin Hickey
ecf4eda6c5
Fix scaffold chart label in helper template
...
The 'app.kubernetes.io/version' label was not being rendered as
expected. It was appending onto the label before it and also
the next label label was appending onto it on the same line.
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
6 years ago
Adam Reese
b97f881be0
ref(*): use time.Duration for timeouts
...
Signed-off-by: Adam Reese <adam@reese.io>
6 years ago
Adam Reese
097834de0a
ref(pkg/chartutil): remove k8s version object dependency
...
Flattens the `.Capabilities` built-in and removes useless kubernetes
runtime metadata.
Signed-off-by: Adam Reese <adam@reese.io>
6 years ago
Matt Farina
2a06c2baba
Merge pull request #5684 from mattfarina/add-app-version
...
Add app version
6 years ago
Adam Reese
2dd4744d23
ref(pkg/kube): extract wait logic from install/update
...
This change adds a new method for waiting for kubernetes resources.
Signed-off-by: Adam Reese <adam@reese.io>
6 years ago
Adam Reese
b8bced2649
fix(pkg/action): load clients after flags have been parsed ( #5681 )
...
Signed-off-by: Adam Reese <adam@reese.io>
6 years ago
Matt Farina
250b63eced
Updating the labels for consistency
...
Signed-off-by: Matt Farina <matt@mattfarina.com>
6 years ago
Luis Davim
82ffe56ca6
Reduce template code duplication. Fixes #5372
...
Signed-off-by: Luis Davim <luis.davim@jet.com>
6 years ago
Luis Davim
4425f86865
Add app.kubernetes.io/version label
...
Signed-off-by: Luis Davim <luis.davim@jet.com>
6 years ago
Matt Farina
af7eab0325
Updating to the k8s label convention
...
Closes #4335
Signed-off-by: Matt Farina <matt@mattfarina.com>
6 years ago
Martin Hickey
75624afb73
Merge pull request #5662 from hickeyma/fix-scaffold-chart-image
...
fix(chartutil): Scaffold chart fails to deploy pod
6 years ago
Josh Dolitsky
a12a396aab
Helm 3: registry login/logout ( #5597 )
...
* login/logout placeholders
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* use latest oras
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* use docker auth system
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* working login+push
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* working on tests
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* fix typo in htpasswd
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* rename credsfile to config.json
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* add flags for username/password
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* disable logout test broken on linux
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* upgrade to oras 0.4.0
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* re-enable logout test
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* panic for uncaught errors
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* move login/logout to new registry subcommand
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
6 years ago
Martin Hickey
f12c2a3111
Change the nginx app version to a set tag
...
Want to avoid moving tags like using 'stable'. Therefore,
specify the specifc nginx version/tag.
Update from comment review:
- https://github.com/helm/helm/pull/5662#discussion_r280122531
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
7 years ago
Martin Hickey
0b809dd078
Validate library chart files after chart loaded
...
Validate the library chart files after the chart and
its depoendencies have been loaded. This is an update
following review comment:
- https://github.com/helm/helm/pull/5441#pullrequestreview-231339425
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
7 years ago
Martin Hickey
86c5e52ac4
Fix the image field for the scaffold chart application
...
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
7 years ago
Martin Hickey
b74fee715e
Add capability for application charts to be used as library charts
...
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
7 years ago
Matthew Fisher
b1ae1acc8b
Merge pull request #5283 from bacongobbler/purge-by-default
...
ref(uninstall): purge release history by default
7 years ago
Martin Hickey
1915358a21
Merge pull request #5538 from hickeyma/add-app-version
...
feat(*): Add app version to history table
7 years ago
Ian Howell
ffff0e8c33
Feat/schema validation ( #5350 )
...
* Add the Schema type and a function to read it
* Added a function to read a schema from a file
* Check that values.yaml matches schema
This commit uses the gojsonschema package to validate a values.yaml file
against a corresponding values.schema.yaml file.
* Add functionality to generate a schema from a values.yaml
* Add Schema to Chart and loader
* Clean up implementation in chartutil
* Add tests for helm install with schema
* Add schema validation to helm lint
* Clean up "matchSchema"
* Modify error output
* Add documentation
* Fix a linter issue
* Fix a test that broke during a rebase
* Clean up documentation
* Specify JSONSchema spec
Since JSONSchema is still in a draft state as of this commit, we need to
specify a particular version of the JSONSchema spec
* Switch to using builtin functionality for file extensions
* Switch to using a third-party library for JSON conversion
* Use the constants from the gojsonschema package
* Updates to unit tests
* Minor change to avoid string cast
* Remove JSON Schema generation
* Change Schema type from map[string]interface{} to []byte
* Convert all Schema YAML to JSON
* Fix some tests that were broken by a rebase
* Fix up YAML/JSON conversions
* This checks subcharts for schema validation
The final coalesced values for a given chart will be validated against
that chart's schema, as well as any dependent subchart's schema
* Add unit tests for ValidateAgainstSchema
* Remove nonessential test files
* Remove a misleading unit test
The TestReadSchema unit test was simply testing the ReadValues function,
which is already being validated in the TestReadValues unit test
* Update documentation to reflect changes to subchart schemas
7 years ago
Michelle Noorali
3dd1765491
Merge pull request #5631 from michelleN/test-run
...
add helm test run
7 years ago
Matthew Fisher
e2ccf13294
Merge pull request #5553 from bacongobbler/same-appversion-as-version
...
ref(create): app version should be 0.1.0
7 years ago
Michelle Noorali
ceab13e9a1
fix test command, move test to test run subcmd
...
Signed-off-by: Michelle Noorali <michellemolu@gmail.com>
7 years ago
Martin Hickey
b72e25cfb9
Change header "APP_VERSION" to "APP VERSION"
...
Update following review comment:
- https://github.com/helm/helm/pull/5538#pullrequestreview-221803355
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
7 years ago
Martin Hickey
b600f6090e
Add app version to history table
...
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
7 years ago
Matthew Fisher
5264ebd5aa
Merge pull request #5572 from ian-howell/fix/template-error-messages
...
fix(pkg/engine): Clean up template error messages
7 years ago
Matthew Fisher
d3f0ac9343
ref(create): app version should be 0.1.0
...
When creating a Helm chart for the first time, the assumption should be that the app version is also 0.1.0, implying this is for a new application.
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Ian Howell
278594fb0f
fix(pkg/engine): Style changes on template errors
...
Signed-off-by: Ian Howell <ian.howell0@gmail.com>
7 years ago
Ian Howell
92b86f6e74
fix(pkg/engine): Catch non-templating errors when rendering templates
...
Signed-off-by: Ian Howell <ian.howell0@gmail.com>
7 years ago
Ian Howell
33b1ede570
fix(pkg/engine): Clean up template error messages
...
Signed-off-by: Ian Howell <ian.howell0@gmail.com>
7 years ago
Abhilash Gnan
d40f3c63ea
fix ByDate sorter to use Time.Unix()
...
Signed-off-by: Abhilash Gnan <abhilashgnan@gmail.com>
7 years ago
Adam Reese
e458a67f0c
ref(pkg/chart): add validation method to chart
...
Consolidate validation of Chart.yaml.
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Abhilash Gnan
f7e2a78374
fix test cases for list.AllNamespaces
...
Signed-off-by: Abhilash Gnan <abhilashgnan@gmail.com>
7 years ago
Abhilash Gnan
f019ec0e8d
add superseded to list filter mask
...
Signed-off-by: Abhilash Gnan <abhilashgnan@gmail.com>
7 years ago
Abhilash Gnan
416667a8e9
Remove redundant ByNameAsc enum value. Handled by default sorting
...
Signed-off-by: Abhilash Gnan <abhilashgnan@gmail.com>
7 years ago
Abhilash Gnan
b9a39e46de
handle default list sort order through enum order
...
Signed-off-by: Abhilash Gnan <abhilashgnan@gmail.com>
7 years ago
Abhilash Gnan
d3c85f97c2
fix sort list with options bug
...
Signed-off-by: Abhilash Gnan <abhilashgnan@gmail.com>
7 years ago
Adam Reese
31819e4796
fix(pkg/chartutil): marshal capabilities VersionSet into slice
...
Marshal capabilities VersionSet into readable value.
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
87e789f01f
fix(pkg/chartutil): only include external objects in capabilities
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
fe06343b8f
Merge pull request #5512 from adamreese/v3/linter-fixes
...
ref(pkg/action): refactoring dup code and linter fixes
7 years ago
Adam Reese
295092cd7d
ref(pkg/action): refactoring dup code and linter fixes
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Matthew Fisher
bb179bdead
chore(dep): bump kubernetes to 1.14.0
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Matthew Fisher
0805a87140
ref(uninstall): purge release history by default
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Matthew Fisher
0b1caa14a7
Merge pull request #5489 from bacongobbler/list-move-namespaces-field
...
ref(list): move namespaces field closer to the name
7 years ago
Matthew Fisher
32712201ec
ref(list): move namespaces field closer to the name
...
Most users want to see the release name and the namespace it was deployed to,
as those are the unique identifiers where the release is stored. I also added
integration tests for `helm list` to better test the command output.
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Adam Reese
779373a276
Merge pull request #5472 from adamreese/v3/template-name
...
fix(cmd/template): allow setting release name for template
7 years ago
Sunyk
223148dac9
Remove a trivial TODO comment
...
Signed-off-by: Sunyk <lentil1016@gmail.com>
7 years ago
Adam Reese
271e4cf111
fix(cmd/template): allow setting release name for template
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
14d8e97d2a
fix(*): resolve new govet issues
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Matthew Fisher
8f37ab48ac
Merge pull request #5455 from bacongobbler/fix-list
...
fix(install): fix issue where chart metadata is not being saved on `helm install`
7 years ago
Matthew Fisher
5c2f235b6c
fix(install): fix issue where chart metadata is not being saved on `helm install`
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Adam Reese
895e9192d4
feat(*): use vanity import helm.sh/helm
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Matthew Fisher
017790d0e3
ref(action): remove ParseShowOutputFormat
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Matthew Fisher
78bd46075b
fix(action): remove test.go
...
test.go is a duplicate of release_testing.go
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Matthew Fisher
f8ed917830
test(cmd): refactor release_testing_test.go
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Matthew Fisher
f185103b60
ref(action): move AddFlags functions back to cmd
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Matthew Fisher
2b775d693d
ref(action): remove io.Writers, return string instead
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Matthew Fisher
6bb9264e89
fix(helm): fix `helm status` output formatting
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Matthew Fisher
2571dbf82f
ref: remove pkg/helm, pkg/hapi, pkg/tiller
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Adam Reese
21d3a40f3b
feat(tests): replace gometalinter with golangci-lint
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
849f27d11f
ref(pkg/engine): make template specific functions private
...
Make template specific functions private to ensure they not misused and
make unit tests simpler. We may export the template helpers later if
needed.
This lays the foundation for the new chart pipeline.
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
d841a1b1d9
fix(engine): make template rendering thread safe
...
See https://github.com/helm/helm/pull/4828
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Matthew Fisher
2a82e6cbe6
docs: s,kubernetes/helm,helm/helm,g ( #5404 )
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Adam Reese
a1a7d3e824
ref(pkg/helm): refactor out `ReleaseStatus`
...
The `ReleaseStatus()` client call returns the same information as
`ReleaseContent()`
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Martin Hickey
ef4d2a6e65
Update after review
...
Review comment:
- https://github.com/helm/helm/pull/5295#pullrequestreview-203519813
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
7 years ago
Martin Hickey
abdaf3ce1b
Add chart type
...
The chart type is added to differentiaite between an application chart
and a library chart. Library charts can be used as dependencies but are
not installable.
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
7 years ago
Adam Reese
e51a9b90c9
Merge pull request #5284 from adamreese/v3/make-format
...
feat(Makefile): add formatting target
7 years ago
Martin Hickey
45fb4b1c44
Fix linter warnings
...
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
7 years ago
Adam Reese
f791421fab
feat(Makefile): add formatting target
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Josh Dolitsky
16b59bfe5b
Helm 3: fix "latest" tag bug ( #5279 )
...
* add extra ref parsing, validation
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* add fix for missing locator
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* add repo and tag fields for clarity
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* small refector
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
7 years ago
Josh Dolitsky
a32f8ebb37
Helm 3: initial registry support ( #5243 )
...
* initial registry support
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* fix dependency mess
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* add extra chart command output
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* sanitize registry path (windows fix)
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* store all sha256 blobs in same dir
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* switch to use chartutil.SaveDir
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* populate chart command long descriptions
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* remove test cache dir in teardown
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* add long description of chart export
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
* clean up table rows code
Signed-off-by: Josh Dolitsky <jdolitsky@gmail.com>
7 years ago
Adam Reese
28d8c7b277
ref(*): remove references to chart 'engine'
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Matthew Fisher
95c865513f
fix appveyor builds ( #4934 )
...
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
7 years ago
Martin Hickey
86d8596763
Feature(Plugins): Enable platform specific commands ( #5176 )
...
* Add logic for platform specific commands to plugins
* Add plugins doc updated to incorporate platform specific commands
* Add condition for os match: If OS matches and there is no more specific match, the command
will be executed
7 years ago
Adam Reese
d94707db86
ref(*): remove helmVersion chart constraint
...
* Remove helmVersion constraint from charts
* Guard compile time set variables behind `internal/`
* Allow configuration of UserAgent for HTTPGetter
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Matt Butcher
f3bfae5ea7
fix: fix a number of style errors ( #5136 )
...
This fixes a dozen or so style errors, almost all of which were just missing comments.
I left several which are fixed in other outstanding PRs, or which belong to code that is about to be removed.
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>
7 years ago
Matt Butcher
425f7a6f6c
feat: add 'pkg/action' for list operations ( #5077 )
...
* feat: add pkg/action to encapsulate action logic
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>
* feat: replace client/server internals with action package
While we removed Tiller, we left the internal client/server architecture mostly intact. This replaces that architecture with the `pkg/action` package.
This implements the action package for list, but nothing else.
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>
* feat: Add install and refactor some tests
This adds install to the action package, and then fixes up a lot of testing.
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>
* fix: Move a bunch of sorters to the releaseutils package
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>
* fix: updated APIs and fixed a failed test
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>
* Use var for timestamper, instead of adding as a struct field
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>
7 years ago
Christian Koeberl
62533e2b0e
fix(pkg/chartutil): conditions for alias and umrella charts ( #3734 )
...
Enable to use charts with dependencies that have conditions (e.g. in
umbrella charts). Allow aliases for dependencies that have dependencies
with conditions.
Closes #3734
Signed-off-by: Christian Koeberl <christian.koeberl@gmail.com>
7 years ago
derkoe
6e1b45888a
feat(cmd/helm): re-add --history-max option to v3
...
Since there is no tiller anymore this option make most sense with the
'helm upgrade' commando.
Origininally this was added in PR #2636 implementing the feature #2081 .
Signed-off-by: Christian Koeberl <christian.koeberl@gmail.com>
7 years ago
Adam Reese
2b81eea1e2
ref(*): replace byte array with map for Release config
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
6fc8c9e079
ref(pkg/chartutil): simplify chart dependency unit tests
...
- simplify unit tests
- refactor typed errors
- unexport internal functions
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
825b15c763
Merge pull request #4972 from adamreese/ref/chart-dependencies
...
ref(pkg/chart): rename Requirements to Dependencies
7 years ago
Adam Reese
85aef0d3d7
ref(pkg/chart): rename Requirements to Dependencies
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
roc
79f88dfc5e
ref(url) update helm's github url ( #4962 )
...
https://github.com/kubernetes/helm -> https://github.com/helm/helm
https://github.com/kubernetes/charts -> https://github.com/helm/charts
Signed-off-by: Roc Chan <roc@imroc.io>
7 years ago
roc
212d326a34
ref(pkg/repo): rename RepoFile to File
...
To resolve the linter warning: name start with package name.
Signed-off-by: Roc <roc@imroc.io>
7 years ago
Adam Reese
9e1e26f01e
ref(pkg/chartutil): attempt to make requirements processing readable
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
f5b6ff2832
ref(pkg/chart): rename files to be consistent with types
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Matt Butcher
7061716406
ref: require name by default on 'helm install' ( #4858 )
...
This is described in the official Helm 3 proposal: https://github.com/helm/community/blob/master/helm-v3/000-helm-v3.md
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>
7 years ago
Adam Reese
3b0ba0f71e
ref(tests): remove broken symlinks in testdata
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
b4ed1de6b8
ref(*): kubernetes v1.12 support
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Matt Butcher
9fda187647
fix(tests): set mock to generate UTC timestamps ( #4716 )
...
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>
7 years ago
Adam Reese
3be0d81da7
ref(driver): refactor out function type errors
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
21259507bd
ref(*): rename requirements.lock to Chart.lock
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
f7a7a157ce
ref(*): merge requirement.yaml into Chart.yaml
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
516c53dae6
ref(chart): use map for chart.Values
...
Signed-off-by: Adam Reese <adam@reese.io>
7 years ago
Adam Reese
4f26b658d8
change copyright to "Copyright The Helm Authors"
7 years ago
Adam Reese
f012940d9c
ref(*): refactor chart/chartutil
...
ref(chartutil): move chart loading out of chartutil into new package
add chart loader interface to allow lazy loading
feat(chart): create chart accessors
ref(*): cleanup requirements
ref(tiller): remove optional template engines
ref(tiller): simplify sorting releases and hooks
ref(*): code simplification
ref(hapi): move chart package out of hapi
ref(chart): add requirements and lock to Chart struct
7 years ago
Adam Reese
7423eddf21
ref(*): kubernetes v1.11 support
7 years ago
Matthew Fisher
195d21d5d7
ref(cmd): rename `helm delete` to `helm uninstall`
...
To match the convention of `helm install`, `helm uninstall` is the inverse.
Other tangential changes in this PR:
- StatusDeleting has been changed to StatusUninstalling
- StatusDeleted has been changed to StatusUninstalled
- `helm list --deleted` has been changed to `helm list --uninstalled`
- `helm list --deleting` has been changed to `helm list --uninstalling`
- `helm.DeleteOption` and all delete options have been renamed to `helm.UninstallOption`
I have not made any changes to the "helm.sh/hook-delete-policy", "pre-delete" and "post-delete" hook annotations because
1. it's a major breaking change to existing helm charts, which we've commited to NOT break in Helm 3
2. there is no "helm.sh/hook-install-policy" to pair with "helm.sh/hook-uninstall-policy", so delete still makes sense here
`helm delete` and `helm del` have been added as aliases to `helm uninstall`, so `helm delete` and `helm del` still works as is.
7 years ago
Adam Reese
aa859e3f88
feat(*): remove Time, Namespace, and Revision from template functions
...
Removes Time, Namespace, and Revision from being exposed to templates to
make template rendering discrete and repeatable.
7 years ago
Adam Reese
a02a598c33
ref(tests): simplify cmd test setup/teardown
...
Ensure proper cleanup of `HELM_HOME` and `HELM_DEBUG`
7 years ago
Adam Reese
b1128abf4f
ref(*): s/tiller/helm/
7 years ago
Adam Reese
726e3c41be
feat(*): print stacktrace on error with debug enabled
7 years ago
Adam Reese
75c4df0b56
ref(tests): use golden files for testing command output
7 years ago
Adam Reese
36e034551f
ref(*): rebuild build version object
8 years ago
Adam Reese
03ea683b9a
ref(pkg/tiller): add flag to enable tilling logging in tests
...
`go test ./pkg/tiller -test.log`
8 years ago
Adam Reese
3b9596c6ab
ref(*): convert const types to strings
8 years ago
Adam Reese
c50813af54
ref(*): remove local repository (dead code)
8 years ago
Adam Reese
3c27143291
fix(kube): output internal object table
...
fixes #3937
8 years ago
Adam Reese
1a508ccdd1
ref(*): move kubeconfig flags to helm/environment
8 years ago
Adam Reese
2a97768b5c
ref(pkg/tiller): flatten package structure for storage
8 years ago
Adam Reese
05da851eb9
fix(hapi): typo in struct json tags
8 years ago
Adam Reese
19398a2ef1
feat(*): store release History in same namespace as release
...
https://github.com/kubernetes-helm/community/blob/master/helm-v3/003-state.md#namespacing-changes
8 years ago
Adam Reese
c19a4ec704
fix(tests): fix race in releasetesting test
8 years ago
Adam Reese
9aa398a7c5
chore(hapi): add missing license headers
8 years ago
Adam Reese
29e772f631
ref(*): replace TillerVersion with HelmVersion
8 years ago
Adam Reese
4c95185164
ref(*): replace chart.config with []byte
8 years ago
Adam Reese
1e2e65ce54
ref(pkg/releasetesting): simplify test setup
8 years ago
Adam Reese
c5151fb7b3
ref(*): cleanup timestamps in tests
8 years ago
Adam Reese
c5a76deba3
ref(*): use go conventions for naming types
8 years ago
Adam Reese
36536d77ba
ref(*): remove protobuf any type
8 years ago
Adam Reese
91a6ebfed5
ref(*): remove protobuf timestamps
8 years ago
Adam Reese
6345f04190
ref(hapi): convert protobuf to go types
8 years ago
Adam Reese
c233336079
ref(pkg/kube): simplify creating versioned objects
8 years ago
Adam Reese
a78aff8d39
ref(*): improve initializing helm clients
8 years ago
Adam Reese
68c0b6a24a
ref(proto): remove unused protobufs
8 years ago
Adam Reese
883371b8ce
ref(pkg/tiller): simplify exported methods
8 years ago
Adam Reese
7f6fa70a91
ref(*): remove grpc
8 years ago
Adam Reese
8f58c9efdc
ref(*): refactor release testing
8 years ago
Adam Reese
358746fee6
ref(*): remove HELM_HOST
8 years ago
Adam Reese
fc4c095cf0
ref(*): remove pkg/kubernetes from tiller and storage
8 years ago
Adam Reese
496ca54183
ref(*): bypass grpc for helm client
8 years ago
Adam Reese
e9478d8f84
ref(pkg/tiller): remove ReleaseModules interface from tiller
8 years ago
Adam Reese
5715ee43d6
ref(*): bypass grpc when invoking helm list
...
TODO reimplement any paging
8 years ago
Adam Reese
62804848c6
ref(pkg/tiller): refactor out imports of pkg/helm
8 years ago
Adam Reese
dbf1ed0ba4
ref(pkg/helm): cleanup unused code
8 years ago
Adam Reese
6bca9686a1
ref(pkg/helm): allow ReleaseContent to call storage directly
8 years ago
Adam Reese
d52faff7b6
ref(cmd,pkg/helm): remove tiller ping
8 years ago
Adam Reese
1d9c3d4651
ref(cmd,pkg/helm): remove server side version
8 years ago
Matthew Fisher
28d87e1780
Merge pull request #3860 from adshmh/refactor-tiller-release-install-unit-tests-using-stubs
...
refactor tiller release install unit tests using chart and install request stubs
8 years ago
Matthew Fisher
6f02d9e412
Merge pull request #3837 from consideRatio/toYaml-pr
...
toYaml - Fix #3470 and #3410's trailing \n issues
8 years ago
eyalbe4
fa62c476fc
Basic auth credentials from repo not used in install/upgrade/fetch commands #3858
8 years ago
Arash Deshmeh
c22492ff01
fix(helm): refactor tiller release install unit tests using chart and install request stubs
...
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
8 years ago
Matthew Fisher
ae7ce6cad4
Merge pull request #3842 from AdamDang/patch-1
...
Fix some typos
8 years ago
AdamDang
ee9ef91df0
typo fix get->Get's
...
typo fix get->Get's
8 years ago
Nikhil Manchanda
b687e9e879
Merge pull request #3709 from adshmh/fix-output-leak-of-tiller-release-install-test
...
fix output leak from tiller release install test
8 years ago
Taylor Thomas
2a86164c1d
Merge pull request #3829 from stuartleeks/set-rollback-failure
...
Fix: Failed rollback left in PENDING_ROLLBACK state
8 years ago
Steffen Windoffer
8bd42d2645
remove unused servicePort from default ingress template
8 years ago
AdamDang
138de17c64
Fix some typos
...
outout->output
Get's->get
the the->the
8 years ago
Erik Sundell
05a1f7f46c
Updated tests for PR #3837
8 years ago
Erik Sundell
6cdf6cee56
toYaml - Fixes #3470 - trailing \n issue
...
`toYaml` was introducing a new line. It is an issue since the new line is part of a functions output, it can't be whitespace chomped away so it would require a `trimSuffix "\n"` pipe. This commit trims one trailing `\n` from the toYaml output.
8 years ago
Erik Sundell
35132d141c
toYaml - Fixes #3410 - trailing \n issue
...
`toYaml` utilized by `.Files` was introducing a new line. It is an issue since the new line is part of a functions output, it can't be whitespace chomped away so it would require a `trimSuffix "\n"` pipe. This commit trims one trailing `\n` from the toYaml output.
8 years ago
Michelle Noorali
72d54f356d
Merge pull request #3831 from michelleN/fix-3655
...
fix(pkg/tiller): saves all previous computed values on reuseValues
8 years ago
Michelle Noorali
e02802ba95
ref(pkg/tiller): clarify reuseValues comment
8 years ago
Michelle Noorali
9266731dc4
fix(pkg/tiller): reuseValues combines all prev val
...
Resolves #3655
We were seeing that when running helm upgrade with the reuse-values
flag enabled that you could end up in the position where overrides
a.k.a computed values from previous revisions were not being saved on
the updated revision. This left us in a weird position where some
computed values would disappear mysteriously in the abyss. That
happened because computed values from previous revisions weren't merged
with the new computed values every time the reuse-values flag was used.
This PR merges computed values from the previous revisions so you don't
end up in that kind of conundrum.
8 years ago
Stuart Leeks
bac4f45f94
Switch flag to update existing record
8 years ago
Zack Williams
1d3ae54185
Fix #3822
8 years ago
Matthew Fisher
e794c48c3a
Merge pull request #3540 from flant/hook_recreate_delete_policy
...
Implement before-hook-creation delete policy
8 years ago
Timofey Kirillov
b7fe0d0bd7
Add hook-delete-policy ReleaseServer unit tests
...
Cover hook-succeeded, hook-failed and before-hook-creation hook delete policies.
8 years ago
fibonacci1729
ad99c1b749
release list paritioning
8 years ago
fibonacci1729
2dd9b34d28
stream releases when listing
8 years ago
Adam Reese
c574e03ed4
Merge pull request #3721 from adamreese/ref/kube-1.10
...
ref(*): kubernetes v1.10 support
8 years ago
adshmh
1a55161a53
Fix several golint warnings ( #3756 )
...
* fix(helm): fix golint warning due to ApiVersionV1 constant name
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
* fix(helm): fix golint warning due to ResolveChartVersionAndGetRepo comment
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
* fix(helm): fix golint warnings on HttpGetter type and SetCredentials method missing a comment
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
* fix(helm):fix golint warning due to comment on FindChartInAuthRepoURL function
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
* fix(helm): fix golint warning due to RepoFile type name
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
* fix(helm): fix golint warning due to ParseString comment
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
8 years ago
Adam Reese
7c19cb7040
ref(*): kubernetes v1.10 support
8 years ago
Matthew Fisher
850c8714a7
Merge pull request #3661 from bacongobbler/fix-default-connection-timeout
...
add back the default 5 second connection timeout
8 years ago
Alexey igrychev
1d4883bf3c
Implement before-hook-creation delete policy
...
Existing helm.sh/hook-delete-policy annotation variables (hook-failed, hook-succeeded) do not allow to leave failed jobs for debugging without blocking the next job launching: every failed job must be deleted manually before the next related release is launching (installing, updating or rolling back).
New policy, before-hook-creation, removes the hook from previous release if there is one before the new hook is launched and can be used with another variable.
8 years ago
Arturo Contreras
a615f80c92
Adding --set-string flag to force string values.
8 years ago
Eyal Ben Moshe
b6335b7dfe
Authentication support for remote charts repositories ( #3206 )
...
Authentication support for remote charts repositories.
8 years ago
Arash Deshmeh
a43ddcc191
fix output leak from tiller release install test
...
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
8 years ago
Taylor Thomas
dede3ff1e1
Merge pull request #3645 from scriptonist/Issue3290
...
Recover from a tiller pod crash caused due to errors in values.yaml
8 years ago
scriptonist
5311bf40a8
Now returning error and stopping installation
8 years ago
Matthew Fisher
3014b9277f
add back the default 5 second connection timeout
...
When Helm v2.8.2 was released, we made a change to the default connection timeout by supplying a value passed from pkg/helm/environment. This broke support for third party clients relying on pkg/helm because now the default connection timeout is zero. Adding a default 5 second timeout back retains old behaviour, while not breaking backwards compatibility because the connection timeout can still be configured.
8 years ago
Taylor Thomas
2c51155724
Merge pull request #3646 from nicdoye/master
...
Accept .yml files as well as .yaml for templates.
8 years ago
Taylor Thomas
c5f2174f26
Merge pull request #3640 from rajatjindal/docfix
...
fix typo in docs and fake
8 years ago
Taylor Thomas
f19daa16c2
Merge pull request #3631 from gburton1/addProxyCapability
...
Respect http proxy settings when using custom Transport
8 years ago
Matthew Fisher
abe958eb87
Merge pull request #3597 from bacongobbler/upgrade-force-replace
...
replace FAILED deployments with `helm upgrade --install --force`
8 years ago
Nic Doye
73017d7e88
Accept .yml files as well as .yaml for templates.
...
See https://github.com/helm/helm-classic/pull/306 for helm classic
and
https://kubernetes.slack.com/archives/C51E88VDG/p1520588964000061
8 years ago
scriptonist
6cfaa4ebf2
Added code to recover from a tiller pod crash in an event of template render failure
8 years ago
Rajat Jindal
92972b0353
fix typo in docs and fake
8 years ago
Matthew Fisher
13730b0dab
replace FAILED deployments with `helm upgrade --install --force`
...
When using `helm upgrade --install`, if the first release fails, Helm will respond with an error saying that it cannot upgrade from an unknown state.
With this feature, `helm upgrade --install --force` automates the same process as `helm delete && helm install --replace`. It will mark the previous release as DELETED, delete any existing resources inside Kubernetes, then replace it as if it was a fresh install. It will then mark the FAILED release as SUPERSEDED.
8 years ago
Matthew Fisher
250d25fdce
fix protoc
...
k8s.io/kubernetes bumped protobuf to 1643683e1b54a9e88ad26d98f81400c8c9d9f4f9
8 years ago
Matthew Fisher
a66a39a171
fix helm init --wait
8 years ago
Greg Burton
bf7e755ef3
Respect env proxy settings when using custom Transport
8 years ago
Matthew Fisher
b5af54d44c
Merge pull request #3567 from jumal/master
...
Create missing directories when saving files in chartutil.SaveDir
8 years ago
Matthew Fisher
86ae040bfa
Merge pull request #3491 from oilbeater/fix-defer-in-loop
...
defer in loop may cause resource leak
8 years ago
Johnny Bergström
5f1a21bc32
fix(tiller): Supersede multiple deployments ( #3539 )
...
* add test for rolling back from a FAILED deployment
* Update naming of release variables
Use same naming as the rest of the file.
* Update rollback test
- Add logging
- Verify other release names not changed
* fix(tiller): Supersede multiple deployments
There are cases when multiple revisions of a release has been
marked with DEPLOYED status. This makes sure any previous deployment
will be set to SUPERSEDED when doing rollbacks.
Closes #2941 #3513 #3275
8 years ago
Matthew Fisher
9c4fd1abae
Merge pull request #3527 from adshmh/fix-output-leak-from-realsetesting-test-suite
...
Fix the output leak to stdout during build by tests under pkg/releasetesting
8 years ago
Matthew Fisher
1a9e0f0d15
Merge pull request #3535 from adshmh/fix-output-leak-from-pkg-tiller-tests
...
fix the output leak from test code under pkg/tiller
8 years ago
Matthew Fisher
642ad5c13e
Merge pull request #3544 from radu-matei/patch-1
...
Fix minor typo
8 years ago
Matthew Fisher
be1e974ccc
Merge pull request #3494 from Thiht/patch-1
...
docs: Fix FromJson comment
8 years ago
Matthew Fisher
e35bd82170
Merge pull request #3514 from joonix/issue-3512
...
Bump client side grpc max msg size
8 years ago
Adam Reese
fa611fe285
fix(plugins): support installing plugins by relative path ( #3568 )
...
Support installing plugins by relative path
```
helm plugins install .
```
8 years ago
Adam Reese
c314e2e2f1
fix(plugins): support newer git ( #3571 )
...
Newer gits don't like checking out an empty string.
```
empty string is not a valid pathspec. please use . instead if you meant to match all paths
```
8 years ago
Jean-Philippe Courson
5b77335415
Create missing directories when saving files in chartutil.SaveDir
8 years ago
Radu Matei
df041c4b19
Fix minor typo
8 years ago
Arash Deshmeh
5b25eef9e2
fix(helm): fix the bug in test code under pkg/tiller that leaks output to stdout during build
...
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
8 years ago
Johnny Bergström
9bac0a4c30
Update deprecated grpc dial timeout
...
The docs say: use DialContext instead.
8 years ago
Johnny Bergström
98e5006ecf
Bump client side grpc max msg size
...
Set it to match the server side, or the default limit
of 4MB will still apply when upgrading charts.
Fixes #3512
8 years ago
Arash Deshmeh
4fcf69bc52
fix(helm): fix the output leak to stdout during build by tests under pkg/releasetesting,
...
by redirecting output from mock clients.
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>
8 years ago
yank1
76a40fc573
fix a typo in client
8 years ago
Thibaut Rousseau
a59d212512
docs: Fix FromJson comment
8 years ago
oilbeater
757db3a062
defer in loop may cause resource leak
8 years ago
Matthew Fisher
8009115d3a
Merge pull request #3465 from oilbeater/fix/defer-in-loop
...
defer in loop may cause resource leak
8 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.
8 years ago
Matt Farina
6a1dd806ed
Merge pull request #3469 from mattfarina/fix/3322
...
fix(grpc): Fixes issue where message sending limited to 4mb
8 years ago
Matthew Fisher
3c39e72189
Merge pull request #3401 from bacongobbler/revert-kubeconfig-flag
...
Revert "feat(helm): adding kubeconfig flag"
8 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.
8 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
8 years ago
Matthew Fisher
2724387506
Merge pull request #3373 from unguiculus/feature/fix-recreate-part-1
...
Fix pod recreation
8 years ago
Matt Farina
4b578f44ff
Merge pull request #3407 from paolomainardi/feature/3173_fix_wait_for_deployments
...
refs #3173 : add appsv1beta1 and appsv1beta2 apiVersion
8 years ago
Paolo Mainardi
b065ffe1fc
adding other missing apiVersions
8 years ago
Paolo Mainardi
2cee4c68fa
add appsv1beta1 and appsv1beta2 apiVersion
8 years ago
Reinhard Nägele
ea520afd3e
Fix 'getSelectorFromObject'
8 years ago
Matthew Fisher
748dbda29a
Merge pull request #3399 from bacongobbler/fix-release-checklist
...
Fix release checklist
8 years ago
Bin Liu
1e0906ed67
Fix type
8 years ago
Matthew Fisher
f13c4d28b5
Revert "feat(helm): adding kubeconfig flag"
...
This reverts commit 371ff8f26d
.
8 years ago
Matthew Fisher
8604e0b832
bump version to v2.8
...
also add it to the release checklist.
8 years ago
Reinhard Nägele
3d05da0109
Fix pod recreation
8 years ago
Matthew Fisher
14c7be1f92
Merge pull request #3288 from bradbeam/starter
...
feat Making starter templates more versatile
8 years ago
Matt Farina
c359ea66ed
Merge pull request #3350 from mattfarina/fix/3349
...
fix(capabilities): Adding GitVersion to default set
8 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>
8 years ago
Adnan Abdulhussein
f432b8ec60
Merge pull request #3337 from unguiculus/feature/helm-create
...
Update initial chart experience
8 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
8 years ago
Matthew Fisher
077be881c4
Merge pull request #3283 from nicolerenee/concurrency
...
feat(pkg/helm): allow concurrent requests on a shared client
8 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
8 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
8 years ago
Matthew Fisher
e1027fae73
refactor symWalk() into pkg/sympath ( #3199 )
...
* refactor symWalk() into pkg/sympath
* add Go copyright
* remove dead code
8 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
8 years ago
Adam Reese
a31e3b8fe1
ref(*): kubernetes v1.9 support ( #3234 )
8 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
8 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
8 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.
8 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.
8 years ago
Tim Hockin
05d4b30730
Convert registry to k8s.gcr.io
8 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
8 years ago
Andrey Klimentyev
cfb7dfa82c
Typo fix
8 years ago
Nauris Sadovskis
bd8178d50b
Fix package url
8 years ago
Matthew Fisher
51a6f3c326
Merge pull request #3247 from mgalgs/missing-repos-help
...
feat(repo): Improve error message for missing repositories.yaml
8 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
8 years ago
Mitchel Humpherys
6de70254fd
feat(repo): Add test for non-existent repo path
8 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.
8 years ago
Matthew Fisher
1042f6118b
Merge pull request #3122 from cofyc/fix_charts_permission
...
Remove executable permission bit for charts downloaded from remote repos
8 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.
8 years ago
Lukas Eichler
1cebc760a0
Fixed warning for missing formating parameter inside error message.
8 years ago
Lukas Eichler
2c338db1bd
fix(helm): Set template context inside tpl function to outer function.
...
docs(helm): Added documentation about tpl function
8 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 /.
8 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
8 years ago
Federico Gimenez
a607fe798d
iinitialize message only when at least one manifest is kept
8 years ago
Federico Gimenez
93bce130fb
fix signatures
8 years ago
Federico Gimenez
81eaf2f263
only check existence for the possibly kept manifests
8 years ago
Adam Reese
e93b77166d
Merge pull request #3025 from beeradb/fakeclient-releasestatus
...
Improvements to FakeClient: support all CRUD operations
8 years ago
Matthew Fisher
cad89240f3
Merge pull request #3187 from hoesler/fix/chart_downloader
...
fix(helm): resolve relative chart paths
8 years ago
Matthew Fisher
7c79d1c765
Merge pull request #3193 from burdiyan/master
...
ref(helm): improve fullname template in starter chart
8 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.
8 years ago
Alexandr Burdiyan
f432e07126
ref(helm): improve fullname template in starter chart
...
Closes #1186
8 years ago
Christoph Hösler
98e0bd2ecb
refactor: rename function and improve error messages
8 years ago
Christoph Hösler
af9190f956
feature: let FindChartInRepoURL return absolute chart URLs
8 years ago
Matthew Fisher
d655911088
Merge pull request #3041 from jlyheden/feature/plugin-http-installer
...
Add plugin installer from http archive
8 years ago
Helgi Þormar Þorbjörnsson
803f7c706e
add a keepalive of 30s to the client ( #3183 )
8 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.
8 years ago
Hadrien Chauvin
838d780894
update google.golang.org/grpc to 1.7.2 ( #3101 )
8 years ago
Matthew Fisher
4c21a0e739
Merge pull request #3123 from Miouge1/fullnameoverride
...
Add fullnameOverride
8 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.
8 years ago
Kai Chen
bc45e90198
Remove unnecessary parentheses in switch statements ( #3175 )
8 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.
8 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.
8 years ago
Federico Gimenez
e6d907ed28
check if resource is present before adding it to remaining or keep on deletion
8 years ago
Maxime Guyot
c864ba179e
Add fullnameOverride
8 years ago
Yecheng Fu
126f0249a7
Remove executable permission bit for chart files.
8 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.
8 years ago
Adam Reese
d6da76f3cf
Merge pull request #3097 from adamreese/fix/2437-upgrade-release
...
fix(tiller): upgrade last deployed release
8 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
8 years ago
Matthew Fisher
1954b51186
Merge pull request #2621 from gardlt/feature/helm/adding-kubeconifg-flag
...
feat(helm): adding kubeconfig flag
8 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
8 years ago
Matt Tucker
b8734a173e
fix(docs): fix code comment for ReuseValues()
8 years ago
gardlt
371ff8f26d
feat(helm): adding kubeconfig flag
8 years ago
Christian Jauvin
8775f632f2
Fix for relative chart path support in index.yaml
8 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
8 years ago
Adam Reese
08c1144f5e
chore(*): bump version to v2.7.0
8 years ago
Johan Lyheden
4831089500
Add tests to HTTPInstaller, update plugin documentation
8 years ago
Johan Lyheden
f1a08adb41
Update to comply with linter rules and gofmt
8 years ago
Johan Lyheden
d43d5ab452
Add plugin installer from http archive
8 years ago
Adam Reese
13912b3cc0
Merge pull request #3008 from adamreese/ref/kube-1.8
...
ref(*): kubernetes v1.8 support
8 years ago
Taylor Thomas
500c9983b5
Merge pull request #3019 from thomastaylor312/fix/crd_ordering
...
fix(tiller): Adds CRD ordering
8 years ago
Brad Bowman
e974ab31d8
[fakeclient-releasestatus] Move ReleaseMock and associates into helm package and update tests to use them
8 years ago
Adam Reese
2b43ecd405
ref(*): kubernetes v1.8 support
8 years ago
Brad Bowman
8d5cf13f69
Add test case for deleting the only existing item
8 years ago
Brad Bowman
cab3057ee3
Add support for all CRUD operations + test coverage for delete and install
8 years ago
Brad Bowman
61f2b24dca
Update comment for accuracy
8 years ago
Brad Bowman
76b70ec70f
gofmt -s
8 years ago
Brad Bowman
3bc7c44f05
Make ReleaseContent and ReleaseStatus respect release names in the fake client
8 years ago
Taylor Thomas
29c3b5276f
fix(tiller): Adds CRD ordering
...
This adds CRD ordering into the explicit kind sorter list
Closes #2925
8 years ago
Remington Reackhof
9af1018bd3
Add secrets storage backend for releases
8 years ago
Taylor Thomas
d4ccef7ba7
Merge pull request #2999 from bonifaido/getterConstructor-err
...
fix(helm): invoking getterConstructor returns downstream error now
8 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
8 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
8 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
8 years ago
Matthew Fisher
d4f5a133dd
Merge pull request #2960 from HotelsDotCom/upgrade-failure-record-release-as-update
...
Correctly persists Release upgrade failure
8 years ago
Taylor Thomas
f8e9741578
Merge pull request #2984 from thomastaylor312/feat/add_missing_test
...
fix(sorter): Adds missing unit test
8 years ago
Matthew Fisher
e3e1116eaa
Merge pull request #2857 from llsheldon/fix-dependency-alias
...
fix(helm):Fix dependency aliaes not working
8 years ago
Matthew Fisher
3799c477d1
Merge pull request #2938 from bacongobbler/remove-facebookgo
...
Remove facebookgo dependencies
8 years ago
Nandor Kracser
a28e5dd2b3
fix(helm): invoking getterConstructor returns downstream error now
8 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
8 years ago
Matthew Fisher
b600ea475e
Merge pull request #2982 from HotelsDotCom/pkg-lint-test-panics
...
Avoid panics if test is failing
8 years ago
Taylor Thomas
acd113ec21
Merge pull request #2978 from HotelsDotCom/detailed-wait-log
...
More helpful wait messages in tiller logs
8 years ago
Taylor Thomas
333f8dd354
fix(sorter): Adds missing unit test
...
Adds a unit test that was missed as part of #2961
8 years ago
Maxim Ivanov
a6872c124a
Avoid panics if test is failing
8 years ago
Taylor Thomas
8c2c9fcf6e
Merge pull request #2961 from NauxLiu/patch-1
...
Load StorageClass before PersistentVolume is loaded.
8 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
8 years ago
Maxim Ivanov
cd8bae1df5
More helpful wait messages in tiller logs
8 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.
8 years ago
Matthew Fisher
f278675f98
remove references to facebookgo/symwalk
8 years ago
Matthew Fisher
211f5f6f33
Revert "fix(helm): resolve symlinks when loading chart"
...
This reverts commit edd4e56112
.
8 years ago
Matthew Fisher
9d1db2851e
Revert "Write repo file using atomicfile"
...
This reverts commit 965cb7fd1c
.
8 years ago
Maxim Ivanov
ff5acc9bd8
Make Memory driver to store copy of releases to stop hiding storage errors during tests
8 years ago
Matthew Fisher
cf3fc21e81
Merge pull request #2898 from zlabjp/kube-version
...
feat(helm): add --kube-version flag to helm template
8 years ago
NauxLiu
c3f9120e02
Load StorageClass before PersistentVolume is loaded.
8 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.
8 years ago
carlory
ad2d8e434a
fix fromjson comment
8 years ago
carlory
b9d3974df0
fix func comment
8 years ago
devinyan
3458ad8e6a
Add the status of pod when using "Helm status" command
8 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).
8 years ago
Kazuki Suda
3508cebbf6
Use the same defaults as done in helm lint for Capabilities
8 years ago
Michelle Noorali
84bb77f183
Merge pull request #1997 from AmandaCameron/issue/1401
...
Take a crack at Issue #1401
8 years ago
Amanda Cameron
415e52bf55
Add support for sub-directory scanning as in issue #1401
8 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
8 years ago
Adnan Abdulhussein
8fd8a7c324
fix(create): port-forward should use internal port
...
cc @james-w
8 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.
8 years ago
Matthew Fisher
1ac5cd78de
Merge pull request #2901 from bacongobbler/bump-lint-k8s-version
...
bump k8s version in helm lint to 1.7
8 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.
8 years ago
Matthew Fisher
8b512808df
bump k8s version in helm lint to 1.7
8 years ago
Matthew Fisher
bb5db61325
Merge pull request #2781 from jascott1/i2755-template
...
feat(helm): add `template` command
8 years ago
DoctorZK
608fd7318a
fix(tiller): fix a warning bug in hook annotation
8 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
8 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
8 years ago
xuhaigang
b79033a715
Merge branch 'fix-update-delete-subcharts' of https://github.com/rocky-nupt/helm into fix-update-delete-subcharts
8 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
8 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
8 years ago
Matt Butcher
a736c2a3fe
Merge pull request #2636 from technosophos/feat/2332-max-history
...
feat(tiller): limit number of versions stored per release
8 years ago
Kira
4c1a47480e
feat(2450): add annotations to chart metadata ( #2671 )
...
* feat(2450): add options to chart metadata
8 years ago
Adam Reese
5bc7c619f8
chore(*): bump to v2.6.0
8 years ago
Taylor Thomas
7a6dbfea49
Merge pull request #2790 from nebril/dependency-update-chart-deletion
...
Delete old deps after chart deps are updated
8 years ago
Justin Scott
f239050996
Update kind subsort to sort unknown kinds alphabetically.
...
Remove length check from subsort test. Add check for unknowns.
8 years ago
Justin Scott
87d2b7ab67
Add test for kind sorter sub sort
8 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
8 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
8 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.
8 years ago
Maxime Guyot
bbf2d6b7e8
Fix the default NOTES.txt
8 years ago
Taylor Thomas
6ab706ae6c
Merge pull request #2768 from JarreyZhou/master
...
feat(helm):use a variable for labels to get tiller pod
8 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
8 years ago
Taylor Thomas
8eaed9d3a7
Merge pull request #2729 from rbwsam/fix/leading_zeros
...
fix(pkg/strvals): preserve leading zeros in vals
8 years ago
Taylor Thomas
e5ea075271
Merge pull request #2727 from Miouge1/master
...
Updating default values.yaml comments
8 years ago
Jerry Zhou
23905f1494
feat(helm):use a variable for labels to get tiller pod
8 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
Adam Reese
a29e610938
fix(helm): fix flag parsing once and for all
8 years ago
Aishwarya Thangappa
915e7023c4
Added more granular statuses
8 years ago
Aishwarya Thangappa
9d67d8b954
Added PENDING status to helm install
8 years ago
Aishwarya Thangappa
3d4361ca1f
Modified release_update and release_rollback
8 years ago
Aishwarya Thangappa
a9c16fc964
Added PENDING status to status.proto and regenerated the pb.go files
8 years ago
Adam Reese
9325d136d4
fix(tests): fix sorting hooks test flake
8 years ago
Taylor Thomas
0a20ed73be
Merge pull request #2644 from alonl/master
...
Fix a bug causing 'helm depndency update' to delete required charts
8 years ago
Steven E. Harris
2aaad8242e
Merge pull request #2666 from adamreese/ref/cleanup-tests
...
ref(tests): simplify kind sorter test
8 years ago
Steven E. Harris
e028be065d
Merge pull request #2650 from arschles/apiservice
...
Add the "APIService" kind to the installation sort order
8 years ago
Aaron Schlesinger
58a2d7b5c1
Adding APIService to the kind sorter tests
8 years ago
Adam Reese
0d1f6b981f
ref(tests): simplify kind sorter test
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
Adam Reese
845e990143
feat(*): use kubernetes 1.7
8 years ago
Matt Butcher
883da97396
Merge pull request #2607 from sushilkm/issues/2599
...
Adds charts in "charts\" directory to dependencies
8 years ago
Aaron Schlesinger
5ff75f84a6
Adding APIService to the sort order
...
This ensures that APIService resources are installed last and
uninstalled first
8 years ago
Alon Lavi
04a7e241e6
Fix a bug causing 'helm depndency update' to delete required charts
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
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
fibonacci1729
a4d05fd2b2
ref(tiller): refactor tiller APIs into logical files
...
*ReleaseServer.UninstallRelease -> tiller/release_uninstall.go
*ReleaseServer.ListReleases -> tiller/release_list.go
8 years ago
fibonacci1729
b5c213d5dd
ref(tiller): refactor tiller APIs into logical files
...
*ReleaseServer.RollbackRelease -> tiller/release_rollback.go
8 years ago
fibonacci1729
62c4b152fd
ref(tiller): refactor tiller APIs into logical files
...
*ReleaseServer.GetReleaseContent -> tiller/release_content.go
*ReleaseServer.GetReleaseStatus -> tiller/release_status.go
8 years ago
fibonacci1729
3b3439f67d
ref(tiller): breakout ReleaseServer APIs into logical files.
...
*ReleaseServer.RunReleaseTest -> tiller/release_testing.go
*ReleaseServer.GetVersion -> tiller/release_version.go
8 years ago
Adam Reese
1e8ebae249
fix(plugins): add error when updating modified plugins
...
If git has autosetuprebase enabled pulling will fail without a
descriptive error message.
8 years ago
Adam Reese
15254e4c5c
feat(*): add generic logging interface ( #2394 )
...
Adds a generic logging interface to importable packages.
8 years ago
lead4good
bb4be33331
fixed fromYaml | toJson
8 years ago
Matt Butcher
2d192b1b1b
Merge pull request #2408 from technosophos/fix/2407-devel-releases
...
fix(helm): add --devel flag to allow dev releases again
8 years ago
Matt Butcher
f9c9e9d6cb
Merge pull request #2455 from technosophos/fix/2393
...
fix(lint): add KubeVersion and TillerVersion to linter
8 years ago
Matt Butcher
28ec92355b
fix(helm): add --devel flag to allow dev releases again
...
2.4.0 stopped installing development versions by default. This allows
the user to re-enable that support by setting --devel.
Closes #2407
8 years ago
Michelle Noorali
a53c29aee7
Merge pull request #2425 from michelleN/helm-test-error-codes
...
fix(*): return non-zero exit code on test failure
8 years ago
Adam Reese
595afc5c17
Merge pull request #2449 from rvbaz/write-index-atomic
...
Write repo file using atomicfile
8 years ago
Matt Butcher
d863d9a886
fix(lint): add KubeVersion and TillerVersion to linter
...
Closes #2393
8 years ago
Raphael Badin
965cb7fd1c
Write repo file using atomicfile
...
This prevents clients from seeing half-written files because
on POSIX systems renaming a file is an atomic operation.
Drive-by: Add test for repo file permission.
8 years ago
Sushil Kumar
ff42dadde4
Adds update option to plugin command ( #2410 )
...
* Adds update option to plugin command
Fixes issues/2385 - helm install silently updates the plugin, if it pre-existed
* Added tests for new methods for plugin update
* Updated docs
* Updated review comments :)
* Return error exit code when there is error
8 years ago
Michelle Noorali
488ca6fdd8
fix(*): return non-zero exit code on test failure
...
* resolves #2089
8 years ago
Adam Reese
35aabdb61a
Merge pull request #2423 from FengyunPan/add-check
...
Add check for getting kubeclient
8 years ago
Taylor Thomas
7d5f4e53db
Merge pull request #2430 from thomastaylor312/fix/deployment_wait
...
fix(tiller): Fixes bug with `--wait` and updated deployments
9 years ago
Matt Butcher
e8aa8d3678
Merge pull request #2405 from sushilkm/linting-fixes
...
Fixed issues reported by test-style
9 years ago
Taylor Thomas
d310612526
fix(tiller): Fixes bug with `--wait` and updated deployments
...
The current methodology generated its own RS slice instead of using
a helper method that uses a `clientset`. This caused some issues where
`FindNewReplicaSet` always returned `nil`. This switches the method and
removes some unneeded API calls and code.
Closes #2426
9 years ago
fengyun.pan
7887185984
Add check for getting kubeclient
9 years ago
Sushil Kumar
24157e4aed
Updated review comments :)
9 years ago
Sushil Kumar
c84fb11a68
Errors out in case requested plugin exists
...
Partially fixes issues/2385 - helm install silently updates the plugin, if it pre-existed
9 years ago