feat(deploy): remote-only, remove local buildpacks/Docker build#7
Merged
Conversation
The CLI no longer builds: it always uploads the source and the platform builds server-side. Removes the local build path (upload_tag, check_environment, upload_logs), the @faabletools/buildpacks dep, and the --local/--buildpack/ --remote flags. Sends source with a manifest only — the builder re-detects the framework and syncs runtime_strategy on completion. No local fallback: a rejected admission (build_mode=local opt-out, kill-switch off) or a build error exits red. The remote builder is unaffected (built from the buildpacks repo, not the npm package). See arch/deploy/remote-artifact-default-cutover.md.
|
🎉 This PR is included in version 1.24.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
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.
Parte del cutover (arch/deploy/remote-artifact-default-cutover.md). Este PR va el ÚLTIMO del rollout.
Cambios
upload_tag.ts,check_environment.ts,upload_logs.ts, la dep@faabletools/buildpacks, y los flags--local/--buildpack/--remote.source: { manifest }sinplan(el builder re-detecta y sincronizaruntime_strategyen el completado).build_mode=local, kill-switch off) o un build error salen en rojo.manifest.tsque citaban--localactualizados.Tests
45 tests AVA verdes; tsc verde; sin referencias a docker en el path de deploy.
Rollout (⚠️ orden)
Mergear/publicar después de: api + deploy-controller + buildpacks desplegados, kill-switches ON en prod, y backfill
$unsethecho. Antes de eso, el gate rechazaría source de apps aún enbuild_mode=localsin fallback.