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

SageMaker on Flyte: installing SageMaker plugins [Alpha] #163

Merged
merged 11 commits into from
Jul 31, 2020

Conversation

bnsblue
Copy link
Contributor

@bnsblue bnsblue commented Jun 27, 2020

TL;DR

This PR enables flyteplugins v0.4.0 for SageMaker support on flytepropeller and add config section needed by SageMaker to the sample config file

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

This PR enables flyteplugins v0.4.0 for SageMaker support and add config section needed by SageMaker to the sample config file

Tracking Issue

flyteorg/flyte#255

Follow-up issue

flyteorg/flyte#431

@bnsblue bnsblue requested review from EngHabu and kumare3 as code owners June 27, 2020 04:34
@codecov-commenter
Copy link

codecov-commenter commented Jun 27, 2020

Codecov Report

Merging #163 into master will not change coverage.
The diff coverage is n/a.

@bnsblue bnsblue changed the title SageMaker on Flyte: installing SageMaker plugins [Alpha] #minor SageMaker on Flyte: installing SageMaker plugins [Alpha] Jul 31, 2020
@bnsblue bnsblue merged commit f9819ab into master Jul 31, 2020
eapolinario pushed a commit to eapolinario/flytepropeller that referenced this pull request Aug 9, 2023
* install the sagemaker plugin

* upgrade flyteplugin

* upgrade dependency on flyteplugins

* upgrade flyteplugins

* upgrade flyteplugins

* go mod

* go mod

* go mod

* add to sandbox config and pull latest flyteidl and flyteplugins

* flyteplugins v0.4.0
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