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: fixing ClientID of log-collector for RGW instance #8889
Conversation
9909c43
to
9546b33
Compare
rook/pkg/operator/ceph/controller/spec.go Line 80 in 1f49b6f
|
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.
There is definitely a bug in the current implementation.
Please write a helper to return the name and unit test it.
Thanks.
9546b33
to
8d9bc5e
Compare
ResourceName: resourceName, | ||
} | ||
daemon := getDaemonName(rgwConfig) | ||
assert.Equal(t, "ceph-client.rgw.default", daemon) |
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.
The daemon log also has the daemon ID which is missing in the example, e,g: ceph-client.rgw.my.store.a.log
. Let's make this example more realistic.
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 would also suggest that we should have 2 test cases for the unit test to make sure that it works as expected for different inputs.
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.
Done
8d9bc5e
to
aa07551
Compare
aa07551
to
aa9c6b8
Compare
The Client_ID generated by operator was different from the log rotate file created The Clinet_ID= rgwceph.client.rook.ceph.rgw.my.store.a and log file name= ceph-client.rgw.my.store.a.log So changed the CLient_ID to ceph-client.rgw.my.store.a for correct working and this follow the patterns how other modules Client_ID is generated Closes: rook#8692 Signed-off-by: parth-gr <paarora@redhat.com>
aa9c6b8
to
fc7905a
Compare
ceph: fixing ClientID of log-collector for RGW instance (backport #8889)
Description of your changes:
Which issue is resolved by this Pull Request:
Resolves # #8692
Checklist:
make codegen
) has been run to update object specifications, if necessary.