Skip to content

[patch] Increase timeout for pvc bind wait #157

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.

Sign up for GitHub

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

Jump to bottom
Merged
durera merged 2 commits into from
Dec 19, 2025
Merged

[patch] Increase timeout for pvc bind wait #157

durera merged 2 commits into from
Dec 19, 2025

Conversation

@durera
Copy link
Contributor

@durera durera commented Dec 19, 2025

We have unnecessarily aggressive polling/timeout when waiting for a PVC to be ready:

19 Dec, 03:13:25 2025-12-19 03:08:24,177   DEBUG    Waiting for postgredb-tekton-results-postgres-0 PVC to be ready
19 Dec, 03:13:25 2025-12-19 03:08:24,195   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be created before checking again ...
19 Dec, 03:13:25 2025-12-19 03:08:29,214   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be created before checking again ...
19 Dec, 03:13:25 2025-12-19 03:08:34,235   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be created before checking again ...
19 Dec, 03:13:25 2025-12-19 03:08:39,257   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be created before checking again ...
19 Dec, 03:13:25 2025-12-19 03:08:44,278   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be created before checking again ...
19 Dec, 03:13:25 2025-12-19 03:08:49,330   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be created before checking again ...
19 Dec, 03:13:25 2025-12-19 03:08:54,350   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be created before checking again ...
19 Dec, 03:13:25 2025-12-19 03:08:59,368   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:04,388   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:09,408   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:14,445   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:19,464   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:24,484   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:29,502   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:34,522   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:39,541   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:44,561   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:49,578   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:54,603   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:09:59,622   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:04,641   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:09,659   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:14,680   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:19,702   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:24,722   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:29,744   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:34,764   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:39,786   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:44,806   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:49,826   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:54,846   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:10:59,866   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:04,885   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:09,905   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:14,926   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:19,948   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:24,967   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:29,989   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:35,010   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:40,029   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:45,052   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:50,073   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:11:55,091   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:00,110   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:05,131   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:10,150   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:15,172   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:20,191   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:25,211   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:30,230   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:35,255   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:40,275   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:45,295   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:50,315   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:12:55,333   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:13:00,366   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:13:05,391   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:13:10,414   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:13:15,435   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:13:20,458   DEBUG    Waiting 5s for PVC postgredb-tekton-results-postgres-0 to be ready before checking again ...
19 Dec, 03:13:25 2025-12-19 03:13:25,484   ERROR    OpenShift Pipelines postgres PVC is NOT ready

We can see that had we waited a little longer the PVC would have bound successfully. This update gives 10 minutes rather than 5, and reduces the poll interval to every 30s.

@durera durera requested a review from a team as a code owner December 19, 2025 11:49
@durera durera merged commit a08a9c8 into stable Dec 19, 2025
2 checks passed
@durera durera deleted the pvcwait branch December 19, 2025 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@durera