Skip to content

fix(lambda): include LastUpdateStatus in function configuration responses#463

Merged
hectorvent merged 1 commit intofloci-io:mainfrom
ctnnguyen:fix/lambda-last-update-status
Apr 15, 2026
Merged

fix(lambda): include LastUpdateStatus in function configuration responses#463
hectorvent merged 1 commit intofloci-io:mainfrom
ctnnguyen:fix/lambda-last-update-status

Conversation

@ctnnguyen
Copy link
Copy Markdown
Contributor

@ctnnguyen ctnnguyen commented Apr 15, 2026

Summary

Adds LastUpdateStatus: "Successful" to buildFunctionConfiguration(), which produces responses for CreateFunction, GetFunction, GetFunctionConfiguration, and UpdateFunctionCode. Closes #439

Per AWS docs, LastUpdateStatus is first set to Successful after function creation completes. Since Floci is synchronous, Successful is always the correct value. LastUpdateStatusReason and LastUpdateStatusReasonCode are omitted because AWS only populates them on failure.

Test failures seem unrelated to my PR.

Type of change

  • Bug fix (fix:)
  • New feature (feat:)
  • Breaking change (feat!: or fix!:)
  • Docs / chore

AWS Compatibility

Verified with AWS CLI v2 against a Docker container built from source. LastUpdateStatus Successful is present in responses for CreateFunction, GetFunction, GetFunctionConfiguration, and UpdateFunctionCode.

Checklist

  • ./mvnw test passes locally
  • New or updated integration test added
  • Commit messages follow Conventional Commits

@ctnnguyen ctnnguyen marked this pull request as ready for review April 15, 2026 16:20
Copy link
Copy Markdown
Collaborator

@hectorvent hectorvent left a comment

Choose a reason for hiding this comment

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

It looks good to me!

Thank @ctnnguyen!

@hectorvent hectorvent merged commit 2590518 into floci-io:main Apr 15, 2026
9 of 10 checks passed
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.

[BUG] Lambda: UpdateFunctionCode missing LastUpdateStatus in response (breaks Terraform re-apply)

2 participants