Skip to content

Harden cloud provider orchestration runtime fallback - #67

Merged
dragonforce2010 merged 4 commits into
mainfrom
codex/pr63-cloud-provider-followup
Jun 22, 2026
Merged

Harden cloud provider orchestration runtime fallback#67
dragonforce2010 merged 4 commits into
mainfrom
codex/pr63-cloud-provider-followup

Conversation

@dragonforce2010

Copy link
Copy Markdown
Owner

Summary

  • add follow-up cloud runtime deployment hardening for veFaaS stable updates and env cleanup
  • add controlled runtime-tool fallback for agent-runtime and host workspace paths
  • keep cloud smoke deterministic via opt-in protocol/env while production stable env drops smoke overrides
  • fix E2E cleanup so host fallback runtime ids are not treated as real E2B sandboxes

Verification

Resource cleanup

  • No real Aliyun FC/OSS resources were created by the verification path; Aliyun coverage uses contract fakes and signed URL checks.
  • Test veFaaS runtime functions from prior cloud smoke were scaled down before this follow-up.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c5e2681b6a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +144 to +145
const configuredInvokeUrl = String(process.env.MAPLE_ALIYUN_FC_INVOKE_URL || defaults.aliyun_fc.invoke_url || "");
const configuredFunctionName = String(process.env.MAPLE_ALIYUN_FC_FUNCTION_NAME || defaults.aliyun_fc.function_name || functionName);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Carry Aliyun FC pool auth into provisioned members

When an Aliyun FC runtime pool uses an existing HTTP endpoint, this path only reads process-wide MAPLE_ALIYUN_FC_* values and later persists member config without the per-workspace/pool invoke_url, function_name, or api_key from runtime_pools[].config / provider_credentials.aliyun. In tenant-scoped deployments where those values are stored on the workspace rather than the API process, provisioning either marks every Aliyun member failed for missing MAPLE_ALIYUN_FC_INVOKE_URL, or sessions run against protected FC endpoints without the Authorization header because runtimePoolMemberAgentRuntime can only reconstruct the runtime from member config_json.

Useful? React with 👍 / 👎.

@dragonforce2010
dragonforce2010 merged commit a25298a into main Jun 22, 2026
2 checks passed
@dragonforce2010
dragonforce2010 deleted the codex/pr63-cloud-provider-followup branch June 22, 2026 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants