From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Jackson Subject: [OSSTEST PATCH 2/2] ms-* html generation: Provide right title for projection Date: Mon, 4 Jan 2016 16:49:38 +0000 Message-ID: <1451926178-10812-3-git-send-email-ian.jackson@eu.citrix.com> References: <1451926178-10812-1-git-send-email-ian.jackson@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1aG8KO-0003VL-F1 for xen-devel@lists.xenproject.org; Mon, 04 Jan 2016 16:49:52 +0000 In-Reply-To: <1451926178-10812-1-git-send-email-ian.jackson@eu.citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: xen-devel@lists.xenproject.org Cc: Ian Jackson , Ian Campbell List-Id: xen-devel@lists.xenproject.org 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 --- 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 "Resource %s - %s\n", - $walker, ($c{DnsDomain} // '?'); + $walker_show, ($c{DnsDomain} // '?'); printf "\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
plan age %sreport at %s