[ui] Move Placement Failures notification above job status panel (#17750)

* Moves the Placement Failures box above job status, should it exist

* Move it for non-service job-types as well
This commit is contained in:
Phil Renaud
2023-06-27 19:32:51 -04:00
committed by GitHub
parent 72a9f2b551
commit f3df01e422
7 changed files with 17 additions and 13 deletions

View File

@@ -8,8 +8,8 @@
<jobPage.ui.Error />
<jobPage.ui.Title />
<jobPage.ui.StatsBox />
<jobPage.ui.StatusPanel @statusMode={{@statusMode}} @setStatusMode={{@setStatusMode}} />
<jobPage.ui.PlacementFailures />
<jobPage.ui.StatusPanel @statusMode={{@statusMode}} @setStatusMode={{@setStatusMode}} />
<jobPage.ui.TaskGroups @sortProperty={{@sortProperty}} @sortDescending={{@sortDescending}} />
<jobPage.ui.RecentAllocations @activeTask={{@activeTask}} @setActiveTaskQueryParam={{@setActiveTaskQueryParam}} />
<jobPage.ui.Meta />

View File

@@ -22,8 +22,8 @@
</span>
</:before-namespace>
</jobPage.ui.StatsBox>
<jobPage.ui.StatusPanel @statusMode={{@statusMode}} @setStatusMode={{@setStatusMode}} />
<jobPage.ui.PlacementFailures />
<jobPage.ui.StatusPanel @statusMode={{@statusMode}} @setStatusMode={{@setStatusMode}} />
<jobPage.ui.TaskGroups @sortProperty={{@sortProperty}} @sortDescending={{@sortDescending}} />
<jobPage.ui.RecentAllocations @activeTask={{@activeTask}} @setActiveTaskQueryParam={{@setActiveTaskQueryParam}} />
<div class="boxed-section">

View File

@@ -4,14 +4,12 @@
~}}
{{#if this.job.hasPlacementFailures}}
<div class="boxed-section is-danger" data-test-placement-failures>
<div class="boxed-section-head">
Placement Failures
</div>
<div class="boxed-section-body">
<Hds::Alert @type="inline" @color="critical" data-test-placement-failures class="boxed-section placement-failures" as |A|>
<A.Title>Placement Failures</A.Title>
<A.Description>
{{#each this.job.taskGroups as |taskGroup|}}
<PlacementFailure @taskGroup={{taskGroup}} />
{{/each}}
</div>
</div>
</A.Description>
</Hds::Alert>
{{/if}}

View File

@@ -9,8 +9,8 @@
<jobPage.ui.Title />
<jobPage.ui.StatsBox />
<jobPage.ui.DasRecommendations />
<jobPage.ui.StatusPanel @statusMode={{@statusMode}} @setStatusMode={{@setStatusMode}} />
<jobPage.ui.PlacementFailures />
<jobPage.ui.StatusPanel @statusMode={{@statusMode}} @setStatusMode={{@setStatusMode}} />
<jobPage.ui.TaskGroups @sortProperty={{@sortProperty}} @sortDescending={{@sortDescending}} />
<jobPage.ui.RecentAllocations @activeTask={{@activeTask}} @setActiveTaskQueryParam={{@setActiveTaskQueryParam}} />
<jobPage.ui.Meta />

View File

@@ -8,8 +8,8 @@
<jobPage.ui.Error />
<jobPage.ui.Title />
<jobPage.ui.StatsBox />
<jobPage.ui.StatusPanel @statusMode={{@statusMode}} @setStatusMode={{@setStatusMode}} />
<jobPage.ui.PlacementFailures />
<jobPage.ui.StatusPanel @statusMode={{@statusMode}} @setStatusMode={{@setStatusMode}} />
<jobPage.ui.TaskGroups @sortProperty={{@sortProperty}} @sortDescending={{@sortDescending}} />
<jobPage.ui.RecentAllocations @activeTask={{@activeTask}} @setActiveTaskQueryParam={{@setActiveTaskQueryParam}} />
<jobPage.ui.Meta />

View File

@@ -9,8 +9,8 @@
<jobPage.ui.Title />
<jobPage.ui.StatsBox />
<jobPage.ui.DasRecommendations />
<jobPage.ui.StatusPanel @statusMode={{@statusMode}} @setStatusMode={{@setStatusMode}} />
<jobPage.ui.PlacementFailures />
<jobPage.ui.StatusPanel @statusMode={{@statusMode}} @setStatusMode={{@setStatusMode}} />
<jobPage.ui.TaskGroups @sortProperty={{@sortProperty}} @sortDescending={{@sortDescending}} />
<jobPage.ui.RecentAllocations @activeTask={{@activeTask}} @setActiveTaskQueryParam={{@setActiveTaskQueryParam}} />
<jobPage.ui.Meta />

View File

@@ -7,7 +7,13 @@
{{#with this.placementFailures as |failures|}}
<h3 class="title is-5" data-test-placement-failure-task-group>
{{this.placementFailures.name}}
<span class="badge is-light" data-test-placement-failure-coalesced-failures>{{inc failures.coalescedFailures}} unplaced</span>
<Hds::Badge
data-test-placement-failure-coalesced-failures
@color="critical"
@type="outlined"
@size="small"
@text="{{inc failures.coalescedFailures}} unplaced"
/>
</h3>
<ul class="simple-list">
{{#if (eq failures.nodesEvaluated 0)}}