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

mimic: mgr/dashboard: Improve error message when backend is unreachable #27748

Closed
wants to merge 2 commits into from
Closed

mimic: mgr/dashboard: Improve error message when backend is unreachable #27748

wants to merge 2 commits into from

Conversation

Exotelis
Copy link
Contributor

The current implementation of the HTTP interceptor does not use the error description in all cases.

Fixes: https://tracker.ceph.com/issues/36190

Signed-off-by: Volker Theile <vtheile@suse.com>
(cherry picked from commit 7d41a4a)
@ricardoasmarques
Copy link
Contributor

Make check is failing because the unit-test-helper file was moved to a different path.

You have to fix the import manually or backport #22857 before this one.

@Exotelis
Copy link
Contributor Author

Make check is failing because the unit-test-helper file was moved to a different path.

You have to fix the import manually or backport #22857 before this one.

It seems, that the file does not exist in Mimic. I'm going to put the imports and providers in the beforeEach block. This should resolve the problem.

Do not import the unit-test-helper, since the file doesn't exist in mimic.
Instead the compilation of the imports and provider will be done in a
beforeEach block

Signed-off-by: Sebastian Krah <skrah@suse.com>
@LenzGr
Copy link
Contributor

LenzGr commented May 3, 2019

I wonder if it would make sense to backport #22857 in order to make it easier for frontend PRs to be backported without having to make changes. @ricardoasmarques or @tspmelo - what do you think?

@LenzGr LenzGr requested a review from Devp00l May 3, 2019 07:47
@smithfarm
Copy link
Contributor

@LenzGr I created http://tracker.ceph.com/issues/39941 to track the backport of #22857

@Exotelis Exotelis closed this May 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants