pattnaik-soumitri / devkube Goto Github PK

Right now the upon clicking the refresh button, only the resources are fetched kubectl get all -o wide.

• Call the getContexts() method as well.
• Refactor the entire refresh functionality into it's own method.
• Change event name from getPods to getResources.

Loading animation goes forever when trying to run with MiniKube

The application currently only works on Windows and Mac. Add Linux support to make it cross-platform.

electron-builder (see here) is more advanced and comes with auto-update feature out of the box. So switch out electron-packager with electron-builder.

For electron packager, add app icon

Add a log/terminal section to the UI where it should show what command is running and what output it's getting.

Move electron and electron-packager to dev-dependencies from dependencies in the package.json file.

-"dependencies": {
-    "electron": "^11.2.0",
-    "electron-packager": "^15.2.0"
-}
+"devDependencies": {
+    "electron": "^11.2.0",
+    "electron-packager": "^15.2.0"
+}

Rather than having to deal with DOM Javascript code, switch to a component-based framework, preferably vue js.

Move all the kubectl commands to a commands constant file.

Currently, the files are not organized into any sort of directory structure.

1. Create a directory structure and move the files.
2. Refactor the imports accordingly.

The deployment and statefulset tables have incorrect content in the shell.

• Add gifs of the application in action.
• Specify Linux support is coming soon.
• Document the manual update process.
• Document kubectl version compatibility.
• Leave links to the kubectl installation documentation.

Substring the node name based on special characters.

Instead of running and parsing kubectl command/responses hit kubectl REST API to perform operations.

The application should react and notify the user about a failure in the kubectl commands.

This issue should fix this (Deployment table has incorrect content in the shell. #8) bug.

Current Approach

As of now the way the resource parsing is hardcoded index based. See below code.

static fromOutputRow(outputRow) {
        let arr = outputRow.split(" ").filter(e => e !== " " && e !== "");

        // The model class
        let pod = new Pod();
        pod.name = arr[0].split("pod/")[1]; // name
        pod.ready = arr[1]; //ready
        pod.status = arr[2]; // status
        pod.restarts = arr[3]; // restarts
        pod.age = arr[4]; // age
        pod.ip = arr[5]; // ip
        pod.node = arr[6]; // node
        
        return pod;
}

The problem

• As kubectl updates the output may change.
• Different users may have different versions of kubectl installed, where this static parsing will result in inconsistent/incorrect output columns.

The solution

• Get rid of the model classed.
• Change resource parsing from index-based to header-based parsing.

The problem

As of now, DevKube relies on pre-installed kubectl. Which has primarily few drawbacks.

• It can be out of date and thus missing features, bug fixes, and performance enhancements.
• As the client system's kubectl version can be anything, we will have to build DevKube based on a minimum kubectl version. In which we will lose out on new features.

The solution

• Shipp kubectl binaries along with DevKube. Can be a post-installation step.

Discovery

• Check if this approach is feasible.