-
Notifications
You must be signed in to change notification settings - Fork 68
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
No validation on backup files - roll back not possible #27
Comments
Thanks for the report. We are looking into a fix. |
No worries @jtran-cloudera I will submit a PR today, I have a number of fixes. My clients raised cases with cloudera too. So it’s in the notes for those cases |
Hi @jtran-cloudera, I am not able to send you my code via public as it's IP. |
Hi @jtran-cloudera, @sdevineni, I see you added the code to validate the backup file, but it's only on jar files, it's also needed on every backup file, like the tar.gz, nar and the new uberjar code. I see you also added
Thanks |
Yes, we are working to update this for nar files as well. if backup fails, it could be because of permissions or space elated issues. hence a fail-fast methodology is adopted to figure our the reason behind the backup creation. |
Hi @sunilgovind, Sounds good. I have already added the sha checksum to the tar.gz and nar too I disagree, from the point of view of automation, I don't want the script to die, it should report issues, not action in those cases and move on. When you have to work on 100's and 1000's of servers to run the patch, you don't have time to stop and debug on Production. All testing needs to be done in NonProd and get all the issue sorted before running in Production Thanks |
…rable class for both HDP and CDH side. Re-run is not needed. (#27)
There is no validation on backup files.
I have a case where the backup path filled up from the script and a number of jar files didn't get backed up, but did get modified.
This means there is no rollback - very bad
The text was updated successfully, but these errors were encountered: