)]}'
{"contrib/devstack/extras.d/70-cue.sh":[{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"f6434fcd818460d7ef9a90a93facfa4a2608389b","unresolved":false,"context_lines":[{"line_number":16,"context_line":"        #echo_summary \"Installing Cue Dashboard\""},{"line_number":17,"context_line":"        # TODO Steve. Disabling the installation of dashboard"},{"line_number":18,"context_line":"        # until the cue gate is fixed."},{"line_number":19,"context_line":"        #install_cuedashboard"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    elif [[ \"$1\" \u003d\u003d \"stack\" \u0026\u0026 \"$2\" \u003d\u003d \"post-config\" ]]; then"},{"line_number":22,"context_line":"        echo_summary \"Configuring Cue\""}],"source_content_type":"text/x-sh","patch_set":20,"id":"7a016987_307b41c0","line":19,"updated":"2015-05-29 16:15:33.000000000","message":"why is this still disabled?","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"21e69b46b19391afc48119be5135cd4af9c195db","unresolved":false,"context_lines":[{"line_number":16,"context_line":"        #echo_summary \"Installing Cue Dashboard\""},{"line_number":17,"context_line":"        # TODO Steve. Disabling the installation of dashboard"},{"line_number":18,"context_line":"        # until the cue gate is fixed."},{"line_number":19,"context_line":"        #install_cuedashboard"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    elif [[ \"$1\" \u003d\u003d \"stack\" \u0026\u0026 \"$2\" \u003d\u003d \"post-config\" ]]; then"},{"line_number":22,"context_line":"        echo_summary \"Configuring Cue\""}],"source_content_type":"text/x-sh","patch_set":20,"id":"7a016987_4b5f2e0f","line":19,"in_reply_to":"7a016987_307b41c0","updated":"2015-05-29 16:21:54.000000000","message":"There are issues with cloning the repo. I have submitted https://review.openstack.org/#/c/186296/ to see if that fixes it. But i would like to get the integration test running and then figure out this cloning issue.","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"23e8ff3959521f4464e0f65b8321988b11ac2916","unresolved":false,"context_lines":[{"line_number":16,"context_line":"        #echo_summary \"Installing Cue Dashboard\""},{"line_number":17,"context_line":"        # TODO Steve. Disabling the installation of dashboard"},{"line_number":18,"context_line":"        # until the cue gate is fixed."},{"line_number":19,"context_line":"        #install_cuedashboard"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    elif [[ \"$1\" \u003d\u003d \"stack\" \u0026\u0026 \"$2\" \u003d\u003d \"post-config\" ]]; then"},{"line_number":22,"context_line":"        echo_summary \"Configuring Cue\""}],"source_content_type":"text/x-sh","patch_set":20,"id":"7a016987_a6d257cf","line":19,"in_reply_to":"7a016987_4b5f2e0f","updated":"2015-05-29 16:55:23.000000000","message":"this effectively disables cuedashboard in devstack, is horizon installed in this environment?  this should actually be run only if horizon is enabled","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"923b6c27011255d58d7d07bb44cc75c5f54847d6","unresolved":false,"context_lines":[{"line_number":16,"context_line":"        #echo_summary \"Installing Cue Dashboard\""},{"line_number":17,"context_line":"        # TODO Steve. Disabling the installation of dashboard"},{"line_number":18,"context_line":"        # until the cue gate is fixed."},{"line_number":19,"context_line":"        #install_cuedashboard"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    elif [[ \"$1\" \u003d\u003d \"stack\" \u0026\u0026 \"$2\" \u003d\u003d \"post-config\" ]]; then"},{"line_number":22,"context_line":"        echo_summary \"Configuring Cue\""}],"source_content_type":"text/x-sh","patch_set":20,"id":"7a016987_6838ada0","line":19,"in_reply_to":"7a016987_a6d257cf","updated":"2015-05-29 17:04:08.000000000","message":"We do have this check. The install happens if horizon is installed.\nHowever, horizon is part of a core devstack project that is always installed in devstack node pool","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"}],"requirements.txt":[{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"f6434fcd818460d7ef9a90a93facfa4a2608389b","unresolved":false,"context_lines":[{"line_number":27,"context_line":"posix_ipc"},{"line_number":28,"context_line":"jinja2"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"taskflow\u003e\u003d0.7.1"},{"line_number":31,"context_line":"-e git+https://github.com/python-zk/kazoo.git#egg\u003dkazoo"},{"line_number":32,"context_line":"#PyMySQL"},{"line_number":33,"context_line":"python-keystoneclient"}],"source_content_type":"text/plain","patch_set":20,"id":"7a016987_d0406d61","line":30,"updated":"2015-05-29 16:15:33.000000000","message":"this should be \u003e\u003d0.10.1","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"d4bea946ad5b515ff1c7052dd3e945e5ac9db2d4","unresolved":false,"context_lines":[{"line_number":27,"context_line":"posix_ipc"},{"line_number":28,"context_line":"jinja2"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"taskflow\u003e\u003d0.7.1"},{"line_number":31,"context_line":"-e git+https://github.com/python-zk/kazoo.git#egg\u003dkazoo"},{"line_number":32,"context_line":"#PyMySQL"},{"line_number":33,"context_line":"python-keystoneclient"}],"source_content_type":"text/plain","patch_set":20,"id":"7a016987_e6796f94","line":30,"in_reply_to":"7a016987_abf462f0","updated":"2015-05-29 16:31:21.000000000","message":"\u003e\u003d0.10.1 is still greater than 0.7.1. So that should satisfied this requirement.","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"923b6c27011255d58d7d07bb44cc75c5f54847d6","unresolved":false,"context_lines":[{"line_number":27,"context_line":"posix_ipc"},{"line_number":28,"context_line":"jinja2"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"taskflow\u003e\u003d0.7.1"},{"line_number":31,"context_line":"-e git+https://github.com/python-zk/kazoo.git#egg\u003dkazoo"},{"line_number":32,"context_line":"#PyMySQL"},{"line_number":33,"context_line":"python-keystoneclient"}],"source_content_type":"text/plain","patch_set":20,"id":"7a016987_2806351c","line":30,"in_reply_to":"7a016987_c69b1316","updated":"2015-05-29 17:04:08.000000000","message":"Ok ill make the change.","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"21e69b46b19391afc48119be5135cd4af9c195db","unresolved":false,"context_lines":[{"line_number":27,"context_line":"posix_ipc"},{"line_number":28,"context_line":"jinja2"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"taskflow\u003e\u003d0.7.1"},{"line_number":31,"context_line":"-e git+https://github.com/python-zk/kazoo.git#egg\u003dkazoo"},{"line_number":32,"context_line":"#PyMySQL"},{"line_number":33,"context_line":"python-keystoneclient"}],"source_content_type":"text/plain","patch_set":20,"id":"7a016987_abf462f0","line":30,"in_reply_to":"7a016987_d0406d61","updated":"2015-05-29 16:21:54.000000000","message":"The version in openstack/global-requirement is 0.7.1 though. Wouldnt it complain?\n\nhttps://github.com/openstack/requirements/blob/master/global-requirements.txt#L174","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"23e8ff3959521f4464e0f65b8321988b11ac2916","unresolved":false,"context_lines":[{"line_number":27,"context_line":"posix_ipc"},{"line_number":28,"context_line":"jinja2"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"taskflow\u003e\u003d0.7.1"},{"line_number":31,"context_line":"-e git+https://github.com/python-zk/kazoo.git#egg\u003dkazoo"},{"line_number":32,"context_line":"#PyMySQL"},{"line_number":33,"context_line":"python-keystoneclient"}],"source_content_type":"text/plain","patch_set":20,"id":"7a016987_c69b1316","line":30,"in_reply_to":"7a016987_e6796f94","updated":"2015-05-29 16:55:23.000000000","message":"our software won\u0027t work with \u003c 0.8, \u003e\u003d0.10.1 is still \u003e\u003d0.7.1 so I don\u0027t see why global requirements would complain","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"}],"tests/integration/api/v1/clusters/test_clusters.py":[{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"2aedc8eb1802428656fd17f20bf49d264cb8de57","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        size \u003d 3"},{"line_number":55,"context_line":"        return self.client.create_cluster(name, size, flavor, network_id)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def test_create_cluster(self):"},{"line_number":58,"context_line":"        \"\"\"Test create cluster and verifies cluster goes to ACTIVE state.\"\"\""},{"line_number":59,"context_line":"        cluster \u003d self._create_cluster()"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"7a016987_ab42220d","line":57,"updated":"2015-05-29 16:21:48.000000000","message":"these should be kept separated","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"f6434fcd818460d7ef9a90a93facfa4a2608389b","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        size \u003d 3"},{"line_number":55,"context_line":"        return self.client.create_cluster(name, size, flavor, network_id)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def test_create_cluster(self):"},{"line_number":58,"context_line":"        \"\"\"Test create cluster and verifies cluster goes to ACTIVE state.\"\"\""},{"line_number":59,"context_line":"        cluster \u003d self._create_cluster()"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"7a016987_50559d1b","line":57,"updated":"2015-05-29 16:15:33.000000000","message":"why are these tests being collapsed to a single test?","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"23e8ff3959521f4464e0f65b8321988b11ac2916","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        size \u003d 3"},{"line_number":55,"context_line":"        return self.client.create_cluster(name, size, flavor, network_id)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def test_create_cluster(self):"},{"line_number":58,"context_line":"        \"\"\"Test create cluster and verifies cluster goes to ACTIVE state.\"\"\""},{"line_number":59,"context_line":"        cluster \u003d self._create_cluster()"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"7a016987_665c5f4b","line":57,"in_reply_to":"7a016987_46b6033f","updated":"2015-05-29 16:55:23.000000000","message":"Having the integration gate take a long time is another motivation for us to fix the lack of multiple cue-worker processes, I would rather than we kept the tests separate as they should be and worked on fixing the actual problem.  If we don\u0027t separate the tests the assertEqual failure messages should be more specific so that the failure provides a better hint on what portion of the test failed.","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"21e69b46b19391afc48119be5135cd4af9c195db","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        size \u003d 3"},{"line_number":55,"context_line":"        return self.client.create_cluster(name, size, flavor, network_id)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def test_create_cluster(self):"},{"line_number":58,"context_line":"        \"\"\"Test create cluster and verifies cluster goes to ACTIVE state.\"\"\""},{"line_number":59,"context_line":"        cluster \u003d self._create_cluster()"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"7a016987_eb313aa3","line":57,"in_reply_to":"7a016987_50559d1b","updated":"2015-05-29 16:21:54.000000000","message":"This is because devstack only spawns one cue-worker.. So it takes a lot longer if we have tests where each creates a cluster. Also, i have seen things go wacky. Moving forward, we should spawn more workers. But in the mean time and just for the sake of having a integration tests gate, i reworked this test.","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"923b6c27011255d58d7d07bb44cc75c5f54847d6","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        size \u003d 3"},{"line_number":55,"context_line":"        return self.client.create_cluster(name, size, flavor, network_id)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def test_create_cluster(self):"},{"line_number":58,"context_line":"        \"\"\"Test create cluster and verifies cluster goes to ACTIVE state.\"\"\""},{"line_number":59,"context_line":"        cluster \u003d self._create_cluster()"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"7a016987_e897fd8d","line":57,"in_reply_to":"7a016987_665c5f4b","updated":"2015-05-29 17:04:08.000000000","message":"The assert message will give you the line numbers of the error. So we will know when and where it failed.\n\nWhile i agree that we should keep the tests separate and not be mocking with them, i feel that having a working and running integration test gate is  far more important. For gating and making sure our code works and also for \u0027big tent\u0027.\n\nThis doesnt mean that all the future tests will be done this way. But at least we have something that we can use now.\n\nOnce we do the work to add more cue-worker, we can separate this tests. Or if it\u0027s a trivial change, which i dont think it is, we can do this now.","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"d4bea946ad5b515ff1c7052dd3e945e5ac9db2d4","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        size \u003d 3"},{"line_number":55,"context_line":"        return self.client.create_cluster(name, size, flavor, network_id)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def test_create_cluster(self):"},{"line_number":58,"context_line":"        \"\"\"Test create cluster and verifies cluster goes to ACTIVE state.\"\"\""},{"line_number":59,"context_line":"        cluster \u003d self._create_cluster()"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"7a016987_46b6033f","line":57,"in_reply_to":"7a016987_ab42220d","updated":"2015-05-29 16:31:21.000000000","message":"I agree but we should have more cue-workers before we can have each test creates its own cluster.","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"}],"tests/integration/common/config.py":[{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"f6434fcd818460d7ef9a90a93facfa4a2608389b","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    identity_group \u003d cfg.OptGroup(name\u003d\u0027identity\u0027)"},{"line_number":28,"context_line":"    identity_options \u003d ["},{"line_number":29,"context_line":"        cfg.StrOpt(\u0027uri\u0027, default\u003d\u0027http://localhost:5000/v2.0\u0027),"},{"line_number":30,"context_line":"        cfg.StrOpt(\u0027username\u0027, default\u003d\u0027demo\u0027),"},{"line_number":31,"context_line":"        cfg.StrOpt(\u0027password\u0027, default\u003d\u0027secretadmin\u0027),"},{"line_number":32,"context_line":"        cfg.StrOpt(\u0027tenant_name\u0027, default\u003d\u0027demo\u0027),"}],"source_content_type":"text/x-python","patch_set":20,"id":"7a016987_b03cd1e1","line":29,"updated":"2015-05-29 16:15:33.000000000","message":"why does this file have to match cue-integration.conf?  If it\u0027s being updated to match cue-integration.conf this should also be set to 127.0.0.1 rather than localhost","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"},{"author":{"_account_id":5390,"name":"Steve Leon","username":"steve-leon","inactive":true},"change_message_id":"21e69b46b19391afc48119be5135cd4af9c195db","unresolved":false,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    identity_group \u003d cfg.OptGroup(name\u003d\u0027identity\u0027)"},{"line_number":28,"context_line":"    identity_options \u003d ["},{"line_number":29,"context_line":"        cfg.StrOpt(\u0027uri\u0027, default\u003d\u0027http://localhost:5000/v2.0\u0027),"},{"line_number":30,"context_line":"        cfg.StrOpt(\u0027username\u0027, default\u003d\u0027demo\u0027),"},{"line_number":31,"context_line":"        cfg.StrOpt(\u0027password\u0027, default\u003d\u0027secretadmin\u0027),"},{"line_number":32,"context_line":"        cfg.StrOpt(\u0027tenant_name\u0027, default\u003d\u0027demo\u0027),"}],"source_content_type":"text/x-python","patch_set":20,"id":"7a016987_2bebd2dd","line":29,"in_reply_to":"7a016987_b03cd1e1","updated":"2015-05-29 16:21:54.000000000","message":"I removed cue-integration.conf. That was not intended to be added to the patch.","commit_id":"1ed8b26dc4e6d0063463e7f5d2be6154103c2e39"}]}