Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[V2] bump k8s to 1.18 and remove knative.dev deps #803

Merged
merged 2 commits into from
May 4, 2020

Conversation

zroubalik
Copy link
Member

@zroubalik zroubalik commented May 4, 2020

To support #802 we need to bump k8s.io version to 1.18 first. k8s.io 1.18 has introduced some breaking changes mainly in client-go interface, some of our deps aren't yet rebuilded to support this, I have worked it out this way:

  • knative.dev/pkg - we are using a very small subset of the code for ducktyping, I have moved the relevant code into our repo, so we don't depend on knative.dev anymore
  • custom-metrics-apiserver - it is still on k8s.io 1.17, I ve created PR there, which needs to be merged. In the meantime KEDA code is refereing to my fork of custom-metrics-apiserver which has been rebuild on k8s.io 1.18 and is working fine

Zbynek Roubalik added 2 commits May 4, 2020 16:16
Signed-off-by: Zbynek Roubalik <zroubali@redhat.com>
Signed-off-by: Zbynek Roubalik <zroubali@redhat.com>
@zroubalik zroubalik requested a review from ahmelsayed as a code owner May 4, 2020 16:41
@ahmelsayed ahmelsayed merged commit 1e4b8db into kedacore:v2 May 4, 2020
zroubalik added a commit to zroubalik/keda that referenced this pull request May 20, 2020
zroubalik added a commit to zroubalik/keda that referenced this pull request Jun 17, 2020
zroubalik added a commit to zroubalik/keda that referenced this pull request Jul 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants