Skip to content

[release/0.3] Remove deprecated ioutil (#490)#516

Merged
athongsamai merged 2 commits into
Mirantis:release/0.3from
athongsamai:remove-ioutil
Oct 29, 2025
Merged

[release/0.3] Remove deprecated ioutil (#490)#516
athongsamai merged 2 commits into
Mirantis:release/0.3from
athongsamai:remove-ioutil

Conversation

@athongsamai

Copy link
Copy Markdown
Contributor

Cherry-pick #490 (commit 89e7b95 )

Proposed Changes

  • Remove the deprecated io/ioutil. (Refer to https://pkg.go.dev/io/ioutil) Instead use the same function from os package.
  • Use logrus.Errorf to append the inner error (following general error convention)

@smerkviladze smerkviladze left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks!

@mlimardo1984 mlimardo1984 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGMT, thanks!

@athongsamai athongsamai merged commit 4c1c397 into Mirantis:release/0.3 Oct 29, 2025
11 checks passed
@athongsamai athongsamai deleted the remove-ioutil branch October 29, 2025 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants