Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
NE-1516: Adds a field in the cluster ingress config object and Ingres…
…s Controller to set EIP via installer and Ingress Controller `config/v1/types_ingress.go` - Adds an API field `eip-allocations` in the cluster ingress config object whose value is set by the installer using install-config.yaml `operator/v1/types_ingress.go` - Adds an API field `eip-allocations` in the Ingress Controller CR object using which the Ingress Operator scales an Ingress Controller with service type balancer whose annotation `service.beta.kubernetes.io/aws-load-balancer-eip-allocations` is set by the value of the field `eip-allocations` of the Ingress Controller. Epic: https://issues.redhat.com/browse/NE-1274 Story: https://issues.redhat.com/browse/NE-1516
- Loading branch information
Showing
37 changed files
with
14,056 additions
and
14 deletions.
There are no files selected for viewing
53 changes: 53 additions & 0 deletions
53
config/v1/tests/ingresses.config.openshift.io/SetEIPForNLBIngressController.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
apiVersion: apiextensions.k8s.io/v1 # Hack because controller-gen complains if we don't have this | ||
name: "Ingress" | ||
crdName: ingresses.config.openshift.io | ||
tests: | ||
onCreate: | ||
- name: Should not allow to set NLB parameters when LBType is Classic. | ||
initial: | | ||
apiVersion: config.openshift.io/v1 | ||
kind: Ingress | ||
metadata: | ||
name: cluster | ||
spec: | ||
loadBalancer: | ||
platform: | ||
type: AWS | ||
aws: | ||
type: Classic | ||
networkLoadBalancer: | ||
eipAllocations: | ||
- eipalloc-12345 | ||
- eipalloc-12346 | ||
expectedError: "Network load balancer parameters are allowed only when load balancer type is NLB." | ||
- name: Should allow to set NLB parameters when LBType is NLB. | ||
initial: | | ||
apiVersion: config.openshift.io/v1 | ||
kind: Ingress | ||
metadata: | ||
name: cluster | ||
spec: | ||
loadBalancer: | ||
platform: | ||
type: AWS | ||
aws: | ||
type: NLB | ||
networkLoadBalancer: | ||
eipAllocations: | ||
- eipalloc-12345 | ||
- eipalloc-12346 | ||
expected: | | ||
apiVersion: config.openshift.io/v1 | ||
kind: Ingress | ||
metadata: | ||
name: cluster | ||
spec: | ||
loadBalancer: | ||
platform: | ||
type: AWS | ||
aws: | ||
type: NLB | ||
networkLoadBalancer: | ||
eipAllocations: | ||
- eipalloc-12345 | ||
- eipalloc-12346 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
562 changes: 562 additions & 0 deletions
562
.../zz_generated.crd-manifests/0000_10_config-operator_01_ingresses-CustomNoUpgrade.crd.yaml
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.