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

Show error message when setClusterMetadata in kubeflow and frameworkcontroller #1045

Closed
wants to merge 76 commits into from

Conversation

SparkSnail
Copy link
Contributor

@SparkSnail SparkSnail commented May 5, 2019

Fix issue #1036

SparkSnail and others added 23 commits March 20, 2019 10:20
Fix remote integration test pipeline (microsoft#914)
Designated gpu devices for NNI trial jobs (microsoft#991)
Copy link

@gaocegege gaocegege left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the enhancement!

} catch(error) {
this.log.error(error);
return Promise.reject(new Error(error));
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume the problem is caused by that the type of error is string? if yes, the problem has been address by this PR, #1064 , it is not necessary to handle this issue case by case.

@SparkSnail SparkSnail closed this May 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants