)]}'
{"specs/liberty/plumgrid-network-metering-driver.rst":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"bbe78ec878f018d20ca4c0ebb122842ddccc7574","unresolved":false,"context_lines":[{"line_number":106,"context_line":"Alternatives"},{"line_number":107,"context_line":"------------"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"None"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"Data model impact"},{"line_number":112,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3afb71cf_9f043918","line":109,"updated":"2015-06-10 05:41:00.000000000","message":"in nova, they actually have functionality to gather metrics periodically, separated from ceilometer polling. these metrics are then published and consumed by ceilometer.\n\nis this a viable alternative? the positive of this is we(ceilometer) don\u0027t hammer any apis and there\u0027s an assumed performance benefit with the polling functionality closer to actual plumgrid networking suite api","commit_id":"e707cab0b15f38e7d0ac05379c5bb179c76b5e29"},{"author":{"_account_id":8279,"name":"Fawad Khaliq","email":"fawadkq@gmail.com","username":"fawadkhaliq"},"change_message_id":"24a610b1ba0daaf12d9456816f67300efdb13be5","unresolved":false,"context_lines":[{"line_number":106,"context_line":"Alternatives"},{"line_number":107,"context_line":"------------"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"None"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"Data model impact"},{"line_number":112,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3afb71cf_b61ef06d","line":109,"in_reply_to":"3afb71cf_6546847c","updated":"2015-06-10 12:14:07.000000000","message":"Makes sense. That would be the right approach and I have covered such aspect for \"Future lifecycle\" at line#195 and this proposal is to be considered a starting point. Trying to take baby steps as having the full solution with Neutron involved is definitely something that will take longer time and more effort. I have kept this as a stepping stone towards a better implementation for future releases.\n\nHad a discussion with Gordon and I clarified the aspects. I will add some of the metric this driver intends to use to complete the approach and intent. \n\nNew patch coming with some of the metric this driver will utilize.","commit_id":"e707cab0b15f38e7d0ac05379c5bb179c76b5e29"},{"author":{"_account_id":8279,"name":"Fawad Khaliq","email":"fawadkq@gmail.com","username":"fawadkhaliq"},"change_message_id":"55cb4c3a30cbb2ceb1da1ed8017d807c3aa431bd","unresolved":false,"context_lines":[{"line_number":106,"context_line":"Alternatives"},{"line_number":107,"context_line":"------------"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"None"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"Data model impact"},{"line_number":112,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3afb71cf_b16cc935","line":109,"in_reply_to":"3afb71cf_9f043918","updated":"2015-06-10 06:11:03.000000000","message":"Thanks a lot for the review. \n\nPlugin/driver functionality exists in Neutron for networking and hence PLUMgrid plugin (which has been part of Neutron since Grizzly) expose the APIs that Neutron offers. Unfortunately, metrics interfaces in Nova cannot be used with Neutron.\n\nThe proposal is exactly like how current in-tree ODL [1] and Contrail [2] network drivers follow.\n\n[1] https://github.com/openstack/ceilometer/tree/master/ceilometer/network/statistics/opendaylight\n[2] https://github.com/openstack/ceilometer/tree/master/ceilometer/network/statistics/opencontrail","commit_id":"e707cab0b15f38e7d0ac05379c5bb179c76b5e29"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"ba84c62fe8da5a1b3b766297738caf16373ddf2e","unresolved":false,"context_lines":[{"line_number":106,"context_line":"Alternatives"},{"line_number":107,"context_line":"------------"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"None"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"Data model impact"},{"line_number":112,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"3afb71cf_6546847c","line":109,"in_reply_to":"3afb71cf_b16cc935","updated":"2015-06-10 10:55:23.000000000","message":"\u003e The proposal is exactly like how current in-tree ODL [1] and Contrail [2] network drivers follow.\n\nI believe the point that gordon was trying to make is maybe we now have an opportunity to do things in a more architecturally correct and efficient fashion. If PLUMgrid were to drive the implementation of neutron emitting notifications that provided metering information that would be a big win for neutron and ceilometer as well as for other monitoring and measuring tools such as monasca and stacktach. One of the discussions we had at the most recent summit is that it is less than ideal for ceilometer to be doing polling: it would be far better for the various services to produces notifications that lots of tools good consume.","commit_id":"e707cab0b15f38e7d0ac05379c5bb179c76b5e29"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"change_message_id":"be3b804c051b696a21815d2b4726eb3e6bffdb8e","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This blueprint will implement a driver in Ceilometer to provide network"},{"line_number":14,"context_line":"monitoring and statistics from PLUMgrid Networking. PLUMgrid Networking"},{"line_number":15,"context_line":"works as a core plugin in Neutron and provides complete networking."},{"line_number":16,"context_line":"Ceilometer PLUMgrid Network monitoring driver will expose network"},{"line_number":17,"context_line":"statistics using meters by communicating with PLUMgrid over REST APIs"},{"line_number":18,"context_line":"through a client."}],"source_content_type":"text/x-rst","patch_set":8,"id":"3afb71cf_7fc276ea","line":15,"updated":"2015-06-10 20:20:43.000000000","message":"core? don\u0027t you have a ML2 driver already?\nThe second part should be rephrase like:\n\"and provides the backend for virtual networking\"","commit_id":"2a0c4152192b43b206b1a1585096ab464b8b52e1"},{"author":{"_account_id":8279,"name":"Fawad Khaliq","email":"fawadkq@gmail.com","username":"fawadkhaliq"},"change_message_id":"2b194eadcc09a07c8f5232a7cbcfd9b9acff6fc2","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This blueprint will implement a driver in Ceilometer to provide network"},{"line_number":14,"context_line":"monitoring and statistics from PLUMgrid Networking. PLUMgrid Networking"},{"line_number":15,"context_line":"works as a core plugin in Neutron and provides complete networking."},{"line_number":16,"context_line":"Ceilometer PLUMgrid Network monitoring driver will expose network"},{"line_number":17,"context_line":"statistics using meters by communicating with PLUMgrid over REST APIs"},{"line_number":18,"context_line":"through a client."}],"source_content_type":"text/x-rst","patch_set":8,"id":"3afb71cf_71f61b5e","line":15,"in_reply_to":"3afb71cf_7fc276ea","updated":"2015-06-11 05:44:01.000000000","message":"Thanks for the reviews Edgar. \n\nPLUMgrid plugin is a core plugin. \n\nI will amend the statement. Thanks!","commit_id":"2a0c4152192b43b206b1a1585096ab464b8b52e1"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"change_message_id":"be3b804c051b696a21815d2b4726eb3e6bffdb8e","unresolved":false,"context_lines":[{"line_number":13,"context_line":"This blueprint will implement a driver in Ceilometer to provide network"},{"line_number":14,"context_line":"monitoring and statistics from PLUMgrid Networking. PLUMgrid Networking"},{"line_number":15,"context_line":"works as a core plugin in Neutron and provides complete networking."},{"line_number":16,"context_line":"Ceilometer PLUMgrid Network monitoring driver will expose network"},{"line_number":17,"context_line":"statistics using meters by communicating with PLUMgrid over REST APIs"},{"line_number":18,"context_line":"through a client."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"3afb71cf_3f463e7b","line":16,"updated":"2015-06-10 20:20:43.000000000","message":"should be Networking for consistency","commit_id":"2a0c4152192b43b206b1a1585096ab464b8b52e1"},{"author":{"_account_id":8279,"name":"Fawad Khaliq","email":"fawadkq@gmail.com","username":"fawadkhaliq"},"change_message_id":"2b194eadcc09a07c8f5232a7cbcfd9b9acff6fc2","unresolved":false,"context_lines":[{"line_number":13,"context_line":"This blueprint will implement a driver in Ceilometer to provide network"},{"line_number":14,"context_line":"monitoring and statistics from PLUMgrid Networking. PLUMgrid Networking"},{"line_number":15,"context_line":"works as a core plugin in Neutron and provides complete networking."},{"line_number":16,"context_line":"Ceilometer PLUMgrid Network monitoring driver will expose network"},{"line_number":17,"context_line":"statistics using meters by communicating with PLUMgrid over REST APIs"},{"line_number":18,"context_line":"through a client."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-rst","patch_set":8,"id":"3afb71cf_31c2e3e9","line":16,"in_reply_to":"3afb71cf_3f463e7b","updated":"2015-06-11 05:44:01.000000000","message":"Fixing it shortly.","commit_id":"2a0c4152192b43b206b1a1585096ab464b8b52e1"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"change_message_id":"be3b804c051b696a21815d2b4726eb3e6bffdb8e","unresolved":false,"context_lines":[{"line_number":93,"context_line":"  |                          |"},{"line_number":94,"context_line":"  |  +---------------------+ |       +----------------------+"},{"line_number":95,"context_line":"  |  |                     | |       |                      |"},{"line_number":96,"context_line":"  |  |   PLUMgrid Network  | |       |  PLUMgrid REST API   |"},{"line_number":97,"context_line":"  |  |                     | |       |                      |"},{"line_number":98,"context_line":"  |  |       Driver        +-------\u003e |        Client        |"},{"line_number":99,"context_line":"  |  |                     | |       |                      |"}],"source_content_type":"text/x-rst","patch_set":8,"id":"3afb71cf_9f1bf280","line":96,"updated":"2015-06-10 20:20:43.000000000","message":"Networking","commit_id":"2a0c4152192b43b206b1a1585096ab464b8b52e1"},{"author":{"_account_id":8279,"name":"Fawad Khaliq","email":"fawadkq@gmail.com","username":"fawadkhaliq"},"change_message_id":"2b194eadcc09a07c8f5232a7cbcfd9b9acff6fc2","unresolved":false,"context_lines":[{"line_number":93,"context_line":"  |                          |"},{"line_number":94,"context_line":"  |  +---------------------+ |       +----------------------+"},{"line_number":95,"context_line":"  |  |                     | |       |                      |"},{"line_number":96,"context_line":"  |  |   PLUMgrid Network  | |       |  PLUMgrid REST API   |"},{"line_number":97,"context_line":"  |  |                     | |       |                      |"},{"line_number":98,"context_line":"  |  |       Driver        +-------\u003e |        Client        |"},{"line_number":99,"context_line":"  |  |                     | |       |                      |"}],"source_content_type":"text/x-rst","patch_set":8,"id":"3afb71cf_51bf9f60","line":96,"in_reply_to":"3afb71cf_9f1bf280","updated":"2015-06-11 05:44:01.000000000","message":"I will amend in the next patch.","commit_id":"2a0c4152192b43b206b1a1585096ab464b8b52e1"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"caa321dc78e1644a86eb3e7e2c56e71ce55cc7fe","unresolved":false,"context_lines":[{"line_number":25,"context_line":"under Ceilometer Central Agent to expose important networking monitoring"},{"line_number":26,"context_line":"and metering information, which is becoming extremely important for operators"},{"line_number":27,"context_line":"so they can have more visibility into one of the complicated pieces"},{"line_number":28,"context_line":"of OpenStack clouds: networking (Neutron). So, while Neutron handles"},{"line_number":29,"context_line":"management aspects of network resources, Ceilometer is designed to get"},{"line_number":30,"context_line":"statistics. A driver for PLUMgrid networking is required, in Ceilometer,"},{"line_number":31,"context_line":"to pull statistics for network resources and transform them into consumable"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1af86dd1_1aa46055","line":28,"updated":"2015-06-12 21:11:27.000000000","message":"Seems like a fairly subjective thing to say here.","commit_id":"74e3ee19872dae11903115e06e3d2498e11c5d82"},{"author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"change_message_id":"caa321dc78e1644a86eb3e7e2c56e71ce55cc7fe","unresolved":false,"context_lines":[{"line_number":29,"context_line":"management aspects of network resources, Ceilometer is designed to get"},{"line_number":30,"context_line":"statistics. A driver for PLUMgrid networking is required, in Ceilometer,"},{"line_number":31,"context_line":"to pull statistics for network resources and transform them into consumable"},{"line_number":32,"context_line":"data through Ceilometer APIs, which will be used by its users."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Proposed change"},{"line_number":35,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":9,"id":"1af86dd1_7abc746c","line":32,"updated":"2015-06-12 21:11:27.000000000","message":"This seems weirdly backwards. I would have thought we would expose the data via the ceilometer/neutron interface, rather than introducing plugins into ceilometer this way.","commit_id":"74e3ee19872dae11903115e06e3d2498e11c5d82"}]}