Sandro Bonazzola has posted comments on this change.

Change subject: packaging: setup: set cluster level CPU id
......................................................................


Patch Set 2: Verified-1

(1 comment)

....................................................
File src/plugins/ovirt-hosted-engine-setup/engine/add_host.py
Line 400:                     ),
Line 401:                     override_iptables=True,
Line 402:                     cpu=self._ovirtsdk_xml.params.CPU(
Line 403:                         
id=self.environment[ohostedcons.VDSMEnv.ENGINE_CPU]
Line 404:                     ),
Cluster here is still created as SandyBridge also if cpu here is Westmere. 
Michael, can you please advise?
Line 405:                 )
Line 406:             )
Line 407: 
Line 408:         except ovirtsdk.infrastructure.errors.RequestError as e:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I141304606f4a1f11bb0f9fda8b1c822669d560c8
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Alex Lourie <alou...@redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadg...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@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

Reply via email to