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

wait for tenant to be active before polling for timeline absence #4856

Merged
merged 2 commits into from
Aug 1, 2023

Conversation

LizardWizzard
Copy link
Contributor

Problem

https://neon-github-public-dev.s3.amazonaws.com/reports/main/5692829577/index.html#suites/f588e0a787c49e67b29490359c589fae/4c50937643d68a66

Summary of changes

wait for tenant to be active after restart before polling for timeline absence

@LizardWizzard LizardWizzard added the c/storage/pageserver Component: storage: pageserver label Jul 31, 2023
@LizardWizzard LizardWizzard requested a review from koivunej July 31, 2023 16:04
@github-actions
Copy link

github-actions bot commented Jul 31, 2023

1240 tests run: 1189 passed, 0 failed, 51 skipped (full report)


Copy link
Member

@koivunej koivunej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good, but wondering about the timeout being enough.

@LizardWizzard LizardWizzard force-pushed the dkr/timeline-delete-flaky branch from 039fce1 to f0d10d9 Compare August 1, 2023 08:51
@koivunej
Copy link
Member

koivunej commented Aug 1, 2023

  • test_concurrent_timeline_delete_stuck_on[in_progress_delete]: debug

Re-running.

@koivunej
Copy link
Member

koivunej commented Aug 1, 2023

Re: timeouts, adding this one here as well: https://neon-github-public-dev.s3.amazonaws.com/reports/pr-4857/5718388045/index.html#suites/f588e0a787c49e67b29490359c589fae/5ef37c7fda6cfb7f

TL;DR: another tenant load not completing in 2s, this time aiming for Broken. I think we should raise all timeouts to 10 seconds on tests which are also ran against real_s3.

More timeouts: https://neon-github-public-dev.s3.amazonaws.com/reports/pr-4860/5725422362/index.html#suites/f588e0a787c49e67b29490359c589fae/23ae7306ad24bf08

@LizardWizzard
Copy link
Contributor Author

I'll rerun workflow for several times to be sure that the fix helped

@LizardWizzard LizardWizzard force-pushed the dkr/timeline-delete-flaky branch from 0b640b1 to 66d8577 Compare August 1, 2023 13:28
@LizardWizzard LizardWizzard merged commit c3fe335 into main Aug 1, 2023
@LizardWizzard LizardWizzard deleted the dkr/timeline-delete-flaky branch August 1, 2023 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/storage/pageserver Component: storage: pageserver
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants