[Bug] Upgrade vke-cluster to the version that is actually live (1.32.1+1)#1267
Conversation
WalkthroughUpdated the Vultr Kubernetes Engine cluster version in apps/infra/src/vultr/vke.ts from v1.29.2+1 to v1.32.1+1. No other properties, control flow, or exported/public signatures changed. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related issues
Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Greptile Summary
This PR updates the Kubernetes version in the Vultr Kubernetes Engine (VKE) cluster configuration from v1.29.2+1 to v1.32.1+1 in the apps/infra/src/vultr/vke.ts file. This is a critical infrastructure fix that addresses a deployment issue where the Pulumi infrastructure code was out of sync with the actual live production cluster.
The change resolves a specific problem where the version specified in the code (v1.29.2+1) has been archived by Vultr, causing deployment failures for the vke-cluster service. According to the issue description and testing evidence provided, the production cluster is already running on v1.32.1+1, as confirmed by both the Vultr web UI and kubectl version output.
This is purely a synchronization fix - the infrastructure code is being updated to match the reality of what's already deployed in production. The change affects the k8sCluster resource definition in the VKE configuration, which is part of the broader Pulumi infrastructure as code setup for managing the bluedot production Kubernetes cluster on Vultr's Amsterdam region.
The fix ensures that future Pulumi deployments will succeed by aligning the declared infrastructure state with the actual deployed state, preventing the mismatch that was causing deployment pipeline failures.
Confidence score: 5/5
- This PR is safe to merge with minimal risk as it synchronizes code with existing production infrastructure
- Score reflects that this is a straightforward version alignment fix with clear evidence and testing provided
- No files require special attention - this is a single-line change in infrastructure configuration
1 file reviewed, no comments
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
apps/infra/src/vultr/vke.ts (1)
7-7: Optional: drive the version from Pulumi config to avoid future code edits.Keeps code stable when bumping versions across stacks.
Apply within this line:
- version: 'v1.32.1+1', + version: clusterVersion,Add near imports:
import * as pulumi from '@pulumi/pulumi'; const cfg = new pulumi.Config(); const clusterVersion = cfg.get('k8sVersion') ?? 'v1.32.1+1';Then set per stack:
- pulumi config set k8sVersion v1.32.1+1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
apps/infra/src/vultr/vke.ts(1 hunks)
🔇 Additional comments (1)
apps/infra/src/vultr/vke.ts (1)
7-7: Pulumi Kubernetes version updated to v1.32.1+1 — no other stale pins found.
|
Great job on finding that it's been updated for us automatically! |
Description
From the issue #1266 :
Issue
#1266
Developer checklist
N/A
Steps to test
On master, running
pulumi preview --diff --show-configshould give an output that looks like this:And on this branch: