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

Updating PodTemplateStore API #516

Merged
merged 21 commits into from
Feb 7, 2023
Merged

Updating PodTemplateStore API #516

merged 21 commits into from
Feb 7, 2023

Conversation

hamersaw
Copy link
Contributor

@hamersaw hamersaw commented Jan 11, 2023

TL;DR

This PR updates the PodTemplateStore API to enable PodTemplate configuration at the task granularity.

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

^^^

Tracking Issue

flyteorg/flyte#3123

Follow-up issue

NA

@codecov
Copy link

codecov bot commented Jan 13, 2023

Codecov Report

Merging #516 (8e52767) into master (2014630) will increase coverage by 0.45%.
The diff coverage is 0.00%.

❗ Current head 8e52767 differs from pull request most recent head 7eae0f0. Consider uploading reports for the commit 7eae0f0 to get more accurate results

@hamersaw hamersaw marked this pull request as ready for review January 13, 2023 21:02
EngHabu
EngHabu previously approved these changes Feb 2, 2023
Signed-off-by: Daniel Rammer <[email protected]>
Signed-off-by: Daniel Rammer <[email protected]>
Signed-off-by: Daniel Rammer <[email protected]>
Signed-off-by: Daniel Rammer <[email protected]>
Signed-off-by: Daniel Rammer <[email protected]>
Signed-off-by: Daniel Rammer <[email protected]>
Signed-off-by: Daniel Rammer <[email protected]>
Signed-off-by: Daniel Rammer <[email protected]>
Signed-off-by: Daniel Rammer <[email protected]>
@hamersaw hamersaw merged commit 92cd1b7 into master Feb 7, 2023
@hamersaw hamersaw deleted the feature/task-pod-template branch February 7, 2023 20:11
@hamersaw hamersaw mentioned this pull request Feb 7, 2023
8 tasks
eapolinario pushed a commit to eapolinario/flytepropeller that referenced this pull request Aug 9, 2023
* updated pod template store handling

Signed-off-by: Daniel Rammer <[email protected]>

* updated flyteidl

Signed-off-by: Daniel Rammer <[email protected]>

* updating flyteplugins

Signed-off-by: Daniel Rammer <[email protected]>

* updated flyteidl and flyteplugins versions

Signed-off-by: Daniel Rammer <[email protected]>

* updated flyteidl and flyteplugins

Signed-off-by: Daniel Rammer <[email protected]>

* bumping go version to 1.19

Signed-off-by: Daniel Rammer <[email protected]>

* fixed linter

Signed-off-by: Daniel Rammer <[email protected]>

* bumping setup-go action

Signed-off-by: Daniel Rammer <[email protected]>

* using different flytetools branch

Signed-off-by: Daniel Rammer <[email protected]>

* fixed go generate

Signed-off-by: Daniel Rammer <[email protected]>

* using master branch of flytetools

Signed-off-by: Daniel Rammer <[email protected]>

* removed secrets from unit tests

Signed-off-by: Daniel Rammer <[email protected]>

* maybe surround go-version in quotes

Signed-off-by: Daniel Rammer <[email protected]>

* specific 1.19.5 version

Signed-off-by: Daniel Rammer <[email protected]>

* moved go-version before secrets

Signed-off-by: Daniel Rammer <[email protected]>

* OMG

Signed-off-by: Daniel Rammer <[email protected]>

* updated flyte deps

Signed-off-by: Daniel Rammer <[email protected]>

---------

Signed-off-by: Daniel Rammer <[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.

3 participants