)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"b252dbcbe5c88a34a12d3f927f256835b2263507","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b4a9ce97_1200895e","updated":"2022-04-12 08:27:39.000000000","message":"check experimental","commit_id":"d4dfa1d2efed519612da67272f7b3234abbca91c"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"e60dc7850094249b872b239471d189bf6b3c04fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"48c92b78_8e9284a4","updated":"2022-04-12 08:27:33.000000000","message":"recheck","commit_id":"d4dfa1d2efed519612da67272f7b3234abbca91c"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"8a1986d177f17e5cddd2aa5a14bebca7ade2cf13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d5ed7df5_91c92bc0","updated":"2022-04-13 11:57:25.000000000","message":"And what about this ? \n\n./zuul.d/jobs.yaml:118:      install_type: source\n./zuul.d/jobs.yaml:129:      install_type: source\n./zuul.d/jobs.yaml:145:      install_type: source\n./zuul.d/jobs.yaml:156:      install_type: source\n./zuul.d/jobs.yaml:166:      install_type: source\n./zuul.d/jobs.yaml:225:      install_type: source\n./zuul.d/jobs.yaml:235:      install_type: source\n","commit_id":"c43195ee6701265d681adba4a3244ec318f9251b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"b7b25670ba976926baf576f95bbf38b5ebf4e366","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"20c4d0ef_6a9d3f7e","updated":"2022-04-13 11:56:29.000000000","message":"I think this can be removed also \n\n./tests/templates/globals-default.j2:9:# Zed dropped install_type so we have it only on upgrades\n./tests/templates/globals-default.j2:11:kolla_install_type: \"{{ install_type }}\"\n\n...as this doesn\u0027t make sense anymore, it\u0027s commit from 2017 year.","commit_id":"c43195ee6701265d681adba4a3244ec318f9251b"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"5be25b9f2acdcea364aae28a4277c9156f18ccc8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"fc72c59b_92d05451","updated":"2022-04-12 16:40:40.000000000","message":"check experimental","commit_id":"c43195ee6701265d681adba4a3244ec318f9251b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"0d8a73fb11f3f735cfccce5b8ed555560078941c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"454eebac_186b7b26","updated":"2022-04-13 11:48:22.000000000","message":"what about reno ? ","commit_id":"c43195ee6701265d681adba4a3244ec318f9251b"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"7d6362da6b62deac7e67f230fa00f957311130b7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a726a342_d9ed69c8","in_reply_to":"20c4d0ef_6a9d3f7e","updated":"2022-04-13 17:05:30.000000000","message":"please comment exact lines so it can be properly read and replied","commit_id":"c43195ee6701265d681adba4a3244ec318f9251b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"64a2841bf88e8627ef1311a836a76638f5c0bbf6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"360aaf0b_a5b8460e","in_reply_to":"a40a7703_0742e432","updated":"2022-04-14 12:55:02.000000000","message":"Aaa, thank you :)","commit_id":"c43195ee6701265d681adba4a3244ec318f9251b"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"bf81ee11fa901d5189cfc2fc3d6e54ed0d0cf63c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a40a7703_0742e432","in_reply_to":"a726a342_d9ed69c8","updated":"2022-04-13 21:49:56.000000000","message":"When patch has topic set it is worth checking what was/is there.\n\nhttps://review.opendev.org/c/openstack/kolla-ansible/+/837291 sorted out lines you mentioned.","commit_id":"c43195ee6701265d681adba4a3244ec318f9251b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"cb775e57e09f828f6e016d0590ea7bc8eb1c1849","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4a88bfdc_96f155e2","in_reply_to":"a726a342_d9ed69c8","updated":"2022-04-13 19:21:13.000000000","message":"how can I comment files which are not included in diff ? :(","commit_id":"c43195ee6701265d681adba4a3244ec318f9251b"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"3240ba09c9d33a9259bba78e09fe36c8c3b71483","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"20017303_bb27166b","updated":"2022-04-20 15:18:26.000000000","message":"Magnum scenario will be fixed by https://review.opendev.org/c/openstack/trove-dashboard/+/838700\nMasakari is broken by SASL (I remember mgoddard working on it)\nI will fix swift.\nUbuntu OVN - could mnasiadka have a look?\nMonasca - anyone cares? This went RED again with no will to fix it - let\u0027s abandon testing?","commit_id":"1620ab5be971008958a528c2d318c41234a3df5d"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"01241e81a1e04b58868d9eab7239cdc0ee7e8155","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"7e84b42d_1d907a08","in_reply_to":"20017303_bb27166b","updated":"2022-04-20 15:37:04.000000000","message":"swift fix in https://review.opendev.org/c/openstack/kolla-ansible/+/838748","commit_id":"1620ab5be971008958a528c2d318c41234a3df5d"}],"doc/source/reference/orchestration-and-nfv/tacker-guide.rst":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f1f88d73f2530ebc7b029b65ff7b2596aacf3df6","unresolved":true,"context_lines":[{"line_number":27,"context_line":"* Networking-sfc"},{"line_number":28,"context_line":"* Opendaylight"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Compatibility"},{"line_number":31,"context_line":"~~~~~~~~~~~~~"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Tacker is supported by the following distros."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* CentOS and RHEL"},{"line_number":36,"context_line":"* Debian"},{"line_number":37,"context_line":"* Ubuntu"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Preparation and Deployment"},{"line_number":40,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":1,"id":"78f58dcb_52745a99","line":37,"range":{"start_line":30,"start_character":0,"end_line":37,"end_character":8},"updated":"2022-04-11 11:08:50.000000000","message":"it\u0027s just supported, we can drop this part","commit_id":"8dd9d55b9faf8346d99a20a7a537805d03c69a17"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"1bf01fddef5f14fc67dd79afb4af4e2c28a2f97a","unresolved":false,"context_lines":[{"line_number":27,"context_line":"* Networking-sfc"},{"line_number":28,"context_line":"* Opendaylight"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"Compatibility"},{"line_number":31,"context_line":"~~~~~~~~~~~~~"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Tacker is supported by the following distros."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* CentOS and RHEL"},{"line_number":36,"context_line":"* Debian"},{"line_number":37,"context_line":"* Ubuntu"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Preparation and Deployment"},{"line_number":40,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~"}],"source_content_type":"text/x-rst","patch_set":1,"id":"c743a2cb_474b719e","line":37,"range":{"start_line":30,"start_character":0,"end_line":37,"end_character":8},"in_reply_to":"78f58dcb_52745a99","updated":"2022-04-11 11:10:53.000000000","message":"Done","commit_id":"8dd9d55b9faf8346d99a20a7a537805d03c69a17"}],"tests/run.yml":[{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"08cd151a90b9162df8bf9711cf7952d806a07819","unresolved":true,"context_lines":[{"line_number":58,"context_line":"        # buster see https://github.com/docker/for-linux/issues/711."},{"line_number":59,"context_line":"        push: \"{{ base_distro !\u003d \u0027debian\u0027 }}\""},{"line_number":60,"context_line":"        base: \"{{ base_distro }}\""},{"line_number":61,"context_line":"        install_type: \"{{ install_type }}\""},{"line_number":62,"context_line":"        registry: \"127.0.0.1:4000\""},{"line_number":63,"context_line":"        namespace: lokolla"},{"line_number":64,"context_line":"        tag: \"{{ build_image_tag }}\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"478cac81_4f15bb4d","side":"PARENT","line":61,"range":{"start_line":61,"start_character":6,"end_line":61,"end_character":42},"updated":"2022-04-12 18:36:41.000000000","message":"Maybe this place should be send as separate with \u0027source\u0027 hardcoded. Then Kolla should merge \u0027drop install_type\u0027 patch and then this patch in k-a.","commit_id":"259a5fc308b02ecbfba625581e3e6f35c5c5314e"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"d36705ab7eef192782c769920457a06663418ddc","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        # buster see https://github.com/docker/for-linux/issues/711."},{"line_number":59,"context_line":"        push: \"{{ base_distro !\u003d \u0027debian\u0027 }}\""},{"line_number":60,"context_line":"        base: \"{{ base_distro }}\""},{"line_number":61,"context_line":"        install_type: \"{{ install_type }}\""},{"line_number":62,"context_line":"        registry: \"127.0.0.1:4000\""},{"line_number":63,"context_line":"        namespace: lokolla"},{"line_number":64,"context_line":"        tag: \"{{ build_image_tag }}\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5258534e_bf755f2b","side":"PARENT","line":61,"range":{"start_line":61,"start_character":6,"end_line":61,"end_character":42},"in_reply_to":"478cac81_4f15bb4d","updated":"2022-04-20 14:36:11.000000000","message":"done other way","commit_id":"259a5fc308b02ecbfba625581e3e6f35c5c5314e"}]}
