Skip to content

[infra] Increase install time limit to 20s#8965

Merged
davebayer merged 2 commits into
NVIDIA:mainfrom
davebayer:bump_install_time_limit
May 13, 2026
Merged

[infra] Increase install time limit to 20s#8965
davebayer merged 2 commits into
NVIDIA:mainfrom
davebayer:bump_install_time_limit

Conversation

@davebayer
Copy link
Copy Markdown
Contributor

I've ran into problems with packaging running out of time during installation. This PR updates the limit from 10s to 20s

@davebayer davebayer requested a review from a team as a code owner May 13, 2026 16:14
@davebayer davebayer requested a review from jrhemstad May 13, 2026 16:14
@github-project-automation github-project-automation Bot moved this to Todo in CCCL May 13, 2026
@cccl-authenticator-app cccl-authenticator-app Bot moved this from Todo to In Review in CCCL May 13, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

Review Change Stack

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: 3c5e1037-c14d-4ed8-a28a-693c113247ab

📥 Commits

Reviewing files that changed from the base of the PR and between 3f440b9 and f149fb1.

📒 Files selected for processing (1)
  • ci/install_packaging.sh

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated installation timeout configuration in the build process to accommodate extended validation timeframes.

Walkthrough

The PR updates a single configuration constant in the CI packaging installation script, increasing the time limit for installation operations from 10 to 20 seconds. This extends the window before timeout enforcement applies to installation variant checks.

Changes

CI installation time limit

Layer / File(s) Summary
Install time limit constant adjustment
ci/install_packaging.sh
install_time_limit constant increased from 10 to 20 seconds, extending allowed execution time for subsequent timing validation checks.

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

@davebayer davebayer enabled auto-merge (squash) May 13, 2026 16:20
@github-actions

This comment has been minimized.

@github-actions
Copy link
Copy Markdown
Contributor

🥳 CI Workflow Results

🟩 Finished in 2h 00m: Pass: 100%/500 | Total: 8d 03h | Max: 1h 42m | Hits: 98%/626672

See results here.

@davebayer davebayer merged commit 58acf4b into NVIDIA:main May 13, 2026
520 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in CCCL May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants