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

rgw: keystone: bump up logging when error is received #18985

Merged
merged 1 commit into from Nov 20, 2017

Conversation

Projects
None yet
6 participants
@theanalyst
Copy link
Member

commented Nov 17, 2017

When keystone engine is used and there is an error (often due to
misconfigured urls etc) these do not show up at the final logs as we
just return -1 at the final stage. Adding logs at level 5 for these
cases, as they can help identify a possible misconfigured url or admin
token. Eg:

5 Failed keystone auth from http://localhost:5000/v3/v3/auth/tokens with 404

Fixes: http://tracker.ceph.com/issues/22151
Signed-off-by: Abhishek Lekshmanan abhishek@suse.com

rgw: keystone: bump up logging when error is received
When keystone engine is used and there is an error (often due to
misconfigured urls etc) these do not show up at the final logs as we
just return  -1 at the final stage. Adding logs at level 5 for these
cases, as they can help identify a possible misconfigured url or admin
token. Eg:

```
5 Failed keystone auth from http://localhost:5000/v3/v3/auth/tokens with 404
```

Fixes: http://tracker.ceph.com/issues/22151
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>

@theanalyst theanalyst requested review from rzarzynski and cbodley Nov 17, 2017

@rzarzynski
Copy link
Contributor

left a comment

LGTM.

@joscollin

This comment has been minimized.

Copy link
Member

commented Nov 20, 2017

@oritwas oritwas merged commit c0e4426 into ceph:master Nov 20, 2017

5 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.