Skip to content

Increase cert-manager startupapicheck timeout to 5m#51

Merged
dims merged 1 commit intoNVIDIA:mainfrom
yuanchen8911:fix/cert-manager-startup-timeout
Feb 3, 2026
Merged

Increase cert-manager startupapicheck timeout to 5m#51
dims merged 1 commit intoNVIDIA:mainfrom
yuanchen8911:fix/cert-manager-startup-timeout

Conversation

@yuanchen8911
Copy link
Contributor

Summary

  • Increase cert-manager startupapicheck job timeout from default 1m to 5m

Problem

The cert-manager startupapicheck job verifies the webhook API is ready after installation. The default 1 minute timeout is often insufficient, causing helm install to fail:

Error: INSTALLATION FAILED: failed post-install: resource not ready, 
name: cert-manager-startupapicheck, kind: Job, status: InProgress
context deadline exceeded

This happens because:

  • Network policies (e.g., nvsentinel metrics-access) can delay webhook connectivity
  • Slow cluster provisioning
  • Other resources competing for startup time

Solution

Add startupapicheck.timeout: 5m to cert-manager values:

startupapicheck:
  timeout: 5m

Test plan

  • Deploy to test cluster and verify cert-manager startupapicheck completes
  • Verify helm install succeeds without timeout errors

🤖 Generated with Claude Code

The default 1 minute timeout for the startupapicheck job is often
insufficient on slow clusters or when network policies delay webhook
connectivity.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@yuanchen8911 yuanchen8911 requested a review from a team as a code owner February 3, 2026 22:16
@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Coverage Report ✅

Metric Value
Coverage 73.8%
Threshold 70%
Status Pass
Coverage Badge
![Coverage](https://img.shields.io/badge/coverage-73.8%25-green)

No Go source files changed in this PR.

@dims dims enabled auto-merge February 3, 2026 22:24
@dims dims disabled auto-merge February 3, 2026 22:27
@dims dims merged commit c645c15 into NVIDIA:main Feb 3, 2026
4 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.

2 participants