)]}'
{"nova/scheduler/client/report.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0796448e68980633c5d4331ead7c3bbeb52f7ed8","unresolved":false,"context_lines":[{"line_number":71,"context_line":"                    \u0027Placement is optional in Newton, but required \u0027"},{"line_number":72,"context_line":"                    \u0027in Ocata. Please enable the placement service \u0027"},{"line_number":73,"context_line":"                    \u0027before upgrading.\u0027))"},{"line_number":74,"context_line":"        except ks_exc.DiscoveryFailure:"},{"line_number":75,"context_line":"            # TODO(_gryf): Looks like DiscoveryFailure is not the only missing"},{"line_number":76,"context_line":"            # exception here. In Pike we should take care about keystoneauth1"},{"line_number":77,"context_line":"            # failures handling globally."}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_43861366","line":74,"updated":"2017-01-20 21:53:29.000000000","message":"We probably also want this in the nova-status upgrade check command (I know, this is all duplication and it sucks):\n\nhttps://github.com/openstack/nova/blob/72390adaf2629653b4e336dbd4c13248b3ef1bb2/nova/cmd/status.py#L189","commit_id":"0a0f78fcfcbbaf1cd2cad9c37b5875d50965e450"},{"author":{"_account_id":13692,"name":"Roman Dobosz","email":"gryf73@gmail.com","username":"gryf"},"change_message_id":"3fdd1d171fb6419c2950dc7c42db236b127fba0d","unresolved":false,"context_lines":[{"line_number":71,"context_line":"                    \u0027Placement is optional in Newton, but required \u0027"},{"line_number":72,"context_line":"                    \u0027in Ocata. Please enable the placement service \u0027"},{"line_number":73,"context_line":"                    \u0027before upgrading.\u0027))"},{"line_number":74,"context_line":"        except ks_exc.DiscoveryFailure:"},{"line_number":75,"context_line":"            # TODO(_gryf): Looks like DiscoveryFailure is not the only missing"},{"line_number":76,"context_line":"            # exception here. In Pike we should take care about keystoneauth1"},{"line_number":77,"context_line":"            # failures handling globally."}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_431fea6b","line":74,"in_reply_to":"5a3905b3_43861366","updated":"2017-01-21 08:03:29.000000000","message":"Roger that.","commit_id":"0a0f78fcfcbbaf1cd2cad9c37b5875d50965e450"}]}