[mikrotik-sdwan] add ping/fetch tests and use worker tasks

Author: geoffhouse

TODO.md

@@ -2,6 +2,8 @@
 
 # Do next
 
+- add link to modules in system/modules to the documentation page
+
 # Backlog
 
 - sort out overlaying menu in bug api table
@@ -16,4 +18,6 @@
 - i need a flag to prevent the module ui from loading - overlayPanel or something? When a status item is so important the panel can't run.
 - check upgrade - the panel enabled should be false and it takes a long time to update
 - use logger rather than console.log
-- fix reload in logs
+- new worker design using tasks
+- update mikrotik modules using new core class
+- mikrotik routes module doesn't understand sub routing tables

src/modules/mikrotik-sdwan/TODO.md

@@ -4,3 +4,12 @@
 - need to handle no data better - critical
 - add test for log route
 - fetchFirewall isn't used!
+- if no routing table entries then hide tab?
+- clear pinger collection in db
+
+- thurs 12/02
+
+Working on default routes.
+Kind of done ping/fetch tests but needs historical data storing and showing in sparkline
+Using comments for bridges maybe isn't the best idea - what about internet routes with client IPs?
+And am I providing functionality to store static IPs and routes?

src/modules/mikrotik-sdwan/client/Module.jsx

@@ -12,6 +12,8 @@ export default function Module(props) {
         <BugModuleWrapper {...props}>
             <Routes>
                 <Route index element={<MainPanel {...props} />} />
+                <Route path="entries" element={<MainPanel {...props} />} />
+                <Route path="routes" element={<MainPanel {...props} />} />
                 <Route path="add" element={<AddPanel {...props} />} />
                 <Route
                     path="config"

src/modules/mikrotik-sdwan/client/components/EntryList.jsx

@@ -18,7 +18,7 @@ import { useEffect } from "react";
 import AddEntryButton from "./AddEntryButton";
 import SetGroupDialog from "./SetGroupDialog";
 
-const EntryStatus = ({ entry, routes }) => {
+const EntryStatus = ({ entry }) => {
     if (entry.static) {
         return <BugStatusLabel>STATIC</BugStatusLabel>;
     }
@@ -31,13 +31,18 @@ const EntryStatus = ({ entry, routes }) => {
     return <BugStatusLabel>UNKNOWN</BugStatusLabel>;
 };
 
-export default function EntryList({ panelId, routes }) {
+export default function EntryList({ panelId }) {
     const sendAlert = useAlert();
     const [forceRefresh, doForceRefresh] = useForceRefresh();
     const { renameDialog } = useBugRenameDialog();
     const { customDialog } = useBugCustomDialog();
     const { confirmDialog } = useBugConfirmDialog();
 
+    const routes = useApiPoller({
+        url: `/container/${panelId}/route`,
+        interval: 2000,
+    });
+
     const entries = useApiPoller({
         url: `/container/${panelId}/entry`,
         interval: 2000,
@@ -222,9 +227,9 @@ export default function EntryList({ panelId, routes }) {
     }
 
     return (
-        <Box sx={{ p: 0.5 }}>
+        <Stack spacing={1}>
             {entries?.data?.map((group) => (
-                <Box key={group.group} sx={{ mb: 1 }}>
+                <Box key={group.group} sx={{ mb: 0 }}>
                     <Typography
                         sx={{
                             p: 2,
@@ -351,6 +356,6 @@ export default function EntryList({ panelId, routes }) {
                     })}
                 </Box>
             ))}
-        </Box>
+        </Stack>
     );
 }