Mike Kolesnik has posted comments on this change.

Change subject: core: DO NOT SUBMIT Fixing DB conf (and conf test) files
......................................................................


Patch Set 7: (3 inline comments)

....................................................
File backend/manager/modules/dal/src/test/resources/fixtures.xml
Line 1584:         <value>88296e00-0cad-4e5a-9291-008a7b7f4399</value>
Please indent properly.

Line 1608:             <value>2010-12-01 14:13:07</value>
I think here's missing the quota_id

Line 1610:         <!-- end of template images -->
This comment should be after the blank image?

--
To view, visit http://gerrit.ovirt.org/1567
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2f5dab28cd9e6c26655733de65c55cf3500f11ce
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to