Yaniv Bronhaim has posted comments on this change.

Change subject: Add domain url to provisioned vds and set name as unchangeable
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.ovirt.org/#/c/35214/3/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/NewHostModel.java
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/NewHostModel.java:

Line 117:                 externalComputeResourceListModel.setIsChangable(true);
Line 118:             }
Line 119:         };
Line 120:         
AsyncDataProvider.getInstance().getExternalProviderComputeResourceList(getComputeResourceQuery,
 provider);
Line 121: 
this move is needed to have the hostgroups entities set before setting the 
hosts, so that we set the host we'll be able to add the domain suffix
Line 122:         AsyncQuery getHostsQuery = new AsyncQuery();
Line 123:         getHostsQuery.asyncCallback = new INewAsyncCallback() {
Line 124:             @Override
Line 125:             public void onSuccess(Object model, Object result) {


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I74e17d69423e1c02028f596ecafb13a21ab3e3ca
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to