Merge lp:~allenap/maas/ntp-service-ui into lp:~maas-committers/maas/trunk

Proposed by Gavin Panella
Status: Merged
Approved by: Gavin Panella
Approved revision: no longer in the source branch.
Merged at revision: 5331
Proposed branch: lp:~allenap/maas/ntp-service-ui
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 17 lines (+7/-0)
1 file modified
src/maasserver/static/partials/node-details.html (+7/-0)
To merge this branch: bzr merge lp:~allenap/maas/ntp-service-ui
Reviewer Review Type Date Requested Status
Andres Rodriguez (community) Approve
Review via email: mp+305119@code.launchpad.net

Commit message

Include NTP in the service monitoring UI.

To post a comment you must log in.
Revision history for this message
Andres Rodriguez (andreserl) wrote :

lgtm!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/maasserver/static/partials/node-details.html'
2--- src/maasserver/static/partials/node-details.html 2016-09-06 10:21:08 +0000
3+++ src/maasserver/static/partials/node-details.html 2016-09-07 15:30:15 +0000
4@@ -454,6 +454,13 @@
5 <span data-ng-if="services.tgt.status_info"> &mdash; {$ services.tgt.status_info $}</span>
6 </span>
7 </li>
8+ <li class="list__item">
9+ <span>
10+ <span class="icon icon--{$ getServiceClass(services.ntp) $}"></span>
11+ &nbsp;ntp
12+ <span data-ng-if="services.ntp.status_info"> &mdash; {$ services.ntp.status_info $}</span>
13+ </span>
14+ </li>
15 </ul>
16 </div>
17 </div>