)]}'
{"etc/tempest.conf.sample":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"d2528ed67f4a34cd16875f5dca6008f8e6addba8","unresolved":false,"context_lines":[{"line_number":238,"context_line":"public_router_id \u003d {$PUBLIC_ROUTER_ID}"},{"line_number":239,"context_line":""},{"line_number":240,"context_line":"# Wheter LBaaS should be used."},{"line_number":241,"context_line":"use_lbaas \u003d True"},{"line_number":242,"context_line":""},{"line_number":243,"context_line":"[volume]"},{"line_number":244,"context_line":"# This section contains the configuration options used when executing tests"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAATH%2F%2FIF4%3D","line":241,"updated":"2013-10-22 19:48:32.000000000","message":"if we want to turn this on in the gate, we probably want to do some config via devstack instead here based on whether use_lbaas is turned on","commit_id":"d9a91ca215e76637eb9e28f7ace22456989a77df"}],"tempest/cli/simple_read_only/test_neutron.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"79875c990aef410c8290e181a4d27be2b359a06b","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    def test_neutron_floatingip_list(self):"},{"line_number":68,"context_line":"        self.neutron(\u0027floatingip-list\u0027)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def test_neutron_lb_healthmonitor_list(self):"},{"line_number":71,"context_line":"        self.neutron(\u0027lb-healthmonitor-list\u0027)"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    def test_neutron_lb_member_list(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATH%2F%2Fwz4%3D","line":70,"updated":"2013-10-10 07:04:41.000000000","message":"I think if we don\u0027t use lbaas, these *-list commands are failed.\nSo could you add config and @testtools.skip_if() to these test cases?","commit_id":"56c78eebc7bf75b45d0982053f8953f53ba486d9"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"50bba910e9c1b4e3e65ee09eb2fa110de2f78f5c","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    def test_neutron_security_group_rule_list(self):"},{"line_number":104,"context_line":"        self.neutron(\u0027security-group-rule-list\u0027)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    def test_neutron_service_provider_list(self):"},{"line_number":107,"context_line":"        self.neutron(\u0027service-provider-list\u0027)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    def test_neutron_subnet_list(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FtJA%3D","line":106,"updated":"2013-10-10 22:52:30.000000000","message":"If we have no service-provider such as LOADBALANCER, this command fails.\nI\u0027m not sure whether this is neutron client\u0027s bug or expected behavior.\n\n $ neutron service-provider-list\n 404 Not Found\n \n The resource could not be found.\n \n $ echo $?\n 1","commit_id":"7c5f3ec8b88577a9fa85ab98bf0a74199600cddd"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"b541def30062e35c7c70eeda155d6af35e277957","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    def test_neutron_security_group_rule_list(self):"},{"line_number":104,"context_line":"        self.neutron(\u0027security-group-rule-list\u0027)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    def test_neutron_service_provider_list(self):"},{"line_number":107,"context_line":"        self.neutron(\u0027service-provider-list\u0027)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    def test_neutron_subnet_list(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2Focg%3D","line":106,"in_reply_to":"AAAATH%2F%2Fo5c%3D","updated":"2013-10-12 03:16:25.000000000","message":"I\u0027ve read the bug report and answer. I saw \u0027that is behavior by design.\u0027\nSo, this is not a bug and will not be fixed.\n\nI also think we should remove this for now.","commit_id":"7c5f3ec8b88577a9fa85ab98bf0a74199600cddd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"bfa6a73a2de5e56326c743ec226f31475fbb31b8","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    def test_neutron_security_group_rule_list(self):"},{"line_number":104,"context_line":"        self.neutron(\u0027security-group-rule-list\u0027)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"    def test_neutron_service_provider_list(self):"},{"line_number":107,"context_line":"        self.neutron(\u0027service-provider-list\u0027)"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"    def test_neutron_subnet_list(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2Fo5c%3D","line":106,"in_reply_to":"AAAATH%2F%2FtJA%3D","updated":"2013-10-11 22:02:24.000000000","message":"I filed a bug report : https://bugs.launchpad.net/neutron/+bug/1238984\n\nI think we may want to remove test_neutron_service_provider for now, push the patch, and re-add test_neutron_service_provider once the bug has been fixed. WDYT ?","commit_id":"7c5f3ec8b88577a9fa85ab98bf0a74199600cddd"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"031c46130cfc2759cf9d98fbafecfd3445a3b373","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        self.neutron(\u0027meter-label-rule-list\u0027)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    @testtools.skipIf(not CONF.network.use_lbaas, \u0027LBaaS disabled\u0027)"},{"line_number":78,"context_line":"    def test_neutron_lb_healthmonitor_list(self):"},{"line_number":79,"context_line":"        self.neutron(\u0027lb-healthmonitor-list\u0027)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    @testtools.skipIf(not CONF.network.use_lbaas, \u0027LBaaS disabled\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATX%2F%2F%2BkE%3D","line":78,"updated":"2013-10-23 12:12:45.000000000","message":"As this test just makes sure the lb commands don\u0027t fail, can we make it pass if neutron client says LBaaS is disabled and not require any extra configuration?","commit_id":"d9a91ca215e76637eb9e28f7ace22456989a77df"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"2ca3dcbcee2dd434f3934f70a91adcba311b3019","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        self.neutron(\u0027meter-label-rule-list\u0027)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    @testtools.skipIf(not CONF.network.use_lbaas, \u0027LBaaS disabled\u0027)"},{"line_number":78,"context_line":"    def test_neutron_lb_healthmonitor_list(self):"},{"line_number":79,"context_line":"        self.neutron(\u0027lb-healthmonitor-list\u0027)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    @testtools.skipIf(not CONF.network.use_lbaas, \u0027LBaaS disabled\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2Fu8A%3D","line":78,"in_reply_to":"AAAATX%2F%2F%2BkE%3D","updated":"2013-10-29 18:39:16.000000000","message":"Seems a bit complicated to me: the command would return 1 and we\u0027d have to parse the message printed to stderr to check whether it\u0027s a 404 or not... I\u0027d rather keep it that way: it\u0027s simple and still makes sense.","commit_id":"d9a91ca215e76637eb9e28f7ace22456989a77df"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"af85de5aac768285ee2e1f3c69698de0c98ba610","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        self.neutron(\u0027meter-label-rule-list\u0027)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    @testtools.skipIf(not CONF.network.use_lbaas, \u0027LBaaS disabled\u0027)"},{"line_number":78,"context_line":"    def test_neutron_lb_healthmonitor_list(self):"},{"line_number":79,"context_line":"        self.neutron(\u0027lb-healthmonitor-list\u0027)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    @testtools.skipIf(not CONF.network.use_lbaas, \u0027LBaaS disabled\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2Fq7g%3D","line":78,"in_reply_to":"AAAATn%2F%2FscA%3D","updated":"2013-10-30 16:04:55.000000000","message":"OK, will do.","commit_id":"d9a91ca215e76637eb9e28f7ace22456989a77df"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"584fa0c612af3c6bdec84c99439ae8dd7d3c42ab","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        self.neutron(\u0027meter-label-rule-list\u0027)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    @testtools.skipIf(not CONF.network.use_lbaas, \u0027LBaaS disabled\u0027)"},{"line_number":78,"context_line":"    def test_neutron_lb_healthmonitor_list(self):"},{"line_number":79,"context_line":"        self.neutron(\u0027lb-healthmonitor-list\u0027)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    @testtools.skipIf(not CONF.network.use_lbaas, \u0027LBaaS disabled\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2FscA%3D","line":78,"in_reply_to":"AAAATn%2F%2Fu8A%3D","updated":"2013-10-30 10:44:17.000000000","message":"At the very least we want a separate test to make sure this command returns a 404 when LBaaS is disabled.  As we want to make sure this command doesn\u0027t cause neutronclient to break when there is no LBaaS","commit_id":"d9a91ca215e76637eb9e28f7ace22456989a77df"}],"tempest/config.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"50bba910e9c1b4e3e65ee09eb2fa110de2f78f5c","unresolved":false,"context_lines":[{"line_number":295,"context_line":"               default\u003d\"\","},{"line_number":296,"context_line":"               help\u003d\"Id of the public router that provides external \""},{"line_number":297,"context_line":"                    \"connectivity\"),"},{"line_number":298,"context_line":"    cfg.StrOpt(\u0027use_lbaas\u0027,"},{"line_number":299,"context_line":"               default\u003dTrue,"},{"line_number":300,"context_line":"               help\u003d\"Whether LbaaS should enabled\"),"},{"line_number":301,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FtKk%3D","line":298,"updated":"2013-10-10 22:52:30.000000000","message":"Could you please add this option to etc/tempest.conf.sample ?","commit_id":"7c5f3ec8b88577a9fa85ab98bf0a74199600cddd"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"bfa6a73a2de5e56326c743ec226f31475fbb31b8","unresolved":false,"context_lines":[{"line_number":295,"context_line":"               default\u003d\"\","},{"line_number":296,"context_line":"               help\u003d\"Id of the public router that provides external \""},{"line_number":297,"context_line":"                    \"connectivity\"),"},{"line_number":298,"context_line":"    cfg.StrOpt(\u0027use_lbaas\u0027,"},{"line_number":299,"context_line":"               default\u003dTrue,"},{"line_number":300,"context_line":"               help\u003d\"Whether LbaaS should enabled\"),"},{"line_number":301,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2Fo7M%3D","line":298,"in_reply_to":"AAAATH%2F%2FtKk%3D","updated":"2013-10-11 22:02:24.000000000","message":"I\u0027ll do that.","commit_id":"7c5f3ec8b88577a9fa85ab98bf0a74199600cddd"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"d2528ed67f4a34cd16875f5dca6008f8e6addba8","unresolved":false,"context_lines":[{"line_number":296,"context_line":"               help\u003d\"Id of the public router that provides external \""},{"line_number":297,"context_line":"                    \"connectivity\"),"},{"line_number":298,"context_line":"    cfg.StrOpt(\u0027use_lbaas\u0027,"},{"line_number":299,"context_line":"               default\u003dTrue,"},{"line_number":300,"context_line":"               help\u003d\"Whether LBaaS should enabled\"),"},{"line_number":301,"context_line":"]"},{"line_number":302,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2FIFw%3D","line":299,"updated":"2013-10-22 19:48:32.000000000","message":"do we really want to default true? is that really the case for most environments?","commit_id":"d9a91ca215e76637eb9e28f7ace22456989a77df"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"2ca3dcbcee2dd434f3934f70a91adcba311b3019","unresolved":false,"context_lines":[{"line_number":296,"context_line":"               help\u003d\"Id of the public router that provides external \""},{"line_number":297,"context_line":"                    \"connectivity\"),"},{"line_number":298,"context_line":"    cfg.StrOpt(\u0027use_lbaas\u0027,"},{"line_number":299,"context_line":"               default\u003dTrue,"},{"line_number":300,"context_line":"               help\u003d\"Whether LBaaS should enabled\"),"},{"line_number":301,"context_line":"]"},{"line_number":302,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2Fz7M%3D","line":299,"in_reply_to":"AAAATH%2F%2FIFw%3D","updated":"2013-10-29 18:39:16.000000000","message":"Probably not, but I wanted to test as many things as possible. Thinking about it, I\u0027m not sure it\u0027d be such a great idea to enable LBaaS in devstack just for these tests, so wa may want to default to False.","commit_id":"d9a91ca215e76637eb9e28f7ace22456989a77df"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"af85de5aac768285ee2e1f3c69698de0c98ba610","unresolved":false,"context_lines":[{"line_number":296,"context_line":"               help\u003d\"Id of the public router that provides external \""},{"line_number":297,"context_line":"                    \"connectivity\"),"},{"line_number":298,"context_line":"    cfg.StrOpt(\u0027use_lbaas\u0027,"},{"line_number":299,"context_line":"               default\u003dTrue,"},{"line_number":300,"context_line":"               help\u003d\"Whether LBaaS should enabled\"),"},{"line_number":301,"context_line":"]"},{"line_number":302,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2Fq68%3D","line":299,"in_reply_to":"AAAATn%2F%2Fsb4%3D","updated":"2013-10-30 16:04:55.000000000","message":"There seem to be some tests in tempest/api/network/test_load_balancer.py but I\u0027m not sure how they are activated.","commit_id":"d9a91ca215e76637eb9e28f7ace22456989a77df"},{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"584fa0c612af3c6bdec84c99439ae8dd7d3c42ab","unresolved":false,"context_lines":[{"line_number":296,"context_line":"               help\u003d\"Id of the public router that provides external \""},{"line_number":297,"context_line":"                    \"connectivity\"),"},{"line_number":298,"context_line":"    cfg.StrOpt(\u0027use_lbaas\u0027,"},{"line_number":299,"context_line":"               default\u003dTrue,"},{"line_number":300,"context_line":"               help\u003d\"Whether LBaaS should enabled\"),"},{"line_number":301,"context_line":"]"},{"line_number":302,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2Fsb4%3D","line":299,"in_reply_to":"AAAATn%2F%2Fz7M%3D","updated":"2013-10-30 10:44:17.000000000","message":"So we don\u0027t test LBaaS API in tempest today?","commit_id":"d9a91ca215e76637eb9e28f7ace22456989a77df"}]}