)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"34be3538b8ebc4ff2b0b6c49a4178db09de830ed","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Wait for 60 seconds on NoValidHost exception in sheduler"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"After schesuler starts it doesn\u0027t have data about hosts capabilities."},{"line_number":10,"context_line":"Let\u0027s sleep and try once more time."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Sleep for 60 seconds as a PeriodicTask interval."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da86d52c_82765504","line":9,"updated":"2015-02-05 21:26:21.000000000","message":"scheduler","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"e16c2d46fc47c9bb02544bc0f3b49fa25ec12ebc","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Wait for 60 seconds on NoValidHost exception in sheduler"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"After schesuler starts it doesn\u0027t have data about hosts capabilities."},{"line_number":10,"context_line":"Let\u0027s sleep and try once more time."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Sleep for 60 seconds as a PeriodicTask interval."},{"line_number":13,"context_line":"Need to re-think it and do not listen creat requests before capabilities"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da86d52c_028265ff","line":10,"updated":"2015-02-05 21:16:07.000000000","message":"once -\u003e one","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"e16c2d46fc47c9bb02544bc0f3b49fa25ec12ebc","unresolved":false,"context_lines":[{"line_number":10,"context_line":"Let\u0027s sleep and try once more time."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Sleep for 60 seconds as a PeriodicTask interval."},{"line_number":13,"context_line":"Need to re-think it and do not listen creat requests before capabilities"},{"line_number":14,"context_line":"data received. Current RPC server implementation doesn\u0027t suppor it."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I351d61342402b892fd4dac1efaa65b595575a39f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da86d52c_4298dd77","line":13,"updated":"2015-02-05 21:16:07.000000000","message":"type \u0027to create\u0027","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"e16c2d46fc47c9bb02544bc0f3b49fa25ec12ebc","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Sleep for 60 seconds as a PeriodicTask interval."},{"line_number":13,"context_line":"Need to re-think it and do not listen creat requests before capabilities"},{"line_number":14,"context_line":"data received. Current RPC server implementation doesn\u0027t suppor it."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I351d61342402b892fd4dac1efaa65b595575a39f"},{"line_number":17,"context_line":"Closes-Bug: #1409012"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da86d52c_829e5561","line":14,"updated":"2015-02-05 21:16:07.000000000","message":"typo suppor -\u003e support","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"}],"cinder/scheduler/flows/create_volume.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"34be3538b8ebc4ff2b0b6c49a4178db09de830ed","unresolved":false,"context_lines":[{"line_number":128,"context_line":"            self.driver_api.schedule_create_volume(context, request_spec,"},{"line_number":129,"context_line":"                                                   filter_properties)"},{"line_number":130,"context_line":"        except exception.NoValidHost as e:"},{"line_number":131,"context_line":"            # NOTE (e0ne): After schesuler starts it doesn\u0027t have data about"},{"line_number":132,"context_line":"            # capabilities. Let\u0027s sleep and try once more time."},{"line_number":133,"context_line":"            # Sleep for 60 seconds as a PeriodicTask interval."},{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_a248b9b3","line":131,"updated":"2015-02-05 21:26:21.000000000","message":"scheduler","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"f762cf46f0390b7b17d45243a61be4e9e9321390","unresolved":false,"context_lines":[{"line_number":128,"context_line":"            self.driver_api.schedule_create_volume(context, request_spec,"},{"line_number":129,"context_line":"                                                   filter_properties)"},{"line_number":130,"context_line":"        except exception.NoValidHost as e:"},{"line_number":131,"context_line":"            # NOTE (e0ne): After schesuler starts it doesn\u0027t have data about"},{"line_number":132,"context_line":"            # capabilities. Let\u0027s sleep and try once more time."},{"line_number":133,"context_line":"            # Sleep for 60 seconds as a PeriodicTask interval."},{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_5d6600ee","line":131,"in_reply_to":"da86d52c_a248b9b3","updated":"2015-02-05 21:48:35.000000000","message":"Done","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"e16c2d46fc47c9bb02544bc0f3b49fa25ec12ebc","unresolved":false,"context_lines":[{"line_number":129,"context_line":"                                                   filter_properties)"},{"line_number":130,"context_line":"        except exception.NoValidHost as e:"},{"line_number":131,"context_line":"            # NOTE (e0ne): After schesuler starts it doesn\u0027t have data about"},{"line_number":132,"context_line":"            # capabilities. Let\u0027s sleep and try once more time."},{"line_number":133,"context_line":"            # Sleep for 60 seconds as a PeriodicTask interval."},{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"},{"line_number":135,"context_line":"            # capabilities data received. Current RPC server implementation"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_c2920d4b","line":132,"updated":"2015-02-05 21:16:07.000000000","message":"once -\u003e one and other typo comments same as the commit message.","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"f762cf46f0390b7b17d45243a61be4e9e9321390","unresolved":false,"context_lines":[{"line_number":129,"context_line":"                                                   filter_properties)"},{"line_number":130,"context_line":"        except exception.NoValidHost as e:"},{"line_number":131,"context_line":"            # NOTE (e0ne): After schesuler starts it doesn\u0027t have data about"},{"line_number":132,"context_line":"            # capabilities. Let\u0027s sleep and try once more time."},{"line_number":133,"context_line":"            # Sleep for 60 seconds as a PeriodicTask interval."},{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"},{"line_number":135,"context_line":"            # capabilities data received. Current RPC server implementation"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_3d6b0cb8","line":132,"in_reply_to":"da86d52c_c2920d4b","updated":"2015-02-05 21:48:35.000000000","message":"Done","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"db34f93d5ffec87c6358b5018d93a9e5eb4aa144","unresolved":false,"context_lines":[{"line_number":133,"context_line":"            # Sleep for 60 seconds as a PeriodicTask interval."},{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"},{"line_number":135,"context_line":"            # capabilities data received. Current RPC server implementation"},{"line_number":136,"context_line":"            # doesn\u0027t suppor it."},{"line_number":137,"context_line":"            time.sleep(60)"},{"line_number":138,"context_line":"            try:"},{"line_number":139,"context_line":"                self.driver_api.schedule_create_volume(context, request_spec,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_82dad514","line":136,"updated":"2015-02-05 21:18:54.000000000","message":"Typo suppor -\u003e support","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"f762cf46f0390b7b17d45243a61be4e9e9321390","unresolved":false,"context_lines":[{"line_number":133,"context_line":"            # Sleep for 60 seconds as a PeriodicTask interval."},{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"},{"line_number":135,"context_line":"            # capabilities data received. Current RPC server implementation"},{"line_number":136,"context_line":"            # doesn\u0027t suppor it."},{"line_number":137,"context_line":"            time.sleep(60)"},{"line_number":138,"context_line":"            try:"},{"line_number":139,"context_line":"                self.driver_api.schedule_create_volume(context, request_spec,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_1d71e8a7","line":136,"in_reply_to":"da86d52c_82dad514","updated":"2015-02-05 21:48:35.000000000","message":"Done","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"e16c2d46fc47c9bb02544bc0f3b49fa25ec12ebc","unresolved":false,"context_lines":[{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"},{"line_number":135,"context_line":"            # capabilities data received. Current RPC server implementation"},{"line_number":136,"context_line":"            # doesn\u0027t suppor it."},{"line_number":137,"context_line":"            time.sleep(60)"},{"line_number":138,"context_line":"            try:"},{"line_number":139,"context_line":"                self.driver_api.schedule_create_volume(context, request_spec,"},{"line_number":140,"context_line":"                                                       filter_properties)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_82ac958d","line":137,"updated":"2015-02-05 21:16:07.000000000","message":"Should we maybe use John\u0027s new looping implementation here?","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":10379,"name":"Rajini Karthik","email":"rrajini@gmail.com","username":"rajinir"},"change_message_id":"db34f93d5ffec87c6358b5018d93a9e5eb4aa144","unresolved":false,"context_lines":[{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"},{"line_number":135,"context_line":"            # capabilities data received. Current RPC server implementation"},{"line_number":136,"context_line":"            # doesn\u0027t suppor it."},{"line_number":137,"context_line":"            time.sleep(60)"},{"line_number":138,"context_line":"            try:"},{"line_number":139,"context_line":"                self.driver_api.schedule_create_volume(context, request_spec,"},{"line_number":140,"context_line":"                                                       filter_properties)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_a24359ad","line":137,"updated":"2015-02-05 21:18:54.000000000","message":"Will specific time of xx guarentee that it will have data ready by then?","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"f762cf46f0390b7b17d45243a61be4e9e9321390","unresolved":false,"context_lines":[{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"},{"line_number":135,"context_line":"            # capabilities data received. Current RPC server implementation"},{"line_number":136,"context_line":"            # doesn\u0027t suppor it."},{"line_number":137,"context_line":"            time.sleep(60)"},{"line_number":138,"context_line":"            try:"},{"line_number":139,"context_line":"                self.driver_api.schedule_create_volume(context, request_spec,"},{"line_number":140,"context_line":"                                                       filter_properties)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_9deb18db","line":137,"in_reply_to":"da86d52c_42053d44","updated":"2015-02-05 21:48:35.000000000","message":"OK I\u0027ll add a config option for it","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"f762cf46f0390b7b17d45243a61be4e9e9321390","unresolved":false,"context_lines":[{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"},{"line_number":135,"context_line":"            # capabilities data received. Current RPC server implementation"},{"line_number":136,"context_line":"            # doesn\u0027t suppor it."},{"line_number":137,"context_line":"            time.sleep(60)"},{"line_number":138,"context_line":"            try:"},{"line_number":139,"context_line":"                self.driver_api.schedule_create_volume(context, request_spec,"},{"line_number":140,"context_line":"                                                       filter_properties)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_fd0ed4e3","line":137,"in_reply_to":"da86d52c_82ac958d","updated":"2015-02-05 21:48:35.000000000","message":"I\u0027ll try it, thanks!","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"34be3538b8ebc4ff2b0b6c49a4178db09de830ed","unresolved":false,"context_lines":[{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"},{"line_number":135,"context_line":"            # capabilities data received. Current RPC server implementation"},{"line_number":136,"context_line":"            # doesn\u0027t suppor it."},{"line_number":137,"context_line":"            time.sleep(60)"},{"line_number":138,"context_line":"            try:"},{"line_number":139,"context_line":"                self.driver_api.schedule_create_volume(context, request_spec,"},{"line_number":140,"context_line":"                                                       filter_properties)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_42053d44","line":137,"in_reply_to":"da86d52c_a24359ad","updated":"2015-02-05 21:26:21.000000000","message":"If you need to wait here, I would suggest using eventlet.sleep().  Also at least make the 60 seconds configurable.","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"f762cf46f0390b7b17d45243a61be4e9e9321390","unresolved":false,"context_lines":[{"line_number":134,"context_line":"            # Need to re-think it and do not listen create requests before"},{"line_number":135,"context_line":"            # capabilities data received. Current RPC server implementation"},{"line_number":136,"context_line":"            # doesn\u0027t suppor it."},{"line_number":137,"context_line":"            time.sleep(60)"},{"line_number":138,"context_line":"            try:"},{"line_number":139,"context_line":"                self.driver_api.schedule_create_volume(context, request_spec,"},{"line_number":140,"context_line":"                                                       filter_properties)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_1d12c808","line":137,"in_reply_to":"da86d52c_a24359ad","updated":"2015-02-05 21:48:35.000000000","message":"No, but for now we didn\u0027t have a way to require this data. At least, I didn\u0027t found it:(.","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"34be3538b8ebc4ff2b0b6c49a4178db09de830ed","unresolved":false,"context_lines":[{"line_number":136,"context_line":"            # doesn\u0027t suppor it."},{"line_number":137,"context_line":"            time.sleep(60)"},{"line_number":138,"context_line":"            try:"},{"line_number":139,"context_line":"                self.driver_api.schedule_create_volume(context, request_spec,"},{"line_number":140,"context_line":"                                                       filter_properties)"},{"line_number":141,"context_line":"            except exception.NoValidHost as e:"},{"line_number":142,"context_line":"                # No host found happened, notify on the scheduler queue and log"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_62aec12f","line":139,"updated":"2015-02-05 21:26:21.000000000","message":"I\u0027m not sure if we should call create_volume again. By default, I think create_volume will be retried 3 times.","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"f762cf46f0390b7b17d45243a61be4e9e9321390","unresolved":false,"context_lines":[{"line_number":136,"context_line":"            # doesn\u0027t suppor it."},{"line_number":137,"context_line":"            time.sleep(60)"},{"line_number":138,"context_line":"            try:"},{"line_number":139,"context_line":"                self.driver_api.schedule_create_volume(context, request_spec,"},{"line_number":140,"context_line":"                                                       filter_properties)"},{"line_number":141,"context_line":"            except exception.NoValidHost as e:"},{"line_number":142,"context_line":"                # No host found happened, notify on the scheduler queue and log"}],"source_content_type":"text/x-python","patch_set":2,"id":"da86d52c_1d29e836","line":139,"in_reply_to":"da86d52c_62aec12f","updated":"2015-02-05 21:48:35.000000000","message":"We need once more retry after sleep","commit_id":"e7b29ff85daf69a363b5e5d61418d42e298decee"}]}