)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"ea4f005b379b5fc2f056d2a1cfd1d2e91e0d3d67","unresolved":false,"context_lines":[{"line_number":17,"context_line":"  in pkg._fix_package_selections"},{"line_number":18,"context_line":"- Also added the exception handling block around utils.execute in"},{"line_number":19,"context_line":"  pkg._fix_package_selections so that correct exception could be visible."},{"line_number":20,"context_line":"- Added exception message to translations."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I8f7bff9dccb7521189bd0eb08bb4b9e67d514ff8"},{"line_number":23,"context_line":"Closes-Bug: #1392870"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ba7be1f8_07bf49d1","line":20,"updated":"2015-02-24 00:13:10.000000000","message":"Please update the commit message to remove that last bullet point.","commit_id":"579b240408d7214d682b6b01b6492aff0ef65667"}],"trove/guestagent/pkg.py":[{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"9afe136a035f7636cbcfdda4883033954a158852","unresolved":false,"context_lines":[{"line_number":273,"context_line":"                utils.execute(\"dpkg\", \"--configure\", \"-a\","},{"line_number":274,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":275,"context_line":"            except ProcessExecutionError:"},{"line_number":276,"context_line":"                raise PkgConfigureError(\"Package cannot be cofigured.\")"},{"line_number":277,"context_line":"            finally:"},{"line_number":278,"context_line":"                os.remove(fname)"},{"line_number":279,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a961159_74d027c9","line":276,"updated":"2014-12-15 14:27:09.000000000","message":"NIT: s/cofigured/configured","commit_id":"9e5a296b380ed9be6ce6576c4edfe11f874a9523"},{"author":{"_account_id":10295,"name":"mariam john","email":"mariamj@us.ibm.com","username":"mariamj"},"change_message_id":"d1b5967ec1ef154a4c1b98417a982a8ab3f354eb","unresolved":false,"context_lines":[{"line_number":273,"context_line":"                utils.execute(\"dpkg\", \"--configure\", \"-a\","},{"line_number":274,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":275,"context_line":"            except ProcessExecutionError:"},{"line_number":276,"context_line":"                raise PkgConfigureError(\"Package cannot be configured.\")"},{"line_number":277,"context_line":"            finally:"},{"line_number":278,"context_line":"                os.remove(fname)"},{"line_number":279,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_637c7d21","line":276,"updated":"2014-12-18 11:23:59.000000000","message":"Doesnt this message need to be translated? Also I think it will be helpful to specify the package name. Thanks.","commit_id":"a59bd4edc884cc95f76773597fd1de6cba9711d5"},{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"5f0b7c2d8e3ae30cda834398f9cd6461696ffff5","unresolved":false,"context_lines":[{"line_number":273,"context_line":"                utils.execute(\"dpkg\", \"--configure\", \"-a\","},{"line_number":274,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":275,"context_line":"            except ProcessExecutionError:"},{"line_number":276,"context_line":"                raise PkgConfigureError(\"Package cannot be configured.\")"},{"line_number":277,"context_line":"            finally:"},{"line_number":278,"context_line":"                os.remove(fname)"},{"line_number":279,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_34f709f7","line":276,"in_reply_to":"3a961159_393f04d8","updated":"2014-12-18 12:54:08.000000000","message":"OK, i got your point, just adding it to translation.","commit_id":"a59bd4edc884cc95f76773597fd1de6cba9711d5"},{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"4d81648d3b50ed0931bcfc1d4049e97664830ee6","unresolved":false,"context_lines":[{"line_number":273,"context_line":"                utils.execute(\"dpkg\", \"--configure\", \"-a\","},{"line_number":274,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":275,"context_line":"            except ProcessExecutionError:"},{"line_number":276,"context_line":"                raise PkgConfigureError(\"Package cannot be configured.\")"},{"line_number":277,"context_line":"            finally:"},{"line_number":278,"context_line":"                os.remove(fname)"},{"line_number":279,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3a961159_393f04d8","line":276,"in_reply_to":"3a961159_637c7d21","updated":"2014-12-18 12:47:25.000000000","message":"Since this is not a message from LOG/logging, so I guess this does not needs to be translated.","commit_id":"a59bd4edc884cc95f76773597fd1de6cba9711d5"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"56d9f11f5ba150b2c928030a531dd81f72aca2cb","unresolved":false,"context_lines":[{"line_number":270,"context_line":"            try:"},{"line_number":271,"context_line":"                utils.execute(\"debconf-set-selections\", fname,"},{"line_number":272,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":273,"context_line":"                utils.execute(\"dpkg\", \"--configure\", \"-a\","},{"line_number":274,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":275,"context_line":"            except ProcessExecutionError:"},{"line_number":276,"context_line":"                raise PkgConfigureError(\"Error configuring package.\")"}],"source_content_type":"text/x-python","patch_set":4,"id":"da86d52c_f3ea37ab","line":273,"updated":"2015-02-15 02:15:44.000000000","message":"shouldn\u0027t this be done on a per-os basis? What would happen on a non debian operating system?\n\nI realize that existing code had this ... but is it worth fixing?","commit_id":"c288a311aa18ad02867ebf8a08d7fb369b0355f2"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"cc60cc25a5800e89b1e706e0d5b3a9a13672ce50","unresolved":false,"context_lines":[{"line_number":268,"context_line":"                fname \u003d f.name"},{"line_number":269,"context_line":"                f.write(selections)"},{"line_number":270,"context_line":"            try:"},{"line_number":271,"context_line":"                utils.execute(\"debconf-set-selections\", fname,"},{"line_number":272,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":273,"context_line":"                utils.execute(\"dpkg\", \"--configure\", \"-a\","},{"line_number":274,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_34ee8eed","line":271,"updated":"2015-02-20 12:30:10.000000000","message":"earlier question about operating systems remains","commit_id":"579b240408d7214d682b6b01b6492aff0ef65667"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"ea4f005b379b5fc2f056d2a1cfd1d2e91e0d3d67","unresolved":false,"context_lines":[{"line_number":268,"context_line":"                fname \u003d f.name"},{"line_number":269,"context_line":"                f.write(selections)"},{"line_number":270,"context_line":"            try:"},{"line_number":271,"context_line":"                utils.execute(\"debconf-set-selections\", fname,"},{"line_number":272,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":273,"context_line":"                utils.execute(\"dpkg\", \"--configure\", \"-a\","},{"line_number":274,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_27dfed3c","line":271,"in_reply_to":"ba7be1f8_34ee8eed","updated":"2015-02-24 00:13:10.000000000","message":"@Amrith -- this is done on a per OS basis.\nThis change is for the DebianPackagerMixin -- no changes are required for other packager mixins since this doesn\u0027t apply to them. \n\nPerhaps gerrit folded too much code that made it look like this was part of the BasePackagerMixin, and hence the confusion?","commit_id":"579b240408d7214d682b6b01b6492aff0ef65667"},{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"47e79a6f2a95fc21d5f8b5e1812df033e74f92ef","unresolved":false,"context_lines":[{"line_number":268,"context_line":"                fname \u003d f.name"},{"line_number":269,"context_line":"                f.write(selections)"},{"line_number":270,"context_line":"            try:"},{"line_number":271,"context_line":"                utils.execute(\"debconf-set-selections\", fname,"},{"line_number":272,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"},{"line_number":273,"context_line":"                utils.execute(\"dpkg\", \"--configure\", \"-a\","},{"line_number":274,"context_line":"                              run_as_root\u003dTrue, root_helper\u003d\"sudo\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba7be1f8_e146f722","line":271,"in_reply_to":"ba7be1f8_34ee8eed","updated":"2015-02-20 13:15:35.000000000","message":"i was not able to find the complimentary method for other os class","commit_id":"579b240408d7214d682b6b01b6492aff0ef65667"}],"trove/locale/trove.pot":[{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"5525f3fc184141b96812c792c17fa8d9b4d35b3f","unresolved":false,"context_lines":[{"line_number":1160,"context_line":"msgid \"Error updating the apt sources\""},{"line_number":1161,"context_line":"msgstr \"\""},{"line_number":1162,"context_line":""},{"line_number":1163,"context_line":"#: trove/guestagent/pkg.py:276"},{"line_number":1164,"context_line":"msgid \"Error configuring package.\""},{"line_number":1165,"context_line":"msgstr \"\""},{"line_number":1166,"context_line":""}],"source_content_type":"application/mspowerpoint","patch_set":4,"id":"da86d52c_5c97d8eb","line":1163,"updated":"2015-02-12 20:17:09.000000000","message":"We shouldn\u0027t translate exceptions https://wiki.openstack.org/wiki/Translations#What_To_Translate.\n\nAnd, in any case, if you want to add a translation, you shouldn\u0027t modify the .pot file but enclose the translatable string with _() (the wrapper for oslo.i18n).","commit_id":"c288a311aa18ad02867ebf8a08d7fb369b0355f2"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"56d9f11f5ba150b2c928030a531dd81f72aca2cb","unresolved":false,"context_lines":[{"line_number":1160,"context_line":"msgid \"Error updating the apt sources\""},{"line_number":1161,"context_line":"msgstr \"\""},{"line_number":1162,"context_line":""},{"line_number":1163,"context_line":"#: trove/guestagent/pkg.py:276"},{"line_number":1164,"context_line":"msgid \"Error configuring package.\""},{"line_number":1165,"context_line":"msgstr \"\""},{"line_number":1166,"context_line":""}],"source_content_type":"application/mspowerpoint","patch_set":4,"id":"da86d52c_9329fbf4","line":1163,"in_reply_to":"da86d52c_5c97d8eb","updated":"2015-02-15 02:15:44.000000000","message":"We shouldn\u0027t change this file, I think it is handled automatically.","commit_id":"c288a311aa18ad02867ebf8a08d7fb369b0355f2"}]}