-
Notifications
You must be signed in to change notification settings - Fork 72
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Changes on the VM Details -> Overview Card: 1. display last 2 events that are associated with the VM 2. provide a "View All" link to a new sub-page: the Events view. Events view features: 1. display events in a table with 3 columns: a) severity b) date (includes time) c) message 2. allow filtering and sorting for all columns via toolbar 3. limit the max number of events that can be fetched to 500 4. do incremental refresh by fetching only new events
- Loading branch information
Showing
19 changed files
with
662 additions
and
11 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,87 @@ | ||
import React, { useMemo } from 'react' | ||
import PropTypes from 'prop-types' | ||
import { connect } from 'react-redux' | ||
import { enumMsg, withMsg } from '_/intl' | ||
import { saveEventFilters } from '_/actions' | ||
import { localeCompare, toJS } from '_/helpers' | ||
import moment from 'moment' | ||
|
||
import { Filters } from '_/components/Toolbar' | ||
|
||
export const SEVERITY = 'severity' | ||
export const DATE = 'date' | ||
export const MESSAGE = 'message' | ||
export const UNKNOWN = 'unknown' | ||
|
||
export const EVENT_SEVERITY = { | ||
error: 3, | ||
warning: 2, | ||
normal: 1, | ||
[UNKNOWN]: 0, | ||
} | ||
|
||
export function filterEvents ({ events, severityFilters, dateFilters, messageFilters }) { | ||
return events?.filter(({ severity, time, description }) => { | ||
const ackFromSeverity = !severityFilters?.length || severityFilters?.some(level => level === severity) | ||
const ackFromTime = !dateFilters?.length || dateFilters?.some(isoDateStr => moment(time).isSame(isoDateStr, 'day')) | ||
const ackFromMessage = !messageFilters?.length || messageFilters?.some(str => description?.includes(str)) | ||
return ackFromSeverity && ackFromTime && ackFromMessage | ||
}) | ||
} | ||
|
||
const composeSeverity = (msg, locale) => { | ||
return { | ||
id: SEVERITY, | ||
title: msg.severity(), | ||
placeholder: msg.eventsFilterTypePlaceholderSeverity(), | ||
filterValues: Object.entries( | ||
Object.keys(EVENT_SEVERITY) | ||
.map((status) => ({ title: enumMsg('EventSeverity', status, msg), id: status })) | ||
.reduce((acc, { title, id }) => { | ||
acc[title] = { ...acc[title], [id]: id } | ||
return acc | ||
}, {})) | ||
.map(([title, ids]) => ({ title, ids })) | ||
.sort((a, b) => localeCompare(a.title, b.title, locale)), | ||
} | ||
} | ||
|
||
const EventFilters = ({ msg, locale, selectedFilters = {}, onFilterUpdate }) => { | ||
const filterTypes = useMemo(() => [ | ||
composeSeverity(msg, locale), | ||
{ | ||
id: DATE, | ||
title: msg.date(), | ||
datePicker: true, | ||
}, | ||
{ | ||
id: MESSAGE, | ||
title: msg.message(), | ||
placeholder: msg.eventsFilterTypePlaceholderMessage(), | ||
}, | ||
], [msg, locale]) | ||
return ( | ||
<Filters | ||
selectedFilters={selectedFilters} | ||
onFilterUpdate={onFilterUpdate} | ||
filterTypes={filterTypes} | ||
textBasedFilterId={MESSAGE} | ||
/> | ||
) | ||
} | ||
|
||
EventFilters.propTypes = { | ||
selectedFilters: PropTypes.object, | ||
onFilterUpdate: PropTypes.func.isRequired, | ||
msg: PropTypes.object.isRequired, | ||
locale: PropTypes.string.isRequired, | ||
} | ||
|
||
export default connect( | ||
({ userMessages }) => ({ | ||
selectedFilters: toJS(userMessages.get('eventFilters')), | ||
}), | ||
(dispatch) => ({ | ||
onFilterUpdate: (filters) => dispatch(saveEventFilters({ filters })), | ||
}) | ||
)(withMsg(EventFilters)) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,44 @@ | ||
import React from 'react' | ||
import PropTypes from 'prop-types' | ||
import { connect } from 'react-redux' | ||
import { Sort } from '_/components/Toolbar' | ||
import { setEventSort } from '_/actions' | ||
import { withMsg } from '_/intl' | ||
import { toJS } from '_/helpers' | ||
|
||
import { SEVERITY, DATE, MESSAGE } from './EventFilters' | ||
|
||
export const SortFields = { | ||
[SEVERITY]: { | ||
id: SEVERITY, | ||
messageDescriptor: { id: 'severity' }, | ||
}, | ||
[DATE]: { | ||
id: DATE, | ||
messageDescriptor: { id: 'date' }, | ||
}, | ||
[MESSAGE]: { | ||
id: MESSAGE, | ||
messageDescriptor: { id: 'message' }, | ||
}, | ||
} | ||
|
||
const EventSort = ({ sort = { ...SortFields[DATE], isAsc: false }, onSortChange }) => <Sort sort={sort} onSortChange={onSortChange} SortFields={SortFields}/> | ||
|
||
EventSort.propTypes = { | ||
sort: PropTypes.shape({ | ||
id: PropTypes.string.isRequired, | ||
messageDescriptor: PropTypes.object.isRequired, | ||
isAsc: PropTypes.bool, | ||
}), | ||
onSortChange: PropTypes.func.isRequired, | ||
} | ||
|
||
export default connect( | ||
({ userMessages }) => ({ | ||
sort: toJS(userMessages.get('eventSort')), | ||
}), | ||
(dispatch) => ({ | ||
onSortChange: (sort) => dispatch(setEventSort({ sort })), | ||
}) | ||
)(withMsg(EventSort)) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
import React from 'react' | ||
import PropTypes from 'prop-types' | ||
import { ExclamationTriangleIcon, ExclamationCircleIcon } from '@patternfly/react-icons/dist/esm/icons' | ||
|
||
export const EventStatus = ({ severity }) => { | ||
switch (severity) { | ||
case 'error': | ||
return <ExclamationCircleIcon color='#c9190b'/> | ||
case 'warning': | ||
return <ExclamationTriangleIcon color='#f0ab00'/> | ||
default: | ||
return null | ||
} | ||
} | ||
|
||
EventStatus.propTypes = { | ||
severity: PropTypes.oneOf(['error', 'warning', 'normal']), | ||
} |
Oops, something went wrong.