304 Commits

Author SHA1 Message Date
Dave Salisbury
1ceec80ec5 Merge remote-tracking branch 'origin/master' into ingress-class-filtering 2022-02-14 16:36:26 +11:00
Andrey Lebedev
4f41229820 Pass stop channel to informer factory instances 2022-01-22 21:31:29 +01:00
Dave Salisbury
097df5c458 Merge remote-tracking branch 'origin/master' into ingress-class-filtering 2021-12-28 14:27:30 +11:00
Michael Gruener
b8a08512e9 Add event handler for OpenShift Route source 2021-11-22 15:16:13 +01:00
Kubernetes Prow Robot
9cf3d4906c
Merge pull request #2400 from miheer/add-router-field-for-ocpRouteSource
Fix CNAME record when multiple router canonical name are defined in Route Status.
2021-11-19 05:50:52 -08:00
Kubernetes Prow Robot
39931b7cb0
Merge pull request #2259 from mper0003/issue-2245
service: fix merging of endpoints - same hostname different set-identifier
2021-11-03 04:32:58 -07:00
Miheer Salunke
ab8a62045e In OCP when you have multiple ingress controllers the route's status Ingress object get populated with multiple router canonical names. So in this case, the external dns tries to add multiple CNAME records for same host in the same hosted zone which is a violation of RFC 1912  and therefore is rejected by standards-compliant DNS services.
This feature adds a router field to the OCP Route Source so that a user can add an ingress controller name in flag --ocp-router-name which will be used to pick up the respective routerCanonicalHostname from Route's Status Ingress Object.

Signed-off-by: Miheer Salunke <miheer.salunke@gmail.com>
2021-11-03 12:52:57 +10:00
Dave Salisbury
b8cbd4bbf4 Merge remote-tracking branch 'origin/master' into ingress-class-filtering 2021-10-25 11:18:56 +11:00
Arjun Naik
d91b7e6b8f Label filtering for Ingress, Service, Openshift Route sources
Currently the `--label-filter` flag can only be used to filter CRDs
which match the label selector passed through that flag. This change
extends the functionality to the Ingress, Service and Openshift Route
type objects. When the flag is not specified the default value is
`labels.Everything()` which is an empty string, the same as before.
Annotation based filter is inefficient because the filtering has to be
done in the controller instead of the API server like with label
filtering.
2021-10-14 11:06:55 +02:00
Dave Salisbury
31bc5bb077 ingress source: fix broken NewIngressSource test
and add an extra one for the mutual exclusivity of ingressClassNames and
ingress.class annotationFilters
2021-10-02 17:08:54 +10:00
Dave Salisbury
6c4a450b25 gofmt simplify new ingress-class changes 2021-10-02 16:54:42 +10:00
Dave Salisbury
0aa3d55450 ingress source: pass back err when requirement fails 2021-10-02 16:50:35 +10:00
Dave Salisbury
a24217a720 ingress source: use shorthand bool comparison 2021-10-02 16:49:51 +10:00
Dave Salisbury
901effbca5 ingress source: ingressClassNames now feed into annotation filter 2021-10-02 16:32:40 +10:00
Dave Salisbury
71a672fe72 ingress source: check for duplicate classname configs
we don't want to get incompatible restrictions by letting someone set
"--ingress-class" and --annotation-filter="kubernetes.io/ingress.class"
at the same time
2021-10-02 16:09:50 +10:00
Dave Salisbury
ac0c4be36a Update tests for ingress class filtering 2021-10-02 15:23:48 +10:00
Dave Salisbury
8da6f99857 Rename ingressClassNameFilter to ingressClassNames
...and update the help text to specify use more clearly
2021-10-02 15:22:01 +10:00
Dave Salisbury
da71b3fff8 ingress source: improve class name filtering and logging 2021-10-02 15:13:45 +10:00
Dave Salisbury
c823eba139 ingress source: update code for networkv1 2021-10-02 14:34:34 +10:00
Dave Salisbury
115e2501af Add ingressClassNameFilter testing 2021-10-02 14:29:34 +10:00
Dave Salisbury
c4898b7e98 Plumb in filtering on ingress class name 2021-10-02 14:29:34 +10:00
Andrew Stuart
0e27f41fa0
Update external-dns to use v1 ingress 2021-09-08 09:29:05 -07:00
Marlene
85c2d278ab
update test 2021-08-29 17:26:21 +10:00
Marlene
c1af8a40ec
do not merge endpoints with different set identifier 2021-08-29 16:25:55 +10:00
Raffaele Di Fazio
9d5f721209 delete one more file
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-08-12 17:19:22 +02:00
Raffaele Di Fazio
d82bee2b53 Merge branch 'master' into raffo/bump-modules
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-08-12 16:28:30 +02:00
Raffaele Di Fazio
3e8b5effd8 changes to contour
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-08-12 16:09:24 +02:00
Raffaele Di Fazio
5c7ddb07c2 drop more concour
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-08-12 14:12:44 +02:00
Raffaele Di Fazio
05cadcb340 remove contour ingressroute
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-08-12 14:08:02 +02:00
Raffaele Di Fazio
17d06f4a88 remove comments
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-08-12 09:56:53 +02:00
Raffaele Di Fazio
b18e723ff2 save
Signed-off-by: Raffaele Di Fazio <raffo@github.com>
2021-08-11 14:46:30 +02:00
Raffaele Di Fazio
376d4653d8 remove kubernetes dependency
Signed-off-by: GitHub <noreply@github.com>
2021-08-10 13:48:11 +00:00
Andy Bursavich
60c649bf5c source: dedupe wait for cache sync
wait
2021-07-28 13:37:17 -07:00
Andy Bursavich
55637abbef source: dedupe event handlers 2021-07-28 13:36:35 -07:00
Andy Bursavich
b032f2864f source: dedupe template execution 2021-07-28 13:35:51 -07:00
Andy Bursavich
638194fccd source: dedupe template parsing 2021-07-28 07:46:35 -07:00
Andy Bursavich
5069c3f0fb source: name files consistently with their source name 2021-07-28 07:46:35 -07:00
Andy Bursavich
76108e1bde source: parallelize tests that wait for syncing 2021-07-26 12:40:15 -07:00
Andy Bursavich
6da9fdd2e2 source: remove NewFakeConfigStore one-liner 2021-07-26 12:40:15 -07:00
Andy Bursavich
53bb0594bf connector: run test server on any open port 2021-07-25 13:32:30 -07:00
Andy Bursavich
f7a0539686 source: create test resources before sources 2021-07-25 13:32:30 -07:00
Andy Bursavich
eb2addb688 source: tests validate endpoint ProviderSpecific and SetIdentifier 2021-07-25 13:32:30 -07:00
Andy Bursavich
e0d42415ee source: fix flakes by sorting endpoint by dns names and targets 2021-07-21 11:50:28 -07:00
Anand Patel
509a07c493
create new source ingress config option IgnoreIngressRulesSpec 2021-06-30 10:03:04 -07:00
Anand Patel
9bccccf414
add ignoreIngressRulesSpec for ingress source 2021-06-30 09:56:58 -07:00
Kubernetes Prow Robot
67fad71451
Merge pull request #2095 from voro015/registry_aws-sd-map-fix
Fixed AWSSDRegistry nil labels map
2021-06-30 01:25:03 -07:00
Stephen Greene
d273066d6b source: Add ocproute source unit tests
source/ocproute_test.go:

New file for source/ocproute.go unit tests.

Inspired by the service source unit tests,
this commit adds basic unit tests to verify
basic functionality of the OCP route source.

Signed-off-by: Stephen Greene <sgreene@redhat.com>
2021-06-28 11:34:00 -04:00
Stephen Greene
b05df8b388 source: Clean up OpenShift Route source
source/ocproute.go:

Add/clean up godocs and general in-line comments.

Change `routeapi` named import to `routev1`.
Update `routev1` calling sites accordingly.

Signed-off-by: Stephen Greene <sgreene@redhat.com>
2021-06-28 11:23:07 -04:00
Kubernetes Prow Robot
0148bca0f5
Merge pull request #2041 from olemarkus/kops
Kops compatibility mode
2021-06-24 06:25:41 -07:00
Ole Markus With
ba30810641 Use kops dns controller instead of just dns controller where it makes sense
Co-authored-by: Nick Jüttner <nick@juni.io>
2021-06-24 08:31:42 +02:00