Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jul 11, 2023. It is now read-only.

Conversation

steeling
Copy link
Contributor

@steeling steeling commented May 3, 2022

The compat layer is used to convert from the old style options
to the new style MRC. This is useful so that we don't have to manage
two implementations of the certificate.Manager.

Signed-off-by: Sean Teeling [email protected]

part of #4713

@steeling steeling force-pushed the feature/mrc-admin branch 6 times, most recently from d9f5525 to 6db2bd0 Compare May 5, 2022 14:46
@steeling steeling force-pushed the feature/mrc-admin branch 5 times, most recently from 43a6447 to 09fd7e5 Compare May 11, 2022 15:28
@codecov-commenter
Copy link

codecov-commenter commented May 11, 2022

Codecov Report

Merging #4718 (397c15b) into main (102baf5) will decrease coverage by 0.05%.
The diff coverage is 71.27%.

@@            Coverage Diff             @@
##             main    #4718      +/-   ##
==========================================
- Coverage   69.56%   69.51%   -0.06%     
==========================================
  Files         220      225       +5     
  Lines       15800    15983     +183     
==========================================
+ Hits        10992    11110     +118     
- Misses       4758     4822      +64     
- Partials       50       51       +1     
Flag Coverage Δ
unittests 69.51% <71.27%> (-0.06%) ⬇️

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

Impacted Files Coverage Δ
cmd/osm-bootstrap/osm-bootstrap.go 39.91% <0.00%> (-3.90%) ⬇️
cmd/osm-controller/osm-controller.go 13.87% <0.00%> (-0.97%) ⬇️
cmd/osm-controller/validate.go 100.00% <ø> (+7.14%) ⬆️
pkg/certificate/providers/types.go 0.00% <ø> (ø)
...certificate/providers/vault/certificate_manager.go 58.57% <0.00%> (ø)
pkg/certificate/types.go 100.00% <ø> (ø)
pkg/certificate/manager.go 81.61% <66.66%> (-3.31%) ⬇️
pkg/certificate/fake_manager.go 52.17% <71.42%> (+10.99%) ⬆️
pkg/certificate/providers/compat.go 80.00% <80.00%> (ø)
pkg/certificate/providers/config.go 84.03% <83.82%> (+7.24%) ⬆️
... and 12 more

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 102baf5...397c15b. Read the comment docs.

@steeling steeling force-pushed the feature/mrc-admin branch 2 times, most recently from b7c55af to cdc6130 Compare May 11, 2022 23:41
@steeling steeling force-pushed the feature/mrc-admin branch from da0730d to d6079f9 Compare May 12, 2022 01:37
Copy link
Contributor

@nojnhuh nojnhuh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a few comments for small stuff, but overall LGTM.

@steeling steeling force-pushed the feature/mrc-admin branch from d6079f9 to 397c15b Compare May 13, 2022 14:32
@steeling
Copy link
Contributor Author

addressed all comments and squashed to single commit

The compat layer is used to convert from the old style options
to the new style MRC. This is useful so that we don't have to manage
to implementations of the certificate.Manager.

Signed-off-by: Sean Teeling <[email protected]>
@steeling steeling force-pushed the feature/mrc-admin branch from 397c15b to 8c3a1e1 Compare May 13, 2022 16:07
@jaellio jaellio merged commit 00bc363 into openservicemesh:main May 13, 2022
@jaellio jaellio mentioned this pull request Aug 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants