Omer Frenkel has posted comments on this change.

Change subject: gluster: BLL commands for gluster volume snapshot actions
......................................................................


Patch Set 5: Code-Review+1

(2 comments)

minor comments

http://gerrit.ovirt.org/#/c/37325/5/backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
File 
backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties:

Line 1175: ACTION_TYPE_FAILED_GLUSTER_VOLUME_SNAPSHOT_NAME_IS_EMPTY=Cannot 
${action} ${type}. Gluster volume snapshot name is empty.
Line 1176: ACTION_TYPE_FAILED_GLUSTER_VOLUME_SNAPSHOT_DOES_NOT_EXIST=Cannot 
${action} ${type}. Gluster volume snapshot ${snapname} does not exist.
Line 1177: ACTION_TYPE_FAILED_GLUSTER_VOLUME_NO_SNAPSHOTS_EXIST=Cannot 
${action} ${type}. No gluster volume snapshots exist for volume ${volumeName}.
Line 1178: ACTION_TYPE_FAILED_GLUSTER_VOLUME_SNAPSHOT_ALREADY_ACTIVATED=Cannot 
${action} ${type}. Gluster volume snapshot ${snapname} is already activated.
Line 1179: 
ACTION_TYPE_FAILED_GLUSTER_VOLUME_SNAPSHOT_ALREADY_DEACTIVATED=Cannot ${action} 
${type}. Gluster volume snapshot ${snapname} is already de-activated.
missing translation for VAR__ACTION__RESTORE
Line 1180: ACTION_TYPE_FAILED_TAG_ID_REQUIRED=Cannot ${action} ${type}. Tag ID 
is required.
Line 1181: 
Line 1182: ACTION_TYPE_FAILED_QOS_CPU_OUT_OF_RANGE_VALUES=Cannot ${action} 
${type}. Values are out of range\n(Legal range is: 1-100).
Line 1183: ACTION_TYPE_FAILED_QOS_STORAGE_OUT_OF_RANGE_VALUES=Cannot ${action} 
${type}. Values are out of range.


http://gerrit.ovirt.org/#/c/37325/5/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
File 
frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties:

Line 1140: ACTION_TYPE_FAILED_GLUSTER_VOLUME_SNAPSHOT_NAME_IS_EMPTY=Cannot 
${action} ${type}. Gluster volume snapshot name is empty.
Line 1141: ACTION_TYPE_FAILED_GLUSTER_VOLUME_SNAPSHOT_DOES_NOT_EXIST=Cannot 
${action} ${type}. Gluster volume snapshot ${snapname} does not exist.
Line 1142: ACTION_TYPE_FAILED_GLUSTER_VOLUME_NO_SNAPSHOTS_EXIST=Cannot 
${action} ${type}. No gluster volume snapshots exist for volume ${volumeName}.
Line 1143: ACTION_TYPE_FAILED_GLUSTER_VOLUME_SNAPSHOT_ALREADY_ACTIVATED=Cannot 
${action} ${type}. Gluster volume snapshot ${snapname} is already activated.
Line 1144: 
ACTION_TYPE_FAILED_GLUSTER_VOLUME_SNAPSHOT_ALREADY_DEACTIVATED=Cannot ${action} 
${type}. Gluster volume snapshot ${snapname} is already de-activated.
also here
Line 1145: ACTION_TYPE_FAILED_TAG_ID_REQUIRED=Cannot ${action} ${type}. Tag ID 
is required.
Line 1146: 
Line 1147: ACTION_TYPE_FAILED_QOS_CPU_OUT_OF_RANGE_VALUES=Cannot ${action} 
${type}. Values are out of range\n(Legal range is: 1-100).
Line 1148: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I74fa567e0910a8f97c3d127951481448983e3fd3
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi <shtri...@redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Ramesh N <rnach...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtri...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to