Skip to content

cache: return api errors on not found imagestreams/deploymentconfigs#9727

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
0xmichalis:return-api-errors
Jul 7, 2016
Merged

cache: return api errors on not found imagestreams/deploymentconfigs#9727
openshift-bot merged 1 commit intoopenshift:masterfrom
0xmichalis:return-api-errors

Conversation

@0xmichalis
Copy link
Contributor

@deads2k
Copy link
Contributor

deads2k commented Jul 7, 2016

There are no ripples from this? Good for merge on green [test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 53f682a

@0xmichalis
Copy link
Contributor Author

There are no ripples from this?

No, only one used is GetConfigForController and there is no special-handling of notFound errors on the other side.

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/5885/)

@0xmichalis
Copy link
Contributor Author

[merge] ?

@openshift-bot
Copy link
Contributor

openshift-bot commented Jul 7, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/5885/) (Image: devenv-rhel7_4543)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 53f682a

@openshift-bot openshift-bot merged commit 11c7ed0 into openshift:master Jul 7, 2016
@0xmichalis 0xmichalis deleted the return-api-errors branch July 7, 2016 15:04
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