chore(deps): bump kube to 1.17

[anik120]

Description of the change:
Updates kube dependencies to 1.17

Reviewer Checklist

• Implementation matches the proposed design, or proposal is updated to match implementation
• Sufficient unit test coverage
• Sufficient end-to-end test coverage
• Docs updated or added to /docs
• Commit messages sensible and descriptive

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: anik120
To complete the pull request process, please assign kevinrizza
You can assign the PR to them by writing /assign @kevinrizza in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[LouisPlisso]

can you please move all k8s modules to native versions: no more v0.0.0

this would be in line with operator-framework/operator-registry#201 and operator-framework/api#17

Also please note that k8s/kubernetes should not be a requirement and is used for locking the versions of the other libraries

Can you please try to remove the import of k8s.io/kubernetes and move all other imports as k8s.io/api v0.17.3 to have them as k8s 1.17.3

Could you please remove the k8s repos from the replace directive down in the go.mod, otherwise any change in the require will be overridden by the replace

[anik120]

@LouisPlisso thank you for your review. It was a WIP PR, apologies for not editing the title with WIP. PTAL now.
Unfortunately though, we still have to keep the replace directive, since we're using the rbacauthorizer of k8s.io/kubernetes, which in turn requires v0.0.0 of all k8s.io modules.

Thoughts @operator-framework/team-olm ?

[anik120]

I've attempted to remove dependency on k8s.io/kubernetes and import all other k8s.io/kubernetes as v0.17.3 by manually vendoring in k8s.io/kubernetes in #1370. I'll close this PR in favour of #1370 if that idea has sufficient votes.

[anik120]

/test unit

[anik120]

/test verify

[openshift-ci-robot]

@anik120: The following tests failed, say /retest to rerun all failed tests:

Test name	Commit	Details	Rerun command
ci/prow/verify	8d21c4c	link	/test verify
ci/prow/e2e-aws-olm	8d21c4c	link	/test e2e-aws-olm
ci/prow/e2e-gcp	8d21c4c	link	/test e2e-gcp
ci/prow/e2e-gcp-upgrade	8d21c4c	link	/test e2e-gcp-upgrade

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[njhale]

@anik120 has this been superseded by #1370?

[anik120]

@njhale yes.

Closing in favor of #1370