)]}'
{"octavia/statistics/drivers/influxdb.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"2066b706e2f5187cdc9a3dbec68f3a50fc523ec3","unresolved":false,"context_lines":[{"line_number":24,"context_line":""},{"line_number":25,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":26,"context_line":"CONF \u003d cfg.CONF"},{"line_number":27,"context_line":"influxdb_opts \u003d ["},{"line_number":28,"context_line":"    cfg.StrOpt(\u0027influxdb_url\u0027, help\u003d\u0027Base ingestion URL for InfluxDB\u0027),"},{"line_number":29,"context_line":"    cfg.StrOpt(\u0027db\u0027, default\u003d\u0027octavia\u0027,"},{"line_number":30,"context_line":"               help\u003d\u0027InfluxDB database name\u0027),"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_4847791e","line":27,"updated":"2020-09-07 21:47:36.000000000","message":"Since this is an in-tree driver, it should be added to the config file example under etc/octavia.conf","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"2066b706e2f5187cdc9a3dbec68f3a50fc523ec3","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    cfg.DictOpt(\u0027tags\u0027, default\u003d{},"},{"line_number":32,"context_line":"                help\u003d\u0027Dictionary of additional tags\u0027)"},{"line_number":33,"context_line":"]"},{"line_number":34,"context_line":"CONF.register_opts(influxdb_opts, group\u003d\u0027influxdb\u0027)"},{"line_number":35,"context_line":"APPLICATION_NAME \u003d \u0027octavia.health_manager\u0027"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"#  measurement,   tag,          tag,           key        timestamp"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_085bc1f5","line":34,"range":{"start_line":34,"start_character":41,"end_line":34,"end_character":49},"updated":"2020-09-07 21:47:36.000000000","message":"Should this be in an octavia namespace to not conflict with other usage?\nMaybe a octavia statistics section and prefix the options with influxdb?","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8a1cc16a9df3fee828d3ad0fb4975c1a94cf6c73","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"class StatsUpdateInfluxDB(stats_base.StatsDriverMixin):"},{"line_number":43,"context_line":"    def __init__(self):"},{"line_number":44,"context_line":"        super(StatsUpdateInfluxDB, self).__init__()"},{"line_number":45,"context_line":"        self.influxdb_url \u003d urllib.parse.urljoin("},{"line_number":46,"context_line":"            CONF.influxdb.influxdb_url,"},{"line_number":47,"context_line":"            \u0027/api/v2/write?bucket\u003d{}\u0027.format(CONF.influxdb.db))"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_0ab590ee","line":44,"updated":"2020-09-04 15:58:31.000000000","message":"pep8: R1725: Consider using Python 3 style super() without arguments (super-with-arguments)","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8a1cc16a9df3fee828d3ad0fb4975c1a94cf6c73","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"class StatsUpdateInfluxDB(stats_base.StatsDriverMixin):"},{"line_number":43,"context_line":"    def __init__(self):"},{"line_number":44,"context_line":"        super(StatsUpdateInfluxDB, self).__init__()"},{"line_number":45,"context_line":"        self.influxdb_url \u003d urllib.parse.urljoin("},{"line_number":46,"context_line":"            CONF.influxdb.influxdb_url,"},{"line_number":47,"context_line":"            \u0027/api/v2/write?bucket\u003d{}\u0027.format(CONF.influxdb.db))"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_2ac97478","line":44,"updated":"2020-09-04 15:58:31.000000000","message":"pep8: R1725: Consider using Python 3 style super() without arguments (super-with-arguments)","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8a1cc16a9df3fee828d3ad0fb4975c1a94cf6c73","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"class StatsUpdateInfluxDB(stats_base.StatsDriverMixin):"},{"line_number":43,"context_line":"    def __init__(self):"},{"line_number":44,"context_line":"        super(StatsUpdateInfluxDB, self).__init__()"},{"line_number":45,"context_line":"        self.influxdb_url \u003d urllib.parse.urljoin("},{"line_number":46,"context_line":"            CONF.influxdb.influxdb_url,"},{"line_number":47,"context_line":"            \u0027/api/v2/write?bucket\u003d{}\u0027.format(CONF.influxdb.db))"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_4ace6883","line":44,"updated":"2020-09-04 15:58:31.000000000","message":"pep8: R1725: Consider using Python 3 style super() without arguments (super-with-arguments)","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8a1cc16a9df3fee828d3ad0fb4975c1a94cf6c73","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"class StatsUpdateInfluxDB(stats_base.StatsDriverMixin):"},{"line_number":43,"context_line":"    def __init__(self):"},{"line_number":44,"context_line":"        super(StatsUpdateInfluxDB, self).__init__()"},{"line_number":45,"context_line":"        self.influxdb_url \u003d urllib.parse.urljoin("},{"line_number":46,"context_line":"            CONF.influxdb.influxdb_url,"},{"line_number":47,"context_line":"            \u0027/api/v2/write?bucket\u003d{}\u0027.format(CONF.influxdb.db))"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_6aea4cd8","line":44,"updated":"2020-09-04 15:58:31.000000000","message":"pep8: R1725: Consider using Python 3 style super() without arguments (super-with-arguments)","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8a1cc16a9df3fee828d3ad0fb4975c1a94cf6c73","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"class StatsUpdateInfluxDB(stats_base.StatsDriverMixin):"},{"line_number":43,"context_line":"    def __init__(self):"},{"line_number":44,"context_line":"        super(StatsUpdateInfluxDB, self).__init__()"},{"line_number":45,"context_line":"        self.influxdb_url \u003d urllib.parse.urljoin("},{"line_number":46,"context_line":"            CONF.influxdb.influxdb_url,"},{"line_number":47,"context_line":"            \u0027/api/v2/write?bucket\u003d{}\u0027.format(CONF.influxdb.db))"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_8ac88074","line":44,"updated":"2020-09-04 15:58:31.000000000","message":"pep8: R1725: Consider using Python 3 style super() without arguments (super-with-arguments)","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8a1cc16a9df3fee828d3ad0fb4975c1a94cf6c73","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"class StatsUpdateInfluxDB(stats_base.StatsDriverMixin):"},{"line_number":43,"context_line":"    def __init__(self):"},{"line_number":44,"context_line":"        super(StatsUpdateInfluxDB, self).__init__()"},{"line_number":45,"context_line":"        self.influxdb_url \u003d urllib.parse.urljoin("},{"line_number":46,"context_line":"            CONF.influxdb.influxdb_url,"},{"line_number":47,"context_line":"            \u0027/api/v2/write?bucket\u003d{}\u0027.format(CONF.influxdb.db))"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_8aefe0e7","line":44,"updated":"2020-09-04 15:58:31.000000000","message":"pep8: R1725: Consider using Python 3 style super() without arguments (super-with-arguments)","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8a1cc16a9df3fee828d3ad0fb4975c1a94cf6c73","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"class StatsUpdateInfluxDB(stats_base.StatsDriverMixin):"},{"line_number":43,"context_line":"    def __init__(self):"},{"line_number":44,"context_line":"        super(StatsUpdateInfluxDB, self).__init__()"},{"line_number":45,"context_line":"        self.influxdb_url \u003d urllib.parse.urljoin("},{"line_number":46,"context_line":"            CONF.influxdb.influxdb_url,"},{"line_number":47,"context_line":"            \u0027/api/v2/write?bucket\u003d{}\u0027.format(CONF.influxdb.db))"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_aabd84d8","line":44,"updated":"2020-09-04 15:58:31.000000000","message":"pep8: R1725: Consider using Python 3 style super() without arguments (super-with-arguments)","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8a1cc16a9df3fee828d3ad0fb4975c1a94cf6c73","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"class StatsUpdateInfluxDB(stats_base.StatsDriverMixin):"},{"line_number":43,"context_line":"    def __init__(self):"},{"line_number":44,"context_line":"        super(StatsUpdateInfluxDB, self).__init__()"},{"line_number":45,"context_line":"        self.influxdb_url \u003d urllib.parse.urljoin("},{"line_number":46,"context_line":"            CONF.influxdb.influxdb_url,"},{"line_number":47,"context_line":"            \u0027/api/v2/write?bucket\u003d{}\u0027.format(CONF.influxdb.db))"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_caba78df","line":44,"updated":"2020-09-04 15:58:31.000000000","message":"pep8: R1725: Consider using Python 3 style super() without arguments (super-with-arguments)","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"8a1cc16a9df3fee828d3ad0fb4975c1a94cf6c73","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"class StatsUpdateInfluxDB(stats_base.StatsDriverMixin):"},{"line_number":43,"context_line":"    def __init__(self):"},{"line_number":44,"context_line":"        super(StatsUpdateInfluxDB, self).__init__()"},{"line_number":45,"context_line":"        self.influxdb_url \u003d urllib.parse.urljoin("},{"line_number":46,"context_line":"            CONF.influxdb.influxdb_url,"},{"line_number":47,"context_line":"            \u0027/api/v2/write?bucket\u003d{}\u0027.format(CONF.influxdb.db))"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_eab77cf4","line":44,"updated":"2020-09-04 15:58:31.000000000","message":"pep8: R1725: Consider using Python 3 style super() without arguments (super-with-arguments)","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"2066b706e2f5187cdc9a3dbec68f3a50fc523ec3","unresolved":false,"context_lines":[{"line_number":86,"context_line":"                    # operator defined tags from config"},{"line_number":87,"context_line":"                    \u0027{extra_tags}\u0027"},{"line_number":88,"context_line":"                    # static tags"},{"line_number":89,"context_line":"                    \u0027amphora_id\u003d{amphora_id},\u0027"},{"line_number":90,"context_line":"                    \u0027listener_id\u003d{listener_id},\u0027"},{"line_number":91,"context_line":"                    \u0027loadbalancer_id\u003d{loadbalancer_id},\u0027"},{"line_number":92,"context_line":"                    \u0027project_id\u003d{project_id} \u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_0810e1ff","line":89,"updated":"2020-09-07 21:47:36.000000000","message":"Should we expose the amphora ID? It\u0027s a \"behind the curtain\" thing. We typically don\u0027t want to publish these as they are used for host validation.","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"2066b706e2f5187cdc9a3dbec68f3a50fc523ec3","unresolved":false,"context_lines":[{"line_number":96,"context_line":"                    \u0027{timestamp}\\n\u0027).format("},{"line_number":97,"context_line":"                    measurement\u003dmetric,"},{"line_number":98,"context_line":"                    extra_tags\u003dextra_tags,"},{"line_number":99,"context_line":"                    amphora_id\u003dstats_object.amphora_id,"},{"line_number":100,"context_line":"                    listener_id\u003dstats_object.listener_id,"},{"line_number":101,"context_line":"                    loadbalancer_id\u003dloadbalancer_id,"},{"line_number":102,"context_line":"                    project_id\u003dproject_id,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_a8fe15ac","line":99,"updated":"2020-09-07 21:47:36.000000000","message":"Same comment, I don\u0027t think the amp ID should be in this message.","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"2066b706e2f5187cdc9a3dbec68f3a50fc523ec3","unresolved":false,"context_lines":[{"line_number":106,"context_line":"                stats_to_send +\u003d formatted_metric"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        # Send stats_to_send to InfluxDB"},{"line_number":109,"context_line":"        r \u003d requests.post(self.influxdb_url, data\u003dstats_to_send)"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        LOG.debug(\"Updating stats in InfluxDB: [%s] %s\","},{"line_number":112,"context_line":"                  r.status_code, stats_to_send)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_c83269a7","line":109,"updated":"2020-09-07 21:47:36.000000000","message":"Shouldn\u0027t there be some kind of try/catch block here?","commit_id":"25f17952024658e41445d9634b133f9e519f933f"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"2066b706e2f5187cdc9a3dbec68f3a50fc523ec3","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        # Send stats_to_send to InfluxDB"},{"line_number":109,"context_line":"        r \u003d requests.post(self.influxdb_url, data\u003dstats_to_send)"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        LOG.debug(\"Updating stats in InfluxDB: [%s] %s\","},{"line_number":112,"context_line":"                  r.status_code, stats_to_send)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_88fb51bc","line":111,"range":{"start_line":111,"start_character":19,"end_line":111,"end_character":27},"updated":"2020-09-07 21:47:36.000000000","message":"nit: Updated","commit_id":"25f17952024658e41445d9634b133f9e519f933f"}]}