)]}'
{"monasca_persister/README.md":[{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"6b72ee19b29ffcccc7e0b77409be563f5daf5677","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"### Package Installation"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Monasca Persister by installed from PyPI via pip. Use your distribution"},{"line_number":13,"context_line":"package manager, or other preferred method, to ensure that you have pip"},{"line_number":14,"context_line":"installed, it is typically called python-pip."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1a6eadb0_051714a4","line":12,"range":{"start_line":12,"start_character":18,"end_line":12,"end_character":20},"updated":"2016-12-12 21:14:58.000000000","message":"should be?","commit_id":"64063309b824d52f863def97212972a138d5bb6e"},{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"6b72ee19b29ffcccc7e0b77409be563f5daf5677","unresolved":false,"context_lines":[{"line_number":89,"context_line":"sudo mkdir /etc/monasca"},{"line_number":90,"context_line":"sudo cp \\"},{"line_number":91,"context_line":"  /usr/local/lib/python2.7/dist-packages/monasca_persister/persister.conf \\"},{"line_number":92,"context_line":"  /etc/monasca  "},{"line_number":93,"context_line":"```"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"Note that the configuration will contain authentication information for"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1a6eadb0_25b410f1","line":92,"range":{"start_line":92,"start_character":14,"end_line":92,"end_character":16},"updated":"2016-12-12 21:14:58.000000000","message":"Trailing spaces","commit_id":"64063309b824d52f863def97212972a138d5bb6e"},{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"6b72ee19b29ffcccc7e0b77409be563f5daf5677","unresolved":false,"context_lines":[{"line_number":115,"context_line":"uri \u003d \u003chost1\u003e:\u003cport1\u003e,\u003chost2\u003e:\u003cport2\u003e,..."},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"[influxdb]"},{"line_number":118,"context_line":"database_name \u003d "},{"line_number":119,"context_line":"ip_address \u003d "},{"line_number":120,"context_line":"port \u003d "},{"line_number":121,"context_line":"user \u003d "}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1a6eadb0_45b02cf5","line":118,"range":{"start_line":118,"start_character":15,"end_line":118,"end_character":16},"updated":"2016-12-12 21:14:58.000000000","message":"Trailing spaces on these options","commit_id":"64063309b824d52f863def97212972a138d5bb6e"},{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"6b72ee19b29ffcccc7e0b77409be563f5daf5677","unresolved":false,"context_lines":[{"line_number":178,"context_line":"WantedBy\u003dmulti-user.target"},{"line_number":179,"context_line":"```"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"For a production deployment, it will almost always be desireable to use the"},{"line_number":182,"context_line":"*Restart* clause in the service file. The *RestartSec* clause is also"},{"line_number":183,"context_line":"important as by default, systemd assumes a delay of 100ms. A number of"},{"line_number":184,"context_line":"failures in quick sucession will cause the unit to enter a failed state,"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1a6eadb0_c5735c4c","line":181,"range":{"start_line":181,"start_character":54,"end_line":181,"end_character":64},"updated":"2016-12-12 21:14:58.000000000","message":"desirable","commit_id":"64063309b824d52f863def97212972a138d5bb6e"},{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"6b72ee19b29ffcccc7e0b77409be563f5daf5677","unresolved":false,"context_lines":[{"line_number":183,"context_line":"important as by default, systemd assumes a delay of 100ms. A number of"},{"line_number":184,"context_line":"failures in quick sucession will cause the unit to enter a failed state,"},{"line_number":185,"context_line":"therefore extending this period is critical."},{"line_number":186,"context_line":""},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"# License"},{"line_number":189,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1a6eadb0_a54900d3","line":186,"updated":"2016-12-12 21:14:58.000000000","message":"Maybe add the below?\n\nAfter creating ``/etc/systemd/system/monasca-persister.service``, you should run (as root):\n\nsystemctl daemon-reload\n\nand\n\nsystemctl  start monasca-persister","commit_id":"64063309b824d52f863def97212972a138d5bb6e"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"fa1db38f6b262318624f316e7e117baeebaedb8f","unresolved":false,"context_lines":[{"line_number":28,"context_line":"to be installed. This can also be achieved via pip:"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"```"},{"line_number":31,"context_line":"sudo pip install influxdb\u003d\u003d3.0.0"},{"line_number":32,"context_line":"```"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Alternatively, pip can be used to install the latest development version"}],"source_content_type":"text/x-gfm","patch_set":2,"id":"1a6eadb0_489d7864","line":31,"updated":"2016-12-13 15:34:57.000000000","message":"I was wondering why the specific version listed.","commit_id":"51c09b90f1960ae7ea1948cdd2de74c6c6099460"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"fa1db38f6b262318624f316e7e117baeebaedb8f","unresolved":false,"context_lines":[{"line_number":48,"context_line":"Using the persister requires that the following components of the Monasca"},{"line_number":49,"context_line":"system are deployed and available:"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"* Kafka (\u0026 Zookeeper)"},{"line_number":52,"context_line":"* InfluxDB"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"If running the persister as a daemon, it is good practice to create a"}],"source_content_type":"text/x-gfm","patch_set":2,"id":"1a6eadb0_b59f4502","line":51,"range":{"start_line":51,"start_character":8,"end_line":51,"end_character":10},"updated":"2016-12-13 15:34:57.000000000","message":"Could put Zookeeper on a separate line. I guess you are trying to imply that Kafka requires ZK, but might be better just to make it concrete.","commit_id":"51c09b90f1960ae7ea1948cdd2de74c6c6099460"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"fa1db38f6b262318624f316e7e117baeebaedb8f","unresolved":false,"context_lines":[{"line_number":55,"context_line":"dedicated system user for the purpose, for example:"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"```"},{"line_number":58,"context_line":"sudo useradd --system monasca-persister"},{"line_number":59,"context_line":"```"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"Additionally, it is good practice to give the daemon a dedicated working"}],"source_content_type":"text/x-gfm","patch_set":2,"id":"1a6eadb0_55e351a3","line":58,"updated":"2016-12-13 15:34:57.000000000","message":"It might be good to shorten this to mon-persister. I believe originally we use monasca-persister, but then tools such as top and others, couldn\u0027t display the username with the default column widths. Everything was truncated to just monasca- or something like that.\n\nSo, there isn\u0027t anything wrong with your directions, but just as a convention, mon-persister will work out better.\n\nAlso, I was wondering if it would be best to first add a \"monasca\" group and then add the monasca-persister to that group, similar to what is done in the DevStack plugin.\n\nIn monasca-api/devstack/plugin.sh we do the following:\n\nsudo groupadd --system monasca || true\nsudo useradd --system -g monasca mon-persister || true","commit_id":"51c09b90f1960ae7ea1948cdd2de74c6c6099460"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"fa1db38f6b262318624f316e7e117baeebaedb8f","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"```"},{"line_number":65,"context_line":"sudo mkdir /var/lib/monasca-persister"},{"line_number":66,"context_line":"sudo chown monasca-persister:monasca-persister /var/lib/monasca-persister"},{"line_number":67,"context_line":"```"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"The persister will write a log file in a location which can be changed"}],"source_content_type":"text/x-gfm","patch_set":2,"id":"1a6eadb0_a8dd2c44","line":66,"updated":"2016-12-13 15:34:57.000000000","message":"If you add a monasca group, then this line would change, as well as some others.","commit_id":"51c09b90f1960ae7ea1948cdd2de74c6c6099460"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"fa1db38f6b262318624f316e7e117baeebaedb8f","unresolved":false,"context_lines":[{"line_number":83,"context_line":"There is minimum amount of configuration which must be performed before"},{"line_number":84,"context_line":"running the persister. Begin with the sample configuration file and"},{"line_number":85,"context_line":"place it somewhere sensible. Note this examples we are assumes the"},{"line_number":86,"context_line":"package was installed in the standard location under Python 2.7."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"```"},{"line_number":89,"context_line":"sudo mkdir /etc/monasca"}],"source_content_type":"text/x-gfm","patch_set":2,"id":"1a6eadb0_e8a3a49c","line":86,"updated":"2016-12-13 15:34:57.000000000","message":"Note this example assumes that the package...","commit_id":"51c09b90f1960ae7ea1948cdd2de74c6c6099460"},{"author":{"_account_id":2419,"name":"Roland Hochmuth","email":"roland.hochmuth@hpe.com","username":"roland-hochmuth-s"},"change_message_id":"6340c17469bad637f0656d95ebc85b87044f78db","unresolved":false,"context_lines":[{"line_number":16,"context_line":"e.g. For Debian/Ubuntu:"},{"line_number":17,"context_line":"```"},{"line_number":18,"context_line":"sudo apt-get install python-pip"},{"line_number":19,"context_line":"```"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Now, to install a particular released version:"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":3,"id":"1a6eadb0_fa1ca66b","line":19,"updated":"2016-12-16 12:13:14.000000000","message":"There might be directions on this available from an official source, such as, https://pip.pypa.io/en/stable/installing/, that would be worth referencing.","commit_id":"72beba59643dfbf1d843da83497cb908472fe0ce"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"b2cce44aa9d7a69842a12cfb96cd148efe9dc5d7","unresolved":false,"context_lines":[{"line_number":71,"context_line":"directory, in the event it ever needs to write any files."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"```"},{"line_number":74,"context_line":"sudo mkdir /var/lib/monasca-persister"},{"line_number":75,"context_line":"sudo chown mon-persister:monasca /var/lib/monasca-persister"},{"line_number":76,"context_line":"```"},{"line_number":77,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":4,"id":"fa31d9ce_2f7718d4","line":74,"updated":"2017-02-14 09:56:57.000000000","message":"Same here (comment below) just for consistency","commit_id":"3ed6b5bcadda9ba07360e6a6a18eb428296fed17"},{"author":{"_account_id":24533,"name":"Steve Simpson","email":"steve@stackhpc.com","username":"stevejims"},"change_message_id":"0111815a9c159d77ea5a1631912c8ac8f52c7d9e","unresolved":false,"context_lines":[{"line_number":71,"context_line":"directory, in the event it ever needs to write any files."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"```"},{"line_number":74,"context_line":"sudo mkdir /var/lib/monasca-persister"},{"line_number":75,"context_line":"sudo chown mon-persister:monasca /var/lib/monasca-persister"},{"line_number":76,"context_line":"```"},{"line_number":77,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":4,"id":"fa31d9ce_eacde331","line":74,"in_reply_to":"fa31d9ce_2f7718d4","updated":"2017-02-14 14:08:02.000000000","message":"Done","commit_id":"3ed6b5bcadda9ba07360e6a6a18eb428296fed17"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"b2cce44aa9d7a69842a12cfb96cd148efe9dc5d7","unresolved":false,"context_lines":[{"line_number":80,"context_line":"be created as follows:"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"```"},{"line_number":83,"context_line":"sudo mkdir /var/log/monasca/persister"},{"line_number":84,"context_line":"sudo chown mon-persister:monasca /var/log/monasca/persister"},{"line_number":85,"context_line":"```"},{"line_number":86,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":4,"id":"fa31d9ce_0f7a54bd","line":83,"updated":"2017-02-14 09:56:57.000000000","message":"I\u0027d safely add -p\n\n  mkdir -p\n\nif /var/log/monasca does not exists, command will fail.","commit_id":"3ed6b5bcadda9ba07360e6a6a18eb428296fed17"},{"author":{"_account_id":24533,"name":"Steve Simpson","email":"steve@stackhpc.com","username":"stevejims"},"change_message_id":"0111815a9c159d77ea5a1631912c8ac8f52c7d9e","unresolved":false,"context_lines":[{"line_number":80,"context_line":"be created as follows:"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"```"},{"line_number":83,"context_line":"sudo mkdir /var/log/monasca/persister"},{"line_number":84,"context_line":"sudo chown mon-persister:monasca /var/log/monasca/persister"},{"line_number":85,"context_line":"```"},{"line_number":86,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":4,"id":"fa31d9ce_0acf9729","line":83,"in_reply_to":"fa31d9ce_0f7a54bd","updated":"2017-02-14 14:08:02.000000000","message":"Done","commit_id":"3ed6b5bcadda9ba07360e6a6a18eb428296fed17"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"b2cce44aa9d7a69842a12cfb96cd148efe9dc5d7","unresolved":false,"context_lines":[{"line_number":95,"context_line":"package was installed in the standard location under Python 2.7."},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"```"},{"line_number":98,"context_line":"sudo mkdir /etc/monasca"},{"line_number":99,"context_line":"sudo cp \\"},{"line_number":100,"context_line":"  /usr/local/lib/python2.7/dist-packages/monasca_persister/persister.conf \\"},{"line_number":101,"context_line":"  /etc/monasca"}],"source_content_type":"text/x-gfm","patch_set":4,"id":"fa31d9ce_cf42fc67","line":98,"updated":"2017-02-14 09:56:57.000000000","message":"-p","commit_id":"3ed6b5bcadda9ba07360e6a6a18eb428296fed17"},{"author":{"_account_id":24533,"name":"Steve Simpson","email":"steve@stackhpc.com","username":"stevejims"},"change_message_id":"0111815a9c159d77ea5a1631912c8ac8f52c7d9e","unresolved":false,"context_lines":[{"line_number":95,"context_line":"package was installed in the standard location under Python 2.7."},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"```"},{"line_number":98,"context_line":"sudo mkdir /etc/monasca"},{"line_number":99,"context_line":"sudo cp \\"},{"line_number":100,"context_line":"  /usr/local/lib/python2.7/dist-packages/monasca_persister/persister.conf \\"},{"line_number":101,"context_line":"  /etc/monasca"}],"source_content_type":"text/x-gfm","patch_set":4,"id":"fa31d9ce_6adad3e9","line":98,"in_reply_to":"fa31d9ce_cf42fc67","updated":"2017-02-14 14:08:02.000000000","message":"Done","commit_id":"3ed6b5bcadda9ba07360e6a6a18eb428296fed17"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"b2cce44aa9d7a69842a12cfb96cd148efe9dc5d7","unresolved":false,"context_lines":[{"line_number":97,"context_line":"```"},{"line_number":98,"context_line":"sudo mkdir /etc/monasca"},{"line_number":99,"context_line":"sudo cp \\"},{"line_number":100,"context_line":"  /usr/local/lib/python2.7/dist-packages/monasca_persister/persister.conf \\"},{"line_number":101,"context_line":"  /etc/monasca"},{"line_number":102,"context_line":"```"},{"line_number":103,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":4,"id":"fa31d9ce_924f4b6a","line":100,"updated":"2017-02-14 09:56:57.000000000","message":"Such file (in that or similar location) won\u0027t exist as setup.cfg for persister does not mention copying it anywhere.","commit_id":"3ed6b5bcadda9ba07360e6a6a18eb428296fed17"},{"author":{"_account_id":24533,"name":"Steve Simpson","email":"steve@stackhpc.com","username":"stevejims"},"change_message_id":"0111815a9c159d77ea5a1631912c8ac8f52c7d9e","unresolved":false,"context_lines":[{"line_number":97,"context_line":"```"},{"line_number":98,"context_line":"sudo mkdir /etc/monasca"},{"line_number":99,"context_line":"sudo cp \\"},{"line_number":100,"context_line":"  /usr/local/lib/python2.7/dist-packages/monasca_persister/persister.conf \\"},{"line_number":101,"context_line":"  /etc/monasca"},{"line_number":102,"context_line":"```"},{"line_number":103,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":4,"id":"fa31d9ce_aaa5ab47","line":100,"in_reply_to":"fa31d9ce_924f4b6a","updated":"2017-02-14 14:08:02.000000000","message":"That\u0027s odd, it exists in my installation.\n\nNever mind - I have changed the doc to refer to the file in source repository instead.","commit_id":"3ed6b5bcadda9ba07360e6a6a18eb428296fed17"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"e0694b583b9cd23bc0e5cab4d7344ef4fab05101","unresolved":false,"context_lines":[{"line_number":141,"context_line":"  --config-file /etc/monasca/persister.conf"},{"line_number":142,"context_line":"```"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"Note however that versions prior to revision 6733c0, the persister cannot be"},{"line_number":145,"context_line":"started via the ``/usr/local/bin/monasca-persister`` script, but as follows:"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"```"}],"source_content_type":"text/x-gfm","patch_set":6,"id":"fa31d9ce_515d31eb","line":144,"updated":"2017-02-17 12:08:17.000000000","message":"Maybe a better marker would be a tag here ?\n\nStill I am not really sure if that makes any sense to mention older monasca-persister, does it ?","commit_id":"a9a6031f70da85c7d31377a3400547ff83ed025f"},{"author":{"_account_id":24533,"name":"Steve Simpson","email":"steve@stackhpc.com","username":"stevejims"},"change_message_id":"92f8a4bec97ccd475e4aa4a0470503c39f15beb6","unresolved":false,"context_lines":[{"line_number":141,"context_line":"  --config-file /etc/monasca/persister.conf"},{"line_number":142,"context_line":"```"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"Note however that versions prior to revision 6733c0, the persister cannot be"},{"line_number":145,"context_line":"started via the ``/usr/local/bin/monasca-persister`` script, but as follows:"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"```"}],"source_content_type":"text/x-gfm","patch_set":6,"id":"ba2be162_e223b103","line":144,"in_reply_to":"fa31d9ce_515d31eb","updated":"2017-03-01 16:52:55.000000000","message":"Removed it","commit_id":"a9a6031f70da85c7d31377a3400547ff83ed025f"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"e0694b583b9cd23bc0e5cab4d7344ef4fab05101","unresolved":false,"context_lines":[{"line_number":166,"context_line":"Documentation\u003dhttps://github.com/openstack/monasca-persister/monasca-persister/README.md"},{"line_number":167,"context_line":"Requires\u003dnetwork.target remote-fs.target"},{"line_number":168,"context_line":"After\u003dnetwork.target remote-fs.target"},{"line_number":169,"context_line":"ConditionPathExists\u003d/etc/monasca/persister.conf"},{"line_number":170,"context_line":"ConditionPathExists\u003d/var/lib/monasca-persister"},{"line_number":171,"context_line":"ConditionPathExists\u003d/var/log/monasca/persister"},{"line_number":172,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":6,"id":"fa31d9ce_f1f545cb","line":169,"updated":"2017-02-17 12:08:17.000000000","message":"That I like","commit_id":"a9a6031f70da85c7d31377a3400547ff83ed025f"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"e0694b583b9cd23bc0e5cab4d7344ef4fab05101","unresolved":false,"context_lines":[{"line_number":176,"context_line":"User\u003dmon-persister"},{"line_number":177,"context_line":"Group\u003dmonasca"},{"line_number":178,"context_line":"WorkingDirectory\u003d/var/lib/monasca-persister"},{"line_number":179,"context_line":"ExecStart\u003d/usr/bin/python /usr/local/lib/python2.7/dist-packages/monasca_persister/persister.py --config-file /etc/monasca/persister.conf"},{"line_number":180,"context_line":"Restart\u003don-failure"},{"line_number":181,"context_line":"RestartSec\u003d5"},{"line_number":182,"context_line":"SyslogIdentifier\u003dmonasca-persister"}],"source_content_type":"text/x-gfm","patch_set":6,"id":"fa31d9ce_31699d8c","line":179,"updated":"2017-02-17 12:08:17.000000000","message":"Should it specify the proper monasca-persister binary ?\n\nI mean you mentioned above that running persister this way was the way prior to certain revision, right ? I;d not encourage this anymore here.","commit_id":"a9a6031f70da85c7d31377a3400547ff83ed025f"},{"author":{"_account_id":24533,"name":"Steve Simpson","email":"steve@stackhpc.com","username":"stevejims"},"change_message_id":"92f8a4bec97ccd475e4aa4a0470503c39f15beb6","unresolved":false,"context_lines":[{"line_number":176,"context_line":"User\u003dmon-persister"},{"line_number":177,"context_line":"Group\u003dmonasca"},{"line_number":178,"context_line":"WorkingDirectory\u003d/var/lib/monasca-persister"},{"line_number":179,"context_line":"ExecStart\u003d/usr/bin/python /usr/local/lib/python2.7/dist-packages/monasca_persister/persister.py --config-file /etc/monasca/persister.conf"},{"line_number":180,"context_line":"Restart\u003don-failure"},{"line_number":181,"context_line":"RestartSec\u003d5"},{"line_number":182,"context_line":"SyslogIdentifier\u003dmonasca-persister"}],"source_content_type":"text/x-gfm","patch_set":6,"id":"ba2be162_c26dcd33","line":179,"in_reply_to":"fa31d9ce_31699d8c","updated":"2017-03-01 16:52:55.000000000","message":"Fixed","commit_id":"a9a6031f70da85c7d31377a3400547ff83ed025f"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"65d3fceca01e9a104faef9239ec45e1a7003a29a","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"### Configuration"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"There is minimum amount of configuration which must be performed before"},{"line_number":93,"context_line":"running the persister. Begin with the sample configuration file,"},{"line_number":94,"context_line":"[persister.conf](../etc/monasca/persister.conf), and place it somewhere"},{"line_number":95,"context_line":"sensible."}],"source_content_type":"text/x-gfm","patch_set":7,"id":"ffe62b97_e3435a12","line":92,"updated":"2017-03-28 06:10:55.000000000","message":"Change where file is copied to the system during installation, has been merged. So I guess this paragraph requires some adjustment.","commit_id":"41a49e1da272aed32cd3bfcb3ea081c52aa11d63"},{"author":{"_account_id":24533,"name":"Steve Simpson","email":"steve@stackhpc.com","username":"stevejims"},"change_message_id":"593b46851203edc870f0d71f4b157a0f1487d181","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"### Configuration"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"There is minimum amount of configuration which must be performed before"},{"line_number":93,"context_line":"running the persister. Begin with the sample configuration file,"},{"line_number":94,"context_line":"[persister.conf](../etc/monasca/persister.conf), and place it somewhere"},{"line_number":95,"context_line":"sensible."}],"source_content_type":"text/x-gfm","patch_set":7,"id":"ffe62b97_a08f7d73","line":92,"in_reply_to":"ffe62b97_e3435a12","updated":"2017-03-28 10:01:47.000000000","message":"Done","commit_id":"41a49e1da272aed32cd3bfcb3ea081c52aa11d63"},{"author":{"_account_id":15178,"name":"Kamil Choroba","email":"kamil.choroba@est.fujitsu.com","username":"kchoroba"},"change_message_id":"e1912b12efe1736c60d3bca18c299f83cbf0020e","unresolved":false,"context_lines":[{"line_number":111,"context_line":"Most of the configuration options should be left at default, but at a"},{"line_number":112,"context_line":"minimum, the following should be changed:"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"```"},{"line_number":115,"context_line":"[zookeeper]"},{"line_number":116,"context_line":"uri \u003d \u003chost1\u003e:\u003cport1\u003e,\u003chost2\u003e:\u003cport2\u003e,..."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"[kafka_alarm_history]"},{"line_number":119,"context_line":"uri \u003d \u003chost1\u003e:\u003cport1\u003e,\u003chost2\u003e:\u003cport2\u003e,..."},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"[kafka_metrics]"},{"line_number":122,"context_line":"uri \u003d \u003chost1\u003e:\u003cport1\u003e,\u003chost2\u003e:\u003cport2\u003e,..."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"[influxdb]"},{"line_number":125,"context_line":"database_name \u003d"},{"line_number":126,"context_line":"ip_address \u003d"},{"line_number":127,"context_line":"port \u003d"},{"line_number":128,"context_line":"user \u003d"},{"line_number":129,"context_line":"password \u003d"},{"line_number":130,"context_line":"```"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"### Running"},{"line_number":133,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":7,"id":"1a1ced50_4aa677c2","line":130,"range":{"start_line":114,"start_character":0,"end_line":130,"end_character":3},"updated":"2017-03-17 11:58:32.000000000","message":"Do you think we should add an example file after this?","commit_id":"41a49e1da272aed32cd3bfcb3ea081c52aa11d63"},{"author":{"_account_id":24533,"name":"Steve Simpson","email":"steve@stackhpc.com","username":"stevejims"},"change_message_id":"5ea16f8e623f616ac2128a22f1865f5b46606e3b","unresolved":false,"context_lines":[{"line_number":111,"context_line":"Most of the configuration options should be left at default, but at a"},{"line_number":112,"context_line":"minimum, the following should be changed:"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"```"},{"line_number":115,"context_line":"[zookeeper]"},{"line_number":116,"context_line":"uri \u003d \u003chost1\u003e:\u003cport1\u003e,\u003chost2\u003e:\u003cport2\u003e,..."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"[kafka_alarm_history]"},{"line_number":119,"context_line":"uri \u003d \u003chost1\u003e:\u003cport1\u003e,\u003chost2\u003e:\u003cport2\u003e,..."},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"[kafka_metrics]"},{"line_number":122,"context_line":"uri \u003d \u003chost1\u003e:\u003cport1\u003e,\u003chost2\u003e:\u003cport2\u003e,..."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"[influxdb]"},{"line_number":125,"context_line":"database_name \u003d"},{"line_number":126,"context_line":"ip_address \u003d"},{"line_number":127,"context_line":"port \u003d"},{"line_number":128,"context_line":"user \u003d"},{"line_number":129,"context_line":"password \u003d"},{"line_number":130,"context_line":"```"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"### Running"},{"line_number":133,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":7,"id":"1a1ced50_b00c513c","line":130,"range":{"start_line":114,"start_character":0,"end_line":130,"end_character":3},"in_reply_to":"1a1ced50_4aa677c2","updated":"2017-03-21 11:32:18.000000000","message":"We could do - though I assume the persister.conf in the repository already serves as an example? The idea here was to illustrate the minimum configuration that might need to change for a particular deployment.","commit_id":"41a49e1da272aed32cd3bfcb3ea081c52aa11d63"},{"author":{"_account_id":15178,"name":"Kamil Choroba","email":"kamil.choroba@est.fujitsu.com","username":"kchoroba"},"change_message_id":"4f95d572d6efa2807b5fd42f5507a686a319cc7e","unresolved":false,"context_lines":[{"line_number":111,"context_line":"Most of the configuration options should be left at default, but at a"},{"line_number":112,"context_line":"minimum, the following should be changed:"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"```"},{"line_number":115,"context_line":"[zookeeper]"},{"line_number":116,"context_line":"uri \u003d \u003chost1\u003e:\u003cport1\u003e,\u003chost2\u003e:\u003cport2\u003e,..."},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"[kafka_alarm_history]"},{"line_number":119,"context_line":"uri \u003d \u003chost1\u003e:\u003cport1\u003e,\u003chost2\u003e:\u003cport2\u003e,..."},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"[kafka_metrics]"},{"line_number":122,"context_line":"uri \u003d \u003chost1\u003e:\u003cport1\u003e,\u003chost2\u003e:\u003cport2\u003e,..."},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"[influxdb]"},{"line_number":125,"context_line":"database_name \u003d"},{"line_number":126,"context_line":"ip_address \u003d"},{"line_number":127,"context_line":"port \u003d"},{"line_number":128,"context_line":"user \u003d"},{"line_number":129,"context_line":"password \u003d"},{"line_number":130,"context_line":"```"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"### Running"},{"line_number":133,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":7,"id":"ffe62b97_196a66b2","line":130,"range":{"start_line":114,"start_character":0,"end_line":130,"end_character":3},"in_reply_to":"1a1ced50_b00c513c","updated":"2017-03-27 08:58:03.000000000","message":"You are right","commit_id":"41a49e1da272aed32cd3bfcb3ea081c52aa11d63"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"a82cddf4b042b527b6c38aa12322e945ee9a0dc7","unresolved":false,"context_lines":[{"line_number":98,"context_line":"desirable to inhibit read access except for the monasca-persister group."},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"```"},{"line_number":101,"context_line":"sudo chown root:monasca-persister /etc/monasca/persister.conf"},{"line_number":102,"context_line":"sudo chmod 640 /etc/monasca/persister.conf"},{"line_number":103,"context_line":"```"},{"line_number":104,"context_line":""}],"source_content_type":"text/x-gfm","patch_set":9,"id":"ffe62b97_6cfc996f","line":101,"updated":"2017-03-29 11:37:22.000000000","message":"root:monasca","commit_id":"ffc36218cade73b61e5bfc5480cadaac2d0a7340"}]}
