Skip to content

Commit

Permalink
add managed-by label
Browse files Browse the repository at this point in the history
  • Loading branch information
ryandawsonuk committed Jun 11, 2020
1 parent 6f32074 commit 6b84f0a
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 0 deletions.
Expand Up @@ -43,6 +43,8 @@ const (
Label_shadow = "shadow"
Label_canary = "canary"
Label_explainer = "explainer"
Label_managed_by = "app.kubernetes.io/managed-by"
Label_value_seldon = "seldon-core"

PODINFO_VOLUME_NAME = "seldon-podinfo"
PODINFO_VOLUME_PATH = "/etc/podinfo"
Expand Down
2 changes: 2 additions & 0 deletions operator/controllers/labels.go
Expand Up @@ -33,10 +33,12 @@ func addLabelsToService(svc *corev1.Service, pu *machinelearningv1.PredictiveUni
if !isEmptyExplainer(p.Explainer) {
svc.Labels[machinelearningv1.Label_explainer] = "true"
}
svc.Labels[machinelearningv1.Label_managed_by] = machinelearningv1.Label_value_seldon
}

func addLabelsToDeployment(deploy *appsv1.Deployment, containerServiceKey, containerServiceValue string) {
deploy.ObjectMeta.Labels[containerServiceKey] = containerServiceValue
deploy.Spec.Selector.MatchLabels[containerServiceKey] = containerServiceValue
deploy.Spec.Template.ObjectMeta.Labels[containerServiceKey] = containerServiceValue
deploy.ObjectMeta.Labels[machinelearningv1.Label_managed_by] = machinelearningv1.Label_value_seldon
}
2 changes: 2 additions & 0 deletions operator/controllers/labels_test.go
Expand Up @@ -34,6 +34,7 @@ func TestAddLabelsToDeployment(t *testing.T) {
g.Expect(d.ObjectMeta.Labels["TestKey"]).To(Equal("TestValue"))
g.Expect(d.Spec.Selector.MatchLabels["TestKey"]).To(Equal("TestValue"))
g.Expect(d.Spec.Template.ObjectMeta.Labels["TestKey"]).To(Equal("TestValue"))
g.Expect(d.ObjectMeta.Labels[machinelearningv1.Label_managed_by]).To(Equal(machinelearningv1.Label_value_seldon))
})
}

Expand All @@ -60,6 +61,7 @@ var _ = Describe("addLabelsToService", func() {
addLabelsToService(svc, pu, p)

Expect(svc.Labels[result]).To(Equal("true"))
Expect(svc.Labels[machinelearningv1.Label_managed_by]).To(Equal(machinelearningv1.Label_value_seldon))
},
Entry("router", machinelearningv1.ROUTER, machinelearningv1.Label_router),
Entry("combiner", machinelearningv1.COMBINER, machinelearningv1.Label_combiner),
Expand Down

0 comments on commit 6b84f0a

Please sign in to comment.