Skip to content
Snippets Groups Projects

CE-2876: Load host status asynchronously in host table

Merged CE-2876: Load host status asynchronously in host table
4 unresolved threads
Merged Domonkos Gulyas requested to merge CE-2876_Load-host-status-async into develop
4 unresolved threads

Load host status asynchronously in host table. Using similar approach to the async status loading in the IOCTable.

Changing tests accordingly.

Minor optimization on fetching logs.

Closes CE-2876

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
9 9 } from "@mui/icons-material";
10 10 import { LabeledIcon } from "@ess-ics/ce-ui-common";
11 11
12 export function HostStatusIcon({ host }) {
  • 16 16 { field: "scope", headerName: "Network scope" }
    17 17 ];
    18 18
    19 export function createRow(hostContainer) {
    20 const host = hostContainer.netBoxHost;
    19 export function createRow(host) {
    21 20 return {
    22 21 id: host.hostId,
    23 bulb: <HostStatusIcon host={hostContainer} />,
  • Johanna Szepanski
    Johanna Szepanski @johannaszepanski started a thread on an outdated change in commit 98976438
  • 34 33 description: <EllipsisText>{host.description}</EllipsisText>,
    35 34 network: <EllipsisText>{host.network}</EllipsisText>,
    36 35 scope: <EllipsisText>{host.scope}</EllipsisText>,
    37 discrepancy: hostContainer.alertSeverity === "WARNING",
    38 inconsistentState: hostContainer.alertSeverity === "ERROR",
    39 36 shortenLongData: true,
  • 1 1 import React from "react";
  • Out of scope for this task, but it seems like The IOC status column and the host table status column has a lot in common. A follow up could be to see if we can refactor those two into a reusable component.

  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    • 26784cee - fixed a broken host table test

    Compare with previous version

  • Johanna Szepanski approved this merge request

    approved this merge request

  • mentioned in commit 3dbfeb2d

  • Please register or sign in to reply
    Loading