OpenStack Compute (nova)

Flavor property "hw_rng:rate_period" should be milliseconds

Bug #1843542 reported by Jordan Ansell
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Takashi Natsume
Pike
In Progress
Medium
Unassigned
Queens
Fix Committed
Medium
Takashi Natsume
Rocky
Fix Committed
Medium
Takashi Natsume
Stein
Fix Committed
Medium
Takashi Natsume
Train
Fix Committed
Medium
Takashi Natsume

Bug Description

Error in "doc/source/user/flavors.rst"

In "Random-number generator" section

Currently reads:
"RATE-PERIOD: (integer) Duration of the read period in seconds."

Should read:
"RATE-PERIOD: (integer) Duration of the read period in milliseconds."

Please see: https://libvirt.org/formatdomain.html#elementsRng for reference.

Either the documentation needs to be updated of the nova needs to convert the given value into milliseconds before passing it to libvirt.

Revision history for this message
Matt Riedemann (mriedem) wrote :

Seems that's been removed from the latest docs:

https://docs.openstack.org/nova/latest/user/flavors.html

tags: added: doc
Revision history for this message
Matt Riedemann (mriedem) wrote :

Oh nevermind it's there on master as well.

Revision history for this message
Matt Riedemann (mriedem) wrote :

Note the glance metadata for the image property says the same:

https://github.com/openstack/glance/blob/54329c6a21b0d3f845b09e79f710fc795976a175/etc/metadefs/compute-randomgen.json#L23

Revision history for this message
Matt Riedemann (mriedem) wrote :

Looking at the libvirt driver code it does not do any conversion from seconds to milliseconds so yeah the docs are wrong when looking at the libvirt docs. The flavor extra spec docs should also mention that is a libvirt-only spec at the moment, no other compute driver is using that spec.

tags: added: low-hanging-fruit
Changed in nova:
status: New → Confirmed
importance: Undecided → Medium
Takashi Natsume (natsume-takashi)
Changed in nova:
assignee: nobody → Takashi NATSUME (natsume-takashi)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.opendev.org/683051

Changed in nova:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.opendev.org/683051
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=976120cb20da46258df87b8fa43ba0f8ee6dc675
Submitter: Zuul
Branch: master

commit 976120cb20da46258df87b8fa43ba0f8ee6dc675
Author: Takashi NATSUME <email address hidden>
Date: Thu Sep 19 16:38:25 2019 +0900

    Fix unit of hw_rng:rate_period

    Replace seconds with milliseconds
    in the description of hw_rng:rate_period.

    * https://libvirt.org/formatdomain.html#elementsRng

    Change-Id: I285f68c81b0587c6a197d4791b09234a1697f5e7
    Closes-Bug: #1843542

Changed in nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/train)

Fix proposed to branch: stable/train
Review: https://review.opendev.org/688083

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/train)

Reviewed: https://review.opendev.org/688083
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=875826df959bf953609aa66d793793afa5ad618a
Submitter: Zuul
Branch: stable/train

commit 875826df959bf953609aa66d793793afa5ad618a
Author: Takashi NATSUME <email address hidden>
Date: Thu Sep 19 16:38:25 2019 +0900

    Fix unit of hw_rng:rate_period

    Replace seconds with milliseconds
    in the description of hw_rng:rate_period.

    * https://libvirt.org/formatdomain.html#elementsRng

    Change-Id: I285f68c81b0587c6a197d4791b09234a1697f5e7
    Closes-Bug: #1843542
    (cherry picked from commit 976120cb20da46258df87b8fa43ba0f8ee6dc675)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/stein)

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/689153

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/689154

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/689155

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/pike)

Fix proposed to branch: stable/pike
Review: https://review.opendev.org/689158

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/stein)

Reviewed: https://review.opendev.org/689153
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=792ec7c1df7fd5c191da6f2feb961ad43f81dfa4
Submitter: Zuul
Branch: stable/stein

commit 792ec7c1df7fd5c191da6f2feb961ad43f81dfa4
Author: Takashi NATSUME <email address hidden>
Date: Thu Sep 19 16:38:25 2019 +0900

    Fix unit of hw_rng:rate_period

    Replace seconds with milliseconds
    in the description of hw_rng:rate_period.

    * https://libvirt.org/formatdomain.html#elementsRng

    Change-Id: I285f68c81b0587c6a197d4791b09234a1697f5e7
    Closes-Bug: #1843542
    (cherry picked from commit 976120cb20da46258df87b8fa43ba0f8ee6dc675)
    (cherry picked from commit 875826df959bf953609aa66d793793afa5ad618a)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/rocky)

Reviewed: https://review.opendev.org/689154
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=ff57ea7fb6a15ff90ff33b0dc7a28426adc840ee
Submitter: Zuul
Branch: stable/rocky

commit ff57ea7fb6a15ff90ff33b0dc7a28426adc840ee
Author: Takashi NATSUME <email address hidden>
Date: Thu Sep 19 16:38:25 2019 +0900

    Fix unit of hw_rng:rate_period

    Replace seconds with milliseconds
    in the description of hw_rng:rate_period.

    * https://libvirt.org/formatdomain.html#elementsRng

    Change-Id: I285f68c81b0587c6a197d4791b09234a1697f5e7
    Closes-Bug: #1843542
    (cherry picked from commit 976120cb20da46258df87b8fa43ba0f8ee6dc675)
    (cherry picked from commit 875826df959bf953609aa66d793793afa5ad618a)
    (cherry picked from commit 792ec7c1df7fd5c191da6f2feb961ad43f81dfa4)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/queens)

Reviewed: https://review.opendev.org/689155
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=55a3796692406e916e17213c5d841281f18e6cc1
Submitter: Zuul
Branch: stable/queens

commit 55a3796692406e916e17213c5d841281f18e6cc1
Author: Takashi NATSUME <email address hidden>
Date: Thu Sep 19 16:38:25 2019 +0900

    Fix unit of hw_rng:rate_period

    Replace seconds with milliseconds
    in the description of hw_rng:rate_period.

    * https://libvirt.org/formatdomain.html#elementsRng

    Change-Id: I285f68c81b0587c6a197d4791b09234a1697f5e7
    Closes-Bug: #1843542
    (cherry picked from commit 976120cb20da46258df87b8fa43ba0f8ee6dc675)
    (cherry picked from commit 875826df959bf953609aa66d793793afa5ad618a)
    (cherry picked from commit 792ec7c1df7fd5c191da6f2feb961ad43f81dfa4)
    (cherry picked from commit ff57ea7fb6a15ff90ff33b0dc7a28426adc840ee)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 17.0.13

This issue was fixed in the openstack/nova 17.0.13 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 20.0.1

This issue was fixed in the openstack/nova 20.0.1 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 19.1.0

This issue was fixed in the openstack/nova 19.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 18.3.0

This issue was fixed in the openstack/nova 18.3.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova (master)

Reviewed: https://review.opendev.org/c/openstack/nova/+/781440
Committed: https://opendev.org/openstack/nova/commit/9dfac329594933864aceaeb3be28457afd13f6ca
Submitter: "Zuul (22348)"
Branch: master

commit 9dfac329594933864aceaeb3be28457afd13f6ca
Author: Stephen Finucane <email address hidden>
Date: Mon Feb 22 18:58:41 2021 +0000

    docs: Add libvirt misc doc

    A new dumping ground for libvirt'y things that can't easily be
    categorized or placed elsewhere.

    Change-Id: I6999b9d66e12e1df7970aff6ce63e5323de6be45
    Signed-off-by: Stephen Finucane <email address hidden>
    Related-Bug: #1843542

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.