fix(operator): use correct parent/child span relationship

[thisthat]

Signed-off-by: Giovanni Liva [email protected]

Fixes #336

[codecov]

Codecov Report

Merging #418 (3760919) into main (fdf3f4b) will increase coverage by 5.14%.
The diff coverage is 55.93%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #418      +/-   ##
==========================================
+ Coverage   32.84%   37.99%   +5.14%     
==========================================
  Files          19       28       +9     
  Lines        1516     2082     +566     
==========================================
+ Hits          498      791     +293     
- Misses        989     1227     +238     
- Partials       29       64      +35     

Impacted Files	Coverage Δ
operator/controllers/common/spanhandler.go	78.57% <ø> (ø)
operator/controllers/common/test_utils.go	0.00% <0.00%> (ø)
operator/controllers/common/phasehandler.go	77.61% <100.00%> (ø)
operator/controllers/keptnappversion/controller.go	71.87% <100.00%> (ø)
operator/controllers/keptntask/job_utils.go	40.00% <0.00%> (ø)
...ers/keptnappversion/reconcile_prepostevaluation.go	81.25% <0.00%> (ø)
...ollers/keptnappversion/reconcile_workloadsstate.go	47.72% <0.00%> (ø)
operator/controllers/keptntask/function_utils.go	77.77% <0.00%> (ø)
...ers/keptnappversion/reconcile_prepostdeployment.go	81.25% <0.00%> (ø)
operator/controllers/keptntask/task_utils.go	57.14% <0.00%> (ø)
... and 2 more

Flag	Coverage Δ
keptn-lifecycle-operator	41.96% <55.93%> (+4.28%)	⬆️
scheduler	4.52% <ø> (ø)

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

[odubajDT]

I guess we might want to fix this in workloadInstance as well https://github.com/keptn/lifecycle-toolkit/blob/main/operator/controllers/keptnworkloadinstance/controller.go#L162

[thisthat]

I guess we might want to fix this in workloadInstance as well main/operator/controllers/keptnworkloadinstance/controller.go#L162

At the moment, this is not problematic since the parent/child relationship for Workload is correct. However, this will be reworked with #410

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[thschue]

lgtm