]> xenbits.xensource.com Git - people/aperard/osstest.git/commitdiff
host reuse: ms-planner: Do not show reuse as shared in the plan
authorIan Jackson <ian.jackson@eu.citrix.com>
Mon, 30 Oct 2017 16:52:24 +0000 (16:52 +0000)
committerIan Jackson <iwj@xenproject.org>
Fri, 2 Oct 2020 15:04:19 +0000 (16:04 +0100)
If the number of shares is 1, do not show it as shared, and also
ignore the Unshare events.

This clarifies the display, especially when used with forthcoming test
host reuse work.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
ms-planner

index d9f3db9114bf4e755cfdb74fb1e2624474a3763e..4e38e4e3188473b71113261e931251f72eae879b 100755 (executable)
@@ -249,6 +249,7 @@ sub launder_check_plan () {
                        Avail => 1,
                        Info => $cshare->{Type},
                        PreviousShare => $cshare,
+                        Shares => $cshare->{Shares},
                    };
                    $cshare= undef;
                }
@@ -678,6 +679,7 @@ sub cmd_show_html () {
        foreach my $evt (@{ $plan->{Events}{$reso} }) {
            my $type= $evt->{Type};
            my $share= $evt->{Share};
+           next if $type eq 'Unshare' && $evt->{Shares} == 1;
            my $show=
                $type eq End ? ($evt->{Share} ? "End $evt->{Info}" : "") :
                 $type eq Unshare ? "$type [".showsharetype($evt->{Info})."]" :
@@ -686,7 +688,7 @@ sub cmd_show_html () {
                (!$evt->{Allocated} ? Booking :
                  ($evt->{Allocated}{Live} ? Allocation : Completed).
                  " $evt->{Allocated}{Task}")." ".$evt->{Info};
-           if ($share) {
+           if ($share && $share->{Shares} != 1) {
                $show .= sprintf(" [%s %d/%d %d]",
                                 showsharetype($share->{Type}),
                                 $share->{Shares} - $evt->{Avail},
@@ -789,7 +791,8 @@ sub cmd_show_html () {
                     $content->{Avail} ||
                     $content->{Type} eq After ||
                     ($content->{Allocated} && !$content->{Allocated}{Live});
-                my $shared = $content->{Share};
+                my $shared = $content->{Share}
+                         && $content->{Share}{Shares} != 1;
                 my $bgcolour=
                     $avail ?
                     ($shared ? "#ccccff" : "#ffffff") :