Skip to content

Commit 070be81

Browse files
committed
support for imagepullsecret
1 parent 97252e8 commit 070be81

File tree

5 files changed

+15
-1
lines changed

5 files changed

+15
-1
lines changed

charts/operators-installer/Chart.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ type: application
1616
# This is the chart version. This version number should be incremented each time you make changes
1717
# to the chart and its templates, including the app version.
1818
# Versions are expected to follow Semantic Versioning (https://semver.org/)
19-
version: 3.0.1
19+
version: 3.1.0
2020

2121
home: https://github.com/redhat-cop/helm-charts
2222

charts/operators-installer/README.md

+1
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ For all of the Subscription parameters see
4242
| operatorGroups[].targetOwnNamespace | `false` | No | If `true` add the OperatorGroup's Namespace as a `targetNamespaces`. If `true` then OperatorGroup will only work for Operators using `OwnNamespace` or `MultiNamespace` `installModes`. If blank and no `otherTargetNamespaces` specified then OperatorGroup will be configured to allow for operators using `installModes` `AllNamespaces`.
4343
| operatorGroups[].otherTargetNamespaces | `[]` | No | List of additional Namespaces to target. If specified OperatorGroup will only work for operators using `SingleNamespace` or `MultiNamespace` `installModes` depending on value of `targetOwnNamespace`.
4444
| installPlanApproverAndVerifyJobsImage | `registry.redhat.io/openshift4/ose-cli:v4.10` | Yes | Image to use for the InstallPlan Approver and Verify Jobs
45+
| installPlanApproverAndVerifyJobsImagePullSecret | `''` | No | Name of existing secret for pulling `installPlanApproverAndVerifyJobsImage` from a private registry
4546
| approveManualInstallPlanViaHook | `true` | No | `true` to create (and clean up) manual InstallPlan approval resources as part of post-install,post-upgrade helm hook<br>`false` to create manual InstallPlan approval resources as part of normal install<br><br>The hook method is nice to not have lingering resources needed for the manual InstallPlan approval but has the downside that no CustomResources using CustomResourceDefinitions installed by the operator can be used in the same chart because the operator InstallPlan wont be approved, and therefor the operator wont be installed, until the post-install,post-upgrade phase which means you will never get to that phase because your CustomResources wont be able to apply because the Operator isn't installed.<br><br>This is is ultimately a trade off between cleaning up these resources or being able to install and configure the operator in the same helm chart that has a dependency on this helm chart.
4647
| installRequiredPythonLibraries | `true` | No | If `true`, install the required Python libraries (openshift-client, semver==2.13.0) dynamically from the given `pythonIndexURL` and `pythonExtraIndexURL` into the `installPlanApproverAndVerifyJobsImage` at run time
4748
| pythonIndexURL | https://pypi.org/simple/ | No | If `installRequiredPythonLibraries` is `true` then use this python index to pull required libraries

charts/operators-installer/templates/Job_installplan-approver.yaml

+4
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,10 @@ spec:
2828
containers:
2929
- name: installplan-approver
3030
image: {{ $.Values.installPlanApproverAndVerifyJobsImage }}
31+
{{- if $.Values.installPlanApproverAndVerifyJobsImagePullSecret }}
32+
imagePullSecrets:
33+
- {{ $.Values. installPlanApproverAndVerifyJobsImagePullSecret }}
34+
{{- end }}
3135
command: ["/bin/sh","-c"]
3236
args:
3337
- >-

charts/operators-installer/templates/Job_installplan-complete-verifier.yaml

+4
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,10 @@ spec:
2727
containers:
2828
- name: installplan-complete-verifier
2929
image: {{ $.Values.installPlanApproverAndVerifyJobsImage }}
30+
{{- if $.Values.installPlanApproverAndVerifyJobsImagePullSecret }}
31+
imagePullSecrets:
32+
- {{ $.Values. installPlanApproverAndVerifyJobsImagePullSecret }}
33+
{{- end }}
3034
command: ["/bin/sh","-c"]
3135
args:
3236
- >-

charts/operators-installer/values.yaml

+5
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,11 @@ approveManualInstallPlanViaHook: true
1212
# Image to use for the InstallPlan Approver and Verify Jobs
1313
installPlanApproverAndVerifyJobsImage: registry.redhat.io/openshift4/ose-cli:v4.15@sha256:3f2123f42ae7358e1fece41d461bf331f144480da8b7711b9a93aca150f33f3f
1414

15+
# If specified, this imagePullSecret will be used for pulling the `installPlanApproverAndVerifyJobsImage`.
16+
# Only needed when using a custom image and pulling from a private registry.
17+
# Accepts a single string which is the name of an existing secret. (This chart does not create or manage the secret.)
18+
installPlanApproverAndVerifyJobsImagePullSecret:
19+
1520
# If `true`, install the required Python libraries (openshift-client, semver==2.13.0) dynamically
1621
# from the given `pythonIndexURL` and `pythonExtraIndexURL` into the `installPlanApproverAndVerifyJobsImage` at run time
1722
#

0 commit comments

Comments
 (0)