)]}'
{"tempest/common/credentials_factory.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"41ece6eeccacef4fe1f3013fcaa8b8e456e4e610","unresolved":false,"context_lines":[{"line_number":219,"context_line":"    \u0027alt_user\u0027: (\u0027identity\u0027, \u0027alt\u0027)"},{"line_number":220,"context_line":"}"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"DEFAULT_PARAMS \u003d {"},{"line_number":223,"context_line":"    \u0027disable_ssl_certificate_validation\u0027:"},{"line_number":224,"context_line":"        CONF.identity.disable_ssl_certificate_validation,"},{"line_number":225,"context_line":"    \u0027ca_certs\u0027: CONF.identity.ca_certificates_file,"},{"line_number":226,"context_line":"    \u0027trace_requests\u0027: CONF.debug.trace_requests"},{"line_number":227,"context_line":"}"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"def get_configured_admin_credentials(fill_in\u003dTrue, identity_version\u003dNone):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f436f4f_a3ba7207","side":"PARENT","line":227,"range":{"start_line":222,"start_character":0,"end_line":227,"end_character":1},"updated":"2017-08-17 14:41:26.000000000","message":"I think we should go with deprecation policy here as we have already declared credentials_factory.py as stable interface for plugin. \n\nCurrently nobody is using this variable in plugin but still we should deprecated it first and then remove.","commit_id":"bb4b0391629fbf8db0983e9a719c22f7135573a5"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"f9f11b0b599d8aeeda76d8a81dbb13602a7fe2ef","unresolved":false,"context_lines":[{"line_number":219,"context_line":"    \u0027alt_user\u0027: (\u0027identity\u0027, \u0027alt\u0027)"},{"line_number":220,"context_line":"}"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"DEFAULT_PARAMS \u003d {"},{"line_number":223,"context_line":"    \u0027disable_ssl_certificate_validation\u0027:"},{"line_number":224,"context_line":"        CONF.identity.disable_ssl_certificate_validation,"},{"line_number":225,"context_line":"    \u0027ca_certs\u0027: CONF.identity.ca_certificates_file,"},{"line_number":226,"context_line":"    \u0027trace_requests\u0027: CONF.debug.trace_requests"},{"line_number":227,"context_line":"}"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"def get_configured_admin_credentials(fill_in\u003dTrue, identity_version\u003dNone):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f436f4f_7b630ab3","side":"PARENT","line":227,"range":{"start_line":222,"start_character":0,"end_line":227,"end_character":1},"in_reply_to":"9f436f4f_a3ba7207","updated":"2017-08-17 18:10:46.000000000","message":"\u003e Currently nobody is using this variable in plugin but still\n \u003e we should deprecated it first and then remove.\n\nDeprecation policy is for helping users. If nobody who needs to be help, the policy doesn\u0027t need to work.\nI know if we break our own policy temporary, it would be hard to keep it in long-term with some exceptions.\n\nTBH, I still am not sure we need to declare this module as stable interface because much configurations are contained and the configuration itself is flexible. That means it is hard to say that is stable for me.","commit_id":"bb4b0391629fbf8db0983e9a719c22f7135573a5"}]}