Grant Spence

Results 46 comments of Grant Spence

/hold Thanks for the review @rfredette. Looking back at this PR after a long break, something feels awkward about this test. I'm going to put it on hold for now....

@lihongan ~~I believe it's due to the same [errors](https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_api/1966/pull-ci-openshift-api-master-integration/1818414773075709952) we see in the integration test.~~ ~~I pushed a fix in this PR. CEL introduced in https://github.com/openshift/api/pull/1826/files#diff-891d2cdcb01603ef2a2c1bfb2f6ef32cf2acf8b1471871a5a6e27308d50050f9R703-R705 should have specified a...

Weird, prow didn't run new tests after my push: /test all

> @lihongan I believe it's due to the same [errors](https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_api/1966/pull-ci-openshift-api-master-integration/1818414773075709952) we see in the integration test. > > I pushed a fix in this PR. CEL introduced in https://github.com/openshift/api/pull/1826/files#diff-891d2cdcb01603ef2a2c1bfb2f6ef32cf2acf8b1471871a5a6e27308d50050f9R703-R705 should...

+1 to option 2: > OR create .IP type load balancers instead of hostnames so that CNI plugins can have rules within the cluster that can shortcircuit this traffic like...

> you could the external aws-load-balancer-controller @kishorj For my and @tssurya's use case (OpenShift), we have an equivalent operator, the https://github.com/openshift/cluster-ingress-operator/ which configures ingress for a variety of platforms. Adding...