Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

pkg/scaffold: update to controller-runtime v0.1.10 and k8s 1.13.1 #1020

Merged

Conversation

hasbro17
Copy link
Contributor

Description of the change:
Update the dep constraints for the controller-runtime version to v0.1.10 and the k8s.io deps to kubernetes-1.13.1.
Also removed the requirement for the k8s.io/code-generator/cmd/openapi-gen pkg since that has been moved to the repo k8s.io/kube-openapi.

Motivation for the change:
Lagging behind the latest controller-runtime updates.

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 29, 2019
@hasbro17 hasbro17 force-pushed the update-controller-runtime-0.1.10 branch from 188815d to 83615f2 Compare January 29, 2019 21:02
@hasbro17 hasbro17 changed the title pkg/scaffold: update to controller-runtime v0.1.10 and k8s 1.13.1 [WIP] pkg/scaffold: update to controller-runtime v0.1.10 and k8s 1.13.1 Jan 29, 2019
@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 29, 2019
@hasbro17 hasbro17 force-pushed the update-controller-runtime-0.1.10 branch from 83615f2 to cc1cf8e Compare January 29, 2019 22:52
@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 29, 2019
@hasbro17 hasbro17 force-pushed the update-controller-runtime-0.1.10 branch from cc1cf8e to 292770e Compare January 29, 2019 23:04
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 29, 2019
@hasbro17 hasbro17 changed the title [WIP] pkg/scaffold: update to controller-runtime v0.1.10 and k8s 1.13.1 pkg/scaffold: update to controller-runtime v0.1.10 and k8s 1.13.1 Jan 29, 2019
Copy link
Member

@joelanford joelanford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hasbro17 hasbro17 merged commit a6a0324 into operator-framework:master Jan 31, 2019
@hasbro17 hasbro17 deleted the update-controller-runtime-0.1.10 branch January 31, 2019 18:40
@hasbro17
Copy link
Contributor Author

Forgot to update the CHANGELOG. I'll do a follow up for that.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants