)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"24a8b2c9ef5cf977f7188385998fda1dff5d9568","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Solve warnings and errors from build_sphinx. Fix rst doc files"},{"line_number":10,"context_line":"and source code docstring issues."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug: #1557047"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ib59df2681c98716c786ff0edf6d7cfcc3e8fe5ec"},{"line_number":15,"context_line":"Signed-off-by: Danny Al-Gaaf \u003cdanny.al-gaaf@bisect.de\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa0719c6_301cf212","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":20},"updated":"2016-03-22 11:02:09.000000000","message":"There are still some warnings in `tox -e docs` output. Maybe Partial-Bug is more appropriate? I don\u0027t see that as a serious issue though - this patch fixes enormous number of warnings.","commit_id":"e11e98bf3aa5e5d3309019842d18a3874d6b54a4"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"62db1c6360cfa0b7335a25b398ef20ab4db2da64","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Solve warnings and errors from build_sphinx. Fix rst doc files"},{"line_number":10,"context_line":"and source code docstring issues."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug: #1557047"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ib59df2681c98716c786ff0edf6d7cfcc3e8fe5ec"},{"line_number":15,"context_line":"Signed-off-by: Danny Al-Gaaf \u003cdanny.al-gaaf@bisect.de\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa0719c6_ada255ac","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":20},"in_reply_to":"fa0719c6_301cf212","updated":"2016-03-22 16:16:33.000000000","message":"I see only some \"document isn\u0027t included in any toctree\" warnings atm, mostly documents in ./cinder/db/sqlalchemy/migrate_repo/versions/0*_placeholder.py no idea how to fix, but I assume it\u0027s not an issue (since all of them are the same without any significant code.","commit_id":"e11e98bf3aa5e5d3309019842d18a3874d6b54a4"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"24a8b2c9ef5cf977f7188385998fda1dff5d9568","unresolved":false,"context_lines":[{"line_number":12,"context_line":"Closes-Bug: #1557047"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ib59df2681c98716c786ff0edf6d7cfcc3e8fe5ec"},{"line_number":15,"context_line":"Signed-off-by: Danny Al-Gaaf \u003cdanny.al-gaaf@bisect.de\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa0719c6_f0cd5ab8","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":54},"updated":"2016-03-22 11:02:09.000000000","message":"Nothing seriously wrong here, but according to [1] we\u0027re not adding this to commit messages.\n\n[1] https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references","commit_id":"e11e98bf3aa5e5d3309019842d18a3874d6b54a4"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"62db1c6360cfa0b7335a25b398ef20ab4db2da64","unresolved":false,"context_lines":[{"line_number":12,"context_line":"Closes-Bug: #1557047"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ib59df2681c98716c786ff0edf6d7cfcc3e8fe5ec"},{"line_number":15,"context_line":"Signed-off-by: Danny Al-Gaaf \u003cdanny.al-gaaf@bisect.de\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa0719c6_072240b9","line":15,"range":{"start_line":15,"start_character":0,"end_line":15,"end_character":54},"in_reply_to":"fa0719c6_f0cd5ab8","updated":"2016-03-22 16:16:33.000000000","message":"My understanding is that it\u0027s not required, but also not an issue.","commit_id":"e11e98bf3aa5e5d3309019842d18a3874d6b54a4"}],"cinder/api/contrib/snapshot_manage.py":[{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"ee794b322c4d80a3b423d29f94a801e84779a712","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        \u0027ref\u0027 is bad."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        Optional elements to \u0027snapshot\u0027 are::"},{"line_number":80,"context_line":"        - `name` - Name for the new snapshot"},{"line_number":81,"context_line":"        - `description` - Description for the new snapshot"},{"line_number":82,"context_line":"        - `metadata` - Key/value pairs to be associated with the new napshot"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"        \"\"\""},{"line_number":85,"context_line":"        context \u003d req.environ[\u0027cinder.context\u0027]"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a18252c_6d99541f","line":82,"range":{"start_line":80,"start_character":4,"end_line":82,"end_character":76},"updated":"2016-04-13 15:54:40.000000000","message":"This is now more difficult to read in the source - which is where many (most?) devs spend their time. Can we get the white space back in without breaking the doc generation?","commit_id":"0ef06174ff6500831428a9846c746dab3c0dfb5e"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"b13d39bd62120707a6f22203e42bfd61748451d5","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        \u0027ref\u0027 is bad."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        Optional elements to \u0027snapshot\u0027 are::"},{"line_number":80,"context_line":"        - `name` - Name for the new snapshot"},{"line_number":81,"context_line":"        - `description` - Description for the new snapshot"},{"line_number":82,"context_line":"        - `metadata` - Key/value pairs to be associated with the new napshot"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"        \"\"\""},{"line_number":85,"context_line":"        context \u003d req.environ[\u0027cinder.context\u0027]"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a18252c_d121154f","line":82,"range":{"start_line":80,"start_character":4,"end_line":82,"end_character":76},"in_reply_to":"5a18252c_6d99541f","updated":"2016-04-13 21:45:41.000000000","message":"Done","commit_id":"0ef06174ff6500831428a9846c746dab3c0dfb5e"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"99573c7fe85b7a184358cfcc4e4868f96f5709b6","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        Optional elements to \u0027snapshot\u0027 are::"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"         name           A Name for the new snapshot."},{"line_number":81,"context_line":"         description    A Description for the new snapshot."},{"line_number":82,"context_line":"         metadata       Key/value pairs to be associated with the new napshot."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"        \"\"\""},{"line_number":85,"context_line":"        context \u003d req.environ[\u0027cinder.context\u0027]"}],"source_content_type":"text/x-python","patch_set":10,"id":"5a18252c_f99861c2","line":82,"range":{"start_line":80,"start_character":0,"end_line":82,"end_character":78},"updated":"2016-04-14 10:03:38.000000000","message":"Looks like you\u0027e made some typos?","commit_id":"98cdb7e5aa4f3a3e812462c1c0ea9f1542a966a0"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"aac85279d429f939e81a444e3faaa769b100b71e","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        Optional elements to \u0027snapshot\u0027 are::"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"         name           A Name for the new snapshot."},{"line_number":81,"context_line":"         description    A Description for the new snapshot."},{"line_number":82,"context_line":"         metadata       Key/value pairs to be associated with the new napshot."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"        \"\"\""},{"line_number":85,"context_line":"        context \u003d req.environ[\u0027cinder.context\u0027]"}],"source_content_type":"text/x-python","patch_set":10,"id":"5a18252c_2792b0f6","line":82,"range":{"start_line":80,"start_character":0,"end_line":82,"end_character":78},"in_reply_to":"5a18252c_a7b24004","updated":"2016-04-14 10:30:32.000000000","message":"Done","commit_id":"98cdb7e5aa4f3a3e812462c1c0ea9f1542a966a0"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"ac0fa1cc70c9a0abc041d7b12b78a858c92548bc","unresolved":false,"context_lines":[{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        Optional elements to \u0027snapshot\u0027 are::"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"         name           A Name for the new snapshot."},{"line_number":81,"context_line":"         description    A Description for the new snapshot."},{"line_number":82,"context_line":"         metadata       Key/value pairs to be associated with the new napshot."},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"        \"\"\""},{"line_number":85,"context_line":"        context \u003d req.environ[\u0027cinder.context\u0027]"}],"source_content_type":"text/x-python","patch_set":10,"id":"5a18252c_a7b24004","line":82,"range":{"start_line":80,"start_character":0,"end_line":82,"end_character":78},"in_reply_to":"5a18252c_f99861c2","updated":"2016-04-14 10:27:00.000000000","message":"Yes, accidental removal of the \u0027s\u0027 sorry","commit_id":"98cdb7e5aa4f3a3e812462c1c0ea9f1542a966a0"}],"cinder/api/contrib/volume_manage.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"92bd3138b852f6c2df19eea9c0755a8ab4ccbb03","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        - `metadata` - Key/value pairs to be associated with the new volume."},{"line_number":91,"context_line":"        - `availability_zone` - The availability zone to associate with \\"},{"line_number":92,"context_line":"        the new volume."},{"line_number":93,"context_line":"        - `bootable` - If set to True, marks the volume as bootable."},{"line_number":94,"context_line":"        \"\"\""},{"line_number":95,"context_line":"        context \u003d req.environ[\u0027cinder.context\u0027]"},{"line_number":96,"context_line":"        authorize(context)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_af5464ca","line":93,"updated":"2016-04-08 11:15:37.000000000","message":"This should be fixed without the need for \\ and preserving the indentation level as in other places","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"fb235e96813143f5ef498f50bf89b75300fde2a0","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        - `metadata` - Key/value pairs to be associated with the new volume."},{"line_number":91,"context_line":"        - `availability_zone` - The availability zone to associate with \\"},{"line_number":92,"context_line":"        the new volume."},{"line_number":93,"context_line":"        - `bootable` - If set to True, marks the volume as bootable."},{"line_number":94,"context_line":"        \"\"\""},{"line_number":95,"context_line":"        context \u003d req.environ[\u0027cinder.context\u0027]"},{"line_number":96,"context_line":"        authorize(context)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_e192e3f5","line":93,"in_reply_to":"9a061dce_af5464ca","updated":"2016-04-08 15:31:41.000000000","message":"Done","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"},{"author":{"_account_id":1207,"name":"Duncan Thomas","email":"duncan.thomas@gmail.com","username":"duncan-thomas"},"change_message_id":"ee794b322c4d80a3b423d29f94a801e84779a712","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        Optional elements to \u0027volume\u0027 are:"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        - `name` - A name for the new volume."},{"line_number":83,"context_line":"        - `description` - A description for the new volume."},{"line_number":84,"context_line":"        - `volume_type` - ID or name of a volume type to associate with"},{"line_number":85,"context_line":"          the new Cinder volume. Does not necessarily guarantee that the"},{"line_number":86,"context_line":"          managed volume will have the properties described in the"},{"line_number":87,"context_line":"          volume_type. The driver may choose to fail if it identifies that"},{"line_number":88,"context_line":"          the specified volume_type is not compatible with the backend"},{"line_number":89,"context_line":"          storage object."},{"line_number":90,"context_line":"        - `metadata` - Key/value pairs to be associated with the new volume."},{"line_number":91,"context_line":"        - `availability_zone` - The availability zone to associate with"},{"line_number":92,"context_line":"          the new volume."},{"line_number":93,"context_line":"        - `bootable` - If set to True, marks the volume as bootable."},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        \"\"\""},{"line_number":96,"context_line":"        context \u003d req.environ[\u0027cinder.context\u0027]"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a18252c_edd724bd","line":93,"range":{"start_line":82,"start_character":11,"end_line":93,"end_character":68},"updated":"2016-04-13 15:54:40.000000000","message":"This is now substantially harder to read in the source form. Can we re-introduce the white space without breaking the doc generation?","commit_id":"0ef06174ff6500831428a9846c746dab3c0dfb5e"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"b13d39bd62120707a6f22203e42bfd61748451d5","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        Optional elements to \u0027volume\u0027 are:"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        - `name` - A name for the new volume."},{"line_number":83,"context_line":"        - `description` - A description for the new volume."},{"line_number":84,"context_line":"        - `volume_type` - ID or name of a volume type to associate with"},{"line_number":85,"context_line":"          the new Cinder volume. Does not necessarily guarantee that the"},{"line_number":86,"context_line":"          managed volume will have the properties described in the"},{"line_number":87,"context_line":"          volume_type. The driver may choose to fail if it identifies that"},{"line_number":88,"context_line":"          the specified volume_type is not compatible with the backend"},{"line_number":89,"context_line":"          storage object."},{"line_number":90,"context_line":"        - `metadata` - Key/value pairs to be associated with the new volume."},{"line_number":91,"context_line":"        - `availability_zone` - The availability zone to associate with"},{"line_number":92,"context_line":"          the new volume."},{"line_number":93,"context_line":"        - `bootable` - If set to True, marks the volume as bootable."},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        \"\"\""},{"line_number":96,"context_line":"        context \u003d req.environ[\u0027cinder.context\u0027]"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a18252c_512d0545","line":93,"range":{"start_line":82,"start_character":11,"end_line":93,"end_character":68},"in_reply_to":"5a18252c_edd724bd","updated":"2016-04-13 21:45:41.000000000","message":"Done","commit_id":"0ef06174ff6500831428a9846c746dab3c0dfb5e"}],"cinder/backup/rpcapi.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"08e0d3c86f397e2f11a41d76c5f45084beaa0e00","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    API version history:"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    .. code-block:: none"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        1.0 - Initial version."},{"line_number":39,"context_line":"        1.1 - Changed methods to accept backup objects instead of IDs."}],"source_content_type":"text/x-python","patch_set":11,"id":"1a122d0e_1d4adc83","line":36,"range":{"start_line":36,"start_character":0,"end_line":36,"end_character":24},"updated":"2016-04-15 19:31:36.000000000","message":"Is code-block a thing for stuff like this?","commit_id":"1323339d3301de6410837bf9ef1c52042097f8ec"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"43383cf68375daf9123ec1454bc28bec1f91d1d7","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    API version history:"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    .. code-block:: none"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        1.0 - Initial version."},{"line_number":39,"context_line":"        1.1 - Changed methods to accept backup objects instead of IDs."}],"source_content_type":"text/x-python","patch_set":11,"id":"1a122d0e_6c26bbbb","line":36,"range":{"start_line":36,"start_character":0,"end_line":36,"end_character":24},"in_reply_to":"1a122d0e_1d4adc83","updated":"2016-04-16 06:58:59.000000000","message":"It was the easiest way to get his block fixed and well displayed in html without reformat the content","commit_id":"1323339d3301de6410837bf9ef1c52042097f8ec"}],"cinder/cmd/volume_usage_audit.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"92bd3138b852f6c2df19eea9c0755a8ab4ccbb03","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    Time periods are specified as \u0027hour\u0027, \u0027month\u0027, \u0027day\u0027 or \u0027year\u0027"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    - `hour` - previous hour. If run at 9:07am, will generate usage for \\"},{"line_number":29,"context_line":"    8-9am."},{"line_number":30,"context_line":"    - `month` - previous month. If the script is run April 1, it will \\"},{"line_number":31,"context_line":"    generate usages for March 1 through March 31."}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_2fdcd445","line":28,"updated":"2016-04-08 11:15:37.000000000","message":"This should be fixed without the need for \\ and preserving the indentation level as in other places","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"fb235e96813143f5ef498f50bf89b75300fde2a0","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    Time periods are specified as \u0027hour\u0027, \u0027month\u0027, \u0027day\u0027 or \u0027year\u0027"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    - `hour` - previous hour. If run at 9:07am, will generate usage for \\"},{"line_number":29,"context_line":"    8-9am."},{"line_number":30,"context_line":"    - `month` - previous month. If the script is run April 1, it will \\"},{"line_number":31,"context_line":"    generate usages for March 1 through March 31."}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_c1199f7b","line":28,"in_reply_to":"9a061dce_2fdcd445","updated":"2016-04-08 15:31:41.000000000","message":"Done","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"}],"cinder/hacking/checks.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"92bd3138b852f6c2df19eea9c0755a8ab4ccbb03","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    we shouldn\u0027t translate debug level logs."},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"    - This check assumes that \u0027LOG\u0027 is a logger."},{"line_number":139,"context_line":"    - Use filename so we can start enforcing this in specific folders \\"},{"line_number":140,"context_line":"    instead of needing to do so all at once."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"    N319"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_ef9f7c63","line":139,"updated":"2016-04-08 11:15:37.000000000","message":"This should be fixed without the need for \\ and preserving the indentation level as in other places.","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"fb235e96813143f5ef498f50bf89b75300fde2a0","unresolved":false,"context_lines":[{"line_number":136,"context_line":"    we shouldn\u0027t translate debug level logs."},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"    - This check assumes that \u0027LOG\u0027 is a logger."},{"line_number":139,"context_line":"    - Use filename so we can start enforcing this in specific folders \\"},{"line_number":140,"context_line":"    instead of needing to do so all at once."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"    N319"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_8836ace5","line":139,"in_reply_to":"9a061dce_ef9f7c63","updated":"2016-04-08 15:31:41.000000000","message":"Done","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"}],"cinder/scheduler/filters/instance_locality_filter.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"92bd3138b852f6c2df19eea9c0755a8ab4ccbb03","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    the instance and the volume are located on the same physical machine."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    In order to work:"},{"line_number":41,"context_line":"    - The Extended Server Attributes extension needs to be active in Nova \\"},{"line_number":42,"context_line":"    (this is by default), so that the \u0027OS-EXT-SRV-ATTR:host\u0027 property is \\"},{"line_number":43,"context_line":"    returned when requesting instance info."},{"line_number":44,"context_line":"    - Either an account with privileged rights for Nova must be \\"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_99bd6e05","line":41,"updated":"2016-04-08 11:15:37.000000000","message":"This should be fixed without the need for \\ and preserving the indentation level as in other places","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"fb235e96813143f5ef498f50bf89b75300fde2a0","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    the instance and the volume are located on the same physical machine."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    In order to work:"},{"line_number":41,"context_line":"    - The Extended Server Attributes extension needs to be active in Nova \\"},{"line_number":42,"context_line":"    (this is by default), so that the \u0027OS-EXT-SRV-ATTR:host\u0027 property is \\"},{"line_number":43,"context_line":"    returned when requesting instance info."},{"line_number":44,"context_line":"    - Either an account with privileged rights for Nova must be \\"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_158c7534","line":41,"in_reply_to":"9a061dce_99bd6e05","updated":"2016-04-08 15:31:41.000000000","message":"Done","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"}],"cinder/volume/drivers/hitachi/hnas_nfs.py":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"11f1d6a59fdaf039ab96147a8a0fba805be09ae4","unresolved":false,"context_lines":[{"line_number":152,"context_line":""},{"line_number":153,"context_line":"class HDSNFSDriver(nfs.NfsDriver):"},{"line_number":154,"context_line":"    \"\"\"Base class for Hitachi NFS driver."},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    Executes commands relating to Volumes."},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"    Version 1.0.0: Initial driver version"}],"source_content_type":"text/x-python","patch_set":2,"id":"1af94dfe_28b8c229","line":155,"updated":"2016-03-17 14:35:06.000000000","message":"Can you please add the: \nVersion history:\n\n.. code-block:: none\n\nHere as well?","commit_id":"72acbb192d650dafb595afb3ad85d321abeeb2be"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"2d707a8ba8a3ffe6eb6edc0e0a87467f6af2d8e3","unresolved":false,"context_lines":[{"line_number":152,"context_line":""},{"line_number":153,"context_line":"class HDSNFSDriver(nfs.NfsDriver):"},{"line_number":154,"context_line":"    \"\"\"Base class for Hitachi NFS driver."},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    Executes commands relating to Volumes."},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"    Version 1.0.0: Initial driver version"}],"source_content_type":"text/x-python","patch_set":2,"id":"1af94dfe_c3312a5c","line":155,"in_reply_to":"1af94dfe_28b8c229","updated":"2016-03-17 21:35:48.000000000","message":"Done","commit_id":"72acbb192d650dafb595afb3ad85d321abeeb2be"}],"doc/source/devref/replication.rst":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"92bd3138b852f6c2df19eea9c0755a8ab4ccbb03","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Additionally we have freeze/thaw methods that will act on the scheduler"},{"line_number":134,"context_line":"but may or may not require something from the driver::"},{"line_number":135,"context_line":"    "},{"line_number":136,"context_line":"    freeze_backend(self, context)"},{"line_number":137,"context_line":"    thaw_backend(self, context)"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a061dce_af574425","line":135,"updated":"2016-04-08 11:15:37.000000000","message":"Unnecessary spaces in this line.","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"fb235e96813143f5ef498f50bf89b75300fde2a0","unresolved":false,"context_lines":[{"line_number":132,"context_line":""},{"line_number":133,"context_line":"Additionally we have freeze/thaw methods that will act on the scheduler"},{"line_number":134,"context_line":"but may or may not require something from the driver::"},{"line_number":135,"context_line":"    "},{"line_number":136,"context_line":"    freeze_backend(self, context)"},{"line_number":137,"context_line":"    thaw_backend(self, context)"},{"line_number":138,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a061dce_35b9b9ed","line":135,"in_reply_to":"9a061dce_af574425","updated":"2016-04-08 15:31:41.000000000","message":"Done","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"}],"doc/source/devref/unit_tests.rst":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"92bd3138b852f6c2df19eea9c0755a8ab4ccbb03","unresolved":false,"context_lines":[{"line_number":11,"context_line":"There are a number of ways to run unit tests currently, and there\u0027s a combination"},{"line_number":12,"context_line":"of frameworks used depending on what commands you use.  The preferred method"},{"line_number":13,"context_line":"is to use tox, which calls ostestr via the tox.ini file.  To run all tests simply run::"},{"line_number":14,"context_line":"    "},{"line_number":15,"context_line":"    tox"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"This will create a virtual environment, load all the packages from test-requirements.txt"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a061dce_cf3438fd","line":14,"updated":"2016-04-08 11:15:37.000000000","message":"Unnecessary spaces in this line.","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"},{"author":{"_account_id":9064,"name":"Danny Al-Gaaf","email":"danny.al-gaaf@bisect.de","username":"dalgaaf"},"change_message_id":"fb235e96813143f5ef498f50bf89b75300fde2a0","unresolved":false,"context_lines":[{"line_number":11,"context_line":"There are a number of ways to run unit tests currently, and there\u0027s a combination"},{"line_number":12,"context_line":"of frameworks used depending on what commands you use.  The preferred method"},{"line_number":13,"context_line":"is to use tox, which calls ostestr via the tox.ini file.  To run all tests simply run::"},{"line_number":14,"context_line":"    "},{"line_number":15,"context_line":"    tox"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"This will create a virtual environment, load all the packages from test-requirements.txt"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a061dce_b594a944","line":14,"in_reply_to":"9a061dce_cf3438fd","updated":"2016-04-08 15:31:41.000000000","message":"Done","commit_id":"ee43f6e8b4be0cc5ebae88d0d7fb7d5c87a64f7d"}]}