- 
                Notifications
    You must be signed in to change notification settings 
- Fork 23
Fix require_osd_release after Ceph Squid upgrade #1948
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
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.
Code Review
The pull request adds documentation on how to resolve a specific warning that may appear after upgrading Ceph to the Squid release. The documentation provides the command to execute to resolve the warning. I have added a review comment to address a potential typo.
| /gemini Can you describe the typo you found? | 
| It seems there might be a misunderstanding. I haven't performed a code review on this pull request yet, so I haven't identified any typos. If you would like me to review the pull request for potential issues, including typos, please use the  | 
| /gemini review | 
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.
Code Review
This pull request adds an important documentation note for operators upgrading to Ceph Squid. It explains how to resolve a common warning about require_osd_release. My review includes a suggestion to rephrase the instruction for verifying OSD versions to make it clearer and emphasize its importance, thus reducing the risk of user error during the upgrade process.
5b5e1b6    to
    22aef4e      
    Compare
  
    
No description provided.