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

Demystify with additional conditions #163

Merged
merged 1 commit into from
Mar 16, 2021
Merged

Demystify with additional conditions #163

merged 1 commit into from
Mar 16, 2021

Conversation

kumare3
Copy link
Contributor

@kumare3 kumare3 commented Mar 16, 2021

Signed-off-by: Ketan Umare [email protected]

TL;DR

In the case when a configmap/secret is not found, the pod stays in pending. this PR ensures that such a pod fails

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

Tracking Issue

flyteorg/flyte#800

 - In the case when a configmap/secret is not found, the pod stays in
pending

Signed-off-by: Ketan Umare <[email protected]>
@codecov
Copy link

codecov bot commented Mar 16, 2021

Codecov Report

Merging #163 (f28d579) into master (4f22a59) will increase coverage by 0.07%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #163      +/-   ##
==========================================
+ Coverage   60.33%   60.40%   +0.07%     
==========================================
  Files         130      130              
  Lines        7027     7027              
==========================================
+ Hits         4240     4245       +5     
+ Misses       2362     2357       -5     
  Partials      425      425              
Flag Coverage Δ
unittests 60.40% <100.00%> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
go/tasks/pluginmachinery/flytek8s/pod_helper.go 64.16% <100.00%> (+4.16%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f22a59...f28d579. Read the comment docs.

@kumare3 kumare3 merged commit 030cdef into master Mar 16, 2021
milton0825 pushed a commit to milton0825/flyteplugins that referenced this pull request May 25, 2021
- In the case when a configmap/secret is not found, the pod stays in
pending

Signed-off-by: Ketan Umare <[email protected]>
Signed-off-by: Chao-Han Tsai <[email protected]>
eapolinario pushed a commit that referenced this pull request Sep 6, 2023
- In the case when a configmap/secret is not found, the pod stays in
pending

Signed-off-by: Ketan Umare <[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