Skip to content

Fix state handling for GET /v3/service_instances/:guid/parameters#2750

Merged
philippthun merged 3 commits intocloudfoundry:mainfrom
sap-contributions:fix-state-handling-for-get-service-instances-params
Jul 4, 2022
Merged

Fix state handling for GET /v3/service_instances/:guid/parameters#2750
philippthun merged 3 commits intocloudfoundry:mainfrom
sap-contributions:fix-state-handling-for-get-service-instances-params

Conversation

@svkrieger
Copy link
Copy Markdown
Contributor

@svkrieger svkrieger commented Apr 4, 2022

Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:

  • Return ServiceInstanceNotFound when service instance state is create failed

An explanation of the use cases your change solves

@svkrieger svkrieger force-pushed the fix-state-handling-for-get-service-instances-params branch from bee3b23 to b33b26d Compare April 4, 2022 14:45
@philippthun philippthun self-assigned this Apr 4, 2022
@philippthun philippthun self-requested a review April 4, 2022 15:55
@svkrieger svkrieger force-pushed the fix-state-handling-for-get-service-instances-params branch 2 times, most recently from fba612c to e6373d9 Compare April 6, 2022 13:51
- Return `ServiceInstanceNotFound` when state is `create failed`
- Return more specific error when state is `delete in progress`
@svkrieger svkrieger force-pushed the fix-state-handling-for-get-service-instances-params branch from e6373d9 to b2148e8 Compare June 24, 2022 11:15
Comment thread app/controllers/v3/service_instances_controller.rb Outdated
Comment thread app/controllers/v3/service_instances_controller.rb Outdated
@philippthun
Copy link
Copy Markdown
Member

Will be merged together with the related PRs...

@philippthun philippthun merged commit eb10971 into cloudfoundry:main Jul 4, 2022
will-gant pushed a commit to sap-contributions/cloud_controller_ng that referenced this pull request Dec 16, 2022
…oudfoundry#2750)

Return `ServiceInstanceNotFound` when state is `create failed`
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.

3 participants