)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"93f3dbd404f995370701346e0b083b19e36183fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d42ff640_611d0748","updated":"2023-02-13 11:19:28.000000000","message":"Removing my -1 because my remaining comment is not a hard blocker but I\u0027d expect this is updated to address my concern with the section name.","commit_id":"73c82152d9ee7cb719558cb4de753bc608f351b2"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"2b2e0a72b846c03ddc7921cda9f68987e780fc69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"288dbd7b_d2ca5e5d","updated":"2023-02-13 16:59:36.000000000","message":"recheck","commit_id":"cdad0e07a3773f8f0af610afa0b12220f9ddfbfd"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"69bcd848588c12e7cf724e2f5b8c093ec888ca43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"57075756_66856cfe","updated":"2023-02-13 20:01:48.000000000","message":"I fixed the unpredictable CI issue with:\nhttps://review.opendev.org/c/openstack/osprofiler/+/873643","commit_id":"36e5a14a487725110da09454ab263cf1cb986686"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"7104507984d4712965fff2b2c686a3574da48086","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"19dd0a86_a13c8578","updated":"2023-02-14 07:50:09.000000000","message":"recheck","commit_id":"36e5a14a487725110da09454ab263cf1cb986686"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"f1fd9a034ec0512bde09dc1405edd71b24ca9098","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a6c6d421_7351c654","updated":"2023-02-14 11:27:19.000000000","message":"recheck timeout","commit_id":"cc3d229d713177ca5ab4da4d103ed6085048c674"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"700c7c93a65463757615535d40d2900ec6981e63","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"a30bd567_5c48b0bf","updated":"2023-02-16 11:03:31.000000000","message":"Sorry, ignore my previous comment, I forgot that osprofiler is an independent deliverable, so no deadline here.","commit_id":"2484b31e6c016e354297a82aa6a88535b359f6f5"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"39999a9a9a5dfea39005346d48a55926fd91cdeb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"095c48ba_77ff3684","updated":"2023-02-16 11:02:10.000000000","message":"We should notice the Antelope (2023.1) is feature freeze deadline is tomorrow\nhttps://releases.openstack.org/antelope/schedule.html#a-ff so if the patches are not ready before tomorrow we would have to wait one or two more weeks for when the deliverables are branched for Antelope, and hence, this patch would go to the Bobcat serie.","commit_id":"2484b31e6c016e354297a82aa6a88535b359f6f5"}],"osprofiler/drivers/jaeger.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2152972652e27edb4b21dff2ee895ec8ad1cb5b3","unresolved":true,"context_lines":[{"line_number":50,"context_line":"                \"reporting_host\": parsed_url.hostname,"},{"line_number":51,"context_line":"                \"reporting_port\": parsed_url.port,"},{"line_number":52,"context_line":"            },"},{"line_number":53,"context_line":"            \"tags\": conf.jaeger.process_tags"},{"line_number":54,"context_line":"        }"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        # Initialize tracer for each profiler"}],"source_content_type":"text/x-python","patch_set":1,"id":"56a7685b_e8485c6a","line":53,"range":{"start_line":53,"start_character":20,"end_line":53,"end_character":44},"updated":"2023-02-13 11:16:12.000000000","message":"I\u0027d suggest we do\n\n( conf.jaeger.process_tags or {} )\n\nto make sure we provide a dict or probably we can set the tags option only if the tag is configured.","commit_id":"73c82152d9ee7cb719558cb4de753bc608f351b2"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2d1ff1414ecd789c798b60c11bdb318c1c207670","unresolved":false,"context_lines":[{"line_number":50,"context_line":"                \"reporting_host\": parsed_url.hostname,"},{"line_number":51,"context_line":"                \"reporting_port\": parsed_url.port,"},{"line_number":52,"context_line":"            },"},{"line_number":53,"context_line":"            \"tags\": conf.jaeger.process_tags"},{"line_number":54,"context_line":"        }"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"        # Initialize tracer for each profiler"}],"source_content_type":"text/x-python","patch_set":1,"id":"d2401ff4_e7780fd5","line":53,"range":{"start_line":53,"start_character":20,"end_line":53,"end_character":44},"in_reply_to":"56a7685b_e8485c6a","updated":"2023-02-13 11:18:30.000000000","message":"Ignore this. process_tags is a DictOpt.","commit_id":"73c82152d9ee7cb719558cb4de753bc608f351b2"}],"osprofiler/opts.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2152972652e27edb4b21dff2ee895ec8ad1cb5b3","unresolved":true,"context_lines":[{"line_number":183,"context_line":""},{"line_number":184,"context_line":"_process_tags \u003d cfg.DictOpt("},{"line_number":185,"context_line":"    \"process_tags\","},{"line_number":186,"context_line":"    default\u003d\"\","},{"line_number":187,"context_line":"    help\u003d\"\"\""},{"line_number":188,"context_line":"Set process tracer tags."},{"line_number":189,"context_line":"\"\"\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"b28db215_4b984982","line":186,"range":{"start_line":186,"start_character":4,"end_line":186,"end_character":15},"updated":"2023-02-13 11:16:12.000000000","message":"I\u0027d prefer None if an empty string does not have real meaning.","commit_id":"73c82152d9ee7cb719558cb4de753bc608f351b2"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"2240a3ed0af7480fcbe0480dbe1087d738e11438","unresolved":false,"context_lines":[{"line_number":183,"context_line":""},{"line_number":184,"context_line":"_process_tags \u003d cfg.DictOpt("},{"line_number":185,"context_line":"    \"process_tags\","},{"line_number":186,"context_line":"    default\u003d\"\","},{"line_number":187,"context_line":"    help\u003d\"\"\""},{"line_number":188,"context_line":"Set process tracer tags."},{"line_number":189,"context_line":"\"\"\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"c3e37e20_8bcf54eb","line":186,"range":{"start_line":186,"start_character":4,"end_line":186,"end_character":15},"in_reply_to":"6e0e5264_4d7462a7","updated":"2023-02-16 12:18:28.000000000","message":"Done","commit_id":"73c82152d9ee7cb719558cb4de753bc608f351b2"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2d1ff1414ecd789c798b60c11bdb318c1c207670","unresolved":true,"context_lines":[{"line_number":183,"context_line":""},{"line_number":184,"context_line":"_process_tags \u003d cfg.DictOpt("},{"line_number":185,"context_line":"    \"process_tags\","},{"line_number":186,"context_line":"    default\u003d\"\","},{"line_number":187,"context_line":"    help\u003d\"\"\""},{"line_number":188,"context_line":"Set process tracer tags."},{"line_number":189,"context_line":"\"\"\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"6e0e5264_4d7462a7","line":186,"range":{"start_line":186,"start_character":4,"end_line":186,"end_character":15},"in_reply_to":"b28db215_4b984982","updated":"2023-02-13 11:18:30.000000000","message":"I noticed this is DictOpt so \"\" is translated to {}. You can ignore this or probably we can use {} instead.","commit_id":"73c82152d9ee7cb719558cb4de753bc608f351b2"}],"releasenotes/notes/jaeger-add-process-tags-79d5f5d7a0b049ef.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ec281eba9aaf9c517428d4f542c30333e12235ff","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    This is adding ability to set tracer process tags to Jaeger via a"},{"line_number":5,"context_line":"    configuration option introduced, `profiler_jaeger/process_tags`."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"d09a9052_5eb6927a","line":4,"range":{"start_line":4,"start_character":3,"end_line":4,"end_character":29},"updated":"2023-02-16 12:03:43.000000000","message":"this is rendered into a release note doc, so people reading the doc does not know what \u0027this\u0027 is. You can start with \"Add ability to ...\".","commit_id":"2484b31e6c016e354297a82aa6a88535b359f6f5"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"6daa548daa3550d087b53ff659b0dc76e7a159e6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    This is adding ability to set tracer process tags to Jaeger via a"},{"line_number":5,"context_line":"    configuration option introduced, `profiler_jaeger/process_tags`."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"ada98d93_f863774c","line":4,"range":{"start_line":4,"start_character":3,"end_line":4,"end_character":29},"in_reply_to":"1378c2cb_564894a5","updated":"2023-02-16 12:18:37.000000000","message":"Done","commit_id":"2484b31e6c016e354297a82aa6a88535b359f6f5"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"179a4e8532cde90cb9064fd0b56313d2b46a67c2","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    This is adding ability to set tracer process tags to Jaeger via a"},{"line_number":5,"context_line":"    configuration option introduced, `profiler_jaeger/process_tags`."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"1378c2cb_564894a5","line":4,"range":{"start_line":4,"start_character":3,"end_line":4,"end_character":29},"in_reply_to":"d09a9052_5eb6927a","updated":"2023-02-16 12:17:55.000000000","message":"Sure, done.","commit_id":"2484b31e6c016e354297a82aa6a88535b359f6f5"}]}