Skip to content

Commit

Permalink
Merge pull request #300 from bjwbell/remove-k8s-unversioned
Browse files Browse the repository at this point in the history
Remove k8s unversioned
  • Loading branch information
caseydavenport authored Dec 16, 2016
2 parents f87d397 + 6ab6c70 commit 81e35fd
Show file tree
Hide file tree
Showing 7 changed files with 129 additions and 139 deletions.
205 changes: 97 additions & 108 deletions glide.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion glide.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ import:
subpackages:
- patricia
- package: k8s.io/client-go
version: 41a99d711af778a177f07402217b85d456b50da1
version: 243d8a9cb66a51ad8676157f79e71033b4014a2a
subpackages:
- kubernetes
- pkg/api
Expand Down
12 changes: 6 additions & 6 deletions lib/backend/k8s/conversion.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,9 @@ import (
cnet "github.com/projectcalico/libcalico-go/lib/net"
"github.com/projectcalico/libcalico-go/lib/numorstring"
kapi "k8s.io/client-go/pkg/api"
"k8s.io/client-go/pkg/api/unversioned"
kapiv1 "k8s.io/client-go/pkg/api/v1"
extensions "k8s.io/client-go/pkg/apis/extensions/v1beta1"
metav1 "k8s.io/client-go/pkg/apis/meta/v1"
)

var (
Expand Down Expand Up @@ -237,7 +237,7 @@ func (c converter) networkPolicyToPolicy(np *extensions.NetworkPolicy) (*model.K

// k8sSelectorToCalico takes a namespaced k8s label selector and returns the Calico
// equivalent.
func (c converter) k8sSelectorToCalico(s *unversioned.LabelSelector, ns *string) string {
func (c converter) k8sSelectorToCalico(s *metav1.LabelSelector, ns *string) string {
// If this is a podSelector, it needs to be namespaced, and it
// uses a different prefix. Otherwise, treat this as a NamespaceSelector.
selectors := []string{}
Expand All @@ -259,13 +259,13 @@ func (c converter) k8sSelectorToCalico(s *unversioned.LabelSelector, ns *string)

// Each selector is formatted differently based on the operator.
switch e.Operator {
case unversioned.LabelSelectorOpIn:
case metav1.LabelSelectorOpIn:
selectors = append(selectors, fmt.Sprintf("%s%s in { '%s' }", prefix, e.Key, valueList))
case unversioned.LabelSelectorOpNotIn:
case metav1.LabelSelectorOpNotIn:
selectors = append(selectors, fmt.Sprintf("%s%s not in { '%s' }", prefix, e.Key, valueList))
case unversioned.LabelSelectorOpExists:
case metav1.LabelSelectorOpExists:
selectors = append(selectors, fmt.Sprintf("has(%s%s)", prefix, e.Key))
case unversioned.LabelSelectorOpDoesNotExist:
case metav1.LabelSelectorOpDoesNotExist:
selectors = append(selectors, fmt.Sprintf("! has(%s%s)", prefix, e.Key))
}
}
Expand Down
Loading

0 comments on commit 81e35fd

Please sign in to comment.