search

Azure / orkestra

Orkestra is a cloud-native release orchestration and lifecycle management (LCM) platform for the fine-grained orchestration of inter-dependent helm charts and their dependencies
https://azure.github.io/orkestra
Other
104 stars 16 forks source link

Add Executor Node Logic to Graph #381

Closed jonathan-innis closed 3 years ago

jonathan-innis commented 3 years ago
codecov-commenter commented 3 years ago

Codecov Report

Merging #381 (64e1b38) into main (e22ff10) will increase coverage by 2.67%. The diff coverage is 83.08%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #381      +/-   ##
==========================================
+ Coverage   67.14%   69.82%   +2.67%     
==========================================
  Files           8        8              
  Lines         350      391      +41     
==========================================
+ Hits          235      273      +38     
+ Misses        106      104       -2     
- Partials        9       14       +5
Impacted Files Coverage Δ
pkg/utils/helpers.go 48.78% <0.00%> (+1.16%) :arrow_up:
pkg/templates/utils.go 66.66% <50.00%> (+16.66%) :arrow_up:
pkg/graph/graph.go 88.09% <80.70%> (-7.83%) :arrow_down:
pkg/templates/templates.go 75.43% <86.84%> (+9.78%) :arrow_up:

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 e22ff10...64e1b38. Read the comment docs.

nitishm commented 3 years ago

Verified that this works with the complex appgroup. Looking good. @jonathan-innis can we resolve the conflicts and let's get this merged in.