starred/zmbackup

Fork 0

mirror of https://github.com/lucascbeyeler/zmbackup.git synced 2026-04-25 07:05:55 +03:00

ParallelAction.sh #137

New issue

Closed

opened 2026-02-27 08:14:33 +03:00 by kerem · 6 comments

kerem commented

2026-02-27 08:14:33 +03:00

Owner

Originally created by @lomikas1 on GitHub (Jan 4, 2022).
Original GitHub issue: https://github.com/lucascbeyeler/zmbackup/issues/169

ISSUE TYPE

Bug Report

ENVIRONMENT VERSION

Zmbackup Version: 1.2.6
Zimbra Version: 8.8.15
Linux Distribution & Version: Ubuntu 18.04.6 LTS

SUMMARY

Restoring ldap entiies from full backup ( in my case to new installation of Zimbra) causing full corruption of ldap directory

STEPS TO REPRODUCE

Make full backup. 
Install new clear zimbra.
Copy full backup to new server.
Replace ldap password in zmbackup.conf
Exec zmbackup -r full_backup or zmbackup -r -ldp full_backup

ACTUAL RESULTS

After that-  ldap is unusable
Error in lib/bash/ParallelAction.sh line 82. 
Contains "$(grep ^dn: "$WORKDIR"/"$1"/"$2".ldiff | awk 'print $2')" > /dev/null 2>&1
Must be "$(grep ^dn: "$WORKDIR"/"$1"/"$2".ldiff | awk '{print $2}')" > /dev/null 2>&1
After correction all works fine

Originally created by @lomikas1 on GitHub (Jan 4, 2022). Original GitHub issue: https://github.com/lucascbeyeler/zmbackup/issues/169   ##### ISSUE TYPE - Bug Report  ##### ENVIRONMENT VERSION - Zmbackup Version: 1.2.6 - Zimbra Version: 8.8.15 - Linux Distribution & Version: Ubuntu 18.04.6 LTS  ##### SUMMARY Restoring ldap entiies from full backup ( in my case to new installation of Zimbra) causing full corruption of ldap directory  ##### STEPS TO REPRODUCE ``` Make full backup. Install new clear zimbra. Copy full backup to new server. Replace ldap password in zmbackup.conf Exec zmbackup -r full_backup or zmbackup -r -ldp full_backup ```  ##### ACTUAL RESULTS ``` After that- ldap is unusable Error in lib/bash/ParallelAction.sh line 82. Contains "$(grep ^dn: "$WORKDIR"/"$1"/"$2".ldiff | awk 'print $2')" > /dev/null 2>&1 Must be "$(grep ^dn: "$WORKDIR"/"$1"/"$2".ldiff | awk '{print $2}')" > /dev/null 2>&1 After correction all works fine ```

kerem closed this issue

2026-02-27 08:14:33 +03:00

kerem commented

2026-02-27 08:14:33 +03:00

Author

Owner

@vdeville commented on GitHub (Jan 11, 2022):

Hello
Yes this code fix one of bug,
For me i'v had the error Ldap bind ldap_bind: Confidentiality required (13)
To fix this i have added -Z arg to ldapdelete and ldapadd

Thanks

@vdeville commented on GitHub (Jan 11, 2022): Hello Yes this code fix one of bug, For me i'v had the error Ldap bind ldap_bind: Confidentiality required (13) To fix this i have added -Z arg to ldapdelete and ldapadd Thanks

kerem commented

2026-02-27 08:14:33 +03:00

Author

Owner

@FlorianHeigl commented on GitHub (Mar 12, 2025):

haha, I spent the last 5 hours NOT noticing this issue report and finally tracked it down to the very same thing.
Good morning to you 3 years old issue :>

(no idea how I ended up without the patch but honestly I don't care anymore after so a few days of sh** like this)

@FlorianHeigl commented on GitHub (Mar 12, 2025): haha, I spent the last 5 hours NOT noticing this issue report and finally tracked it down to the very same thing. Good morning to you 3 years old issue :> (no idea how I ended up without the patch but honestly I don't care anymore after so a few days of sh** like this)

kerem commented

2026-02-27 08:14:33 +03:00

Author

Owner

@FlorianHeigl commented on GitHub (Mar 12, 2025):

@vdeville @lucascbeyeler @iqkotze if either of you would like a paypal'd pizza as a thanks, let me know. having, at last, some ability to proceed to a clean setup after a chain of issues, to get rid of the emergency duct tape I had put in place is thanks to you all.

(hopefully Zimbra or a fork thereof will get a few good architects who give it the design boost it needed since 20 years or so. like M$ had to do for their cloud offering too. anyway. I love watching a restore happening!)

@FlorianHeigl commented on GitHub (Mar 12, 2025): @vdeville @lucascbeyeler @iqkotze if either of you would like a paypal'd pizza as a thanks, let me know. having, at last, some ability to proceed to a clean setup after a chain of issues, to get rid of the emergency duct tape I had put in place is thanks to you all. (hopefully Zimbra or a fork thereof will get a few good architects who give it the design boost it needed since 20 years or so. like M$ had to do for their cloud offering too. anyway. I love watching a restore happening!)

kerem commented

2026-02-27 08:14:33 +03:00

Author

Owner

@lucascbeyeler commented on GitHub (Mar 17, 2025):

Don't paypal me - I did nothing aside merge this patch. My job position that I have today made me distant from Zimbra so it's impossible to provide any form of update.

The real hero here was @alsyundawy - the one who fixed the LDAP.

Closing the issue since looks like the problem was solved in the end.

@lucascbeyeler commented on GitHub (Mar 17, 2025): Don't paypal me - I did nothing aside merge this patch. My job position that I have today made me distant from Zimbra so it's impossible to provide any form of update. The real hero here was @alsyundawy - the one who fixed the LDAP. Closing the issue since looks like the problem was solved in the end.

kerem commented

2026-02-27 08:14:33 +03:00

Author

Owner

@alsyundawy commented on GitHub (Mar 31, 2025):

thx

@alsyundawy commented on GitHub (Mar 31, 2025): thx

kerem commented

2026-02-27 08:14:34 +03:00

Author

Owner

@FlorianHeigl commented on GitHub (Feb 18, 2026):

spoiler: I never updated the installed version in /usr/local/lib and, yes, I ran into this again and had absolutely no recollection of this being the case less than a year ago.
4 days...

@FlorianHeigl commented on GitHub (Feb 18, 2026): spoiler: I never updated the installed version in /usr/local/lib and, yes, I ran into this again and had absolutely no recollection of this being the case less than a year ago. 4 days...

kerem referenced this issue

2026-02-27 08:14:49 +03:00

[PR #137] [MERGED] Use .ldif extension for LDIF files #189