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: update endpoint with IP for external RGW server #9010
Conversation
@thotz can you explain more about the changes like what is the problem... just for my understanding? Thanks |
Basically fix based on my code reading, so I might be wrong here as well. The current code always generate |
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'm not sure why this is needed, the service should be present and pointing to the ExternalRgwEndpoints
already. I haven't fully read the issue but I don't see how this code is fixing anything (yet?). Thanks
Actually, if the service is not there, it might be that it's created later? |
That's possible I guess |
For external RGW server use the IP mentioned in Gateway for admin Ops operattions. Fixes: rook#8916 Signed-off-by: Jiffin Tony Thottan <thottanjiffin@gmail.com>
cb3dd8b
to
d4562f6
Compare
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 looks more like the right fix for this issue. Can we validate this via the controller_test.go?
@leseb : The |
Sounds good and I just realized there are no unit tests for external mode either... #9025 |
ceph: update endpoint with IP for external RGW server (backport #9010)
Description of your changes:
For external RGW server use the IP mentioned in Gateway for admin Ops
operattions.
Signed-off-by: Jiffin Tony Thottan thottanjiffin@gmail.com
Which issue is resolved by this Pull Request:
Resolves #8916
Checklist:
make codegen
) has been run to update object specifications, if necessary.