fix(e2e): increase TimeoutShort from 2min to 3min

[skevetter]

Summary

• E2E tests doing Docker builds with apt-get install consistently timeout at 2 minutes on GitHub Actions due to slow Ubuntu apt mirror response times
• This caused 4 consecutive CI failures on main (commits d2ca833 through ba9fbb9) despite the underlying code being correct
• Increases TimeoutShort() from 2 minutes to 3 minutes to accommodate realistic Docker build times on CI runners
• The original semantic merge conflict (WriteResultJSON missing warnings param from PR feat(envelope): surface hostRequirements warnings in CLI result JSON #204 vs PR feat(cmd): add run-user-commands lifecycle command #203) was already fixed in PR feat(cmd): add --remove-volumes flag to down command #207

Resolves consistent "Test up-features && suite" and "Test up-docker-compose && config" CI failures on main.

Summary by CodeRabbit

• Chores
  • Adjusted test framework timeout configuration on non-Windows platforms to improve test reliability.

[netlify]

✅ Deploy Preview for devsydev canceled.

Name	Link
🔨 Latest commit	1c0c33e
🔍 Latest deploy log	https://app.netlify.com/projects/devsydev/deploys/69f8dc041a085b0008eeea08

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0a741000-8430-4d62-9a52-562b5bd85beb

📥 Commits

Reviewing files that changed from the base of the PR and between ba9fbb9 and 1c0c33e.

📒 Files selected for processing (1)

• e2e/framework/timeouts.go

---

📝 Walkthrough

Walkthrough

The TimeoutShort() function in the e2e test framework is updated to return 3 * time.Minute instead of 2 * time.Minute for non-Windows platforms, while Windows platforms continue to use 10 * time.Minute.

Changes

Timeout Configuration Update

Layer / File(s)	Summary
Constant Adjustment e2e/framework/timeouts.go	TimeoutShort() non-Windows return value increased from 2 * time.Minute to 3 * time.Minute.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ 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%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely summarizes the main change: increasing TimeoutShort from 2 minutes to 3 minutes in the e2e framework.
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

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