)]}'
{"releasenotes/notes/subunit_describe_calls-ad7df689b9d63e3f.yaml":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"237b30bc8d7c663e9d2d56abbfcaa73a1cd24879","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"3fa7e38b_15066987","line":5,"updated":"2019-11-22 08:49:10.000000000","message":"Did you want to add `deprecations` section like this?\nhttps://opendev.org/openstack/tempest/src/branch/master/releasenotes/notes/deprecate-vnc-server-header-529f07d592aefb62.yaml#L2","commit_id":"94af2ada2a3499fc31ed776363d5bab31a5c5b07"}],"setup.cfg":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8d68cc69c8cb8ca74db7030c2837e65ab5b725d9","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    workspace_remove \u003d tempest.cmd.workspace:TempestWorkspaceRemove"},{"line_number":47,"context_line":"    workspace_list \u003d tempest.cmd.workspace:TempestWorkspaceList"},{"line_number":48,"context_line":"    run \u003d tempest.cmd.run:TempestRun"},{"line_number":49,"context_line":"    subunit-describe-calls \u003d tempest.cmd.subunit_describe_calls:SubunitDescribeCalls"},{"line_number":50,"context_line":"oslo.config.opts \u003d"},{"line_number":51,"context_line":"    tempest.config \u003d tempest.config:list_opts"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":4,"id":"3fa7e38b_bd9b1364","line":49,"updated":"2019-09-24 02:24:14.000000000","message":"This makes the command interface as `tempest` subcommand.\nOriginal: `subunit-describe-calls`\nWith this patch: `tempest subunit-describe-calls`\n\nI think we don\u0027t need this in `tempest` command.","commit_id":"73a90515253eec5b2efbb0f7de7ef90226599f66"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"0f0ad00b147c4bae4cbc36610f6ff35239233268","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    subunit-describe-calls \u003d tempest.cmd.subunit_describe_calls:entry_point"},{"line_number":37,"context_line":"tempest.cm \u003d"},{"line_number":38,"context_line":"    account-generator \u003d tempest.cmd.account_generator:TempestAccountGenerator"},{"line_number":39,"context_line":"    subunit-describe-calls \u003d tempest.cmd.subunit_describe_calls:TempestSubunitDescribeCalls"},{"line_number":40,"context_line":"    init \u003d tempest.cmd.init:TempestInit"},{"line_number":41,"context_line":"    cleanup \u003d tempest.cmd.cleanup:TempestCleanup"},{"line_number":42,"context_line":"    list-plugins \u003d tempest.cmd.list_plugins:TempestListPlugins"}],"source_content_type":"text/x-ttcn-cfg","patch_set":7,"id":"3fa7e38b_fe8dd3d3","line":39,"updated":"2019-10-30 01:32:56.000000000","message":"I think a reno is needed if we add a new subcommand to tempest. And the commit message should mention about it too. And the original subunit-describe-calls should be deprecated to avoid confusion.\n\nI also think this change can be split to another patch. I think it\u0027s a different topic from the commit message.","commit_id":"122cd9be21c7a77f7e12ae605c8d811112e2dc63"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"29e50a9e5b72ba0ffbba4dbe396a26d5b56aeacb","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    subunit-describe-calls \u003d tempest.cmd.subunit_describe_calls:entry_point"},{"line_number":37,"context_line":"tempest.cm \u003d"},{"line_number":38,"context_line":"    account-generator \u003d tempest.cmd.account_generator:TempestAccountGenerator"},{"line_number":39,"context_line":"    subunit-describe-calls \u003d tempest.cmd.subunit_describe_calls:TempestSubunitDescribeCalls"},{"line_number":40,"context_line":"    init \u003d tempest.cmd.init:TempestInit"},{"line_number":41,"context_line":"    cleanup \u003d tempest.cmd.cleanup:TempestCleanup"},{"line_number":42,"context_line":"    list-plugins \u003d tempest.cmd.list_plugins:TempestListPlugins"}],"source_content_type":"text/x-ttcn-cfg","patch_set":7,"id":"3fa7e38b_b6b8a9c7","line":39,"in_reply_to":"3fa7e38b_fe8dd3d3","updated":"2019-11-04 07:37:36.000000000","message":"Masayuki Igawa, I will update it in another patch.","commit_id":"122cd9be21c7a77f7e12ae605c8d811112e2dc63"}],"tempest/cmd/subunit_describe_calls.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8d68cc69c8cb8ca74db7030c2837e65ab5b725d9","unresolved":false,"context_lines":[{"line_number":243,"context_line":"class ArgumentParser(Command):"},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"    _description \u003d \"Outputs all HTTP calls a given test made that were logged.\""},{"line_number":246,"context_line":"    "},{"line_number":247,"context_line":"    def __init__(self):"},{"line_number":248,"context_line":"        super(ArgumentParser, self).__init__(self, \"subunit-describe-calls\")"},{"line_number":249,"context_line":"    "}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_77943645","line":246,"updated":"2019-09-24 02:24:14.000000000","message":"Please remove these white spaces.","commit_id":"9507777d07ad53a4608582e778cb42ce9d051b68"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"626010d2a6221565424420c92e36408627b4f2d9","unresolved":false,"context_lines":[{"line_number":243,"context_line":"class ArgumentParser(Command):"},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"    _description \u003d \"Outputs all HTTP calls a given test made that were logged.\""},{"line_number":246,"context_line":"    "},{"line_number":247,"context_line":"    def __init__(self):"},{"line_number":248,"context_line":"        super(ArgumentParser, self).__init__(self, \"subunit-describe-calls\")"},{"line_number":249,"context_line":"    "}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_23570429","line":246,"in_reply_to":"3fa7e38b_6323bc57","updated":"2019-09-24 07:04:09.000000000","message":"oops, sorry, igrnore them. This is remaining as a draft comment on my gerrit and then, published with another review.","commit_id":"9507777d07ad53a4608582e778cb42ce9d051b68"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"1ed7157cea6812434bcb6bb5608532cda40ea239","unresolved":false,"context_lines":[{"line_number":243,"context_line":"class ArgumentParser(Command):"},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"    _description \u003d \"Outputs all HTTP calls a given test made that were logged.\""},{"line_number":246,"context_line":"    "},{"line_number":247,"context_line":"    def __init__(self):"},{"line_number":248,"context_line":"        super(ArgumentParser, self).__init__(self, \"subunit-describe-calls\")"},{"line_number":249,"context_line":"    "}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_6323bc57","line":246,"in_reply_to":"3fa7e38b_77943645","updated":"2019-09-24 06:22:50.000000000","message":"This is older patch. Please have a look at the new one.","commit_id":"9507777d07ad53a4608582e778cb42ce9d051b68"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8d68cc69c8cb8ca74db7030c2837e65ab5b725d9","unresolved":false,"context_lines":[{"line_number":246,"context_line":"    "},{"line_number":247,"context_line":"    def __init__(self):"},{"line_number":248,"context_line":"        super(ArgumentParser, self).__init__(self, \"subunit-describe-calls\")"},{"line_number":249,"context_line":"    "},{"line_number":250,"context_line":"    def get_parser(self):"},{"line_number":251,"context_line":"        self.prog \u003d \"subunit-describe-calls\""},{"line_number":252,"context_line":"        parser \u003d super(ArgumentParser, self).get_parser(self.prog)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_9791b255","line":249,"updated":"2019-09-24 02:24:14.000000000","message":"ditto","commit_id":"9507777d07ad53a4608582e778cb42ce9d051b68"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"fe2509dda8d1a12a331be5ac5419ab2a94ce7fbd","unresolved":false,"context_lines":[{"line_number":301,"context_line":"def _parser_add_args(parser):"},{"line_number":302,"context_line":"    parser.add_argument("},{"line_number":303,"context_line":"        \"-s\", \"--subunit\", metavar\u003d\"\u003csubunit file\u003e\","},{"line_number":304,"context_line":"        nargs\u003d\"?\", type\u003dcliff._argparse.FileType(\u0027rb\u0027), default\u003dsys.stdin,"},{"line_number":305,"context_line":"        help\u003d\"The path to the subunit output file.\""},{"line_number":306,"context_line":"    )"},{"line_number":307,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3fa7e38b_c39b9063","line":304,"updated":"2019-09-24 06:21:56.000000000","message":"This generates a bit weird output like following.. I have no idea to fix this right now, though.\n\npython3.7:\n  -s [\u003csubunit file\u003e], --subunit [\u003csubunit file\u003e]\n                        The path to the subunit output file. (default:\n                        \u003c_io.TextIOWrapper name\u003d\u0027\u003cstdin\u003e\u0027 mode\u003d\u0027r\u0027\n                        encoding\u003d\u0027UTF-8\u0027\u003e)\n\npython2.7:\n  -s [\u003csubunit file\u003e], --subunit [\u003csubunit file\u003e]\n                        The path to the subunit output file. (default: \u003copen\n                        file \u0027\u003cstdin\u003e\u0027, mode \u0027r\u0027 at 0x7f2a6f1520c0\u003e)","commit_id":"ba4b0eb3fbe89bcd777b254f0529e69ac5b2bac5"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8da45fb0b716402246d085a60db0475d8bfb95ca","unresolved":false,"context_lines":[{"line_number":301,"context_line":"def _parser_add_args(parser):"},{"line_number":302,"context_line":"    parser.add_argument("},{"line_number":303,"context_line":"        \"-s\", \"--subunit\", metavar\u003d\"\u003csubunit file\u003e\","},{"line_number":304,"context_line":"        nargs\u003d\"?\", type\u003dargparse.FileType(\u0027rb\u0027, 0), default\u003d\"sys.stdin\","},{"line_number":305,"context_line":"        help\u003d\"The path to the subunit output file.\""},{"line_number":306,"context_line":"    )"},{"line_number":307,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_5ce448e1","line":304,"updated":"2019-09-25 07:30:16.000000000","message":"It looks this doesn\u0027t work?\n\n$ cat  ./.stestr/45 | subunit-describe-calls\nusage: subunit-describe-calls [-h] [-s [\u003csubunit file\u003e]]\n                              [-n \u003cnon subunit name\u003e] [-o \u003coutput file\u003e]\n                              [-p \u003cports file\u003e] [-v]\nsubunit-describe-calls: error: argument -s/--subunit: can\u0027t open \u0027sys.stdin\u0027: [Errno 2] No such file or directory: \u0027sys.stdin\u0027\n\nBut this works\n\n  $ cat  ./.stestr/45 | subunit-describe-calls -s -","commit_id":"dd5d59e03cea8ca36a3b8dfadf6cabb5d81a4a24"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"65719004f9cfbe35aaa02899c83befb542d4a73e","unresolved":false,"context_lines":[{"line_number":302,"context_line":"                    item.get(\u0027response_headers\u0027)))"},{"line_number":303,"context_line":"                sys.stdout.write(\u0027\\t\\t- response body: {0}\\n\u0027.format("},{"line_number":304,"context_line":"                    item.get(\u0027response_body\u0027)))"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"        sys.stdout.write(\u0027\\n\u0027)"},{"line_number":307,"context_line":""},{"line_number":308,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"3fa7e38b_82880f71","line":305,"updated":"2019-10-29 13:54:09.000000000","message":"unnecessary new line","commit_id":"122cd9be21c7a77f7e12ae605c8d811112e2dc63"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"65719004f9cfbe35aaa02899c83befb542d4a73e","unresolved":false,"context_lines":[{"line_number":307,"context_line":""},{"line_number":308,"context_line":""},{"line_number":309,"context_line":"def entry_point(cl_args\u003dNone):"},{"line_number":310,"context_line":""},{"line_number":311,"context_line":"    if not cl_args:"},{"line_number":312,"context_line":"        cl_args \u003d ArgumentParser().parse_args()"},{"line_number":313,"context_line":"    parser \u003d parse(cl_args.subunit, cl_args.non_subunit_name, cl_args.ports)"}],"source_content_type":"text/x-python","patch_set":7,"id":"3fa7e38b_22791b81","line":310,"updated":"2019-10-29 13:54:09.000000000","message":"unnecessary new line","commit_id":"122cd9be21c7a77f7e12ae605c8d811112e2dc63"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"29e50a9e5b72ba0ffbba4dbe396a26d5b56aeacb","unresolved":false,"context_lines":[{"line_number":307,"context_line":""},{"line_number":308,"context_line":""},{"line_number":309,"context_line":"def entry_point(cl_args\u003dNone):"},{"line_number":310,"context_line":""},{"line_number":311,"context_line":"    if not cl_args:"},{"line_number":312,"context_line":"        cl_args \u003d ArgumentParser().parse_args()"},{"line_number":313,"context_line":"    parser \u003d parse(cl_args.subunit, cl_args.non_subunit_name, cl_args.ports)"}],"source_content_type":"text/x-python","patch_set":7,"id":"3fa7e38b_561db50c","line":310,"in_reply_to":"3fa7e38b_22791b81","updated":"2019-11-04 07:37:36.000000000","message":"Thanks, I would address these in upcoming patch","commit_id":"122cd9be21c7a77f7e12ae605c8d811112e2dc63"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"0f0ad00b147c4bae4cbc36610f6ff35239233268","unresolved":false,"context_lines":[{"line_number":318,"context_line":"    parser.add_argument("},{"line_number":319,"context_line":"        \"-s\", \"--subunit\", metavar\u003d\"\u003csubunit file\u003e\","},{"line_number":320,"context_line":"        nargs\u003d\"?\", type\u003dargparse.FileType(\u0027rb\u0027), default\u003dsys.stdin,"},{"line_number":321,"context_line":"        help\u003d\"The path to the subunit output file.\""},{"line_number":322,"context_line":"    )"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"    parser.add_argument("}],"source_content_type":"text/x-python","patch_set":7,"id":"3fa7e38b_1e8b0fc0","line":321,"updated":"2019-10-30 01:32:56.000000000","message":"I think a description for the default value(stdin) would be helpful for users in this help. But we can add it in a following patch.","commit_id":"122cd9be21c7a77f7e12ae605c8d811112e2dc63"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"29e50a9e5b72ba0ffbba4dbe396a26d5b56aeacb","unresolved":false,"context_lines":[{"line_number":318,"context_line":"    parser.add_argument("},{"line_number":319,"context_line":"        \"-s\", \"--subunit\", metavar\u003d\"\u003csubunit file\u003e\","},{"line_number":320,"context_line":"        nargs\u003d\"?\", type\u003dargparse.FileType(\u0027rb\u0027), default\u003dsys.stdin,"},{"line_number":321,"context_line":"        help\u003d\"The path to the subunit output file.\""},{"line_number":322,"context_line":"    )"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"    parser.add_argument("}],"source_content_type":"text/x-python","patch_set":7,"id":"3fa7e38b_b6d3c937","line":321,"in_reply_to":"3fa7e38b_1e8b0fc0","updated":"2019-11-04 07:37:36.000000000","message":"Sure","commit_id":"122cd9be21c7a77f7e12ae605c8d811112e2dc63"}]}