ci: make production deploy manual-only (remove push:main auto-deploy)#105
Closed
jeffersonBastos wants to merge 1 commit into
Closed
ci: make production deploy manual-only (remove push:main auto-deploy)#105jeffersonBastos wants to merge 1 commit into
jeffersonBastos wants to merge 1 commit into
Conversation
Merging to main previously auto-triggered deploy.yml, which (on the push path, with no skip_schema_drop input) drops the `programmatic_orders` schema and re-indexes from scratch — taking the synced production instance offline for hours. Remove the `push: main` trigger so deploys are explicit-only via workflow_dispatch. This lets us merge develop -> main for code review without disturbing the live, fully-synced instance; the final version is then deployed manually (with skip_schema_drop=true to preserve progress). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
Author
|
Applied directly to |
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
1 participant
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.
Problem
deploy.ymlauto-triggers onpush: main. On the push path theskip_schema_dropinput is empty, so the workflow runsDROP SCHEMA IF EXISTS "programmatic_orders" CASCADEand re-indexes from scratch — hours of downtime / empty data for the live instance. So mergingdevelop → mainwould destroy the currently-synced production state.Change
Remove the
push: maintrigger. Deploys become manual-only viaworkflow_dispatch(Actions → Run workflow), whereskip_schema_drop=truecan preserve indexing progress.Effect
develop → main(for code review / handover) no longer touches production.Since the workflow file in the merged commit no longer has the
pushtrigger, the merge-to-main itself will not fire a deploy.🤖 Generated with Claude Code