)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"2cb4843bb21b40fdcb32c35cbf2d9f11126c6463","unresolved":false,"context_lines":[{"line_number":12,"context_line":"under 0.25s to post 5000 points. the only thing that took \u003e3s was"},{"line_number":13,"context_line":"getting \u003e80K datapoints and this workflow is only pushing, and only"},{"line_number":14,"context_line":"getting small chunks occasionally. making it 6s because it seems"},{"line_number":15,"context_line":"keystone adds decent overhead."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"do not set timeout for upgrade because it might take longer than normal"},{"line_number":18,"context_line":"workflow"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"da36d5c6_c319ae23","line":15,"updated":"2017-02-25 09:12:57.000000000","message":"yeah if the keystone that gnocchi uses is slow to validate taht might be long","commit_id":"7c6160e3a19ed50562e51137b2779c328a750bce"}],"ceilometer/dispatcher/gnocchi_opts.py":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"e8b71704f3d5947c5cd4cd9648f7b50191b392e5","unresolved":false,"context_lines":[{"line_number":30,"context_line":"               help\u003d(\u0027The Yaml file that defines mapping between samples \u0027"},{"line_number":31,"context_line":"                     \u0027and gnocchi resources/metrics\u0027)),"},{"line_number":32,"context_line":"    cfg.FloatOpt(\u0027request_timeout\u0027, default\u003d3.05, min\u003d0.0,"},{"line_number":33,"context_line":"                 help\u003d\u0027Number of seconds before request to gnocchi times out\u0027),"},{"line_number":34,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":4,"id":"da36d5c6_7222bc53","line":33,"range":{"start_line":33,"start_character":59,"end_line":33,"end_character":66},"updated":"2017-02-23 21:18:43.000000000","message":"Gnocchi","commit_id":"74071df2564abbd8704ef5208aa48b3d95e98dae"}],"ceilometer/gnocchi_client.py":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"93cd02a8df47ddb5d76b891bf8d8754ee1f40fb6","unresolved":false,"context_lines":[{"line_number":20,"context_line":"LOG \u003d log.getLogger(__name__)"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"def get_gnocchiclient(conf, timeout_override\u003dFalse):"},{"line_number":24,"context_line":"    group \u003d conf.dispatcher_gnocchi.auth_section"},{"line_number":25,"context_line":"    timeout \u003d (None if (not conf.dispatcher_gnocchi.request_timeout or"},{"line_number":26,"context_line":"                        timeout_override)"}],"source_content_type":"text/x-python","patch_set":10,"id":"da36d5c6_98080e2a","line":23,"range":{"start_line":23,"start_character":28,"end_line":23,"end_character":50},"updated":"2017-02-24 20:05:34.000000000","message":"you could just set timeout\u003dNone that would be simpler rather than an override","commit_id":"0c8df150983de1ca4f3fd63029588b03c45b0859"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"46e71b80faacfbc248e4cebbeb6bc80f9f74f9b4","unresolved":false,"context_lines":[{"line_number":20,"context_line":"LOG \u003d log.getLogger(__name__)"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"def get_gnocchiclient(conf, timeout_override\u003dFalse):"},{"line_number":24,"context_line":"    group \u003d conf.dispatcher_gnocchi.auth_section"},{"line_number":25,"context_line":"    timeout \u003d (None if (not conf.dispatcher_gnocchi.request_timeout or"},{"line_number":26,"context_line":"                        timeout_override)"}],"source_content_type":"text/x-python","patch_set":10,"id":"da36d5c6_b8a79209","line":23,"range":{"start_line":23,"start_character":28,"end_line":23,"end_character":50},"in_reply_to":"da36d5c6_98080e2a","updated":"2017-02-24 20:09:44.000000000","message":"i didn\u0027t want to it to be another timeout input... if we add another timeout option here, i guess it will have higher priority over conf option?\n\ni\u0027m ok with this.","commit_id":"0c8df150983de1ca4f3fd63029588b03c45b0859"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"6214d51e1b81a96cdbb95bc8cefa4108daf97047","unresolved":false,"context_lines":[{"line_number":20,"context_line":"LOG \u003d log.getLogger(__name__)"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"def get_gnocchiclient(conf, timeout_override\u003dFalse):"},{"line_number":24,"context_line":"    group \u003d conf.dispatcher_gnocchi.auth_section"},{"line_number":25,"context_line":"    timeout \u003d (None if (not conf.dispatcher_gnocchi.request_timeout or"},{"line_number":26,"context_line":"                        timeout_override)"}],"source_content_type":"text/x-python","patch_set":10,"id":"da36d5c6_b8431212","line":23,"range":{"start_line":23,"start_character":28,"end_line":23,"end_character":50},"in_reply_to":"da36d5c6_b8a79209","updated":"2017-02-24 20:15:33.000000000","message":"actually it might get confusing. if i put timeout\u003dNone i don\u0027t know when it\u0027s just an unset timeout \u0027override\u0027 or i want to make it never timeout","commit_id":"0c8df150983de1ca4f3fd63029588b03c45b0859"}],"ceilometer/keystone_client.py":[{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"19497ff247556ed5721833368603d0cdfbfbd464","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    group \u003d group or DEFAULT_GROUP"},{"line_number":33,"context_line":"    auth_plugin \u003d ka_loading.load_auth_from_conf_options(conf, group)"},{"line_number":34,"context_line":"    kwargs \u003d {\u0027auth\u0027: auth_plugin, \u0027session\u0027: requests_session}"},{"line_number":35,"context_line":"    session \u003d ka_loading.load_session_from_conf_options(conf, group, **kwargs)"},{"line_number":36,"context_line":"    return session"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"da36d5c6_eaa13e47","line":35,"updated":"2017-02-24 13:36:51.000000000","message":"i removed it here... if i pass in timeout, it fails when trying to upgrade gnocchi\n\n2017-02-23 22:36:26.691 | ++ /opt/stack/new/ceilometer/devstack/plugin.sh:init_ceilometer:377 :   /usr/local/bin/ceilometer-upgrade --skip-metering-database\n2017-02-23 22:36:27.766 | 2017-02-23 22:36:27.766 22639 WARNING oslo_reports.guru_meditation_report [-] Guru meditation now registers SIGUSR1 and SIGUSR2 by default for backward compatibility. SIGUSR1 will no longer be registered in a future release, so please use SIGUSR2 to generate reports.\n2017-02-23 22:36:27.766 | 2017-02-23 22:36:27.766 22639 INFO ceilometer.cmd.storage [-] Skipping metering database upgrade\n2017-02-23 22:36:27.767 | 2017-02-23 22:36:27.766 22639 DEBUG ceilometer.cmd.storage [-] Upgrading Gnocchi resource types upgrade /opt/stack/new/ceilometer/ceilometer/cmd/storage.py:51\n2017-02-23 22:36:31.637 | 2017-02-23 22:36:31.636 22639 WARNING keystoneauth.identity.generic.base [-] Discovering versions from the identity service failed when creating the password plugin. Attempting to determine version from URL.\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 CRITICAL ceilometer [-] DiscoveryFailure: Could not determine a suitable URL for the plugin\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer Traceback (most recent call last):\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/bin/ceilometer-upgrade\", line 10, in \u003cmodule\u003e\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     sys.exit(upgrade())\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/opt/stack/new/ceilometer/ceilometer/cmd/storage.py\", line 53, in upgrade\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     gnocchi_client.upgrade_resource_types(conf)\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/opt/stack/new/ceilometer/ceilometer/gnocchi_client.py\", line 122, in upgrade_resource_types\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     gnocchi.resource_type.get(name\u003dname)\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/gnocchiclient/v1/resource_type.py\", line 44, in get\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     headers\u003d{\u0027Content-Type\u0027: \"application/json\"}).json()\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/gnocchiclient/v1/base.py\", line 37, in _get\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     return self.client.api.get(*args, **kwargs)\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/keystoneauth1/adapter.py\", line 217, in get\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     return self.request(url, \u0027GET\u0027, **kwargs)\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/gnocchiclient/client.py\", line 35, in request\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     **kwargs)\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/keystoneauth1/adapter.py\", line 142, in request\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     return self.session.request(url, method, **kwargs)\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/positional/__init__.py\", line 101, in inner\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     return wrapped(*args, **kwargs)\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/keystoneauth1/session.py\", line 491, in request\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     auth_headers \u003d self.get_auth_headers(auth)\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/keystoneauth1/session.py\", line 818, in get_auth_headers\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     return auth.get_headers(self, **kwargs)\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/keystoneauth1/plugin.py\", line 90, in get_headers\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     token \u003d self.get_token(session)\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/keystoneauth1/identity/base.py\", line 90, in get_token\n2017-02-23 22:36:31.646 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     return self.get_access(session).auth_token\n2017-02-23 22:36:31.647 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/keystoneauth1/identity/base.py\", line 136, in get_access\n2017-02-23 22:36:31.647 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     self.auth_ref \u003d self.get_auth_ref(session)\n2017-02-23 22:36:31.647 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/keystoneauth1/identity/generic/base.py\", line 196, in get_auth_ref\n2017-02-23 22:36:31.647 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     self._plugin \u003d self._do_create_plugin(session)\n2017-02-23 22:36:31.647 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer   File \"/usr/local/lib/python2.7/dist-packages/keystoneauth1/identity/generic/base.py\", line 191, in _do_create_plugin\n2017-02-23 22:36:31.647 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer     raise exceptions.DiscoveryFailure(\u0027Could not determine a suitable URL \u0027\n2017-02-23 22:36:31.647 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer DiscoveryFailure: Could not determine a suitable URL for the plugin\n2017-02-23 22:36:31.647 | 2017-02-23 22:36:31.637 22639 ERROR ceilometer","commit_id":"a490539733d60df6afdc20898d9b7135e1873df1"}]}