Skip to content

Commit

Permalink
fix(action): populate all fields of Job from CronJob (#15259) (#15727) (
Browse files Browse the repository at this point in the history
#15730)

Signed-off-by: sergey.ladutko <[email protected]>
Co-authored-by: SergeyLadutko <[email protected]>
Co-authored-by: sergey.ladutko <[email protected]>
  • Loading branch information
3 people authored Sep 29, 2023
1 parent b3fabc2 commit ea34029
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -47,11 +47,7 @@ ownerRef.uid = obj.metadata.uid
job.metadata.ownerReferences = {}
job.metadata.ownerReferences[1] = ownerRef

job.spec = {}
job.spec.suspend = false
job.spec.template = {}
job.spec.template.metadata = deepCopy(obj.spec.jobTemplate.spec.template.metadata)
job.spec.template.spec = deepCopy(obj.spec.jobTemplate.spec.template.spec)
job.spec = deepCopy(obj.spec.jobTemplate.spec)

local impactedResource = {}
impactedResource.operation = "create"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ spec:
annotations:
my: annotation
spec:
ttlSecondsAfterFinished: 100
template:
metadata:
labels:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
annotations:
my: annotation
spec:
ttlSecondsAfterFinished: 100
template:
metadata:
labels:
Expand Down

0 comments on commit ea34029

Please sign in to comment.