)]}'
{"openstack/auth/identity/v2.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"1dbfce55add6f3415d097f04f27ee91df3dd14a0","unresolved":false,"context_lines":[{"line_number":44,"context_line":"class Auth(base.BaseIdentityPlugin):"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    #: Valid options for this plugin"},{"line_number":47,"context_line":"    valid_options \u003d ["},{"line_number":48,"context_line":"        \u0027access_info\u0027,"},{"line_number":49,"context_line":"        \u0027auth_url\u0027,"},{"line_number":50,"context_line":"        \u0027user_name\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_6724bc0e","line":47,"updated":"2015-02-15 16:26:01.000000000","message":"Should update these to match the args, same with the v3 one","commit_id":"cdcca5eb5e334136317a0f9fddd6ddcb68ca9ae4"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"f49f99e84a80502ad98f31540c66480714d77978","unresolved":false,"context_lines":[{"line_number":44,"context_line":"class Auth(base.BaseIdentityPlugin):"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    #: Valid options for this plugin"},{"line_number":47,"context_line":"    valid_options \u003d ["},{"line_number":48,"context_line":"        \u0027access_info\u0027,"},{"line_number":49,"context_line":"        \u0027auth_url\u0027,"},{"line_number":50,"context_line":"        \u0027user_name\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_c288a324","line":47,"in_reply_to":"da86d52c_6724bc0e","updated":"2015-02-17 22:12:23.000000000","message":"Done","commit_id":"cdcca5eb5e334136317a0f9fddd6ddcb68ca9ae4"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"1dbfce55add6f3415d097f04f27ee91df3dd14a0","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        auth \u003d {\u0027password\u0027: self.password}"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"        if self.user_name:"},{"line_number":144,"context_line":"            auth[\u0027user_name\u0027] \u003d self.user_name"},{"line_number":145,"context_line":"        elif self.user_id:"},{"line_number":146,"context_line":"            auth[\u0027userId\u0027] \u003d self.user_id"},{"line_number":147,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_a7e314a3","line":144,"updated":"2015-02-15 16:26:01.000000000","message":"This should be username without the underscore","commit_id":"cdcca5eb5e334136317a0f9fddd6ddcb68ca9ae4"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"f49f99e84a80502ad98f31540c66480714d77978","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        auth \u003d {\u0027password\u0027: self.password}"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"        if self.user_name:"},{"line_number":144,"context_line":"            auth[\u0027user_name\u0027] \u003d self.user_name"},{"line_number":145,"context_line":"        elif self.user_id:"},{"line_number":146,"context_line":"            auth[\u0027userId\u0027] \u003d self.user_id"},{"line_number":147,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_4295933e","line":144,"in_reply_to":"da86d52c_a7e314a3","updated":"2015-02-17 22:12:23.000000000","message":"Done","commit_id":"cdcca5eb5e334136317a0f9fddd6ddcb68ca9ae4"}],"openstack/auth/identity/v3.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"1dbfce55add6f3415d097f04f27ee91df3dd14a0","unresolved":false,"context_lines":[{"line_number":171,"context_line":"    the factory method and don\u0027t work as well with AuthConstructors."},{"line_number":172,"context_line":"    \"\"\""},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"    _method_parameters \u003d []"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"    def __init__(self, **kwargs):"},{"line_number":177,"context_line":"        for param in self._method_parameters:"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_a775d4fb","line":174,"updated":"2015-02-15 16:26:01.000000000","message":"Is there any sense in making this valid_options and keeping it public, similar to how we have on the Auth classes?","commit_id":"cdcca5eb5e334136317a0f9fddd6ddcb68ca9ae4"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"4263f68fde686da80be34d433544f4f493bf7965","unresolved":false,"context_lines":[{"line_number":171,"context_line":"    the factory method and don\u0027t work as well with AuthConstructors."},{"line_number":172,"context_line":"    \"\"\""},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"    _method_parameters \u003d []"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"    def __init__(self, **kwargs):"},{"line_number":177,"context_line":"        for param in self._method_parameters:"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_76c1797f","line":174,"in_reply_to":"da86d52c_626a971b","updated":"2015-02-17 23:32:09.000000000","message":"Makes sense","commit_id":"cdcca5eb5e334136317a0f9fddd6ddcb68ca9ae4"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"f49f99e84a80502ad98f31540c66480714d77978","unresolved":false,"context_lines":[{"line_number":171,"context_line":"    the factory method and don\u0027t work as well with AuthConstructors."},{"line_number":172,"context_line":"    \"\"\""},{"line_number":173,"context_line":""},{"line_number":174,"context_line":"    _method_parameters \u003d []"},{"line_number":175,"context_line":""},{"line_number":176,"context_line":"    def __init__(self, **kwargs):"},{"line_number":177,"context_line":"        for param in self._method_parameters:"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_626a971b","line":174,"in_reply_to":"da86d52c_a775d4fb","updated":"2015-02-17 22:12:23.000000000","message":"I\u0027d sooner leave this for now to try and minimize the differences between ksc and the sdk.  There are differences already, but if we want to try and merge again, it will make it harder.","commit_id":"cdcca5eb5e334136317a0f9fddd6ddcb68ca9ae4"}],"openstack/tests/auth/identity/test_v2.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"1dbfce55add6f3415d097f04f27ee91df3dd14a0","unresolved":false,"context_lines":[{"line_number":26,"context_line":"class TestV2Auth(testtools.TestCase):"},{"line_number":27,"context_line":"    def test_password(self):"},{"line_number":28,"context_line":"        kargs \u003d {\u0027trust_id\u0027: common.TEST_TRUST_ID,"},{"line_number":29,"context_line":"                 \u0027tenant_id\u0027: common.TEST_TENANT_ID,"},{"line_number":30,"context_line":"                 \u0027tenant_name\u0027: common.TEST_TENANT_NAME}"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"        sot \u003d v2.Password(TEST_URL, common.TEST_USER, common.TEST_PASS,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_475ed877","line":29,"updated":"2015-02-15 16:26:01.000000000","message":"Since we\u0027re to be converging on project instead of tenant elsewhere, should we do that here?","commit_id":"cdcca5eb5e334136317a0f9fddd6ddcb68ca9ae4"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"f49f99e84a80502ad98f31540c66480714d77978","unresolved":false,"context_lines":[{"line_number":26,"context_line":"class TestV2Auth(testtools.TestCase):"},{"line_number":27,"context_line":"    def test_password(self):"},{"line_number":28,"context_line":"        kargs \u003d {\u0027trust_id\u0027: common.TEST_TRUST_ID,"},{"line_number":29,"context_line":"                 \u0027tenant_id\u0027: common.TEST_TENANT_ID,"},{"line_number":30,"context_line":"                 \u0027tenant_name\u0027: common.TEST_TENANT_NAME}"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"        sot \u003d v2.Password(TEST_URL, common.TEST_USER, common.TEST_PASS,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_82f86b4b","line":29,"in_reply_to":"da86d52c_475ed877","updated":"2015-02-17 22:12:23.000000000","message":"I added in some comapatibility to support either for now.","commit_id":"cdcca5eb5e334136317a0f9fddd6ddcb68ca9ae4"}]}