)]}'
{"openstackclient/api/auth_plugin.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"614ea06c1ca9315ad3ebb031d0a945c392642702","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import logging"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from six.moves.urllib import parse as urlparse"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_config import cfg"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from keystoneclient.auth import base"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a80dd14_3aac5235","line":19,"updated":"2015-03-06 08:28:23.000000000","message":"group these 2 together","commit_id":"c9a8a8c0c64537a4133acfe7dca704251eefa03b"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"f73cd1a4e2a76cdca57d3d89526bef6e7c7fc3fc","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import logging"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from six.moves.urllib import parse as urlparse"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_config import cfg"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from keystoneclient.auth import base"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a80dd14_bdc11756","line":19,"in_reply_to":"9a80dd14_3aac5235","updated":"2015-03-06 10:33:26.000000000","message":"should these 3 actually be grouped together, since they are all 3rd party library","commit_id":"c9a8a8c0c64537a4133acfe7dca704251eefa03b"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"cb28c836ccb148881aa3755dc8e78944af0aa810","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import logging"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from six.moves.urllib import parse as urlparse"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_config import cfg"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from keystoneclient.auth import base"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a80dd14_b63931a6","line":19,"in_reply_to":"9a80dd14_bdc11756","updated":"2015-03-06 13:54:45.000000000","message":"Sure, if that\u0027s what it takes.  This was just copied from auth.py...","commit_id":"c9a8a8c0c64537a4133acfe7dca704251eefa03b"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"9d46aa3419175239476ce9e57ad14b489b13e26c","unresolved":false,"context_lines":[{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        options.extend(["},{"line_number":68,"context_line":"            # Maintain name \u0027url\u0027 for compatibility"},{"line_number":69,"context_line":"            cfg.StrOpt(\u0027url\u0027,"},{"line_number":70,"context_line":"                       help\u003d\u0027Specific service endpoint to use\u0027),"},{"line_number":71,"context_line":"            cfg.StrOpt(\u0027token\u0027,"},{"line_number":72,"context_line":"                       secret\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a80dd14_d0fd6e6f","line":69,"updated":"2015-03-10 03:22:42.000000000","message":"don\u0027t this need to be os_url and os_token to maintain backward compatibility?","commit_id":"749920bc65d4859eeaf44d16ae94f842d88af178"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"872e417aec4feef324e513c223615d54cac93796","unresolved":false,"context_lines":[{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        options.extend(["},{"line_number":68,"context_line":"            # Maintain name \u0027url\u0027 for compatibility"},{"line_number":69,"context_line":"            cfg.StrOpt(\u0027url\u0027,"},{"line_number":70,"context_line":"                       help\u003d\u0027Specific service endpoint to use\u0027),"},{"line_number":71,"context_line":"            cfg.StrOpt(\u0027token\u0027,"},{"line_number":72,"context_line":"                       secret\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a80dd14_44454341","line":69,"in_reply_to":"9a80dd14_d0fd6e6f","updated":"2015-03-10 03:49:42.000000000","message":"No, the auth plugins do not use the os- prefix, OSC adds that elsewhere.","commit_id":"749920bc65d4859eeaf44d16ae94f842d88af178"}],"openstackclient/common/clientmanager.py":[{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"872e43371f78ed98686343ca1cf636453dbb9769","unresolved":false,"context_lines":[{"line_number":120,"context_line":"        # password auth is requested."},{"line_number":121,"context_line":"        if (self.auth_plugin_name.endswith(\u0027password\u0027) and"},{"line_number":122,"context_line":"                not self._cli_options.os_password):"},{"line_number":123,"context_line":"            self._cli_options.os_password \u003d self._pw_callback()"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"        (auth_plugin, self._auth_params) \u003d auth.build_auth_params("},{"line_number":126,"context_line":"            self.auth_plugin_name,"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a80dd14_0487cbb5","line":123,"updated":"2015-03-10 04:26:09.000000000","message":"good catch","commit_id":"749920bc65d4859eeaf44d16ae94f842d88af178"}],"openstackclient/shell.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"614ea06c1ca9315ad3ebb031d0a945c392642702","unresolved":false,"context_lines":[{"line_number":188,"context_line":"            description,"},{"line_number":189,"context_line":"            version)"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"        # service token auth argument"},{"line_number":192,"context_line":"        parser.add_argument("},{"line_number":193,"context_line":"            \u0027--os-url\u0027,"},{"line_number":194,"context_line":"            metavar\u003d\u0027\u003curl\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a80dd14_5a82e694","side":"PARENT","line":191,"updated":"2015-03-06 08:28:23.000000000","message":"i\u0027m not super clear on why this was removed? the commit message doesn\u0027t really say why, should it be in a different commit?","commit_id":"dc00d7c758a431845750974cc10a2734e14f9d03"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"cb28c836ccb148881aa3755dc8e78944af0aa810","unresolved":false,"context_lines":[{"line_number":188,"context_line":"            description,"},{"line_number":189,"context_line":"            version)"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"        # service token auth argument"},{"line_number":192,"context_line":"        parser.add_argument("},{"line_number":193,"context_line":"            \u0027--os-url\u0027,"},{"line_number":194,"context_line":"            metavar\u003d\u0027\u003curl\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a80dd14_a32ad58d","side":"PARENT","line":191,"in_reply_to":"9a80dd14_5a82e694","updated":"2015-03-06 13:54:45.000000000","message":"This was here because the TokenEndpoint plugin was not found and therefore not picking up the option defined in the plugin.  I never figured out why until now.  Because of the way the options are ORed together, --os-token was picked up from the Token plugin.","commit_id":"dc00d7c758a431845750974cc10a2734e14f9d03"}]}
