Skip to content

fix: keep Studio minimum instance at one - #949

Draft
evanlowe wants to merge 1 commit into
volcengine:mainfrom
evanlowe:fix/studio-min-instance-one
Draft

fix: keep Studio minimum instance at one#949
evanlowe wants to merge 1 commit into
volcengine:mainfrom
evanlowe:fix/studio-min-instance-one

Conversation

@evanlowe

@evanlowe evanlowe commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • set VeFaaS MinInstance to 1 after successful deploy and synchronous updates
  • omit MaxInstance, ReservedFrozenInstance, and all unrelated resource fields
  • add vefaas:UpdateFunctionResource to Studio deploy precheck and runtime IAM policy
  • let legacy Studio in-app updates continue with an actionable warning when the existing Function role lacks vefaas:UpdateFunctionResource; other resource-update errors still stop the release

Legacy deployment behavior

Existing Function roles cannot grant themselves new IAM permissions. For those deployments, an in-app update still publishes the new Revision but leaves MinInstance unchanged and records a warning. Run veadk studio update once, or grant vefaas:UpdateFunctionResource manually, to migrate the role and enforce MinInstance=1.

Validation

  • pre-commit run --files ...
  • focused CLI, cloud, deploy-permission, and runtime-policy tests: 108 passed
  • BytePlus live Studio deploy: MinInstance=1, MaxInstance=10
  • BytePlus live CLI update: MinInstance=1, MaxInstance=10
  • BytePlus live in-app update to a new Revision: MinInstance=1, MaxInstance=10

@evanlowe
evanlowe force-pushed the fix/studio-min-instance-one branch from b140688 to 9cecea5 Compare August 20, 2026 10:50
@evanlowe
evanlowe force-pushed the fix/studio-min-instance-one branch from 9cecea5 to 5c6c8ac Compare August 20, 2026 10:54
@yaozheng-fang
yaozheng-fang marked this pull request as draft August 21, 2026 07:46
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.

1 participant