Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Deprecated io/utils #2496

Merged
merged 1 commit into from
Jan 8, 2023
Merged

Conversation

cheyang
Copy link
Collaborator

@cheyang cheyang commented Jan 6, 2023

Signed-off-by: cheyang [email protected]

Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

fixes #2495

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@fluid-e2e-bot
Copy link

fluid-e2e-bot bot commented Jan 6, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from cheyang by writing /assign @cheyang in a comment. For more information see:The Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@codecov
Copy link

codecov bot commented Jan 6, 2023

Codecov Report

Merging #2496 (7a421dc) into master (76b0e5b) will increase coverage by 0.08%.
The diff coverage is 92.00%.

@@            Coverage Diff             @@
##           master    #2496      +/-   ##
==========================================
+ Coverage   67.34%   67.42%   +0.08%     
==========================================
  Files         367      367              
  Lines       21225    21225              
==========================================
+ Hits        14294    14312      +18     
+ Misses       5249     5231      -18     
  Partials     1682     1682              
Impacted Files Coverage Δ
pkg/ddc/thin/master_internal.go 0.00% <0.00%> (ø)
pkg/csi/recover/recover.go 66.45% <100.00%> (ø)
pkg/ddc/alluxio/load_data.go 83.33% <100.00%> (ø)
pkg/ddc/alluxio/master_internal.go 68.18% <100.00%> (ø)
pkg/ddc/eac/master_internal.go 61.90% <100.00%> (ø)
pkg/ddc/goosefs/load_data.go 83.14% <100.00%> (ø)
pkg/ddc/goosefs/master_internal.go 68.18% <100.00%> (ø)
pkg/ddc/jindo/load_data.go 64.76% <100.00%> (ø)
pkg/ddc/jindo/master_internal.go 56.41% <100.00%> (ø)
pkg/ddc/juicefs/data_load.go 77.41% <100.00%> (ø)
... and 7 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@cheyang
Copy link
Collaborator Author

cheyang commented Jan 6, 2023

/test fluid-e2e

@TrafalgarZZZ
Copy link
Member

/retest

@TrafalgarZZZ
Copy link
Member

/lgtm

@fluid-e2e-bot fluid-e2e-bot bot added the lgtm label Jan 8, 2023
@cheyang cheyang merged commit 2967f96 into fluid-cloudnative:master Jan 8, 2023
cheyang added a commit to cheyang/fluid that referenced this pull request Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG]"io/ioutil" has been deprecated since Go 1.16
2 participants