Skip to content

Commit

Permalink
Merge 628ed59 into 102228c
Browse files Browse the repository at this point in the history
  • Loading branch information
josephglanville committed Jun 12, 2020
2 parents 102228c + 628ed59 commit 0f28fb3
Show file tree
Hide file tree
Showing 10 changed files with 168 additions and 104 deletions.
28 changes: 17 additions & 11 deletions go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -24,23 +24,27 @@ require (
github.com/exoscale/egoscale v0.18.1
github.com/ffledgling/pdns-go v0.0.0-20180219074714-524e7daccd99
github.com/gobs/pretty v0.0.0-20180724170744-09732c25a95b // indirect
github.com/golang/groupcache v0.0.0-20190702054246-869f871628b6 // indirect
github.com/golang/sync v0.0.0-20180314180146-1d60e4601c6f
github.com/gophercloud/gophercloud v0.1.0
github.com/gorilla/mux v1.7.4 // indirect
github.com/hashicorp/golang-lru v0.5.4 // indirect
github.com/infobloxopen/infoblox-go-client v0.0.0-20180606155407-61dc5f9b0a65
github.com/json-iterator/go v1.1.9 // indirect
github.com/linki/instrumented_http v0.2.0
github.com/linode/linodego v0.15.0
github.com/linode/linodego v0.19.0
github.com/maxatome/go-testdeep v1.4.0
github.com/miekg/dns v1.1.25
github.com/nesv/go-dynect v0.6.0
github.com/nic-at/rc0go v1.1.0
github.com/openshift/api v0.0.0-20200302134843-001335d6cc34
github.com/openshift/client-go v0.0.0-20200116145930-eb24d03d8420
github.com/openshift/api v0.0.0-20200605231317-fb2a6ca106ae
github.com/openshift/client-go v0.0.0-20200608144219-584632b8fc73
github.com/oracle/oci-go-sdk v1.8.0
github.com/ovh/go-ovh v0.0.0-20181109152953-ba5adb4cf014
github.com/pkg/errors v0.9.1
github.com/projectcontour/contour v1.4.0
github.com/projectcontour/contour v1.5.0
github.com/prometheus/client_golang v1.1.0
github.com/prometheus/client_model v0.2.0 // indirect
github.com/sanyu/dynectsoap v0.0.0-20181203081243-b83de5edc4e0
github.com/satori/go.uuid v1.2.0 // indirect
github.com/sirupsen/logrus v1.4.2
Expand All @@ -50,18 +54,19 @@ require (
github.com/transip/gotransip v5.8.2+incompatible
github.com/vinyldns/go-vinyldns v0.0.0-20190611170422-7119fe55ed92
github.com/vultr/govultr v0.3.2
go.etcd.io/etcd v0.5.0-alpha.5.0.20200401174654-e694b7bb0875
go.etcd.io/etcd v0.0.0-20191023171146-3cf2f69b5738
golang.org/x/net v0.0.0-20200202094626-16171245cfb2
golang.org/x/oauth2 v0.0.0-20190604053449-0f29369cfe45
golang.org/x/time v0.0.0-20190921001708-c4c64cad1fd0 // indirect
google.golang.org/api v0.15.0
google.golang.org/appengine v1.6.5 // indirect
gopkg.in/ns1/ns1-go.v2 v2.0.0-20190322154155-0dafb5275fd1
gopkg.in/yaml.v2 v2.2.8
istio.io/api v0.0.0-20200324230725-4b064f75ad8f
istio.io/client-go v0.0.0-20200324231043-96a582576da1
k8s.io/api v0.17.5
k8s.io/apimachinery v0.17.5
k8s.io/client-go v0.17.5
k8s.io/klog v1.0.0
istio.io/api v0.0.0-20200529165953-72dad51d4ffc
istio.io/client-go v0.0.0-20200529172309-31c16ea3f751
k8s.io/api v0.18.3
k8s.io/apimachinery v0.18.3
k8s.io/client-go v0.18.3
)

replace (
Expand All @@ -70,5 +75,6 @@ replace (
github.com/Azure/go-autorest/autorest/adal => github.com/Azure/go-autorest/autorest/adal v0.6.0
github.com/Azure/go-autorest/autorest/azure/auth => github.com/Azure/go-autorest/autorest/azure/auth v0.3.0
github.com/golang/glog => github.com/kubermatic/glog-logrus v0.0.0-20180829085450-3fa5b9870d1d
google.golang.org/grpc => google.golang.org/grpc v1.26.0
k8s.io/klog => github.com/mikkeloscar/knolog v0.0.0-20190326191552-80742771eb6b
)
176 changes: 115 additions & 61 deletions go.sum

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions source/crd.go
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ func (cs *crdSource) List(opts *metav1.ListOptions) (result *endpoint.DNSEndpoin
Namespace(cs.namespace).
Resource(cs.crdResource).
VersionedParams(opts, cs.codec).
Do().
Do(context.Background()).
Into(result)
return
}
Expand All @@ -200,7 +200,7 @@ func (cs *crdSource) UpdateStatus(dnsEndpoint *endpoint.DNSEndpoint) (result *en
Name(dnsEndpoint.Name).
SubResource("status").
Body(dnsEndpoint).
Do().
Do(context.Background()).
Into(result)
return
}
Expand Down
2 changes: 1 addition & 1 deletion source/gateway.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ func NewIstioGatewaySource(
// Endpoints returns endpoint objects for each host-target combination that should be processed.
// Retrieves all gateway resources in the source's namespace(s).
func (sc *gatewaySource) Endpoints() ([]*endpoint.Endpoint, error) {
gwList, err := sc.istioClient.NetworkingV1alpha3().Gateways(sc.namespace).List(metav1.ListOptions{})
gwList, err := sc.istioClient.NetworkingV1alpha3().Gateways(sc.namespace).List(context.Background(), metav1.ListOptions{})
if err != nil {
return nil, err
}
Expand Down
9 changes: 5 additions & 4 deletions source/gateway_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ limitations under the License.
package source

import (
"context"
"testing"

"github.com/pkg/errors"
Expand Down Expand Up @@ -64,7 +65,7 @@ func (suite *GatewaySuite) SetupTest() {
}

for _, service := range suite.lbServices {
_, err = fakeKubernetesClient.CoreV1().Services(service.Namespace).Create(service)
_, err = fakeKubernetesClient.CoreV1().Services(service.Namespace).Create(context.Background(), service, metav1.CreateOptions{})
suite.NoError(err, "should succeed")
}

Expand Down Expand Up @@ -1078,14 +1079,14 @@ func testGatewayEndpoints(t *testing.T) {

for _, lb := range ti.lbServices {
service := lb.Service()
_, err := fakeKubernetesClient.CoreV1().Services(service.Namespace).Create(service)
_, err := fakeKubernetesClient.CoreV1().Services(service.Namespace).Create(context.Background(), service, metav1.CreateOptions{})
require.NoError(t, err)
}

fakeIstioClient := NewFakeConfigStore()
for _, config := range ti.configItems {
gatewayCfg := config.Config()
_, err := fakeIstioClient.NetworkingV1alpha3().Gateways(ti.targetNamespace).Create(&gatewayCfg)
_, err := fakeIstioClient.NetworkingV1alpha3().Gateways(ti.targetNamespace).Create(context.Background(), &gatewayCfg, metav1.CreateOptions{})
require.NoError(t, err)
}

Expand Down Expand Up @@ -1119,7 +1120,7 @@ func newTestGatewaySource(loadBalancerList []fakeIngressGatewayService) (*gatewa

for _, lb := range loadBalancerList {
service := lb.Service()
_, err := fakeKubernetesClient.CoreV1().Services(service.Namespace).Create(service)
_, err := fakeKubernetesClient.CoreV1().Services(service.Namespace).Create(context.Background(), service, metav1.CreateOptions{})
if err != nil {
return nil, err
}
Expand Down
5 changes: 3 additions & 2 deletions source/ingress_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ limitations under the License.
package source

import (
"context"
"testing"
"time"

Expand Down Expand Up @@ -62,7 +63,7 @@ func (suite *IngressSuite) SetupTest() {
hostnames: []string{"v1"},
annotations: map[string]string{ALBDualstackAnnotationKey: ALBDualstackAnnotationValue},
}).Ingress()
_, err = fakeClient.ExtensionsV1beta1().Ingresses(suite.fooWithTargets.Namespace).Create(suite.fooWithTargets)
_, err = fakeClient.ExtensionsV1beta1().Ingresses(suite.fooWithTargets.Namespace).Create(context.Background(), suite.fooWithTargets, metav1.CreateOptions{})
suite.NoError(err, "should succeed")
}

Expand Down Expand Up @@ -1009,7 +1010,7 @@ func testIngressEndpoints(t *testing.T) {
ti.ignoreHostnameAnnotation,
)
for _, ingress := range ingresses {
_, err := fakeClient.ExtensionsV1beta1().Ingresses(ingress.Namespace).Create(ingress)
_, err := fakeClient.ExtensionsV1beta1().Ingresses(ingress.Namespace).Create(context.Background(), ingress, metav1.CreateOptions{})
require.NoError(t, err)
}

Expand Down
2 changes: 1 addition & 1 deletion source/ingressroute.go
Original file line number Diff line number Diff line change
Expand Up @@ -287,7 +287,7 @@ func (sc *ingressRouteSource) targetsFromContourLoadBalancer() (targets endpoint
if err != nil {
return nil, err
}
if svc, err := sc.kubeClient.CoreV1().Services(lbNamespace).Get(lbName, metav1.GetOptions{}); err != nil {
if svc, err := sc.kubeClient.CoreV1().Services(lbNamespace).Get(context.Background(), lbName, metav1.GetOptions{}); err != nil {
log.Warn(err)
} else {
for _, lb := range svc.Status.LoadBalancer.Ingress {
Expand Down
14 changes: 7 additions & 7 deletions source/ingressroute_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ func (suite *IngressRouteSuite) SetupTest() {
name: "contour",
}).Service()

_, err = fakeKubernetesClient.CoreV1().Services(suite.loadBalancer.Namespace).Create(suite.loadBalancer)
_, err = fakeKubernetesClient.CoreV1().Services(suite.loadBalancer.Namespace).Create(context.Background(), suite.loadBalancer, metav1.CreateOptions{})
suite.NoError(err, "should succeed")

suite.source, err = NewContourIngressRouteSource(
Expand All @@ -85,7 +85,7 @@ func (suite *IngressRouteSuite) SetupTest() {
suite.Error(err)
}

_, err = fakeDynamicClient.Resource(contour.IngressRouteGVR).Namespace(suite.ingressRoute.Namespace).Create(unstructuredIngressRoute, metav1.CreateOptions{})
_, err = fakeDynamicClient.Resource(contour.IngressRouteGVR).Namespace(suite.ingressRoute.Namespace).Create(context.Background(), unstructuredIngressRoute, metav1.CreateOptions{})
suite.NoError(err, "should succeed")
}

Expand All @@ -104,7 +104,7 @@ func newDynamicKubernetesClient() (*fakeDynamic.FakeDynamicClient, *runtime.Sche

func convertToUnstructured(ir *contour.IngressRoute, s *runtime.Scheme) (*unstructured.Unstructured, error) {
unstructuredIngressRoute := &unstructured.Unstructured{}
if err := s.Convert(ir, unstructuredIngressRoute, context.TODO()); err != nil {
if err := s.Convert(ir, unstructuredIngressRoute, context.Background()); err != nil {
return nil, err
}
return unstructuredIngressRoute, nil
Expand Down Expand Up @@ -1006,7 +1006,7 @@ func testIngressRouteEndpoints(t *testing.T) {
fakeKubernetesClient := fakeKube.NewSimpleClientset()

lbService := ti.loadBalancer.Service()
_, err := fakeKubernetesClient.CoreV1().Services(lbService.Namespace).Create(lbService)
_, err := fakeKubernetesClient.CoreV1().Services(lbService.Namespace).Create(context.Background(), lbService, metav1.CreateOptions{})
if err != nil {
require.NoError(t, err)
}
Expand All @@ -1015,7 +1015,7 @@ func testIngressRouteEndpoints(t *testing.T) {
for _, ingressRoute := range ingressRoutes {
converted, err := convertToUnstructured(ingressRoute, scheme)
require.NoError(t, err)
_, err = fakeDynamicClient.Resource(contour.IngressRouteGVR).Namespace(ingressRoute.Namespace).Create(converted, metav1.CreateOptions{})
_, err = fakeDynamicClient.Resource(contour.IngressRouteGVR).Namespace(ingressRoute.Namespace).Create(context.Background(), converted, metav1.CreateOptions{})
require.NoError(t, err)
}

Expand Down Expand Up @@ -1049,7 +1049,7 @@ func newTestIngressRouteSource(loadBalancer fakeLoadBalancerService) (*ingressRo
fakeDynamicClient, _ := newDynamicKubernetesClient()

lbService := loadBalancer.Service()
_, err := fakeKubernetesClient.CoreV1().Services(lbService.Namespace).Create(lbService)
_, err := fakeKubernetesClient.CoreV1().Services(lbService.Namespace).Create(context.Background(), lbService, metav1.CreateOptions{})
if err != nil {
return nil, err
}
Expand Down Expand Up @@ -1133,7 +1133,7 @@ func (ir fakeIngressRoute) IngressRoute() *contour.IngressRoute {
spec = contour.IngressRouteSpec{}
} else {
spec = contour.IngressRouteSpec{
VirtualHost: &projectcontour.VirtualHost{
VirtualHost: &contour.VirtualHost{
Fqdn: ir.host,
},
}
Expand Down
3 changes: 2 additions & 1 deletion source/node_test.go
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
package source

import (
"context"
"testing"

"github.com/stretchr/testify/assert"
Expand Down Expand Up @@ -306,7 +307,7 @@ func testNodeSourceEndpoints(t *testing.T) {
},
}

_, err := kubernetes.CoreV1().Nodes().Create(node)
_, err := kubernetes.CoreV1().Nodes().Create(context.Background(), node, metav1.CreateOptions{})
require.NoError(t, err)

// Create our object under test and get the endpoints.
Expand Down
29 changes: 15 additions & 14 deletions source/service_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ limitations under the License.
package source

import (
"context"
"net"
"testing"
"time"
Expand Down Expand Up @@ -75,7 +76,7 @@ func (suite *ServiceSuite) SetupTest() {

suite.NoError(err, "should initialize service source")

_, err = fakeClient.CoreV1().Services(suite.fooWithTargets.Namespace).Create(suite.fooWithTargets)
_, err = fakeClient.CoreV1().Services(suite.fooWithTargets.Namespace).Create(context.Background(), suite.fooWithTargets, metav1.CreateOptions{})
suite.NoError(err, "should successfully create service")

}
Expand Down Expand Up @@ -1095,7 +1096,7 @@ func testServiceSourceEndpoints(t *testing.T) {
},
}

_, err := kubernetes.CoreV1().Services(service.Namespace).Create(service)
_, err := kubernetes.CoreV1().Services(service.Namespace).Create(context.Background(), service, metav1.CreateOptions{})
require.NoError(t, err)

// Create our object under test and get the endpoints.
Expand Down Expand Up @@ -1266,7 +1267,7 @@ func TestClusterIpServices(t *testing.T) {
},
}

_, err := kubernetes.CoreV1().Services(service.Namespace).Create(service)
_, err := kubernetes.CoreV1().Services(service.Namespace).Create(context.Background(), service, metav1.CreateOptions{})
require.NoError(t, err)

// Create our object under test and get the endpoints.
Expand Down Expand Up @@ -1551,7 +1552,7 @@ func TestNodePortServices(t *testing.T) {

// Create the nodes
for _, node := range tc.nodes {
if _, err := kubernetes.CoreV1().Nodes().Create(node); err != nil {
if _, err := kubernetes.CoreV1().Nodes().Create(context.Background(), node, metav1.CreateOptions{}); err != nil {
t.Fatal(err)
}
}
Expand All @@ -1575,7 +1576,7 @@ func TestNodePortServices(t *testing.T) {
},
}

_, err := kubernetes.CoreV1().Pods(tc.svcNamespace).Create(pod)
_, err := kubernetes.CoreV1().Pods(tc.svcNamespace).Create(context.Background(), pod, metav1.CreateOptions{})
require.NoError(t, err)
}

Expand All @@ -1598,7 +1599,7 @@ func TestNodePortServices(t *testing.T) {
},
}

_, err := kubernetes.CoreV1().Services(service.Namespace).Create(service)
_, err := kubernetes.CoreV1().Services(service.Namespace).Create(context.Background(), service, metav1.CreateOptions{})
require.NoError(t, err)

// Create our object under test and get the endpoints.
Expand Down Expand Up @@ -1876,7 +1877,7 @@ func TestHeadlessServices(t *testing.T) {
},
Status: v1.ServiceStatus{},
}
_, err := kubernetes.CoreV1().Services(service.Namespace).Create(service)
_, err := kubernetes.CoreV1().Services(service.Namespace).Create(context.Background(), service, metav1.CreateOptions{})
require.NoError(t, err)

var addresses, notReadyAddresses []v1.EndpointAddress
Expand All @@ -1897,7 +1898,7 @@ func TestHeadlessServices(t *testing.T) {
},
}

_, err = kubernetes.CoreV1().Pods(tc.svcNamespace).Create(pod)
_, err = kubernetes.CoreV1().Pods(tc.svcNamespace).Create(context.Background(), pod, metav1.CreateOptions{})
require.NoError(t, err)

address := v1.EndpointAddress{
Expand Down Expand Up @@ -1927,7 +1928,7 @@ func TestHeadlessServices(t *testing.T) {
},
},
}
_, err = kubernetes.CoreV1().Endpoints(tc.svcNamespace).Create(endpointsObject)
_, err = kubernetes.CoreV1().Endpoints(tc.svcNamespace).Create(context.Background(), endpointsObject, metav1.CreateOptions{})
require.NoError(t, err)

// Create our object under test and get the endpoints.
Expand Down Expand Up @@ -2177,7 +2178,7 @@ func TestHeadlessServicesHostIP(t *testing.T) {
},
Status: v1.ServiceStatus{},
}
_, err := kubernetes.CoreV1().Services(service.Namespace).Create(service)
_, err := kubernetes.CoreV1().Services(service.Namespace).Create(context.Background(), service, metav1.CreateOptions{})
require.NoError(t, err)

var addresses []v1.EndpointAddress
Expand All @@ -2199,7 +2200,7 @@ func TestHeadlessServicesHostIP(t *testing.T) {
},
}

_, err = kubernetes.CoreV1().Pods(tc.svcNamespace).Create(pod)
_, err = kubernetes.CoreV1().Pods(tc.svcNamespace).Create(context.Background(), pod, metav1.CreateOptions{})
require.NoError(t, err)

address := v1.EndpointAddress{
Expand Down Expand Up @@ -2229,7 +2230,7 @@ func TestHeadlessServicesHostIP(t *testing.T) {
},
},
}
_, err = kubernetes.CoreV1().Endpoints(tc.svcNamespace).Create(endpointsObject)
_, err = kubernetes.CoreV1().Endpoints(tc.svcNamespace).Create(context.Background(), endpointsObject, metav1.CreateOptions{})
require.NoError(t, err)

// Create our object under test and get the endpoints.
Expand Down Expand Up @@ -2334,7 +2335,7 @@ func TestExternalServices(t *testing.T) {
},
Status: v1.ServiceStatus{},
}
_, err := kubernetes.CoreV1().Services(service.Namespace).Create(service)
_, err := kubernetes.CoreV1().Services(service.Namespace).Create(context.Background(), service, metav1.CreateOptions{})
require.NoError(t, err)

// Create our object under test and get the endpoints.
Expand Down Expand Up @@ -2387,7 +2388,7 @@ func BenchmarkServiceEndpoints(b *testing.B) {
},
}

_, err := kubernetes.CoreV1().Services(service.Namespace).Create(service)
_, err := kubernetes.CoreV1().Services(service.Namespace).Create(context.Background(), service, metav1.CreateOptions{})
require.NoError(b, err)

client, err := NewServiceSource(kubernetes, v1.NamespaceAll, "", "", false, "", false, false, false, []string{}, false)
Expand Down

0 comments on commit 0f28fb3

Please sign in to comment.