)]}'
{"etc/tempest.conf.sample":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"6d40fa706327dd0db4d376fefb85fb568c4f2765","unresolved":false,"context_lines":[{"line_number":649,"context_line":"# Is the v3 identity API enabled (boolean value)"},{"line_number":650,"context_line":"#api_v3\u003dtrue"},{"line_number":651,"context_line":""},{"line_number":652,"context_line":"# If false, skip all identify api tests with xml (boolean"},{"line_number":653,"context_line":"# value)"},{"line_number":654,"context_line":"#xml_api\u003dfalse"},{"line_number":655,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":6,"id":"5a890539_1b756231","line":652,"updated":"2014-11-18 07:58:06.000000000","message":"This line break is not required. (Fails on pep8)","commit_id":"9b2a55e29d3e0fe89f017db17c135fce5e0d2646"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"22b56d156b048f147831b0e5709ca6f38e0f132c","unresolved":false,"context_lines":[{"line_number":649,"context_line":"# Is the v3 identity API enabled (boolean value)"},{"line_number":650,"context_line":"#api_v3\u003dtrue"},{"line_number":651,"context_line":""},{"line_number":652,"context_line":"# If false, skip all identify api tests with xml (boolean"},{"line_number":653,"context_line":"# value)"},{"line_number":654,"context_line":"#xml_api\u003dfalse"},{"line_number":655,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":6,"id":"5a890539_6fa4c008","line":652,"in_reply_to":"5a890539_1b756231","updated":"2014-11-18 21:49:28.000000000","message":"Done","commit_id":"9b2a55e29d3e0fe89f017db17c135fce5e0d2646"}],"tempest/api/identity/base.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"0ca8b8b63d2b9b6ec2f5fe99a45d5c74e70492dd","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    def resource_setup(cls):"},{"line_number":33,"context_line":"        super(BaseIdentityAdminTest, cls).resource_setup()"},{"line_number":34,"context_line":"        if getattr(cls, \u0027_interface\u0027, None) \u003d\u003d \u0027xml\u0027:"},{"line_number":35,"context_line":"            if not CONF.identity_feature_group.xml_api:"},{"line_number":36,"context_line":"                raise cls.skipException(\u0027XML API is not enabled\u0027)"},{"line_number":37,"context_line":"        cls.os_adm \u003d clients.AdminManager(interface\u003dcls._interface)"},{"line_number":38,"context_line":"        cls.os \u003d clients.Manager(interface\u003dcls._interface)"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_240cdfec","line":35,"updated":"2014-10-07 20:32:17.000000000","message":"This should be:\n\nif not CONF.identity_feature_enabled.xml_api:","commit_id":"1b2825c20b5a4c3dbbea0ffd52e3bafb4d405b59"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"24ebfa113820ba8aa867e28efa0ce0822b940cd3","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    def resource_setup(cls):"},{"line_number":33,"context_line":"        super(BaseIdentityAdminTest, cls).resource_setup()"},{"line_number":34,"context_line":"        if getattr(cls, \u0027_interface\u0027, None) \u003d\u003d \u0027xml\u0027:"},{"line_number":35,"context_line":"            if not CONF.identity_feature_group.xml_api:"},{"line_number":36,"context_line":"                raise cls.skipException(\u0027XML API is not enabled\u0027)"},{"line_number":37,"context_line":"        cls.os_adm \u003d clients.AdminManager(interface\u003dcls._interface)"},{"line_number":38,"context_line":"        cls.os \u003d clients.Manager(interface\u003dcls._interface)"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa201ad_ea562105","line":35,"in_reply_to":"baa201ad_240cdfec","updated":"2014-10-07 20:53:10.000000000","message":"Done, good catch","commit_id":"1b2825c20b5a4c3dbbea0ffd52e3bafb4d405b59"}],"tempest/config.py":[{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"f1e427b389a1575c663d831267498e1437b47bb8","unresolved":false,"context_lines":[{"line_number":124,"context_line":"                default\u003dTrue,"},{"line_number":125,"context_line":"                help\u003d\u0027Is the v3 identity API enabled\u0027),"},{"line_number":126,"context_line":"    cfg.BoolOpt(\u0027xml_api\u0027,"},{"line_number":127,"context_line":"                default\u003dFalse,"},{"line_number":128,"context_line":"                help\u003d\u0027If true, enable identity XML support\u0027),"},{"line_number":129,"context_line":"]"},{"line_number":130,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"baa201ad_5de6b845","line":127,"updated":"2014-10-10 19:32:36.000000000","message":"I think this needs to default to True in order to make icehouse continue to test xml.","commit_id":"0ebd97e1ac79bedfaa2b38119c686e4bed5be61d"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"e942d8b7582928167ce24c05e60a84b14cdce80d","unresolved":false,"context_lines":[{"line_number":124,"context_line":"                default\u003dTrue,"},{"line_number":125,"context_line":"                help\u003d\u0027Is the v3 identity API enabled\u0027),"},{"line_number":126,"context_line":"    cfg.BoolOpt(\u0027xml_api\u0027,"},{"line_number":127,"context_line":"                default\u003dFalse,"},{"line_number":128,"context_line":"                help\u003d\u0027If true, enable identity XML support\u0027),"},{"line_number":129,"context_line":"]"},{"line_number":130,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"baa201ad_7dfffcaa","line":127,"in_reply_to":"baa201ad_5de6b845","updated":"2014-10-10 19:37:29.000000000","message":"Done","commit_id":"0ebd97e1ac79bedfaa2b38119c686e4bed5be61d"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"4442fcf03bd66e35ee8f65163c335024f8efcef4","unresolved":false,"context_lines":[{"line_number":125,"context_line":"                help\u003d\u0027Is the v3 identity API enabled\u0027),"},{"line_number":126,"context_line":"    cfg.BoolOpt(\u0027xml_api\u0027,"},{"line_number":127,"context_line":"                default\u003dFalse,"},{"line_number":128,"context_line":"                help\u003d\u0027If true, enable identity XML support\u0027),"},{"line_number":129,"context_line":"]"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"compute_group \u003d cfg.OptGroup(name\u003d\u0027compute\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"baa201ad_c4cdcc32","line":128,"updated":"2014-10-08 04:06:20.000000000","message":"Tempest doesn\u0027t provide identify XML support. Keystone does.\n\nThe description of compute XML tests option is following:\n\n If false skip all v2 api tests with xml\n\nSo how about\n\n If false, skip all identify api tests with xml\n\nfor consistency?","commit_id":"0ebd97e1ac79bedfaa2b38119c686e4bed5be61d"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"e942d8b7582928167ce24c05e60a84b14cdce80d","unresolved":false,"context_lines":[{"line_number":125,"context_line":"                help\u003d\u0027Is the v3 identity API enabled\u0027),"},{"line_number":126,"context_line":"    cfg.BoolOpt(\u0027xml_api\u0027,"},{"line_number":127,"context_line":"                default\u003dFalse,"},{"line_number":128,"context_line":"                help\u003d\u0027If true, enable identity XML support\u0027),"},{"line_number":129,"context_line":"]"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"compute_group \u003d cfg.OptGroup(name\u003d\u0027compute\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"baa201ad_9d2bc032","line":128,"in_reply_to":"baa201ad_c4cdcc32","updated":"2014-10-10 19:37:29.000000000","message":"Done","commit_id":"0ebd97e1ac79bedfaa2b38119c686e4bed5be61d"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"f02e7143b1e5664ee1e7a539c4730874c6d3a3a6","unresolved":false,"context_lines":[{"line_number":124,"context_line":"                default\u003dTrue,"},{"line_number":125,"context_line":"                help\u003d\u0027Is the v3 identity API enabled\u0027),"},{"line_number":126,"context_line":"    cfg.BoolOpt(\u0027xml_api\u0027,"},{"line_number":127,"context_line":"                default\u003dFalse,"},{"line_number":128,"context_line":"                help\u003d\u0027If false, skip all identify api tests with xml\u0027),"},{"line_number":129,"context_line":"]"},{"line_number":130,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"baa201ad_ba678750","line":127,"updated":"2014-10-13 14:51:07.000000000","message":"Why do you change the default value to False?\nThat was David\u0027s point on PS3.","commit_id":"246fe3f944d42ead231ed2a44454befcd36dfb41"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"dc7433afbd7f853c10d93b2bf0722956493eb88d","unresolved":false,"context_lines":[{"line_number":124,"context_line":"                default\u003dTrue,"},{"line_number":125,"context_line":"                help\u003d\u0027Is the v3 identity API enabled\u0027),"},{"line_number":126,"context_line":"    cfg.BoolOpt(\u0027xml_api\u0027,"},{"line_number":127,"context_line":"                default\u003dFalse,"},{"line_number":128,"context_line":"                help\u003d\u0027If false, skip all identify api tests with xml\u0027),"},{"line_number":129,"context_line":"]"},{"line_number":130,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"baa201ad_c071bcdf","line":127,"in_reply_to":"baa201ad_ba678750","updated":"2014-10-13 15:57:29.000000000","message":"I had a conversation with David in -qa on Friday about this. \n\nhttp://eavesdrop.openstack.org/irclogs/%23openstack-qa/%23openstack-qa.2014-10-10.log\n\naround 2014-10-10T19:44:46","commit_id":"246fe3f944d42ead231ed2a44454befcd36dfb41"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"79eeb4d5310881030da5bfbe2401870bf86aafb2","unresolved":false,"context_lines":[{"line_number":124,"context_line":"                default\u003dTrue,"},{"line_number":125,"context_line":"                help\u003d\u0027Is the v3 identity API enabled\u0027),"},{"line_number":126,"context_line":"    cfg.BoolOpt(\u0027xml_api\u0027,"},{"line_number":127,"context_line":"                default\u003dFalse,"},{"line_number":128,"context_line":"                help\u003d\u0027If false, skip all identify api tests with xml\u0027),"},{"line_number":129,"context_line":"]"},{"line_number":130,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"baa201ad_83b33ee8","line":127,"in_reply_to":"baa201ad_c071bcdf","updated":"2014-10-13 16:15:34.000000000","message":"uhm, maybe the commit-id of devstack seems necessary in the commit message.\nI also do the same thing for Nova API on https://review.openstack.org/#/c/126815/ , and I got such comment from David.","commit_id":"246fe3f944d42ead231ed2a44454befcd36dfb41"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"9a7910d32f4a14326a8d88826e2bb98f56a6d5a0","unresolved":false,"context_lines":[{"line_number":145,"context_line":"                help\u003d\u0027Is the v3 identity API enabled\u0027),"},{"line_number":146,"context_line":"    cfg.BoolOpt(\u0027xml_api\u0027,"},{"line_number":147,"context_line":"                default\u003dFalse,"},{"line_number":148,"context_line":"                help\u003d\u0027If false, skip all identify api tests with xml\u0027),"},{"line_number":149,"context_line":"]"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"compute_group \u003d cfg.OptGroup(name\u003d\u0027compute\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a890539_13aeebe7","line":148,"updated":"2014-11-19 16:45:17.000000000","message":"change identify to identity","commit_id":"d550bc4a5f3ff243d6c604d8663571f680094237"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"9481a971c36c8ba2c22e1e2f1b50284fd832b7eb","unresolved":false,"context_lines":[{"line_number":145,"context_line":"                help\u003d\u0027Is the v3 identity API enabled\u0027),"},{"line_number":146,"context_line":"    cfg.BoolOpt(\u0027xml_api\u0027,"},{"line_number":147,"context_line":"                default\u003dFalse,"},{"line_number":148,"context_line":"                help\u003d\u0027If false, skip all identify api tests with xml\u0027),"},{"line_number":149,"context_line":"]"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"compute_group \u003d cfg.OptGroup(name\u003d\u0027compute\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"5a890539_f3b87fe7","line":148,"in_reply_to":"5a890539_13aeebe7","updated":"2014-11-19 16:50:43.000000000","message":"Done","commit_id":"d550bc4a5f3ff243d6c604d8663571f680094237"}]}
