[stable/gcloud-endpoints] #1785 namespace defined templates with char…

[kevinschumacher]

…t name
#1785

[dhilipkumars]

/assign
/ok-to-test

[dhilipkumars]

/test pull-charts-e2e

[dhilipkumars]

/test pull-charts-e2e

[kevinschumacher]

/test pull-charts-e2e

[kevinschumacher]

/retest

[kevinschumacher]

@dhilipkumars the tests are failing because the .Values.image url is no longer valid. I have submitted a PR to fix that here: #2321

[dhilipkumars]

cc: @ultimateboy could you suggest? The CI fails as the image seems to be invalid anymore.

[dhilipkumars]

@kubernetes/charts-maintainers Im going to merge this inspite of failing test, the chart is resonably un- maintained.

[unguiculus]

/retest

[unguiculus]

No idea why this still fails.

[unguiculus]

@ultimateboy Could you have a look at this?

[dhilipkumars]

Should we just ignore this chart and move on ?

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kevinschumacher
We suggest the following additional approver: seanknox

Assign the PR to them by writing /assign @seanknox in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these OWNERS Files:

• OWNERS

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

[mattfarina]

Do we need to increase the build time?

/retest

[unguiculus]

Increasing the timeout won't help. The pods don't come up.

I0116 16:04:27.748] gcloud--4730-gcloud-endp-d46c784cf-wvz2w   0/1       CrashLoopBackOff   6         9m

If @ultimateboy does not react, I'd deprecate the chart. The chart was alread broken which was revealed in this PR.

[timstoop]

/retest
I'm going to assume nothing has really changed here, so if this test fails again, I'm going to start the process of deprecating the chart, as described in https://github.com/kubernetes/charts/blob/master/PROCESSES.md.

[k8s-ci-robot]

@kevinschumacher: The following test failed, say /retest to rerun them all:

Test name	Commit	Details	Rerun command
pull-charts-e2e	dab6b70	link	/test pull-charts-e2e

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[stale]

This issue is being automatically closed due to inactivity.