[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] [OSSTEST PATCH 2/2] ms-* html generation: Provide right title for projection



When ms-queuedaemon generates a resource-projection.html, it sometimes
does so from data-plan.pl (see proc report-plan).  This means that
ms-planner does not get a reliable indication of whether it is being
run for the plan or the projection, and the resource-project.html
sometimes claims to be the plan.

Fix with a new ms-planner option -W which tells it what to put in the
title, defaulting to the value passed to -w.

DEPLOYMENT NOTE:

The new ms-planner works with the old queuedaemon, so when upgrading,
it is OK to simply update the daemons-testing.git and then restart the
ms-queuedaemon.

If it is necessary to downgrade, rewinding to the old commit with a
running ms-queuedaemon will cause errors from the old ms-planner being
passed -w -- but these errors are trapped and ignored.  So in this
case reports will be out of datte until ms-queuedaemon is also
restarted.

In either case nothing will go badly wrong.

Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
---
 ms-planner     |    7 ++++++-
 ms-queuedaemon |    2 +-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/ms-planner b/ms-planner
index 7ce0405..6f4a511 100755
--- a/ms-planner
+++ b/ms-planner
@@ -34,6 +34,7 @@ use Osstest::Executive;
 open DEBUG, ">/dev/null" or die $!;
 
 our $walker = 'plan';
+our $walker_show;
 
 while (@ARGV and $ARGV[0] =~ m/^-/) {
     $_= shift @ARGV;
@@ -43,12 +44,16 @@ while (@ARGV and $ARGV[0] =~ m/^-/) {
             open DEBUG, ">&STDERR" or die $!;
         } elsif (s/^-w(.+)/-/) {
            $walker = $1;
+        } elsif (s/^-W(.+)/-/) {
+           $walker_show = $1;
         } else {
             die "$_ ?";
         }
     }
 }
 
+$walker_show //= $walker;
+
 csreadconfig();
 
 our ($plan);
@@ -688,7 +693,7 @@ sub cmd_show_html () {
     #    Dumper(\@rows, \@newoutcols);
 
     printf "<html><head><title>Resource %s - %s</title></head><body>\n",
-        $walker, ($c{DnsDomain} // '?');
+        $walker_show, ($c{DnsDomain} // '?');
 
     printf "<table rules=all><tr><td>plan age %s</td><td>report at %s</td>\n",
         show_rel_time($now);
diff --git a/ms-queuedaemon b/ms-queuedaemon
index 08f88cf..2b8d621 100755
--- a/ms-queuedaemon
+++ b/ms-queuedaemon
@@ -301,7 +301,7 @@ proc report-plan {w wo} {
     global c
     catching-internally "showing $w html" {
        set outputfile "$c(WebspaceFile)/resource-$wo.html"
-       exec ./ms-planner -w$w show-html > $outputfile
+       exec ./ms-planner -w$w -W$wo show-html > $outputfile
     } {
        set out data-$wo.final.pl
        file copy -force data-$w.pl $out.new
-- 
1.7.10.4


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.