-
Notifications
You must be signed in to change notification settings - Fork 6k
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
radosgw-admin: skip GC init on read-only admin ops #41027
Conversation
jenkins test make check |
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.
overall, looks fine to me
src/rgw/rgw_rados.h
Outdated
@@ -480,6 +480,7 @@ class RGWRados | |||
RGWIndexCompletionManager *index_completion_manager{nullptr}; | |||
|
|||
bool use_cache{false}; | |||
bool readonly{false}; |
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.
I guess this makes sense. I am more strongly preferring the idea of pulling admin into a remote service all the time.
1bc086c
to
d3fb7d8
Compare
jenkins test make check |
e1e85e8
to
fecb9cc
Compare
updated as discussed on stadup, |
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.
this actually now does look good to me, read it earlier in the day ;)
fecb9cc
to
91b8521
Compare
Fixes: https://tracker.ceph.com/issues/50520 Signed-off-by: Mark Kogan <mkogan@redhat.com>
91b8521
to
9ac1991
Compare
Fixes: https://tracker.ceph.com/issues/50520
Signed-off-by: Mark Kogan mkogan@redhat.com
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox