)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Arx Cruz \u003carxcruz@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-01-14 18:20:11 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Ad addtest command to tempest-skiplist"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the addtest command to tempest-skiplist in order to"},{"line_number":10,"context_line":"avoid mistakes like duplicated entries, invalid yaml file format, add"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"1bfad431_660c690e","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"updated":"2021-01-15 17:44:13.000000000","message":"Add","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9132c74cddfd4448ea6d237119fa1fb45a51989b","unresolved":true,"context_lines":[{"line_number":11,"context_line":"namespace instead of full test path. The idea here is to help ruck and"},{"line_number":12,"context_line":"rovers to add tests to skiplist, without the need to edit long yaml"},{"line_number":13,"context_line":"file, and be able to quickly select tests under a specific namespace to"},{"line_number":14,"context_line":"be added without all the boilplate of create entries for each test."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ibedd2c1307940f01229ecce2a683fdfd51dc612e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"768a546e_ac5cf344","line":14,"range":{"start_line":14,"start_character":25,"end_line":14,"end_character":34},"updated":"2021-01-19 08:26:00.000000000","message":"boilerplate","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"}],"doc/source/addtest/addtest.rst":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":5,"context_line":"Adding tests"},{"line_number":6,"context_line":"------------"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"Of course is possible to directly edit the yaml file and add the test itself,"},{"line_number":9,"context_line":"but that leads to failures, as for example duplicated entries, identation"},{"line_number":10,"context_line":"issues, have a namespace test instead of full path test name."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"dd01ba53_9d63fc47","line":8,"range":{"start_line":8,"start_character":10,"end_line":8,"end_character":12},"updated":"2021-01-15 17:44:13.000000000","message":"it is","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":6,"context_line":"------------"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"Of course is possible to directly edit the yaml file and add the test itself,"},{"line_number":9,"context_line":"but that leads to failures, as for example duplicated entries, identation"},{"line_number":10,"context_line":"issues, have a namespace test instead of full path test name."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"In order to avoid that, you can use the `addtest` command that will identify if"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5de25c86_fcab707a","line":9,"range":{"start_line":9,"start_character":9,"end_line":9,"end_character":14},"updated":"2021-01-15 17:44:13.000000000","message":"I\u0027d use \"may lead\" as it doesn\u0027t always leads to failures.","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":6,"context_line":"------------"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"Of course is possible to directly edit the yaml file and add the test itself,"},{"line_number":9,"context_line":"but that leads to failures, as for example duplicated entries, identation"},{"line_number":10,"context_line":"issues, have a namespace test instead of full path test name."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"In order to avoid that, you can use the `addtest` command that will identify if"}],"source_content_type":"text/x-rst","patch_set":7,"id":"a6bfb157_bfdfca74","line":9,"range":{"start_line":9,"start_character":63,"end_line":9,"end_character":73},"updated":"2021-01-15 17:44:13.000000000","message":"indentation","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"Of course is possible to directly edit the yaml file and add the test itself,"},{"line_number":9,"context_line":"but that leads to failures, as for example duplicated entries, identation"},{"line_number":10,"context_line":"issues, have a namespace test instead of full path test name."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"In order to avoid that, you can use the `addtest` command that will identify if"},{"line_number":13,"context_line":"the test is already on the skipplist, avoiding duplication. It will also"}],"source_content_type":"text/x-rst","patch_set":7,"id":"7221e3f5_f1b2dac8","line":10,"range":{"start_line":10,"start_character":8,"end_line":10,"end_character":12},"updated":"2021-01-15 17:44:13.000000000","message":"having","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":10,"context_line":"issues, have a namespace test instead of full path test name."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"In order to avoid that, you can use the `addtest` command that will identify if"},{"line_number":13,"context_line":"the test is already on the skipplist, avoiding duplication. It will also"},{"line_number":14,"context_line":"generate the yaml file properly as well as avoid the use of test namespace. For"},{"line_number":15,"context_line":"example, if the user try to add the tempest.scenario, it will skip all the"},{"line_number":16,"context_line":"tests under tempest.scenario, which is not the desirable behavior. However, it"}],"source_content_type":"text/x-rst","patch_set":7,"id":"23362a7d_e60b79c3","line":13,"range":{"start_line":13,"start_character":30,"end_line":13,"end_character":32},"updated":"2021-01-15 17:44:13.000000000","message":"just one \"p\"","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":12,"context_line":"In order to avoid that, you can use the `addtest` command that will identify if"},{"line_number":13,"context_line":"the test is already on the skipplist, avoiding duplication. It will also"},{"line_number":14,"context_line":"generate the yaml file properly as well as avoid the use of test namespace. For"},{"line_number":15,"context_line":"example, if the user try to add the tempest.scenario, it will skip all the"},{"line_number":16,"context_line":"tests under tempest.scenario, which is not the desirable behavior. However, it"},{"line_number":17,"context_line":"is a lot of work to add each entry under tempest.scenario, since we must repeat"},{"line_number":18,"context_line":"all the reasons, bugzilla, releases, etc."}],"source_content_type":"text/x-rst","patch_set":7,"id":"d3d01959_21a82bb5","line":15,"range":{"start_line":15,"start_character":12,"end_line":15,"end_character":15},"updated":"2021-01-15 17:44:13.000000000","message":"a","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":12,"context_line":"In order to avoid that, you can use the `addtest` command that will identify if"},{"line_number":13,"context_line":"the test is already on the skipplist, avoiding duplication. It will also"},{"line_number":14,"context_line":"generate the yaml file properly as well as avoid the use of test namespace. For"},{"line_number":15,"context_line":"example, if the user try to add the tempest.scenario, it will skip all the"},{"line_number":16,"context_line":"tests under tempest.scenario, which is not the desirable behavior. However, it"},{"line_number":17,"context_line":"is a lot of work to add each entry under tempest.scenario, since we must repeat"},{"line_number":18,"context_line":"all the reasons, bugzilla, releases, etc."}],"source_content_type":"text/x-rst","patch_set":7,"id":"ab52f42a_42c26c31","line":15,"range":{"start_line":15,"start_character":21,"end_line":15,"end_character":24},"updated":"2021-01-15 17:44:13.000000000","message":"tries","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":17,"context_line":"is a lot of work to add each entry under tempest.scenario, since we must repeat"},{"line_number":18,"context_line":"all the reasons, bugzilla, releases, etc."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"The addtest command solve all this problems for you. First of all, when you try"},{"line_number":21,"context_line":"to add a test passing a test namespace, addtest will gave you a list of tests"},{"line_number":22,"context_line":"under that particular namespace, where you can choose from the list which ones"},{"line_number":23,"context_line":"you would like to add. Select the ones you want and you are done!"}],"source_content_type":"text/x-rst","patch_set":7,"id":"5eb9d269_fc4da7be","line":20,"range":{"start_line":20,"start_character":20,"end_line":20,"end_character":25},"updated":"2021-01-15 17:44:13.000000000","message":"solves","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":17,"context_line":"is a lot of work to add each entry under tempest.scenario, since we must repeat"},{"line_number":18,"context_line":"all the reasons, bugzilla, releases, etc."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"The addtest command solve all this problems for you. First of all, when you try"},{"line_number":21,"context_line":"to add a test passing a test namespace, addtest will gave you a list of tests"},{"line_number":22,"context_line":"under that particular namespace, where you can choose from the list which ones"},{"line_number":23,"context_line":"you would like to add. Select the ones you want and you are done!"}],"source_content_type":"text/x-rst","patch_set":7,"id":"8bd877e7_60973739","line":20,"range":{"start_line":20,"start_character":30,"end_line":20,"end_character":34},"updated":"2021-01-15 17:44:13.000000000","message":"these","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":18,"context_line":"all the reasons, bugzilla, releases, etc."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"The addtest command solve all this problems for you. First of all, when you try"},{"line_number":21,"context_line":"to add a test passing a test namespace, addtest will gave you a list of tests"},{"line_number":22,"context_line":"under that particular namespace, where you can choose from the list which ones"},{"line_number":23,"context_line":"you would like to add. Select the ones you want and you are done!"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-rst","patch_set":7,"id":"2776ed30_c075bc89","line":21,"range":{"start_line":21,"start_character":53,"end_line":21,"end_character":57},"updated":"2021-01-15 17:44:13.000000000","message":"give","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":25,"context_line":"Examples"},{"line_number":26,"context_line":"--------"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"The command below add the test tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_connectivity_between_vms_on_different_networks::"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    $ tempest-skip addtest \\"},{"line_number":31,"context_line":"                   --file roles/validate-tempest/vars/tempest_skip.yml \\"}],"source_content_type":"text/x-rst","patch_set":7,"id":"a34534fe_066d8535","line":28,"range":{"start_line":28,"start_character":18,"end_line":28,"end_character":21},"updated":"2021-01-15 17:44:13.000000000","message":"adds","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"You can use the arrow keys on your keyboard to navigate through the list, and"},{"line_number":66,"context_line":"space to select. Once you are done, just press enter, the command will go"},{"line_number":67,"context_line":"through each test, check if the test exists or not. If exists, it will also"},{"line_number":68,"context_line":"check the release exists, and properly add the test."},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Once you are done, you can validate if the yaml file was generated properly"}],"source_content_type":"text/x-rst","patch_set":7,"id":"6aad1997_2517009f","line":67,"range":{"start_line":67,"start_character":54,"end_line":67,"end_character":55},"updated":"2021-01-15 17:44:13.000000000","message":"it","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"0fc407465beac92b6ff73b5f9544057ba58031a8","unresolved":true,"context_lines":[{"line_number":25,"context_line":"Examples"},{"line_number":26,"context_line":"--------"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"The command below add the test tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_connectivity_between_vms_on_different_networks::"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    $ tempest-skip addtest \\"},{"line_number":31,"context_line":"                   --file roles/validate-tempest/vars/tempest_skip.yml \\"}],"source_content_type":"text/x-rst","patch_set":11,"id":"0c04917d_cc1654dd","line":28,"updated":"2021-01-18 18:43:26.000000000","message":"adds","commit_id":"e74fc737147408d6d752a80ee0f11b507184d8bb"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9132c74cddfd4448ea6d237119fa1fb45a51989b","unresolved":true,"context_lines":[{"line_number":32,"context_line":"                   --release master \\"},{"line_number":33,"context_line":"                   --test tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_connectivity_between_vms_on_different_networks \\"},{"line_number":34,"context_line":"                   --reason \u0027Failing on network\u0027 \\"},{"line_number":35,"context_line":"                   --lp https://launchpad.net/bug/12345"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"In this example, we are adding the full path test, and so, the command will not"},{"line_number":38,"context_line":"prompt you a list of tests that you want to choose. If for example, only the"}],"source_content_type":"text/x-rst","patch_set":11,"id":"4465fae6_79dda1c8","line":35,"updated":"2021-01-19 08:26:00.000000000","message":"i tried following this example but it didn\u0027t work for me in particular it doesn\u0027t like the value of \u0027--reason\u0027:\n\n[m@fedora openstack-tempest-skiplist]$ source .tox/venv/bin/activate\n(venv) [m@fedora openstack-tempest-skiplist]$ tempest-skip\n(tempest-skip) addtest\nusage: addtest [-h] --file FILE [--release RELEASE] [--job JOB] --test TEST (--lp LP | --bz BZ)\n               --reason REASON [--deployment {overcloud,undercloud}]\naddtest: error: the following arguments are required: --file, --test, --reason\n\n\n(tempest-skip) addtest  --file roles/validate-tempest/vars/tempest_skip.yml  --release master --test tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_connectivity_between_vms_on_different_networks --reason \u0027Failing on network\u0027   --lp https://launchpad.net/bug/12345\nusage: addtest [-h] --file FILE [--release RELEASE] [--job JOB] --test TEST (--lp LP | --bz BZ)\n               --reason REASON [--deployment {overcloud,undercloud}]\naddtest: error: unrecognized arguments: on network\n\n\nwhen i leave only one work then it works OK:\n\n(tempest-skip) addtest  --file roles/validate-tempest/vars/tempest_skip.yml  --release master --test tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_connectivity_between_vms_on_different_networks --reason \"Failing\" --lp https://launchpad.net/bug/12345\n2021-01-19 10:23:40.954 52307 INFO tempest [-] Using tempest config file /etc/tempest/tempest.conf\n/home/m/Documents/WORK/NewEclipsey/Github/Tuskar/openstack-tempest-skiplist/.tox/venv/lib/python3.9/site-packages/oslo_config/types.py:57: UserWarning: converting \u00273000\u0027 to a string\n  warnings.warn(\u0027converting \\\u0027%s\\\u0027 to a string\u0027 % str_val)\n(tempest-skip)","commit_id":"e74fc737147408d6d752a80ee0f11b507184d8bb"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"27b80beb550689d773ee4565bd26eebc78222cf3","unresolved":true,"context_lines":[{"line_number":32,"context_line":"                   --release master \\"},{"line_number":33,"context_line":"                   --test tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_connectivity_between_vms_on_different_networks \\"},{"line_number":34,"context_line":"                   --reason \u0027Failing on network\u0027 \\"},{"line_number":35,"context_line":"                   --lp https://launchpad.net/bug/12345"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"In this example, we are adding the full path test, and so, the command will not"},{"line_number":38,"context_line":"prompt you a list of tests that you want to choose. If for example, only the"}],"source_content_type":"text/x-rst","patch_set":11,"id":"30b494fa_6e7be82e","line":35,"in_reply_to":"4465fae6_79dda1c8","updated":"2021-01-19 08:28:17.000000000","message":"OK, when i ran it like this it works OK:\n\n(venv) [m@fedora openstack-tempest-skiplist]$ tempest-skip addtest \\\n\u003e                    --file roles/validate-tempest/vars/tempest_skip.yml \\\n\u003e                    --release master \\\n\u003e                    --test tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_connectivity_between_vms_on_different_networks \\\n\u003e                    --reason \u0027Failing on network\u0027 \\\n\u003e                    --lp https://launchpad.net/bug/12345\n2021-01-19 10:27:28.766 52463 INFO tempest [-] Using tempest config file /etc/tempest/tempest.conf\n/home/m/Documents/WORK/NewEclipsey/Github/Tuskar/openstack-tempest-skiplist/.tox/venv/lib/python3.9/site-packages/oslo_config/types.py:57: UserWarning: converting \u00273000\u0027 to a string\n  warnings.warn(\u0027converting \\\u0027%s\\\u0027 to a string\u0027 % str_val)\n2021-01-19 10:27:30.192 52463 WARNING tempest_skip.add_test [-] Test tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_connectivity_between_vms_on_different_networks already exist for release master, doing nothing","commit_id":"e74fc737147408d6d752a80ee0f11b507184d8bb"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"0fc407465beac92b6ff73b5f9544057ba58031a8","unresolved":true,"context_lines":[{"line_number":36,"context_line":""},{"line_number":37,"context_line":"In this example, we are adding the full path test, and so, the command will not"},{"line_number":38,"context_line":"prompt you a list of tests that you want to choose. If for example, only the"},{"line_number":39,"context_line":"namespace be parsed, you will be prompted with a list of tests under that"},{"line_number":40,"context_line":"namespace::"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    $ tempest-skip addtest \\"}],"source_content_type":"text/x-rst","patch_set":11,"id":"db4bf5f0_34e7753e","line":39,"updated":"2021-01-18 18:43:26.000000000","message":"parsed or passed? not sure of the meaning here","commit_id":"e74fc737147408d6d752a80ee0f11b507184d8bb"}],"tempest_skip/add_test.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"dfec1387d56a442e02674e584b131c3880757614","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        test_list \u003d self._get_tests_list()"},{"line_number":36,"context_line":"        filter_tests \u003d list(filter(lambda x: parsed_args.test in x, test_list))"},{"line_number":37,"context_line":"        if len(filter_tests) \u003e 1:"},{"line_number":38,"context_line":"            question \u003d inquirer.CheckBox(\u0027list_tests\u0027,"},{"line_number":39,"context_line":"                                         \u0027These are the tests available on the\u0027"},{"line_number":40,"context_line":"                                         \u0027 namespace, choose which ones you \u0027"},{"line_number":41,"context_line":"                                         \u0027want to add\u0027, choices\u003dfilter_tests)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_cfb3139a","line":38,"updated":"2020-09-29 13:06:36.000000000","message":"pep8: F841 local variable \u0027question\u0027 is assigned to but never used","commit_id":"be2996d8b98c6d21a57b18543d7b4ac0a6025753"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"dfec1387d56a442e02674e584b131c3880757614","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        return tempest_cmd.take_action(parser)"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def get_parser(self, prog_name):"},{"line_number":76,"context_line":"        parser \u003d super(AddTest, self).get_parser(prog_name)"},{"line_number":77,"context_line":"        parser.add_argument(\u0027--file\u0027, dest\u003d\u0027file\u0027, required\u003dTrue,"},{"line_number":78,"context_line":"                            help\u003d\u0027Skiplist config file\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_afb85f75","line":75,"updated":"2020-09-29 13:06:36.000000000","message":"pep8: E303 too many blank lines (2)","commit_id":"be2996d8b98c6d21a57b18543d7b4ac0a6025753"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"dfec1387d56a442e02674e584b131c3880757614","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        parser.add_argument(\u0027--reason\u0027, dest\u003d\u0027reason\u0027, required\u003dTrue,"},{"line_number":91,"context_line":"                            help\u003d\u0027Reason to test be skipped\u0027)"},{"line_number":92,"context_line":"        return parser"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f560f44_0fad0b32","line":93,"updated":"2020-09-29 13:06:36.000000000","message":"pep8: W391 blank line at end of file","commit_id":"be2996d8b98c6d21a57b18543d7b4ac0a6025753"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"12899c453082232368b1833fe2726d598ee0ef81","unresolved":false,"context_lines":[{"line_number":93,"context_line":"            print(\u0027Test already exist, doing nothing\u0027)"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    def _get_tests_list(self):"},{"line_number":97,"context_line":"        tempest_run \u003d TempestRun(__name__, [])"},{"line_number":98,"context_line":"        tempest_init \u003d TempestInit(__name__, [])"},{"line_number":99,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_43f99c4c","line":96,"updated":"2020-09-29 15:57:02.000000000","message":"pep8: E303 too many blank lines (2)","commit_id":"25264cf83cd0c46b7f99e1326b0740c51c97ac7b"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"12899c453082232368b1833fe2726d598ee0ef81","unresolved":false,"context_lines":[{"line_number":134,"context_line":"        return [i.split(\u0027[\u0027)[0] for i in ids]"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    def get_parser(self, prog_name):"},{"line_number":138,"context_line":"        parser \u003d super(AddTest, self).get_parser(prog_name)"},{"line_number":139,"context_line":"        parser.add_argument(\u0027--file\u0027, dest\u003d\u0027file\u0027, required\u003dTrue,"},{"line_number":140,"context_line":"                            help\u003d\u0027Skiplist config file\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_23f6285f","line":137,"updated":"2020-09-29 15:57:02.000000000","message":"pep8: E303 too many blank lines (2)","commit_id":"25264cf83cd0c46b7f99e1326b0740c51c97ac7b"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"12899c453082232368b1833fe2726d598ee0ef81","unresolved":false,"context_lines":[{"line_number":152,"context_line":"        parser.add_argument(\u0027--reason\u0027, dest\u003d\u0027reason\u0027, required\u003dTrue,"},{"line_number":153,"context_line":"                            help\u003d\u0027Reason to test be skipped\u0027)"},{"line_number":154,"context_line":"        return parser"},{"line_number":155,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9f560f44_8390d4f5","line":155,"updated":"2020-09-29 15:57:02.000000000","message":"pep8: W391 blank line at end of file","commit_id":"25264cf83cd0c46b7f99e1326b0740c51c97ac7b"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"f9e5f92ef62583e2eb56c448f890efaee49cd216","unresolved":true,"context_lines":[{"line_number":59,"context_line":"                                   bz\u003dparsed_args.bz, job\u003dparsed_args.job)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        with open(self.abs_path, \u0027w\u0027) as f:"},{"line_number":62,"context_line":"            yaml.dump(self.yaml_file, f, Dumper\u003dyaml.RoundTripDumper)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def _add_test_in_yaml(self, test_name, deployment, release, reason,"},{"line_number":65,"context_line":"                          lp\u003dNone, bz\u003dNone, job\u003dNone):"}],"source_content_type":"text/x-python","patch_set":11,"id":"f6589e9f_ed8cc241","line":62,"range":{"start_line":62,"start_character":17,"end_line":62,"end_character":21},"updated":"2021-01-20 08:18:10.000000000","message":"i was gonna suggest using sort_keys\u003dFalse here to solve your keys ordering diff but it seems there is no such thing in ruamel\n\nby digging though I came across ruamel round_trip_load and round_trip_dump which I believe preserves the original format (e.g. there https://stackoverflow.com/questions/37139553/pyyaml-safe-dump-adding-line-breaks-and-indent-to-the-yaml-file#37140044 )","commit_id":"e74fc737147408d6d752a80ee0f11b507184d8bb"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"fffcab6948f431fa86c186c8a70e8da7c71fd043","unresolved":true,"context_lines":[{"line_number":59,"context_line":"                                   bz\u003dparsed_args.bz, job\u003dparsed_args.job)"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        with open(self.abs_path, \u0027w\u0027) as f:"},{"line_number":62,"context_line":"            yaml.dump(self.yaml_file, f, Dumper\u003dyaml.RoundTripDumper)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def _add_test_in_yaml(self, test_name, deployment, release, reason,"},{"line_number":65,"context_line":"                          lp\u003dNone, bz\u003dNone, job\u003dNone):"}],"source_content_type":"text/x-python","patch_set":11,"id":"2e659c00_48e61f7d","line":62,"range":{"start_line":62,"start_character":17,"end_line":62,"end_character":21},"in_reply_to":"f6589e9f_ed8cc241","updated":"2021-01-25 10:30:37.000000000","message":"Thanks, i update the patch with this, although it keeps the format, it removes the \u0027\u0027 on some texts, but it\u0027s much better than before","commit_id":"e74fc737147408d6d752a80ee0f11b507184d8bb"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"86d54672392ae99266aa706b9733c1f432a23cb7","unresolved":true,"context_lines":[{"line_number":36,"context_line":"    def take_action(self, parsed_args):"},{"line_number":37,"context_line":"        self.log.debug(\u0027Running add_test command\u0027)"},{"line_number":38,"context_line":"        self.abs_path \u003d os.path.abspath(parsed_args.file)"},{"line_number":39,"context_line":"        self.yaml_file \u003d yaml.round_trip_load(open(self.abs_path))"},{"line_number":40,"context_line":"        test_list \u003d self._get_tests_list()"},{"line_number":41,"context_line":"        filter_tests \u003d list(filter(lambda x: parsed_args.test in x, test_list))"},{"line_number":42,"context_line":"        if len(filter_tests) \u003e 1:"}],"source_content_type":"text/x-python","patch_set":13,"id":"22ee293d_86f10e9e","line":39,"range":{"start_line":39,"start_character":30,"end_line":39,"end_character":45},"updated":"2021-01-26 08:59:33.000000000","message":"this has a preserve_quotes\u003dTrue argument you should use it\n\n 39         self.yaml_file \u003d yaml.round_trip_load(open(self.abs_path), preserve_quotes\u003dTrue)\n\n\nthen the result is even better, it includes the \u0027\u0027 but starts to wrap those quoted lines e.g. \n\n\n-        reason: \u0027This test was enabled recently on ovn and we miss configuration in tripleo jobs, for sake of CI skipping them for now, but we will enabled them as fast as possible\u0027\n\n+        reason: \u0027This test was enabled recently on ovn and we miss configuration in\n+            tripleo jobs, for sake of CI skipping them for now, but we will enabled\n+            them as fast as possible\u0027\n\n\nmaybe you can dig some more at that but seriously ruamel docs are S***T","commit_id":"e4cc20b59ad5195227e72bed40a3e4da8527e2f4"}],"test-requirements.txt":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"ccfbafe80b7334065486a50a20ce49e33d840107","unresolved":true,"context_lines":[{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"flake8\u003d\u003d3.8.3 # MIT"},{"line_number":6,"context_line":"mock"},{"line_number":7,"context_line":"oslotest\u003e\u003d3.2.0 # Apache-2.0"},{"line_number":8,"context_line":"stestr\u003e\u003d1.1.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":7,"id":"a660e996_d643cb3b","line":6,"range":{"start_line":6,"start_character":0,"end_line":6,"end_character":4},"updated":"2021-01-15 17:44:13.000000000","message":"what about using a builtin module - unittest.mock?","commit_id":"23ae950a48f8676937bd0ec71d6c9d1e3d38cd7b"}]}