Juan Hernandez has posted comments on this change.
Change subject: restapi: 'concurrent' Not Displayed For Primary PM Agent
......................................................................
Patch Set 1:
(1 comment)
....................................................
File
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
Line 488: if (!StringUtils.isEmpty(entity.getPmSecondaryIp())) {
Line 489: boolean concurrent = entity.isPmSecondaryConcurrent();
Line 490: // When a second agent exists, 'concurrent' field is
relevant for both agents, so here we
Line 491: // set it retroactively in the first agent.
Line 492:
model.getAgents().getAgents().get(0).setConcurrent(concurrent);
I agree that such "rule" may not make sense in the future, that is why I
suggest to put it in a place where it is a bit easier to remove.
Line 493: agent = new Agent();
Line 494: agent.setType(entity.getPmSecondaryType());
Line 495: agent.setAddress(entity.getPmSecondaryIp());
Line 496: agent.setUsername(entity.getPmSecondaryUser());
--
To view, visit http://gerrit.ovirt.org/21781
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If0882d1db6c13b24c767e0cb40b9b797e8b61860
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ori Liel <ol...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Ori Liel <ol...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
