Eli Mesika has uploaded a new change for review.

Change subject: core: Bad request error doesn't returns enough...
......................................................................

core: Bad request error doesn't returns enough...

Bad request error doesn't returns enough details (cannot understand
from the error what was bad/missing in request)

Removing the notNull annotation from the managementIp field since
although that this is true for new PM definitions, it is not mandatory
for updates and should be define exactly as the secondary management IP
field.

Change-Id: I163166bb6d58c475c7e3361821e928bc3f7b3a47
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1013301
Signed-off-by: Eli Mesika <[email protected]>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java
1 file changed, 0 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/76/19676/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java
index 7d6e5fd..7ae485f 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java
@@ -37,7 +37,6 @@
 
     @EditableField
     @HostnameOrIp(message = 
"VALIDATION.VDS.POWER_MGMT.ADDRESS.HOSTNAME_OR_IP", groups = 
PowerManagementCheck.class)
-    @NotNull(groups = PowerManagementCheck.class)
     @Size(max = BusinessEntitiesDefinitions.HOST_IP_SIZE)
     private String managementIp;
 


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

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

Reply via email to