search

pravega / pravega-operator

Pravega Kubernetes Operator
Apache License 2.0
41 stars 38 forks source link

Issue 616: Update k8 and Controller-run time versions #617

Closed anishakj closed 2 years ago

anishakj commented 2 years ago

Signed-off-by: anishakj anisha.kj@dell.com

Change log description

Updating controller-runtime to 11.0 and k8 version to 23.0

Purpose of the change

Fixes #616

What the code does

Updated the versions in go.mod and removed replace directives Changed the Api calls accordance with new controller run time Rewrite the e2e tests using envtest

How to verify it

All unit and e2e tests should pass

codecov-commenter commented 2 years ago

Codecov Report

Merging #617 (6b0eee5) into master (46bbd1e) will increase coverage by 7.85%. The diff coverage is 67.33%.

@@            Coverage Diff             @@
##           master     #617      +/-   ##
==========================================
+ Coverage   75.09%   82.94%   +7.85%     
==========================================
  Files          16       17       +1     
  Lines        4525     4099     -426     
==========================================
+ Hits         3398     3400       +2     
+ Misses        993      565     -428     
  Partials      134      134
Impacted Files Coverage Δ
api/v1alpha1/bookkeeper.go 100.00% <ø> (ø)
api/v1alpha1/pravega.go 100.00% <ø> (ø)
api/v1alpha1/pravegacluster_types.go 90.47% <ø> (ø)
api/v1alpha1/status.go 100.00% <ø> (ø)
api/v1alpha1/zz_generated.deepcopy.go 100.00% <ø> (ø)
api/v1beta1/pravega.go 98.88% <ø> (ø)
api/v1beta1/status.go 98.41% <ø> (ø)
api/v1beta1/zz_generated.deepcopy.go 99.03% <ø> (ø)
api/v1beta1/pravegacluster_webhook.go 44.21% <44.21%> (ø)
controllers/pravegacluster_controller.go 54.98% <75.38%> (ø)
... and 5 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 46bbd1e...6b0eee5. Read the comment docs.