caicloud / ciao

Kernel for Kubeflow in Jupyter Notebook
Apache License 2.0
67 stars 18 forks source link

fix: Fix job label #80

Closed minshenglin closed 5 years ago

minshenglin commented 5 years ago

What this PR does / why we need it:

the hyphen in job name label has been changed to use underscore, so pods will never be found.

Which issue(s) this PR is related to (optional, link to 3rd issue(s)):

Fixes #79

Special notes for your reviewer:

/cc @your-reviewer

Release note:

NONE
caicloud-bot commented 5 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: gaocegege

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/caicloud/ciao/blob/master/OWNERS)~~ [gaocegege] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 71


Totals Coverage Status
Change from base Build 70: 0.0%
Covered Lines: 357
Relevant Lines: 408

💛 - Coveralls
coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 71


Totals Coverage Status
Change from base Build 70: 0.0%
Covered Lines: 357
Relevant Lines: 408

💛 - Coveralls
gaocegege commented 5 years ago

/release-note-none

gaocegege commented 5 years ago

@minshenglin

Thanks for your contribution! :tada: :+1: