Skip to content

[CFX-5557] tweak templates setup timeouts#424

Open
ajalon1 wants to merge 4 commits intodatarobot-oss:mainfrom
ajalon1:aj/CFX-5557-tweak-timeouts
Open

[CFX-5557] tweak templates setup timeouts#424
ajalon1 wants to merge 4 commits intodatarobot-oss:mainfrom
ajalon1:aj/CFX-5557-tweak-timeouts

Conversation

@ajalon1
Copy link
Copy Markdown
Contributor

@ajalon1 ajalon1 commented Apr 7, 2026

RATIONALE

Flaky test now: https://github.com/datarobot-oss/cli/actions/runs/24057232285/attempts/1

But it succeeded on the second try: https://github.com/datarobot-oss/cli/actions/runs/24057232285

I think its because I changed timeouts to 30s for Phase 0, but that's not really necessary. Let's see if the previous value of 360s works here.

CHANGES

PR Automation

Comment-Commands: Trigger CI by commenting on the PR:

  • /trigger-smoke-test or /trigger-test-smoke - Run smoke tests
  • /trigger-install-test or /trigger-test-install - Run installation tests

Labels: Apply labels to trigger workflows:

  • run-smoke-tests or go - Run smoke tests on demand (only works for non-forked PRs)

Important

For Forked PRs: If you're an external contributor, the run-smoke-tests label won't work. A maintainer must manually trigger the "Fork PR Smoke Tests" workflow from the Actions tab, providing your PR number. Please comment requesting a maintainer review if you need smoke tests to run.


Note

Low Risk
Low risk: only changes an Expect timeout in smoke-test automation to reduce flakiness; no product/runtime logic is affected.

Overview
Reduces smoke-test flakiness in smoke_test_scripts/expect_templates_setup.exp by removing the shortened Phase 0 timeout, allowing the existing longer timeout to cover the Pulumi pre-wizard flow before the dotenv wizard starts.

Reviewed by Cursor Bugbot for commit 1827237. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

🔐 Fork smoke tests started by maintainer

⏳ Security scans passed. Running smoke tests...

Commit: 182723734c378d25b401477715fd3de672f187dd
View run

@ajalon1
Copy link
Copy Markdown
Contributor Author

ajalon1 commented Apr 7, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

All smoke tests passed! (Fork PR)

✅ Security Scan: success
✅ Linux: success
✅ Windows: success

View run details

@ajalon1 ajalon1 requested a review from a team April 7, 2026 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants