need to improve the snmp inspector by using bulkCmd

Bug #1286132 reported by Lianhao Lu
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Wishlist
Lianhao Lu

Bug Description

Just like eglynn suggested in his comment of https://review.openstack.org/#/c/43073/, as I quote here:

However, I would like to revisit the issue of fine-grained versus bulk SNMP commands at some point in the future. IIUC the interaction could be made significantly less chatty if the bulk approach was used. I take your point that this is not supported by older SNMP versions, but I wonder could we always first assume that the bulk mode is supported by each host, and only revert to fine-grained commands if the bulk version fails in a way that suggests that mode is not supported by the SNMP daemon running on that host (and cache that per-host status).

Please note that in older version of SNMP(version<2c), bulkCmd is not supported. So we need to take that into account too.

Julien Danjou (jdanjou)
Changed in ceilometer:
status: New → Triaged
Lianhao Lu (lianhao-lu)
Changed in ceilometer:
assignee: nobody → Lianhao Lu (lianhao-lu)
Changed in ceilometer:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/108938
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=e7046c44c01596abae9ef867cf0cd95a51919dac
Submitter: Jenkins
Branch: master

commit e7046c44c01596abae9ef867cf0cd95a51919dac
Author: Lianhao Lu <email address hidden>
Date: Fri Jul 25 13:19:32 2014 +0800

    Added new hardware inspector interface

    Added a new hardware inspector interface: inspect_generic and its snmp
    implementation. The old interface will be removed later.

    Partial implements: blueprint snmp-improvement.
    Partial-Bug: 1286132

    Change-Id: I847cfdad5b6571c4d066250617050a8f0b1cfbc4

Eoghan Glynn (eglynn)
Changed in ceilometer:
milestone: none → juno-3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/109207
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=e9e6c97f68c985e1de3d61e7cd91445ea5d8d159
Submitter: Jenkins
Branch: master

commit e9e6c97f68c985e1de3d61e7cd91445ea5d8d159
Author: Lianhao Lu <email address hidden>
Date: Thu Jul 24 15:55:11 2014 +0800

    Rebase hardware pollsters to use new inspector interface

    Change all the hardware pollsters to use new inspector interface
    inspect_generic(), and remove the old interfaces.

    Partial implements: blueprint snmp-improvement.
    Partial-Bug: 1286132

    Change-Id: Icc566ec8c4dbe1c0883c7e0d653480c5659f30b9

Lianhao Lu (lianhao-lu)
Changed in ceilometer:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: juno-3 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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