)]}'
{"tempest/templates/bin/_run-tests.sh.tpl":[{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"dd92cdfd398b84ce3b254e4f934553e897e1e5db","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"set -ex"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"{{ if .Values.conf.cleanup.enabled }}"},{"line_number":22,"context_line":"tempest cleanup --init-saved-state"},{"line_number":23,"context_line":"{{- end }}"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"{{ .Values.conf.script }}"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"{{ if .Values.conf.cleanup.enabled }}"},{"line_number":28,"context_line":"tempest cleanup"},{"line_number":29,"context_line":"{{- end }}"}],"source_content_type":"text/x-smarty","patch_set":1,"id":"3f79a3b5_dec2e785","line":29,"range":{"start_line":21,"start_character":0,"end_line":29,"end_character":10},"updated":"2018-08-07 13:47:21.000000000","message":"If cleanup does need to be configurable (given a proper use case), we should just include the cleanup commands in the .Values.conf.script value and get rid of these conditionals","commit_id":"9385066f929f18602317f6077c9cef025ca753b9"}],"tempest/values.yaml":[{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"dd92cdfd398b84ce3b254e4f934553e897e1e5db","unresolved":false,"context_lines":[{"line_number":119,"context_line":"  #   - (?:tempest\\.scenario\\.test_network_basic_ops\\.TestNetworkBasicOps\\.test_update_router_admin_state)"},{"line_number":120,"context_line":"  #   - (?:tempest\\.scenario\\.test_network_basic_ops\\.TestNetworkBasicOps\\.test_router_rescheduling)"},{"line_number":121,"context_line":"  #   - (?:tempest\\.scenario\\.test_network_basic_ops\\.TestNetworkBasicOps\\.test_update_instance_port_admin_state)"},{"line_number":122,"context_line":"  cleanup:"},{"line_number":123,"context_line":"    enabled: true"},{"line_number":124,"context_line":"  tempest:"},{"line_number":125,"context_line":"    auth:"},{"line_number":126,"context_line":"      use_dynamic_credentials: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_bedccb74","line":123,"range":{"start_line":122,"start_character":0,"end_line":123,"end_character":17},"updated":"2018-08-07 13:47:21.000000000","message":"Is there a situation where we wouldn\u0027t want Tempest to cleanup after itself?","commit_id":"9385066f929f18602317f6077c9cef025ca753b9"},{"author":{"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"},"change_message_id":"d2e55770f16e6cf9fb6cef4026cfd44ac8dbbc29","unresolved":false,"context_lines":[{"line_number":119,"context_line":"  #   - (?:tempest\\.scenario\\.test_network_basic_ops\\.TestNetworkBasicOps\\.test_update_router_admin_state)"},{"line_number":120,"context_line":"  #   - (?:tempest\\.scenario\\.test_network_basic_ops\\.TestNetworkBasicOps\\.test_router_rescheduling)"},{"line_number":121,"context_line":"  #   - (?:tempest\\.scenario\\.test_network_basic_ops\\.TestNetworkBasicOps\\.test_update_instance_port_admin_state)"},{"line_number":122,"context_line":"  cleanup:"},{"line_number":123,"context_line":"    enabled: true"},{"line_number":124,"context_line":"  tempest:"},{"line_number":125,"context_line":"    auth:"},{"line_number":126,"context_line":"      use_dynamic_credentials: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_d93be135","line":123,"range":{"start_line":122,"start_character":0,"end_line":123,"end_character":17},"in_reply_to":"3f79a3b5_bedccb74","updated":"2018-08-07 14:37:26.000000000","message":"The tempest utility that is configured here will take a snapshot.  After the snapashot and test run it will delete any resources that are not in the snapshot. If a tenant create a resource outside of the test run it will be deleted.  So this is something you do not want to do in an live lab. We are going to propose a spec to only review project for resources to allow this type of process on a live lab not use a gate lab.","commit_id":"9385066f929f18602317f6077c9cef025ca753b9"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"e0ac00d20c69fbd6615eb99c39dfa7cd11ad1693","unresolved":false,"context_lines":[{"line_number":119,"context_line":"  #   - (?:tempest\\.scenario\\.test_network_basic_ops\\.TestNetworkBasicOps\\.test_update_router_admin_state)"},{"line_number":120,"context_line":"  #   - (?:tempest\\.scenario\\.test_network_basic_ops\\.TestNetworkBasicOps\\.test_router_rescheduling)"},{"line_number":121,"context_line":"  #   - (?:tempest\\.scenario\\.test_network_basic_ops\\.TestNetworkBasicOps\\.test_update_instance_port_admin_state)"},{"line_number":122,"context_line":"  cleanup:"},{"line_number":123,"context_line":"    enabled: true"},{"line_number":124,"context_line":"  tempest:"},{"line_number":125,"context_line":"    auth:"},{"line_number":126,"context_line":"      use_dynamic_credentials: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_f9e9fd7f","line":123,"range":{"start_line":122,"start_character":0,"end_line":123,"end_character":17},"in_reply_to":"3f79a3b5_d93be135","updated":"2018-08-07 14:44:25.000000000","message":"Okay, that makes sense. In that case, I\u0027d rather us just move to include the cleanup flags in the default script passed in via values, then it can be overridden when required. I think that\u0027s a cleaner approach than creating a new flag for this use case.","commit_id":"9385066f929f18602317f6077c9cef025ca753b9"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"dd92cdfd398b84ce3b254e4f934553e897e1e5db","unresolved":false,"context_lines":[{"line_number":208,"context_line":"    storage: 2Gi"},{"line_number":209,"context_line":"  storage_class: general"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"mountlocation:"},{"line_number":212,"context_line":"  tempest_data: /var/lib/tempest/data"},{"line_number":213,"context_line":"  test_run_script: /tmp/run-tests.sh"},{"line_number":214,"context_line":"  test_run_script_name: run-tests.sh"},{"line_number":215,"context_line":"  tempest_conf_location: /etc/tempest/tempest.conf"},{"line_number":216,"context_line":"  tempest_conf_folder_location: /etc/tempest"},{"line_number":217,"context_line":"  tempest_conf_name: tempest.conf"},{"line_number":218,"context_line":"  tempest_blacklist_folder_location: /etc/tempest/test-blacklist"},{"line_number":219,"context_line":"  tempest_blacklist_folder: test-blacklist"},{"line_number":220,"context_line":"  tempest_whitelist_folder_location: /etc/tempest/test-whitelist"},{"line_number":221,"context_line":"  tempest_whitelist_folder: test-whitelist"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"secrets:"},{"line_number":224,"context_line":"  identity:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_5eeed722","line":221,"range":{"start_line":211,"start_character":0,"end_line":221,"end_character":42},"updated":"2018-08-07 13:47:21.000000000","message":"This needs to be cleaned up, and preferably moved under the pod.mounts key. We shouldn\u0027t create an arbitrary top level key for defining these.","commit_id":"9385066f929f18602317f6077c9cef025ca753b9"}]}