)]}'
{"tempest/cmd/cleanup.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"e64b518308ccc04a8ea2804138e8dc8ac2322fa1","unresolved":false,"context_lines":[{"line_number":43,"context_line":"be preserved from the clean up command. This should be done prior to running"},{"line_number":44,"context_line":"tempest tests."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"**--delete-tempest-conf-objects**: If set to ``true``, then the command will"},{"line_number":47,"context_line":"delete the Tempest and alternate Tempest tenants in addition to the objects"},{"line_number":48,"context_line":"associated with them on clean up. If set to ``false``, the command will"},{"line_number":49,"context_line":"delete the objects associated with the Tempest and alternate Tempest tenants"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a30ddce_467b4328","line":46,"updated":"2017-03-10 09:05:09.000000000","message":"I feel this \u0027set to ``true``\u0027 is a bit confusing. Because this option doesn\u0027t take a boolean value.","commit_id":"79f36b72578ddaabe41edd8e0fb77b92b591bd7e"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"716999383e8a7eff8bba13ef8156866af00eb9a3","unresolved":false,"context_lines":[{"line_number":43,"context_line":"be preserved from the clean up command. This should be done prior to running"},{"line_number":44,"context_line":"tempest tests."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"**--delete-tempest-conf-objects**: If set to ``true``, then the command will"},{"line_number":47,"context_line":"delete the Tempest and alternate Tempest tenants in addition to the objects"},{"line_number":48,"context_line":"associated with them on clean up. If set to ``false``, the command will"},{"line_number":49,"context_line":"delete the objects associated with the Tempest and alternate Tempest tenants"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a30ddce_e4efc584","line":46,"in_reply_to":"9a30ddce_467b4328","updated":"2017-03-10 13:57:03.000000000","message":"Good catch! I\u0027ll make this more clear that this is switched on the presence or absence of the option.","commit_id":"79f36b72578ddaabe41edd8e0fb77b92b591bd7e"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"daa78050fbd28b1cfb0d8a532bde4634f3ebd441","unresolved":false,"context_lines":[{"line_number":19,"context_line":"**Usage:** ``tempest cleanup [--help] [OPTIONS]``"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If run with no arguments, ``tempest cleanup`` will query your OpenStack"},{"line_number":22,"context_line":"deployment and build a list of objects to delete after filtering out the"},{"line_number":23,"context_line":"objects that are listed in ``saved_state.json`` as well as the"},{"line_number":24,"context_line":"**--delete-tempest-conf-objects** flag. By default the Tempest and alternate"},{"line_number":25,"context_line":"Tempest users and tenants are not deleted and the admin user specified in"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_65f5efc9","line":22,"updated":"2017-03-16 09:06:02.000000000","message":"s/objects/resources/","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"a2511b063d2cc6fdf77dcee78dd99071c7276fb5","unresolved":false,"context_lines":[{"line_number":19,"context_line":"**Usage:** ``tempest cleanup [--help] [OPTIONS]``"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If run with no arguments, ``tempest cleanup`` will query your OpenStack"},{"line_number":22,"context_line":"deployment and build a list of objects to delete after filtering out the"},{"line_number":23,"context_line":"objects that are listed in ``saved_state.json`` as well as the"},{"line_number":24,"context_line":"**--delete-tempest-conf-objects** flag. By default the Tempest and alternate"},{"line_number":25,"context_line":"Tempest users and tenants are not deleted and the admin user specified in"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_facdea5d","line":22,"in_reply_to":"3a1ff146_65f5efc9","updated":"2017-03-16 16:11:49.000000000","message":"Done","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"daa78050fbd28b1cfb0d8a532bde4634f3ebd441","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If run with no arguments, ``tempest cleanup`` will query your OpenStack"},{"line_number":22,"context_line":"deployment and build a list of objects to delete after filtering out the"},{"line_number":23,"context_line":"objects that are listed in ``saved_state.json`` as well as the"},{"line_number":24,"context_line":"**--delete-tempest-conf-objects** flag. By default the Tempest and alternate"},{"line_number":25,"context_line":"Tempest users and tenants are not deleted and the admin user specified in"},{"line_number":26,"context_line":"``tempest.conf`` is never deleted."}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_8504abc0","line":23,"updated":"2017-03-16 09:06:02.000000000","message":"same","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"a2511b063d2cc6fdf77dcee78dd99071c7276fb5","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If run with no arguments, ``tempest cleanup`` will query your OpenStack"},{"line_number":22,"context_line":"deployment and build a list of objects to delete after filtering out the"},{"line_number":23,"context_line":"objects that are listed in ``saved_state.json`` as well as the"},{"line_number":24,"context_line":"**--delete-tempest-conf-objects** flag. By default the Tempest and alternate"},{"line_number":25,"context_line":"Tempest users and tenants are not deleted and the admin user specified in"},{"line_number":26,"context_line":"``tempest.conf`` is never deleted."}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_1add660e","line":23,"in_reply_to":"3a1ff146_8504abc0","updated":"2017-03-16 16:11:49.000000000","message":"Done","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fd7e53f477c333b509d965a6fc081526812c057d","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` should be with the"},{"line_number":29,"context_line":"**--init-saved-state** option. Doing so will initialize the saved state of"},{"line_number":30,"context_line":"the OpenStack deployment such that when the clean up command is given it"},{"line_number":31,"context_line":"will only delete objects that do not need to be kept. Alternatively, you can"},{"line_number":32,"context_line":"create a ``saved_state.json`` file manually that will specify the objects"},{"line_number":33,"context_line":"that ``tempest cleanup`` shall not delete. For an example of the format of"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_98189386","line":30,"range":{"start_line":30,"start_character":50,"end_line":30,"end_character":52},"updated":"2017-03-16 08:42:45.000000000","message":"it should be cleanup.","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"a2511b063d2cc6fdf77dcee78dd99071c7276fb5","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` should be with the"},{"line_number":29,"context_line":"**--init-saved-state** option. Doing so will initialize the saved state of"},{"line_number":30,"context_line":"the OpenStack deployment such that when the clean up command is given it"},{"line_number":31,"context_line":"will only delete objects that do not need to be kept. Alternatively, you can"},{"line_number":32,"context_line":"create a ``saved_state.json`` file manually that will specify the objects"},{"line_number":33,"context_line":"that ``tempest cleanup`` shall not delete. For an example of the format of"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_3ad8621d","line":30,"range":{"start_line":30,"start_character":50,"end_line":30,"end_character":52},"in_reply_to":"3a1ff146_98189386","updated":"2017-03-16 16:11:49.000000000","message":"Done","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"daa78050fbd28b1cfb0d8a532bde4634f3ebd441","unresolved":false,"context_lines":[{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` should be with the"},{"line_number":29,"context_line":"**--init-saved-state** option. Doing so will initialize the saved state of"},{"line_number":30,"context_line":"the OpenStack deployment such that when the clean up command is given it"},{"line_number":31,"context_line":"will only delete objects that do not need to be kept. Alternatively, you can"},{"line_number":32,"context_line":"create a ``saved_state.json`` file manually that will specify the objects"},{"line_number":33,"context_line":"that ``tempest cleanup`` shall not delete. For an example of the format of"},{"line_number":34,"context_line":"this file, run the command with the **--init-saved-state** flag and look at"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_0527db64","line":31,"updated":"2017-03-16 09:06:02.000000000","message":"dito","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"a2511b063d2cc6fdf77dcee78dd99071c7276fb5","unresolved":false,"context_lines":[{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` should be with the"},{"line_number":29,"context_line":"**--init-saved-state** option. Doing so will initialize the saved state of"},{"line_number":30,"context_line":"the OpenStack deployment such that when the clean up command is given it"},{"line_number":31,"context_line":"will only delete objects that do not need to be kept. Alternatively, you can"},{"line_number":32,"context_line":"create a ``saved_state.json`` file manually that will specify the objects"},{"line_number":33,"context_line":"that ``tempest cleanup`` shall not delete. For an example of the format of"},{"line_number":34,"context_line":"this file, run the command with the **--init-saved-state** flag and look at"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_5a051e78","line":31,"in_reply_to":"3a1ff146_0527db64","updated":"2017-03-16 16:11:49.000000000","message":"Done","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fd7e53f477c333b509d965a6fc081526812c057d","unresolved":false,"context_lines":[{"line_number":40,"context_line":"**--init-saved-state**: Typically used on an initial run of the command,"},{"line_number":41,"context_line":"it will initialize the saved state of the OpenStack deployment and output a"},{"line_number":42,"context_line":"``saved_state.json`` file containing objects from your deployment that will"},{"line_number":43,"context_line":"be preserved from the clean up command. This should be done prior to running"},{"line_number":44,"context_line":"tempest tests."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"**--delete-tempest-conf-objects**: If option is present, then the command will"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_1342c43e","line":43,"range":{"start_line":43,"start_character":22,"end_line":43,"end_character":30},"updated":"2017-03-16 08:42:45.000000000","message":"It should be \"cleanup\".","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"a2511b063d2cc6fdf77dcee78dd99071c7276fb5","unresolved":false,"context_lines":[{"line_number":40,"context_line":"**--init-saved-state**: Typically used on an initial run of the command,"},{"line_number":41,"context_line":"it will initialize the saved state of the OpenStack deployment and output a"},{"line_number":42,"context_line":"``saved_state.json`` file containing objects from your deployment that will"},{"line_number":43,"context_line":"be preserved from the clean up command. This should be done prior to running"},{"line_number":44,"context_line":"tempest tests."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"**--delete-tempest-conf-objects**: If option is present, then the command will"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_7a081a92","line":43,"range":{"start_line":43,"start_character":22,"end_line":43,"end_character":30},"in_reply_to":"3a1ff146_1342c43e","updated":"2017-03-16 16:11:49.000000000","message":"Done","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fd7e53f477c333b509d965a6fc081526812c057d","unresolved":false,"context_lines":[{"line_number":44,"context_line":"tempest tests."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"**--delete-tempest-conf-objects**: If option is present, then the command will"},{"line_number":47,"context_line":"delete the Tempest and alternate Tempest tenants in addition to the objects"},{"line_number":48,"context_line":"associated with them on clean up. If option is not present, the command will"},{"line_number":49,"context_line":"delete the objects associated with the Tempest and alternate Tempest tenants"},{"line_number":50,"context_line":"but will not delete the tenants themselves."}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_98eed389","line":47,"range":{"start_line":47,"start_character":33,"end_line":47,"end_character":48},"updated":"2017-03-16 08:42:45.000000000","message":"It should be Tempest users and tenants.","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"daa78050fbd28b1cfb0d8a532bde4634f3ebd441","unresolved":false,"context_lines":[{"line_number":44,"context_line":"tempest tests."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"**--delete-tempest-conf-objects**: If option is present, then the command will"},{"line_number":47,"context_line":"delete the Tempest and alternate Tempest tenants in addition to the objects"},{"line_number":48,"context_line":"associated with them on clean up. If option is not present, the command will"},{"line_number":49,"context_line":"delete the objects associated with the Tempest and alternate Tempest tenants"},{"line_number":50,"context_line":"but will not delete the tenants themselves."}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_a59b6771","line":47,"range":{"start_line":47,"start_character":33,"end_line":47,"end_character":48},"in_reply_to":"3a1ff146_98eed389","updated":"2017-03-16 09:06:02.000000000","message":"Are you sure the actual Tempest keystone users are deleted by cleaneup ?","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"a2511b063d2cc6fdf77dcee78dd99071c7276fb5","unresolved":false,"context_lines":[{"line_number":44,"context_line":"tempest tests."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"**--delete-tempest-conf-objects**: If option is present, then the command will"},{"line_number":47,"context_line":"delete the Tempest and alternate Tempest tenants in addition to the objects"},{"line_number":48,"context_line":"associated with them on clean up. If option is not present, the command will"},{"line_number":49,"context_line":"delete the objects associated with the Tempest and alternate Tempest tenants"},{"line_number":50,"context_line":"but will not delete the tenants themselves."}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_638d427b","line":47,"range":{"start_line":47,"start_character":33,"end_line":47,"end_character":48},"in_reply_to":"3a1ff146_98eed389","updated":"2017-03-16 16:11:49.000000000","message":"Done","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"a2511b063d2cc6fdf77dcee78dd99071c7276fb5","unresolved":false,"context_lines":[{"line_number":44,"context_line":"tempest tests."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"**--delete-tempest-conf-objects**: If option is present, then the command will"},{"line_number":47,"context_line":"delete the Tempest and alternate Tempest tenants in addition to the objects"},{"line_number":48,"context_line":"associated with them on clean up. If option is not present, the command will"},{"line_number":49,"context_line":"delete the objects associated with the Tempest and alternate Tempest tenants"},{"line_number":50,"context_line":"but will not delete the tenants themselves."}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_23a0aa90","line":47,"range":{"start_line":47,"start_character":33,"end_line":47,"end_character":48},"in_reply_to":"3a1ff146_a59b6771","updated":"2017-03-16 16:11:49.000000000","message":"Looking at the cleanup_service.py code for the UserService class it looks that way. Check out https://github.com/openstack/tempest/blob/master/tempest/cmd/cleanup_service.py#L788-L828, keeping in mind that the \"is_preserve\" flag is the inverse of whether this flag is set or not.","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fd7e53f477c333b509d965a6fc081526812c057d","unresolved":false,"context_lines":[{"line_number":46,"context_line":"**--delete-tempest-conf-objects**: If option is present, then the command will"},{"line_number":47,"context_line":"delete the Tempest and alternate Tempest tenants in addition to the objects"},{"line_number":48,"context_line":"associated with them on clean up. If option is not present, the command will"},{"line_number":49,"context_line":"delete the objects associated with the Tempest and alternate Tempest tenants"},{"line_number":50,"context_line":"but will not delete the tenants themselves."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"**--dry-run**: Creates a report (``./dry_run.json``) of the tenants that will"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_d89f6b01","line":49,"range":{"start_line":49,"start_character":61,"end_line":49,"end_character":76},"updated":"2017-03-16 08:42:45.000000000","message":"It should be Tempest users and tenants.","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"a2511b063d2cc6fdf77dcee78dd99071c7276fb5","unresolved":false,"context_lines":[{"line_number":46,"context_line":"**--delete-tempest-conf-objects**: If option is present, then the command will"},{"line_number":47,"context_line":"delete the Tempest and alternate Tempest tenants in addition to the objects"},{"line_number":48,"context_line":"associated with them on clean up. If option is not present, the command will"},{"line_number":49,"context_line":"delete the objects associated with the Tempest and alternate Tempest tenants"},{"line_number":50,"context_line":"but will not delete the tenants themselves."},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"**--dry-run**: Creates a report (``./dry_run.json``) of the tenants that will"}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_e3b4d2d2","line":49,"range":{"start_line":49,"start_character":61,"end_line":49,"end_character":76},"in_reply_to":"3a1ff146_d89f6b01","updated":"2017-03-16 16:11:49.000000000","message":"Done","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fd7e53f477c333b509d965a6fc081526812c057d","unresolved":false,"context_lines":[{"line_number":52,"context_line":"**--dry-run**: Creates a report (``./dry_run.json``) of the tenants that will"},{"line_number":53,"context_line":"be cleaned up (in the ``_tenants_to_clean`` dictionary [1]_) and the global"},{"line_number":54,"context_line":"objects that will be removed (domains, flavors, images, roles, tenants,"},{"line_number":55,"context_line":"and users). Once cleanup is executed (e.g. run without parameters), running"},{"line_number":56,"context_line":"it again with **--dry-run** should yield an empty report."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"**--help**: Print the help text for the command and parameters."}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_73e3f833","line":55,"range":{"start_line":55,"start_character":17,"end_line":55,"end_character":25},"updated":"2017-03-16 08:42:45.000000000","message":"It should be \"cleanup command\".","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"a2511b063d2cc6fdf77dcee78dd99071c7276fb5","unresolved":false,"context_lines":[{"line_number":52,"context_line":"**--dry-run**: Creates a report (``./dry_run.json``) of the tenants that will"},{"line_number":53,"context_line":"be cleaned up (in the ``_tenants_to_clean`` dictionary [1]_) and the global"},{"line_number":54,"context_line":"objects that will be removed (domains, flavors, images, roles, tenants,"},{"line_number":55,"context_line":"and users). Once cleanup is executed (e.g. run without parameters), running"},{"line_number":56,"context_line":"it again with **--dry-run** should yield an empty report."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"**--help**: Print the help text for the command and parameters."}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_03f80e7c","line":55,"range":{"start_line":55,"start_character":17,"end_line":55,"end_character":25},"in_reply_to":"3a1ff146_73e3f833","updated":"2017-03-16 16:11:49.000000000","message":"Done","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"fd7e53f477c333b509d965a6fc081526812c057d","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    tenants that ``tempest cleanup`` will loop through to delete child"},{"line_number":62,"context_line":"    objects, but the command will, by default, not delete the tenants"},{"line_number":63,"context_line":"    themselves. This may differ from the ``tenants`` list as you can clean"},{"line_number":64,"context_line":"    the Tempest and alternate Tempest tenants but they will not be deleted"},{"line_number":65,"context_line":"    unless the **--delete-tempest-conf-objects** flag is used to force their"},{"line_number":66,"context_line":"    deletion."},{"line_number":67,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3a1ff146_5859bb59","line":64,"range":{"start_line":64,"start_character":30,"end_line":64,"end_character":45},"updated":"2017-03-16 08:42:45.000000000","message":"It should be Tempest users and tenants.","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"a2511b063d2cc6fdf77dcee78dd99071c7276fb5","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    tenants that ``tempest cleanup`` will loop through to delete child"},{"line_number":62,"context_line":"    objects, but the command will, by default, not delete the tenants"},{"line_number":63,"context_line":"    themselves. This may differ from the ``tenants`` list as you can clean"},{"line_number":64,"context_line":"    the Tempest and alternate Tempest tenants but they will not be deleted"},{"line_number":65,"context_line":"    unless the **--delete-tempest-conf-objects** flag is used to force their"},{"line_number":66,"context_line":"    deletion."},{"line_number":67,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_e3393243","line":64,"range":{"start_line":64,"start_character":30,"end_line":64,"end_character":45},"in_reply_to":"3a1ff146_5859bb59","updated":"2017-03-16 16:11:49.000000000","message":"Done","commit_id":"8d88303af1803d934e32036269a95e52bb63c49a"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"b55d6ace81278c065eb34428ded4b3c9214df67a","unresolved":false,"context_lines":[{"line_number":19,"context_line":"**Usage:** ``tempest cleanup [--help] [OPTIONS]``"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If run with no arguments, ``tempest cleanup`` will query your OpenStack"},{"line_number":22,"context_line":"deployment and build a list of resources to delete after filtering out the"},{"line_number":23,"context_line":"resources that are listed in ``saved_state.json`` as well as the"},{"line_number":24,"context_line":"**--delete-tempest-conf-objects** flag. By default the Tempest and alternate"},{"line_number":25,"context_line":"Tempest users and tenants are not deleted and the admin user specified in"},{"line_number":26,"context_line":"``tempest.conf`` is never deleted."},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"ffe62b97_4991bcf0","line":24,"range":{"start_line":22,"start_character":41,"end_line":24,"end_character":39},"updated":"2017-03-29 14:56:47.000000000","message":"This part of the sentence is not very clear, what about breaking it down:\n...list of resources to delete.\nCreated list will exclude the resources from ``saved_state.json``. Option ``--delete-tempest-conf-objects`` will be ignored (if present).","commit_id":"def4a528231977df0a6fbb8ac2bd167fc7eb2f33"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"b55d6ace81278c065eb34428ded4b3c9214df67a","unresolved":false,"context_lines":[{"line_number":25,"context_line":"Tempest users and tenants are not deleted and the admin user specified in"},{"line_number":26,"context_line":"``tempest.conf`` is never deleted."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` should be with the"},{"line_number":29,"context_line":"**--init-saved-state** option. Doing so will initialize the saved state of"},{"line_number":30,"context_line":"the OpenStack deployment such that when the cleanup command is given it"},{"line_number":31,"context_line":"will only delete resources that do not need to be kept. Alternatively, you can"}],"source_content_type":"text/x-python","patch_set":6,"id":"ffe62b97_8910e458","line":28,"range":{"start_line":28,"start_character":60,"end_line":28,"end_character":78},"updated":"2017-03-29 14:56:47.000000000","message":"replace with \"utilize\"","commit_id":"def4a528231977df0a6fbb8ac2bd167fc7eb2f33"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"b55d6ace81278c065eb34428ded4b3c9214df67a","unresolved":false,"context_lines":[{"line_number":26,"context_line":"``tempest.conf`` is never deleted."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` should be with the"},{"line_number":29,"context_line":"**--init-saved-state** option. Doing so will initialize the saved state of"},{"line_number":30,"context_line":"the OpenStack deployment such that when the cleanup command is given it"},{"line_number":31,"context_line":"will only delete resources that do not need to be kept. Alternatively, you can"},{"line_number":32,"context_line":"create a ``saved_state.json`` file manually that will specify the resources"}],"source_content_type":"text/x-python","patch_set":6,"id":"ffe62b97_048dd930","line":29,"range":{"start_line":29,"start_character":45,"end_line":29,"end_character":71},"updated":"2017-03-29 14:56:47.000000000","message":"sed \u0027s/initialize the saved state/save the existing resources/g\u0027","commit_id":"def4a528231977df0a6fbb8ac2bd167fc7eb2f33"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"b55d6ace81278c065eb34428ded4b3c9214df67a","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` should be with the"},{"line_number":29,"context_line":"**--init-saved-state** option. Doing so will initialize the saved state of"},{"line_number":30,"context_line":"the OpenStack deployment such that when the cleanup command is given it"},{"line_number":31,"context_line":"will only delete resources that do not need to be kept. Alternatively, you can"},{"line_number":32,"context_line":"create a ``saved_state.json`` file manually that will specify the resources"},{"line_number":33,"context_line":"that ``tempest cleanup`` shall not delete. For an example of the format of"}],"source_content_type":"text/x-python","patch_set":6,"id":"ffe62b97_a4aced9e","line":30,"range":{"start_line":30,"start_character":63,"end_line":30,"end_character":68},"updated":"2017-03-29 14:56:47.000000000","message":"sed \u0027s/given/executed;/g\u0027","commit_id":"def4a528231977df0a6fbb8ac2bd167fc7eb2f33"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"1c7826b89243f2cf9ace5aa79132e8c7f79bb002","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If run with no arguments, ``tempest cleanup`` will query your OpenStack"},{"line_number":22,"context_line":"deployment and build a list of resources to delete and destroy them. This list"},{"line_number":23,"context_line":"will exclude the resources from ``saved_state.json`` and will include the"},{"line_number":24,"context_line":"Tempest and alternate Tempest users and tenants if the"},{"line_number":25,"context_line":"``--delete-tempest-conf-objects`` flag is specified. By default the Tempest"},{"line_number":26,"context_line":"and alternate Tempest users and tenants are not deleted and the admin user"},{"line_number":27,"context_line":"specified in ``tempest.conf`` is never deleted."},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"dfeb2761_aaa00ea7","line":25,"range":{"start_line":23,"start_character":53,"end_line":25,"end_character":51},"updated":"2017-03-30 21:23:26.000000000","message":"Tempest and Tempest alt tenants are not defined anymore.\nConfig only includes an admin account. \nAny other account is either created on the fly or stored in accounts.yaml.\n\nLooking at the code, this covers only the admin account.","commit_id":"c34f6683af00d23dbafeee48cd2c1be9ea9d0df2"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"30993f006a1bdc6b89cef8da8aeb7d991cc2a94a","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If run with no arguments, ``tempest cleanup`` will query your OpenStack"},{"line_number":22,"context_line":"deployment and build a list of resources to delete and destroy them. This list"},{"line_number":23,"context_line":"will exclude the resources from ``saved_state.json`` and will include the"},{"line_number":24,"context_line":"Tempest and alternate Tempest users and tenants if the"},{"line_number":25,"context_line":"``--delete-tempest-conf-objects`` flag is specified. By default the Tempest"},{"line_number":26,"context_line":"and alternate Tempest users and tenants are not deleted and the admin user"},{"line_number":27,"context_line":"specified in ``tempest.conf`` is never deleted."},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"dfeb2761_1f195a78","line":25,"range":{"start_line":23,"start_character":53,"end_line":25,"end_character":51},"in_reply_to":"dfeb2761_aaa00ea7","updated":"2017-03-31 17:37:53.000000000","message":"Ahh, good to know, thanks for the clarification, Andrea. I\u0027ll make this clear in the next patchset.","commit_id":"c34f6683af00d23dbafeee48cd2c1be9ea9d0df2"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"1c7826b89243f2cf9ace5aa79132e8c7f79bb002","unresolved":false,"context_lines":[{"line_number":45,"context_line":"be preserved from the cleanup command. This should be done prior to running"},{"line_number":46,"context_line":"Tempest tests."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"**--delete-tempest-conf-objects**: If option is present, then the command will"},{"line_number":49,"context_line":"delete the Tempest and alternate Tempest users and tenants in addition to the"},{"line_number":50,"context_line":"resources associated with them on clean up. If option is not present, the"},{"line_number":51,"context_line":"command will delete the resources associated with the Tempest and alternate"},{"line_number":52,"context_line":"Tempest users and tenants but will not delete the tenants themselves."}],"source_content_type":"text/x-python","patch_set":7,"id":"dfeb2761_8a8e6a2c","line":49,"range":{"start_line":48,"start_character":35,"end_line":49,"end_character":58},"updated":"2017-03-30 21:23:26.000000000","message":"Ditto","commit_id":"c34f6683af00d23dbafeee48cd2c1be9ea9d0df2"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"30993f006a1bdc6b89cef8da8aeb7d991cc2a94a","unresolved":false,"context_lines":[{"line_number":45,"context_line":"be preserved from the cleanup command. This should be done prior to running"},{"line_number":46,"context_line":"Tempest tests."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"**--delete-tempest-conf-objects**: If option is present, then the command will"},{"line_number":49,"context_line":"delete the Tempest and alternate Tempest users and tenants in addition to the"},{"line_number":50,"context_line":"resources associated with them on clean up. If option is not present, the"},{"line_number":51,"context_line":"command will delete the resources associated with the Tempest and alternate"},{"line_number":52,"context_line":"Tempest users and tenants but will not delete the tenants themselves."}],"source_content_type":"text/x-python","patch_set":7,"id":"dfeb2761_bf214ea2","line":49,"range":{"start_line":48,"start_character":35,"end_line":49,"end_character":58},"in_reply_to":"dfeb2761_8a8e6a2c","updated":"2017-03-31 17:37:53.000000000","message":"I\u0027ll do the same here. Thanks!","commit_id":"c34f6683af00d23dbafeee48cd2c1be9ea9d0df2"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"bfc563cf5b9c381fe39dd7979f5f3ec246e1046b","unresolved":false,"context_lines":[{"line_number":25,"context_line":"specified. By default the admin tenants is not deleted and the admin user"},{"line_number":26,"context_line":"specified in ``tempest.conf`` is never deleted."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` utilize the"},{"line_number":29,"context_line":"**--init-saved-state** option. Doing so will enumerate the existing"},{"line_number":30,"context_line":"resources of the OpenStack deployment in the ``saved_state.json`` file such"},{"line_number":31,"context_line":"that when the cleanup command is executed it will only delete resources that"}],"source_content_type":"text/x-python","patch_set":8,"id":"dfeb2761_6bc2e487","line":28,"updated":"2017-03-31 20:04:58.000000000","message":"You probably should mention you do this before a tempest test run too. It might not be clear from the way this is worded. It is below in the argument description, but this section is describing the typical workflow, and that\u0027s an important thing to mention.","commit_id":"55aeed4aa1f5240e162d78128ee3cd51d2c2381b"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"03923b6a45f14c8ade767a5056d383de268c17e6","unresolved":false,"context_lines":[{"line_number":22,"context_line":"deployment and build a list of resources to delete and destroy them. This list"},{"line_number":23,"context_line":"will exclude the resources from ``saved_state.json`` and will include the"},{"line_number":24,"context_line":"configured admin account if the ``--delete-tempest-conf-objects`` flag is"},{"line_number":25,"context_line":"specified. By default the admin tenants is not deleted and the admin user"},{"line_number":26,"context_line":"specified in ``tempest.conf`` is never deleted."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` utilize the"}],"source_content_type":"text/x-python","patch_set":9,"id":"bff0334d_f4122409","line":25,"range":{"start_line":25,"start_character":38,"end_line":25,"end_character":39},"updated":"2017-04-07 11:47:10.000000000","message":"tenant - or better, project","commit_id":"562b8fa42ded7861fdfcc92c7762fbf173bb0e61"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"ad4a703ca9992a7cc9c64462e7d133725a2932f8","unresolved":false,"context_lines":[{"line_number":22,"context_line":"deployment and build a list of resources to delete and destroy them. This list"},{"line_number":23,"context_line":"will exclude the resources from ``saved_state.json`` and will include the"},{"line_number":24,"context_line":"configured admin account if the ``--delete-tempest-conf-objects`` flag is"},{"line_number":25,"context_line":"specified. By default the admin tenants is not deleted and the admin user"},{"line_number":26,"context_line":"specified in ``tempest.conf`` is never deleted."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` utilize the"}],"source_content_type":"text/x-python","patch_set":9,"id":"9ff52f3f_e07ceeb6","line":25,"range":{"start_line":25,"start_character":38,"end_line":25,"end_character":39},"in_reply_to":"bff0334d_f4122409","updated":"2017-04-13 15:17:08.000000000","message":"Done","commit_id":"562b8fa42ded7861fdfcc92c7762fbf173bb0e61"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"03923b6a45f14c8ade767a5056d383de268c17e6","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"**Usage:** ``tempest cleanup [--help] [OPTIONS]``"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If run with no arguments, ``tempest cleanup`` will query your OpenStack"},{"line_number":22,"context_line":"deployment and build a list of resources to delete and destroy them. This list"},{"line_number":23,"context_line":"will exclude the resources from ``saved_state.json`` and will include the"},{"line_number":24,"context_line":"configured admin account if the ``--delete-tempest-conf-objects`` flag is"},{"line_number":25,"context_line":"specified. By default the admin tenants is not deleted and the admin user"},{"line_number":26,"context_line":"specified in ``tempest.conf`` is never deleted."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` utilize the"},{"line_number":29,"context_line":"**--init-saved-state** option and be executed before running any sort of"},{"line_number":30,"context_line":"Tempest test run. Doing so will enumerate the existing resources of the"},{"line_number":31,"context_line":"OpenStack deployment inthe ``saved_state.json`` file such that when the cleanup"},{"line_number":32,"context_line":"command is executed it will only delete resources that do not need to be kept."},{"line_number":33,"context_line":"Alternatively, you can create a ``saved_state.json`` file manually that will"},{"line_number":34,"context_line":"specify the resources that ``tempest cleanup`` shall not delete. For an example"},{"line_number":35,"context_line":"of the format of this file, run the command with the **--init-saved-state**"},{"line_number":36,"context_line":"flag and look at the generated ``./saved_state.json`` file."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Runtime Arguments"},{"line_number":39,"context_line":"-----------------"}],"source_content_type":"text/x-python","patch_set":9,"id":"bff0334d_d7a3a29d","line":36,"range":{"start_line":21,"start_character":0,"end_line":36,"end_character":59},"updated":"2017-04-07 11:47:10.000000000","message":"The meaning of each config option is explained below already, so I think what would be useful to add instead is an example of the standard why to use this, i.e.\n\n$ tempest-cleanup --init-saved-state\n$ [test run]\n$ tempest-cleanup\n\nso it\u0027s clear what to do.\n\nI would also add a BIG warning, saying that if step 1 is skipped, the cleanup may delete resources that were in the cloud before the test run.","commit_id":"562b8fa42ded7861fdfcc92c7762fbf173bb0e61"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"ad4a703ca9992a7cc9c64462e7d133725a2932f8","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"**Usage:** ``tempest cleanup [--help] [OPTIONS]``"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"If run with no arguments, ``tempest cleanup`` will query your OpenStack"},{"line_number":22,"context_line":"deployment and build a list of resources to delete and destroy them. This list"},{"line_number":23,"context_line":"will exclude the resources from ``saved_state.json`` and will include the"},{"line_number":24,"context_line":"configured admin account if the ``--delete-tempest-conf-objects`` flag is"},{"line_number":25,"context_line":"specified. By default the admin tenants is not deleted and the admin user"},{"line_number":26,"context_line":"specified in ``tempest.conf`` is never deleted."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"It is recommended that the first run of ``tempest cleanup`` utilize the"},{"line_number":29,"context_line":"**--init-saved-state** option and be executed before running any sort of"},{"line_number":30,"context_line":"Tempest test run. Doing so will enumerate the existing resources of the"},{"line_number":31,"context_line":"OpenStack deployment inthe ``saved_state.json`` file such that when the cleanup"},{"line_number":32,"context_line":"command is executed it will only delete resources that do not need to be kept."},{"line_number":33,"context_line":"Alternatively, you can create a ``saved_state.json`` file manually that will"},{"line_number":34,"context_line":"specify the resources that ``tempest cleanup`` shall not delete. For an example"},{"line_number":35,"context_line":"of the format of this file, run the command with the **--init-saved-state**"},{"line_number":36,"context_line":"flag and look at the generated ``./saved_state.json`` file."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Runtime Arguments"},{"line_number":39,"context_line":"-----------------"}],"source_content_type":"text/x-python","patch_set":9,"id":"9ff52f3f_fad13659","line":36,"range":{"start_line":21,"start_character":0,"end_line":36,"end_character":59},"in_reply_to":"bff0334d_d7a3a29d","updated":"2017-04-13 15:17:08.000000000","message":"Done, I\u0027ve removed this section and replaced it with an \"Example Run\" section with a bolded warning stating that if you skip the first step very bad things could happen. Thanks!","commit_id":"562b8fa42ded7861fdfcc92c7762fbf173bb0e61"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"03923b6a45f14c8ade767a5056d383de268c17e6","unresolved":false,"context_lines":[{"line_number":38,"context_line":"Runtime Arguments"},{"line_number":39,"context_line":"-----------------"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"**--init-saved-state**: Typically used on an initial run of the command,"},{"line_number":42,"context_line":"it will initialize the saved state of the OpenStack deployment and output a"},{"line_number":43,"context_line":"``saved_state.json`` file containing resources from your deployment that will"},{"line_number":44,"context_line":"be preserved from the cleanup command. This should be done prior to running"}],"source_content_type":"text/x-python","patch_set":9,"id":"bff0334d_f7b586ce","line":41,"range":{"start_line":41,"start_character":24,"end_line":41,"end_character":72},"updated":"2017-04-07 11:47:10.000000000","message":"nit: I would only state what this does, so remove this line","commit_id":"562b8fa42ded7861fdfcc92c7762fbf173bb0e61"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"ad4a703ca9992a7cc9c64462e7d133725a2932f8","unresolved":false,"context_lines":[{"line_number":38,"context_line":"Runtime Arguments"},{"line_number":39,"context_line":"-----------------"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"**--init-saved-state**: Typically used on an initial run of the command,"},{"line_number":42,"context_line":"it will initialize the saved state of the OpenStack deployment and output a"},{"line_number":43,"context_line":"``saved_state.json`` file containing resources from your deployment that will"},{"line_number":44,"context_line":"be preserved from the cleanup command. This should be done prior to running"}],"source_content_type":"text/x-python","patch_set":9,"id":"9ff52f3f_7af886c3","line":41,"range":{"start_line":41,"start_character":24,"end_line":41,"end_character":72},"in_reply_to":"bff0334d_f7b586ce","updated":"2017-04-13 15:17:08.000000000","message":"Done","commit_id":"562b8fa42ded7861fdfcc92c7762fbf173bb0e61"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"9c65d79ca6ef8290fab3e7cbdcc81981995283b9","unresolved":false,"context_lines":[{"line_number":28,"context_line":"Example Run"},{"line_number":29,"context_line":"-----------"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"**WARNING: If step 1 is skipped in the example below, the cleanup procedure "},{"line_number":32,"context_line":"may delete resources that existed in the cloud before the test run. This "},{"line_number":33,"context_line":"may cause an unwanted destruction of cloud resources, so use caution with "},{"line_number":34,"context_line":"this command.**"}],"source_content_type":"text/x-python","patch_set":10,"id":"7ffa3b31_c6979d90","line":31,"updated":"2017-04-17 17:58:33.000000000","message":"We need to remove the end of space at this line as\n\nhttp://logs.openstack.org/41/444041/10/check/gate-tempest-pep8-ubuntu-xenial/8d20b21/console.html#_2017-04-17_08_36_50_944464\n\nThis patch contains another invalid coding style, please check the above log more.","commit_id":"630e3cb79e8e3c79f74ca4cc99b8bf59c67e1b0b"},{"author":{"_account_id":17780,"name":"Dustin Schoenbrun","email":"dschoenb@redhat.com","username":"dschoenb"},"change_message_id":"f4d0b3680323a92fae79726a09dc2750892ec9d7","unresolved":false,"context_lines":[{"line_number":28,"context_line":"Example Run"},{"line_number":29,"context_line":"-----------"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"**WARNING: If step 1 is skipped in the example below, the cleanup procedure "},{"line_number":32,"context_line":"may delete resources that existed in the cloud before the test run. This "},{"line_number":33,"context_line":"may cause an unwanted destruction of cloud resources, so use caution with "},{"line_number":34,"context_line":"this command.**"}],"source_content_type":"text/x-python","patch_set":10,"id":"7ffa3b31_b94dd373","line":31,"in_reply_to":"7ffa3b31_c6979d90","updated":"2017-04-17 20:37:02.000000000","message":"Trailing whitespace, the bane of my patchsets! I\u0027ll push up another without them and see if that solves our issues.","commit_id":"630e3cb79e8e3c79f74ca4cc99b8bf59c67e1b0b"}]}