)]}'
{"examples/osc-lib.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"554b46ec5ec82a1d373bd04582da73c472b94c9e","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    for mod in clientmanager.PLUGIN_MODULES:"},{"line_number":56,"context_line":"        default_version \u003d getattr(mod, \u0027DEFAULT_API_VERSION\u0027, None)"},{"line_number":57,"context_line":"        option \u003d mod.API_VERSION_OPTION.replace(\u0027os_\u0027, \u0027\u0027)"},{"line_number":58,"context_line":"        version_opt \u003d str(cloud.config.get(option, default_version))"},{"line_number":59,"context_line":"        if version_opt:"},{"line_number":60,"context_line":"            api \u003d mod.API_NAME"},{"line_number":61,"context_line":"            api_version[api] \u003d version_opt"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_25b3bd4e","line":58,"updated":"2015-12-03 13:16:41.000000000","message":"I guess my concern here is the defaults from cloud config don\u0027t line up exactly with what osc want at the moment.  Do they?","commit_id":"2969fe1dae8c8c63fe43dc829120bc053210cf25"},{"author":{"_account_id":18858,"name":"ashish singh","email":"ashishsingh2k8@gmail.com","username":"Ashishsingh2k8"},"change_message_id":"630c3011cbef908bd3401145999d7a95f7a51924","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    for mod in clientmanager.PLUGIN_MODULES:"},{"line_number":56,"context_line":"        default_version \u003d getattr(mod, \u0027DEFAULT_API_VERSION\u0027, None)"},{"line_number":57,"context_line":"        option \u003d mod.API_VERSION_OPTION.replace(\u0027os_\u0027, \u0027\u0027)"},{"line_number":58,"context_line":"        version_opt \u003d str(cloud.config.get(option, default_version))"},{"line_number":59,"context_line":"        if version_opt:"},{"line_number":60,"context_line":"            api \u003d mod.API_NAME"},{"line_number":61,"context_line":"            api_version[api] \u003d version_opt"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_0c5ca149","line":58,"in_reply_to":"9a8ffd7b_25b3bd4e","updated":"2015-12-03 14:02:39.000000000","message":"And with the current implementation, there is no compute API version here, but it is used below in the example, thus It throws error. So after this fix it will have all API versions included from which user may use any service with osc. Am I making sense?","commit_id":"2969fe1dae8c8c63fe43dc829120bc053210cf25"},{"author":{"_account_id":18858,"name":"ashish singh","email":"ashishsingh2k8@gmail.com","username":"Ashishsingh2k8"},"change_message_id":"4e4abebdc03c2c0f844b650ca24895cd5fbdb475","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    for mod in clientmanager.PLUGIN_MODULES:"},{"line_number":56,"context_line":"        default_version \u003d getattr(mod, \u0027DEFAULT_API_VERSION\u0027, None)"},{"line_number":57,"context_line":"        option \u003d mod.API_VERSION_OPTION.replace(\u0027os_\u0027, \u0027\u0027)"},{"line_number":58,"context_line":"        version_opt \u003d str(cloud.config.get(option, default_version))"},{"line_number":59,"context_line":"        if version_opt:"},{"line_number":60,"context_line":"            api \u003d mod.API_NAME"},{"line_number":61,"context_line":"            api_version[api] \u003d version_opt"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a8ffd7b_4c362952","line":58,"in_reply_to":"9a8ffd7b_25b3bd4e","updated":"2015-12-03 13:59:16.000000000","message":"yes, But then the user should be able to look example as a reference if he is going to write a client with osc. That is why I have included these defaults.","commit_id":"2969fe1dae8c8c63fe43dc829120bc053210cf25"}]}