← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1693923] [NEW] compute_monitors help incorrectly shows multiple values in a list

 

Public bug reported:

This issue was actually nicely written up here: https://iclykofte.com
/physical-host-metrics-in-ceilometer-an-openstack-liberty-tale/  ...but
apparently never made it into a bug/patch. If you supply more than one
driver  to compute_monitors in a list per the setting help, the list
will be nested inside another list (cfg_monitors).

Steps to reproduce:
1) set /etc/nova/nova.conf DEFAULT compute_monitors to ["cpu.virt_driver", "numa_mem_bw.virt_driver"]
2) in nova-manage shell, load the config, and populate the cfg_monitors list
3) cfg_monitors will = ['nova.compute.monitors.["cpu.virt_driver"', 'nova.compute.monitors."numa_mem_bw.virt_driver"]']

Expected result:
cfg_monitors = ['nova.compute.monitors.cpu.virt_driver', 'nova.compute.monitors.cpu.numa_mem_bw.virt_driver']

This can be achieved by setting compute_monitors = cpu.virt_driver,
numa_mem_bw.virt_driver

** Affects: nova
     Importance: Undecided
     Assignee: Matt Wisch (matt-wisch)
         Status: In Progress

** Changed in: nova
       Status: New => In Progress

** Changed in: nova
     Assignee: (unassigned) => Matt Wisch (matt-wisch)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1693923

Title:
  compute_monitors help incorrectly shows multiple values in a list

Status in OpenStack Compute (nova):
  In Progress

Bug description:
  This issue was actually nicely written up here: https://iclykofte.com
  /physical-host-metrics-in-ceilometer-an-openstack-liberty-tale/
  ...but apparently never made it into a bug/patch. If you supply more
  than one driver  to compute_monitors in a list per the setting help,
  the list will be nested inside another list (cfg_monitors).

  Steps to reproduce:
  1) set /etc/nova/nova.conf DEFAULT compute_monitors to ["cpu.virt_driver", "numa_mem_bw.virt_driver"]
  2) in nova-manage shell, load the config, and populate the cfg_monitors list
  3) cfg_monitors will = ['nova.compute.monitors.["cpu.virt_driver"', 'nova.compute.monitors."numa_mem_bw.virt_driver"]']

  Expected result:
  cfg_monitors = ['nova.compute.monitors.cpu.virt_driver', 'nova.compute.monitors.cpu.numa_mem_bw.virt_driver']

  This can be achieved by setting compute_monitors = cpu.virt_driver,
  numa_mem_bw.virt_driver

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1693923/+subscriptions


Follow ups