Fix eslint warning

[kyoto]

Summary by CodeRabbit

• Bug Fixes
  • Improved log preview error messages by ensuring they use the latest available translations.

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

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

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

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

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

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

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: a0f84e94-576a-4091-8be3-e0196d85c302

📥 Commits

Reviewing files that changed from the base of the PR and between de8b98d and fdbb002.

📒 Files selected for processing (1)

• src/components/AttachLogModal.tsx

---

📝 Walkthrough

Walkthrough

AttachLogModal moves the react-hooks/exhaustive-deps suppression comment from the loadPreview definition to its dependency array. No exported APIs or runtime behavior changed.

Changes

AttachLogModal lint suppression

Layer / File(s)	Summary
Move dependency lint suppression src/components/AttachLogModal.tsx	The react-hooks/exhaustive-deps suppression comment was relocated to immediately precede the loadPreview dependency array.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: joshuawilson, xrajesh

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title matches the change: a small ESLint warning fix in AttachLogModal.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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

[kyoto]

/cherry-pick release-4.19
/cherry-pick pattern-fly-5

[openshift-cherrypick-robot]

@kyoto: once the present PR merges, I will cherry-pick it on top of pattern-fly-5, release-4.19 in new PRs and assign them to you.

Details

In response to this:

/cherry-pick release-4.19
/cherry-pick pattern-fly-5

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.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/components/AttachLogModal.tsx`:
- Line 352: Update the effect dependencies around the debounced loadPreview so
cleanup calls loadPreview.cancel() before the locale-dependent function is
replaced, preventing pending timers from using stale t and updating
previewError. Keep the existing preview-loading behavior unchanged.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 888d7910-faa2-43ba-8db9-9e01345438dd

📥 Commits

Reviewing files that changed from the base of the PR and between 4a897c3 and de8b98d.

📒 Files selected for processing (1)

• src/components/AttachLogModal.tsx

[openshift-ci]

New changes are detected. LGTM label has been removed.

[kyoto]

/lgtm

Override test failure unrelated to this change:
/override "Red Hat Konflux / e2e-console-pf6 / lightspeed-console"

[openshift-ci]

@kyoto: you cannot LGTM your own PR.

Details

In response to this:

/lgtm

Override test failure unrelated to this change:
/override "Red Hat Konflux / e2e-console-pf6 / lightspeed-console"

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.

[openshift-ci]

@kyoto: Overrode contexts on behalf of kyoto: Red Hat Konflux / e2e-console-pf6 / lightspeed-console

Details

In response to this:

/lgtm

Override test failure unrelated to this change:
/override "Red Hat Konflux / e2e-console-pf6 / lightspeed-console"

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.

[openshift-cherrypick-robot]

@kyoto: #2220 failed to apply on top of branch "release-4.19":

Applying: Fix eslint warning
Using index info to reconstruct a base tree...
M	src/components/AttachLogModal.tsx
Falling back to patching base and 3-way merge...
Auto-merging src/components/AttachLogModal.tsx
CONFLICT (content): Merge conflict in src/components/AttachLogModal.tsx
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Fix eslint warning

Details

In response to this:

/cherry-pick release-4.19
/cherry-pick pattern-fly-5

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.

[openshift-cherrypick-robot]

@kyoto: #2220 failed to apply on top of branch "pattern-fly-5":

Applying: Fix eslint warning
Using index info to reconstruct a base tree...
M	src/components/AttachLogModal.tsx
Falling back to patching base and 3-way merge...
Auto-merging src/components/AttachLogModal.tsx
CONFLICT (content): Merge conflict in src/components/AttachLogModal.tsx
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Fix eslint warning

Details

In response to this:

/cherry-pick release-4.19
/cherry-pick pattern-fly-5

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.