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

Consistent service offering/plans guid across restarts #71

Open
gberche-orange opened this issue Oct 16, 2020 · 0 comments
Open

Consistent service offering/plans guid across restarts #71

gberche-orange opened this issue Oct 16, 2020 · 0 comments

Comments

@gberche-orange
Copy link

Expected behavior

As an overview-broker user deployed in cloudfoundry

  • in order to support application restarts while still serving incoming OSB requests
  • I need that offering/plans guid to not change across application restarts

Observed behavior

Following e11bba2 the uuids are randomly generated at application restart.

Consequently, if the overview broker was registered in cloudfoundry the following errors will be observed

  • service instance provisionning requests fail with message Could not find service 895749e2-8543-4f9d-8134-3a71b4a388c4, plan 8fb7951a-f309-496a-89b2-7cbdec5a3dcc
  • if there are existing service instances, errors will occur while trying to deprovision them with message Could not find service 895749e2-8543-4f9d-8134-3a71b4a388c4, plan 8fb7951a-f309-496a-89b2-7cbdec5a3dcc
  • if the broker registration in cloudfoundry is requested, then it fails with the following message, because the services are already registered with a same name but distinct ids
Server error, status code: 502, error code: 270012, message: Service broker catalog is invalid: 

Service names must be unique within a broker. Services with names ["overview-service"] already exist

Workaround

Systematically perform a cf purge-service-offering -f overview-service -b overview-broker prior to execute a cf update-service-broker

Suggested fix

  • Use consistent hash (e.g. md5) from names (respectively service name and service plan name) as Guids
  • Add an opt-in (e.g. OSB_GUID_OFFSET) which adds an offset to the consistent hash, enabling registration of multiple overview broker instances in K8S svcat (CF support multi service registration out of the box)
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

No branches or pull requests

1 participant