backport of #7558 from master

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

This is a backport for PR 7443.

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

Changes to the Kubernetes API server and kubectl libraries caused
the status to no longer display when helm status was run for a
release. This change restores the status display.

Generation of the tables for display was moved server
side. A request for the data as a table is made and a kubectl
printer for tables can display this data. Kubectl uses this setup and
the structure here closely resembles kubectl. kubectl is still
able to display objects as tables from prior to server side
printing but only prints limited information.

Note, an extra request is made because table responses cannot be
easily transformed into Go objects for Kubernetes types to work
with. There is one request to get the resources for display in
a table and a second request to get the resources to lookup the
related pods. The related pods are now requested as a table as
well for display purposes.

This is likely part of the larger trend to move features like
this server side so that more libraries in more languages can
get to the feature.

Closes #6896

Signed-off-by: Matt Farina <matt@mattfarina.com>
(cherry picked from commit e8396c92)

* align both formats behaviors and now they will just differ in how to discover their paths
* add coverage for exports format and fix expected assertions for parent-child format to match the logic child values always wins
* just partially revert dda84976, this way parents values could be overridden when coalescing
* after getting better coverage we were able to refact both formats behaviors by merging their propagation logics into a single code path.

(cherry picked from commit b7d2947d)

* fix #7182

Signed-off-by: zwwhdls <zwwhdls@hotmail.com>

* fix typo

Signed-off-by: zwwhdls <zwwhdls@hotmail.com>

* add testcase

Signed-off-by: zwwhdls <zwwhdls@hotmail.com>

* fix conflict subdirectory when untardir is the clashing directory

Signed-off-by: zwwhdls <zwwhdls@hotmail.com>
(cherry picked from commit 63e1c9bc)

Signed-off-by: yury <yury@panoply.io>
(cherry picked from commit dc2f5bcb)

For some reason, many users experince corrupted storage with the
ConfigMaps storage backend. Specifically, several Releases are marked as
DEPLOYED. This patch improved handling of such situations, by taking the latest
DEPLOYED Release. Eventually, the storage will clean itself out, after
the corrupted Releases are deleted due to --history-max.

Closes #6031

Signed-off-by: Cristian Klein <cristian.klein@elastisys.com>
(cherry picked from commit 840e0e27)

Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>
(cherry picked from commit ab797325)

Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>
(cherry picked from commit c3060582)

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

In several of the job checks and other conversions we were using legacyscheme.
I don't know why it was working before, but I am guessing something changed
between k8s 1.15 and 1.16. To fix I changed the references to use the default
scheme in client-go

Signed-off-by: Taylor Thomas <taylor.thomas@microsoft.com>
(cherry picked from commit e2894a18)

This happened to be a bug we identified in Helm 3 and did not check if
it existed in Helm 2. The improved logic for job waiting used an automatic
retry. However, when we were creating the watcher, we were listing on everything
of that same api version and kind. So if you had more than 1 hook and the first
was successful, it would think everything was successful. I have validated that
this now fails as intended if a job is failing

Closes #6767

Signed-off-by: Taylor Thomas <taylor.thomas@microsoft.com>
(cherry picked from commit 77c97342)

Signed-off-by: Yagnesh Mistry <ysh@live.in>
(cherry picked from commit b66ae749)

Signed-off-by: Yagnesh Mistry <ysh@live.in>
(cherry picked from commit 465870a2)

Signed-off-by: Yagnesh Mistry <ysh@live.in>
(cherry picked from commit 7786caa6)

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 394ba588)

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 16852f04)

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 45d72335)

I found another package that took code from a BSD licensed open source project. I copied their approach for reusing that license.

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 5ab89c97)

This is probably not the correct apporach to using bsd licensed code under an apache license.

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 55ea6987)

For some reason, gofmt -s -w produced a different result than running gofmt on my host os

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 3d78e6a1)

Neeted to run formatting on fs_test.go

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit c292ba5c)

Unused var was in fs.go. Removed it.

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit aacffe5f)

Our style settings don't like underscores in variable names, so I switched to camel case. Also, err was unused in some places, so I made an error a fail condition for some test.

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 3d15ec71)

Root can access all files, so the tests for inaccessible files were failing. Now we will skip these tests when running as root.

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 3469c9a1)

Recreated the features of the test package in dep that were necessary to run the tests in the fs package.

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 60bd2a56)

The fs package tests are going to be a lot of work to get working without the internal test package used by golang/dep, so I removed all of the unused ones by this project to make the work easier.

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 51dbd7c1)

helm dependency upgrade wasn't working on certain file system because it assumes that os.rename is available. Since rename isn't available for subfolders in docker containers, I ripped a fallback rename strategy from dep (https://github.com/golang/dep/blob/5b1fe9e6d89372487d0aac78d9c5362a517857e7/internal/fs/fs.go#L103-L118

) that works in docker.

Signed-off-by: Brice Rising <brice.rising@slalom.com>
(cherry picked from commit 94d87ef9)

Signed-off-by: Matt Farina <matt@mattfarina.com>

Update language on search usage

This is a followup to PR #6780 that implements some discussed
language changes.

Signed-off-by: Matt Farina <matt@mattfarina.com>

fix(repo/search): fix helm repo search command to display proper versions

Fix error when loading irregular files

Signed-off-by: Matt Farina <matt@mattfarina.com>

ref(glide.yaml): kubernetes 1.16.2

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

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

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

fix(wait): Removes ingress checks

Closes #6751

After doing some more digging, I found out that updating the status
of an `Ingress` object is completely optional. Because of this, Helm
cannot support ingresses with the `--wait` flag because there is no
standard way to identify that they are ready

Signed-off-by: Taylor Thomas <taylor.thomas@microsoft.com>