)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"280585daa2f6e551b3956362f31a589b4ac9b2be","unresolved":false,"context_lines":[{"line_number":10,"context_line":"controlled from the global timeout set in tempest.conf"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"We should ensure future tests follow this convention where Timeout values"},{"line_number":13,"context_line":"are specified in the template."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ie4a9048771e3a106e1d9c6b69f244461f6d5fd8d"},{"line_number":16,"context_line":"Partial-Bug: #1297560"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAXH%2F%2F5zw%3D","line":13,"updated":"2014-04-16 15:36:41.000000000","message":"Could we add a hacking check or some other static analysis to enforce this, maybe a unit test that loads all the templates. Without an automated check to ensure this convention things will slip through.","commit_id":"0d00dd8be166c99097d7eb67200eb2a4ab4dc9a2"},{"author":{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},"change_message_id":"8965e4a915a7f1d4d18efa4fcaefc68c46c2269f","unresolved":false,"context_lines":[{"line_number":10,"context_line":"controlled from the global timeout set in tempest.conf"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"We should ensure future tests follow this convention where Timeout values"},{"line_number":13,"context_line":"are specified in the template."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ie4a9048771e3a106e1d9c6b69f244461f6d5fd8d"},{"line_number":16,"context_line":"Partial-Bug: #1297560"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAXH%2F%2F5h4%3D","line":13,"in_reply_to":"AAAAXH%2F%2F5zw%3D","updated":"2014-04-16 16:03:08.000000000","message":"That sounds like it would be quite easy, but I\u0027m not sure where such a test should live?","commit_id":"0d00dd8be166c99097d7eb67200eb2a4ab4dc9a2"}],"tempest/api/orchestration/stacks/templates/neutron_basic.yaml":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c9cd6c70761b594c9345b64aa187da1b713c51fd","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    depends_on: Server"},{"line_number":68,"context_line":"    properties:"},{"line_number":69,"context_line":"      Handle: {get_resource: WaitHandleNeutron}"},{"line_number":70,"context_line":"      Timeout: {get_param: timeout}"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"AAAAXH%2F%2F590%3D","line":70,"updated":"2014-04-16 15:25:40.000000000","message":"This is \"Timeout: {Ref: timeout}\" in the other yaml file, why is it different here?","commit_id":"0d00dd8be166c99097d7eb67200eb2a4ab4dc9a2"},{"author":{"_account_id":4328,"name":"Steven Hardy","email":"steven.hardy@suse.com","username":"shardy"},"change_message_id":"8965e4a915a7f1d4d18efa4fcaefc68c46c2269f","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    depends_on: Server"},{"line_number":68,"context_line":"    properties:"},{"line_number":69,"context_line":"      Handle: {get_resource: WaitHandleNeutron}"},{"line_number":70,"context_line":"      Timeout: {get_param: timeout}"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"AAAAXH%2F%2F5g4%3D","line":70,"in_reply_to":"AAAAXH%2F%2F590%3D","updated":"2014-04-16 16:03:08.000000000","message":"Because this is a HOT syntax template, and the other one is (yaml rendered) CloudFormation syntax, which uses Ref instead of get_param.\n\nYou can tell the difference by the version string:\nheat_template_version: \u00272013-05-23\u0027 is HOT\n\nHeatTemplateFormatVersion: \u00272012-12-12\u0027 is YAML rendered CFN","commit_id":"0d00dd8be166c99097d7eb67200eb2a4ab4dc9a2"}]}