Skip to content

Prevent binding creation when instance is in state 'create failed'#2808

Merged
philippthun merged 1 commit intocloudfoundry:mainfrom
sap-contributions:no-binding-creation-when-instance-in-failed-state
Jul 5, 2022
Merged

Prevent binding creation when instance is in state 'create failed'#2808
philippthun merged 1 commit intocloudfoundry:mainfrom
sap-contributions:no-binding-creation-when-instance-in-failed-state

Conversation

@philippthun
Copy link
Copy Markdown
Member

@philippthun philippthun commented May 18, 2022

  • Apply changes done in ServiceCredentialBindingAppCreate [1] also to ServiceCredentialBindingKeyCreate and ServiceRouteBindingCreate.
  • Move common code into ServiceBindingCreate.

[1] #2756

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@philippthun philippthun force-pushed the no-binding-creation-when-instance-in-failed-state branch from 330cc20 to cff756a Compare May 19, 2022 09:52
- Apply changes done in ServiceCredentialBindingAppCreate also to
  ServiceCredentialBindingKeyCreate and ServiceRouteBindingCreate.
- Move common code into ServiceBindingCreate.
@philippthun philippthun force-pushed the no-binding-creation-when-instance-in-failed-state branch from cff756a to 90bb223 Compare July 4, 2022 09:39
@philippthun philippthun changed the title Prevent binding creation when instance is in failed state Prevent binding creation when instance is in state 'create failed' Jul 4, 2022
@philippthun philippthun marked this pull request as ready for review July 4, 2022 10:48
@philippthun philippthun merged commit 1c08d23 into cloudfoundry:main Jul 5, 2022
@FloThinksPi FloThinksPi deleted the no-binding-creation-when-instance-in-failed-state branch July 5, 2022 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants