Signed-off-by: Takao Shibata <chise.alter.pasta@gmail.com>

Signed-off-by: zhoulin xie <zhoulin.xie@daocloud.io>

Add content on how community members can help review PRs

fix: ignore pax_global_header "file" in chart validation

Signed-off-by: Geoff Baskwill <me@geoffbaskwill.ca>

* update nginx example to work multi-arch
Signed-off-by: Patrick Lang <patrick.lang@hotmail.com>

* Fix alpine to use a multiarch tag
Signed-off-by: Patrick Lang <patrick.lang@hotmail.com>

As noted in Slack by a community member, release names with periods are
considered usable. Switching to RFC 1123 subdomain verification
continues to block bad release names like BAD_NAME, but allows names
like good.name to continue working.
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>

The 'cmd/helm/delete.go' file permission is 755, change to same as another file.
Signed-off-by: Koichi Shiraishi <zchee.io@gmail.com>

Updated text to better elaborate on he process and also some small other
nits/updates.
Signed-off-by: Martin Hickey <martin.hickey@ie.ibm.com>

Signed-off-by: ds-ms <desattir@microsoft.com>

#2715 add child NOTES.txt rendering

Signed-off-by: Jon Leonard <jon.leonard@sanofi.com>

* fix: perform extra validation on paths in tar archives
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>

* fix: Cover a few Windows cases and also remove a duplicate tar reader
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>

* fix: removed debug output
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>

* fix: Expand again preserves the files verbatim

Also added tests for Expand
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>

* fix: add license block and remove println
Signed-off-by: Matt Butcher <matt.butcher@microsoft.com>

Signed-off-by: Axel Köhler <koehler.ax3l@gmail.com>

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

Signed-off-by: JoeWrightss <zhoulin.xie@daocloud.io>

* 5138 Recommended using crd-install instead of pre-install
Signed-off-by: Eric Thiebaut-George <ethiebautgeorge@tycoint.com>

* 5138 Updated section title
Signed-off-by: Eric Thiebaut-George <ethiebautgeorge@tycoint.com>

doc(release_checklist): Adds steps for categorizing changelogs

Closes #5119

Also includes some formatting fixes to wrap lines at 80 chars
Signed-off-by: Taylor Thomas <taylor.thomas@microsoft.com>

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

* autohelm we renamed to reckoner for copyright concerns. This commit renames autohelm in the documentation
Signed-off-by: EJ Etherington <ej@reactiveops.com>

* moving to be in alphabetical order
Signed-off-by: EJ Etherington <ej@reactiveops.com>

This reverts commit e2a0e7fa.
Signed-off-by: Matthew Fisher <matt.fisher@microsoft.com>

Fix(helm): Use spaces in ingress template

The rest of the ingress template was using spaces, just these two lines
were tabs.
Signed-off-by: Alex Humphreys <alex.humphreys@here.com>

chore(deps): bump kubernetes to v1.13.1

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

Renaming Issue Templates correctly and adding a pull request template

Signed-off-by: JoeWrightss <zhoulin.xie@daocloud.io>

Signed-off-by: Kévin Dunglas <dunglas@gmail.com>

Signed-off-by: Ben Tyler <benjamin.tyler@booking.com>

ref(*): kubernetes v1.13 support

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

Remove k8s.io/kubernetes deps for simple pod and svc util methods

Signed-off-by: tariqibrahim <tariq.ibrahim@microsoft.com>

* Refactor test run to separate method

This will allow us to parallelise it more easily
Signed-off-by: Frank Hamand <frankhamand@gmail.com>

* Add --parallel flag to helm test

(No functionality in this commit)
Signed-off-by: Frank Hamand <frankhamand@gmail.com>

* Run helm tests in parallel with --parallel flag
Signed-off-by: Frank Hamand <frankhamand@gmail.com>

* Add a mutex to helm test message streams

This is to protect against data races when running tests in parallel.
Signed-off-by: Frank Hamand <frankhamand@gmail.com>

* Add tests for --parallel flag
Signed-off-by: Frank Hamand <frankhamand@gmail.com>

* Add concurrency limit for parallel helm tests
Signed-off-by: Frank Hamand <frankhamand@gmail.com>

* Add test for concurrency limit
Signed-off-by: Frank Hamand <frankhamand@gmail.com>

* Fix rebase introduced errors
Signed-off-by: Frank Hamand <frankhamand@gmail.com>

* Return empty string instead of nil when linting on required

This allows lint to work in scenarios when required is used in secrets or it's output is passed to another function.
Due to lint mode no longer failing on missing value in required it is passing nil through which not all functions can accept.

Fixes #4747
Signed-off-by: Helgi Þorbjörnsson <helgith@gmail.com>

* Apply suggestions from code review
Co-Authored-By: helgi <70530+helgi@users.noreply.github.com>
Signed-off-by: Helgi Þorbjörnsson <helgith@gmail.com>

* Add tests
Signed-off-by: Helgi Þorbjörnsson <helgith@gmail.com>

Make the current check for the number of templates on create more
resilient by using a varible to store the expected number
of templates. In addition, fix the error message so that it displays
the correct number expected of templates.

Closes #5009
Signed-off-by: Henry Nash <henry.nash@uk.ibm.com>

Provide comment for constant to avoid golint warning.

Closes #5020
Signed-off-by: Henry Nash <henry.nash@uk.ibm.com>

ref(kube): Gets rid of superfluous Sprintf call

Signed-off-by: tariqibrahim <tariq.ibrahim@microsoft.com>