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

Pick up latest plugins resource resolution & implement changes #333

Merged
merged 10 commits into from
Oct 11, 2021

Conversation

katrogan
Copy link
Contributor

@katrogan katrogan commented Oct 8, 2021

TL;DR

Pick up latest plugins resource resolution and implements changes as required by latest TaskExecutionMetadata interface

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

Testing documented in plugins PR description: flyteorg/flyteplugins#215

Tracking Issue

flyteorg/flyte#1450
flyteorg/flyte#1452

Follow-up issue

NA

Signed-off-by: Katrina Rogan <[email protected]>
Signed-off-by: Katrina Rogan <[email protected]>
Signed-off-by: Katrina Rogan <[email protected]>
Signed-off-by: Katrina Rogan <[email protected]>
Signed-off-by: Katrina Rogan <[email protected]>
Signed-off-by: Katrina Rogan <[email protected]>
Signed-off-by: Katrina Rogan <[email protected]>
@katrogan katrogan changed the title wip: Pick up latest plugins resource resolution & implement changes Pick up latest plugins resource resolution & implement changes Oct 8, 2021
@codecov
Copy link

codecov bot commented Oct 8, 2021

Codecov Report

Merging #333 (02d5b1a) into master (27f2f56) will decrease coverage by 0.02%.
The diff coverage is 89.47%.

Signed-off-by: Katrina Rogan <[email protected]>
EngHabu
EngHabu previously approved these changes Oct 11, 2021
Signed-off-by: Katrina Rogan <[email protected]>
Signed-off-by: Haytham Abuelfutuh <[email protected]>
@katrogan katrogan merged commit 1e0b62b into master Oct 11, 2021
bimtauer pushed a commit to bimtauer/flytepropeller that referenced this pull request Oct 20, 2021
eapolinario pushed a commit to eapolinario/flytepropeller that referenced this pull request Aug 9, 2023
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