)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16986,"name":"Shuquan Huang","email":"shuquan.huang@live.com","username":"shuquan"},"change_message_id":"de5c3b1af65301177c076c640fc61f5f1ed19dbb","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-01-07 01:50:26 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adds auth_version property to dashboard config"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #1531594"},{"line_number":10,"context_line":"Change-Id: I060a7ceb19c2c2872b065c94d31255b0dc1750fc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa69d971_70f4e886","line":8,"updated":"2016-01-07 10:58:22.000000000","message":"Can you put more detail on the commit messages here? Thanks","commit_id":"9bb6703675bd3552c5e8d00ebe8d01dcfbbc72da"}],"tempest/config.py":[{"author":{"_account_id":7179,"name":"Brad Pokorny","email":"brad_pokorny@symantec.com","username":"bpokorny"},"change_message_id":"4e60fe7e93ab99b98e4df3367d1f841e3f1bdd0e","unresolved":false,"context_lines":[{"line_number":971,"context_line":"               deprecated_for_removal\u003dTrue),"},{"line_number":972,"context_line":"    cfg.StrOpt(\u0027auth_version\u0027,"},{"line_number":973,"context_line":"               default\u003d\u0027v2\u0027,"},{"line_number":974,"context_line":"               help\u003d\"Identity API version to be used for authentication \""},{"line_number":975,"context_line":"                    \"used by Horizon Dashboard\"),"},{"line_number":976,"context_line":"]"},{"line_number":977,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_3b50ac31","line":974,"updated":"2016-01-06 23:04:42.000000000","message":"(nit) Suggest changing this help text to something like \"Identity API version the Horizon Dashboard will use for authentication\"","commit_id":"e3d9822364272d5630cf10823527eee248acdc33"},{"author":{"_account_id":6162,"name":"Dan Nguyen","email":"dan.nguyens.mail@gmail.com","username":"daniel-a-nguyen"},"change_message_id":"97148abcfba055270a9985d7fe1d5e13a5e2d9ec","unresolved":false,"context_lines":[{"line_number":971,"context_line":"               deprecated_for_removal\u003dTrue),"},{"line_number":972,"context_line":"    cfg.StrOpt(\u0027auth_version\u0027,"},{"line_number":973,"context_line":"               default\u003d\u0027v2\u0027,"},{"line_number":974,"context_line":"               help\u003d\"Identity API version to be used for authentication \""},{"line_number":975,"context_line":"                    \"used by Horizon Dashboard\"),"},{"line_number":976,"context_line":"]"},{"line_number":977,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_9ea666ac","line":974,"in_reply_to":"fa69d971_3b50ac31","updated":"2016-01-06 23:16:07.000000000","message":"Done","commit_id":"e3d9822364272d5630cf10823527eee248acdc33"}],"tempest/scenario/test_dashboard_basic_ops.py":[{"author":{"_account_id":7179,"name":"Brad Pokorny","email":"brad_pokorny@symantec.com","username":"bpokorny"},"change_message_id":"cdf132e2fcab9098349bdae12f02456fba702e71","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        if not CONF.service_available.horizon:"},{"line_number":73,"context_line":"            raise cls.skipException(\"Horizon support is required\")"},{"line_number":74,"context_line":"        if CONF.dashboard.auth_version !\u003d \u0027v2\u0027:"},{"line_number":75,"context_line":"            raise cls.skipException(\"Horizon auth version is incompatible\")"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    @classmethod"},{"line_number":78,"context_line":"    def setup_credentials(cls):"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa69d971_50032710","line":75,"updated":"2016-01-11 19:53:10.000000000","message":"I didn\u0027t think about this earlier, but it might be more instructive for the user to return a message like \"Horizon tempest tests don\u0027t currently support Keystone v3 authentication\".\n\nIf I understand this change correctly, we won\u0027t run any dashboard tests if the auth version isn\u0027t v2.","commit_id":"ed30c701dc3f2bbfbb4cf18eb7e456bd8bf63960"},{"author":{"_account_id":6162,"name":"Dan Nguyen","email":"dan.nguyens.mail@gmail.com","username":"daniel-a-nguyen"},"change_message_id":"f1f65c86ebed6f572fcdfa9b92cb1b52dbf547c3","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        if not CONF.service_available.horizon:"},{"line_number":73,"context_line":"            raise cls.skipException(\"Horizon support is required\")"},{"line_number":74,"context_line":"        if CONF.dashboard.auth_version !\u003d \u0027v2\u0027:"},{"line_number":75,"context_line":"            raise cls.skipException(\"Horizon auth version is incompatible\")"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    @classmethod"},{"line_number":78,"context_line":"    def setup_credentials(cls):"}],"source_content_type":"text/x-python","patch_set":6,"id":"da6ed579_6395fe3e","line":75,"in_reply_to":"fa69d971_50032710","updated":"2016-01-12 01:33:48.000000000","message":"This is the only test that I found so far that does\u0027t run when horizon is set to use v3","commit_id":"ed30c701dc3f2bbfbb4cf18eb7e456bd8bf63960"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"f9d3403346dd4fac697e41b195f71f5c76eb6db7","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        super(TestDashboardBasicOps, cls).skip_checks()"},{"line_number":72,"context_line":"        if not CONF.service_available.horizon:"},{"line_number":73,"context_line":"            raise cls.skipException(\"Horizon support is required\")"},{"line_number":74,"context_line":"        if CONF.dashboard.auth_version !\u003d \u0027v2\u0027:"},{"line_number":75,"context_line":"            raise cls.skipException(\"Horizon auth version is not compatible \""},{"line_number":76,"context_line":"                                    \"with this test, the user\u0027s domain name \""},{"line_number":77,"context_line":"                                    \"is required in Keystone v3\")"}],"source_content_type":"text/x-python","patch_set":7,"id":"da6ed579_d9f91f4f","line":74,"updated":"2016-01-12 13:46:03.000000000","message":"I think that some TODO message about adding support of V3 would be helpful here.","commit_id":"bc9356c7fbeca3a613fed2c2aa093666363a6082"},{"author":{"_account_id":6162,"name":"Dan Nguyen","email":"dan.nguyens.mail@gmail.com","username":"daniel-a-nguyen"},"change_message_id":"e244c5199cf6dd89d4ea15f1cfa52bddb8836b16","unresolved":false,"context_lines":[{"line_number":71,"context_line":"        super(TestDashboardBasicOps, cls).skip_checks()"},{"line_number":72,"context_line":"        if not CONF.service_available.horizon:"},{"line_number":73,"context_line":"            raise cls.skipException(\"Horizon support is required\")"},{"line_number":74,"context_line":"        if CONF.dashboard.auth_version !\u003d \u0027v2\u0027:"},{"line_number":75,"context_line":"            raise cls.skipException(\"Horizon auth version is not compatible \""},{"line_number":76,"context_line":"                                    \"with this test, the user\u0027s domain name \""},{"line_number":77,"context_line":"                                    \"is required in Keystone v3\")"}],"source_content_type":"text/x-python","patch_set":7,"id":"da6ed579_fae075fc","line":74,"in_reply_to":"da6ed579_d9f91f4f","updated":"2016-01-12 23:17:59.000000000","message":"good idea, the current plan is to refactor this test to allow both.  I\u0027ll push a new patch set soon.  thx for the comment!","commit_id":"bc9356c7fbeca3a613fed2c2aa093666363a6082"}]}