Skip to content

Commit

Permalink
apply changes proposed in PR review comments
Browse files Browse the repository at this point in the history
Signed-off-by: Tim Ramlot <[email protected]>
  • Loading branch information
inteon committed Jul 8, 2024
1 parent 3477b5d commit 14cb9e0
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 8 deletions.
2 changes: 1 addition & 1 deletion deploy/charts/openshift-routes/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apiVersion: v2

name: cert-manager-openshift-routes
name: openshift-routes
type: application
description: A Helm chart to deploy openshift-routes adapter for cert-manager on Kubernetes

Expand Down
3 changes: 1 addition & 2 deletions deploy/charts/openshift-routes/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ repository: jetstack/cert-manager-openshift-routes
#### **image.repository** ~ `string`
> Default value:
> ```yaml
> quay.io/jetstack/cert-manager-openshift-routes
> ghcr.io/cert-manager/cert-manager-openshift-routes
> ```
Target image repository.
Expand Down Expand Up @@ -142,7 +142,6 @@ For example:
```yaml
resources:
limits:
cpu: 100m
memory: 128Mi
requests:
cpu: 100m
Expand Down
4 changes: 2 additions & 2 deletions deploy/charts/openshift-routes/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@
"type": "string"
},
"helm-values.image.repository": {
"default": "quay.io/jetstack/cert-manager-openshift-routes",
"default": "ghcr.io/cert-manager/cert-manager-openshift-routes",
"description": "Target image repository.",
"type": "string"
},
Expand Down Expand Up @@ -215,7 +215,7 @@
},
"helm-values.resources": {
"default": {},
"description": "Kubernetes pod resources\nref: https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/\n\nFor example:\nresources:\n limits:\n cpu: 100m\n memory: 128Mi\n requests:\n cpu: 100m\n memory: 128Mi",
"description": "Kubernetes pod resources\nref: https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/\n\nFor example:\nresources:\n limits:\n memory: 128Mi\n requests:\n cpu: 100m\n memory: 128Mi",
"type": "object"
},
"helm-values.securityContext": {
Expand Down
3 changes: 1 addition & 2 deletions deploy/charts/openshift-routes/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ image:
# registry: quay.io

# Target image repository.
repository: quay.io/jetstack/cert-manager-openshift-routes
repository: ghcr.io/cert-manager/cert-manager-openshift-routes

# Override the image tag to deploy by setting this variable.
# If no value is set, the chart's appVersion is used.
Expand Down Expand Up @@ -93,7 +93,6 @@ securityContext:
# For example:
# resources:
# limits:
# cpu: 100m
# memory: 128Mi
# requests:
# cpu: 100m
Expand Down
2 changes: 1 addition & 1 deletion make/00_mod.mk
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ deploy_name := openshift-routes
deploy_namespace := cert-manager

helm_chart_source_dir := deploy/charts/openshift-routes
helm_chart_name := cert-manager-openshift-routes
helm_chart_name := openshift-routes
helm_chart_version := $(VERSION:v%=%)
helm_labels_template_name := cert-manager-openshift-routes.labels
helm_docs_use_helm_tool := 1
Expand Down

0 comments on commit 14cb9e0

Please sign in to comment.