[Code] Persist clone error messages

x-pack/plugins/code/model/repository.ts

@@ -117,6 +117,7 @@ export interface WorkerProgress {
   progress: number;
   timestamp: Date;
   revision?: string;
+  errorMessage?: string;
 }
 
 export interface CloneProgress {

x-pack/plugins/code/public/components/admin_page/project_item.tsx

@@ -12,6 +12,7 @@ import {
   EuiProgress,
   EuiText,
   EuiTextColor,
+  EuiToolTip,
 } from '@elastic/eui';
 import moment from 'moment';
 import React from 'react';
@@ -82,7 +83,14 @@ class CodeProjectItem extends React.PureComponent<{
       footer = <ErrorFooter>ERROR INDEX REPO</ErrorFooter>;
       hasError = true;
     } else if (status.state === RepoState.CLONE_ERROR) {
-      footer = <ErrorFooter>ERROR CLONE REPO</ErrorFooter>;
+      footer = (
+        <ErrorFooter>
+          ERROR CLONE REPO&nbsp;
+          <EuiToolTip position="top" content={status.errorMessage}>
+            <EuiIcon type="iInCircle" />
+          </EuiToolTip>
+        </ErrorFooter>
+      );
       // Disable repo link is clone failed.
       disableRepoLink = true;
       hasError = true;
@@ -104,7 +112,8 @@ class CodeProjectItem extends React.PureComponent<{
       status &&
       status.state !== RepoState.CLONING &&
       status.state !== RepoState.DELETING &&
-      status.state !== RepoState.INDEXING;
+      status.state !== RepoState.INDEXING &&
+      status.state !== RepoState.CLONE_ERROR;
     const indexVisibility = indexShow ? 'visible' : 'hidden';
 
     const deleteShow = status && status.state !== RepoState.DELETING;

x-pack/plugins/code/public/reducers/status.ts

@@ -33,6 +33,7 @@ export interface RepoStatus {
   cloneProgress?: any;
   timestamp?: Date;
   state?: RepoState;
+  errorMessage?: string;
 }
 
 export interface StatusState {

x-pack/plugins/code/server/indexer/schema/document.ts

@@ -100,6 +100,9 @@ export const DocumentSchema = {
       revision: {
         type: 'keyword',
       },
+      errorMessage: {
+        type: 'text',
+      },
       cloneProgress: {
         properties: {
           isCloned: {

x-pack/plugins/code/server/queue/abstract_git_worker.ts

@@ -68,20 +68,26 @@ export abstract class AbstractGitWorker extends AbstractWorker {
     }
   }
 
-  public async updateProgress(job: Job, progress: number, cloneProgress?: CloneProgress) {
+  public async updateProgress(
+    job: Job,
+    progress: number,
+    error?: Error,
+    cloneProgress?: CloneProgress
+  ) {
     const { uri } = job.payload;
     const p: CloneWorkerProgress = {
       uri,
       progress,
       timestamp: new Date(),
       cloneProgress,
+      errorMessage: error ? error.message : undefined,
     };
     try {
       return await this.objectClient.updateRepositoryGitStatus(uri, p);
-    } catch (error) {
+    } catch (err) {
       // This is a warning since it's not blocking anything.
       this.log.warn(`Update git clone progress error.`);
-      this.log.warn(error);
+      this.log.warn(err);
     }
   }
 }

x-pack/plugins/code/server/queue/abstract_worker.ts

@@ -121,7 +121,7 @@ export abstract class AbstractWorker implements Worker {
         res.job
       )} seconds.`
     );
-    return await this.updateProgress(res.job, WorkerReservedProgress.ERROR);
+    return await this.updateProgress(res.job, WorkerReservedProgress.ERROR, res.error);
   }
 
   public async onJobTimeOut(res: any) {
@@ -130,10 +130,10 @@ export abstract class AbstractWorker implements Worker {
         res.job
       )} seconds.`
     );
-    return await this.updateProgress(res.job, WorkerReservedProgress.TIMEOUT);
+    return await this.updateProgress(res.job, WorkerReservedProgress.TIMEOUT, res.error);
   }
 
-  public async updateProgress(job: Job, progress: number) {
+  public async updateProgress(job: Job, progress: number, error?: Error) {
     // This is an abstract class. Do nothing here. You should override this.
     return new Promise<WorkerResult>((resolve, _) => {
       resolve();

x-pack/plugins/code/server/queue/clone_worker.ts

@@ -63,7 +63,7 @@ export class CloneWorker extends AbstractGitWorker {
     const repo = RepositoryUtils.buildRepository(url);
     return await repoService.clone(repo, (progress: number, cloneProgress?: CloneProgress) => {
       job.payload.uri = repo.uri;
-      this.updateProgress(job, progress, cloneProgress);
+      this.updateProgress(job, progress, undefined, cloneProgress);
     });
   }
 

x-pack/plugins/code/server/repository_service.ts

@@ -227,7 +227,7 @@ export class RepositoryService {
         const msg = `Clone repository from ${repo.url} error.`;
         this.log.error(msg);
         this.log.error(error);
-        throw new Error(msg);
+        throw new Error(error.message);
       }
     }
   }