Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

MPI Operator plugin interface #217

Merged
merged 4 commits into from
Oct 5, 2021
Merged

MPI Operator plugin interface #217

merged 4 commits into from
Oct 5, 2021

Conversation

yindia
Copy link
Contributor

@yindia yindia commented Aug 27, 2021

TL;DR

  • Added Plugin contract for MPI

Dependent PR

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

How did you fix the bug, make the feature etc. Link to any design docs etc

Tracking Issue

Remove the 'fixes' keyword if there will be multiple PRs to fix the linked issue

flyteorg/flyte#1543

Follow-up issue

NA
OR
https://github.com/flyteorg/flyte/issues/

yindia added 2 commits August 27, 2021 18:45
Signed-off-by: Yuvraj <[email protected]>
Signed-off-by: Yuvraj <[email protected]>
@yindia yindia marked this pull request as ready for review August 30, 2021 12:04
@yindia yindia requested review from katrogan, EngHabu and kumare3 August 30, 2021 12:04
@yindia yindia mentioned this pull request Aug 30, 2021
8 tasks
@yindia yindia requested a review from katrogan September 2, 2021 21:15
Copy link
Contributor

@katrogan katrogan left a comment

Choose a reason for hiding this comment

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

thank you for adding comments!

@kumare3
Copy link
Contributor

kumare3 commented Oct 1, 2021

can you create an issue for this, then +1

@kumare3 kumare3 changed the title Added Mpi plugin MPI Operator plugin interface Oct 1, 2021
@yindia yindia removed the request for review from EngHabu October 4, 2021 03:18
@EngHabu EngHabu merged commit 1967714 into flyteorg:master Oct 5, 2021
eapolinario pushed a commit that referenced this pull request Sep 8, 2023
* Added mpi plugin

Signed-off-by: Yuvraj <[email protected]>

* Rename variable name

Signed-off-by: Yuvraj <[email protected]>

* Added docs for mpi

Signed-off-by: Yuvraj <[email protected]>
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