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

Support synchronous managed service instance creation #3481

Closed
georgethebeatle opened this issue Sep 18, 2024 · 0 comments
Closed

Support synchronous managed service instance creation #3481

georgethebeatle opened this issue Sep 18, 2024 · 0 comments

Comments

@georgethebeatle
Copy link
Member

Background

According to the OSBAPI spec the broker may create the service instance synchronously and respond with 201 Created. The spec does not make it clear if getting last operation status will be possible in this case. Some brokers may not be serving the service instance last operation endpoint for synchronous creates, so we should make sure we only get last operation when the broker responeded with 202 Accepted and do nothing otherwise.

Acceptance

GIVEN a managed CFServiceInstance resource is created
WHEN the broker responds with 201 Created to the PUT /v2/service_instances/:instance_id request
THEN the CFServiceInstance becomes ready
AND no GET /v2/service_instances/:instance_id/last_operation request was sent to the OSBAPI broker

Dev Notes

  • As the synchronous/asynchronous nature of the OSBAPI request is a low level implementation detail it is not practical to write high level tests (e2e/smoke) for this feature. Instead we should test it with a controller test, where the OSBAPI client is a fake, which makes it easy to check what requests have been performed against the broker
@github-project-automation github-project-automation bot moved this to 🧊 Icebox in Korifi - Backlog Sep 18, 2024
@georgethebeatle georgethebeatle moved this from 🧊 Icebox to 🇪🇺 To do in Korifi - Backlog Sep 18, 2024
@georgethebeatle georgethebeatle added this to the v0.13 milestone Sep 20, 2024
zabanov-lab added a commit to zabanov-lab/korifi that referenced this issue Sep 27, 2024
zabanov-lab added a commit to zabanov-lab/korifi that referenced this issue Sep 27, 2024
@github-project-automation github-project-automation bot moved this from 🔄 In progress to ✅ Done in Korifi - Backlog Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants