Repository navigation
feat: add leaderless signaling and parallel full-stack deployment - #3
Merged
Merged
Conversation
Signed-off-by: Hang Yin <h4x3rotab@gmail.com>
Signed-off-by: Hang Yin <h4x3rotab@gmail.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.
Summary
phala_cvm_envresources. Remove the shell driver, targeted phases, and generated roster inputs. Parent apps no longer overwrite instance-owned slot environments.modules/mesh-coordinators: callers put shared values inenvand per-instance values ininstances[name].env; the module derives the signaling roster without an input/output dependency cycle. Include state moves for the former flat resources and preserve legacy app-only placement.Dependencies / merge order
The examples require the provider's new
phala_cvm_env, allowlist-onlyenv_keys,env_update_scope,wait_for_identity, and readiness semantics. Those are not available in published0.3.0-beta.4, which remains the current version constraint. Validation used a locally built provider via Terraform development overrides, as documented. A provider release and example version-constraint update are needed before registry-only deployment is supported.The pinned verifier image includes the linked verifier fix. No phala-cloud-monorepo source changes are included.
Prior live validation: prod14 + prod15
Fresh eight-CVM deployment: three coordinators, three PostgreSQL nodes, and two webdemo nodes, split evenly across the hosts.
/allrequests exercised 1,024 inner calls with zero errors.Identity allocation remains a genuine dependency, now expressed in Terraform's graph; independent boots and updates overlap. Cloud create/reboot time remains and no global latency bound is claimed.
Native single-apply validation
signaling.auto.tfvars.jsonand legacy driver inputs after backing up state. Keep the suppliedmovedblocks, preserve names and placement, and initialize the local module. The guide documents ownership handover and readiness versus application-health checks.Automated checks and PR split
protocol/package, verification CI, proof documentation links, and formal-verification changelog entries move to stacked PR test: add formal verification of the leaderless signaling protocol #4: test: add formal verification of the leaderless signaling protocol #4. Merge this implementation first, then retarget test: add formal verification of the leaderless signaling protocol #4 tomain.Local credentials, Terraform state, generated deployment files, and temporary overrides are excluded.