Tomas Jelinek has uploaded a new change for review.

Change subject: webadmin: display address override message
......................................................................

webadmin: display address override message

The patch 63867923f1d80fc2527cc78cb3a3dd92b0eff9a6
added possibility to configure the console address of a host
but did not add a validation message to all the required places.

Change-Id: I5b292c74a7b02796cc780dcebd3c2b66e643d306
Bug-Url: https://bugzilla.redhat.com/928298
Signed-off-by: Tomas Jelinek <[email protected]>
---
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java
M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
M 
frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
M 
frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M 
frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/UserPortal.gwt.xml
M 
frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
M 
frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/WebAdmin.gwt.xml
7 files changed, 9 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/80/13780/1

diff --git 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java
 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java
index 7e63b63..de147d4 100644
--- 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java
+++ 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcBllMessages.java
@@ -78,6 +78,7 @@
 
     ACTION_LIST_CANNOT_BE_EMPTY,
     ACTION_TYPE_FAILED_NAME_ALREADY_USED,
+    VALIDATION_VDS_CONSOLEADDRESSS_HOSTNAME_OR_IP,
     ACTION_TYPE_FAILED_VM_MAX_RESOURCE_EXEEDED,
     ACTION_TYPE_FAILED_VM_IN_PREVIEW,
     ACTION_TYPE_FAILED_DISKS_LOCKED,
diff --git 
a/backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties 
b/backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
index a9759af..ab3b88e 100644
--- 
a/backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
+++ 
b/backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties
@@ -590,6 +590,7 @@
 VALIDATION.VDS_GROUP.NAME.MAX=Cluster name must not exceed 40 characters
 VALIDATION.VDS_GROUP.NAME.INVALID=Cluster name must be formed of alphanumeric 
characters, numbers or "-_"
 VALIDATION_VDS_NAME_INVALID=Host name must be formed of alphanumeric 
characters, numbers or "-_."
+VALIDATION_VDS_CONSOLEADDRESSS_HOSTNAME_OR_IP=Console address must be a FQDN 
or a valid IP address
 VALIDATION.VDS_GROUP.MigrateOnError.NOT_NULL=Cluster migrate on error option 
is required
 VALIDATION.STORAGE_POOL.ID.NOT_NULL=Data Center ID is required
 VALIDATION.VDS.HOSTNAME.HOSTNAME_OR_IP=Host address must be a FQDN or a valid 
IP address
diff --git 
a/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
 
b/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
index 2b91bbb..ef0a18a 100644
--- 
a/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
+++ 
b/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java
@@ -1621,6 +1621,9 @@
     @DefaultStringValue("Host name must be formed of alphanumeric characters, 
numbers or \"-_.\"")
     String VALIDATION_VDS_NAME_INVALID();
 
+    @DefaultStringValue("Console address must be a FQDN or a valid IP address")
+    String VALIDATION_VDS_CONSOLEADDRESSS_HOSTNAME_OR_IP();
+
     @DefaultStringValue("email format is not valid")
     String VALIDATION_EVENTS_EMAIL_FORMAT();
 
diff --git 
a/frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
 
b/frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
index 3ce4e3b..ec04461 100644
--- 
a/frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
+++ 
b/frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
@@ -332,6 +332,7 @@
 VDS_GROUP_CANNOT_DO_ACTION_NAME_IN_USE=Cannot ${action} Cluster. Cluster name 
is already in use.
 NETWORK_NAME_ALREADY_EXISTS=Cannot ${action} ${type}. Network name already 
exists.
 ACTION_TYPE_FAILED_NAME_ALREADY_USED=Cannot ${action} ${type}. The ${type} 
name is already in use, please choose a unique name and try again.
+VALIDATION_VDS_CONSOLEADDRESSS_HOSTNAME_OR_IP=Console address must be a FQDN 
or a valid IP address
 ACTION_TYPE_FAILED_STORAGE_CONNECTION_NOT_EXIST=Cannot ${action} ${type}. 
Storage connection doesn't exist.
 ACTION_TYPE_FAILED_STORAGE_CONNECTION_ALREADY_EXISTS=Cannot ${action} ${type}. 
Storage connection already exists.
 ACTION_TYPE_FAILED_STORAGE_CONNECTION_UNSUPPORTED_ACTION_FOR_STORAGE=Cannot 
${action} ${type}. Storage connection parameters can be edited only for NFS 
data domains in maintenance.
diff --git 
a/frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/UserPortal.gwt.xml
 
b/frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/UserPortal.gwt.xml
index 1975db1..2e935fe 100644
--- 
a/frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/UserPortal.gwt.xml
+++ 
b/frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/UserPortal.gwt.xml
@@ -39,7 +39,7 @@
 
        <!-- Supported locales (defined via maven property) -->
        <extend-property name="locale" values="en_US,es,fr,ja,pt_BR,zh_CN,de" />
-       <set-property name="locale" value="${gwt.locale}" />
+       <set-property name="locale" value="en_US" />
        <set-property-fallback name="locale" value="en_US" />
 
 </module>
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
 
b/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
index 346e84d..9df7963 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
+++ 
b/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties
@@ -330,6 +330,7 @@
 VDS_GROUP_CANNOT_DO_ACTION_NAME_IN_USE=Cannot ${action} Cluster. Cluster name 
is already in use.
 NETWORK_NAME_ALREADY_EXISTS=Cannot ${action} ${type}. Network name already 
exists.
 ACTION_TYPE_FAILED_NAME_ALREADY_USED=Cannot ${action} ${type}. The ${type} 
name is already in use, please choose a unique name and try again.
+VALIDATION_VDS_CONSOLEADDRESSS_HOSTNAME_OR_IP=Console address must be a FQDN 
or a valid IP address
 ACTION_TYPE_FAILED_STORAGE_CONNECTION_NOT_EXIST=Cannot ${action} ${type}. 
Storage connection doesn't exist.
 ACTION_TYPE_FAILED_STORAGE_CONNECTION_ALREADY_EXISTS=Cannot ${action} ${type}. 
Storage connection already exists.
 ACTION_TYPE_FAILED_STORAGE_CONNECTION_UNSUPPORTED_ACTION_FOR_STORAGE=Cannot 
${action} ${type}. Storage connection parameters can be edited only for NFS 
data domains in maintenance.
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/WebAdmin.gwt.xml
 
b/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/WebAdmin.gwt.xml
index ea0c16c..d442c5c 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/WebAdmin.gwt.xml
+++ 
b/frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/WebAdmin.gwt.xml
@@ -39,7 +39,7 @@
 
        <!-- Supported locales (defined via maven property) -->
        <extend-property name="locale" values="en_US,es,fr,ja,pt_BR,zh_CN,de" />
-       <set-property name="locale" value="${gwt.locale}" />
+       <set-property name="locale" value="en_US" />
        <set-property-fallback name="locale" value="en_US" />
 
 </module>


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5b292c74a7b02796cc780dcebd3c2b66e643d306
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to