Greg Padgett has posted comments on this change. Change subject: WIP restapi: Add fields for CPU Thread Handling (4/5) ......................................................................
Patch Set 2: Hi Michael, thanks. The reason for two threads_as_cores is that one is what vdsm is doing on the host side (reporting threads as cores through its api), and the other is what the user's setting is on the engine side. We want to effectively deprecate the vdsm setting, but we need to see it in order to figure out if vdsm is lying to us about the number of cores. If it is [lying] then we want to make sure to not double-count threads. -- To view, visit http://gerrit.ovirt.org/10169 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iee8eebee7db2611276f58169aef196053542c7a5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadg...@redhat.com> Gerrit-Reviewer: Doron Fediuck <dfedi...@redhat.com> Gerrit-Reviewer: Greg Padgett <gpadg...@redhat.com> Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
