)]}'
{"releasenotes/notes/network-statistics-from-opendaylight-787df77484d8d751.yaml":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"3586a20dbb119480a263863ad99bc0e0f9c26a03","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"prelude: \u003e"},{"line_number":3,"context_line":"    Network Statistics From OpenDaylight."},{"line_number":4,"context_line":"features:"},{"line_number":5,"context_line":"  - Add a ceilometer driver to collect network"},{"line_number":6,"context_line":"    statistics information using REST APIs exposed by"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3f1d235d_51f14800","line":3,"range":{"start_line":3,"start_character":28,"end_line":3,"end_character":40},"updated":"2017-07-07 05:38:56.000000000","message":"i don\u0027t see anything odl specific in this patch.\ncan you explain?","commit_id":"bf406f4c259db89c0e9890635cef5ba70077c96b"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"ab603376bc44775e6676952792fd191c9580e253","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"prelude: \u003e"},{"line_number":3,"context_line":"    Network Statistics From OpenDaylight."},{"line_number":4,"context_line":"features:"},{"line_number":5,"context_line":"  - Add a ceilometer driver to collect network"},{"line_number":6,"context_line":"    statistics information using REST APIs exposed by"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3f1d235d_b89078ee","line":3,"range":{"start_line":3,"start_character":28,"end_line":3,"end_character":40},"in_reply_to":"3f1d235d_51f14800","updated":"2017-07-07 10:26:57.000000000","message":"This patch is one of the patches which implements the blueprint network-statistics-from-opendaylight. This particular review is one of the base code changes required for implementing the driver.\n\nRef: https://review.openstack.org/#/q/topic:bp/network-statistics-from-opendaylight","commit_id":"bf406f4c259db89c0e9890635cef5ba70077c96b"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"3586a20dbb119480a263863ad99bc0e0f9c26a03","unresolved":false,"context_lines":[{"line_number":5,"context_line":"  - Add a ceilometer driver to collect network"},{"line_number":6,"context_line":"    statistics information using REST APIs exposed by"},{"line_number":7,"context_line":"    network-statistics module in OpenDaylight."},{"line_number":8,"context_line":"  - Add support for network statistics meters with gnocchi"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3f1d235d_f1d97482","line":8,"range":{"start_line":8,"start_character":0,"end_line":8,"end_character":58},"updated":"2017-07-07 05:38:56.000000000","message":"i guess this is not about this patch.","commit_id":"bf406f4c259db89c0e9890635cef5ba70077c96b"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"ab603376bc44775e6676952792fd191c9580e253","unresolved":false,"context_lines":[{"line_number":5,"context_line":"  - Add a ceilometer driver to collect network"},{"line_number":6,"context_line":"    statistics information using REST APIs exposed by"},{"line_number":7,"context_line":"    network-statistics module in OpenDaylight."},{"line_number":8,"context_line":"  - Add support for network statistics meters with gnocchi"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3f1d235d_98a49c4c","line":8,"range":{"start_line":8,"start_character":0,"end_line":8,"end_character":58},"in_reply_to":"3f1d235d_f1d97482","updated":"2017-07-07 10:26:57.000000000","message":"The features aims to add a new ODL driver. The SDN counters are not yet supported with gnocchi. https://review.openstack.org/#/c/478159/ handles it.","commit_id":"bf406f4c259db89c0e9890635cef5ba70077c96b"}],"setup.cfg":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"2ae05ab6cb7c677c24dd72e7269f6b836ed32b24","unresolved":false,"context_lines":[{"line_number":152,"context_line":"    port.receive.bytes \u003d ceilometer.network.statistics.port_v2:PortPollsterReceiveBytes"},{"line_number":153,"context_line":"    port.transmit.bytes \u003d ceilometer.network.statistics.port_v2:PortPollsterTransmitBytes"},{"line_number":154,"context_line":"    port.receive.drops \u003d ceilometer.network.statistics.port_v2:PortPollsterReceiveDrops"},{"line_number":155,"context_line":"    port.receive.errors \u003d ceilometer.network.statistics.port_v2:PortPollsterReceiveErrors"},{"line_number":156,"context_line":"    rgw.containers.objects \u003d ceilometer.objectstore.rgw:ContainersObjectsPollster"},{"line_number":157,"context_line":"    rgw.containers.objects.size \u003d ceilometer.objectstore.rgw:ContainersSizePollster"},{"line_number":158,"context_line":"    rgw.objects \u003d ceilometer.objectstore.rgw:ObjectsPollster"}],"source_content_type":"text/x-ttcn-cfg","patch_set":1,"id":"5f201791_6d2dbda6","line":155,"updated":"2017-06-27 12:22:38.000000000","message":"missing:\nPortPollsterTransmitDrops\nPortPollsterTransmitErrors\nPortPollsterReceiveFrameErrors\nPortPollsterReceiveOverrunErrors\nPortPollsterReceiveCRCErrors\nPortPollsterCollisionCount","commit_id":"fc028079b7f291f1c1deca3a21efbed754d1a901"},{"author":{"_account_id":9361,"name":"Vivekanandan Narasimhan","email":"n.vivekanandan@ericsson.com","username":"viveknarasimhan"},"change_message_id":"d3e781ca0e1cdcc18d3b9297836b07d323a6e1ba","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    switch.port.receive.frame_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveFrameErrors"},{"line_number":183,"context_line":"    switch.port.receive.overrun_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveOverrunErrors"},{"line_number":184,"context_line":"    switch.port.receive.crc_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveCRCErrors"},{"line_number":185,"context_line":"    switch.port.collision.count \u003d ceilometer.network.statistics.port:PortPollsterCollisionCount"},{"line_number":186,"context_line":"    switch.table \u003d ceilometer.network.statistics.table:TablePollster"},{"line_number":187,"context_line":"    switch.table.active.entries \u003d ceilometer.network.statistics.table:TablePollsterActiveEntries"},{"line_number":188,"context_line":"    switch.table.lookup.packets \u003d ceilometer.network.statistics.table:TablePollsterLookupPackets"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_1b6c3606","line":185,"updated":"2017-06-27 18:01:38.000000000","message":"are the above lines from 174 - 185 required to be retained?","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"e8cd2ee99d4f42e60874e8beaaef906666159b19","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    switch.port.receive.frame_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveFrameErrors"},{"line_number":183,"context_line":"    switch.port.receive.overrun_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveOverrunErrors"},{"line_number":184,"context_line":"    switch.port.receive.crc_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveCRCErrors"},{"line_number":185,"context_line":"    switch.port.collision.count \u003d ceilometer.network.statistics.port:PortPollsterCollisionCount"},{"line_number":186,"context_line":"    switch.table \u003d ceilometer.network.statistics.table:TablePollster"},{"line_number":187,"context_line":"    switch.table.active.entries \u003d ceilometer.network.statistics.table:TablePollsterActiveEntries"},{"line_number":188,"context_line":"    switch.table.lookup.packets \u003d ceilometer.network.statistics.table:TablePollsterLookupPackets"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_b10af4ce","line":185,"in_reply_to":"5f201791_1b6c3606","updated":"2017-06-28 05:04:38.000000000","message":"switch.port.(transmit|receive).(bytes|packets) are used by opencontrail driver.\n\nRest of the meters are used in existing opendaylight driver. I think these meters can be removed once we remove exsting driver. Thoughts?\n\nHowever at the moment, these meters are not supported with gnocchi","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"9e731bba0567f8d1659a75ec28290f94679c0a47","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    switch.port.receive.frame_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveFrameErrors"},{"line_number":183,"context_line":"    switch.port.receive.overrun_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveOverrunErrors"},{"line_number":184,"context_line":"    switch.port.receive.crc_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveCRCErrors"},{"line_number":185,"context_line":"    switch.port.collision.count \u003d ceilometer.network.statistics.port:PortPollsterCollisionCount"},{"line_number":186,"context_line":"    switch.table \u003d ceilometer.network.statistics.table:TablePollster"},{"line_number":187,"context_line":"    switch.table.active.entries \u003d ceilometer.network.statistics.table:TablePollsterActiveEntries"},{"line_number":188,"context_line":"    switch.table.lookup.packets \u003d ceilometer.network.statistics.table:TablePollsterLookupPackets"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_1d953287","line":185,"in_reply_to":"5f201791_34f7f28d","updated":"2017-06-28 09:35:48.000000000","message":"As discussed, we will not handle removal of ODL driver now and retain these counters as some of them are used by opencontrail driver.","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"},{"author":{"_account_id":9361,"name":"Vivekanandan Narasimhan","email":"n.vivekanandan@ericsson.com","username":"viveknarasimhan"},"change_message_id":"5705e7b4145c708a4d347a02fc9d2fe6ec7ebd62","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    switch.port.receive.frame_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveFrameErrors"},{"line_number":183,"context_line":"    switch.port.receive.overrun_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveOverrunErrors"},{"line_number":184,"context_line":"    switch.port.receive.crc_error \u003d ceilometer.network.statistics.port:PortPollsterReceiveCRCErrors"},{"line_number":185,"context_line":"    switch.port.collision.count \u003d ceilometer.network.statistics.port:PortPollsterCollisionCount"},{"line_number":186,"context_line":"    switch.table \u003d ceilometer.network.statistics.table:TablePollster"},{"line_number":187,"context_line":"    switch.table.active.entries \u003d ceilometer.network.statistics.table:TablePollsterActiveEntries"},{"line_number":188,"context_line":"    switch.table.lookup.packets \u003d ceilometer.network.statistics.table:TablePollsterLookupPackets"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_34f7f28d","line":185,"in_reply_to":"5f201791_b10af4ce","updated":"2017-06-28 05:45:21.000000000","message":"I am starting to think that we will remove the old OpenDaylight driver..\n\nWe can quote that in our Spec itself and then remove that driver as part of this review.\n\nAlso we donot then need to name the new drivers as v2.\n\nWhat do you think?","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"},{"author":{"_account_id":9361,"name":"Vivekanandan Narasimhan","email":"n.vivekanandan@ericsson.com","username":"viveknarasimhan"},"change_message_id":"d3e781ca0e1cdcc18d3b9297836b07d323a6e1ba","unresolved":false,"context_lines":[{"line_number":186,"context_line":"    switch.table \u003d ceilometer.network.statistics.table:TablePollster"},{"line_number":187,"context_line":"    switch.table.active.entries \u003d ceilometer.network.statistics.table:TablePollsterActiveEntries"},{"line_number":188,"context_line":"    switch.table.lookup.packets \u003d ceilometer.network.statistics.table:TablePollsterLookupPackets"},{"line_number":189,"context_line":"    switch.table.matched.packets \u003d ceilometer.network.statistics.table:TablePollsterMatchedPackets"},{"line_number":190,"context_line":"    switch \u003d ceilometer.network.statistics.switch:SWPollster"},{"line_number":191,"context_line":"    switch.ports \u003d ceilometer.network.statistics.switch:SwitchPollsterPorts"},{"line_number":192,"context_line":"    switch.flow \u003d ceilometer.network.statistics.flow:FlowPollster"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_fb15fa49","line":189,"updated":"2017-06-27 18:01:38.000000000","message":"are we going to support switch.table pollsters with a v2 version?","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"},{"author":{"_account_id":9361,"name":"Vivekanandan Narasimhan","email":"n.vivekanandan@ericsson.com","username":"viveknarasimhan"},"change_message_id":"5705e7b4145c708a4d347a02fc9d2fe6ec7ebd62","unresolved":false,"context_lines":[{"line_number":186,"context_line":"    switch.table \u003d ceilometer.network.statistics.table:TablePollster"},{"line_number":187,"context_line":"    switch.table.active.entries \u003d ceilometer.network.statistics.table:TablePollsterActiveEntries"},{"line_number":188,"context_line":"    switch.table.lookup.packets \u003d ceilometer.network.statistics.table:TablePollsterLookupPackets"},{"line_number":189,"context_line":"    switch.table.matched.packets \u003d ceilometer.network.statistics.table:TablePollsterMatchedPackets"},{"line_number":190,"context_line":"    switch \u003d ceilometer.network.statistics.switch:SWPollster"},{"line_number":191,"context_line":"    switch.ports \u003d ceilometer.network.statistics.switch:SwitchPollsterPorts"},{"line_number":192,"context_line":"    switch.flow \u003d ceilometer.network.statistics.flow:FlowPollster"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_f4ecdabb","line":189,"in_reply_to":"5f201791_1101a8af","updated":"2017-06-28 05:45:21.000000000","message":"what is switch table active entries mean.. what flow entries are considered active in a table?","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"9e731bba0567f8d1659a75ec28290f94679c0a47","unresolved":false,"context_lines":[{"line_number":186,"context_line":"    switch.table \u003d ceilometer.network.statistics.table:TablePollster"},{"line_number":187,"context_line":"    switch.table.active.entries \u003d ceilometer.network.statistics.table:TablePollsterActiveEntries"},{"line_number":188,"context_line":"    switch.table.lookup.packets \u003d ceilometer.network.statistics.table:TablePollsterLookupPackets"},{"line_number":189,"context_line":"    switch.table.matched.packets \u003d ceilometer.network.statistics.table:TablePollsterMatchedPackets"},{"line_number":190,"context_line":"    switch \u003d ceilometer.network.statistics.switch:SWPollster"},{"line_number":191,"context_line":"    switch.ports \u003d ceilometer.network.statistics.switch:SwitchPollsterPorts"},{"line_number":192,"context_line":"    switch.flow \u003d ceilometer.network.statistics.flow:FlowPollster"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_5df4caf0","line":189,"in_reply_to":"5f201791_fb15fa49","updated":"2017-06-28 09:35:48.000000000","message":"It shows number of flows installed on the table as per openflow spec.","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"e8cd2ee99d4f42e60874e8beaaef906666159b19","unresolved":false,"context_lines":[{"line_number":186,"context_line":"    switch.table \u003d ceilometer.network.statistics.table:TablePollster"},{"line_number":187,"context_line":"    switch.table.active.entries \u003d ceilometer.network.statistics.table:TablePollsterActiveEntries"},{"line_number":188,"context_line":"    switch.table.lookup.packets \u003d ceilometer.network.statistics.table:TablePollsterLookupPackets"},{"line_number":189,"context_line":"    switch.table.matched.packets \u003d ceilometer.network.statistics.table:TablePollsterMatchedPackets"},{"line_number":190,"context_line":"    switch \u003d ceilometer.network.statistics.switch:SWPollster"},{"line_number":191,"context_line":"    switch.ports \u003d ceilometer.network.statistics.switch:SwitchPollsterPorts"},{"line_number":192,"context_line":"    switch.flow \u003d ceilometer.network.statistics.flow:FlowPollster"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_1101a8af","line":189,"in_reply_to":"5f201791_fb15fa49","updated":"2017-06-28 05:04:38.000000000","message":"OpenDaylight v2 driver will support only switch.table.active.entries.","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"},{"author":{"_account_id":9361,"name":"Vivekanandan Narasimhan","email":"n.vivekanandan@ericsson.com","username":"viveknarasimhan"},"change_message_id":"d3e781ca0e1cdcc18d3b9297836b07d323a6e1ba","unresolved":false,"context_lines":[{"line_number":190,"context_line":"    switch \u003d ceilometer.network.statistics.switch:SWPollster"},{"line_number":191,"context_line":"    switch.ports \u003d ceilometer.network.statistics.switch:SwitchPollsterPorts"},{"line_number":192,"context_line":"    switch.flow \u003d ceilometer.network.statistics.flow:FlowPollster"},{"line_number":193,"context_line":"    switch.flow.bytes \u003d ceilometer.network.statistics.flow:FlowPollsterBytes"},{"line_number":194,"context_line":"    switch.flow.duration.nanoseconds \u003d ceilometer.network.statistics.flow:FlowPollsterDurationNanoseconds"},{"line_number":195,"context_line":"    switch.flow.duration.seconds \u003d ceilometer.network.statistics.flow:FlowPollsterDurationSeconds"},{"line_number":196,"context_line":"    switch.flow.packets \u003d ceilometer.network.statistics.flow:FlowPollsterPackets"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_db7b5d89","line":193,"updated":"2017-06-27 18:01:38.000000000","message":"the same question as for \u0027table\u0027 applies for switch.flow here as well..","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"},{"author":{"_account_id":9361,"name":"Vivekanandan Narasimhan","email":"n.vivekanandan@ericsson.com","username":"viveknarasimhan"},"change_message_id":"5705e7b4145c708a4d347a02fc9d2fe6ec7ebd62","unresolved":false,"context_lines":[{"line_number":190,"context_line":"    switch \u003d ceilometer.network.statistics.switch:SWPollster"},{"line_number":191,"context_line":"    switch.ports \u003d ceilometer.network.statistics.switch:SwitchPollsterPorts"},{"line_number":192,"context_line":"    switch.flow \u003d ceilometer.network.statistics.flow:FlowPollster"},{"line_number":193,"context_line":"    switch.flow.bytes \u003d ceilometer.network.statistics.flow:FlowPollsterBytes"},{"line_number":194,"context_line":"    switch.flow.duration.nanoseconds \u003d ceilometer.network.statistics.flow:FlowPollsterDurationNanoseconds"},{"line_number":195,"context_line":"    switch.flow.duration.seconds \u003d ceilometer.network.statistics.flow:FlowPollsterDurationSeconds"},{"line_number":196,"context_line":"    switch.flow.packets \u003d ceilometer.network.statistics.flow:FlowPollsterPackets"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_142076f6","line":193,"in_reply_to":"5f201791_d1be90d3","updated":"2017-06-28 05:45:21.000000000","message":"This aligns with my question in the first comment of this file.","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"},{"author":{"_account_id":25171,"name":"Deepthi V V","email":"deepthi.v.v@ericsson.com","username":"deepthivv88"},"change_message_id":"e8cd2ee99d4f42e60874e8beaaef906666159b19","unresolved":false,"context_lines":[{"line_number":190,"context_line":"    switch \u003d ceilometer.network.statistics.switch:SWPollster"},{"line_number":191,"context_line":"    switch.ports \u003d ceilometer.network.statistics.switch:SwitchPollsterPorts"},{"line_number":192,"context_line":"    switch.flow \u003d ceilometer.network.statistics.flow:FlowPollster"},{"line_number":193,"context_line":"    switch.flow.bytes \u003d ceilometer.network.statistics.flow:FlowPollsterBytes"},{"line_number":194,"context_line":"    switch.flow.duration.nanoseconds \u003d ceilometer.network.statistics.flow:FlowPollsterDurationNanoseconds"},{"line_number":195,"context_line":"    switch.flow.duration.seconds \u003d ceilometer.network.statistics.flow:FlowPollsterDurationSeconds"},{"line_number":196,"context_line":"    switch.flow.packets \u003d ceilometer.network.statistics.flow:FlowPollsterPackets"}],"source_content_type":"text/x-ttcn-cfg","patch_set":2,"id":"5f201791_d1be90d3","line":193,"in_reply_to":"5f201791_db7b5d89","updated":"2017-06-28 05:04:38.000000000","message":"Switch.flow meters will not be supported by OpenDaylight v2 driver.","commit_id":"c3c76817ff03d15cccf8583b550f437d9b963348"}]}
