Initialize HelmRepoManager on demand (#2591)

src/main/helm/helm-repo-manager.ts

@@ -35,7 +35,7 @@ export class HelmRepoManager extends Singleton {
   protected helmEnv: HelmEnv;
   protected initialized: boolean;
 
-  async loadAvailableRepos(): Promise<HelmRepo[]> {
+  public static async loadAvailableRepos(): Promise<HelmRepo[]> {
     const res = await customRequestPromise({
       uri: "https://github.com/lensapp/artifact-hub-repositories/releases/download/latest/repositories.json",
       json: true,
@@ -46,18 +46,18 @@ export class HelmRepoManager extends Singleton {
     return orderBy<HelmRepo>(res.body, repo => repo.name);
   }
 
-  async init() {
+  private async init() {
     helmCli.setLogger(logger);
     await helmCli.ensureBinary();
 
     if (!this.initialized) {
-      this.helmEnv = await this.parseHelmEnv();
-      await this.update();
+      this.helmEnv = await HelmRepoManager.parseHelmEnv();
+      await HelmRepoManager.update();
       this.initialized = true;
     }
   }
 
-  protected async parseHelmEnv() {
+  protected static async parseHelmEnv() {
     const helm = await helmCli.binaryPath();
     const { stdout } = await promiseExec(`"${helm}" env`).catch((error) => {
       throw(error.stderr);
@@ -78,6 +78,10 @@ export class HelmRepoManager extends Singleton {
 
   public async repositories(): Promise<HelmRepo[]> {
     try {
+      if (!this.initialized) {
+        await this.init();
+      }
+
       const repoConfigFile = this.helmEnv.HELM_REPOSITORY_CONFIG;
       const { repositories }: HelmRepoConfig = await readFile(repoConfigFile, "utf8")
         .then((yamlContent: string) => yaml.safeLoad(yamlContent))
@@ -86,7 +90,7 @@ export class HelmRepoManager extends Singleton {
         }));
 
       if (!repositories.length) {
-        await this.addRepo({ name: "bitnami", url: "https://charts.bitnami.com/bitnami" });
+        await HelmRepoManager.addRepo({ name: "bitnami", url: "https://charts.bitnami.com/bitnami" });
 
         return await this.repositories();
       }
@@ -102,13 +106,7 @@ export class HelmRepoManager extends Singleton {
     }
   }
 
-  public async repository(name: string) {
-    const repositories = await this.repositories();
-
-    return repositories.find(repo => repo.name == name);
-  }
-
-  public async update() {
+  public static async update() {
     const helm = await helmCli.binaryPath();
     const { stdout } = await promiseExec(`"${helm}" repo update`).catch((error) => {
       return { stdout: error.stdout };
@@ -117,7 +115,7 @@ export class HelmRepoManager extends Singleton {
     return stdout;
   }
 
-  public async addRepo({ name, url }: HelmRepo) {
+  public static async addRepo({ name, url }: HelmRepo) {
     logger.info(`[HELM]: adding repo "${name}" from ${url}`);
     const helm = await helmCli.binaryPath();
     const { stdout } = await promiseExec(`"${helm}" repo add ${name} ${url}`).catch((error) => {
@@ -127,7 +125,7 @@ export class HelmRepoManager extends Singleton {
     return stdout;
   }
 
-  public async addСustomRepo(repoAttributes : HelmRepo) {
+  public static async addСustomRepo(repoAttributes : HelmRepo) {
     logger.info(`[HELM]: adding repo "${repoAttributes.name}" from ${repoAttributes.url}`);
     const helm = await helmCli.binaryPath();
 
@@ -146,7 +144,7 @@ export class HelmRepoManager extends Singleton {
     return stdout;
   }
 
-  public async removeRepo({ name, url }: HelmRepo): Promise<string> {
+  public static async removeRepo({ name, url }: HelmRepo): Promise<string> {
     logger.info(`[HELM]: removing repo "${name}" from ${url}`);
     const helm = await helmCli.binaryPath();
     const { stdout } = await promiseExec(`"${helm}" repo remove ${name}`).catch((error) => {

src/main/helm/helm-service.ts

@@ -34,7 +34,8 @@ class HelmService {
       readme: "",
       versions: {}
     };
-    const repo = await HelmRepoManager.getInstance().repository(repoName);
+    const repos = await HelmRepoManager.getInstance().repositories();
+    const repo = repos.find(repo => repo.name === repoName);
     const chartManager = new HelmChartManager(repo);
     const chart = await chartManager.chart(chartName);
 
@@ -45,7 +46,8 @@ class HelmService {
   }
 
   public async getChartValues(repoName: string, chartName: string, version = "") {
-    const repo = await HelmRepoManager.getInstance().repository(repoName);
+    const repos = await HelmRepoManager.getInstance().repositories();
+    const repo = repos.find(repo => repo.name === repoName);
     const chartManager = new HelmChartManager(repo);
 
     return chartManager.getValues(chartName, version);

src/main/index.ts

@@ -32,6 +32,7 @@ import { IpcRendererNavigationEvents } from "../renderer/navigation/events";
 import { CatalogPusher } from "./catalog-pusher";
 import { catalogEntityRegistry } from "../common/catalog-entity-registry";
 import { HotbarStore } from "../common/hotbar-store";
+import { HelmRepoManager } from "./helm/helm-repo-manager";
 
 const workingDir = path.join(app.getPath("appData"), appName);
 
@@ -104,6 +105,8 @@ app.on("ready", async () => {
   const extensionsStore = ExtensionsStore.createInstance();
   const filesystemStore = FilesystemProvisionerStore.createInstance();
 
+  HelmRepoManager.createInstance(); // create the instance
+
   logger.info("💾 Loading stores");
   // preload
   await Promise.all([