Skip to content

return gone on unbind from non-existent templateinstance #18416

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

Merged
merged 1 commit into from
Feb 7, 2018

Conversation

bparees
Copy link
Contributor

@bparees bparees commented Feb 2, 2018

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 2, 2018
@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 2, 2018
@bparees bparees changed the title return gone on unbind from non-existence templateinstance [WIP] return gone on unbind from non-existence templateinstance Feb 2, 2018
@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 2, 2018
@spadgett
Copy link
Member

spadgett commented Feb 2, 2018

Looks correct to me per the open service broker spec

@bparees bparees changed the title [WIP] return gone on unbind from non-existence templateinstance [WIP] return gone on unbind from non-existent templateinstance Feb 2, 2018
@bparees bparees force-pushed the tsb_binding branch 2 times, most recently from 8249c88 to 8bfa1c4 Compare February 3, 2018 20:10
@bparees bparees changed the title [WIP] return gone on unbind from non-existent templateinstance return gone on unbind from non-existent templateinstance Feb 3, 2018
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 3, 2018
@bparees
Copy link
Contributor Author

bparees commented Feb 3, 2018

disabled the new test for now since it won't pass until the TSB changes make it into a published image.

@bparees
Copy link
Contributor Author

bparees commented Feb 3, 2018

/retest

@jim-minter
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 5, 2018
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bparees, jim-minter

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@bparees
Copy link
Contributor Author

bparees commented Feb 5, 2018

/retest

1 similar comment
@bparees
Copy link
Contributor Author

bparees commented Feb 5, 2018

/retest

@sdodson
Copy link
Member

sdodson commented Feb 6, 2018

GCP job updated to force log level 5.
/test gcp

@bparees
Copy link
Contributor Author

bparees commented Feb 6, 2018

/retest

1 similar comment
@bparees
Copy link
Contributor Author

bparees commented Feb 6, 2018

/retest

@bparees
Copy link
Contributor Author

bparees commented Feb 6, 2018

/retest

@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit 4957392 into openshift:master Feb 7, 2018
openshift-merge-robot added a commit that referenced this pull request Feb 7, 2018
Automatic merge from submit-queue.

make sure we can unbind a deleted templateinstance

won't pass tests until #18416 has merged and new images are pushed.
@bparees bparees deleted the tsb_binding branch February 9, 2018 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. retest-not-required size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants