From Dan Kenigsberg <[email protected]>:

Dan Kenigsberg has submitted this change and it was merged.

Change subject: mailbox: Add tests for mailbox validation
......................................................................


mailbox: Add tests for mailbox validation

When SPM mailbox is reading new mail, it validates the mailbox checksum.
Add tests before modifying this code.

Change-Id: I4ec15704cc913777b5bf9dd8bd983cdec6336673
Bug-Url: https://bugzilla.redhat.com/1426762
Signed-off-by: Nir Soffer <[email protected]>
---
M lib/vdsm/storage/mailbox.py
M tests/storage_mailbox_test.py
2 files changed, 32 insertions(+), 2 deletions(-)

Approvals:
  Adam Litke: Looks good to me, approved
  Nir Soffer: Verified
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/73077
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4ec15704cc913777b5bf9dd8bd983cdec6336673
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to