)]}'
{"tools/migrate_from_tempest.sh":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"37c95b21251e60ac53befec08a764de1fedae630","unresolved":false,"context_lines":[{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Use this script to move over a set of files from tempest master with commit"},{"line_number":4,"context_line":"# history into the tempest lib. You must only do this for files that haven\u0027t"},{"line_number":5,"context_line":"# been migrated over already."},{"line_number":6,"context_line":"# "},{"line_number":7,"context_line":"# To use:"},{"line_number":8,"context_line":"#  1. Create a new branch in the tempest-lib repo so not to destroy your current"}],"source_content_type":"text/x-sh","patch_set":2,"id":"fa98f980_fc34c4fc","line":5,"updated":"2014-09-18 05:31:01.000000000","message":"space.","commit_id":"8830a673b56f7393c2bf75e513884bc75deaa209"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"1725e0ded8522bc7c332457823a6588b1a7743f0","unresolved":false,"context_lines":[{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Use this script to move over a set of files from tempest master with commit"},{"line_number":4,"context_line":"# history into the tempest lib. You must only do this for files that haven\u0027t"},{"line_number":5,"context_line":"# been migrated over already."},{"line_number":6,"context_line":"# "},{"line_number":7,"context_line":"# To use:"},{"line_number":8,"context_line":"#  1. Create a new branch in the tempest-lib repo so not to destroy your current"}],"source_content_type":"text/x-sh","patch_set":2,"id":"fa98f980_11908395","line":5,"in_reply_to":"fa98f980_fc34c4fc","updated":"2014-09-18 14:03:01.000000000","message":"Done","commit_id":"8830a673b56f7393c2bf75e513884bc75deaa209"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"37c95b21251e60ac53befec08a764de1fedae630","unresolved":false,"context_lines":[{"line_number":105,"context_line":"file_list\u003d\u0027\u0027"},{"line_number":106,"context_line":"for file in $files; do"},{"line_number":107,"context_line":"    filename\u003d`basename $file`"},{"line_number":108,"context_line":"    if [ -n \"$output_dir\" ]; then"},{"line_number":109,"context_line":"        cp \"$tmpdir/$file\" \"$output_dir/$filename\""},{"line_number":110,"context_line":"    else"},{"line_number":111,"context_line":"        cp \"$tmpdir/$file\" \"tempest_lib/$filename\""}],"source_content_type":"text/x-sh","patch_set":2,"id":"fa98f980_bc8ebcce","line":108,"updated":"2014-09-18 05:31:01.000000000","message":"Whats about moving complete directory? We might need to move schema dir completely . OR Its ok to migrate those file by file as dir migration is one time process. And modified files between migration and applying framework in tempest can be migrated as per current tool.","commit_id":"8830a673b56f7393c2bf75e513884bc75deaa209"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"1725e0ded8522bc7c332457823a6588b1a7743f0","unresolved":false,"context_lines":[{"line_number":105,"context_line":"file_list\u003d\u0027\u0027"},{"line_number":106,"context_line":"for file in $files; do"},{"line_number":107,"context_line":"    filename\u003d`basename $file`"},{"line_number":108,"context_line":"    if [ -n \"$output_dir\" ]; then"},{"line_number":109,"context_line":"        cp \"$tmpdir/$file\" \"$output_dir/$filename\""},{"line_number":110,"context_line":"    else"},{"line_number":111,"context_line":"        cp \"$tmpdir/$file\" \"tempest_lib/$filename\""}],"source_content_type":"text/x-sh","patch_set":2,"id":"fa98f980_1fda4f17","line":108,"in_reply_to":"fa98f980_bc8ebcce","updated":"2014-09-18 14:03:01.000000000","message":"Well I can change the cp to be \u0027cp -r\u0027 which will copy whole directories. Things here will behave the same if I do that. I had originally assumed we\u0027d be only moving over single files. I\u0027ll correct that on the next rev.\n\nI\u0027m also not sure we want to move over the schemas, but I think that raises a larger discussion about tempest-lib scope. (ie should we migrate the individual service clients, or just the rest_client so that building service clients are easier)","commit_id":"8830a673b56f7393c2bf75e513884bc75deaa209"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a8a7a22a73c7fb93abfd8bd4b5a7c78b1019afea","unresolved":false,"context_lines":[{"line_number":118,"context_line":"done"},{"line_number":119,"context_line":"#Cleanup temporary tempest repo"},{"line_number":120,"context_line":"rm -rf $tmpdir"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"#Generate a migration commit"},{"line_number":123,"context_line":"git add ."},{"line_number":124,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"fa98f980_bc715c46","line":121,"updated":"2014-09-18 05:11:05.000000000","message":"1 suggestion- git show can be printed here to verify migration and after verification, it can be continue to commit.","commit_id":"8830a673b56f7393c2bf75e513884bc75deaa209"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"1725e0ded8522bc7c332457823a6588b1a7743f0","unresolved":false,"context_lines":[{"line_number":118,"context_line":"done"},{"line_number":119,"context_line":"#Cleanup temporary tempest repo"},{"line_number":120,"context_line":"rm -rf $tmpdir"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"#Generate a migration commit"},{"line_number":123,"context_line":"git add ."},{"line_number":124,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"fa98f980_bfa14348","line":121,"in_reply_to":"fa98f980_bc715c46","updated":"2014-09-18 14:03:01.000000000","message":"I\u0027m not sure what a git show would do here. Until the new files are actually committed git show will just show the previous commit. I also am trying to avoid making this script having a step which requires manual intervention, because I think having uniformity on migration commits is desirable. You just run the tool and it creates a migration commit for the files specified. \n\nIf verification that the migration commit was generated correctly is needed I feel it should be a manual step after running this tool, because there isn\u0027t a consistent way to automate that.\n\nHowever, thinking about this I did realize that I probably should not do a blanket git add below, so in the next rev I\u0027ll only add files from the migration.","commit_id":"8830a673b56f7393c2bf75e513884bc75deaa209"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"3ecbbdf47bd3616c2968829a2b0c5d27597e446f","unresolved":false,"context_lines":[{"line_number":9,"context_line":"#  2. Run the script from the repo dir and specify the file paths relative to"},{"line_number":10,"context_line":"#     the root tempest dir(only code and unit tests):"},{"line_number":11,"context_line":"#"},{"line_number":12,"context_line":"#   tools/migrate_from_tempest.sh tempest/"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"function usage {"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_03560a5c","line":12,"updated":"2014-10-07 05:49:38.000000000","message":"This doesn\u0027t work in my environment.. A lot of errors occurred like below:\n\n $ ./tools/migrate_from_tempest.sh tempest/\n [..]\n Rewrite f88913f26e99957b88bdcf03bade78730b75a4c3 (1/5163)/usr/lib/git-core/git-filter-branch: 2: [: f88913f26e99957b88bdcf03bade78730b75a4c3: unexpected operator\n Rewrite 0d9bbd14716bc7454f60925adc37f09518f9ccd4 (2/5163)/usr/lib/git-core/git-filter-branch: 2: [: 0d9bbd14716bc7454f60925adc37f09518f9ccd4: unexpected operator\n Rewrite f4457093e12e0fb466e36bd56f5119f61f8660b9 (3/5163)/usr/lib/git-core/git-filter-branch: 2: [: f4457093e12e0fb466e36bd56f5119f61f8660b9: unexpected operator\n [..]\n\nBut this patch is not the cause of this error because this error occurs without this patch.","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"576019f993d6ac39e322096f29ef6522023e40a6","unresolved":false,"context_lines":[{"line_number":9,"context_line":"#  2. Run the script from the repo dir and specify the file paths relative to"},{"line_number":10,"context_line":"#     the root tempest dir(only code and unit tests):"},{"line_number":11,"context_line":"#"},{"line_number":12,"context_line":"#   tools/migrate_from_tempest.sh tempest/"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"function usage {"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_87ad98eb","line":12,"in_reply_to":"baa201ad_03560a5c","updated":"2014-10-07 15:06:16.000000000","message":"Oh, sorry, I\u0027ll update this in the next rev it should be:\n\ntools/migrate_from_tempest.sh tempest/file.py tempest/sub_dir ...\n\nI\u0027m not sure what would happen if you specified migrating the entirety of tempest, although I don\u0027t know why it would fail.","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"3ecbbdf47bd3616c2968829a2b0c5d27597e446f","unresolved":false,"context_lines":[{"line_number":94,"context_line":"EOF"},{"line_number":95,"context_line":")"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"#Prune just the commits relevant to what is being migrated"},{"line_number":98,"context_line":"git filter-branch --index-filter \"$pruner\" --parent-filter \"$set_roots\" --commit-filter \"$skip_empty\" HEAD"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"SHA1_LIST\u003d`git log --oneline`"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_431e52bc","line":97,"updated":"2014-10-07 05:49:38.000000000","message":"It\u0027s better to add a white space for the consistency.","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"576019f993d6ac39e322096f29ef6522023e40a6","unresolved":false,"context_lines":[{"line_number":94,"context_line":"EOF"},{"line_number":95,"context_line":")"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"#Prune just the commits relevant to what is being migrated"},{"line_number":98,"context_line":"git filter-branch --index-filter \"$pruner\" --parent-filter \"$set_roots\" --commit-filter \"$skip_empty\" HEAD"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"SHA1_LIST\u003d`git log --oneline`"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_6cb33b98","line":97,"in_reply_to":"baa201ad_2bf04490","updated":"2014-10-07 15:06:16.000000000","message":"Done","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0caec6b92ecc8bbbce1dc2544c95197ad235fb10","unresolved":false,"context_lines":[{"line_number":94,"context_line":"EOF"},{"line_number":95,"context_line":")"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"#Prune just the commits relevant to what is being migrated"},{"line_number":98,"context_line":"git filter-branch --index-filter \"$pruner\" --parent-filter \"$set_roots\" --commit-filter \"$skip_empty\" HEAD"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"SHA1_LIST\u003d`git log --oneline`"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_2bf04490","line":97,"in_reply_to":"baa201ad_431e52bc","updated":"2014-10-07 08:33:44.000000000","message":"nit but agree.","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0caec6b92ecc8bbbce1dc2544c95197ad235fb10","unresolved":false,"context_lines":[{"line_number":109,"context_line":"        git add \"$output_dir/$filename\""},{"line_number":110,"context_line":"    else"},{"line_number":111,"context_line":"        cp -r \"$tmpdir/$file\" \"tempest_lib/$filename\""},{"line_number":112,"context_line":"        git add \"tempest_lib/$filename\""},{"line_number":113,"context_line":"    fi"},{"line_number":114,"context_line":"    if [[ -z \"$file_list\" ]]; then"},{"line_number":115,"context_line":"        file_list\u003d\"$filename\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_4be02896","line":112,"updated":"2014-10-07 08:33:44.000000000","message":"How about\n\n if [ -n \"$output_dir\" ]; then\n     dest_file\u003d\"$output_dir/$filename\"\n else\n     dest_file\u003d\"tempest_lib/$filename\"\n cp -r \"$tmpdir/$file\" \"$dest_file\"\n git add \"$dest_file\"\n\n?","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"576019f993d6ac39e322096f29ef6522023e40a6","unresolved":false,"context_lines":[{"line_number":109,"context_line":"        git add \"$output_dir/$filename\""},{"line_number":110,"context_line":"    else"},{"line_number":111,"context_line":"        cp -r \"$tmpdir/$file\" \"tempest_lib/$filename\""},{"line_number":112,"context_line":"        git add \"tempest_lib/$filename\""},{"line_number":113,"context_line":"    fi"},{"line_number":114,"context_line":"    if [[ -z \"$file_list\" ]]; then"},{"line_number":115,"context_line":"        file_list\u003d\"$filename\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_2cb6938a","line":112,"in_reply_to":"baa201ad_4be02896","updated":"2014-10-07 15:06:16.000000000","message":"Done","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0caec6b92ecc8bbbce1dc2544c95197ad235fb10","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        file_list\u003d\"$file_list, $filename\""},{"line_number":118,"context_line":"    fi"},{"line_number":119,"context_line":"done"},{"line_number":120,"context_line":"#Cleanup temporary tempest repo"},{"line_number":121,"context_line":"rm -rf $tmpdir"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"#Generate a migration commit"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_4bed88b6","line":120,"updated":"2014-10-07 08:33:44.000000000","message":"ditto","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"576019f993d6ac39e322096f29ef6522023e40a6","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        file_list\u003d\"$file_list, $filename\""},{"line_number":118,"context_line":"    fi"},{"line_number":119,"context_line":"done"},{"line_number":120,"context_line":"#Cleanup temporary tempest repo"},{"line_number":121,"context_line":"rm -rf $tmpdir"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"#Generate a migration commit"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_aca9a325","line":120,"in_reply_to":"baa201ad_4bed88b6","updated":"2014-10-07 15:06:16.000000000","message":"Done","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0caec6b92ecc8bbbce1dc2544c95197ad235fb10","unresolved":false,"context_lines":[{"line_number":120,"context_line":"#Cleanup temporary tempest repo"},{"line_number":121,"context_line":"rm -rf $tmpdir"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"#Generate a migration commit"},{"line_number":124,"context_line":"commit_message\u003d\"Migrated $file_list from tempest\""},{"line_number":125,"context_line":"pre_list\u003d$\"This migrates the above files from tempest. This includes tempest\\ncommits:\""},{"line_number":126,"context_line":"post_list\u003d$\"to see the commit history for these files refer to the above sha1s in\\nthe tempest repository\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_6b2decfd","line":123,"updated":"2014-10-07 08:33:44.000000000","message":"ditto","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"576019f993d6ac39e322096f29ef6522023e40a6","unresolved":false,"context_lines":[{"line_number":120,"context_line":"#Cleanup temporary tempest repo"},{"line_number":121,"context_line":"rm -rf $tmpdir"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"#Generate a migration commit"},{"line_number":124,"context_line":"commit_message\u003d\"Migrated $file_list from tempest\""},{"line_number":125,"context_line":"pre_list\u003d$\"This migrates the above files from tempest. This includes tempest\\ncommits:\""},{"line_number":126,"context_line":"post_list\u003d$\"to see the commit history for these files refer to the above sha1s in\\nthe tempest repository\""}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_cca4274d","line":123,"in_reply_to":"baa201ad_6b2decfd","updated":"2014-10-07 15:06:16.000000000","message":"Done","commit_id":"f19d2764f89c7d803b5d8c079a4ea615b42a2548"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"374f73cd64e5263977aa6fffacbf0b426f9268c5","unresolved":false,"context_lines":[{"line_number":105,"context_line":"for file in $files; do"},{"line_number":106,"context_line":"    filename\u003d`basename $file`"},{"line_number":107,"context_line":"    if [ -n \"$output_dir\" ]; then"},{"line_number":108,"context_line":"        dest_file\u003d\"$output_dir/$filename\" "},{"line_number":109,"context_line":"    else"},{"line_number":110,"context_line":"        dest_file\u003d\"tempest_lib/$filename\""},{"line_number":111,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":4,"id":"baa201ad_2d6e8dc9","line":108,"updated":"2014-10-07 15:52:49.000000000","message":"unnecessary space at the end of line..","commit_id":"41d1b5ecce88e56fb6f0501757891d4c0cd55365"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"30060cf3365a3086fef29519eebfcea157c04d3a","unresolved":false,"context_lines":[{"line_number":105,"context_line":"for file in $files; do"},{"line_number":106,"context_line":"    filename\u003d`basename $file`"},{"line_number":107,"context_line":"    if [ -n \"$output_dir\" ]; then"},{"line_number":108,"context_line":"        dest_file\u003d\"$output_dir/$filename\" "},{"line_number":109,"context_line":"    else"},{"line_number":110,"context_line":"        dest_file\u003d\"tempest_lib/$filename\""},{"line_number":111,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":4,"id":"baa201ad_ad027db1","line":108,"in_reply_to":"baa201ad_2d6e8dc9","updated":"2014-10-07 16:00:09.000000000","message":"Sigh,,,\n\nDone","commit_id":"41d1b5ecce88e56fb6f0501757891d4c0cd55365"}]}