Migrate away from deprecated ioutil

[sebrandon1]

ioutil has been deprecated since Go 1.16: https://go.dev/doc/go1.16#ioutil

Tracking issue: redhat-best-practices-for-k8s/telco-bot#52

[coderabbitai]

Walkthrough

Replaced deprecated ioutil.WriteFile with the standard library os.WriteFile function in the RestoreAsset implementation, with corresponding import cleanup. Functionality and control flow remain unchanged.

Changes

Cohort / File(s)	Summary
Stdlib function migration pkg/operator/assets/bindata.go	Replaced ioutil.WriteFile with os.WriteFile in RestoreAsset function; removed ioutil import

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Straightforward, mechanical replacement of deprecated function with standard library equivalent
• No logic or behavior changes
• Single file, single location edit
• Verify import removal doesn't cause any other references to ioutil elsewhere in the codebase (though unlikely in this context)

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between 4ded1de and 0b03e5d.

📒 Files selected for processing (1)

• pkg/operator/assets/bindata.go (1 hunks)

🧰 Additional context used

📓 Path-based instructions (1)

**

⚙️ CodeRabbit configuration file

-Focus on major issues impacting performance, readability, maintainability and security. Avoid nitpicks and avoid verbosity.

Files:

• pkg/operator/assets/bindata.go

🔇 Additional comments (1)

pkg/operator/assets/bindata.go (1)

3327-3327: LGTM! Correct migration from deprecated ioutil.WriteFile verified.

The change to os.WriteFile on line 3327 is the proper migration path for the deprecated ioutil.WriteFile function. Verification confirms:

• os.WriteFile is correctly used with identical function signature
• ioutil import has been removed (not present in lines 68-74)
• No remaining ioutil references exist in the file

Since this is auto-generated code (go-bindata), verify the generator tool is also updated to prevent ioutil references in future regenerations.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: sebrandon1
Once this PR has been reviewed and has the lgtm label, please assign mytreya-rh for approval. For more information see the 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:

• OWNERS

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

[openshift-ci]

@sebrandon1: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/verify	0b03e5d	link	true	/test verify

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

[lunarwhite]

@sebrandon1 Thanks for your contribution! But as you could see this is from an auto-generated bindata:

cert-manager-operator/pkg/operator/assets/bindata.go

Line 1 in 0b03e5d

// Code generated for package assets by go-bindata DO NOT EDIT. (@generated)

I'm not sure if we really need to mitigate it.

[sebrandon1]

Ah yes, sorry I thought I was avoiding all of the bindata. I'll fix it.

[sebrandon1]

Closing as this is just bindata. Thanks!