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

SUBMARINE-814. Install local pytorchjob chart via helm golang API#577

Merged
kevin85421 merged 1 commit intoapache:masterfrom
Kenchu123:SUBMARINE-814
May 9, 2021
Merged

SUBMARINE-814. Install local pytorchjob chart via helm golang API#577
kevin85421 merged 1 commit intoapache:masterfrom
Kenchu123:SUBMARINE-814

Conversation

@Kenchu123
Copy link
Copy Markdown
Contributor

What is this PR for?

pytorch chart:

https://github.com/apache/submarine/tree/master/helm-charts/submarine/charts/pytorchjob

Helm golang API:

https://github.com/apache/submarine/blob/master/submarine-cloud-v2/pkg/helm/helm.go

What type of PR is it?

[Feature]

Todos

What is the Jira issue?

https://issues.apache.org/jira/projects/SUBMARINE/issues/SUBMARINE-814

How should this be tested?

https://www.travis-ci.com/github/Kenchu123/submarine/builds/224734346

Screenshots (if appropriate)

By operator:

helm

helm

pods

deployments

services

replicaset

Questions:

  • Do the license files need updating? No
  • Are there breaking changes for older versions? No
  • Does this need new documentation? No

Copy link
Copy Markdown
Member

@xunliu xunliu left a comment

Choose a reason for hiding this comment

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

@Kenchu123 Please fixed conflicts. Thanks.

@Kenchu123 Kenchu123 force-pushed the SUBMARINE-814 branch 2 times, most recently from 36e2731 to c5ba3fd Compare May 6, 2021 14:44
@kevin85421
Copy link
Copy Markdown
Member

LGTM

@kevin85421 kevin85421 merged commit 244fca9 into apache:master May 9, 2021
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.

3 participants