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
ceph: enable debug for adminops client if the rook loglevel <= debug #8208
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Funny, I have a similar patch locally that I haven't pushed yet, you beat me to it :)
e85eb08
to
5fb4721
Compare
Just to answer anyone who might have concerns, turning on DEBUG on the API does not leak connection credentials 😌 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not the healthchecker btw? /pkg/operator/ceph/object/health.go
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not the healthchecker btw? /pkg/operator/ceph/object/health.go
This pull request has merge conflicts that must be resolved before it can be merged. @thotz please rebase it. https://rook.io/docs/rook/master/development-flow.html#updating-your-fork |
The debug for adminOps client can be enabled by setting `Debug` flag. In this PR, it is enabled for OBC and cephobjectstore not healthchecker. Also add similar changes while calling `NewS3Agent` in the bucket provisioner code path. Signed-off-by: Jiffin Tony Thottan <thottanjiffin@gmail.com>
5fb4721
to
560f44b
Compare
Done |
ceph: enable debug for adminops client if the rook loglevel <= debug (backport #8208)
The debug for adminOps client can be enabled by setting
Debug
flag.In this PR, it is enabled for OBC and cephobjectstore not healthchecker.
Also add similar changes while calling
NewS3Agent
in the bucketprovisioner code path.
Signed-off-by: Jiffin Tony Thottan thottanjiffin@gmail.com
Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:
make codegen
) has been run to update object specifications, if necessary.