)]}'
{"projects/70_cinder/upgrade.sh":[{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"55e1d85d5e7ecee727ce3d83b267e1f579d18883","unresolved":false,"context_lines":[{"line_number":65,"context_line":"# Don\u0027t succeed unless the services come up"},{"line_number":66,"context_line":"ensure_services_started cinder-api"},{"line_number":67,"context_line":"ensure_logs_exist c-api"},{"line_number":68,"context_line":"if is_service_enabled c-vol; then"},{"line_number":69,"context_line":"    ensure_logs_exist c-vol"},{"line_number":70,"context_line":"fi"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"3ac371cc_70fc5272","line":68,"range":{"start_line":68,"start_character":3,"end_line":68,"end_character":28},"updated":"2016-08-17 17:06:03.000000000","message":"What if c-vol service did not come back due to the upgrade?\nI think a flag should be added before the migration (probably line 59) and here (line68) if flag was set then verify log exist.\n\nAlso what about cinder scheduler log/service?","commit_id":"8a609fe6b75b83c936dd55557d00aad5c86c9de2"}]}