Skip to content

Conversation

@chengoramazon
Copy link
Contributor

Problem

The default 5-minute timeout was insufficient for Amazon Q Feature Dev tests that involve code generation with multiple iterations. This was causing tests to fail prematurely before they could complete their full execution cycle.

Solution

Extended the test timeout to 15 minutes (900,000ms) specifically for the code generation test with multi-iteration scenarios. This provides adequate time for multi-iteration code generation tests to complete their execution while maintaining the existing test behavior.


  • Treat all work as PUBLIC. Private feature/x branches will not be squash-merged at release time.
  • Your code changes must meet the guidelines in CONTRIBUTING.md.
  • License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@chengoramazon chengoramazon requested a review from a team as a code owner April 29, 2025 18:31
@chengoramazon
Copy link
Contributor Author

/runIntegrationTests

@chengoramazon
Copy link
Contributor Author

chengoramazon commented Apr 29, 2025

- AmazonQLSP AmazonQLSPInstaller resolve() resolves
- Amazon Q Welcome page shows 3 times new tabs
- AmazonQWorkspaceLSP AmazonQWorkspaceLSPInstaller resolve() resolves

Failing tests are unrelated to /dev e2e test.

FeatureDev E2E only run in E2E Tests - stable which timeouts after 1 hour.

@chengoramazon
Copy link
Contributor Author

chengoramazon commented Apr 29, 2025

/retryBuilds

@chengoramazon
Copy link
Contributor Author

/runIntegrationTests

@chengoramazon
Copy link
Contributor Author

/runIntegrationTests

@chengoramazon
Copy link
Contributor Author

chengoramazon commented May 2, 2025

It looks the test timeout in master as well. https://d1ihu6zq92vp9p.cloudfront.net/96d2fe6f-ecce-4741-a45a-1c4038771295/report.html

@justinmk3 justinmk3 merged commit e9ea808 into aws:master May 6, 2025
29 of 31 checks passed
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.

4 participants