)]}'
{"tempest/config.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"2198a75c5ac4166a9ce44d7d2a5da3aa254d8e84","unresolved":false,"context_lines":[{"line_number":419,"context_line":"                     \u0027with multiple ports on the same network? This is only \u0027"},{"line_number":420,"context_line":"                     \u0027valid when using Neutron.\u0027),"},{"line_number":421,"context_line":"    cfg.BoolOpt(\u0027config_drive\u0027,"},{"line_number":422,"context_line":"                default\u003dFalse,"},{"line_number":423,"context_line":"                help\u003d\u0027Enable special configuration drive with metadata.\u0027),"},{"line_number":424,"context_line":"]"},{"line_number":425,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"ba15a1d1_c175f463","line":422,"updated":"2015-09-18 09:44:38.000000000","message":"It should default to True. Otherwise the new part of the test will never be executed in the Gate, which we don\"t want.","commit_id":"2f249ae56c5e4ab1c8db7bec9e376174ae4718f0"}],"tempest/scenario/test_server_basic_ops.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"531f9346c49efc47a4d302d42e0bbb4c378a94d8","unresolved":false,"context_lines":[{"line_number":126,"context_line":"                                                  \u0027%s is empty.\u0027 % md_url)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    def verify_metadata_on_config_drive(self):"},{"line_number":129,"context_line":"        if self.run_ssh and CONF.compute_feature_enabled.metadata_service:"},{"line_number":130,"context_line":"            # Verify metadata service on config_drive"},{"line_number":131,"context_line":"            cmd_blkid \u003d \u0027blkid -t LABEL\u003dconfig-2 -o device\u0027"},{"line_number":132,"context_line":"            dev_name \u003d self.ssh_client.exec_command(cmd_blkid)"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba15a1d1_7b79f492","line":129,"updated":"2015-09-14 15:44:22.000000000","message":"metadata_service is used to test whether the metadata service  (accessed through 169.254.169.254) is available. Config drive is something else. You should introduce a new config flag or at least not used CONF.compute_feature_enabled.metadata_service","commit_id":"52dd31b5b73118caa7d93d536136a4d947d2df62"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"531f9346c49efc47a4d302d42e0bbb4c378a94d8","unresolved":false,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    def verify_metadata_on_config_drive(self):"},{"line_number":129,"context_line":"        if self.run_ssh and CONF.compute_feature_enabled.metadata_service:"},{"line_number":130,"context_line":"            # Verify metadata service on config_drive"},{"line_number":131,"context_line":"            cmd_blkid \u003d \u0027blkid -t LABEL\u003dconfig-2 -o device\u0027"},{"line_number":132,"context_line":"            dev_name \u003d self.ssh_client.exec_command(cmd_blkid)"},{"line_number":133,"context_line":"            dev_name \u003d dev_name.rstrip()"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba15a1d1_9b908897","line":130,"updated":"2015-09-14 15:44:22.000000000","message":"just remove the \u0027service\u0027 word here. In OpenStack, service is (wrongly ?) assumed to be an HTTP service.","commit_id":"52dd31b5b73118caa7d93d536136a4d947d2df62"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"531f9346c49efc47a4d302d42e0bbb4c378a94d8","unresolved":false,"context_lines":[{"line_number":136,"context_line":"            result \u003d self.ssh_client.exec_command(cmd_md)"},{"line_number":137,"context_line":"            self.ssh_client.exec_command(\u0027sudo umount /mnt\u0027)"},{"line_number":138,"context_line":"            result \u003d json.loads(result)"},{"line_number":139,"context_line":"            self.assertIn(\u0027meta\u0027, result.keys())"},{"line_number":140,"context_line":"            msg \u003d (\u0027Failed while verifying metadata on config_drive on server.\u0027"},{"line_number":141,"context_line":"                   \u0027 Result of command \"%s\" is NOT \"%s\"\u0027 % (cmd_md, self.meta))"},{"line_number":142,"context_line":"            self.assertEqual(self.meta, result[\u0027meta\u0027], msg)"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba15a1d1_bb69cc8d","line":139,"range":{"start_line":139,"start_character":29,"end_line":139,"end_character":38},"updated":"2015-09-14 15:44:22.000000000","message":"self.assertIn(\u0027meta\u0027, result)\n\nis enough. no need for the \u0027keys()\u0027 call.","commit_id":"52dd31b5b73118caa7d93d536136a4d947d2df62"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"e16c762bdb651a1d98ce7b6195f63d09c5331a99","unresolved":false,"context_lines":[{"line_number":150,"context_line":"        self.boot_instance()"},{"line_number":151,"context_line":"        self.verify_ssh()"},{"line_number":152,"context_line":"        self.verify_metadata()"},{"line_number":153,"context_line":"        self.verify_metadata_on_config_drive()"},{"line_number":154,"context_line":"        self.servers_client.delete_server(self.instance[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":6,"id":"ba15a1d1_77c3eaed","line":153,"updated":"2015-09-16 12:14:05.000000000","message":"We should have a config switch to deactivate config_drive. Not all the hypervisors support it.","commit_id":"7c82b83912935f455b209f08210c087769a9a82e"}]}