Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(229)

Unified Diff: src/scripts/image_to_live.sh

Issue 1572034: Refine image_to_live script to log all output from devserver. (Closed)
Patch Set: Created 10 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/scripts/image_to_live.sh
diff --git a/src/scripts/image_to_live.sh b/src/scripts/image_to_live.sh
index 49974453dfa3ce388ac815dc53025beac1c42547..b735341e2be546ec1423ff23f4b76916eb1e195d 100755
--- a/src/scripts/image_to_live.sh
+++ b/src/scripts/image_to_live.sh
@@ -18,6 +18,8 @@ DEFINE_boolean ignore_hostname ${FLAGS_TRUE} \
"Ignore existing AU hostname on running instance use this hostname"
DEFINE_boolean update_known_hosts ${FLAGS_FALSE} \
"Update your known_hosts with the new remote instance's key"
+DEFINE_boolean verbose ${FLAGS_FALSE} \
+ "Whether to output verbose information for debugging."
function kill_all_devservers {
# Using ! here to avoid exiting with set -e is insufficient, so use
@@ -38,7 +40,11 @@ function remote_reboot_sh {
function start_dev_server {
kill_all_devservers
- ./enter_chroot.sh "./start_devserver > /dev/null 2>&1" &
+ if [ ${FLAGS_verbose} -eq ${FLAGS_FALSE} ]; then
+ ./enter_chroot.sh "./start_devserver > /dev/null 2>&1" &
+ else
+ ./enter_chroot.sh "./start_devserver" &
+ fi
echo -n "Waiting on devserver to start"
until netstat -anp 2>&1 | grep 8080 > /dev/null; do
sleep .5
@@ -169,7 +175,7 @@ function main() {
if ! run_auto_update; then
echo "Update was not successful."
- exit
+ exit 1
fi
remote_reboot
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698