)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"a196c6c9195ccc659f82a22f604d6c6c09781dac","unresolved":true,"context_lines":[{"line_number":10,"context_line":"This will be usefull to allow people to not easily remove tests from"},{"line_number":11,"context_line":"skiplist file, without the need to edit it manually."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I229608dd26a27d8b6f305e0044b65f70f252d676"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5a0ad2679d67c91328a18c75e15787b25fc6f34b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"d46a5497_b0511e99","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":5},"updated":"2022-09-21 12:48:41.000000000","message":"why do we have 2 of these?","commit_id":"c882aaf238691f6357ab78b652dace87406efa00"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"96c81c2039ebb49e88311c6557be5e27e209b028","unresolved":false,"context_lines":[{"line_number":10,"context_line":"This will be usefull to allow people to not easily remove tests from"},{"line_number":11,"context_line":"skiplist file, without the need to edit it manually."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I229608dd26a27d8b6f305e0044b65f70f252d676"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5a0ad2679d67c91328a18c75e15787b25fc6f34b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"46e65469_3aa9b0b3","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":5},"in_reply_to":"d46a5497_b0511e99","updated":"2022-09-21 18:35:35.000000000","message":"Done","commit_id":"c882aaf238691f6357ab78b652dace87406efa00"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"a196c6c9195ccc659f82a22f604d6c6c09781dac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"65337a32_b404b50c","updated":"2022-09-21 12:48:41.000000000","message":"it still behaves weird, it removes a comment with Note unexpectedly:\nhttps://paste.openstack.org/show/b2nGgjWsR1RG4h3VtO1B/","commit_id":"c882aaf238691f6357ab78b652dace87406efa00"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ec61110f4b5633a37da9fa97c42f66039acad6a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"84139798_e6824314","updated":"2022-09-22 08:11:22.000000000","message":"one more question, check out the output - https://paste.openstack.org/show/b1nbUOz41NOhdaMJg8KV/ - i tested a few edge scenarios, although, it doesn\u0027t need to be a bug, it depends on what we actually want remove command to remove","commit_id":"766862272b0dde82148de6296e65cdd2e9309c3d"},{"author":{"_account_id":30750,"name":"amolkahat","display_name":"Amol Kahat","email":"amolkahat@gmail.com","username":"amolkahat"},"change_message_id":"34dc86b72f8e441d71d4b60595c015fea03512fd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":14,"id":"e41bf869_50ddbbfc","in_reply_to":"84139798_e6824314","updated":"2022-09-23 11:37:00.000000000","message":"It also remove comments which starts with \u0027#\u0027. It may be because of the file rewrite. \n\nWe need some option to preserve comments. It is different discussion we can talk offline.","commit_id":"766862272b0dde82148de6296e65cdd2e9309c3d"}],"doc/source/removetest/removetest.rst":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5e62771a0d86290d0443a3f5fa379569694cf4d5","unresolved":true,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The `--file` and `--test` arguments are required, however the `--release`,"},{"line_number":15,"context_line":"`--job`, `--deployment` are not, and passing these arguments will refine the"},{"line_number":16,"context_line":"search for the job to be removed. For example, if you want to remove the test"},{"line_number":17,"context_line":"tempest.scenario.test1 from release master, but don\u0027t want to touch the"},{"line_number":18,"context_line":"deployment or jobs, just use the following command::"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"633ef8dd_89b1c5b2","line":16,"range":{"start_line":16,"start_character":15,"end_line":16,"end_character":18},"updated":"2021-02-03 19:21:17.000000000","message":"test?","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"0093b32f68b90267236e5d6defa5d014c07b7b3b","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The `--file` and `--test` arguments are required, however the `--release`,"},{"line_number":15,"context_line":"`--job`, `--deployment` are not, and passing these arguments will refine the"},{"line_number":16,"context_line":"search for the job to be removed. For example, if you want to remove the test"},{"line_number":17,"context_line":"tempest.scenario.test1 from release master, but don\u0027t want to touch the"},{"line_number":18,"context_line":"deployment or jobs, just use the following command::"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-rst","patch_set":5,"id":"5d0e64cf_8772f24d","line":16,"range":{"start_line":16,"start_character":15,"end_line":16,"end_character":18},"in_reply_to":"633ef8dd_89b1c5b2","updated":"2022-09-20 11:11:07.000000000","message":"Done","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5e62771a0d86290d0443a3f5fa379569694cf4d5","unresolved":true,"context_lines":[{"line_number":34,"context_line":"The command above will remove the test from release master and the deployment"},{"line_number":35,"context_line":"undercloud as well."},{"line_number":36,"context_line":"If you want to only remove the deployment, but keep all the releases that the"},{"line_number":37,"context_line":"test should skipp, all you have to do is don\u0027t pass the `--release` argument::"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    $ tempest-skip remove --file roles/validate-tempest/vars/tempest_skip.yml \\"}],"source_content_type":"text/x-rst","patch_set":5,"id":"2be2210e_b081f60f","line":37,"range":{"start_line":37,"start_character":15,"end_line":37,"end_character":17},"updated":"2021-02-03 19:21:17.000000000","message":"one \u0027p\u0027 extra","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"b694c3816be8593351aaa0fcaf08f711e645fdbe","unresolved":false,"context_lines":[{"line_number":34,"context_line":"The command above will remove the test from release master and the deployment"},{"line_number":35,"context_line":"undercloud as well."},{"line_number":36,"context_line":"If you want to only remove the deployment, but keep all the releases that the"},{"line_number":37,"context_line":"test should skipp, all you have to do is don\u0027t pass the `--release` argument::"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    $ tempest-skip remove --file roles/validate-tempest/vars/tempest_skip.yml \\"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1fdbda02_b3a02798","line":37,"range":{"start_line":37,"start_character":15,"end_line":37,"end_character":17},"in_reply_to":"2be2210e_b081f60f","updated":"2022-09-21 18:36:14.000000000","message":"Done","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ec61110f4b5633a37da9fa97c42f66039acad6a0","unresolved":true,"context_lines":[{"line_number":11,"context_line":"In order to avoid that, you can use the `remove` command that will search for"},{"line_number":12,"context_line":"the match passed by argument line, and remove it."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The `--file` and `--release` arguments is required, however the `--test`,"},{"line_number":15,"context_line":"`--job`, `--deployment` are not, and passing these arguments will refine the"},{"line_number":16,"context_line":"search for the test to be removed. For example, if you want to remove the test"},{"line_number":17,"context_line":"tempest.scenario.test1 from release master, but don\u0027t want to touch the"}],"source_content_type":"text/x-rst","patch_set":14,"id":"c3d854ae_91bd078b","line":14,"range":{"start_line":14,"start_character":39,"end_line":14,"end_character":41},"updated":"2022-09-22 08:11:22.000000000","message":"nit: are","commit_id":"766862272b0dde82148de6296e65cdd2e9309c3d"}],"tempest_skip/removetest.py":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5e62771a0d86290d0443a3f5fa379569694cf4d5","unresolved":true,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":"# Copyright 2021 Red Hat, Inc."},{"line_number":3,"context_line":"# All Rights Reserved."},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":5,"id":"07f536a6_4867e014","line":1,"updated":"2021-02-03 19:21:17.000000000","message":"extra line?","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"0093b32f68b90267236e5d6defa5d014c07b7b3b","unresolved":false,"context_lines":[{"line_number":1,"context_line":""},{"line_number":2,"context_line":"# Copyright 2021 Red Hat, Inc."},{"line_number":3,"context_line":"# All Rights Reserved."},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":5,"id":"93ffa374_7f0178a6","line":1,"in_reply_to":"07f536a6_4867e014","updated":"2022-09-20 11:11:07.000000000","message":"Done","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5e62771a0d86290d0443a3f5fa379569694cf4d5","unresolved":true,"context_lines":[{"line_number":67,"context_line":"        parser.add_argument(\u0027--file\u0027, dest\u003d\u0027file\u0027, required\u003dTrue,"},{"line_number":68,"context_line":"                            help\u003d\u0027Skiplist config file\u0027)"},{"line_number":69,"context_line":"        parser.add_argument(\u0027--release\u0027, dest\u003d\u0027release\u0027,"},{"line_number":70,"context_line":"                            help\u003d\u0027Release where the test will be added\u0027)"},{"line_number":71,"context_line":"        parser.add_argument(\u0027--job\u0027, dest\u003d\u0027job\u0027,"},{"line_number":72,"context_line":"                            help\u003d\u0027Specify in which job this test will be \u0027"},{"line_number":73,"context_line":"                                 \u0027skipped\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"cdbe5070_e852e97d","line":70,"range":{"start_line":70,"start_character":65,"end_line":70,"end_character":70},"updated":"2021-02-03 19:21:17.000000000","message":"removed","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"0093b32f68b90267236e5d6defa5d014c07b7b3b","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        parser.add_argument(\u0027--file\u0027, dest\u003d\u0027file\u0027, required\u003dTrue,"},{"line_number":68,"context_line":"                            help\u003d\u0027Skiplist config file\u0027)"},{"line_number":69,"context_line":"        parser.add_argument(\u0027--release\u0027, dest\u003d\u0027release\u0027,"},{"line_number":70,"context_line":"                            help\u003d\u0027Release where the test will be added\u0027)"},{"line_number":71,"context_line":"        parser.add_argument(\u0027--job\u0027, dest\u003d\u0027job\u0027,"},{"line_number":72,"context_line":"                            help\u003d\u0027Specify in which job this test will be \u0027"},{"line_number":73,"context_line":"                                 \u0027skipped\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"54e26d16_3f7f01f9","line":70,"range":{"start_line":70,"start_character":65,"end_line":70,"end_character":70},"in_reply_to":"cdbe5070_e852e97d","updated":"2022-09-20 11:11:07.000000000","message":"Done","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5e62771a0d86290d0443a3f5fa379569694cf4d5","unresolved":true,"context_lines":[{"line_number":70,"context_line":"                            help\u003d\u0027Release where the test will be added\u0027)"},{"line_number":71,"context_line":"        parser.add_argument(\u0027--job\u0027, dest\u003d\u0027job\u0027,"},{"line_number":72,"context_line":"                            help\u003d\u0027Specify in which job this test will be \u0027"},{"line_number":73,"context_line":"                                 \u0027skipped\u0027)"},{"line_number":74,"context_line":"        parser.add_argument(\u0027--test\u0027, dest\u003d\u0027test\u0027, required\u003dTrue,"},{"line_number":75,"context_line":"                            help\u003d\u0027Test to be skipped\u0027)"},{"line_number":76,"context_line":"        parser.add_argument(\u0027--deployment\u0027, dest\u003d\u0027deployment\u0027, required\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":5,"id":"d173ff2d_fe9d981f","line":73,"range":{"start_line":73,"start_character":33,"end_line":73,"end_character":43},"updated":"2021-02-03 19:21:17.000000000","message":"this probably applies on addtest scenario too","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"0093b32f68b90267236e5d6defa5d014c07b7b3b","unresolved":false,"context_lines":[{"line_number":70,"context_line":"                            help\u003d\u0027Release where the test will be added\u0027)"},{"line_number":71,"context_line":"        parser.add_argument(\u0027--job\u0027, dest\u003d\u0027job\u0027,"},{"line_number":72,"context_line":"                            help\u003d\u0027Specify in which job this test will be \u0027"},{"line_number":73,"context_line":"                                 \u0027skipped\u0027)"},{"line_number":74,"context_line":"        parser.add_argument(\u0027--test\u0027, dest\u003d\u0027test\u0027, required\u003dTrue,"},{"line_number":75,"context_line":"                            help\u003d\u0027Test to be skipped\u0027)"},{"line_number":76,"context_line":"        parser.add_argument(\u0027--deployment\u0027, dest\u003d\u0027deployment\u0027, required\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":5,"id":"03ab9fb7_e4783307","line":73,"range":{"start_line":73,"start_character":33,"end_line":73,"end_character":43},"in_reply_to":"d173ff2d_fe9d981f","updated":"2022-09-20 11:11:07.000000000","message":"Done","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5e62771a0d86290d0443a3f5fa379569694cf4d5","unresolved":true,"context_lines":[{"line_number":72,"context_line":"                            help\u003d\u0027Specify in which job this test will be \u0027"},{"line_number":73,"context_line":"                                 \u0027skipped\u0027)"},{"line_number":74,"context_line":"        parser.add_argument(\u0027--test\u0027, dest\u003d\u0027test\u0027, required\u003dTrue,"},{"line_number":75,"context_line":"                            help\u003d\u0027Test to be skipped\u0027)"},{"line_number":76,"context_line":"        parser.add_argument(\u0027--deployment\u0027, dest\u003d\u0027deployment\u0027, required\u003dFalse,"},{"line_number":77,"context_line":"                            choices\u003d[\u0027overcloud\u0027, \u0027undercloud\u0027])"},{"line_number":78,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":5,"id":"4c4e5ac4_64d80f45","line":75,"range":{"start_line":75,"start_character":35,"end_line":75,"end_character":52},"updated":"2021-02-03 19:21:17.000000000","message":"ditto","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"0093b32f68b90267236e5d6defa5d014c07b7b3b","unresolved":false,"context_lines":[{"line_number":72,"context_line":"                            help\u003d\u0027Specify in which job this test will be \u0027"},{"line_number":73,"context_line":"                                 \u0027skipped\u0027)"},{"line_number":74,"context_line":"        parser.add_argument(\u0027--test\u0027, dest\u003d\u0027test\u0027, required\u003dTrue,"},{"line_number":75,"context_line":"                            help\u003d\u0027Test to be skipped\u0027)"},{"line_number":76,"context_line":"        parser.add_argument(\u0027--deployment\u0027, dest\u003d\u0027deployment\u0027, required\u003dFalse,"},{"line_number":77,"context_line":"                            choices\u003d[\u0027overcloud\u0027, \u0027undercloud\u0027])"},{"line_number":78,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":5,"id":"2dd7f3db_de9b5740","line":75,"range":{"start_line":75,"start_character":35,"end_line":75,"end_character":52},"in_reply_to":"4c4e5ac4_64d80f45","updated":"2022-09-20 11:11:07.000000000","message":"Done","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5e62771a0d86290d0443a3f5fa379569694cf4d5","unresolved":true,"context_lines":[{"line_number":73,"context_line":"                                 \u0027skipped\u0027)"},{"line_number":74,"context_line":"        parser.add_argument(\u0027--test\u0027, dest\u003d\u0027test\u0027, required\u003dTrue,"},{"line_number":75,"context_line":"                            help\u003d\u0027Test to be skipped\u0027)"},{"line_number":76,"context_line":"        parser.add_argument(\u0027--deployment\u0027, dest\u003d\u0027deployment\u0027, required\u003dFalse,"},{"line_number":77,"context_line":"                            choices\u003d[\u0027overcloud\u0027, \u0027undercloud\u0027])"},{"line_number":78,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":5,"id":"53645894_5a9d5267","line":76,"range":{"start_line":76,"start_character":29,"end_line":76,"end_character":41},"updated":"2021-02-03 19:21:17.000000000","message":"no help msg?","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"0093b32f68b90267236e5d6defa5d014c07b7b3b","unresolved":false,"context_lines":[{"line_number":73,"context_line":"                                 \u0027skipped\u0027)"},{"line_number":74,"context_line":"        parser.add_argument(\u0027--test\u0027, dest\u003d\u0027test\u0027, required\u003dTrue,"},{"line_number":75,"context_line":"                            help\u003d\u0027Test to be skipped\u0027)"},{"line_number":76,"context_line":"        parser.add_argument(\u0027--deployment\u0027, dest\u003d\u0027deployment\u0027, required\u003dFalse,"},{"line_number":77,"context_line":"                            choices\u003d[\u0027overcloud\u0027, \u0027undercloud\u0027])"},{"line_number":78,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":5,"id":"d47fb01b_7abfe5d4","line":76,"range":{"start_line":76,"start_character":29,"end_line":76,"end_character":41},"in_reply_to":"53645894_5a9d5267","updated":"2022-09-20 11:11:07.000000000","message":"Done","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"}],"tempest_skip/tests/test_remove_test.py":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"5e62771a0d86290d0443a3f5fa379569694cf4d5","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright 2020 Red Hat, Inc."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":5,"id":"32943143_e0836cc0","line":1,"range":{"start_line":1,"start_character":12,"end_line":1,"end_character":16},"updated":"2021-02-03 19:21:17.000000000","message":"2021? I\u0027ve just noticed that the previous file has 2021, but I don\u0027t really care .. 😊","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"ded18dd7c1f98990c82743b4b5eca975e73aeebe","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2020 Red Hat, Inc."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":5,"id":"c2f83b8a_be7efca9","line":1,"range":{"start_line":1,"start_character":12,"end_line":1,"end_character":16},"in_reply_to":"32943143_e0836cc0","updated":"2022-09-20 11:11:48.000000000","message":"Done","commit_id":"e26faa6a651ff6f40492bc4f9dc8d47773b26319"}]}