Closed dominicgunn closed 4 years ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign dominicgunn
You can assign the PR to them by writing /assign @dominicgunn
in a comment when ready.
The full list of commands accepted by this bot can be found here.
Merging #1864 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #1864 +/- ##
=======================================
Coverage 24.42% 24.42%
=======================================
Files 98 98
Lines 5114 5114
=======================================
Hits 1249 1249
Misses 3725 3725
Partials 140 140
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 69ed6a7...6127eff. Read the comment docs.
Changes
This takes the canal configuration from kops, and moves installation into a initContainer as oppose to trying to run both the installation and canal containers side-by-side.