ci: temp state delete for renamed experimental-ext-tasks repo#126
Merged
Conversation
The repo was renamed to ext-tasks in #125, but Pulumi state still holds the old RepositoryCollaborators resource. The first 'pulumi up' (which runs before refresh) tries to delete it and 404s, blocking all deploys. Drop it from state explicitly before 'make up'. Idempotent via '|| true'; remove this line after one successful deploy.
Pulumi PreviewClick to expand preview output |
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.
Deploys have been failing since #125 renamed
experimental-ext-tasks→ext-tasks.Pulumi state still holds the old
RepositoryCollaborators::repo-experimental-ext-tasksresource. The firstpulumi upinmake up(which runs before refresh, by design — see Makefile comment re: dynamic providers) tries to delete it and gets a 404 because the repo no longer exists under that name:This blocks every deploy, including the
up --refreshstep that would reconcile other drift (e.g. team memberships removed manually in the GitHub UI).Fix: drop the stale resource from state explicitly before
make up.|| truekeeps it idempotent so subsequent runs no-op once the entry is gone.Follow-up: remove this line after one successful deploy on
main. Longer-term, repo renames inrepoAccess.tsshould use Pulumialiasesso the resource is renamed in state instead of delete+create.