add comment

Author: HaoyL666

app/charts/GrafanaEventChart.tsx

@@ -7,10 +7,6 @@ interface EventChartProps {
     token: string;
 }
 
-// interface SoloStyles {
-//     height: number;
-//     width: number;
-// }
 
 type TimeFrame = '5m' | '15m' | '30m' | '1h' | '2h' | '1d' | '2d';
 
@@ -27,16 +23,10 @@ const GrafanaEventChart: React.FC<EventChartProps> = React.memo(props => {
     const [type, setType] = useState(['timeserie']);
     const [timeFrame, setTimeFrame] = useState('5m');
 
-    // const [solo, setSolo] = useState<SoloStyles | null>(null);
     console.log("graphType: ", graphType)
     console.log("type: ", type)
     console.log("inside GrafanaEventChart")
 
-    // setInterval(() => {
-    //     if (solo !== soloStyle) {
-    //         setSolo(soloStyle);
-    //     }
-    // }, 20);
     console.log("metricName: ", metricName)
     let uid = metricName.replace(/.*\/.*\//g, '')
     if (uid.length >= 40) {
@@ -47,8 +37,8 @@ const GrafanaEventChart: React.FC<EventChartProps> = React.memo(props => {
     console.log("uid: ", uid)
     console.log("parsedName: ", parsedName)
 
+    // make request to update dashboard
     const handleSelectionChange = async (event) => {
-        //setGraphType(event.target.value);
         setType([...type, graphType]);
         await fetch('http://localhost:1111/api/updateDashboard', {
             method: 'POST',

app/containers/EventContainer.tsx

@@ -135,6 +135,7 @@ const EventContainer: React.FC<EventContainerProps> = React.memo(props => {
         // }
       }
     }
+    // currently, we only display graph using grafana. It can be implement as a option to choose between ploty and grafana for future iterations
     // if (isGrafana) {
     console.log(grafanaChartsArray)
     setEventChartsArr(grafanaChartsArray);

app/containers/Inspect.jsx

@@ -6,10 +6,6 @@ import '../stylesheets/Inspect.scss';
 import { CSS2DRenderer, CSS2DObject } from 'three/examples/jsm/renderers/CSS2DRenderer.js';
 
 
-
-// currently, Insepect only works statically, but it should be possible to make it dynamic. To do so, we need to make get kubernetes resources information from the backend, and then pass it to the graph.
-// It will be great if future iteration group can make it dynamic.
-
 const Inspect = () => {
     useEffect(() => {
         // controls
@@ -124,7 +120,6 @@ const Inspect = () => {
         <h2>Infrastructure</h2>
         <div id="graph" />;
     </div>
-    // <div id="graph" />;
 };
 
 export default Inspect;

chronos_npm_package/server/output.csv

@@ -2,10 +2,10 @@ size,path
 "",Namespace: default
 "",Namespace: default/Service: grafana/Pod: grafana-85d768d55b-jkr84
 "",Namespace: default/Service: grafana
-"",Namespace: default/Service: frontend/Pod: nginx-frontend-55b44fbc7-45xp7
-"",Namespace: default/Service: frontend/Pod: nginx-frontend-55b44fbc7-5n2dh
+"",Namespace: default/Service: frontend/Pod: nginx-frontend-55b44fbc7-mzbp2
+"",Namespace: default/Service: frontend/Pod: nginx-frontend-55b44fbc7-vmxpm
 "",Namespace: default/Service: frontend
-"",Namespace: default/Service: backend/Pod: node-backend-d597768c-z7mkw
+"",Namespace: default/Service: backend/Pod: node-backend-d597768c-944xr
 "",Namespace: default/Service: backend
-"",Namespace: default/Service: prometheus-service/Pod: prometheus-deployment-96898bbc9-qg7tl
+"",Namespace: default/Service: prometheus-service/Pod: prometheus-deployment-96898bbc9-2jp9j
 "",Namespace: default/Service: prometheus-service

chronos_npm_package/server/server.js

@@ -60,13 +60,6 @@ app.get('/api/kuberData', kuberControllers.getResources, fileControllers.saveCSV
 });
 
 
-
-
-
-
-
-
-
 app.use('*', (req, res) => {
     res.status(404).send('Not Found');
 });