Skip to content
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

backup mdbox -> rbox #118

Closed
jrse opened this issue Jan 22, 2018 · 3 comments
Closed

backup mdbox -> rbox #118

jrse opened this issue Jan 22, 2018 · 3 comments

Comments

@jrse
Copy link
Contributor

jrse commented Jan 22, 2018

t10.mdbox.log
t10.rbox.log
diff.log

in some cases there is a difference of the keyword order:
e.g.
rbox: $label1 $label4 $label5
mdbox: $label4 $label1 $label5

to reproduce:

  • create mdbox mailbox with imaptest
  • do doveadm backup to rbox

validate:

compare.sh

users t1 - t100

#!/bin/bash
for i in seq 1 100;
do
doveadm fetch -u t$i "guid date.received date.sent flags pop3.uidl seq size.virtual uid user mailbox-guid mailbox" ALL > diff.text1
doveadm -c path to mdbox dovecot.conf fetch -u t$i "guid date.received date.sent flags pop3.uidl seq size.virtual uid user mailbox-guid mailbox" ALL > diff.text2

result=$(diff -y -W 200 diff.text1 diff.text2)

if [ $? -eq 0 ]
then
echo "all the same t$i"
else
echo "files are different t$i"
echo "$result"
fi
done

echo "done"

#################

@jrse
Copy link
Contributor Author

jrse commented Jan 22, 2018

a retest with a backup from mdbox to sdbox shows that the keyword order is also not preserved.

@jrse
Copy link
Contributor Author

jrse commented Jan 22, 2018

Due to the fact that sdbox storage plugin is handling keywords in the same way, i looked for answers regarding this issue and got the answer:

"there is nothing that guarantees their ordering
iow, you should not expect that to be preserved"

So i think we need to modify our check scripts to handle this "random" ordering of keywords.

@pburow
Copy link
Contributor

pburow commented Jan 25, 2018

I changed the keyword checking script by omitting the field "flags" now. But I extended the script with manual flags verification.

@pburow pburow closed this as completed Jan 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants