)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"dd8e99412ddd2c366da5bba55e00b84df33316dc","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ciprian Barbu \u003cciprian.barbu@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-12 13:06:41 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[Verify] Add flavor_ref_ram, flavor_ref_alt_ram and heat_instance_type_ram"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"With these new options it is possible to configure the RAM size of"},{"line_number":10,"context_line":"the two flavors used when running tempest and for the orchestration instance"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dab17558_82679ae9","line":7,"updated":"2016-05-12 11:47:37.000000000","message":"This line is too long. Pep8 complaints on it. It can be:\n\n\n[Verify] Add flavor related options to config\n\n\nor something like that.","commit_id":"263b68608db7a5e5f3d1a370e50e5f1ea0e39079"},{"author":{"_account_id":20947,"name":"Ciprian Barbu","email":"ciprian.barbu@gmail.com","username":"ciprian.barbu"},"change_message_id":"d50c472c30984b231f2ae2f5238149412188803b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ciprian Barbu \u003cciprian.barbu@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-12 13:06:41 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[Verify] Add flavor_ref_ram, flavor_ref_alt_ram and heat_instance_type_ram"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"With these new options it is possible to configure the RAM size of"},{"line_number":10,"context_line":"the two flavors used when running tempest and for the orchestration instance"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dab17558_22b4eecf","line":7,"in_reply_to":"dab17558_82679ae9","updated":"2016-05-12 13:35:27.000000000","message":"Ok, sorry about that, I will fix it.","commit_id":"263b68608db7a5e5f3d1a370e50e5f1ea0e39079"}],"etc/rally/rally.conf.sample":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"e7ddd0a84fd1cbfeec0cb78d89ffea09338a7f96","unresolved":false,"context_lines":[{"line_number":660,"context_line":"#name_regex \u003d ^.*(cirros|testvm).*$"},{"line_number":661,"context_line":""},{"line_number":662,"context_line":"# The primary flavor used for most of the tests (integer value)"},{"line_number":663,"context_line":"#flavor_ref_ram \u003d 64"},{"line_number":664,"context_line":""},{"line_number":665,"context_line":"# The secondary flavor used for tests that need two flavors, like tests that"},{"line_number":666,"context_line":"# perform a resize (integer value)"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"dab17558_7290514f","line":663,"updated":"2016-05-13 10:35:59.000000000","message":"This (and below) option is assumed to be used only by Tempest, but is placed to [image] section - this looks like a configuration mess. Tempest is just a third-party testing tool which can be launched by Rally. Maybe create separate section [tempest] ?","commit_id":"375cb652e6234eee5ad999d15910b36d2783b9db"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"4634422b6481e579fb524c28a0145de95122bbcd","unresolved":false,"context_lines":[{"line_number":660,"context_line":"#name_regex \u003d ^.*(cirros|testvm).*$"},{"line_number":661,"context_line":""},{"line_number":662,"context_line":"# The primary flavor used for most of the tests (integer value)"},{"line_number":663,"context_line":"#flavor_ref_ram \u003d 64"},{"line_number":664,"context_line":""},{"line_number":665,"context_line":"# The secondary flavor used for tests that need two flavors, like tests that"},{"line_number":666,"context_line":"# perform a resize (integer value)"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"dab17558_8bc49c92","line":663,"in_reply_to":"dab17558_7290514f","updated":"2016-05-13 11:13:32.000000000","message":"+1 for Tempest section! But we can leave it as it is for now to move all options to \u0027[tempest]\u0027 section once.","commit_id":"375cb652e6234eee5ad999d15910b36d2783b9db"},{"author":{"_account_id":20947,"name":"Ciprian Barbu","email":"ciprian.barbu@gmail.com","username":"ciprian.barbu"},"change_message_id":"c8f3a45e5636118733b7aeb38fc43dfe43d41202","unresolved":false,"context_lines":[{"line_number":660,"context_line":"#name_regex \u003d ^.*(cirros|testvm).*$"},{"line_number":661,"context_line":""},{"line_number":662,"context_line":"# The primary flavor used for most of the tests (integer value)"},{"line_number":663,"context_line":"#flavor_ref_ram \u003d 64"},{"line_number":664,"context_line":""},{"line_number":665,"context_line":"# The secondary flavor used for tests that need two flavors, like tests that"},{"line_number":666,"context_line":"# perform a resize (integer value)"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"bab6814e_dfc4d241","line":663,"in_reply_to":"dab17558_8bc49c92","updated":"2016-05-18 12:00:12.000000000","message":"So is there anything left to do for this change? Jenkins verification failed and I can\u0027t figure out why, can you help me with that maybe?","commit_id":"375cb652e6234eee5ad999d15910b36d2783b9db"}],"rally/verification/tempest/config.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"64bda13a8de1a6c5503a48e9ade90d44ba04e536","unresolved":false,"context_lines":[{"line_number":388,"context_line":"        if \"heat\" in self.available_services:"},{"line_number":389,"context_line":"            self._configure_option(\"orchestration\", \"instance_type\","},{"line_number":390,"context_line":"                                   self._discover_or_create_flavor,"},{"line_number":391,"context_line":"                                   flavor_ref_ram)"},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"        _write_config(self.conf_path, self.conf)"},{"line_number":394,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_e0714caa","line":391,"updated":"2016-05-12 08:45:04.000000000","message":"I think that here should be separate option differ from \"Reference flavor RAM size used when creating images\" As I understand they are not always the same","commit_id":"0f236c0f4c4c74edbe1cd3d8706a67a3d0846813"},{"author":{"_account_id":20947,"name":"Ciprian Barbu","email":"ciprian.barbu@gmail.com","username":"ciprian.barbu"},"change_message_id":"a376d43cb8a6d53bdb3bf0c9c6ccb3ad0f5e9ae3","unresolved":false,"context_lines":[{"line_number":388,"context_line":"        if \"heat\" in self.available_services:"},{"line_number":389,"context_line":"            self._configure_option(\"orchestration\", \"instance_type\","},{"line_number":390,"context_line":"                                   self._discover_or_create_flavor,"},{"line_number":391,"context_line":"                                   flavor_ref_ram)"},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"        _write_config(self.conf_path, self.conf)"},{"line_number":394,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"dab17558_a62e3cf8","line":391,"in_reply_to":"dab17558_e0714caa","updated":"2016-05-12 09:30:06.000000000","message":"I can add a new option, of course, but how should I name it? I can\u0027t really tell what is different between the flavor used by orchestration and reference flavor.","commit_id":"0f236c0f4c4c74edbe1cd3d8706a67a3d0846813"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"dd8e99412ddd2c366da5bba55e00b84df33316dc","unresolved":false,"context_lines":[{"line_number":59,"context_line":"                    \"In this case Rally will create needed resources by \""},{"line_number":60,"context_line":"                    \"itself if the values for the corresponding config \""},{"line_number":61,"context_line":"                    \"options are not specified in the Tempest config file\"),"},{"line_number":62,"context_line":"    cfg.StrOpt(\"flavor_ref_ram\","},{"line_number":63,"context_line":"               default\u003d\"64\","},{"line_number":64,"context_line":"               help\u003d\"Primar flavor RAM size used by most of the test cases\"),"},{"line_number":65,"context_line":"    cfg.StrOpt(\"flavor_ref_alt_ram\","}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_5f858dab","line":62,"range":{"start_line":62,"start_character":8,"end_line":62,"end_character":14},"updated":"2016-05-12 11:47:37.000000000","message":"What about IntOpt here?","commit_id":"263b68608db7a5e5f3d1a370e50e5f1ea0e39079"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"dd8e99412ddd2c366da5bba55e00b84df33316dc","unresolved":false,"context_lines":[{"line_number":62,"context_line":"    cfg.StrOpt(\"flavor_ref_ram\","},{"line_number":63,"context_line":"               default\u003d\"64\","},{"line_number":64,"context_line":"               help\u003d\"Primar flavor RAM size used by most of the test cases\"),"},{"line_number":65,"context_line":"    cfg.StrOpt(\"flavor_ref_alt_ram\","},{"line_number":66,"context_line":"               default\u003d\"128\","},{"line_number":67,"context_line":"               help\u003d\"Alternate reference flavor RAM size used by test that\""},{"line_number":68,"context_line":"               \"need two flavors, like those that resize an instnace\"),"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_1fbae56e","line":65,"range":{"start_line":65,"start_character":8,"end_line":65,"end_character":14},"updated":"2016-05-12 11:47:37.000000000","message":"ditto","commit_id":"263b68608db7a5e5f3d1a370e50e5f1ea0e39079"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"dd8e99412ddd2c366da5bba55e00b84df33316dc","unresolved":false,"context_lines":[{"line_number":66,"context_line":"               default\u003d\"128\","},{"line_number":67,"context_line":"               help\u003d\"Alternate reference flavor RAM size used by test that\""},{"line_number":68,"context_line":"               \"need two flavors, like those that resize an instnace\"),"},{"line_number":69,"context_line":"    cfg.StrOpt(\"heat_instance_type_ram\","},{"line_number":70,"context_line":"               default\u003d\"64\","},{"line_number":71,"context_line":"               help\u003d\"RAM size flavor used for orchestration test cases\")"},{"line_number":72,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_7f7691d9","line":69,"range":{"start_line":69,"start_character":8,"end_line":69,"end_character":15},"updated":"2016-05-12 11:47:37.000000000","message":"ditto","commit_id":"263b68608db7a5e5f3d1a370e50e5f1ea0e39079"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"dd8e99412ddd2c366da5bba55e00b84df33316dc","unresolved":false,"context_lines":[{"line_number":369,"context_line":"                               self._discover_or_create_image)"},{"line_number":370,"context_line":"        self._configure_option(\"compute\", \"flavor_ref\","},{"line_number":371,"context_line":"                               self._discover_or_create_flavor,"},{"line_number":372,"context_line":"                               int(CONF.image.flavor_ref_ram))"},{"line_number":373,"context_line":"        self._configure_option(\"compute\", \"flavor_ref_alt\","},{"line_number":374,"context_line":"                               self._discover_or_create_flavor,"},{"line_number":375,"context_line":"                               int(CONF.image.flavor_ref_alt_ram))"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_3f43696e","line":372,"range":{"start_line":372,"start_character":31,"end_line":372,"end_character":35},"updated":"2016-05-12 11:47:37.000000000","message":"Then int() will not be needed here.","commit_id":"263b68608db7a5e5f3d1a370e50e5f1ea0e39079"},{"author":{"_account_id":20947,"name":"Ciprian Barbu","email":"ciprian.barbu@gmail.com","username":"ciprian.barbu"},"change_message_id":"d50c472c30984b231f2ae2f5238149412188803b","unresolved":false,"context_lines":[{"line_number":369,"context_line":"                               self._discover_or_create_image)"},{"line_number":370,"context_line":"        self._configure_option(\"compute\", \"flavor_ref\","},{"line_number":371,"context_line":"                               self._discover_or_create_flavor,"},{"line_number":372,"context_line":"                               int(CONF.image.flavor_ref_ram))"},{"line_number":373,"context_line":"        self._configure_option(\"compute\", \"flavor_ref_alt\","},{"line_number":374,"context_line":"                               self._discover_or_create_flavor,"},{"line_number":375,"context_line":"                               int(CONF.image.flavor_ref_alt_ram))"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_6291d652","line":372,"range":{"start_line":372,"start_character":31,"end_line":372,"end_character":35},"in_reply_to":"dab17558_3f43696e","updated":"2016-05-12 13:35:27.000000000","message":"Right, that makes more sense :)","commit_id":"263b68608db7a5e5f3d1a370e50e5f1ea0e39079"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"dd8e99412ddd2c366da5bba55e00b84df33316dc","unresolved":false,"context_lines":[{"line_number":372,"context_line":"                               int(CONF.image.flavor_ref_ram))"},{"line_number":373,"context_line":"        self._configure_option(\"compute\", \"flavor_ref_alt\","},{"line_number":374,"context_line":"                               self._discover_or_create_flavor,"},{"line_number":375,"context_line":"                               int(CONF.image.flavor_ref_alt_ram))"},{"line_number":376,"context_line":"        if \"neutron\" in self.available_services:"},{"line_number":377,"context_line":"            neutronclient \u003d self.clients.neutron()"},{"line_number":378,"context_line":"            if neutronclient.list_networks(shared\u003dTrue)[\"networks\"]:"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_7f5df147","line":375,"range":{"start_line":375,"start_character":31,"end_line":375,"end_character":35},"updated":"2016-05-12 11:47:37.000000000","message":"ditto","commit_id":"263b68608db7a5e5f3d1a370e50e5f1ea0e39079"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"dd8e99412ddd2c366da5bba55e00b84df33316dc","unresolved":false,"context_lines":[{"line_number":390,"context_line":"        if \"heat\" in self.available_services:"},{"line_number":391,"context_line":"            self._configure_option(\"orchestration\", \"instance_type\","},{"line_number":392,"context_line":"                                   self._discover_or_create_flavor,"},{"line_number":393,"context_line":"                                   int(CONF.image.heat_instance_type_ram))"},{"line_number":394,"context_line":""},{"line_number":395,"context_line":"        _write_config(self.conf_path, self.conf)"},{"line_number":396,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_df34ddfc","line":393,"range":{"start_line":393,"start_character":35,"end_line":393,"end_character":39},"updated":"2016-05-12 11:47:37.000000000","message":"ditto","commit_id":"263b68608db7a5e5f3d1a370e50e5f1ea0e39079"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"03c78590cf1e58c82242483ecb58a1ecda4f8032","unresolved":false,"context_lines":[{"line_number":59,"context_line":"                    \"In this case Rally will create needed resources by \""},{"line_number":60,"context_line":"                    \"itself if the values for the corresponding config \""},{"line_number":61,"context_line":"                    \"options are not specified in the Tempest config file\"),"},{"line_number":62,"context_line":"    cfg.IntOpt(\"flavor_ref_ram\","},{"line_number":63,"context_line":"               default\u003d\"64\","},{"line_number":64,"context_line":"               help\u003d\"Primar flavor RAM size used by most of the test cases\"),"},{"line_number":65,"context_line":"    cfg.IntOpt(\"flavor_ref_alt_ram\","}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_3f2f853f","line":62,"updated":"2016-05-25 15:16:44.000000000","message":"You register flavor related options in the image group. I don\u0027t think that there is a sense to create the flavor group and register these options there. Let\u0027s create the only \u0027tempest\u0027 group as Alexander Maretskiy suggested, move old options there and add new options.","commit_id":"375cb652e6234eee5ad999d15910b36d2783b9db"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"1a2e856beaa671f01284439b1ae5a90d37decc47","unresolved":false,"context_lines":[{"line_number":39,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"TEMPEST_OPTS \u003d ["},{"line_number":42,"context_line":"    cfg.StrOpt(\"cirros_img_url\","},{"line_number":43,"context_line":"               default\u003d\"http://download.cirros-cloud.net/\""},{"line_number":44,"context_line":"                       \"0.3.4/cirros-0.3.4-x86_64-disk.img\","},{"line_number":45,"context_line":"               help\u003d\"CirrOS image URL\"),"},{"line_number":46,"context_line":"    cfg.StrOpt(\"disk_format\","},{"line_number":47,"context_line":"               default\u003d\"qcow2\","},{"line_number":48,"context_line":"               help\u003d\"Image disk format to use when creating the image\"),"}],"source_content_type":"text/x-python","patch_set":5,"id":"9abb7d3a_c64e9fc0","line":45,"range":{"start_line":42,"start_character":4,"end_line":45,"end_character":39},"updated":"2016-06-03 09:14:29.000000000","message":"You should add\n\ndeprecated_group\u003d\"image\"\n\nhere.\n\n\ncfg.StrOpt(\"cirros_img_url\",\n           default\u003d\"http://download.cirros-cloud.net/\"\n                   \"0.3.4/cirros-0.3.4-x86_64-disk.img\",\n           deprecated_group\u003d\"image\", help\u003d\"CirrOS image URL\")","commit_id":"cd09ce312e8e025f0b0a7aed0392c302942dbfae"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"1a2e856beaa671f01284439b1ae5a90d37decc47","unresolved":false,"context_lines":[{"line_number":43,"context_line":"               default\u003d\"http://download.cirros-cloud.net/\""},{"line_number":44,"context_line":"                       \"0.3.4/cirros-0.3.4-x86_64-disk.img\","},{"line_number":45,"context_line":"               help\u003d\"CirrOS image URL\"),"},{"line_number":46,"context_line":"    cfg.StrOpt(\"disk_format\","},{"line_number":47,"context_line":"               default\u003d\"qcow2\","},{"line_number":48,"context_line":"               help\u003d\"Image disk format to use when creating the image\"),"},{"line_number":49,"context_line":"    cfg.StrOpt(\"container_format\","}],"source_content_type":"text/x-python","patch_set":5,"id":"9abb7d3a_46d10fcd","line":46,"updated":"2016-06-03 09:14:29.000000000","message":"ditto","commit_id":"cd09ce312e8e025f0b0a7aed0392c302942dbfae"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"1a2e856beaa671f01284439b1ae5a90d37decc47","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    cfg.StrOpt(\"disk_format\","},{"line_number":47,"context_line":"               default\u003d\"qcow2\","},{"line_number":48,"context_line":"               help\u003d\"Image disk format to use when creating the image\"),"},{"line_number":49,"context_line":"    cfg.StrOpt(\"container_format\","},{"line_number":50,"context_line":"               default\u003d\"bare\","},{"line_number":51,"context_line":"               help\u003d\"Image container format to use when creating the image\"),"},{"line_number":52,"context_line":"    cfg.StrOpt(\"name_regex\","}],"source_content_type":"text/x-python","patch_set":5,"id":"9abb7d3a_86dbf7ec","line":49,"updated":"2016-06-03 09:14:29.000000000","message":"ditto","commit_id":"cd09ce312e8e025f0b0a7aed0392c302942dbfae"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"1a2e856beaa671f01284439b1ae5a90d37decc47","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    cfg.StrOpt(\"container_format\","},{"line_number":50,"context_line":"               default\u003d\"bare\","},{"line_number":51,"context_line":"               help\u003d\"Image container format to use when creating the image\"),"},{"line_number":52,"context_line":"    cfg.StrOpt(\"name_regex\","},{"line_number":53,"context_line":"               default\u003d\"^.*(cirros|testvm).*$\","},{"line_number":54,"context_line":"               help\u003d\"Regular expression for name of a public image to \""},{"line_number":55,"context_line":"                    \"discover it in the cloud and use it for the tests. \""}],"source_content_type":"text/x-python","patch_set":5,"id":"9abb7d3a_06eee78a","line":52,"updated":"2016-06-03 09:14:29.000000000","message":"ditto","commit_id":"cd09ce312e8e025f0b0a7aed0392c302942dbfae"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"1a2e856beaa671f01284439b1ae5a90d37decc47","unresolved":false,"context_lines":[{"line_number":71,"context_line":"               help\u003d\"RAM size flavor used for orchestration test cases\")"},{"line_number":72,"context_line":"]"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"ROLE_OPTS \u003d ["},{"line_number":75,"context_line":"    cfg.StrOpt(\"swift_operator_role\","},{"line_number":76,"context_line":"               default\u003d\"Member\","},{"line_number":77,"context_line":"               help\u003d\"Role required for users \""},{"line_number":78,"context_line":"                    \"to be able to create Swift containers\"),"},{"line_number":79,"context_line":"    cfg.StrOpt(\"swift_reseller_admin_role\","},{"line_number":80,"context_line":"               default\u003d\"ResellerAdmin\","},{"line_number":81,"context_line":"               help\u003d\"User role that has reseller admin\"),"},{"line_number":82,"context_line":"    cfg.StrOpt(\"heat_stack_owner_role\","},{"line_number":83,"context_line":"               default\u003d\"heat_stack_owner\","},{"line_number":84,"context_line":"               help\u003d\"Role required for users \""},{"line_number":85,"context_line":"                    \"to be able to manage Heat stacks\"),"},{"line_number":86,"context_line":"    cfg.StrOpt(\"heat_stack_user_role\","},{"line_number":87,"context_line":"               default\u003d\"heat_stack_user\","},{"line_number":88,"context_line":"               help\u003d\"Role for Heat template-defined users\")"},{"line_number":89,"context_line":"]"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"CONF \u003d cfg.CONF"},{"line_number":92,"context_line":"CONF.register_opts(TEMPEST_OPTS, \"tempest\")"}],"source_content_type":"text/x-python","patch_set":5,"id":"9abb7d3a_8672d77a","line":89,"range":{"start_line":74,"start_character":0,"end_line":89,"end_character":1},"updated":"2016-06-03 09:14:29.000000000","message":"These options should be also moved to TEMPEST_OPTS.","commit_id":"cd09ce312e8e025f0b0a7aed0392c302942dbfae"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"e3e1255a4d418178f2bd4180362ace08a297848a","unresolved":false,"context_lines":[{"line_number":83,"context_line":"               help\u003d\"Role for Heat template-defined users\"),"},{"line_number":84,"context_line":"    cfg.IntOpt(\"flavor_ref_ram\","},{"line_number":85,"context_line":"               default\u003d\"64\","},{"line_number":86,"context_line":"               help\u003d\"Primar flavor RAM size used by most of the test cases\"),"},{"line_number":87,"context_line":"    cfg.IntOpt(\"flavor_ref_alt_ram\","},{"line_number":88,"context_line":"               default\u003d\"128\","},{"line_number":89,"context_line":"               help\u003d\"Alternate reference flavor RAM size used by test that\""}],"source_content_type":"text/x-python","patch_set":6,"id":"9abb7d3a_996071a6","line":86,"range":{"start_line":86,"start_character":21,"end_line":86,"end_character":27},"updated":"2016-06-03 10:47:13.000000000","message":"Primar -\u003e Primary","commit_id":"c3fd8ccb9a48a4f79e94df0030023657a79380b2"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":39,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"TEMPEST_OPTS \u003d ["},{"line_number":42,"context_line":"    cfg.StrOpt(\"cirros_img_url\","},{"line_number":43,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":44,"context_line":"               default\u003d\"http://download.cirros-cloud.net/\""},{"line_number":45,"context_line":"                       \"0.3.4/cirros-0.3.4-x86_64-disk.img\","}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_3464d1e2","line":42,"range":{"start_line":42,"start_character":16,"end_line":42,"end_character":30},"updated":"2016-06-07 09:51:59.000000000","message":"Let\u0027s rename this option into \u0027img_url\u0027 because a user may want to use another image.","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"TEMPEST_OPTS \u003d ["},{"line_number":42,"context_line":"    cfg.StrOpt(\"cirros_img_url\","},{"line_number":43,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":44,"context_line":"               default\u003d\"http://download.cirros-cloud.net/\""},{"line_number":45,"context_line":"                       \"0.3.4/cirros-0.3.4-x86_64-disk.img\","},{"line_number":46,"context_line":"               help\u003d\"CirrOS image URL\"),"}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_942ae52a","line":43,"range":{"start_line":43,"start_character":15,"end_line":43,"end_character":39},"updated":"2016-06-07 09:51:59.000000000","message":"Then here you should use\ndeprecated_opts\u003d[cfg.DeprecatedOpt(\"cirros_img_url\", group\u003d\"image\")]","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":43,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":44,"context_line":"               default\u003d\"http://download.cirros-cloud.net/\""},{"line_number":45,"context_line":"                       \"0.3.4/cirros-0.3.4-x86_64-disk.img\","},{"line_number":46,"context_line":"               help\u003d\"CirrOS image URL\"),"},{"line_number":47,"context_line":"    cfg.StrOpt(\"disk_format\","},{"line_number":48,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":49,"context_line":"               default\u003d\"qcow2\","}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_b45ba186","line":46,"range":{"start_line":46,"start_character":21,"end_line":46,"end_character":27},"updated":"2016-06-07 09:51:59.000000000","message":"It should be removed.","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":44,"context_line":"               default\u003d\"http://download.cirros-cloud.net/\""},{"line_number":45,"context_line":"                       \"0.3.4/cirros-0.3.4-x86_64-disk.img\","},{"line_number":46,"context_line":"               help\u003d\"CirrOS image URL\"),"},{"line_number":47,"context_line":"    cfg.StrOpt(\"disk_format\","},{"line_number":48,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":49,"context_line":"               default\u003d\"qcow2\","},{"line_number":50,"context_line":"               help\u003d\"Image disk format to use when creating the image\"),"}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_b4ac41a1","line":47,"range":{"start_line":47,"start_character":16,"end_line":47,"end_character":27},"updated":"2016-06-07 09:51:59.000000000","message":"img_disk_format","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":45,"context_line":"                       \"0.3.4/cirros-0.3.4-x86_64-disk.img\","},{"line_number":46,"context_line":"               help\u003d\"CirrOS image URL\"),"},{"line_number":47,"context_line":"    cfg.StrOpt(\"disk_format\","},{"line_number":48,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":49,"context_line":"               default\u003d\"qcow2\","},{"line_number":50,"context_line":"               help\u003d\"Image disk format to use when creating the image\"),"},{"line_number":51,"context_line":"    cfg.StrOpt(\"container_format\","}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_3478512a","line":48,"range":{"start_line":48,"start_character":15,"end_line":48,"end_character":39},"updated":"2016-06-07 09:51:59.000000000","message":"deprecated_opts\u003d[cfg.DeprecatedOpt(\"disk_format\", group\u003d\"image\")]","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":48,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":49,"context_line":"               default\u003d\"qcow2\","},{"line_number":50,"context_line":"               help\u003d\"Image disk format to use when creating the image\"),"},{"line_number":51,"context_line":"    cfg.StrOpt(\"container_format\","},{"line_number":52,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":53,"context_line":"               default\u003d\"bare\","},{"line_number":54,"context_line":"               help\u003d\"Image container format to use when creating the image\"),"}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_b49a8161","line":51,"range":{"start_line":51,"start_character":16,"end_line":51,"end_character":32},"updated":"2016-06-07 09:51:59.000000000","message":"img_container_format","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":49,"context_line":"               default\u003d\"qcow2\","},{"line_number":50,"context_line":"               help\u003d\"Image disk format to use when creating the image\"),"},{"line_number":51,"context_line":"    cfg.StrOpt(\"container_format\","},{"line_number":52,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":53,"context_line":"               default\u003d\"bare\","},{"line_number":54,"context_line":"               help\u003d\"Image container format to use when creating the image\"),"},{"line_number":55,"context_line":"    cfg.StrOpt(\"name_regex\","}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_14aa7570","line":52,"range":{"start_line":52,"start_character":15,"end_line":52,"end_character":39},"updated":"2016-06-07 09:51:59.000000000","message":"deprecated_opts\u003d[cfg.DeprecatedOpt(\"container_format\", group\u003d\"image\")]","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":52,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":53,"context_line":"               default\u003d\"bare\","},{"line_number":54,"context_line":"               help\u003d\"Image container format to use when creating the image\"),"},{"line_number":55,"context_line":"    cfg.StrOpt(\"name_regex\","},{"line_number":56,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":57,"context_line":"               default\u003d\"^.*(cirros|testvm).*$\","},{"line_number":58,"context_line":"               help\u003d\"Regular expression for name of a public image to \""}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_d4c62d48","line":55,"range":{"start_line":55,"start_character":16,"end_line":55,"end_character":26},"updated":"2016-06-07 09:51:59.000000000","message":"img_name_regex","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":53,"context_line":"               default\u003d\"bare\","},{"line_number":54,"context_line":"               help\u003d\"Image container format to use when creating the image\"),"},{"line_number":55,"context_line":"    cfg.StrOpt(\"name_regex\","},{"line_number":56,"context_line":"               deprecated_group\u003d\"image\","},{"line_number":57,"context_line":"               default\u003d\"^.*(cirros|testvm).*$\","},{"line_number":58,"context_line":"               help\u003d\"Regular expression for name of a public image to \""},{"line_number":59,"context_line":"                    \"discover it in the cloud and use it for the tests. \""}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_54e19dcf","line":56,"range":{"start_line":56,"start_character":15,"end_line":56,"end_character":39},"updated":"2016-06-07 09:51:59.000000000","message":"deprecated_opts\u003d[cfg.DeprecatedOpt(\"name_regex\", group\u003d\"image\")]","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":137,"context_line":"            CONF.tempest.cirros_img_url).path.split(\"/\")[-1]"},{"line_number":138,"context_line":"        self._download_cirros_image()"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"    def _download_cirros_image(self):"},{"line_number":141,"context_line":"        img_path \u003d os.path.join(self.data_dir, self.image_name)"},{"line_number":142,"context_line":"        if os.path.isfile(img_path):"},{"line_number":143,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_14e2759f","line":140,"range":{"start_line":140,"start_character":8,"end_line":140,"end_character":30},"updated":"2016-06-07 09:51:59.000000000","message":"_download_image","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":145,"context_line":"        try:"},{"line_number":146,"context_line":"            response \u003d requests.get(CONF.tempest.cirros_img_url, stream\u003dTrue)"},{"line_number":147,"context_line":"        except requests.ConnectionError as err:"},{"line_number":148,"context_line":"            msg \u003d _(\"Failed to download CirrOS image. \""},{"line_number":149,"context_line":"                    \"Possibly there is no connection to Internet. \""},{"line_number":150,"context_line":"                    \"Error: %s.\") % (str(err) or \"unknown\")"},{"line_number":151,"context_line":"            raise exceptions.TempestConfigCreationFailure(msg)"}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_94eca590","line":148,"range":{"start_line":148,"start_character":40,"end_line":148,"end_character":46},"updated":"2016-06-07 09:51:59.000000000","message":"It should be deleted.","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":159,"context_line":"            os.rename(img_path + \".tmp\", img_path)"},{"line_number":160,"context_line":"        else:"},{"line_number":161,"context_line":"            if response.status_code \u003d\u003d 404:"},{"line_number":162,"context_line":"                msg \u003d _(\"Failed to download CirrOS image. \""},{"line_number":163,"context_line":"                        \"Image was not found.\")"},{"line_number":164,"context_line":"            else:"},{"line_number":165,"context_line":"                msg \u003d _(\"Failed to download CirrOS image. \""}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_54999d1e","line":162,"range":{"start_line":162,"start_character":44,"end_line":162,"end_character":50},"updated":"2016-06-07 09:51:59.000000000","message":"ditto","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"88a36c39502e694556b35a093f3d63b15c52c6ab","unresolved":false,"context_lines":[{"line_number":162,"context_line":"                msg \u003d _(\"Failed to download CirrOS image. \""},{"line_number":163,"context_line":"                        \"Image was not found.\")"},{"line_number":164,"context_line":"            else:"},{"line_number":165,"context_line":"                msg \u003d _(\"Failed to download CirrOS image. \""},{"line_number":166,"context_line":"                        \"HTTP error code %d.\") % response.status_code"},{"line_number":167,"context_line":"            raise exceptions.TempestConfigCreationFailure(msg)"},{"line_number":168,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_94a3054f","line":165,"range":{"start_line":165,"start_character":44,"end_line":165,"end_character":50},"updated":"2016-06-07 09:51:59.000000000","message":"ditto","commit_id":"c428ed859e9ffb7ce7587d2daebc83e361c6c854"}]}