)]}'
{"tempest/api/orchestration/base.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"bf19646ee17d1d8221b123bd8108f06f66c89269","unresolved":false,"context_lines":[{"line_number":104,"context_line":"                pass"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    @classmethod"},{"line_number":107,"context_line":"    def load_template(cls, name, ext\u003d\u0027yaml\u0027):"},{"line_number":108,"context_line":"        fullpath \u003d os.path.join((\"tempest\", \"api\", \"orchestration\","},{"line_number":109,"context_line":"                                 \"stacks\", \"templates\", \"%s.%s\" % (name, ext)))"},{"line_number":110,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2BkHA%3D","line":107,"updated":"2014-03-25 21:21:19.000000000","message":"I think this should probably be unit tested, despite how simple it is, it\u0027s a common mechanism we\u0027ll be using throughout the heat tests.","commit_id":"714354da0bdddcaae636e6e08b4dd1b0581297aa"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"bf19646ee17d1d8221b123bd8108f06f66c89269","unresolved":false,"context_lines":[{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        with open(fullpath, \"r\") as f:"},{"line_number":112,"context_line":"            if ext \u003d\u003d \u0027yaml\u0027:"},{"line_number":113,"context_line":"                return yaml.load(f)"},{"line_number":114,"context_line":"            elif ext \u003d\u003d \u0027json\u0027:"},{"line_number":115,"context_line":"                return json.loads(f.read())"},{"line_number":116,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAWH%2F%2BkFI%3D","line":113,"updated":"2014-03-25 21:21:19.000000000","message":"So doing the load here is different than passing the string to the client.","commit_id":"714354da0bdddcaae636e6e08b4dd1b0581297aa"}],"tempest/api/orchestration/stacks/templates/non_empty_stack.yaml":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"7564d1b862b61a76a8be8389b2e824dfc95545ac","unresolved":false,"context_lines":[{"line_number":29,"context_line":"      Fn::Replace:"},{"line_number":30,"context_line":"      - nick: {Ref: fluffy}"},{"line_number":31,"context_line":"      - |"},{"line_number":32,"context_line":"        #nick"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"AAAAWH%2F%2BeYo%3D","line":32,"updated":"2014-03-26 15:09:27.000000000","message":"Is this comment needed? or does it have special significance in the template format that I don\u0027t know about?","commit_id":"e0a65d1436590432fd61b3c09ec86dffd6b0239e"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"eca22492a32acf074b4b7fb8cd95d4c4f5e1b511","unresolved":false,"context_lines":[{"line_number":29,"context_line":"      Fn::Replace:"},{"line_number":30,"context_line":"      - nick: {Ref: fluffy}"},{"line_number":31,"context_line":"      - |"},{"line_number":32,"context_line":"        #nick"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"AAAAWH%2F%2BeUg%3D","line":32,"in_reply_to":"AAAAWH%2F%2BeYo%3D","updated":"2014-03-26 15:11:56.000000000","message":"it\u0027s not actually a comment because it\u0027s in a text block","commit_id":"e0a65d1436590432fd61b3c09ec86dffd6b0239e"}],"tempest/api/orchestration/stacks/test_non_empty_stack.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"93aa6aef0b3759d4424cfd0e80f18aaf847ae2f9","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    def setUpClass(cls):"},{"line_number":28,"context_line":"        super(StacksTestJSON, cls).setUpClass()"},{"line_number":29,"context_line":"        cls.client \u003d cls.orchestration_client"},{"line_number":30,"context_line":"        stack_name \u003d data_utils.rand_name(\u0027heat\u0027)"},{"line_number":31,"context_line":"        template \u003d cls.load_template(\u0027non_empty_stack\u0027)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        # create the stack"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWH%2F%2BkrA%3D","line":30,"updated":"2014-03-25 20:05:44.000000000","message":"I think the stack name is referenced below.","commit_id":"a3dbd163e59ba2c1d96f6581914e6cb1823b6df3"}],"tempest/api/orchestration/stacks/test_templates.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"7564d1b862b61a76a8be8389b2e824dfc95545ac","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"class TemplateYAMLTestJSON(base.BaseOrchestrationTest):"},{"line_number":19,"context_line":"    template \u003d \"\"\""},{"line_number":20,"context_line":"HeatTemplateFormatVersion: \u00272012-12-12\u0027"},{"line_number":21,"context_line":"Description: |"},{"line_number":22,"context_line":"  Template which creates only a new user"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F%2BeW0%3D","line":19,"updated":"2014-03-26 15:09:27.000000000","message":"Why not put this in a file too? I get that it\u0027s simple but I think that we should be explicit in either doing it inline or in a separate file.","commit_id":"e0a65d1436590432fd61b3c09ec86dffd6b0239e"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"eca22492a32acf074b4b7fb8cd95d4c4f5e1b511","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"class TemplateYAMLTestJSON(base.BaseOrchestrationTest):"},{"line_number":19,"context_line":"    template \u003d \"\"\""},{"line_number":20,"context_line":"HeatTemplateFormatVersion: \u00272012-12-12\u0027"},{"line_number":21,"context_line":"Description: |"},{"line_number":22,"context_line":"  Template which creates only a new user"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F%2BeTg%3D","line":19,"in_reply_to":"AAAAWH%2F%2BeW0%3D","updated":"2014-03-26 15:11:56.000000000","message":"mostly because it was really small. So it seemed fine to be reviewable this way.","commit_id":"e0a65d1436590432fd61b3c09ec86dffd6b0239e"}],"tempest/api/orchestration/stacks/test_templates_negative.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"7564d1b862b61a76a8be8389b2e824dfc95545ac","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class TemplateYAMLNegativeTestJSON(base.BaseOrchestrationTest):"},{"line_number":21,"context_line":"    template \u003d \"\"\""},{"line_number":22,"context_line":"HeatTemplateFormatVersion: \u00272012-12-12\u0027"},{"line_number":23,"context_line":"Description: |"},{"line_number":24,"context_line":"  Template which creates only a new user"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAWH%2F%2BeWU%3D","line":21,"updated":"2014-03-26 15:09:27.000000000","message":"Although maybe not for negative tests, that does seem like a different case","commit_id":"e0a65d1436590432fd61b3c09ec86dffd6b0239e"}]}