utkuozdemir / pv-migrate

CLI tool to easily migrate Kubernetes persistent volumes
Apache License 2.0
1.57k stars 76 forks source link

feat: improve node selection, refactor #237

Closed utkuozdemir closed 1 year ago

utkuozdemir commented 1 year ago

Do lots of refactoring to have better error wrapping, simplify some parts. Reorganize packages, remove internal package.

Improve the node affinity logic. Closes #224.

codecov-commenter commented 1 year ago

Codecov Report

Merging #237 (c05bca8) into master (1f88348) will decrease coverage by 1.79%. The diff coverage is 43.45%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
- Coverage   71.43%   69.65%   -1.79%     
==========================================
  Files          24       23       -1     
  Lines        1782     1865      +83     
==========================================
+ Hits         1273     1299      +26     
- Misses        407      449      +42     
- Partials      102      117      +15     
Files Changed Coverage Δ
app/app.go 100.00% <ø> (ø)
app/completion.go 13.13% <0.00%> (ø)
app/root.go 88.88% <0.00%> (ø)
k8s/completion.go 0.00% <0.00%> (ø)
k8s/helm.go 77.77% <ø> (ø)
rsync/cmd.go 79.31% <0.00%> (ø)
ssh/ssh.go 73.19% <0.00%> (ø)
util/util.go 87.50% <ø> (ø)
k8s/portforward.go 59.45% <14.28%> (ø)
strategy/strategy.go 63.77% <17.64%> (ø)
... and 13 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more