)]}'
{"nova/api/openstack/placement/handlers/resource_provider.py":[{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"5712b8a2e8610e77fba8682d090621caec67d065","unresolved":false,"context_lines":[{"line_number":188,"context_line":"    except db_exc.DBDuplicateEntry as exc:"},{"line_number":189,"context_line":"        raise webob.exc.HTTPConflict("},{"line_number":190,"context_line":"            _(\u0027Conflicting resource provider %(rp_uuid)s already exists.\u0027) %"},{"line_number":191,"context_line":"            {\u0027rp_uuid\u0027: uuid},"},{"line_number":192,"context_line":"            json_formatter\u003dutil.json_error_formatter)"},{"line_number":193,"context_line":"    except exception.ObjectActionError as exc:"},{"line_number":194,"context_line":"        raise webob.exc.HTTPBadRequest("}],"source_content_type":"text/x-python","patch_set":5,"id":"da36d5c6_1b4978d0","line":191,"updated":"2017-02-24 21:18:22.000000000","message":"I think previously, error message explained which key gets the duplicate entry. But this error message is misleading, conflict in name is also presented as uuid conflict like in the gabbi test \"try to create same name again\".","commit_id":"a6fd39e716f9270c25abec1421a278ff84fdd0cd"},{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"36595892ef31d48b7d60f93addcca1ee5f02b225","unresolved":false,"context_lines":[{"line_number":188,"context_line":"    except db_exc.DBDuplicateEntry as exc:"},{"line_number":189,"context_line":"        raise webob.exc.HTTPConflict("},{"line_number":190,"context_line":"            _(\u0027Conflicting resource provider %(rp_uuid)s already exists.\u0027) %"},{"line_number":191,"context_line":"            {\u0027rp_uuid\u0027: uuid},"},{"line_number":192,"context_line":"            json_formatter\u003dutil.json_error_formatter)"},{"line_number":193,"context_line":"    except exception.ObjectActionError as exc:"},{"line_number":194,"context_line":"        raise webob.exc.HTTPBadRequest("}],"source_content_type":"text/x-python","patch_set":5,"id":"da36d5c6_23d7b324","line":191,"in_reply_to":"da36d5c6_1b4978d0","updated":"2017-02-28 00:19:16.000000000","message":"Using name instead","commit_id":"a6fd39e716f9270c25abec1421a278ff84fdd0cd"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6c7223677f136e1c53a25cd0055a75af6046e13f","unresolved":false,"context_lines":[{"line_number":339,"context_line":"        resource_provider.save()"},{"line_number":340,"context_line":"    except db_exc.DBDuplicateEntry as exc:"},{"line_number":341,"context_line":"        raise webob.exc.HTTPConflict("},{"line_number":342,"context_line":"            _(\u0027Conflicting resource provider already exists: %(error)s\u0027) %"},{"line_number":343,"context_line":"            {\u0027error\u0027: exc},"},{"line_number":344,"context_line":"            json_formatter\u003dutil.json_error_formatter)"},{"line_number":345,"context_line":"    except exception.ObjectActionError as exc:"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba2be162_413226a8","line":342,"updated":"2017-03-04 14:06:04.000000000","message":"How about updating this one too?","commit_id":"beafc0b932450a17ace6ea4248a8377242ff3e07"},{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"61b041ef1743dec133b375d36a03715d60dd2408","unresolved":false,"context_lines":[{"line_number":339,"context_line":"        resource_provider.save()"},{"line_number":340,"context_line":"    except db_exc.DBDuplicateEntry as exc:"},{"line_number":341,"context_line":"        raise webob.exc.HTTPConflict("},{"line_number":342,"context_line":"            _(\u0027Conflicting resource provider already exists: %(error)s\u0027) %"},{"line_number":343,"context_line":"            {\u0027error\u0027: exc},"},{"line_number":344,"context_line":"            json_formatter\u003dutil.json_error_formatter)"},{"line_number":345,"context_line":"    except exception.ObjectActionError as exc:"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba2be162_ff65671d","line":342,"in_reply_to":"ba2be162_413226a8","updated":"2017-03-06 00:36:47.000000000","message":"Done","commit_id":"beafc0b932450a17ace6ea4248a8377242ff3e07"}],"nova/tests/functional/api/openstack/placement/gabbits/resource-provider.yaml":[{"author":{"_account_id":18337,"name":"Sujitha","email":"sujitha.neti@intel.com","username":"Sujitha"},"change_message_id":"5712b8a2e8610e77fba8682d090621caec67d065","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      uuid: ada30fb5-566d-4fe1-b43b-28a9e988790c"},{"line_number":65,"context_line":"  status: 409"},{"line_number":66,"context_line":"  response_strings:"},{"line_number":67,"context_line":"      - Conflicting resource provider ada30fb5-566d-4fe1-b43b-28a9e988790c already exists"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"- name: confirm the correct post"},{"line_number":70,"context_line":"  GET: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"da36d5c6_5b8bf01e","line":67,"updated":"2017-02-24 21:18:22.000000000","message":"Conflict in this case is due to resource provider name not uuid. This error message would be misleading saying the resource provider with that uuid already exists but actually resource provider with that name is duplicated.","commit_id":"a6fd39e716f9270c25abec1421a278ff84fdd0cd"},{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"36595892ef31d48b7d60f93addcca1ee5f02b225","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      uuid: ada30fb5-566d-4fe1-b43b-28a9e988790c"},{"line_number":65,"context_line":"  status: 409"},{"line_number":66,"context_line":"  response_strings:"},{"line_number":67,"context_line":"      - Conflicting resource provider ada30fb5-566d-4fe1-b43b-28a9e988790c already exists"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"- name: confirm the correct post"},{"line_number":70,"context_line":"  GET: /resource_providers/$ENVIRON[\u0027RP_UUID\u0027]"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"da36d5c6_a31c8314","line":67,"in_reply_to":"da36d5c6_5b8bf01e","updated":"2017-02-28 00:19:16.000000000","message":"Make sense, changed to name","commit_id":"a6fd39e716f9270c25abec1421a278ff84fdd0cd"}]}