From faaca31e533a36a9c391c288365f24c50a6fb4cf Mon Sep 17 00:00:00 2001 From: guodan1 Date: Thu, 31 Jan 2019 20:36:57 +0800 Subject: [PATCH] mgr/dashboard: Add date range and log search functionality Fixes: http://tracker.ceph.com/issues/37387t st Signed-off-by: guodan1 --- .../src/app/ceph/cluster/cluster.module.ts | 6 +- .../app/ceph/cluster/logs/logs.component.html | 80 ++++++++++++- .../app/ceph/cluster/logs/logs.component.scss | 102 ++++++++++++++++ .../ceph/cluster/logs/logs.component.spec.ts | 112 +++++++++++++++++- .../app/ceph/cluster/logs/logs.component.ts | 87 +++++++++++++- .../frontend/src/locale/messages.xlf | 38 +++++- 6 files changed, 413 insertions(+), 12 deletions(-) diff --git a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/cluster.module.ts b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/cluster.module.ts index a84f0544c49..fce9b6ab988 100644 --- a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/cluster.module.ts +++ b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/cluster.module.ts @@ -5,9 +5,11 @@ import { RouterModule } from '@angular/router'; import { TreeModule } from 'ng2-tree'; import { AlertModule } from 'ngx-bootstrap/alert'; +import { BsDatepickerModule } from 'ngx-bootstrap/datepicker'; import { BsDropdownModule } from 'ngx-bootstrap/dropdown'; import { ModalModule } from 'ngx-bootstrap/modal'; import { TabsModule } from 'ngx-bootstrap/tabs'; +import { TimepickerModule } from 'ngx-bootstrap/timepicker'; import { TooltipModule } from 'ngx-bootstrap/tooltip'; import { SharedModule } from '../../shared/shared.module'; @@ -51,7 +53,9 @@ import { PrometheusListComponent } from './prometheus/prometheus-list/prometheus AlertModule.forRoot(), TooltipModule.forRoot(), TreeModule, - MgrModulesModule + MgrModulesModule, + TimepickerModule.forRoot(), + BsDatepickerModule.forRoot() ], declarations: [ HostsComponent, diff --git a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.html b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.html index d14dbf0be6b..a2c7d2389eb 100644 --- a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.html +++ b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.html @@ -1,10 +1,11 @@
+
-
-

+

+

{{ line.stamp }} {{ line.priority }} {{ line.message }} @@ -19,8 +20,8 @@

-
-

+

+

{{ line.stamp }} {{ line.priority }} {{ line.message }} @@ -33,3 +34,74 @@

+ + +
+
+ + +
+
+ +
+ + + + + + + +
+
+
+ +
+ + + + +
+
+
+
+ + + +  —  + + +
+
+
diff --git a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.scss b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.scss index b084aada8cc..8c91d4ad8a9 100644 --- a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.scss +++ b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.scss @@ -39,3 +39,105 @@ p { color: $color-brand-teal; } } + +::ng-deep timepicker table tbody tr td { + .bs-timepicker-field { + width: 3.5rem; + font-size: 1rem; + padding: 4px 6px; + } + .btn { + font-size: 1rem; + } +} + +.log-filters { + margin-bottom: 5px; + padding: 0 30px; + * { + box-sizing: border-box; + } + + .filter-box { + margin: 0; + padding: 0 15px 5px 0; + display: -webkit-flex; + display: flex; + justify-content: flex-start; + align-items: center; + label { + padding-top: 5px; + padding-right: 5px; + } + } + + @media (max-width: 991px) { + .time-box { + margin-top: 20px; + } + } + + @media (min-width: 1200px) { + .cd-col-4 { + width: 28vw; + } + + .cd-col-3 { + width: 20vw; + } + + .cd-col-2 { + width: 16vw; + } + .cd-col-1 { + width: 14vw; + } + } + + @media (min-width: 1400px) { + .cd-col-4 { + width: 24vw; + } + + .cd-col-3 { + width: 18vw; + } + + .cd-col-2 { + width: 14vw; + } + .cd-col-1 { + width: 12vw; + } + } + + @media (min-width: 1600px) { + .cd-col-4 { + width: 22vw; + } + + .cd-col-3 { + width: 16vw; + } + + .cd-col-2 { + width: 12vw; + } + .cd-col-1 { + width: 10vw; + } + } + + @media (min-width: 1800px) { + .cd-col-3 { + width: 14vw; + } + + .cd-col-2 { + width: 11vw; + } + .cd-col-1 { + width: 9vw; + } + } +} diff --git a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.spec.ts b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.spec.ts index 6a8007edbb0..8b4436cc7ef 100644 --- a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.spec.ts +++ b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.spec.ts @@ -1,7 +1,10 @@ import { HttpClientTestingModule } from '@angular/common/http/testing'; import { ComponentFixture, TestBed } from '@angular/core/testing'; +import { FormsModule } from '@angular/forms'; +import { BsDatepickerModule } from 'ngx-bootstrap/datepicker'; import { TabsModule } from 'ngx-bootstrap/tabs'; +import { TimepickerModule } from 'ngx-bootstrap/timepicker'; import { configureTestBed } from '../../../../testing/unit-test-helper'; import { SharedModule } from '../../../shared/shared.module'; @@ -12,7 +15,14 @@ describe('LogsComponent', () => { let fixture: ComponentFixture; configureTestBed({ - imports: [HttpClientTestingModule, TabsModule.forRoot(), SharedModule], + imports: [ + HttpClientTestingModule, + TabsModule.forRoot(), + SharedModule, + BsDatepickerModule.forRoot(), + TimepickerModule.forRoot(), + FormsModule + ], declarations: [LogsComponent] }); @@ -25,4 +35,104 @@ describe('LogsComponent', () => { it('should create', () => { expect(component).toBeTruthy(); }); + + describe('abstractfilters', () => { + it('after initializaed', () => { + const filters = component.abstractfilters(); + expect(filters.priority).toBe('All'); + expect(filters.key).toBe(''); + expect(filters.yearMonthDay).toBe(''); + expect(filters.sTime).toBe(0); + expect(filters.eTime).toBe(1439); + }); + it('change date', () => { + component.selectedDate = new Date(2019, 0, 1); + component.startTime = new Date(2019, 1, 1, 1, 10); + component.endTime = new Date(2019, 1, 1, 12, 10); + const filters = component.abstractfilters(); + expect(filters.yearMonthDay).toBe('2019-01-01'); + expect(filters.sTime).toBe(70); + expect(filters.eTime).toBe(730); + }); + }); + + describe('filterLogs', () => { + const contentData = { + clog: [ + { + name: 'priority', + stamp: '2019-02-21 09:39:49.572801', + message: 'Manager daemon localhost is now available', + priority: '[ERR]' + }, + { + name: 'search', + stamp: '2019-02-21 09:39:49.572801', + message: 'Activating manager daemon localhost', + priority: '[INF]' + }, + { + name: 'date', + stamp: '2019-01-21 09:39:49.572801', + message: 'Manager daemon localhost is now available', + priority: '[INF]' + }, + { + name: 'time', + stamp: '2019-02-21 01:39:49.572801', + message: 'Manager daemon localhost is now available', + priority: '[INF]' + } + ], + audit_log: [] + }; + const resetFilter = () => { + component.selectedDate = null; + component.priority = 'All'; + component.search = ''; + component.startTime.setHours(0, 0); + component.endTime.setHours(23, 59); + }; + beforeEach(() => { + component.contentData = contentData; + }); + + it('show all log', () => { + component.filterLogs(); + expect(component.clog.length).toBe(4); + }); + + it('filter by search key', () => { + resetFilter(); + component.search = 'Activating'; + component.filterLogs(); + expect(component.clog.length).toBe(1); + expect(component.clog[0].name).toBe('search'); + }); + + it('filter by date', () => { + resetFilter(); + component.selectedDate = new Date(2019, 0, 21); + component.filterLogs(); + expect(component.clog.length).toBe(1); + expect(component.clog[0].name).toBe('date'); + }); + + it('filter by priority', () => { + resetFilter(); + component.priority = '[ERR]'; + component.filterLogs(); + expect(component.clog.length).toBe(1); + expect(component.clog[0].name).toBe('priority'); + }); + + it('filter by time range', () => { + resetFilter(); + component.startTime.setHours(1, 0); + component.endTime.setHours(2, 0); + component.filterLogs(); + expect(component.clog.length).toBe(1); + expect(component.clog[0].name).toBe('time'); + }); + }); }); diff --git a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.ts b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.ts index d3a9a341ad7..cd20503c3a3 100644 --- a/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.ts +++ b/src/pybind/mgr/dashboard/frontend/src/app/ceph/cluster/logs/logs.component.ts @@ -9,9 +9,29 @@ import { LogsService } from '../../../shared/api/logs.service'; }) export class LogsComponent implements OnInit, OnDestroy { contentData: any; - interval: number; + clog: Array; + audit_log: Array; - constructor(private logsService: LogsService) {} + interval: number; + bsConfig = { + dateInputFormat: 'YYYY-MM-DD', + containerClass: 'theme-default' + }; + prioritys: Array<{ name: string; value: string }> = [ + { name: 'Info', value: '[INF]' }, + { name: 'Warning', value: '[WRN]' }, + { name: 'Error', value: '[ERR]' }, + { name: 'All', value: 'All' } + ]; + priority = 'All'; + search = ''; + selectedDate: Date; + startTime: Date = new Date(); + endTime: Date = new Date(); + constructor(private logsService: LogsService) { + this.startTime.setHours(0, 0); + this.endTime.setHours(23, 59); + } ngOnInit() { this.getInfo(); @@ -27,6 +47,69 @@ export class LogsComponent implements OnInit, OnDestroy { getInfo() { this.logsService.getLogs().subscribe((data: any) => { this.contentData = data; + this.filterLogs(); }); } + + abstractfilters(): any { + const priority = this.priority; + const key = this.search.toLowerCase().replace(/,/g, ''); + + let yearMonthDay: string; + if (this.selectedDate) { + const m = this.selectedDate.getMonth() + 1; + const d = this.selectedDate.getDate(); + + const year = this.selectedDate.getFullYear().toString(); + const month = m <= 9 ? `0${m}` : `${m}`; + const day = d <= 9 ? `0${d}` : `${d}`; + yearMonthDay = `${year}-${month}-${day}`; + } else { + yearMonthDay = ''; + } + + const sHour = this.startTime ? this.startTime.getHours() : 0; + const sMinutes = this.startTime ? this.startTime.getMinutes() : 0; + const sTime = sHour * 60 + sMinutes; + + const eHour = this.endTime ? this.endTime.getHours() : 23; + const eMinutes = this.endTime ? this.endTime.getMinutes() : 59; + const eTime = eHour * 60 + eMinutes; + + return { priority, key, yearMonthDay, sTime, eTime }; + } + + filterExecutor(logs: Array, filters: any): Array { + return logs.filter((line) => { + const hour = parseInt(line.stamp.slice(11, 13), 10); + const minutes = parseInt(line.stamp.slice(14, 16), 10); + let prio: string, y_m_d: string, timeSpan: number; + + prio = filters.priority === 'All' ? line.priority : filters.priority; + y_m_d = filters.yearMonthDay ? filters.yearMonthDay : line.stamp; + timeSpan = hour * 60 + minutes; + return ( + line.priority === prio && + line.message.toLowerCase().indexOf(filters.key) !== -1 && + line.stamp.indexOf(y_m_d) !== -1 && + timeSpan >= filters.sTime && + timeSpan <= filters.eTime + ); + }); + } + + filterLogs() { + const filters = this.abstractfilters(); + this.clog = this.filterExecutor(this.contentData.clog, filters); + this.audit_log = this.filterExecutor(this.contentData.audit_log, filters); + } + + clearSearchKey() { + this.search = ''; + this.filterLogs(); + } + clearDate() { + this.selectedDate = null; + this.filterLogs(); + } } diff --git a/src/pybind/mgr/dashboard/frontend/src/locale/messages.xlf b/src/pybind/mgr/dashboard/frontend/src/locale/messages.xlf index 11c6b814a7e..2f184d852d1 100644 --- a/src/pybind/mgr/dashboard/frontend/src/locale/messages.xlf +++ b/src/pybind/mgr/dashboard/frontend/src/locale/messages.xlf @@ -1394,23 +1394,53 @@ No entries found app/ceph/cluster/logs/logs.component.html - 14 + 15 app/ceph/cluster/logs/logs.component.html - 30 + 31 Cluster Logs app/ceph/cluster/logs/logs.component.html - 4 + 5 Audit Logs app/ceph/cluster/logs/logs.component.html - 20 + 21 + + + Priority: + + app/ceph/cluster/logs/logs.component.html + 41 + + + Keyword: + + app/ceph/cluster/logs/logs.component.html + 51 + + + Date: + + app/ceph/cluster/logs/logs.component.html + 70 + + + Datepicker + + app/ceph/cluster/logs/logs.component.html + 75 + + + Time range: + + app/ceph/cluster/logs/logs.component.html + 91 Loading configuration... -- 2.39.5