)]}'
{"ceilometer/alarm/service.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"3cfe48fcd87622109515f001d451d1784d88bbf0","unresolved":false,"context_lines":[{"line_number":84,"context_line":"                insecure\u003dauth_config.insecure,"},{"line_number":85,"context_line":"                timeout\u003dcfg.CONF.http_timeout,"},{"line_number":86,"context_line":"                os_user_domain_id\u003dauth_config.os_user_domain_id,"},{"line_number":87,"context_line":"                os_project_name\u003dauth_config.os_project_name,"},{"line_number":88,"context_line":"                os_project_domain_id\u003dauth_config.os_project_domain_id,"},{"line_number":89,"context_line":"            )"},{"line_number":90,"context_line":"            self.api_client \u003d ceiloclient.get_client(2, **creds)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_b721444f","line":87,"updated":"2015-07-28 17:44:43.000000000","message":"why do we need project_name and tenant_name?","commit_id":"0b09e570906479aeee5aa19199e81ce8e2e08d5d"},{"author":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"change_message_id":"b939487dd24b5f4f4ae85534066aa29b2fd181aa","unresolved":false,"context_lines":[{"line_number":84,"context_line":"                insecure\u003dauth_config.insecure,"},{"line_number":85,"context_line":"                timeout\u003dcfg.CONF.http_timeout,"},{"line_number":86,"context_line":"                os_user_domain_id\u003dauth_config.os_user_domain_id,"},{"line_number":87,"context_line":"                os_project_name\u003dauth_config.os_project_name,"},{"line_number":88,"context_line":"                os_project_domain_id\u003dauth_config.os_project_domain_id,"},{"line_number":89,"context_line":"            )"},{"line_number":90,"context_line":"            self.api_client \u003d ceiloclient.get_client(2, **creds)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_d6faba74","line":87,"in_reply_to":"3a50d1a3_b721444f","updated":"2015-07-29 06:03:39.000000000","message":"Done","commit_id":"0b09e570906479aeee5aa19199e81ce8e2e08d5d"},{"author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"change_message_id":"d26b3e90b7aae85d229a954310cedf5109265a8f","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            creds \u003d dict("},{"line_number":79,"context_line":"                os_auth_url\u003dauth_config.os_auth_url,"},{"line_number":80,"context_line":"                os_region_name\u003dauth_config.os_region_name,"},{"line_number":81,"context_line":"                os_tenant_name\u003dauth_config.os_tenant_name,"},{"line_number":82,"context_line":"                os_password\u003dauth_config.os_password,"},{"line_number":83,"context_line":"                os_username\u003dauth_config.os_username,"},{"line_number":84,"context_line":"                os_cacert\u003dauth_config.os_cacert,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_f0e4960d","line":81,"updated":"2015-07-29 08:50:34.000000000","message":"I guess gordon meant this line, the tenant_name is used for keystone V2 and the project_name is used for keystone V3, we just need project_name, because ceilometerclient will automatically translate. see [1]\n\n[1] http://git.openstack.org/cgit/openstack/python-ceilometerclient/tree/ceilometerclient/client.py?id\u003d1.0.13#n119","commit_id":"c6c45c68e312f5ab9cbe396f16612d3877c32956"},{"author":{"_account_id":8290,"name":"liusheng","email":"liusheng2048@gmail.com","username":"LiuSheng"},"change_message_id":"facbbecfd3ed6b3837e1e3d0ffa096d584bebe7b","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            creds \u003d dict("},{"line_number":79,"context_line":"                os_auth_url\u003dauth_config.os_auth_url,"},{"line_number":80,"context_line":"                os_region_name\u003dauth_config.os_region_name,"},{"line_number":81,"context_line":"                os_tenant_name\u003dauth_config.os_tenant_name,"},{"line_number":82,"context_line":"                os_password\u003dauth_config.os_password,"},{"line_number":83,"context_line":"                os_username\u003dauth_config.os_username,"},{"line_number":84,"context_line":"                os_cacert\u003dauth_config.os_cacert,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_5359d1fb","line":81,"in_reply_to":"3a50d1a3_2b3cd3df","updated":"2015-07-29 10:16:54.000000000","message":"actually, the ceilometerclient has been changed to cover both of the two options, see[1][2], but there is a problem that current devstack installation use the client packages from pypi. the changes [1] hasn\u0027t been released to pypi. So if we use this work around way, we can remove above line. or we need to wait the client package released and keep this the change you have made in PS1. The current ceilometerclient in ceilometer requirements is1.0.13\n\n[1] http://git.openstack.org/cgit/openstack/python-ceilometerclient/tree/ceilometerclient/client.py#n405\n[2] http://git.openstack.org/cgit/openstack/python-ceilometerclient/tree/ceilometerclient/client.py#n234","commit_id":"c6c45c68e312f5ab9cbe396f16612d3877c32956"},{"author":{"_account_id":12927,"name":"Lan Qi song","email":"lqs933@foxmail.com","username":"lqisong"},"change_message_id":"91d65788bf06c685825c7fe9ad81cc58b85e5a94","unresolved":false,"context_lines":[{"line_number":78,"context_line":"            creds \u003d dict("},{"line_number":79,"context_line":"                os_auth_url\u003dauth_config.os_auth_url,"},{"line_number":80,"context_line":"                os_region_name\u003dauth_config.os_region_name,"},{"line_number":81,"context_line":"                os_tenant_name\u003dauth_config.os_tenant_name,"},{"line_number":82,"context_line":"                os_password\u003dauth_config.os_password,"},{"line_number":83,"context_line":"                os_username\u003dauth_config.os_username,"},{"line_number":84,"context_line":"                os_cacert\u003dauth_config.os_cacert,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_2b3cd3df","line":81,"in_reply_to":"3a50d1a3_f0e4960d","updated":"2015-07-29 09:28:16.000000000","message":"Hi Liu Sheng, thanks for the information. But, if we remove line 81, should we also remove  \"os-tenant-name\" option in \"service_credentials\" group?  since it\u0027s useless any more.\n\nhttps://github.com/openstack/ceilometer/blob/master/ceilometer/service.py#L65-L68\n\nI think we\u0027d better support both \"tenant_name\" and \"project_name\" option, and user only need to specify one of them.","commit_id":"c6c45c68e312f5ab9cbe396f16612d3877c32956"}],"ceilometer/service.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"7de050fdd3efa833e66b013a2b4bd5612330bc71","unresolved":false,"context_lines":[{"line_number":63,"context_line":"               default\u003dos.environ.get(\u0027OS_TENANT_ID\u0027, \u0027\u0027),"},{"line_number":64,"context_line":"               help\u003d\u0027Tenant ID to use for OpenStack service access.\u0027),"},{"line_number":65,"context_line":"    cfg.StrOpt(\u0027os-tenant-name\u0027,"},{"line_number":66,"context_line":"               deprecated_group\u003d\"DEFAULT\","},{"line_number":67,"context_line":"               default\u003dos.environ.get(\u0027OS_TENANT_NAME\u0027, \u0027admin\u0027),"},{"line_number":68,"context_line":"               help\u003d\u0027Tenant name to use for OpenStack service access.\u0027),"},{"line_number":69,"context_line":"    cfg.StrOpt(\u0027os-cacert\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_af51be63","line":66,"updated":"2015-07-29 14:56:24.000000000","message":"remove this.","commit_id":"c6c45c68e312f5ab9cbe396f16612d3877c32956"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"7de050fdd3efa833e66b013a2b4bd5612330bc71","unresolved":false,"context_lines":[{"line_number":92,"context_line":"    cfg.StrOpt(\u0027os-project-domain-id\u0027,"},{"line_number":93,"context_line":"               default\u003dos.environ.get(\u0027OS_PROJECT_DOMAIN_ID\u0027, \u0027default\u0027),"},{"line_number":94,"context_line":"               help\u003d\u0027The domain id of the user project\u0027),"},{"line_number":95,"context_line":"    cfg.StrOpt(\u0027os-project-name\u0027,"},{"line_number":96,"context_line":"               default\u003dos.environ.get(\u0027OS_PROJECT_NAME\u0027, \u0027admin\u0027),"},{"line_number":97,"context_line":"               help\u003d\u0027The user project name\u0027),"},{"line_number":98,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_ef6d8603","line":95,"updated":"2015-07-29 14:56:24.000000000","message":"add\n\n  deprecated_opts\u003d[cfg.DeprecatedOpt(\"os-tenant-name\", \"DEFAULT\"),\n                  cfg.DeprecatedOpt(\"os-tenant-name\", \"service_credentials\")]","commit_id":"c6c45c68e312f5ab9cbe396f16612d3877c32956"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"7de050fdd3efa833e66b013a2b4bd5612330bc71","unresolved":false,"context_lines":[{"line_number":93,"context_line":"               default\u003dos.environ.get(\u0027OS_PROJECT_DOMAIN_ID\u0027, \u0027default\u0027),"},{"line_number":94,"context_line":"               help\u003d\u0027The domain id of the user project\u0027),"},{"line_number":95,"context_line":"    cfg.StrOpt(\u0027os-project-name\u0027,"},{"line_number":96,"context_line":"               default\u003dos.environ.get(\u0027OS_PROJECT_NAME\u0027, \u0027admin\u0027),"},{"line_number":97,"context_line":"               help\u003d\u0027The user project name\u0027),"},{"line_number":98,"context_line":"]"},{"line_number":99,"context_line":"cfg.CONF.register_cli_opts(CLI_OPTS, group\u003d\"service_credentials\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"3a50d1a3_cf0f6a37","line":96,"updated":"2015-07-29 14:56:24.000000000","message":"change to\n\n  os.environ.get(\u0027OS_PROJECT_NAME\u0027, os.environ.get(\u0027OS_TENANT_NAME\u0027, \u0027admin\u0027))","commit_id":"c6c45c68e312f5ab9cbe396f16612d3877c32956"}]}