Skip to content
This repository was archived by the owner on Sep 18, 2024. It is now read-only.

Commit

Permalink
[Kubeflow training service] fix bug that wrongly split kube delete cm…
Browse files Browse the repository at this point in the history
…d into 2 lines (#425)

* [Kubeflow training service] fix bug that wrongly split kube delete cmd into 2 lines

* Adjust white space
  • Loading branch information
yds05 authored Nov 30, 2018
1 parent bc44c74 commit 5426cfe
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -282,8 +282,8 @@ class KubeflowTrainingService implements TrainingService {
return Promise.reject(errorMessage);
}

const result: cpp.childProcessPromise.Result = await cpp.exec(`kubectl delete
${this.kubeflowJobPlural} -l app=${this.NNI_KUBEFLOW_TRIAL_LABEL},expId=${getExperimentId()},trialId=${trialJobId}`);
const result: cpp.childProcessPromise.Result = await cpp.exec(`kubectl delete ${this.kubeflowJobPlural} -l `
+ `app=${this.NNI_KUBEFLOW_TRIAL_LABEL},expId=${getExperimentId()},trialId=${trialJobId}`);
if(result.stderr) {
const errorMessage: string = `kubectl delete ${this.kubeflowJobPlural} for trial ${trialJobId} failed: ${result.stderr}`;
this.log.error(errorMessage);
Expand Down Expand Up @@ -318,7 +318,7 @@ class KubeflowTrainingService implements TrainingService {
this.log.error(mountError);
throw new Error(mountError);
}
}else if(this.kubeflowClusterConfig.keyVault && this.kubeflowClusterConfig.azureStorage){
} else if(this.kubeflowClusterConfig.keyVault && this.kubeflowClusterConfig.azureStorage){
const vaultName = this.kubeflowClusterConfig.keyVault.vaultName;
const valutKeyName = this.kubeflowClusterConfig.keyVault.name;
this.azureStorageAccountName = this.kubeflowClusterConfig.azureStorage.accountName;
Expand All @@ -340,7 +340,7 @@ class KubeflowTrainingService implements TrainingService {
+ `--from-literal=azurestorageaccountname=${this.azureStorageAccountName} `
+ `--from-literal=azurestorageaccountkey=${storageAccountKey}`)

}catch(error){
} catch(error){
this.log.error(`command error: ${error}`);
throw new Error(error);
}
Expand Down

0 comments on commit 5426cfe

Please sign in to comment.