Migrate away from deprecated ioutil #1160
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ioutilhas been deprecated since Go 1.16: https://go.dev/doc/go1.16#ioutilTracking issue: redhat-best-practices-for-k8s/telco-bot#52
Migration from
ioutiltoosfor file and directory operations:Replaced
ioutil.ReadFilewithos.ReadFileandioutil.ReadDirwithos.ReadDirin core configuration and bundle loading logic, such as inpkg/profiling/config/config.goandstaging/api/pkg/manifests/bundleloader.go. [1] [2] [3] [4]Updated validation and testing files to use
os.ReadFileinstead ofioutil.ReadFile, including files likestaging/api/pkg/validation/internal/community.go,crd_test.go,csv_test.go,object_test.go, andoperatorgroup_test.go. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]Updated additional internal and registry logic to use
os.ReadFile,os.WriteFile, andos.MkdirTempin place of theirioutilcounterparts, such as inoperatorhub.go,buildahregistry/_options.go, andprovisioner_kind.go. [1] [2] [3] [4] [5] [6]