- 
                Notifications
    You must be signed in to change notification settings 
- Fork 23
feat: add step to perform image data migration #1953
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
base: stackhpc/2025.1
Are you sure you want to change the base?
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
This pull request updates the documentation to include a necessary data migration step for Pulp after an upgrade. The changes are a good addition. I have a couple of suggestions to improve the clarity and robustness of the documentation, including fixing a formatting issue and adding a note to clarify when the new commands should be run.
| Why not add that to etc/kayobe/containers/pulp/post.yml? I understand that it won't break anything? | 
134ca78    to
    f8a5be0      
    Compare
  
    The command `pulpcore-manager container-handle-image-data`[0] can be used to resolve issues with missing image fields. This will happen post deployment of Pulp. [0]: pulp/pulp_container#1554
f8a5be0    to
    59d035b      
    Compare
  
    | 
 Updated docs and added task to  | 
The command
pulpcore-manager container-handle-image-data0 can be used to resolve issues with missing image fields.