Skip to content

Feat Async function deployments #843

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 20 commits into from
May 21, 2024

Conversation

byawitz
Copy link
Member

@byawitz byawitz commented May 19, 2024

What does this PR do?

Deploy functions asynchronously in parallel.

  • Show the status and progress for each function.
  • Using a wrapper around the cli-progress library for easy reuse.
  • Validating function questions before deploying, allowing hassle-free asynchronous deployment.

Test Plan

Tested with a variety of use cases against the 1.5.5 instance.

@byawitz byawitz requested review from stnguyen90, Meldiron and loks0n May 19, 2024 16:38
@byawitz byawitz self-assigned this May 19, 2024
@byawitz
Copy link
Member Author

byawitz commented May 19, 2024

Example 👇 of errors, and questions

async-func.mp4

@byawitz byawitz requested a review from Meldiron May 20, 2024 17:32
@loks0n
Copy link
Member

loks0n commented May 20, 2024

Is there a way we can have column headers for the deployment status?

In your video the difference between function name and ID is not too clear

@christyjacob4 christyjacob4 merged commit c91aa21 into feat-cli-g2 May 21, 2024
34 checks passed
@christyjacob4 christyjacob4 deleted the feat-async-function-deployments branch May 21, 2024 12:04
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.

4 participants