Skip to content

Increase vault pod readiness retry count from 20 to 40#147

Merged
darkdoc merged 1 commit into
validatedpatterns:mainfrom
rhjcd:increase-vault-readiness-retries
Jul 15, 2026
Merged

Increase vault pod readiness retry count from 20 to 40#147
darkdoc merged 1 commit into
validatedpatterns:mainfrom
rhjcd:increase-vault-readiness-retries

Conversation

@rhjcd

@rhjcd rhjcd commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Doubles the retry count from 20 to 40 for the vault namespace check and vault pod readiness check in roles/vault_utils/tasks/vault_status.yaml
  • Extends the maximum wait time from ~15 minutes to ~30 minutes per check (retries x 45s delay)
  • Addresses an 18.9% failure rate on RHDP sandbox environments where shared infrastructure causes vault pod startup to exceed the current 15-minute timeout

Context

On Red Hat Demo Platform sandbox environments, vault pod startup frequently takes longer than 15 minutes due to:

  • ArgoCD deployment sequencing delays
  • PVC provisioning on shared storage
  • Operator unseal job timing

The delay interval (45 seconds) is unchanged -- only the retry count increases.

Test plan

  • Deploy a validated pattern on a sandbox environment and confirm vault readiness checks succeed within the extended window
  • Verify no regression on environments where vault starts quickly (the check still exits as soon as the pod is ready)

On RHDP sandbox environments with shared infrastructure, vault pod
startup frequently exceeds the current 15-minute timeout (20 retries
x 45s delay) due to ArgoCD deployment sequencing, PVC provisioning,
and operator unseal job timing. This causes an 18.9% failure rate.

Double the retry count to 40 (30 min max per check) while keeping
the delay unchanged at 45 seconds.
@rhpds-bot

Copy link
Copy Markdown

@darkdoc will you review and merge please?

@darkdoc darkdoc merged commit e673ade into validatedpatterns:main Jul 15, 2026
7 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