Change in ovirt-engine[master]: webadmin: Add check for the previous state of portMirroring ...

Code Review gerrit at ovirt.org
Wed Aug 30 18:42:02 UTC 2017


>From Alexander Wels <awels at redhat.com>:

Alexander Wels has submitted this change and it was merged. ( https://gerrit.ovirt.org/81202 )

Change subject: webadmin: Add check for the previous state of portMirroring and passthrough
......................................................................


webadmin: Add check for the previous state of
portMirroring and passthrough

Add check that previous state is not overwritten by the new state.
Copy the old changeProhibitionReason message in case it is needed.

Change-Id: I913be053b86092e99447070858822dae70e88ea7
Signed-off-by: Ales Musil <amusil at redhat.com>
Bug-Url: https://bugzilla.redhat.com/1435094
Bug-Url: https://bugzilla.redhat.com/1486547
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/profiles/VnicProfileModel.java
1 file changed, 7 insertions(+), 3 deletions(-)

Approvals:
  Ales Musil: Verified
  Alexander Wels: Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/81202
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I913be053b86092e99447070858822dae70e88ea7
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ales Musil <amusil at redhat.com>
Gerrit-Reviewer: Ales Musil <amusil at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list