kubernetes-retired / service-catalog

Consume services in Kubernetes using the Open Service Broker API
https://svc-cat.io
Apache License 2.0
1.05k stars 385 forks source link

Register broker not working on kubernetes 1.18 #2793

Closed georgifarashev closed 4 years ago

georgifarashev commented 4 years ago

Bug Report

When I try to register a broker: svcat register mybroker --url <broker_url> --basic-secret <secret_name> I get the following error: Error: register request failed (ServiceBroker.servicecatalog.k8s.io "mybroker" is invalid: metadata.managedFields.fieldsType: Invalid value: "": must be FieldsV1\)

What you expected to happen: The broker to be registered successfully.

How to reproduce it (as minimally and precisely as possible): 1) create a k8s cluster with version 1.18.x 2) install service catalog version 0.3.0-beta2 3) register a broker using svcat

Anything else we need to know?:

Environment:

mszostok commented 4 years ago

Hi @georgifarashev

Yes, the service catalog does not support 1.18 yet. AFAIK there are some changes in client-go so we need to bump that dependency.

Kubernetes v.1.18.0 introduces following change https://kubernetes.io/docs/reference/using-api/api-concepts/#server-side-apply

alexwo commented 4 years ago

Hi @mszostok ,

Thanks for your quick response! and the rapid PR to update everything (https://github.com/kubernetes-sigs/service-catalog/pull/2796/commits)

Is the fix for supporting k8s 1.18 version is planned to be released as part of the 0.3.0 milestone? or perhaps implemented and released more early?

Thanks, Alex

mszostok commented 4 years ago

Hi @alexwo

so the plan is as follow:

  1. Switch service catalog to go 1.13 with go modules (https://github.com/kubernetes-sigs/service-catalog/pull/2795)
  2. Remove unused code from api-server (https://github.com/kubernetes-sigs/service-catalog/pull/2720)
  3. Bump to K8s version (https://github.com/kubernetes-sigs/service-catalog/pull/2796)

We hope to finish all that stuff this week and do the 0.3.0 release as adding beta prefix (v0.3.0-beta.3) does not make sense anymore as we are using the CRD already for half a year and we didn't have any problems with this implementation.

Rest issues from the 0.3.0 milestone will be addressed later and release with 0.3.x versions :wink:

mszostok commented 4 years ago

Hi @alexwo & @georgifarashev 🙂

The new release is now available https://github.com/kubernetes-sigs/service-catalog/releases/tag/v0.3.0 with support for k8s 1.18 🎉

please let me know if you will encounter any problem with that release and sorry for the delay but we had a few problems (e.g. downtime of quey.io, switching from go dep to go mod etc.)