Kubectl: Remove dependency on k8s.io/kubernetes/pkg/apis/policy from kubectl

Created on 20 Oct 2017  路  4Comments  路  Source: kubernetes/kubectl

Remove dependencies from packages under k8s.io/kubernetes/pkg/kubectl/... on k8s.io/kubernetes/pkg/apis/policy by converting them to use the k8s.io/api/policy/v1beta1 package

good first issue kincleanup prioritimportant-longterm sicli sizS typcode-cleanup
Source
picture pwittrock

Most helpful comment

@alessbn, my apologies, I didn't see your comment on this issue when I started working on another PR that fixes some Pod Disruption Budget issues. https://github.com/kubernetes/kubernetes/pull/54411 fixes partially fixes this, but there're still some other files to update. I already started working on them, but as you commented here first that you'll pick the issue, I will stop working on it.

picture frodenas on 23 Oct 2017
👍2

All 4 comments

Hi @pwittrock I'd like to pick this issue.

alessbn picture alessbn on 23 Oct 2017
👍1

@alessbn, my apologies, I didn't see your comment on this issue when I started working on another PR that fixes some Pod Disruption Budget issues. https://github.com/kubernetes/kubernetes/pull/54411 fixes partially fixes this, but there're still some other files to update. I already started working on them, but as you commented here first that you'll pick the issue, I will stop working on it.

frodenas picture frodenas on 23 Oct 2017
👍2

@frodenas thank you, as a new contribuitor is not easy for me pick an issue :sweat_smile:

alessbn picture alessbn on 23 Oct 2017
👍1

Hi @frodenas you can keep working on the issue I'll pick up issue 95 馃槃

alessbn picture alessbn on 23 Oct 2017
👍1
Was this page helpful?
0 / 5 - 0 ratings

Related issues

drpaneas picture drpaneas  路  3Comments
mshahat picture mshahat  路  3Comments
mb-m picture mb-m  路  5Comments
pwittrock picture pwittrock  路  6Comments
cbluth picture cbluth  路  6Comments