)]}'
{"specs/tempest-cli-improvements.rst":[{"author":{"_account_id":9234,"name":"Steve Heyman","email":"steve@heyman.com","username":"hockeynut"},"change_message_id":"e1f9e5e6f555358962dd66421baef06f154ba4d6","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"There are currently some Tempest CLI endpoints created when tempest is"},{"line_number":21,"context_line":"installed but there is no consistency in the console command names or function."},{"line_number":22,"context_line":"Also there is no way to simple way execute a tempest run via a console"},{"line_number":23,"context_line":"command currently."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Proposed Change"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_850e198a","line":22,"updated":"2015-05-21 21:20:25.000000000","message":"nit - \"Also there is no simple way to execute...\".  Plus this gets my name on the interested parties list :-)","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ac673cf5b4bd60bdb6163989f009bc9cd57866dd","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    tempest config"},{"line_number":83,"context_line":"        arguments:"},{"line_number":84,"context_line":"          --create [optional_tempest.conf_path]"},{"line_number":85,"context_line":"            --validate"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    tempest javelin"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_5c10f858","line":84,"updated":"2015-03-07 04:41:16.000000000","message":"So this doesn\u0027t exist yet, there is been a spec which has been basically dead to handle this. I\u0027d rather not see this cli work conflated with that.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"14b926abc0724618a455672b10c0be57e93b3f9e","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    tempest config"},{"line_number":83,"context_line":"        arguments:"},{"line_number":84,"context_line":"          --create [optional_tempest.conf_path]"},{"line_number":85,"context_line":"            --validate"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    tempest javelin"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_74caa36f","line":84,"in_reply_to":"9a80dd14_3cd4c546","updated":"2015-03-11 15:03:03.000000000","message":"\"tempest config\" would be a top level command then subcommand flags would have been tempest config --create and tempest config --validate.\n\ntempest config --create would call the downstream config script dkranz has written.\ntempest config --validate - would call the existing validation script.\n\nI will update spec with removal of --config.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"7b339dae3f554a6c478da7225e1c04ebfad46b03","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    tempest config"},{"line_number":83,"context_line":"        arguments:"},{"line_number":84,"context_line":"          --create [optional_tempest.conf_path]"},{"line_number":85,"context_line":"            --validate"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    tempest javelin"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_3cd4c546","line":84,"in_reply_to":"9a80dd14_5c10f858","updated":"2015-03-10 16:02:37.000000000","message":"I do not agree that it is dead but do agree that it should not be part of the first cut at this. I suggest removing the config suboption and changing it to \u0027tempest validate\u0027 which does the same as verify-tempest-config today.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ac673cf5b4bd60bdb6163989f009bc9cd57866dd","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    tempest config"},{"line_number":83,"context_line":"        arguments:"},{"line_number":84,"context_line":"          --create [optional_tempest.conf_path]"},{"line_number":85,"context_line":"            --validate"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    tempest javelin"},{"line_number":88,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_dc2ce8ae","line":85,"updated":"2015-03-07 04:41:16.000000000","message":"I assume this is what is basically verify-tempest-config today.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"14b926abc0724618a455672b10c0be57e93b3f9e","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    tempest config"},{"line_number":83,"context_line":"        arguments:"},{"line_number":84,"context_line":"          --create [optional_tempest.conf_path]"},{"line_number":85,"context_line":"            --validate"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"    tempest javelin"},{"line_number":88,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_170fa9e2","line":85,"in_reply_to":"9a80dd14_dc2ce8ae","updated":"2015-03-11 15:03:03.000000000","message":"mtreinish: yes","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"0a22e00090b05f098098dd4028b425c5e852d298","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    tempest javelin"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    tempest run"},{"line_number":90,"context_line":"        arguments:"},{"line_number":91,"context_line":"          --smoke"},{"line_number":92,"context_line":"         --gate"},{"line_number":93,"context_line":"         --regex \"{some_regex}\""}],"source_content_type":"text/x-rst","patch_set":4,"id":"ba7be1f8_2e1c6c13","line":90,"updated":"2015-02-27 14:30:43.000000000","message":"I would be nice to have a --dry-run (or list) argument here as well, to see what tests would be executed based on the other arguments.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"b2e99eb033aff650ef6d90be46b6bf0415539f1d","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    tempest javelin"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    tempest run"},{"line_number":90,"context_line":"        arguments:"},{"line_number":91,"context_line":"          --smoke"},{"line_number":92,"context_line":"         --gate"},{"line_number":93,"context_line":"         --regex \"{some_regex}\""}],"source_content_type":"text/x-rst","patch_set":4,"id":"7a016987_f13c88c9","line":90,"in_reply_to":"9a80dd14_b4c09b2b","updated":"2015-05-20 23:16:26.000000000","message":"Right, but what I\u0027d like to see is the tests that would be listed with the given regex","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"14b926abc0724618a455672b10c0be57e93b3f9e","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    tempest javelin"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    tempest run"},{"line_number":90,"context_line":"        arguments:"},{"line_number":91,"context_line":"          --smoke"},{"line_number":92,"context_line":"         --gate"},{"line_number":93,"context_line":"         --regex \"{some_regex}\""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_b4c09b2b","line":90,"in_reply_to":"9a80dd14_df5f7a0d","updated":"2015-03-11 15:03:03.000000000","message":"Good idea, I will update spec with this feature, on the fence about the flag name, both work.  I\u0027ll go with --list for now.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ac673cf5b4bd60bdb6163989f009bc9cd57866dd","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    tempest javelin"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    tempest run"},{"line_number":90,"context_line":"        arguments:"},{"line_number":91,"context_line":"          --smoke"},{"line_number":92,"context_line":"         --gate"},{"line_number":93,"context_line":"         --regex \"{some_regex}\""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_df5f7a0d","line":90,"in_reply_to":"ba7be1f8_2e1c6c13","updated":"2015-03-07 04:41:16.000000000","message":"I think --list/-l is better for that.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ac673cf5b4bd60bdb6163989f009bc9cd57866dd","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        arguments:"},{"line_number":91,"context_line":"          --smoke"},{"line_number":92,"context_line":"         --gate"},{"line_number":93,"context_line":"         --regex \"{some_regex}\""},{"line_number":94,"context_line":"         ..."},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_3c1fc46d","line":93,"updated":"2015-03-07 04:41:16.000000000","message":"The one thing we must support here is have an exclude list from file arg here. This is one of the most asked for features and I feel is key to the run subcommand. The big reason for adding a run subcommand is so we can workaround the testr UI.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"7b339dae3f554a6c478da7225e1c04ebfad46b03","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        arguments:"},{"line_number":91,"context_line":"          --smoke"},{"line_number":92,"context_line":"         --gate"},{"line_number":93,"context_line":"         --regex \"{some_regex}\""},{"line_number":94,"context_line":"         ..."},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_5c98996a","line":93,"in_reply_to":"9a80dd14_3c1fc46d","updated":"2015-03-10 16:02:37.000000000","message":"+1 fo a file argument. You also need to be able to pass the other -- arguments to testr. I actually think it would be better to have these arguments look like a call to testr plus the file argument. Having some argument shorthand like \"gate\" or \"smoke\" to represent a more complex regexp may be useful but if we do that I think it would be better to have a --type argument or something that itself took a name for a defined and useful class of tests.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"14b926abc0724618a455672b10c0be57e93b3f9e","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        arguments:"},{"line_number":91,"context_line":"          --smoke"},{"line_number":92,"context_line":"         --gate"},{"line_number":93,"context_line":"         --regex \"{some_regex}\""},{"line_number":94,"context_line":"         ..."},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_f7edb526","line":93,"in_reply_to":"9a80dd14_5c98996a","updated":"2015-03-11 15:03:03.000000000","message":"Will add tempest run --exclude-file\n\ndkranz: --regex flag was kind of what you are looking for testr args.  How about adding --tester-args (or -testr-args) argument?  Which basically just passes whatever you provide as input to testr?\n\nThe combination of --tester-args and --exclude-file would provide the functionality you are looking for.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"0a22e00090b05f098098dd4028b425c5e852d298","unresolved":false,"context_lines":[{"line_number":91,"context_line":"          --smoke"},{"line_number":92,"context_line":"         --gate"},{"line_number":93,"context_line":"         --regex \"{some_regex}\""},{"line_number":94,"context_line":"         ..."},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"Alternatives"},{"line_number":97,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"ba7be1f8_8ea0f8a6","line":94,"updated":"2015-02-27 14:30:43.000000000","message":"We may need an option about whether to use venv/tox or not.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ac673cf5b4bd60bdb6163989f009bc9cd57866dd","unresolved":false,"context_lines":[{"line_number":91,"context_line":"          --smoke"},{"line_number":92,"context_line":"         --gate"},{"line_number":93,"context_line":"         --regex \"{some_regex}\""},{"line_number":94,"context_line":"         ..."},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"Alternatives"},{"line_number":97,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"ba7be1f8_fb7002de","line":94,"in_reply_to":"ba7be1f8_8ea0f8a6","updated":"2015-03-07 04:41:16.000000000","message":"So I actually think this should just be a testr wrapper. We will let tox or something else handle the venv. So once this is implemented the tox.ini commands will be switched to use tempest run --regex instead of pretty_tox.sh.\n\nThis is a detail which would have been good to include in the spec, but we can add it later.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ac673cf5b4bd60bdb6163989f009bc9cd57866dd","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    tempest-cleanup"},{"line_number":104,"context_line":"    tempest-config"},{"line_number":105,"context_line":"    tempest-javelin"},{"line_number":106,"context_line":"    tempest-run"},{"line_number":107,"context_line":"    ..."},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_fc00cc03","line":106,"updated":"2015-03-07 04:41:16.000000000","message":"Personally I prefer this, but that\u0027s probably just me. I\u0027m fine with the subcommands especially since everyone else seems to favor it.","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"14b926abc0724618a455672b10c0be57e93b3f9e","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    tempest-cleanup"},{"line_number":104,"context_line":"    tempest-config"},{"line_number":105,"context_line":"    tempest-javelin"},{"line_number":106,"context_line":"    tempest-run"},{"line_number":107,"context_line":"    ..."},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"Implementation"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_9492b7f4","line":106,"in_reply_to":"9a80dd14_fc00cc03","updated":"2015-03-11 15:03:03.000000000","message":"Seems more in-line with other OS CLIs to have the suggested format: \n[client] [do-something], example: \"nova list\" therefore: \"tempest run\"","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ac673cf5b4bd60bdb6163989f009bc9cd57866dd","unresolved":false,"context_lines":[{"line_number":119,"context_line":"----------"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"Target Milestone for completion:"},{"line_number":122,"context_line":"  Kilo-2"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"Work Items"},{"line_number":125,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_9c0d1041","line":122,"updated":"2015-03-07 04:41:16.000000000","message":"Well kilo-3 is \u003c 2 weeks away, but no big deal. Sorry this has sat for so long...","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"14b926abc0724618a455672b10c0be57e93b3f9e","unresolved":false,"context_lines":[{"line_number":119,"context_line":"----------"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"Target Milestone for completion:"},{"line_number":122,"context_line":"  Kilo-2"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"Work Items"},{"line_number":125,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9a80dd14_54af5f2b","line":122,"in_reply_to":"9a80dd14_9c0d1041","updated":"2015-03-11 15:03:03.000000000","message":"Done","commit_id":"36566687173d8cd02113113dc71fd04451b29b3b"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"6dd4dd7fc77f9a9c57a1172717fca0e3df4be5b6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    https://etherpad.openstack.org/p/tempest-cli"},{"line_number":8,"context_line":"    https://etherpad.openstack.org/p/YVR-QA-Tempest-CLI"},{"line_number":9,"context_line":"    https://etherpad.openstack.org/p/YVR-QA-Liberty-Priorities"},{"line_number":10,"context_line":"    http://git.openstack.org/cgit/openstack/tripleo-image-elements/tree/elements/tempest/tests2skip.py"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":13,"context_line":"Tempest CLI Improvements"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_14455d16","line":10,"updated":"2015-05-26 19:23:53.000000000","message":"I think we should add an external references section at the bottom of the spec and list these there. Take a look at the rendered sphinx output from the docs job:\n\nhttp://docs-draft.openstack.org/85/138785/6/check/gate-qa-specs-docs/2f0857e//doc/build/html/specs/tempest-cli-improvements.html\n\nto see what I mean about making it a list. http://sphinx-doc.org/rest.html#lists-and-quote-like-blocks might be useful\n\nAlso don\u0027t think we need the last link since it\u0027s only slightly related.","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"20d3735a391f80b9a05dab619ce8203d51c1bf06","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    https://etherpad.openstack.org/p/tempest-cli"},{"line_number":8,"context_line":"    https://etherpad.openstack.org/p/YVR-QA-Tempest-CLI"},{"line_number":9,"context_line":"    https://etherpad.openstack.org/p/YVR-QA-Liberty-Priorities"},{"line_number":10,"context_line":"    http://git.openstack.org/cgit/openstack/tripleo-image-elements/tree/elements/tempest/tests2skip.py"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":13,"context_line":"Tempest CLI Improvements"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_e1096196","line":10,"in_reply_to":"7a016987_14455d16","updated":"2015-05-29 14:29:34.000000000","message":"Yeah that looks bad when rendered. Wasn\u0027t aware of docs-draft.openstack.org.  Very useful, thanks.","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"6dd4dd7fc77f9a9c57a1172717fca0e3df4be5b6","unresolved":false,"context_lines":[{"line_number":54,"context_line":"   from cliff.commandmanager import CommandManager"},{"line_number":55,"context_line":"   from cliff.command import Command"},{"line_number":56,"context_line":"   # ..."},{"line_number":57,"context_line":"   "},{"line_number":58,"context_line":"   class Main(App):"},{"line_number":59,"context_line":"   "},{"line_number":60,"context_line":"       log \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_b40209b3","line":57,"updated":"2015-05-26 19:23:53.000000000","message":"A bunch of white space issues here and below","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"20d3735a391f80b9a05dab619ce8203d51c1bf06","unresolved":false,"context_lines":[{"line_number":54,"context_line":"   from cliff.commandmanager import CommandManager"},{"line_number":55,"context_line":"   from cliff.command import Command"},{"line_number":56,"context_line":"   # ..."},{"line_number":57,"context_line":"   "},{"line_number":58,"context_line":"   class Main(App):"},{"line_number":59,"context_line":"   "},{"line_number":60,"context_line":"       log \u003d logging.getLogger(__name__)"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_61f7b1d3","line":57,"in_reply_to":"7a016987_b40209b3","updated":"2015-05-29 14:29:34.000000000","message":"Done","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"6dd4dd7fc77f9a9c57a1172717fca0e3df4be5b6","unresolved":false,"context_lines":[{"line_number":176,"context_line":""},{"line_number":177,"context_line":"- Add support for Cliff"},{"line_number":178,"context_line":"- Define endpoints and commands in setup.cfg."},{"line_number":179,"context_line":"- Create stubbed tempest.cmd.main module providing main cliff-based CLI facade."},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"Dependencies"},{"line_number":182,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_14131d0f","line":179,"updated":"2015-05-26 19:23:53.000000000","message":"You don\u0027t want to mention the work on migrating the additional commands (or adding a run one to this list)","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"20d3735a391f80b9a05dab619ce8203d51c1bf06","unresolved":false,"context_lines":[{"line_number":176,"context_line":""},{"line_number":177,"context_line":"- Add support for Cliff"},{"line_number":178,"context_line":"- Define endpoints and commands in setup.cfg."},{"line_number":179,"context_line":"- Create stubbed tempest.cmd.main module providing main cliff-based CLI facade."},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"Dependencies"},{"line_number":182,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_81faadbb","line":179,"in_reply_to":"7a016987_14131d0f","updated":"2015-05-29 14:29:34.000000000","message":"I thought the command migration was going to be completed in additional patches?  At the summit we talked about making this first patch limited to just getting cliff integrated then migrating/refactoring the existing commands in future patches.\n\nPerhaps add another section, \"Followup Work items\"?","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"3ec45809ae92c8f6e1b1c9c36da908c61b9b1b8f","unresolved":false,"context_lines":[{"line_number":176,"context_line":""},{"line_number":177,"context_line":"- Add support for Cliff"},{"line_number":178,"context_line":"- Define endpoints and commands in setup.cfg."},{"line_number":179,"context_line":"- Create stubbed tempest.cmd.main module providing main cliff-based CLI facade."},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"Dependencies"},{"line_number":182,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_0405ab78","line":179,"in_reply_to":"7a016987_81faadbb","updated":"2015-05-29 14:45:55.000000000","message":"The spec/bp doesn\u0027t have a 1:1 mapping to a single patch. It\u0027s meant to track a longer, larger effort which almost always involves several patches to complete. Sure, the first patch will just be just the initial cliff integration but I don\u0027t really think that\u0027s enough to do say the cli improvements are done.\n\nYou should put all the items here that you think need to be finished before you can say this effort is done to completion. If you think that includes the existing command migration (even if it\u0027s in a patches later in a series) that should be on the list. (which I think is the right thing to do)","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"6dd4dd7fc77f9a9c57a1172717fca0e3df4be5b6","unresolved":false,"context_lines":[{"line_number":181,"context_line":"Dependencies"},{"line_number":182,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"cliff"},{"line_number":185,"context_line":"pbr"},{"line_number":186,"context_line":"setuptools"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_f4c3e188","line":184,"updated":"2015-05-26 19:23:53.000000000","message":"Might just want to mention why cliff is a dep here. (although I guess it\u0027s kinda obvious from above)","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"20d3735a391f80b9a05dab619ce8203d51c1bf06","unresolved":false,"context_lines":[{"line_number":181,"context_line":"Dependencies"},{"line_number":182,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"cliff"},{"line_number":185,"context_line":"pbr"},{"line_number":186,"context_line":"setuptools"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_410035ab","line":184,"in_reply_to":"7a016987_f4c3e188","updated":"2015-05-29 14:29:34.000000000","message":"Done","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"6dd4dd7fc77f9a9c57a1172717fca0e3df4be5b6","unresolved":false,"context_lines":[{"line_number":183,"context_line":""},{"line_number":184,"context_line":"cliff"},{"line_number":185,"context_line":"pbr"},{"line_number":186,"context_line":"setuptools"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_94e38d33","line":186,"updated":"2015-05-26 19:23:53.000000000","message":"Don\u0027t actually need to list these bottom 2 since they\u0027re already required for tempest.","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"20d3735a391f80b9a05dab619ce8203d51c1bf06","unresolved":false,"context_lines":[{"line_number":183,"context_line":""},{"line_number":184,"context_line":"cliff"},{"line_number":185,"context_line":"pbr"},{"line_number":186,"context_line":"setuptools"}],"source_content_type":"text/x-rst","patch_set":6,"id":"7a016987_611e910d","line":186,"in_reply_to":"7a016987_94e38d33","updated":"2015-05-29 14:29:34.000000000","message":"Done","commit_id":"150ab4e3351035f6519e7e88a473db96a3bd9bf7"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"5724a0ef2670fbdda4845f793b732c3d02e2556f","unresolved":false,"context_lines":[{"line_number":107,"context_line":"            --init-saved-state"},{"line_number":108,"context_line":"            --delete-tempest-conf-objects"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    tempest config-create"},{"line_number":111,"context_line":"        arguments:"},{"line_number":112,"context_line":"            --release"},{"line_number":113,"context_line":"            --create"}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a016987_f1a488a6","line":110,"updated":"2015-05-29 18:47:07.000000000","message":"Looks like these came from the proposed config script. I think discussion about the right arguments for config-create should happen around the config script and this spec should say TBD for this section. It is not clear if there will be config code in the repo to implement this part when the first delivery of the cli feature lands.","commit_id":"c60d33072932f32da09135f13b6e20777e49f076"},{"author":{"_account_id":10644,"name":"David Paterson","email":"davpat2112@yahoo.com","username":"dpaterson"},"change_message_id":"21db9cd2f3e28f927247e8b32d416faca9b987c9","unresolved":false,"context_lines":[{"line_number":107,"context_line":"            --init-saved-state"},{"line_number":108,"context_line":"            --delete-tempest-conf-objects"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    tempest config-create"},{"line_number":111,"context_line":"        arguments:"},{"line_number":112,"context_line":"            --release"},{"line_number":113,"context_line":"            --create"}],"source_content_type":"text/x-rst","patch_set":9,"id":"7a016987_b804a114","line":110,"in_reply_to":"7a016987_f1a488a6","updated":"2015-05-31 00:55:19.000000000","message":"Done","commit_id":"c60d33072932f32da09135f13b6e20777e49f076"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"63cd195197e79aba2e2daa167a76634af8587851","unresolved":false,"context_lines":[{"line_number":107,"context_line":"            --init-saved-state"},{"line_number":108,"context_line":"            --delete-tempest-conf-objects"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    tempest config-create"},{"line_number":111,"context_line":"        arguments:"},{"line_number":112,"context_line":"            TBD"},{"line_number":113,"context_line":""}],"source_content_type":"text/x-rst","patch_set":10,"id":"1af86dd1_0931edec","line":110,"updated":"2015-06-11 22:14:10.000000000","message":"curious about this one..","commit_id":"129e261c8ccbe8075c4433f08d947a199e2a6500"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"63cd195197e79aba2e2daa167a76634af8587851","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        arguments:"},{"line_number":112,"context_line":"            TBD"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"    tempest config-verify"},{"line_number":115,"context_line":"        arguments:"},{"line_number":116,"context_line":"            --update, -u"},{"line_number":117,"context_line":"            --output, -o"}],"source_content_type":"text/x-rst","patch_set":10,"id":"1af86dd1_493ec5b9","line":114,"updated":"2015-06-11 22:14:10.000000000","message":"+2","commit_id":"129e261c8ccbe8075c4433f08d947a199e2a6500"}]}