murano uses deprecated psutil.NUM_CPUS

Bug #1598208 reported by Kirill Zaitsev on 2016-07-01
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Murano
Wishlist
Kirill Zaitsev
Mitaka
High
Kirill Zaitsev
Newton
Wishlist
Kirill Zaitsev
Ubuntu Cloud Archive
Undecided
Unassigned
Mitaka
Undecided
Unassigned
Newton
Undecided
Unassigned
murano (Ubuntu)
Undecided
Unassigned
Xenial
Undecided
Unassigned
Yakkety
Undecided
Unassigned
Zesty
Undecided
Unassigned

Bug Description

In [2]: psutil.NUM_CPUS
/Users/teferi/murano/horizon/.tox/venv/bin/ipython:1: DeprecationWarning: NUM_CPUS constant is deprecated; use cpu_count() instead
  #!/Users/teferi/murano/horizon/.tox/venv/bin/python2.7

Fix proposed to branch: master
Review: https://review.openstack.org/336619

Changed in murano:
status: Confirmed → In Progress

Reviewed: https://review.openstack.org/336619
Committed: https://git.openstack.org/cgit/openstack/murano/commit/?id=8fb54da2f91a88286fcc1b444d26b387cd04d174
Submitter: Jenkins
Branch: master

commit 8fb54da2f91a88286fcc1b444d26b387cd04d174
Author: Kirill Zaitsev <email address hidden>
Date: Fri Jul 1 18:20:38 2016 +0300

    Use multiprocessing.cpu_count instead of psutil.NUM_CPUS

    psutil.NUM_CPUS has been deprecated in psutil 2.x
    While currently global-requirements restrict usage of psutil to
    versions <2.0.0 it can be good to be prepared for upgrade in advance.
    This commit makes use of multiprocessing module instead.

    Change-Id: I6a4653eb7e5c26c3068a4d766acb4bc219920f75
    Closes-Bug: #1598208

Changed in murano:
status: In Progress → Fix Released

This issue was fixed in the openstack/murano 3.0.0.0b2 development milestone.

Kirill Zaitsev (kzaitsev) wrote :

This is actually the case with UCA packages of murano. Ubuntu packages 3.4.2 even though global-requirements has <2.0.0

Reviewed: https://review.openstack.org/360727
Committed: https://git.openstack.org/cgit/openstack/murano/commit/?id=521f031490f07f109c33f9409f8370db96555873
Submitter: Jenkins
Branch: stable/mitaka

commit 521f031490f07f109c33f9409f8370db96555873
Author: Kirill Zaitsev <email address hidden>
Date: Fri Jul 1 18:20:38 2016 +0300

    Use multiprocessing.cpu_count instead of psutil.NUM_CPUS

    psutil.NUM_CPUS has been deprecated in psutil 2.x
    While currently global-requirements restrict usage of psutil to
    versions <2.0.0 it can be good to be prepared for upgrade in advance.
    This commit makes use of multiprocessing module instead.

    Change-Id: I6a4653eb7e5c26c3068a4d766acb4bc219920f75
    Closes-Bug: #1598208
    (cherry picked from commit 8fb54da2f91a88286fcc1b444d26b387cd04d174)

This issue was fixed in the openstack/murano 2.0.2 release.

James Page (james-page) on 2016-11-10
Changed in murano (Ubuntu Yakkety):
status: New → Fix Released
Changed in murano (Ubuntu Zesty):
status: New → Fix Released
Changed in murano (Ubuntu Xenial):
status: New → Triaged
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers