Mike Kolesnik has posted comments on this change. Change subject: engine: Introduce VnicProfileValidator ......................................................................
Patch Set 22: (2 inline comments) .................................................... File backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties Line 358: -Please upgrade your Cluster to a later Compatibility version first. Line 359: VDS_GROUP_CPU_TYPE_CANNOT_BE_NULL=Cannot add Cluster. CPU type must be specified Line 360: VDS_GROUP_CANNOT_DO_ACTION_NAME_IN_USE=Cannot ${action} Cluster. Cluster name is already in use. Line 361: NETWORK_NAME_ALREADY_EXISTS=Cannot ${action} ${type}. Network name already exists. Line 362: VNIC_PROFILE_NOT_EXISTS=The specified VM network interface profile doesn't exist. Why no $action $type for this one? Line 363: VNIC_PROFILE_NAME_IN_USE=Cannot ${action} ${type}. The VM network interface profile's name is already used by an existing profile in the same data-center.\n-Please choose a different name. Line 364: VNIC_PROFILE_IN_USE=Cannot ${action} ${type}. Several ${entities} (${ENTITIES_USING_VNIC_PROFILE_COUNTER}) are using this VM network interface profile:\n${ENTITIES_USING_VNIC_PROFILE}\n - Please remove it from all ${entities} that are using it and try again. Line 365: CANNOT_CHANGE_VNIC_PROFILE_NETWORK=Cannot ${action} ${type}. VM network interface profile's network cannot be changed. Line 366: ACTION_TYPE_FAILED_NAME_ALREADY_USED=Cannot ${action} ${type}. The ${type} name is already in use, please choose a unique name and try again. Line 359: VDS_GROUP_CPU_TYPE_CANNOT_BE_NULL=Cannot add Cluster. CPU type must be specified Line 360: VDS_GROUP_CANNOT_DO_ACTION_NAME_IN_USE=Cannot ${action} Cluster. Cluster name is already in use. Line 361: NETWORK_NAME_ALREADY_EXISTS=Cannot ${action} ${type}. Network name already exists. Line 362: VNIC_PROFILE_NOT_EXISTS=The specified VM network interface profile doesn't exist. Line 363: VNIC_PROFILE_NAME_IN_USE=Cannot ${action} ${type}. The VM network interface profile's name is already used by an existing profile in the same data-center.\n-Please choose a different name. Please use ACTION_TYPE_FAILED_ prefix for keys that require translation for action/type Line 364: VNIC_PROFILE_IN_USE=Cannot ${action} ${type}. Several ${entities} (${ENTITIES_USING_VNIC_PROFILE_COUNTER}) are using this VM network interface profile:\n${ENTITIES_USING_VNIC_PROFILE}\n - Please remove it from all ${entities} that are using it and try again. Line 365: CANNOT_CHANGE_VNIC_PROFILE_NETWORK=Cannot ${action} ${type}. VM network interface profile's network cannot be changed. Line 366: ACTION_TYPE_FAILED_NAME_ALREADY_USED=Cannot ${action} ${type}. The ${type} name is already in use, please choose a unique name and try again. Line 367: ACTION_TYPE_FAILED_URL_INVALID=Cannot ${action} ${type}. The URL is not valid, please enter a valid URL and try again. -- To view, visit http://gerrit.ovirt.org/16887 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6af74a398d1d7ffb97a1802305c56ad536acb346 Gerrit-PatchSet: 22 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masa...@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkap...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches