)]}'
{"sahara/service/validations/clusters.py":[{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"9e0e669bf64a03ff6294966f64c12f530f932079","unresolved":false,"context_lines":[{"line_number":90,"context_line":"    else:"},{"line_number":91,"context_line":"        if CONF.use_neutron:"},{"line_number":92,"context_line":"            raise ex.NotFoundException(\u0027neutron_management_network\u0027,"},{"line_number":93,"context_line":"                                       \"\u0027%s\u0027 field is not found\")"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"def _get_cluster_field(cluster, field):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_39bf0d62","line":93,"updated":"2014-05-23 01:32:58.000000000","message":"is there supposed to be  a parameter to go with the %s?","commit_id":"206da8319643cc455753793b6ccac24212af958f"},{"author":{"_account_id":8411,"name":"Andrew Lazarev","email":"alazarev@mirantis.com","username":"alazarev"},"change_message_id":"f9f5de59e7403f52a2733276dce94b295ed9e0bc","unresolved":false,"context_lines":[{"line_number":90,"context_line":"    else:"},{"line_number":91,"context_line":"        if CONF.use_neutron:"},{"line_number":92,"context_line":"            raise ex.NotFoundException(\u0027neutron_management_network\u0027,"},{"line_number":93,"context_line":"                                       \"\u0027%s\u0027 field is not found\")"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"def _get_cluster_field(cluster, field):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_6b02823f","line":93,"in_reply_to":"7adec928_30454970","updated":"2014-05-23 17:59:40.000000000","message":"Made it more explicit.","commit_id":"206da8319643cc455753793b6ccac24212af958f"},{"author":{"_account_id":8411,"name":"Andrew Lazarev","email":"alazarev@mirantis.com","username":"alazarev"},"change_message_id":"44fb56f76de2e554cffe702ce964a3f8363aa0ac","unresolved":false,"context_lines":[{"line_number":90,"context_line":"    else:"},{"line_number":91,"context_line":"        if CONF.use_neutron:"},{"line_number":92,"context_line":"            raise ex.NotFoundException(\u0027neutron_management_network\u0027,"},{"line_number":93,"context_line":"                                       \"\u0027%s\u0027 field is not found\")"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"def _get_cluster_field(cluster, field):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_5aecc429","line":93,"in_reply_to":"7adec928_39bf0d62","updated":"2014-05-23 07:06:29.000000000","message":"\u0027neutron_management_network\u0027 is a parameter. I also don\u0027t like how NotFoundException implemented, but this is out of scope for this CR.","commit_id":"206da8319643cc455753793b6ccac24212af958f"},{"author":{"_account_id":8754,"name":"Mike May","email":"themistymay@gmail.com","username":"themistymay"},"change_message_id":"df04987f92e7aebd8ba7cac660b9b0148b46734a","unresolved":false,"context_lines":[{"line_number":90,"context_line":"    else:"},{"line_number":91,"context_line":"        if CONF.use_neutron:"},{"line_number":92,"context_line":"            raise ex.NotFoundException(\u0027neutron_management_network\u0027,"},{"line_number":93,"context_line":"                                       \"\u0027%s\u0027 field is not found\")"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"def _get_cluster_field(cluster, field):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_30454970","line":93,"in_reply_to":"7adec928_5aecc429","updated":"2014-05-23 17:23:13.000000000","message":"Wow, did not know this either. Very interesting...","commit_id":"206da8319643cc455753793b6ccac24212af958f"}],"sahara/tests/unit/service/validation/utils.py":[{"author":{"_account_id":8754,"name":"Mike May","email":"themistymay@gmail.com","username":"themistymay"},"change_message_id":"df04987f92e7aebd8ba7cac660b9b0148b46734a","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from sahara.conductor import resource as r"},{"line_number":20,"context_line":"from sahara.plugins.vanilla import plugin"},{"line_number":21,"context_line":"import sahara.service.validation as v"},{"line_number":22,"context_line":"from sahara.tests.unit import base"},{"line_number":23,"context_line":"from sahara.tests.unit import testutils as tu"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_d02a658f","line":22,"updated":"2014-05-23 17:23:13.000000000","message":"Is this change worth updating the commit message to add as an item changed?  Or is this leftover from a previous cleanup?","commit_id":"206da8319643cc455753793b6ccac24212af958f"},{"author":{"_account_id":8411,"name":"Andrew Lazarev","email":"alazarev@mirantis.com","username":"alazarev"},"change_message_id":"f9f5de59e7403f52a2733276dce94b295ed9e0bc","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from sahara.conductor import resource as r"},{"line_number":20,"context_line":"from sahara.plugins.vanilla import plugin"},{"line_number":21,"context_line":"import sahara.service.validation as v"},{"line_number":22,"context_line":"from sahara.tests.unit import base"},{"line_number":23,"context_line":"from sahara.tests.unit import testutils as tu"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_8b3a6e32","line":22,"in_reply_to":"7adec928_d02a658f","updated":"2014-05-23 17:59:40.000000000","message":"This is to use override_config.","commit_id":"206da8319643cc455753793b6ccac24212af958f"}]}