USHIFT-6708: Fix dnf_retry.sh to be more robust to package list separators

[ggiguash]

Introduced by a0492bb

• e2e-aws-ai-model-serving-nightly
• e2e-aws-footprint-and-performance-nightly

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Run ID: 264ccf3f-c1c0-4e35-971b-bbc0872d6d3a

📥 Commits

Reviewing files that changed from the base of the PR and between 302a842 and 44628df.

📒 Files selected for processing (1)

• scripts/dnf_retry.sh

---

Walkthrough

Modified scripts/dnf_retry.sh to change package argument construction. The script now iterates over individual package words from the DNF_PACK variable and appends each as a separate argument to DNF_CMD, rather than appending the entire array at once.

Changes

Cohort / File(s)	Summary
DNF Package Handling scripts/dnf_retry.sh	Modified loop logic to iterate over each package word individually and append as separate arguments to the DNF command, changing argument semantics for space-separated values.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

Tip

CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.

OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required.

[ggiguash]

/test ?

[ggiguash]

/test e2e-aws-ai-model-serving
/test e2e-aws-footprint-and-performance

[openshift-ci-robot]

@ggiguash: This pull request references USHIFT-6708 which is a valid jira issue.

Details

In response to this:

Introduced by a0492bb

• e2e-aws-ai-model-serving-nightly
• e2e-aws-footprint-and-performance-nightly

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 openshift-eng/jira-lifecycle-plugin repository.

[openshift-ci]

@ggiguash: all tests passed!

Full PR test history. Your PR dashboard.

Details

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.

[ggiguash]

/verified by ci

[openshift-ci-robot]

@ggiguash: This PR has been marked as verified by ci.

Details

In response to this:

/verified by ci

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 openshift-eng/jira-lifecycle-plugin repository.

[pacevedom]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ggiguash, pacevedom

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [ggiguash,pacevedom]

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