From 2caf8fddff77368110b1588af50a72a3c70f2aac Mon Sep 17 00:00:00 2001
From: Jesse Andrews <anotherjesse@gmail.com>
Date: Mon, 12 Sep 2011 16:15:11 -0700
Subject: [PATCH] comments on how to improve

---
 README          | 17 +++++++++++++----
 files/sudo/nova |  1 +
 stack.sh        |  8 +++++++-
 3 files changed, 21 insertions(+), 5 deletions(-)
 create mode 100644 files/sudo/nova

diff --git a/README b/README
index 4a78f6e..2efa315 100644
--- a/README
+++ b/README
@@ -1,7 +1,16 @@
-Tool to build nfs export openstack environments.  This is useful for development
+Tool to build (nfs export) openstack dev environments
+
+## Tod
+
+* move back to using sudo
+* Add volume support
+* Add quantum support
+* Add python-novaclient cli support
+* allow changing of git locations (specify different tag/branch or repos)
+* change method of invoking stack.sh to ./stack.sh all _or_ ./stack.sh nova-compute,nova-network,nova-api,nova-volume
+* allow sql/rabbit connection to be specified via environment variables with sensible defaults
+
+# Future
 
-* todo: move as much from run to install as possible
-  * issue: HOST_IP will be different on each machine - might need to update / change stuff
-* todo: use local disk on nfs boots for instances
 * idea: create a live-cd / vmware preview image using this?
 * idea: use lxc to make the proto-image have everything?  so launching a new server is just turn on / updating / run?
diff --git a/files/sudo/nova b/files/sudo/nova
new file mode 100644
index 0000000..3843416
--- /dev/null
+++ b/files/sudo/nova
@@ -0,0 +1 @@
+socat
diff --git a/stack.sh b/stack.sh
index ad6a9cb..00bb891 100755
--- a/stack.sh
+++ b/stack.sh
@@ -37,8 +37,12 @@ NET_MAN=${NET_MAN:-VlanManager}
 #             ip or you risk breaking things.
 # FLAT_INTERFACE=eth0
 
+# TODO: set rabbitmq conn string explicitly as well
+# TODO: switch to mysql for all services
 SQL_CONN=sqlite:///$NOVA_DIR/nova.sqlite
 
+# FIXME: commands should be: stack.sh all or list of services to install/run
+
 # You should only have to run this once
 if [ "$CMD" == "install" ]; then
     # install apt requirements
@@ -105,7 +109,7 @@ if [ "$CMD" == "install" ]; then
     # create an empty directory to use as our 
     mkdir $DASH_DIR/.blackhole
     # FIXME(ja): can't figure out how to make $DASH_DIR work in sed, also install to available/a2e it 
-    cat $DIR/files/000-default.template | sed "s/%DASH_DIR%/\/opt\/dash/g" > /etc/apache2/sites-enabled/000-default
+    cat $DIR/files/000-default.template | sed 's/%DASH_DIR%/\/opt\/dash/g' > /etc/apache2/sites-enabled/000-default
 
     chown -R www-data:www-data $DASH_DIR
 
@@ -121,6 +125,8 @@ if [ "$CMD" == "install" ]; then
 
     # add useful screenrc
     cp $DIR/files/screenrc ~/.screenrc
+
+    # TODO: update current user to allow sudo for all commands in files/sudo/*
     exit
 fi