chore(operator): refactor KeptnTask controller logic

[odubajDT]

Fixes: #1500

Changes:

• split JobRunner interface function CreateContainerWithVolumes to two separate ones retrieving containers and volumes separately
• remove unneeded k8s API calls from KeptnTask reconcile loop
• refactored job-utils to use the newly adapted interface
• set TaskType as part of TaskContext
• refactored ContainerBuilder to use the updated JobRunner interface
• renamed DunctionBuilder to RuntimeBuilder
• refactored RuntimeBuilder to implement the updated JobRunner interface

[netlify]

✅ Deploy Preview for keptn-lifecycle-toolkit ready!

Name	Link
🔨 Latest commit	136d72e
🔍 Latest deploy log	https://app.netlify.com/sites/keptn-lifecycle-toolkit/deploys/64872287d414e20008547c6c
😎 Deploy Preview	https://deploy-preview-1536--keptn-lifecycle-toolkit.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[codecov]

Codecov Report

Merging #1536 (c9f62b9) into main (908b03d) will decrease coverage by 0.07%.
The diff coverage is 79.36%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1536      +/-   ##
==========================================
- Coverage   63.22%   63.15%   -0.07%     
==========================================
  Files         142      142              
  Lines       10794    10787       -7     
==========================================
- Hits         6824     6813      -11     
- Misses       3715     3716       +1     
- Partials      255      258       +3     

Impacted Files	Coverage Δ
...ator/controllers/lifecycle/keptntask/controller.go	56.33% <50.00%> (+3.63%)	⬆️
...rator/controllers/lifecycle/keptntask/job_utils.go	74.16% <64.28%> (+5.66%)	⬆️
...controllers/lifecycle/keptntask/runtime_builder.go	70.44% <78.12%> (ø)
...ntrollers/lifecycle/keptntask/container_builder.go	100.00% <100.00%> (ø)
...trollers/lifecycle/keptntask/job_runner_builder.go	100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes

Flag	Coverage Δ
certificate-operator	68.55% <ø> (ø)
component-tests	58.49% <68.25%> (-0.33%)	⬇️
lifecycle-operator	84.43% <77.77%> (+0.04%)	⬆️
metrics-operator	81.66% <ø> (ø)
scheduler	30.45% <ø> (ø)

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

[bacherfl]

lgtm

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication