)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4e12fd2653e9fb406f62763f82bd11c2f2694df7","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Alex Weeks \u003calex.weeks@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-06-18 14:20:21 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add metric reporting spec."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ie3fb3d7c122f902fa89a8e2fd2f3777938be9cba"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1ae5cdf2_7d546afe","line":7,"updated":"2014-06-23 11:20:42.000000000","message":"Should duplicate the header: \"Add metrics reporting to Ironic\"","commit_id":"66acab7ddbb4ecd179c3a3558f089d4db41c1dd7"}],"specs/juno/add-metrics.rst":[{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"a72273eb9539e0e441ad44663e409114d2887bd7","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Add metrics reporting to Ironic"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"This blueprint proposes the addition of metric data reporting features to Ironic.  These data will initially be reported via statsd, but should be sufficiently generic to permit the creation of alternative back-ends."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Problem description"},{"line_number":14,"context_line":"\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":1,"id":"1ae5cdf2_18e06bf9","line":11,"updated":"2014-06-18 06:39:24.000000000","message":"You\u0027ll need to not split lines here :)","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"a72273eb9539e0e441ad44663e409114d2887bd7","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Problem description"},{"line_number":14,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Currently, Ironic does not report any metric data.  Metric data are extremely useful--even critical--for recognizing and diagnosing problems in software.  These metrics will can be used to monitor the health and performance of Ironic in a production environment."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Proposed change"},{"line_number":19,"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":1,"id":"1ae5cdf2_d8bc43de","line":16,"updated":"2014-06-18 06:39:24.000000000","message":"Perhaps \"Metric data is extremely useful - even critical - for recognizing...\" might be a better phrasing","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"a72273eb9539e0e441ad44663e409114d2887bd7","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"* Add a metric reporting system API"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"* Add support for statsd as the initial back-end."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"* Instrument code in Ironic--particularly RPCs--to report metric data."},{"line_number":26,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_58c83375","line":23,"updated":"2014-06-18 06:39:24.000000000","message":"May want to add a reference to statsd into the references section and refer to it here","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"a72273eb9539e0e441ad44663e409114d2887bd7","unresolved":false,"context_lines":[{"line_number":21,"context_line":"* Add a metric reporting system API"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"* Add support for statsd as the initial back-end."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"* Instrument code in Ironic--particularly RPCs--to report metric data."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_383e072a","line":24,"updated":"2014-06-18 06:39:24.000000000","message":"Where do you expect to collect this metric data?  From what parts of Ironic?","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"a72273eb9539e0e441ad44663e409114d2887bd7","unresolved":false,"context_lines":[{"line_number":27,"context_line":"Alternatives"},{"line_number":28,"context_line":"------------"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Reporting data to Ceilometer is a possibility, but in the near-term that project is focused on collecting metering billing for billing purposes.  Ceilometer support may be added as an alternative back-end in the future."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Data model impact"},{"line_number":33,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_d8a363af","line":30,"updated":"2014-06-18 06:39:24.000000000","message":"Splitting words here","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"a72273eb9539e0e441ad44663e409114d2887bd7","unresolved":false,"context_lines":[{"line_number":67,"context_line":"Performance"},{"line_number":68,"context_line":"------------------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Collecting and reporting metrics does affect performance, but these effects are likely to be small if discretion is used in how code is instrumented.  The overhead of sending a single piece of metric data is very small; in particular, statsd metrics are sent via UDP (non-blocking) to a local daemon that aggregates the metrics before forwarding them to one of its supported back-ends.  Should this back-end become unresponsive or overloaded metric data will be lost, but without any other performance issues."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Other deployer impact"},{"line_number":73,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_38944743","line":70,"updated":"2014-06-18 06:39:24.000000000","message":"Splitting words here","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"a72273eb9539e0e441ad44663e409114d2887bd7","unresolved":false,"context_lines":[{"line_number":68,"context_line":"------------------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Collecting and reporting metrics does affect performance, but these effects are likely to be small if discretion is used in how code is instrumented.  The overhead of sending a single piece of metric data is very small; in particular, statsd metrics are sent via UDP (non-blocking) to a local daemon that aggregates the metrics before forwarding them to one of its supported back-ends.  Should this back-end become unresponsive or overloaded metric data will be lost, but without any other performance issues."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Other deployer impact"},{"line_number":73,"context_line":"---------------------"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_582033f8","line":71,"updated":"2014-06-18 06:39:24.000000000","message":"Should we consider a mechanism to turn off metrics collection?  What happens if a deployment doesn\u0027t wish to collect metrics data?  Will that be configurable?  And will turning off collection result in zero performance impact?","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":11739,"name":"Alex Weeks","email":"alex.weeks@gmail.com","username":"aweeks"},"change_message_id":"6b0cfe33b1905f9ebd880c590a237f1e9da7aa86","unresolved":false,"context_lines":[{"line_number":68,"context_line":"------------------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Collecting and reporting metrics does affect performance, but these effects are likely to be small if discretion is used in how code is instrumented.  The overhead of sending a single piece of metric data is very small; in particular, statsd metrics are sent via UDP (non-blocking) to a local daemon that aggregates the metrics before forwarding them to one of its supported back-ends.  Should this back-end become unresponsive or overloaded metric data will be lost, but without any other performance issues."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Other deployer impact"},{"line_number":73,"context_line":"---------------------"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_ddd33936","line":71,"in_reply_to":"1ae5cdf2_582033f8","updated":"2014-06-18 17:48:49.000000000","message":"My current plan is to disable it by default, which would reduce, but not totally eliminate the performance hit.  At the very least, we\u0027d have to check a conditional at each place where a metric could be reported.  Additionally, arguments may end up being evaluated unnecessarily depending on exactly how/where the conditional checking is done.\n\nI\u0027ll add some more information about this in the spec.","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"ba2532baf629039d540a14542b3d997727381f45","unresolved":false,"context_lines":[{"line_number":72,"context_line":"Other deployer impact"},{"line_number":73,"context_line":"---------------------"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"Configuration options for statsd will be required.  Deployers will have to install and configure statsd, as well as any other metrics software (such as Graphite) they wish to use."},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Developer impact"},{"line_number":78,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_4f7f9466","line":75,"updated":"2014-06-18 00:35:53.000000000","message":"Probably want to call our Ceilometer here, for $raisins","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"a72273eb9539e0e441ad44663e409114d2887bd7","unresolved":false,"context_lines":[{"line_number":72,"context_line":"Other deployer impact"},{"line_number":73,"context_line":"---------------------"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"Configuration options for statsd will be required.  Deployers will have to install and configure statsd, as well as any other metrics software (such as Graphite) they wish to use."},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"Developer impact"},{"line_number":78,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_9899db7b","line":75,"updated":"2014-06-18 06:39:24.000000000","message":"Splitting words here again.  May also want to link a reference to Graphite","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"06a57f80e39d40a2618b6db207b3d673aa37d71b","unresolved":false,"context_lines":[{"line_number":103,"context_line":"Dependencies"},{"line_number":104,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The statsd procol is simple enough to justify implementing it ourselves in order to avoid introducing dependencies.  Alternatively, the python-statsd module could be used."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Testing"},{"line_number":109,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_af61f8d4","line":106,"updated":"2014-06-18 00:39:53.000000000","message":"Might be useful to note that swift has an implementation already, and maybe that we could add that to oslo.","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"ba2532baf629039d540a14542b3d997727381f45","unresolved":false,"context_lines":[{"line_number":103,"context_line":"Dependencies"},{"line_number":104,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The statsd procol is simple enough to justify implementing it ourselves in order to avoid introducing dependencies.  Alternatively, the python-statsd module could be used."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Testing"},{"line_number":109,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_2f6ec8c9","line":106,"updated":"2014-06-18 00:35:53.000000000","message":"spelling: s/procol/protocol/","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"a72273eb9539e0e441ad44663e409114d2887bd7","unresolved":false,"context_lines":[{"line_number":103,"context_line":"Dependencies"},{"line_number":104,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"The statsd procol is simple enough to justify implementing it ourselves in order to avoid introducing dependencies.  Alternatively, the python-statsd module could be used."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"Testing"},{"line_number":109,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_9870fb41","line":106,"updated":"2014-06-18 06:39:24.000000000","message":"splitting words here","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"a72273eb9539e0e441ad44663e409114d2887bd7","unresolved":false,"context_lines":[{"line_number":108,"context_line":"Testing"},{"line_number":109,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"Unit tests will be required.  Additional work may be required to test the statsd network code."},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"Documentation Impact"},{"line_number":114,"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":1,"id":"1ae5cdf2_38d0c7f6","line":111,"updated":"2014-06-18 06:39:24.000000000","message":"I don\u0027t think you mean that you\u0027re going to need to test statsd code, but rather our code to interact with it, and how statsd consumes what we send to it, right?","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"06a57f80e39d40a2618b6db207b3d673aa37d71b","unresolved":false,"context_lines":[{"line_number":113,"context_line":"Documentation Impact"},{"line_number":114,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"Appropriate documentation describing how to use the new features will be required."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"References"},{"line_number":119,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1ae5cdf2_cfb3a44b","line":116,"updated":"2014-06-18 00:39:53.000000000","message":"I would probably s/use/configure and deploy/","commit_id":"68cfeb846bc2e040f5268c63f0dd47aa9770722f"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"06ff7be80e4c9d149ab7102efa788619331d43d5","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"* Add support for statsd as the initial back-end [1]."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* Instrument code in Ironic--particularly RPCs--to report metric data."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Alternatives"},{"line_number":33,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1ae5cdf2_0c4ffe92","line":30,"updated":"2014-06-19 22:19:38.000000000","message":"I don\u0027t think mrda\u0027s question about where we expect to collect metric data was addressed in this patch.\n\nI\u0027m hoping the answer is \u0027everywhere\u0027, but a few examples might be worth providing. I\u0027m thinking:\n\n1) Provisioning times\n2) Metrics sent from an agent on a box being provisioned (it took me X seconds to download the image and Y seconds to image it)\n3) Timing on all RPC calls\n4) Counts of some common actions or failures (ipmi failed to work on node X 10% of the time)","commit_id":"66acab7ddbb4ecd179c3a3558f089d4db41c1dd7"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4e12fd2653e9fb406f62763f82bd11c2f2694df7","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"* Add support for statsd as the initial back-end [1]."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* Instrument code in Ironic--particularly RPCs--to report metric data."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Alternatives"},{"line_number":33,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1ae5cdf2_1d5c4619","line":30,"in_reply_to":"1ae5cdf2_0c4ffe92","updated":"2014-06-23 11:20:42.000000000","message":"I also vote for providing at least preliminary least of metrics to collect. Jay\u0027s list looks good, may make sense to also add timing of DB requests.","commit_id":"66acab7ddbb4ecd179c3a3558f089d4db41c1dd7"},{"author":{"_account_id":1420,"name":"Adam Gandelman","email":"adamg@ubuntu.com","username":"gandelman-a"},"change_message_id":"780957de74f56c44a2e4a9f21365681cf33b5755","unresolved":false,"context_lines":[{"line_number":59,"context_line":"Security impact"},{"line_number":60,"context_line":"---------------"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"None"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Other end user impact"},{"line_number":65,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1ae5cdf2_f161c339","line":62,"updated":"2014-06-20 04:34:44.000000000","message":"If I understand correctly, this requires installing a new daemon on a service host that will open a network port and listen for metrics broadcast by other Ironic components but also agents in deployment ramdisks?  There are no associated security concerns here?  How mature is the statsd project?  Are they responsive to upstream bugs and security issues?","commit_id":"66acab7ddbb4ecd179c3a3558f089d4db41c1dd7"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4e12fd2653e9fb406f62763f82bd11c2f2694df7","unresolved":false,"context_lines":[{"line_number":69,"context_line":"Scalability impact"},{"line_number":70,"context_line":"------------------"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"None"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"Performance Impact"},{"line_number":75,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1ae5cdf2_fde31a32","line":72,"updated":"2014-06-23 11:20:42.000000000","message":"There can be some scalability impact, depending on the answer for Josh\u0027s question below. I would like to see how HA can be this solution.","commit_id":"66acab7ddbb4ecd179c3a3558f089d4db41c1dd7"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"1aaf8547a5cd2df5bb01d4c78df7307a3800c219","unresolved":false,"context_lines":[{"line_number":77,"context_line":"Collecting and reporting metrics does affect performance, but these effects"},{"line_number":78,"context_line":"are likely to be small if discretion is used in how code is instrumented.  The"},{"line_number":79,"context_line":"overhead of sending a single piece of metric data is very small; in"},{"line_number":80,"context_line":"particular, statsd metrics are sent via UDP (non-blocking) to a local daemon"},{"line_number":81,"context_line":"that aggregates the metrics before forwarding them to one of its supported"},{"line_number":82,"context_line":"back-ends.  Should this back-end become unresponsive or overloaded, then"},{"line_number":83,"context_line":"metric data will be lost, but without any other performance issues."}],"source_content_type":"text/x-rst","patch_set":3,"id":"1ae5cdf2_05043113","line":80,"updated":"2014-06-20 00:11:36.000000000","message":"What\u0027s the daemon? Is it a prewritten statsd package? Something you\u0027ll be writing? Is it local to each node or is it a \"each cluster has a local-to-the-cluster statsd daemon\"?","commit_id":"66acab7ddbb4ecd179c3a3558f089d4db41c1dd7"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"1aaf8547a5cd2df5bb01d4c78df7307a3800c219","unresolved":false,"context_lines":[{"line_number":92,"context_line":"Other deployer impact"},{"line_number":93,"context_line":"---------------------"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"Configuration options for statsd will be required.  If metrics reporting is"},{"line_number":96,"context_line":"enabled, then Deployers will have to install and configure statsd, as well as"},{"line_number":97,"context_line":"any other metrics software (such as Graphite [2]) they wish to use."},{"line_number":98,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1ae5cdf2_c515e93d","line":95,"updated":"2014-06-20 00:11:36.000000000","message":"What are these options?","commit_id":"66acab7ddbb4ecd179c3a3558f089d4db41c1dd7"},{"author":{"_account_id":10380,"name":"Josh Gachnang","email":"josh@pcsforeducation.com","username":"JoshNang"},"change_message_id":"1aaf8547a5cd2df5bb01d4c78df7307a3800c219","unresolved":false,"context_lines":[{"line_number":96,"context_line":"enabled, then Deployers will have to install and configure statsd, as well as"},{"line_number":97,"context_line":"any other metrics software (such as Graphite [2]) they wish to use."},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"Developer impact"},{"line_number":100,"context_line":"----------------"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"None"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1ae5cdf2_c2279fc5","line":99,"updated":"2014-06-20 00:11:36.000000000","message":"If there is going to be a local daemon as noted above, there\u0027ll be additional things deployers need to do for each control plane node. Will it only be on conductors? API nodes as well?","commit_id":"66acab7ddbb4ecd179c3a3558f089d4db41c1dd7"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"4e12fd2653e9fb406f62763f82bd11c2f2694df7","unresolved":false,"context_lines":[{"line_number":135,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"Unit tests will be required.  Additional work may be required to test the code"},{"line_number":138,"context_line":"that generates and sends metrics via the statsd procotol."},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"Documentation Impact"},{"line_number":141,"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":"1ae5cdf2_9d43f612","line":138,"updated":"2014-06-23 11:20:42.000000000","message":"Will support for statsd be added to devstack? Enabling it there will at least check that we\u0027re not breaking everything.","commit_id":"66acab7ddbb4ecd179c3a3558f089d4db41c1dd7"}]}