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

Use correct k8 client #563

Merged
merged 2 commits into from
May 18, 2023
Merged

Conversation

sonjaer
Copy link
Contributor

@sonjaer sonjaer commented May 16, 2023

TL;DR

See issue flyteorg/flyte#3688

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

fixes https://github.com/flyteorg/flyte/issues/

Follow-up issue

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

Signed-off-by: Sonja Ericsson <[email protected]>
@sonjaer sonjaer force-pushed the use-correct-k8-client branch from d05650d to 132cc45 Compare May 16, 2023 15:59
Signed-off-by: Sonja Ericsson <[email protected]>
@sonjaer sonjaer marked this pull request as ready for review May 16, 2023 17:51
@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Merging #563 (dbee96c) into master (f97ff74) will increase coverage by 0.42%.
The diff coverage is 75.00%.

❗ Current head dbee96c differs from pull request most recent head f7a4be2. Consider uploading reports for the commit f7a4be2 to get more accurate results

@hamersaw hamersaw merged commit 33ca38c into flyteorg:master May 18, 2023
eapolinario pushed a commit to eapolinario/flytepropeller that referenced this pull request Aug 9, 2023
* Use correct k8 client

Signed-off-by: Sonja Ericsson <[email protected]>

* Use correct k8 client

Signed-off-by: Sonja Ericsson <[email protected]>

---------

Signed-off-by: Sonja Ericsson <[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.

2 participants