)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"899bb1751f2909099f138d6d9f62bf89ffdf2cdd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"48553102_8b58964b","updated":"2022-06-15 06:44:20.000000000","message":"-1 for visibility please check inline","commit_id":"f08d44a19389b2bd25858a25a9ec2c41e0e79dca"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"91c9bf7a79be57238a4d21fd5b9a776572e3e445","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7ca159e9_26ffc23a","updated":"2022-06-15 05:55:45.000000000","message":"Looks good to me,\n\nOne inline comment about task-core.","commit_id":"f08d44a19389b2bd25858a25a9ec2c41e0e79dca"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"e84411a0edd83d9a78051cee3b14a4850acad97c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"efc1e9be_27745f81","updated":"2022-06-15 10:01:03.000000000","message":"Looks good to me now, thanks.","commit_id":"36c2eb3081d03745dc1c73330cc6a442bf368ec3"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"1cb503f13ef893325bfe753b407c9fddf81df276","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9fda3e64_b2516a64","updated":"2022-06-15 13:34:22.000000000","message":"Not using my core here yet - want other ppl to have time to check that out!","commit_id":"36c2eb3081d03745dc1c73330cc6a442bf368ec3"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"82194e86165e655a8b0d85ec99f878d2bbf399e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1cad9cb4_522f4783","in_reply_to":"9fda3e64_b2516a64","updated":"2022-06-16 06:41:48.000000000","message":"anything in particular you are concerned about? i mean other than \u0027another service enabled\u0027 but this is only turned on for the ci/environments files so will not affect anything in \u0027normal\u0027 deploys","commit_id":"36c2eb3081d03745dc1c73330cc6a442bf368ec3"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"df0472b3a1b7fbd91423237d7d27d5bd6fb3dcd7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"03dc6111_3656ce75","updated":"2022-06-21 19:20:36.000000000","message":"recheck","commit_id":"6db7153b342b8aad874f9b139f33a19269571a0e"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"17bde0c58eb582f6d76eb6862c2f8bf7ac132add","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c5d7f4d8_59db6011","updated":"2022-06-23 03:01:35.000000000","message":"recheck","commit_id":"6db7153b342b8aad874f9b139f33a19269571a0e"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"76e88378d8bcd294943bae010c4314ebaee7f289","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d6efee4e_c0b10469","updated":"2022-06-22 08:15:23.000000000","message":"recheck","commit_id":"6db7153b342b8aad874f9b139f33a19269571a0e"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2785348f7f0222cc6e7fd5598e7a9b369d7ed238","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"572fc837_480d1cda","updated":"2022-06-20 23:28:20.000000000","message":"recheck\n\nFailed at \"Enable Ceph User\" which I looks irrelevant to this change.","commit_id":"6db7153b342b8aad874f9b139f33a19269571a0e"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"6e1eaa2ffe9feca1b5c9b5cec755714ba52f152f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2e0ee57d_ee2d200b","updated":"2022-07-04 11:37:15.000000000","message":"recheck\n\nFailed because of the issue caused by pacemaker, which we already sorted out.","commit_id":"6db7153b342b8aad874f9b139f33a19269571a0e"}],"ci/environments/scenario001-standalone.yaml":[{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"91c9bf7a79be57238a4d21fd5b9a776572e3e445","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  # NOTE(mwhahaha): cheat and use the swift-ring-builder \"service\" for our test"},{"line_number":35,"context_line":"  # service since the service is disabled. This allows us to not have to keep"},{"line_number":36,"context_line":"  # some test service in the defaults just for this case."},{"line_number":37,"context_line":"  OS::TripleO::Services::SwiftRingBuilder: ../../deployment/tests/test-task-core.yaml"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"parameter_defaults:"},{"line_number":40,"context_line":"  NodeDataLookup:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1b00d860_4dc2b313","line":37,"range":{"start_line":37,"start_character":48,"end_line":37,"end_character":85},"updated":"2022-06-15 05:55:45.000000000","message":"afaik.. we decided not to go ahead with task-core, Do we need to swap Swift here for test service or can we drop task core completely.\n\nI wonder if its okay to get rid of task core files from everywhere (in followups)","commit_id":"f08d44a19389b2bd25858a25a9ec2c41e0e79dca"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ab5c43ab08fade9e9768d9c252a1135f6c24ca35","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  # NOTE(mwhahaha): cheat and use the swift-ring-builder \"service\" for our test"},{"line_number":35,"context_line":"  # service since the service is disabled. This allows us to not have to keep"},{"line_number":36,"context_line":"  # some test service in the defaults just for this case."},{"line_number":37,"context_line":"  OS::TripleO::Services::SwiftRingBuilder: ../../deployment/tests/test-task-core.yaml"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"parameter_defaults:"},{"line_number":40,"context_line":"  NodeDataLookup:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"390b5295_9f72d7b1","line":37,"range":{"start_line":37,"start_character":48,"end_line":37,"end_character":85},"in_reply_to":"14bc710a_365c77d2","updated":"2022-06-15 07:41:13.000000000","message":"Maybe I should rebase this on top of https://review.opendev.org/c/openstack/tripleo-heat-templates/+/841131","commit_id":"f08d44a19389b2bd25858a25a9ec2c41e0e79dca"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"899bb1751f2909099f138d6d9f62bf89ffdf2cdd","unresolved":true,"context_lines":[{"line_number":34,"context_line":"  # NOTE(mwhahaha): cheat and use the swift-ring-builder \"service\" for our test"},{"line_number":35,"context_line":"  # service since the service is disabled. This allows us to not have to keep"},{"line_number":36,"context_line":"  # some test service in the defaults just for this case."},{"line_number":37,"context_line":"  OS::TripleO::Services::SwiftRingBuilder: ../../deployment/tests/test-task-core.yaml"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"parameter_defaults:"},{"line_number":40,"context_line":"  NodeDataLookup:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"14bc710a_365c77d2","line":37,"range":{"start_line":37,"start_character":48,"end_line":37,"end_character":85},"in_reply_to":"1b00d860_4dc2b313","updated":"2022-06-15 06:44:20.000000000","message":"probably this is nit from dev environment","commit_id":"f08d44a19389b2bd25858a25a9ec2c41e0e79dca"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2ba2af0423b88e8983d9652db19a1ceaa91b3dd5","unresolved":false,"context_lines":[{"line_number":34,"context_line":"  # NOTE(mwhahaha): cheat and use the swift-ring-builder \"service\" for our test"},{"line_number":35,"context_line":"  # service since the service is disabled. This allows us to not have to keep"},{"line_number":36,"context_line":"  # some test service in the defaults just for this case."},{"line_number":37,"context_line":"  OS::TripleO::Services::SwiftRingBuilder: ../../deployment/tests/test-task-core.yaml"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"parameter_defaults:"},{"line_number":40,"context_line":"  NodeDataLookup:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"a8f84530_7efd122b","line":37,"range":{"start_line":37,"start_character":48,"end_line":37,"end_character":85},"in_reply_to":"390b5295_9f72d7b1","updated":"2022-06-15 07:44:50.000000000","message":"Done","commit_id":"f08d44a19389b2bd25858a25a9ec2c41e0e79dca"}]}