Repository navigation
fix(palworld): pin operator image to live digest 8036766f - #43
Merged
Merged
Conversation
Default/example digest 89efffac (b21803c2, 07-29) is 27 commits older than the running controller (57e7f73, 09-07); an apply would downgrade. Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Updates
palworld_operator_imagedefault (and the tfvars example) fromsha256:89efffac…(commit b21803c2, 2026-07-29) tosha256:8036766feeb4…— Harbor tag57e7f73, the exact digest the prd-apps controller is running (palworld-operator-controller-managerimageID). Prevents a future apply from downgrading the operator ~27 commits. Local (gitignored)terraform.tfvarsupdated to match.The server-manager sidecar (
dd00444c…, commit 57e9dcc) is set on thePalworldServerCR, which Terraform does not manage — unchanged.Verification (no apply)
State backed up to
~/terraform-state-backups/rancher-deploy/20261004-203034/. Ran against a copy of state:terraform plan -refresh=false -target=null_resource.deploy_palworld_operator_{prd,nprd,poc}_apps→Plan: 1 to add, 0 to change, 0 to destroy, with triggerpalworld_operator_image = …@sha256:8036766f….null_resource.deploy_palworld_operator_prd_appsis not in state (operator was deployed out-of-band), so the plan shows a create; an apply re-runs thekubectl apply --server-side --force-conflictsprovisioner.kubectl diff --server-sideof the renderedtemplates/palworld-operator.yaml.tplvs live: Deployment image identical (no change), but the template's CRD is older than live (dropsmodManager/serverManagerschema etc., which the livepalworld-serverCR uses) and the ClusterRole lacksserviceaccounts,httproutes, andrbac roles/rolebindingspermissions. Applying today would downgrade the CRD/RBAC. Template should be regenerated from operator commit 57e7f73 (or HEAD) before plan Upgrade CloudNativePG operator 1.28.0 → 1.30.0 #19.Note
Low Risk
Config-only digest pin; no runtime or auth changes in this diff, though a full operator apply may still need template/CRD alignment separately.
Overview
Updates the
palworld_operator_imagedigest pin fromsha256:89efffac…tosha256:8036766feeb4…(Harbor tag57e7f73) invariables.tfandterraform.tfvars.example, with comments noting the 2026-10-04 prd-apps imageID.This aligns Terraform defaults with the controller already running in production so a future apply does not roll the operator back to an older build. No install flags, manifests, or deploy logic change—only the image reference and documentation in the example tfvars.
Reviewed by Cursor Bugbot for commit edf4792. Bugbot is set up for automated code reviews on this repo. Configure here.