-
Notifications
You must be signed in to change notification settings - Fork 16.9k
cert-manager: fast-forward to upstream 9a8f4371 #10830
cert-manager: fast-forward to upstream 9a8f4371 #10830
Conversation
c12ba87
to
7617292
Compare
* Bump version strings for v0.6.0 (cert-manager/cert-manager#1247) * Set CA sync backoff limit & add verifying installation docs (cert-manager/cert-manager#1245) * Added securityContext to cert-manager deployment pods in helm chart (cert-manager/cert-manager#1237) * Move Helm chart to deploy/charts/cert-manager (cert-manager/cert-manager#1235) * chart: allow deployment strategy customization * webhook: remove hardcoded image (cert-manager/cert-manager#1039) * Remove crd-install annotation from Challenges and Orders * Add Challenge and Order CRDs * Add certificates and issuers to aggregated RBAC for view, edit and admin ClusterRoles. (cert-manager/cert-manager#872) * Use fixed image tag for ca-helper and add description Signed-off-by: James Munnelly <james@munnelly.eu>
7617292
to
7b96b5b
Compare
/retest |
1 similar comment
/retest |
@@ -14,6 +14,10 @@ spec: | |||
matchLabels: | |||
app: {{ template "cert-manager.name" . }} | |||
release: {{ .Release.Name }} | |||
{{- with .Values.strategy }} | |||
strategy: | |||
{{- . | toYaml | nindent 4 }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Think this is meant to say indent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought this too 😄 but it in fact adds a newline along with the indent
@unguiculus looks like the e2e test is trying to install a chart inside the directory of this chart ( |
To clarify, this sub-chart should be installed, and is installed just fine as part of the main cert-manager chart. The issue is as @prydonius says, the e2e script also attempts to install the 'webhook' chart on its own (seemingly because it treats the sub-chart as an independent chart) I guess this is down to the way the chart-testing code discovers charts that need testing? |
@munnerz have a look at the failing test? |
Just to clarify, this is an example of the actual test failure:
https://gubernator.k8s.io/build/kubernetes-jenkins/pr-logs/pull/helm_charts/10830/pull-charts-e2e/19312/
I added a 'test' commit to try something out, hence the current tip of this
branch is failing for a different reason right now.
But the test failure linked here is the real problem!! 😬
…On Wed, 23 Jan 2019 at 07:41, David J. M. Karlsen ***@***.***> wrote:
@munnerz <https://github.com/munnerz> have a look at the failing test?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#10830 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAMbP0SSrzSevIxdrqPgVyhsOEfFHueNks5vGBJBgaJpZM4aNLn_>
.
|
d4ba826
to
7b96b5b
Compare
4d3213a
to
7b96b5b
Compare
…rs (#90) Fixes a regression introduced in #73 so that chart directories were no longer required to be in a direct subdirectory of configured chart directories. This caused problems in helm/charts#10830. cc @munnerz
/retest |
3 similar comments
/retest |
/retest |
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: munnerz, simonswine The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
* Bump version strings for v0.6.0 (cert-manager/cert-manager#1247) * Set CA sync backoff limit & add verifying installation docs (cert-manager/cert-manager#1245) * Added securityContext to cert-manager deployment pods in helm chart (cert-manager/cert-manager#1237) * Move Helm chart to deploy/charts/cert-manager (cert-manager/cert-manager#1235) * chart: allow deployment strategy customization * webhook: remove hardcoded image (cert-manager/cert-manager#1039) * Remove crd-install annotation from Challenges and Orders * Add Challenge and Order CRDs * Add certificates and issuers to aggregated RBAC for view, edit and admin ClusterRoles. (cert-manager/cert-manager#872) * Use fixed image tag for ca-helper and add description Signed-off-by: James Munnelly <james@munnelly.eu> Signed-off-by: Victor Boissiere <victor.boissiere@neo9.fr>
Checklist
/cc @kragniz