Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>

See https://github.com/helm/chart-testing/issues/48Signed-by: Scott Rigby <scott@r6by.com>
Signed-off-by: Scott Rigby <scott@r6by.com>

* Fail on failed chart repository update

* Fix failed test

* Add repo update strict flag

Solves #3722 by making the changes in #3539 more compatible with the previous behavior.

This gives a recovery option for "oops I deleted my helm release" by allowing rollback, which is intended to be a working feature of helm. Note that purging releases removes the history required to rollback, so this doesn't work in that case. However, purging takes significantly more time, so it's harder to accidentally purge everything.
Signed-off-by: Brent <bmperrea@gmail.com>

Upgrading a release and override existing values doesn't work as expected for nested values. Maps should be merged recursively, but currently maps are treated just like values and replaced at the top level.

If the existing values are:
```yaml
resources:
  requests:
    cpu: 400m
  something: else
```
and an update is done with ```--set=resources.requests.cpu=500m```, it currently ends up as
```yaml
resources:
  requests:
    cpu: 500m
```
but it should have been
```yaml
resources:
  requests:
    cpu: 500m
  something: else
```

This PR updates the way override values are merged into the existing set of values to merge rather than replace maps.

Closes: #4792
Signed-off-by: Morten Torkildsen <mortent@google.com>

fix(helm): Use line breaks consistently in status output

Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>

Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>

* note on naming of resources
Signed-off-by: Ryan Dawson <ryan.dawson@alfresco.com>

* put whitespace back in
Signed-off-by: Ryan Dawson <ryan.dawson@alfresco.com>

* put whitespace back
Signed-off-by: Ryan Dawson <ryan.dawson@alfresco.com>

* put whitespace back
Signed-off-by: Ryan Dawson <ryan.dawson@alfresco.com>

* clarify template fullname
Signed-off-by: Ryan Dawson <ryan.dawson@alfresco.com>

* fix formatting problem by escaping underscore
Signed-off-by: ryandawsonuk <ryandawson@cantab.net>

* no need to change whitespace elsewhere in doc
Signed-off-by: ryandawsonuk <ryandawson@cantab.net>

* remove import, export, add backup instead
Signed-off-by: Maor <maor.friedman@nuvo-group.com>

* add helm plugin utility functions
Signed-off-by: Maor <maor.friedman@nuvo-group.com>

* move helm-plugin-utils into helm plugins section
Signed-off-by: Maor <maor.friedman@nuvo-group.com>

* fix(tiller) added mock kube client to return failure on delete
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>

* fix(tiller) added object's name to delete release error message
Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>

The essence of this commit is to help people get started with a better
indentation practice than this:

```yaml
spec:
  labels:
{{ toYaml .Values.labels | indent 4 }}
```

The previous indentation practice is harder to read. Instead this commit
introduces an indentation practice using `nindent` like this:

```yaml
spec:
  labels:
    {{- toYaml .Values.labels | nindent 4 }}
```
Signed-off-by: Erik Sundell <erik.i.sundell@gmail.com>

for command helm template and helm install
when cusotmer input invalid release name, return a error
Signed-off-by: jliao <jliao@alauda.io>

Signed-off-by: Arash Deshmeh <adeshmeh@ca.ibm.com>

Signed-off-by: rimas <rmocius@gmail.com>

Signed-off-by: Timothy Hobbs <timothy@hobbs.cz>

Added `--classic` to the snap install command.
Signed-off-by: Mike Garuccio <mgaruccio@gmail.com>

Signed-off-by: Dr Nic Williams <drnicwilliams@gmail.com>

Remove the engine `currentTemplates` field which was shared state
across threads and thus not thread safe, and instead just pass these
reference templates as parameters down recursively.

Closes #4819
Signed-off-by: Sean Eagan <sean.eagan@att.com>

Signed-off-by: Daniel M Barlow <danielmbarlow@gmail.com>

Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>

Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>

Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>

ref(kube): use external api types where possible

Signed-off-by: Adam Reese <adam@reese.io>

Signed-off-by: Zachary Seguin <zachary@zacharyseguin.ca>

While investigating a tiller crash on v2.10.0 (see recent comments in #3125), I pulled down the code
and wrote a test replicating the crash I was experiencing. I then
realized that the crash had been fixed, and was able to locate the fix
in #4630 after running a quck bisect.

Since there don't appear to be any tests that cover this crash, and I
had written one myself, I figured I might as well put up a PR for it.

Here's what the test failure on v2.10.0 looks like:
```
-- FAIL: TestUpdateReleasePendingInstall_Force (0.00s)
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
        panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x50 pc=0x1d128d8]

goroutine 235 [running]:
testing.tRunner.func1(0xc420493c20)
        /usr/local/Cellar/go/1.10/libexec/src/testing/testing.go:742 +0x29d
panic(0x1eb8d80, 0x2a12db0)
        /usr/local/Cellar/go/1.10/libexec/src/runtime/panic.go:505 +0x229
k8s.io/helm/pkg/tiller.(*ReleaseServer).performUpdateForce(0xc4208210b0, 0xc4202c6dc0, 0x0, 0x0, 0x2174220)
        /Users/mattrasmus/go/src/k8s.io/helm/pkg/tiller/release_update.go:166 +0x188
k8s.io/helm/pkg/tiller.(*ReleaseServer).UpdateRelease(0xc4208210b0, 0x2191780, 0xc420820f30, 0xc4202c6dc0, 0x29aeb90, 0x38, 0x2d2)
        /Users/mattrasmus/go/src/k8s.io/helm/pkg/tiller/release_update.go:43 +0x245
k8s.io/helm/pkg/tiller.TestUpdateReleasePendingInstall_Force(0xc420493c20)
        /Users/mattrasmus/go/src/k8s.io/helm/pkg/tiller/release_update_test.go:549 +0x120
testing.tRunner(0xc420493c20, 0x20e5c70)
        /usr/local/Cellar/go/1.10/libexec/src/testing/testing.go:777 +0xd0
created by testing.(*T).Run
        /usr/local/Cellar/go/1.10/libexec/src/testing/testing.go:824 +0x2e0
FAIL    k8s.io/helm/pkg/tiller  0.118s
```
Signed-off-by: Matt Rasmus <mrasmus@betterworks.com>

Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>
(cherry picked from commit 3a551d01)

Update circleCI badge to more consistent shield.

Update quickstart.md

Signed-off-by: Maor <maor.friedman@nuvo-group.com>

Signed-off-by: Bartel Sielski <bsielski@nalys-group.com>

Signed-off-by: masahiro331 <m_fujimura@r.recruit.co.jp>

Signed-off-by: Louis-Etienne Dorval <ledor473@hotmail.com>

* Update Contributing.md to remove mention of CLA
Signed-off-by: Marc Khouzam <marc.khouzam@ville.montreal.qc.ca>

* Add hyperlink to git page in CONTRIBUTING.md
Signed-off-by: Marc Khouzam <marc.khouzam@ville.montreal.qc.ca>

The output from helm status does not have consistent use of line breaks.
For some resources there is a line break after the kind header, for
others there is not. This is caused by how the printer handles column
headers. This removes a line break for all but the first resource listed.
Signed-off-by: Morten Torkildsen <mortent@google.com>

Signed-off-by: Alpha <jhexe2002@gmail.com>

Update of the client-go package changed the status output to only include
the age of resources. The new printer in client-go only formats the output
to include details of specific resources if the internal representation
of resources are passed into the printer. This PR updates helm to convert
resources to the internal type before printing.

Closes #4712
Signed-off-by: Morten Torkildsen <mortent@google.com>

Updated the MySQL release, and put more output that gets sent to
STDOUT with `helm install stable/mysql`
Signed-off-by: JJ Asghar <jja@ibm.com>

Signed-off-by: Marc Khouzam <marc.khouzam@ville.montreal.qc.ca>