Skip to content

Commit

Permalink
Merge pull request #1115 from alvaroaleman/defaulting
Browse files Browse the repository at this point in the history
Fix defaulting of issuer url
  • Loading branch information
openshift-merge-robot authored Mar 7, 2022
2 parents a0a1fc9 + ae6817e commit 27690f3
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion api/v1alpha1/hostedcluster_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,8 @@ type HostedClusterSpec struct {
//
// +kubebuilder:default:="https://kubernetes.default.svc"
// +immutable
IssuerURL string `json:"issuerURL"`
// +optional
IssuerURL string `json:"issuerURL,omitempty"`

// Configuration specifies configuration for individual OCP components in the
// cluster, represented as embedded resources that correspond to the openshift
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -928,7 +928,6 @@ spec:
type: string
type: object
required:
- issuerURL
- networking
- platform
- pullSecret
Expand Down
2 changes: 2 additions & 0 deletions docs/content/reference/api.md
Original file line number Diff line number Diff line change
Expand Up @@ -277,6 +277,7 @@ string
</em>
</td>
<td>
<em>(Optional)</em>
<p>IssuerURL is an OIDC issuer URL which is used as the issuer in all
ServiceAccount tokens generated by the control plane API server. The
default value is kubernetes.default.svc, which only works for in-cluster
Expand Down Expand Up @@ -2426,6 +2427,7 @@ string
</em>
</td>
<td>
<em>(Optional)</em>
<p>IssuerURL is an OIDC issuer URL which is used as the issuer in all
ServiceAccount tokens generated by the control plane API server. The
default value is kubernetes.default.svc, which only works for in-cluster
Expand Down
1 change: 0 additions & 1 deletion hack/app-sre/saas_template.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20504,7 +20504,6 @@ objects:
type: string
type: object
required:
- issuerURL
- networking
- platform
- pullSecret
Expand Down

0 comments on commit 27690f3

Please sign in to comment.