Change in vdsm[master]: machinetype: split and slightly refactor cpu_models function

danken at redhat.com danken at redhat.com
Wed Mar 9 13:03:47 UTC 2016


Dan Kenigsberg has submitted this change and it was merged.

Change subject: machinetype: split and slightly refactor cpu_models function
......................................................................


machinetype: split and slightly refactor cpu_models function

cpu_models has grown large as it has to

a) having to find the correct architecture element,
b) find the cpu models in that element.

This patch splits the function and updates variables to have more
descriptive names.

Change-Id: I264209f94ef7140e27c534f1df408c7a3a17bbbf
Signed-off-by: Martin Polednik <mpolednik at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/54301
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <fromani at redhat.com>
---
M lib/vdsm/machinetype.py
1 file changed, 30 insertions(+), 26 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, approved
  Martin Polednik: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I264209f94ef7140e27c534f1df408c7a3a17bbbf
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list