Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: increase sleep time to allow GC to perform cleanup #20019

Merged
merged 2 commits into from
Sep 23, 2024

Conversation

mcollovati
Copy link
Collaborator

@mcollovati mcollovati commented Sep 23, 2024

On CI systems the test randomly fails because the GC seems not to clean up ComponentLocator memory within the 5 milliseconds. However, running the test in isolation works fine. This change increases the delay between the GC requests to try to make the test more reliable.

On CI system the test randomly fails because the GC seems not
to clean up ComponentLocator memory within the 5 seconds.
However, running the test in isolation works fine.
This change increases the delay between the GC requests
to try to make the test more reliable.
Copy link

github-actions bot commented Sep 23, 2024

Test Results

1 137 files  ± 0  1 137 suites  ±0   1h 10m 11s ⏱️ + 2m 37s
7 399 tests ± 0  7 349 ✅ ± 0  50 💤 ±0  0 ❌ ±0 
7 766 runs  +13  7 706 ✅ +13  60 💤 ±0  0 ❌ ±0 

Results for commit 344ff35. ± Comparison against base commit 12633cd.

♻️ This comment has been updated with latest results.

Copy link

sonarcloud bot commented Sep 23, 2024

@mcollovati mcollovati merged commit 8819f2c into main Sep 23, 2024
42 checks passed
@mcollovati mcollovati deleted the test/component-locator-memory-release-test branch September 23, 2024 09:42
vaadin-bot pushed a commit that referenced this pull request Sep 23, 2024
On CI system the test randomly fails because the GC seems not
to clean up ComponentLocator memory within the 5 milliseconds.
However, running the test in isolation works fine.
This change increases the delay between the GC requests
to try to make the test more reliable.
vaadin-bot pushed a commit that referenced this pull request Sep 23, 2024
On CI system the test randomly fails because the GC seems not
to clean up ComponentLocator memory within the 5 milliseconds.
However, running the test in isolation works fine.
This change increases the delay between the GC requests
to try to make the test more reliable.
vaadin-bot pushed a commit that referenced this pull request Sep 23, 2024
On CI system the test randomly fails because the GC seems not
to clean up ComponentLocator memory within the 5 milliseconds.
However, running the test in isolation works fine.
This change increases the delay between the GC requests
to try to make the test more reliable.
vaadin-bot added a commit that referenced this pull request Sep 23, 2024
…0022)

On CI system the test randomly fails because the GC seems not
to clean up ComponentLocator memory within the 5 milliseconds.
However, running the test in isolation works fine.
This change increases the delay between the GC requests
to try to make the test more reliable.

Co-authored-by: Marco Collovati <marco@vaadin.com>
vaadin-bot added a commit that referenced this pull request Sep 23, 2024
…0023)

On CI system the test randomly fails because the GC seems not
to clean up ComponentLocator memory within the 5 milliseconds.
However, running the test in isolation works fine.
This change increases the delay between the GC requests
to try to make the test more reliable.

Co-authored-by: Marco Collovati <marco@vaadin.com>
vaadin-bot added a commit that referenced this pull request Sep 23, 2024
…0021)

On CI system the test randomly fails because the GC seems not
to clean up ComponentLocator memory within the 5 milliseconds.
However, running the test in isolation works fine.
This change increases the delay between the GC requests
to try to make the test more reliable.

Co-authored-by: Marco Collovati <marco@vaadin.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants