)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2014-10-22 14:40:50 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Generic Notification Sensor Meter Plugin Design Spec"},{"line_number":8,"context_line":"for blue print generic-notification-sensor-meter-plugin"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I5a131e022e5d6f2d70521165f180b47d95f38d4e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9aa7fdbe_c3b19be2","line":8,"range":{"start_line":8,"start_character":4,"end_line":8,"end_character":14},"updated":"2014-10-23 08:23:49.000000000","message":"blueprint - one word","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2014-10-22 14:40:50 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Generic Notification Sensor Meter Plugin Design Spec"},{"line_number":8,"context_line":"for blue print generic-notification-sensor-meter-plugin"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I5a131e022e5d6f2d70521165f180b47d95f38d4e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9aa7fdbe_d1d6b2f0","line":8,"in_reply_to":"9aa7fdbe_c3b19be2","updated":"2014-10-23 17:38:47.000000000","message":"ok","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Generic Notification Sensor Meter Plugin Design Spec"},{"line_number":8,"context_line":"for blue print generic-notification-sensor-meter-plugin"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I5a131e022e5d6f2d70521165f180b47d95f38d4e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9aa7fdbe_03d2c3ff","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":6},"updated":"2014-10-23 08:23:49.000000000","message":"add Related-Blueprint or Implements-Blueprint hook here for generic-notification-sensor-meter-plugin","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Generic Notification Sensor Meter Plugin Design Spec"},{"line_number":8,"context_line":"for blue print generic-notification-sensor-meter-plugin"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I5a131e022e5d6f2d70521165f180b47d95f38d4e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9aa7fdbe_7160a601","line":10,"in_reply_to":"9aa7fdbe_03d2c3ff","updated":"2014-10-23 17:38:47.000000000","message":"Done","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"}],"specs/kilo/generic-notification-sensor-meter-plugin.rst":[{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":12,"context_line":"https://blueprints.launchpad.net/ceilometer/+spec/generic-notification-sensor-meter-plugin"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Provide support for accepting sensor meter notifications from platform"},{"line_number":15,"context_line":"providers in addition to IPMI.    Currently only IPMI based sensor meter"},{"line_number":16,"context_line":"notification events are supported in Ceilometer."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Problem description"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_8305d37b","line":15,"range":{"start_line":15,"start_character":31,"end_line":15,"end_character":34},"updated":"2014-10-23 08:23:49.000000000","message":"too much whitespaces","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"4dd66e0f6783d65ce8f68a191bc8f8d940522fe9","unresolved":false,"context_lines":[{"line_number":12,"context_line":"https://blueprints.launchpad.net/ceilometer/+spec/generic-notification-sensor-meter-plugin"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Provide support for accepting sensor meter notifications from platform"},{"line_number":15,"context_line":"providers in addition to IPMI.    Currently only IPMI based sensor meter"},{"line_number":16,"context_line":"notification events are supported in Ceilometer."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Problem description"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_2466d1c1","line":15,"in_reply_to":"9aa7fdbe_6d01ac37","updated":"2014-10-24 08:18:50.000000000","message":"80 symbols does not mean you need to have 80 symbols per line :) It means you need to have not more than 80 symbols :) That\u0027s why your multiple whitespaces are not needed actually.","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":12,"context_line":"https://blueprints.launchpad.net/ceilometer/+spec/generic-notification-sensor-meter-plugin"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Provide support for accepting sensor meter notifications from platform"},{"line_number":15,"context_line":"providers in addition to IPMI.    Currently only IPMI based sensor meter"},{"line_number":16,"context_line":"notification events are supported in Ceilometer."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Problem description"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_6d01ac37","line":15,"in_reply_to":"9aa7fdbe_8305d37b","updated":"2014-10-23 17:38:47.000000000","message":"I fixed formatting as best I could to reduce some of the  whitespace, but with a limit of 80 characters per line what you see in the next doc is the best I could do.","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":19,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"The current Ceilometer sensor meter notification plugin is implemented to"},{"line_number":22,"context_line":"support notifications for IPMI sensors sent from ironic.  Notification"},{"line_number":23,"context_line":"messages are sent from ironic to Ceilometer with IPMI specific event_type of"},{"line_number":24,"context_line":"\u0027hardware.ipmi.*\u0027 and IPMI sensors of type Temperature, Fan, Voltage,"},{"line_number":25,"context_line":"and Current are supported by the Ceilometer plugin."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_03f9e37f","line":22,"range":{"start_line":22,"start_character":49,"end_line":22,"end_character":55},"updated":"2014-10-23 08:23:49.000000000","message":"Ironic + too many whitespaces","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":19,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"The current Ceilometer sensor meter notification plugin is implemented to"},{"line_number":22,"context_line":"support notifications for IPMI sensors sent from ironic.  Notification"},{"line_number":23,"context_line":"messages are sent from ironic to Ceilometer with IPMI specific event_type of"},{"line_number":24,"context_line":"\u0027hardware.ipmi.*\u0027 and IPMI sensors of type Temperature, Fan, Voltage,"},{"line_number":25,"context_line":"and Current are supported by the Ceilometer plugin."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_ed9cbc11","line":22,"in_reply_to":"9aa7fdbe_03f9e37f","updated":"2014-10-23 17:38:47.000000000","message":"Done, all lower case ironic -\u003e Ironic","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"The current Ceilometer sensor meter notification plugin is implemented to"},{"line_number":22,"context_line":"support notifications for IPMI sensors sent from ironic.  Notification"},{"line_number":23,"context_line":"messages are sent from ironic to Ceilometer with IPMI specific event_type of"},{"line_number":24,"context_line":"\u0027hardware.ipmi.*\u0027 and IPMI sensors of type Temperature, Fan, Voltage,"},{"line_number":25,"context_line":"and Current are supported by the Ceilometer plugin."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_832cf303","line":23,"range":{"start_line":23,"start_character":23,"end_line":23,"end_character":29},"updated":"2014-10-23 08:23:49.000000000","message":"Ironic","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":25,"context_line":"and Current are supported by the Ceilometer plugin."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"I would like to provide support for non IPMI based sensor providers in the"},{"line_number":28,"context_line":"ironic conductor as well as enabling support new sensor types. This can be"},{"line_number":29,"context_line":"accomplished by changing sensor notifications from the ironic conductor"},{"line_number":30,"context_line":"to Ceilometer to be provider generic."},{"line_number":31,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_c336fb51","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":6},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"I would like to provide support for non IPMI based sensor providers in the"},{"line_number":28,"context_line":"ironic conductor as well as enabling support new sensor types. This can be"},{"line_number":29,"context_line":"accomplished by changing sensor notifications from the ironic conductor"},{"line_number":30,"context_line":"to Ceilometer to be provider generic."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"This change will enable support for the creation of Ceilometer meters"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_034003a4","line":29,"range":{"start_line":29,"start_character":55,"end_line":29,"end_character":61},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":36,"context_line":"Proposed change"},{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Support for multiple sensor provider notifications requires two changes to"},{"line_number":40,"context_line":"the way in which sensors meters are managed by Ceilometer.   The first change"},{"line_number":41,"context_line":"is to the way in which sensor meters are currently named and the second change"},{"line_number":42,"context_line":"is notification plugin support for generic sensor provider meter"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_631c6fc2","line":39,"range":{"start_line":39,"start_character":28,"end_line":39,"end_character":36},"updated":"2014-10-23 08:23:49.000000000","message":"providerS I guess","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":36,"context_line":"Proposed change"},{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Support for multiple sensor provider notifications requires two changes to"},{"line_number":40,"context_line":"the way in which sensors meters are managed by Ceilometer.   The first change"},{"line_number":41,"context_line":"is to the way in which sensor meters are currently named and the second change"},{"line_number":42,"context_line":"is notification plugin support for generic sensor provider meter"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_0dd020c8","line":39,"in_reply_to":"9aa7fdbe_631c6fc2","updated":"2014-10-23 17:38:47.000000000","message":"Done","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":37,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Support for multiple sensor provider notifications requires two changes to"},{"line_number":40,"context_line":"the way in which sensors meters are managed by Ceilometer.   The first change"},{"line_number":41,"context_line":"is to the way in which sensor meters are currently named and the second change"},{"line_number":42,"context_line":"is notification plugin support for generic sensor provider meter"},{"line_number":43,"context_line":"notification messages."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_e3877fd5","line":40,"range":{"start_line":40,"start_character":58,"end_line":40,"end_character":61},"updated":"2014-10-23 08:23:49.000000000","message":"too many whitespaces","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":52,"context_line":"+--------------------------+--------------------------+"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Note:"},{"line_number":55,"context_line":"  The SensorType comes from the notification metric name.  The SensorID comes"},{"line_number":56,"context_line":"  from a per sensor message payload field \u0027Sensor ID\u0027.   The IronicNodeUUID"},{"line_number":57,"context_line":"  comes from the message payload \u0027node_uuid\u0027 field."},{"line_number":58,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_036e432f","line":55,"range":{"start_line":55,"start_character":57,"end_line":55,"end_character":59},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"Note:"},{"line_number":55,"context_line":"  The SensorType comes from the notification metric name.  The SensorID comes"},{"line_number":56,"context_line":"  from a per sensor message payload field \u0027Sensor ID\u0027.   The IronicNodeUUID"},{"line_number":57,"context_line":"  comes from the message payload \u0027node_uuid\u0027 field."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"To support generic sensor providers,  I would like to propose making the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_63738f15","line":56,"range":{"start_line":56,"start_character":54,"end_line":56,"end_character":57},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":56,"context_line":"  from a per sensor message payload field \u0027Sensor ID\u0027.   The IronicNodeUUID"},{"line_number":57,"context_line":"  comes from the message payload \u0027node_uuid\u0027 field."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"To support generic sensor providers,  I would like to propose making the"},{"line_number":60,"context_line":"the following changes to the current Ceilometer sensor meter naming."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"* The \"ipmi\" will be removed from the Ceilometer Name and moved into a per"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_2350a75d","line":59,"range":{"start_line":59,"start_character":36,"end_line":59,"end_character":38},"updated":"2014-10-23 08:23:49.000000000","message":"ditto, making -\u003e contributing/committing will be nicer","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":56,"context_line":"  from a per sensor message payload field \u0027Sensor ID\u0027.   The IronicNodeUUID"},{"line_number":57,"context_line":"  comes from the message payload \u0027node_uuid\u0027 field."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"To support generic sensor providers,  I would like to propose making the"},{"line_number":60,"context_line":"the following changes to the current Ceilometer sensor meter naming."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"* The \"ipmi\" will be removed from the Ceilometer Name and moved into a per"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_6d536c6b","line":59,"in_reply_to":"9aa7fdbe_2350a75d","updated":"2014-10-23 17:38:47.000000000","message":"Done","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":57,"context_line":"  comes from the message payload \u0027node_uuid\u0027 field."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"To support generic sensor providers,  I would like to propose making the"},{"line_number":60,"context_line":"the following changes to the current Ceilometer sensor meter naming."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"* The \"ipmi\" will be removed from the Ceilometer Name and moved into a per"},{"line_number":63,"context_line":"  sensor message payload field \u0027sensor_provider\u0027.   Each provider of a"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_c3d27bd1","line":60,"range":{"start_line":60,"start_character":67,"end_line":60,"end_character":68},"updated":"2014-10-23 08:23:49.000000000","message":". -\u003e :","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":57,"context_line":"  comes from the message payload \u0027node_uuid\u0027 field."},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"To support generic sensor providers,  I would like to propose making the"},{"line_number":60,"context_line":"the following changes to the current Ceilometer sensor meter naming."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"* The \"ipmi\" will be removed from the Ceilometer Name and moved into a per"},{"line_number":63,"context_line":"  sensor message payload field \u0027sensor_provider\u0027.   Each provider of a"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_b104ce18","line":60,"in_reply_to":"9aa7fdbe_c3d27bd1","updated":"2014-10-23 17:38:47.000000000","message":"Done","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":59,"context_line":"To support generic sensor providers,  I would like to propose making the"},{"line_number":60,"context_line":"the following changes to the current Ceilometer sensor meter naming."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"* The \"ipmi\" will be removed from the Ceilometer Name and moved into a per"},{"line_number":63,"context_line":"  sensor message payload field \u0027sensor_provider\u0027.   Each provider of a"},{"line_number":64,"context_line":"  sensor record will be required to create this field so that a sensor"},{"line_number":65,"context_line":"  record consumer can distinguish the provider which generated the sensor."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_0303a342","line":62,"range":{"start_line":62,"start_character":6,"end_line":62,"end_character":12},"updated":"2014-10-23 08:23:49.000000000","message":"\u0027the \"ipmi\" prefix\u0027 will look better imho","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":59,"context_line":"To support generic sensor providers,  I would like to propose making the"},{"line_number":60,"context_line":"the following changes to the current Ceilometer sensor meter naming."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"* The \"ipmi\" will be removed from the Ceilometer Name and moved into a per"},{"line_number":63,"context_line":"  sensor message payload field \u0027sensor_provider\u0027.   Each provider of a"},{"line_number":64,"context_line":"  sensor record will be required to create this field so that a sensor"},{"line_number":65,"context_line":"  record consumer can distinguish the provider which generated the sensor."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_31f29e06","line":62,"in_reply_to":"9aa7fdbe_0303a342","updated":"2014-10-23 17:38:47.000000000","message":"The removal of the provider from the Ceilometer name will permit users to address sensors via generic names if users don\u0027t care what driver actually provided the sensor.   Putting the \u0027sensor_provider\u0027 in the resource_metadata will enable users that want driver specific information from the resource_metadata to easily be able to determine if the information they want is available.","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":60,"context_line":"the following changes to the current Ceilometer sensor meter naming."},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"* The \"ipmi\" will be removed from the Ceilometer Name and moved into a per"},{"line_number":63,"context_line":"  sensor message payload field \u0027sensor_provider\u0027.   Each provider of a"},{"line_number":64,"context_line":"  sensor record will be required to create this field so that a sensor"},{"line_number":65,"context_line":"  record consumer can distinguish the provider which generated the sensor."},{"line_number":66,"context_line":"  Note: all fields in the per sensor message payload are placed into the meter"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_63e1cf99","line":63,"range":{"start_line":63,"start_character":49,"end_line":63,"end_character":52},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":66,"context_line":"  Note: all fields in the per sensor message payload are placed into the meter"},{"line_number":67,"context_line":"  resource_metadata."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"* The Resource ID will be changed to contain only the IronicNodeUUID.  This"},{"line_number":70,"context_line":"  will enable quick access to all the sensors for a given Ironic node using"},{"line_number":71,"context_line":"  the Ceilometer resource list API."},{"line_number":72,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_43fdab39","line":69,"range":{"start_line":69,"start_character":69,"end_line":69,"end_character":71},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":70,"context_line":"  will enable quick access to all the sensors for a given Ironic node using"},{"line_number":71,"context_line":"  the Ceilometer resource list API."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"* The SensorID will be appended to the Ceilometer name as \".SensorID\".   This"},{"line_number":74,"context_line":"  will provide a unique name per sensor of a given type."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"The resulting naming for sensor meters will be as follows based on the above"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_a399f720","line":73,"range":{"start_line":73,"start_character":70,"end_line":73,"end_character":73},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":89,"context_line":"  http://lists.openstack.org/pipermail/openstack-dev/2014-October/048631.html"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"I would like to propose that a new generic notification plugin be developed in"},{"line_number":93,"context_line":"Ceilometer to create a meter/sample for every sensor payload received with"},{"line_number":94,"context_line":"event_type \u0027hardware.sensors\u0027.   The message payload will contain fields"},{"line_number":95,"context_line":"\u0027event_type\u0027, \u0027ironic_node_uuid\u0027, \u0027project_id\u0027, \u0027timestamp\u0027 and \u0027user_id\u0027"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_e3a3ff4c","line":92,"range":{"start_line":92,"start_character":62,"end_line":92,"end_character":63},"updated":"2014-10-23 08:23:49.000000000","message":"*should* be developed","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":89,"context_line":"  http://lists.openstack.org/pipermail/openstack-dev/2014-October/048631.html"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"I would like to propose that a new generic notification plugin be developed in"},{"line_number":93,"context_line":"Ceilometer to create a meter/sample for every sensor payload received with"},{"line_number":94,"context_line":"event_type \u0027hardware.sensors\u0027.   The message payload will contain fields"},{"line_number":95,"context_line":"\u0027event_type\u0027, \u0027ironic_node_uuid\u0027, \u0027project_id\u0027, \u0027timestamp\u0027 and \u0027user_id\u0027"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_ed675c4b","line":92,"in_reply_to":"9aa7fdbe_e3a3ff4c","updated":"2014-10-23 17:38:47.000000000","message":"Done","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":91,"context_line":""},{"line_number":92,"context_line":"I would like to propose that a new generic notification plugin be developed in"},{"line_number":93,"context_line":"Ceilometer to create a meter/sample for every sensor payload received with"},{"line_number":94,"context_line":"event_type \u0027hardware.sensors\u0027.   The message payload will contain fields"},{"line_number":95,"context_line":"\u0027event_type\u0027, \u0027ironic_node_uuid\u0027, \u0027project_id\u0027, \u0027timestamp\u0027 and \u0027user_id\u0027"},{"line_number":96,"context_line":"along with a list of sensors \u0027sensors\u0027 where each sensor in the list"},{"line_number":97,"context_line":"must contain the following fields:"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_038ac3be","line":94,"range":{"start_line":94,"start_character":30,"end_line":94,"end_character":33},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":101,"context_line":"* \u0027sensor_reading\u0027 - Ceilometer Sample volume value"},{"line_number":102,"context_line":"* \u0027sensor_unit\u0027 - Ceilometer Sample unit string"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"I am proposing that the fully qualified sensor name be generated by the"},{"line_number":105,"context_line":"ironic conductor instead of by the Ceilometer notification plugin so that"},{"line_number":106,"context_line":"consistency in naming of sensors can be provided when other metering system"},{"line_number":107,"context_line":"(like Monasca) need to be supported by the ironic conductor."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_4394cbe1","line":104,"range":{"start_line":104,"start_character":52,"end_line":104,"end_character":54},"updated":"2014-10-23 08:23:49.000000000","message":"will be","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":101,"context_line":"* \u0027sensor_reading\u0027 - Ceilometer Sample volume value"},{"line_number":102,"context_line":"* \u0027sensor_unit\u0027 - Ceilometer Sample unit string"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"I am proposing that the fully qualified sensor name be generated by the"},{"line_number":105,"context_line":"ironic conductor instead of by the Ceilometer notification plugin so that"},{"line_number":106,"context_line":"consistency in naming of sensors can be provided when other metering system"},{"line_number":107,"context_line":"(like Monasca) need to be supported by the ironic conductor."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_d4186077","line":104,"in_reply_to":"9aa7fdbe_4394cbe1","updated":"2014-10-23 17:38:47.000000000","message":"Done","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":102,"context_line":"* \u0027sensor_unit\u0027 - Ceilometer Sample unit string"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"I am proposing that the fully qualified sensor name be generated by the"},{"line_number":105,"context_line":"ironic conductor instead of by the Ceilometer notification plugin so that"},{"line_number":106,"context_line":"consistency in naming of sensors can be provided when other metering system"},{"line_number":107,"context_line":"(like Monasca) need to be supported by the ironic conductor."},{"line_number":108,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_638f0fcd","line":105,"range":{"start_line":105,"start_character":0,"end_line":105,"end_character":6},"updated":"2014-10-23 08:23:49.000000000","message":"Ironic","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":102,"context_line":"* \u0027sensor_unit\u0027 - Ceilometer Sample unit string"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"I am proposing that the fully qualified sensor name be generated by the"},{"line_number":105,"context_line":"ironic conductor instead of by the Ceilometer notification plugin so that"},{"line_number":106,"context_line":"consistency in naming of sensors can be provided when other metering system"},{"line_number":107,"context_line":"(like Monasca) need to be supported by the ironic conductor."},{"line_number":108,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_f4280488","line":105,"in_reply_to":"9aa7fdbe_638f0fcd","updated":"2014-10-23 17:38:47.000000000","message":"Done","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":104,"context_line":"I am proposing that the fully qualified sensor name be generated by the"},{"line_number":105,"context_line":"ironic conductor instead of by the Ceilometer notification plugin so that"},{"line_number":106,"context_line":"consistency in naming of sensors can be provided when other metering system"},{"line_number":107,"context_line":"(like Monasca) need to be supported by the ironic conductor."},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"The generic Ceilometer plugin will only require one notification event type,"},{"line_number":110,"context_line":"\u0027hardware.sensors\u0027, instead of a notification event type per SensorType."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_03b1e38e","line":107,"range":{"start_line":107,"start_character":14,"end_line":107,"end_character":15},"updated":"2014-10-23 08:23:49.000000000","message":"will need ? + ironic -\u003e Ironic","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":104,"context_line":"I am proposing that the fully qualified sensor name be generated by the"},{"line_number":105,"context_line":"ironic conductor instead of by the Ceilometer notification plugin so that"},{"line_number":106,"context_line":"consistency in naming of sensors can be provided when other metering system"},{"line_number":107,"context_line":"(like Monasca) need to be supported by the ironic conductor."},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"The generic Ceilometer plugin will only require one notification event type,"},{"line_number":110,"context_line":"\u0027hardware.sensors\u0027, instead of a notification event type per SensorType."}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_717ee61a","line":107,"in_reply_to":"9aa7fdbe_03b1e38e","updated":"2014-10-23 17:38:47.000000000","message":"Done","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":108,"context_line":""},{"line_number":109,"context_line":"The generic Ceilometer plugin will only require one notification event type,"},{"line_number":110,"context_line":"\u0027hardware.sensors\u0027, instead of a notification event type per SensorType."},{"line_number":111,"context_line":"Use of a single notification event along with ironic conductor meter name"},{"line_number":112,"context_line":"generation will alleviate the need to change the Ceilometer plugin whenever"},{"line_number":113,"context_line":"a new SensorType needs to be supported."},{"line_number":114,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_63a62f55","line":111,"range":{"start_line":111,"start_character":46,"end_line":111,"end_character":52},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":116,"context_line":"------------"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"There were alternative naming discussions in the email thread I linked to"},{"line_number":119,"context_line":"above.    One possible alternative discussed was to keep the provider in the"},{"line_number":120,"context_line":"Ceilometer name string."},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"An alternative plugin design would be along the lines of what"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_c36efb00","line":119,"range":{"start_line":119,"start_character":6,"end_line":119,"end_character":10},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":120,"context_line":"Ceilometer name string."},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"An alternative plugin design would be along the lines of what"},{"line_number":123,"context_line":"currently exists for IPMI notifications.   This will certainly work, but"},{"line_number":124,"context_line":"requires changes whenever a new sensor type needs to be supported."},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"Data model impact"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_e3713fe1","line":123,"range":{"start_line":123,"start_character":40,"end_line":123,"end_character":43},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":163,"context_line":"----------------"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"This plugin should be able to be developed and deployed independently"},{"line_number":166,"context_line":"of the ironic conductor changes as long as the current Ceilometer IPMI"},{"line_number":167,"context_line":"notification plugin is still available."},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_43820b95","line":166,"range":{"start_line":166,"start_character":7,"end_line":166,"end_character":13},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":193,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":194,"context_line":""},{"line_number":195,"context_line":"Through the creation of a generic sensor plugin, changes to Ceilometer in order"},{"line_number":196,"context_line":"to support new sensor types will not be required.    At some point, the"},{"line_number":197,"context_line":"existing IPMI sensor notification plugin can be deprecated."},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_834b1386","line":196,"range":{"start_line":196,"start_character":49,"end_line":196,"end_character":53},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":209,"context_line":"Testing"},{"line_number":210,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"Unit tests for the new plugin and naming will need to be developed.   I will be"},{"line_number":213,"context_line":"doing point testing with ProLiant platforms to verify correct operation."},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"Documentation Impact"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_43304b01","line":212,"range":{"start_line":212,"start_character":67,"end_line":212,"end_character":70},"updated":"2014-10-23 08:23:49.000000000","message":"ditto","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"6d4ebec10ddd93cb5bb61a51ce87364e9253c5dd","unresolved":false,"context_lines":[{"line_number":217,"context_line":""},{"line_number":218,"context_line":"I\u0027m not sure what Ceilometer documentation would need to be updated based on"},{"line_number":219,"context_line":"what I have proposed here and I would appreciate some insight on what needs to"},{"line_number":220,"context_line":"be done."},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"References"},{"line_number":223,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_e33f7fd4","line":220,"updated":"2014-10-23 08:23:49.000000000","message":"I won\u0027t agree here, actually. There is section \u0027Ironic Hardware IPMI Sensor Data\u0027 in the measurements.rst page, I would suggest to change this part of documentation to present generic sensor concept","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"94a94e002bd3806a15dffaa7c49b50c3ffdc2891","unresolved":false,"context_lines":[{"line_number":217,"context_line":""},{"line_number":218,"context_line":"I\u0027m not sure what Ceilometer documentation would need to be updated based on"},{"line_number":219,"context_line":"what I have proposed here and I would appreciate some insight on what needs to"},{"line_number":220,"context_line":"be done."},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"References"},{"line_number":223,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa7fdbe_f12e5642","line":220,"in_reply_to":"9aa7fdbe_e33f7fd4","updated":"2014-10-23 17:38:47.000000000","message":"I added the need to update measurements.rst to this section.\nThanks for the info","commit_id":"0a1f4f920edea8e6701765f9558d997052ebf4ed"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"a0b9d4c31386b9d23a17845a5a2641974864e626","unresolved":false,"context_lines":[{"line_number":41,"context_line":"plugin support for generic sensor provider meter notification messages."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"The current names for sensor meters in Juno are generated by the Ceilometer IPMI"},{"line_number":44,"context_line":"notification plugin and are formatted as show in the following table:"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"+--------------------------+--------------------------+"},{"line_number":47,"context_line":"| Name                     | Resource ID              |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_a6316c0d","line":44,"updated":"2015-03-04 19:55:34.000000000","message":"s/as show/as shown/","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":41,"context_line":"plugin support for generic sensor provider meter notification messages."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"The current names for sensor meters in Juno are generated by the Ceilometer IPMI"},{"line_number":44,"context_line":"notification plugin and are formatted as show in the following table:"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"+--------------------------+--------------------------+"},{"line_number":47,"context_line":"| Name                     | Resource ID              |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_54c26885","line":44,"in_reply_to":"9a80dd14_a6316c0d","updated":"2015-03-10 17:38:28.000000000","message":"Done","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"a0b9d4c31386b9d23a17845a5a2641974864e626","unresolved":false,"context_lines":[{"line_number":49,"context_line":"| hardware.ipmi.SensorType | IronicNodeUUID-SensorID  |"},{"line_number":50,"context_line":"+--------------------------+--------------------------+"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Note:"},{"line_number":53,"context_line":"  The SensorType comes from the notification metric name.  The SensorID comes"},{"line_number":54,"context_line":"  from a per sensor message payload field \u0027Sensor ID\u0027.   The IronicNodeUUID"},{"line_number":55,"context_line":"  comes from the message payload \u0027node_uuid\u0027 field."}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_899c796b","line":52,"updated":"2015-03-04 19:55:34.000000000","message":"i think you need double colon \u0027::\u0027 and a new line to gen proper doc","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":49,"context_line":"| hardware.ipmi.SensorType | IronicNodeUUID-SensorID  |"},{"line_number":50,"context_line":"+--------------------------+--------------------------+"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Note:"},{"line_number":53,"context_line":"  The SensorType comes from the notification metric name.  The SensorID comes"},{"line_number":54,"context_line":"  from a per sensor message payload field \u0027Sensor ID\u0027.   The IronicNodeUUID"},{"line_number":55,"context_line":"  comes from the message payload \u0027node_uuid\u0027 field."}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_f4251c5c","line":52,"in_reply_to":"9a80dd14_899c796b","updated":"2015-03-10 17:38:28.000000000","message":"Done","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"3afb922f2abc44b00d15c802e5778885665b4ec9","unresolved":false,"context_lines":[{"line_number":50,"context_line":"+--------------------------+--------------------------+"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Note:"},{"line_number":53,"context_line":"  The SensorType comes from the notification metric name.  The SensorID comes"},{"line_number":54,"context_line":"  from a per sensor message payload field \u0027Sensor ID\u0027.   The IronicNodeUUID"},{"line_number":55,"context_line":"  comes from the message payload \u0027node_uuid\u0027 field."},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_9ede493e","line":53,"updated":"2015-03-05 12:34:41.000000000","message":"By notification metric name you mean the name in the notification that Ironic emits?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":50,"context_line":"+--------------------------+--------------------------+"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"Note:"},{"line_number":53,"context_line":"  The SensorType comes from the notification metric name.  The SensorID comes"},{"line_number":54,"context_line":"  from a per sensor message payload field \u0027Sensor ID\u0027.   The IronicNodeUUID"},{"line_number":55,"context_line":"  comes from the message payload \u0027node_uuid\u0027 field."},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_94eb00db","line":53,"in_reply_to":"9a80dd14_9ede493e","updated":"2015-03-10 17:38:28.000000000","message":"Yes, I change the text to indicate it is the notification from Ironic.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"3afb922f2abc44b00d15c802e5778885665b4ec9","unresolved":false,"context_lines":[{"line_number":61,"context_line":"  per sensor message payload field \u0027sensor_provider\u0027.   Each provider of a"},{"line_number":62,"context_line":"  sensor record will be required to create this field so that a sensor"},{"line_number":63,"context_line":"  record consumer can distinguish the provider which generated the sensor."},{"line_number":64,"context_line":"  Note: all fields in the per sensor message payload are placed into the meter"},{"line_number":65,"context_line":"  resource_metadata."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"I would like to propose that a generic notification plugin should be developed"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_1808ac2c","line":64,"updated":"2015-03-05 12:34:41.000000000","message":"Is there any material that can show a sample payload?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"0a25303476f1a80376f432d3e86a4bb230af06d9","unresolved":false,"context_lines":[{"line_number":61,"context_line":"  per sensor message payload field \u0027sensor_provider\u0027.   Each provider of a"},{"line_number":62,"context_line":"  sensor record will be required to create this field so that a sensor"},{"line_number":63,"context_line":"  record consumer can distinguish the provider which generated the sensor."},{"line_number":64,"context_line":"  Note: all fields in the per sensor message payload are placed into the meter"},{"line_number":65,"context_line":"  resource_metadata."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"I would like to propose that a generic notification plugin should be developed"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa963961_ff1c1879","line":64,"in_reply_to":"1a6ced46_6c2d0b95","updated":"2015-03-26 17:09:08.000000000","message":"I just added sample payload to the doc and should have an update doc posted sometime today.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":61,"context_line":"  per sensor message payload field \u0027sensor_provider\u0027.   Each provider of a"},{"line_number":62,"context_line":"  sensor record will be required to create this field so that a sensor"},{"line_number":63,"context_line":"  record consumer can distinguish the provider which generated the sensor."},{"line_number":64,"context_line":"  Note: all fields in the per sensor message payload are placed into the meter"},{"line_number":65,"context_line":"  resource_metadata."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"I would like to propose that a generic notification plugin should be developed"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_a42a5b2a","line":64,"in_reply_to":"9a80dd14_1808ac2c","updated":"2015-03-10 17:38:28.000000000","message":"I can put a sample payload in this section of you think it would be helpful.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"b8388c4afba9f9eb927ec2ab7573911e9a5a64c9","unresolved":false,"context_lines":[{"line_number":61,"context_line":"  per sensor message payload field \u0027sensor_provider\u0027.   Each provider of a"},{"line_number":62,"context_line":"  sensor record will be required to create this field so that a sensor"},{"line_number":63,"context_line":"  record consumer can distinguish the provider which generated the sensor."},{"line_number":64,"context_line":"  Note: all fields in the per sensor message payload are placed into the meter"},{"line_number":65,"context_line":"  resource_metadata."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"I would like to propose that a generic notification plugin should be developed"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6ced46_6c2d0b95","line":64,"in_reply_to":"9a80dd14_a42a5b2a","updated":"2015-03-26 05:29:11.000000000","message":"Yes, please provide a sample of a payload and the resulting ceilometer sample. Thank would be helpful.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"b8388c4afba9f9eb927ec2ab7573911e9a5a64c9","unresolved":false,"context_lines":[{"line_number":62,"context_line":"  sensor record will be required to create this field so that a sensor"},{"line_number":63,"context_line":"  record consumer can distinguish the provider which generated the sensor."},{"line_number":64,"context_line":"  Note: all fields in the per sensor message payload are placed into the meter"},{"line_number":65,"context_line":"  resource_metadata."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"I would like to propose that a generic notification plugin should be developed"},{"line_number":68,"context_line":"in Ceilometer to create a meter/sample for every sensor payload received with"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1a6ced46_4c4f6f6b","line":65,"updated":"2015-03-26 05:29:11.000000000","message":"I\u0027m thinking of the gnocchi integration. Does this change means that we\u0027ll have the samples with the same meter_name/resource_id but with different metadata? Can gnocchi support this?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"0a25303476f1a80376f432d3e86a4bb230af06d9","unresolved":false,"context_lines":[{"line_number":62,"context_line":"  sensor record will be required to create this field so that a sensor"},{"line_number":63,"context_line":"  record consumer can distinguish the provider which generated the sensor."},{"line_number":64,"context_line":"  Note: all fields in the per sensor message payload are placed into the meter"},{"line_number":65,"context_line":"  resource_metadata."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"I would like to propose that a generic notification plugin should be developed"},{"line_number":68,"context_line":"in Ceilometer to create a meter/sample for every sensor payload received with"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa963961_6293cbfb","line":65,"in_reply_to":"1a6ced46_4c4f6f6b","updated":"2015-03-26 17:09:08.000000000","message":"With the current implementation, and with my changes as well,  it is possible to have sensor samples with the same meter_name/resource_id and different metadata.     i.e. A sensor sample may contain platform modifiable values like Temperature Thresholds.  I ran across this fact when I initially started working with sensor data.  (See: https://bugs.launchpad.net/bugs/1348651) \n\nEven though the meta data is referred to as \"resource_metadata\", it is provided when creating each sample, so if the provider of sample decides to change metadata, you end up with meter_name/resource_id samples with different metadata.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"3afb922f2abc44b00d15c802e5778885665b4ec9","unresolved":false,"context_lines":[{"line_number":64,"context_line":"  Note: all fields in the per sensor message payload are placed into the meter"},{"line_number":65,"context_line":"  resource_metadata."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"I would like to propose that a generic notification plugin should be developed"},{"line_number":68,"context_line":"in Ceilometer to create a meter/sample for every sensor payload received with"},{"line_number":69,"context_line":"event_type \u0027hardware.sensors\u0027.   The message payload will contain fields"},{"line_number":70,"context_line":"\u0027event_type\u0027, \u0027ironic_node_uuid\u0027, \u0027project_id\u0027, \u0027timestamp\u0027 and \u0027user_id\u0027 along"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_9ea00945","line":67,"updated":"2015-03-05 12:34:41.000000000","message":"This plugin would replace the currently existing one, right?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":64,"context_line":"  Note: all fields in the per sensor message payload are placed into the meter"},{"line_number":65,"context_line":"  resource_metadata."},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"I would like to propose that a generic notification plugin should be developed"},{"line_number":68,"context_line":"in Ceilometer to create a meter/sample for every sensor payload received with"},{"line_number":69,"context_line":"event_type \u0027hardware.sensors\u0027.   The message payload will contain fields"},{"line_number":70,"context_line":"\u0027event_type\u0027, \u0027ironic_node_uuid\u0027, \u0027project_id\u0027, \u0027timestamp\u0027 and \u0027user_id\u0027 along"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_749e8c64","line":67,"in_reply_to":"9a80dd14_9ea00945","updated":"2015-03-10 17:38:28.000000000","message":"Yes, This plugin would replace the current one.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"3afb922f2abc44b00d15c802e5778885665b4ec9","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"I am proposing that the fully qualified sensor name be generated by the Ironic"},{"line_number":80,"context_line":"conductor instead of by the Ceilometer notification plugin so that consistency"},{"line_number":81,"context_line":"in naming of sensors can be provided when other metering systems (like Monasca)"},{"line_number":82,"context_line":"are supported by the Ironic conductor."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"The generic Ceilometer plugin will only require one notification event type,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_bed5cdba","line":81,"updated":"2015-03-05 12:34:41.000000000","message":"Is something like this already supported in Monasca or are there plans for it?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"bf05b6236ac268d78a6b46271e2039ee4d0f7b82","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"I am proposing that the fully qualified sensor name be generated by the Ironic"},{"line_number":80,"context_line":"conductor instead of by the Ceilometer notification plugin so that consistency"},{"line_number":81,"context_line":"in naming of sensors can be provided when other metering systems (like Monasca)"},{"line_number":82,"context_line":"are supported by the Ironic conductor."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"The generic Ceilometer plugin will only require one notification event type,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_7ff2aa07","line":81,"in_reply_to":"9a80dd14_944300ae","updated":"2015-03-10 17:47:01.000000000","message":"Apart from all this other stuff, we should endeavor to fix this difference. Can you find me on irc sometime (cdent) or on the os-dev mailing list so we can noodle about it?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":78,"context_line":""},{"line_number":79,"context_line":"I am proposing that the fully qualified sensor name be generated by the Ironic"},{"line_number":80,"context_line":"conductor instead of by the Ceilometer notification plugin so that consistency"},{"line_number":81,"context_line":"in naming of sensors can be provided when other metering systems (like Monasca)"},{"line_number":82,"context_line":"are supported by the Ironic conductor."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"The generic Ceilometer plugin will only require one notification event type,"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_944300ae","line":81,"in_reply_to":"9a80dd14_bed5cdba","updated":"2015-03-10 17:38:28.000000000","message":"Yes, Metrics have names in Monasca.   FYI: The acceptable characters that can be used to generate Metric names differs between Monasca and Ceilometer.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"a0b9d4c31386b9d23a17845a5a2641974864e626","unresolved":false,"context_lines":[{"line_number":85,"context_line":"\u0027hardware.sensors\u0027, instead of a notification event type per SensorType.  Use"},{"line_number":86,"context_line":"of a single notification event in conjunction with having the Ironic conductor"},{"line_number":87,"context_line":"generate the meter name will alleviate the need to change the Ceilometer plugin"},{"line_number":88,"context_line":"whenever a new SensorType needs to be supported."},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"Alternatives"},{"line_number":91,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_49ce015c","line":88,"updated":"2015-03-04 19:55:34.000000000","message":"is there metadata? i\u0027m assuming no?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":85,"context_line":"\u0027hardware.sensors\u0027, instead of a notification event type per SensorType.  Use"},{"line_number":86,"context_line":"of a single notification event in conjunction with having the Ironic conductor"},{"line_number":87,"context_line":"generate the meter name will alleviate the need to change the Ceilometer plugin"},{"line_number":88,"context_line":"whenever a new SensorType needs to be supported."},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"Alternatives"},{"line_number":91,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_14471078","line":88,"in_reply_to":"9a80dd14_49ce015c","updated":"2015-03-10 17:38:28.000000000","message":"There is still the metadata.   The metadata is the sensor as sent by ironic sans the fields which are are part of the actual sample (meter_name, meter_id, sensor_unit, sensor_reading).    I saw no reason to replicate this information in the metadata.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"a0b9d4c31386b9d23a17845a5a2641974864e626","unresolved":false,"context_lines":[{"line_number":90,"context_line":"Alternatives"},{"line_number":91,"context_line":"------------"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"There were alternative naming discussions in the email thread I linked to above."},{"line_number":94,"context_line":"One possible alternative discussed was to keep the provider in the Ceilometer"},{"line_number":95,"context_line":"name string."},{"line_number":96,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_e90c1597","line":93,"updated":"2015-03-04 19:55:34.000000000","message":"i apologise if i\u0027m being blind/stupid but what email thread?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":90,"context_line":"Alternatives"},{"line_number":91,"context_line":"------------"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"There were alternative naming discussions in the email thread I linked to above."},{"line_number":94,"context_line":"One possible alternative discussed was to keep the provider in the Ceilometer"},{"line_number":95,"context_line":"name string."},{"line_number":96,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_b7a486ba","line":93,"in_reply_to":"9a80dd14_e90c1597","updated":"2015-03-10 17:38:28.000000000","message":"Sorry,  I removed the section that had the link.  I updated and added the link back in.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"3afb922f2abc44b00d15c802e5778885665b4ec9","unresolved":false,"context_lines":[{"line_number":146,"context_line":"\"hardware.ipmi.SensorType\" will no longer be updated, but these meters will"},{"line_number":147,"context_line":"still exist in ceilomter."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"There is no plan to change the existing IPMI pollster meter naming.    The IPMI"},{"line_number":150,"context_line":"pollster meters will continue to have \"ipmi\" in their names."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_183b4c39","line":149,"updated":"2015-03-05 12:34:41.000000000","message":"Hmm. The sentence in the \u0027end user impact\u0027 section made me think that it will be changed. Could you please clarify this?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":146,"context_line":"\"hardware.ipmi.SensorType\" will no longer be updated, but these meters will"},{"line_number":147,"context_line":"still exist in ceilomter."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"There is no plan to change the existing IPMI pollster meter naming.    The IPMI"},{"line_number":150,"context_line":"pollster meters will continue to have \"ipmi\" in their names."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_779e8e64","line":149,"in_reply_to":"9a80dd14_183b4c39","updated":"2015-03-10 17:38:28.000000000","message":"Moved to end user impact section","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"a0b9d4c31386b9d23a17845a5a2641974864e626","unresolved":false,"context_lines":[{"line_number":147,"context_line":"still exist in ceilomter."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"There is no plan to change the existing IPMI pollster meter naming.    The IPMI"},{"line_number":150,"context_line":"pollster meters will continue to have \"ipmi\" in their names."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Implementation"},{"line_number":153,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_29546d53","line":150,"updated":"2015-03-04 19:55:34.000000000","message":"this should probably be in end user impact section","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":147,"context_line":"still exist in ceilomter."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"There is no plan to change the existing IPMI pollster meter naming.    The IPMI"},{"line_number":150,"context_line":"pollster meters will continue to have \"ipmi\" in their names."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Implementation"},{"line_number":153,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_97706241","line":150,"in_reply_to":"9a80dd14_29546d53","updated":"2015-03-10 17:38:28.000000000","message":"Moved to end user impact section","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"3afb922f2abc44b00d15c802e5778885665b4ec9","unresolved":false,"context_lines":[{"line_number":192,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"Unit tests for the new plugin and naming will need to be developed.   I will be"},{"line_number":195,"context_line":"doing point testing with ProLiant platforms to verify correct operation."},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"Documentation Impact"},{"line_number":198,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_f87758d2","line":195,"updated":"2015-03-05 12:34:41.000000000","message":"Is there any chance to test this in the OpenStack CI?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":192,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":193,"context_line":""},{"line_number":194,"context_line":"Unit tests for the new plugin and naming will need to be developed.   I will be"},{"line_number":195,"context_line":"doing point testing with ProLiant platforms to verify correct operation."},{"line_number":196,"context_line":""},{"line_number":197,"context_line":"Documentation Impact"},{"line_number":198,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_77c3ee37","line":195,"in_reply_to":"9a80dd14_f87758d2","updated":"2015-03-10 17:38:28.000000000","message":"I think it would be good to test this in the OpenStack CI, but I don\u0027t know what it would take to do this.     Any information on how to test patches using the the OpenStack CI would be helpful.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"a0b9d4c31386b9d23a17845a5a2641974864e626","unresolved":false,"context_lines":[{"line_number":197,"context_line":"Documentation Impact"},{"line_number":198,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"The \"Ironic Hardware IPMI Sensor Data\" section in doc/source/measurements.rst"},{"line_number":201,"context_line":"will need to be updated/changed to reflect the new Ceilometer naming."},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"References"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_09740998","line":200,"updated":"2015-03-04 19:55:34.000000000","message":"measurements have moved... http://docs.openstack.org/admin-guide-cloud/content/section_telemetry-measurements.html","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":197,"context_line":"Documentation Impact"},{"line_number":198,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"The \"Ironic Hardware IPMI Sensor Data\" section in doc/source/measurements.rst"},{"line_number":201,"context_line":"will need to be updated/changed to reflect the new Ceilometer naming."},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"References"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_494e1898","line":200,"in_reply_to":"9a80dd14_09740998","updated":"2015-03-10 17:38:28.000000000","message":"Ok I\u0027\u0027ll need to update the appropriate parts in this section of the admin guide.   Where does the source code reside for  this section in the admin-guide?","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"3afb922f2abc44b00d15c802e5778885665b4ec9","unresolved":false,"context_lines":[{"line_number":198,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"The \"Ironic Hardware IPMI Sensor Data\" section in doc/source/measurements.rst"},{"line_number":201,"context_line":"will need to be updated/changed to reflect the new Ceilometer naming."},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"References"},{"line_number":204,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_389490e1","line":201,"updated":"2015-03-05 12:34:41.000000000","message":"The new Measurements section in OpenStack Manuals: http://docs.openstack.org/admin-guide-cloud/content/section_telemetry-ironic-metrics.html . A DocImpact flag will be needed in the patch, which implements the new meters/meter names.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":198,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"The \"Ironic Hardware IPMI Sensor Data\" section in doc/source/measurements.rst"},{"line_number":201,"context_line":"will need to be updated/changed to reflect the new Ceilometer naming."},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"References"},{"line_number":204,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_4436ffb5","line":201,"in_reply_to":"9a80dd14_35a56eb5","updated":"2015-03-10 17:38:28.000000000","message":"As people add meters, they will simply need to update the section_telemetry-measurements.html to indicate the new meters.   I don\u0027t see this as any different than what would have needed to be done before.    I think I see your point though, if you don\u0027t need to change code in Ceilometer to add a new sensor meter, then you might not know you need to update the Ceilometer documentation.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"b78a6ca802cc0eb0f5a0cd2a6d4c98e840ff3a9b","unresolved":false,"context_lines":[{"line_number":198,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"The \"Ironic Hardware IPMI Sensor Data\" section in doc/source/measurements.rst"},{"line_number":201,"context_line":"will need to be updated/changed to reflect the new Ceilometer naming."},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"References"},{"line_number":204,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_35a56eb5","line":201,"in_reply_to":"9a80dd14_389490e1","updated":"2015-03-05 15:33:41.000000000","message":"Another question that came out on the meeting today is that is there any way to figure out the possibly available list of meters? As we have an exact list of metrics in that documentation and with this generic meter source idea it is not clear at the moment how can we extend this.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"f6f908d2a7f8b95ff2c9df9e4ac270adcf8edc77","unresolved":false,"context_lines":[{"line_number":198,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"The \"Ironic Hardware IPMI Sensor Data\" section in doc/source/measurements.rst"},{"line_number":201,"context_line":"will need to be updated/changed to reflect the new Ceilometer naming."},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"References"},{"line_number":204,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9a80dd14_44b51fcf","line":201,"in_reply_to":"9a80dd14_389490e1","updated":"2015-03-10 17:38:28.000000000","message":"See my reply above to Gordon\u0027s about admin guide changes.      DocImpact flag on the patch make sense, but I don\u0027t know the mechanics of this as I\u0027ve never submitted patches before to openstack.","commit_id":"16e5043039544feec1255346b97d6c35f06975d1"},{"author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"change_message_id":"5ef0c8d2e9048b7c72bd6cb825f6df0770af3356","unresolved":false,"context_lines":[{"line_number":121,"context_line":"  {"},{"line_number":122,"context_line":"    \"counter_name\": \"hardware.temperature\","},{"line_number":123,"context_line":"    \"user_id\": null,"},{"line_number":124,"context_line":"    \"resource_id\": \"03960b7d-3383-456f-9a8e-d76d795320ea-09-vr_p1_zone_(0x12)\","},{"line_number":125,"context_line":"    \"timestamp\": \"2015-03-26T16:27:59.240831\","},{"line_number":126,"context_line":"    \"recorded_at\": \"2015-03-26T16:27:59.325135\","},{"line_number":127,"context_line":"    \"message_id\": \"10e5b6a0-d3d5-11e4-a0ad-080027e2e50b\","}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa963961_6b827854","line":124,"updated":"2015-03-27 01:57:06.000000000","message":"I\u0027m just wondering, if the temperature is from other sources than IPMI, is it possible that the resource_id will be the same as that one from IPMI? My initial guessing is not, am I wrong?","commit_id":"109372390475368bb14f2f05db8ea2f2f4fdfc59"},{"author":{"_account_id":13448,"name":"Jim Mankovich","email":"jmank@hpe.com","username":"jmank"},"change_message_id":"ac35e2cd2aff60fa37d5c16733413a5e979c7c54","unresolved":false,"context_lines":[{"line_number":121,"context_line":"  {"},{"line_number":122,"context_line":"    \"counter_name\": \"hardware.temperature\","},{"line_number":123,"context_line":"    \"user_id\": null,"},{"line_number":124,"context_line":"    \"resource_id\": \"03960b7d-3383-456f-9a8e-d76d795320ea-09-vr_p1_zone_(0x12)\","},{"line_number":125,"context_line":"    \"timestamp\": \"2015-03-26T16:27:59.240831\","},{"line_number":126,"context_line":"    \"recorded_at\": \"2015-03-26T16:27:59.325135\","},{"line_number":127,"context_line":"    \"message_id\": \"10e5b6a0-d3d5-11e4-a0ad-080027e2e50b\","}],"source_content_type":"text/x-rst","patch_set":4,"id":"fa963961_5ebbe6c4","line":124,"in_reply_to":"fa963961_6b827854","updated":"2015-03-27 13:25:06.000000000","message":"IPMI provider reported sensors will always have a different resource_id than sensors reported by other providers due to way in which \"ipmitool\" displays the platform reported sensor id.    The \"ipmitool\" displays a sensor name as the IPMI reported sensor id appended with a IPMI specific sensor record index.   This ipmitool reported name is what is use in the creation the ceilometer resource_id string.\n\nHere is an example of ilo provider health information as reported to ceilometer from the ironic ilo sensor provider.       The ilo sensor provider only reports sensors that are not available via IPMI.\n\n        {\n            \"platform_uuid\": \"37363536-3636-4D32-3232-303130324A41\", \n            \"status\": \"OK\", \n            \"Sensor ID\": \"Drive\", \n            \"meter_name\": \"hardware.health\", \n            \"sensor_unit\": \"Percent\", \n            \"node\": \"03960b7d-3383-456f-9a8e-d76d795320ea\", \n            \"platform_name\": \"ProLiant ML110 G7\", \n            \"meter_id\": \"-drive\", \n            \"sensor_provider\": \"ilo\", \n            \"sensor_reading\": 100.0\n        }","commit_id":"109372390475368bb14f2f05db8ea2f2f4fdfc59"}]}