OCPBUGS-63354: Increase Node ISO creation timeout from 15 to 30 minutes

[mhanss]

This PR increases the timeout for Node ISO creation from 15 minutes to 30 minutes.

[openshift-ci-robot]

@mhanss: This pull request references Jira Issue OCPBUGS-63354, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug

• bug is open, matching expected state (open)
• bug target version (4.21.0) matches configured target version for branch (4.21.0)
• bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @mhanss

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

This PR increases the timeout for Node ISO creation from 15 minutes to 30 minutes.

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]

@openshift-ci-robot: GitHub didn't allow me to request PR reviews from the following users: mhanss.

Note that only openshift members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

@mhanss: This pull request references Jira Issue OCPBUGS-63354, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug

• bug is open, matching expected state (open)
• bug target version (4.21.0) matches configured target version for branch (4.21.0)
• bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @mhanss

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

This PR increases the timeout for Node ISO creation from 15 minutes to 30 minutes.

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.

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]

Walkthrough

Replaced two 15-minute wait.PollUntilContextTimeout timeouts with a new nodeJoinerPollTimeout constant set to 30 minutes in the node image creation workflow, extending poll durations used during node-joiner pod execution and workflow monitoring.

Changes

Cohort / File(s)	Change Summary
Timeout Configuration Update pkg/cli/admin/nodeimage/create.go	Added nodeJoinerPollTimeout = 30m; replaced two occurrences of 15-minute timeouts in wait.PollUntilContextTimeout calls (in nodeJoinerPodExec and monitorWorkflowReport) with nodeJoinerPollTimeout

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

• Areas to spot-check: nodeJoinerPodExec and monitorWorkflowReport call sites in pkg/cli/admin/nodeimage/create.go to ensure the new constant is used consistently and timeout semantics remain correct.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The pull request title "OCPBUGS-63354: Increase Node ISO creation timeout from 15 to 30 minutes" is fully aligned with the changeset, which added a new nodeJoinerPollTimeout constant set to 30 minutes and replaced hardcoded 15-minute timeouts with this new constant. The title is specific, clear, and accurately describes the primary change without unnecessary noise or vague terms. A teammate scanning the history would immediately understand that this PR increases a timeout value from 15 to 30 minutes.
Description Check	✅ Passed	The pull request description "This pull request increases the timeout for Node ISO creation from 15 minutes to 30 minutes" is directly related to the changeset and accurately describes the modifications made. The description matches the title and clearly conveys what was changed, with sufficient clarity regarding the timeout adjustment even if it lacks implementation details. This meets the lenient criteria for the description check.

✨ 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 0e39ed6 and 6c04d0a.

📒 Files selected for processing (1)

• pkg/cli/admin/nodeimage/create.go (3 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• pkg/cli/admin/nodeimage/create.go

---

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.}

[coderabbitai]

Actionable comments posted: 1

📜 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 e6eeeb7 and 0e39ed6.

📒 Files selected for processing (1)

• pkg/cli/admin/nodeimage/create.go (1 hunks)

[andfasano]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andfasano, mhanss

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

The pull request process is described here

Needs approval from an approver in each of these files:

• pkg/cli/admin/nodeimage/OWNERS [andfasano]

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

[mhanss]

/verified bypass

[openshift-ci-robot]

@mhanss: The verified label has been added.

In response to this:

/verified bypass

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]

@mhanss: 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/okd-scos-e2e-aws-ovn	6c04d0a	link	false	/test okd-scos-e2e-aws-ovn

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.