Skip to content

Commit e641ebd

Browse files
author
Travis Clarke
committed
Remove old k8s client from "artifact-uploader"
1 parent 1709219 commit e641ebd

File tree

4 files changed

+9
-8
lines changed

4 files changed

+9
-8
lines changed

prow/artifact-uploader/BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ go_library(
66
importpath = "k8s.io/test-infra/prow/artifact-uploader",
77
visibility = ["//visibility:public"],
88
deps = [
9+
"//prow/client/clientset/versioned/typed/prowjobs/v1:go_default_library",
910
"//prow/gcsupload:go_default_library",
1011
"//prow/kube:go_default_library",
1112
"//prow/pod-utils/downwardapi:go_default_library",

prow/artifact-uploader/controller.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ import (
3232
core "k8s.io/client-go/kubernetes/typed/core/v1"
3333
"k8s.io/client-go/tools/cache"
3434
"k8s.io/client-go/util/workqueue"
35-
35+
prowv1 "k8s.io/test-infra/prow/client/clientset/versioned/typed/prowjobs/v1"
3636
"k8s.io/test-infra/prow/gcsupload"
3737
"k8s.io/test-infra/prow/kube"
3838
"k8s.io/test-infra/prow/pod-utils/downwardapi"
@@ -53,7 +53,7 @@ type item struct {
5353
prowJobId string
5454
}
5555

56-
func NewController(client core.CoreV1Interface, prowJobClient *kube.Client, gcsConfig *gcsupload.Options) Controller {
56+
func NewController(client core.CoreV1Interface, prowJobClient prowv1.ProwJobInterface, gcsConfig *gcsupload.Options) Controller {
5757
queue := workqueue.NewRateLimitingQueue(workqueue.DefaultControllerRateLimiter())
5858
optionsModifier := func(options *metav1.ListOptions) {
5959
req, _ := labels.NewRequirement(kube.ProwJobIDLabel, selection.Exists, []string{})
@@ -116,7 +116,7 @@ type Controller struct {
116116
informer cache.Controller
117117

118118
client core.CoreV1Interface
119-
prowJobClient *kube.Client
119+
prowJobClient prowv1.ProwJobInterface
120120

121121
gcsConfig *gcsupload.Options
122122
}
@@ -153,7 +153,7 @@ func (c *Controller) processNextItem() bool {
153153

154154
workItem := key.(item)
155155

156-
prowJob, err := c.prowJobClient.GetProwJob(workItem.prowJobId)
156+
prowJob, err := c.prowJobClient.Get(workItem.prowJobId, metav1.GetOptions{})
157157
if err != nil {
158158
c.handleErr(err, workItem)
159159
return true

prow/cmd/artifact-uploader/BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@ go_library(
88
visibility = ["//visibility:private"],
99
deps = [
1010
"//prow/artifact-uploader:go_default_library",
11+
"//prow/client/clientset/versioned:go_default_library",
1112
"//prow/gcsupload:go_default_library",
12-
"//prow/kube:go_default_library",
1313
"//prow/logrusutil:go_default_library",
1414
"//prow/pod-utils/options:go_default_library",
1515
"@com_github_sirupsen_logrus//:go_default_library",

prow/cmd/artifact-uploader/main.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,8 @@ import (
2828
"k8s.io/client-go/tools/clientcmd"
2929

3030
artifact_uploader "k8s.io/test-infra/prow/artifact-uploader"
31+
prow "k8s.io/test-infra/prow/client/clientset/versioned"
3132
"k8s.io/test-infra/prow/gcsupload"
32-
"k8s.io/test-infra/prow/kube"
3333
"k8s.io/test-infra/prow/logrusutil"
3434
"k8s.io/test-infra/prow/pod-utils/options"
3535
)
@@ -135,12 +135,12 @@ func (o *Options) Run() error {
135135
return err
136136
}
137137

138-
prowJobClient, err := kube.NewClientInCluster(o.ProwJobNamespace)
138+
prowJobClient, err := prow.NewForConfig(clusterConfig)
139139
if err != nil {
140140
return err
141141
}
142142

143-
controller := artifact_uploader.NewController(client.CoreV1(), prowJobClient, o.Options)
143+
controller := artifact_uploader.NewController(client.CoreV1(), prowJobClient.ProwV1().ProwJobs(o.ProwJobNamespace), o.Options)
144144

145145
stop := make(chan struct{})
146146
defer close(stop)

0 commit comments

Comments
 (0)