-
Notifications
You must be signed in to change notification settings - Fork 3
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
Allow concurrent actions on packages #68
Conversation
This change allows package dependencies to be read into a dependency tree. From this tree we are able to determine which package actions can be run concurrently. This allows us to drastically speed up the time it takes to deploy packages.
I don't think this is useful and makes the table really wide which causes display issues.
Task linked: CU-86by2x2u5 Start platform packages concurrenctly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rcrichton Looks great to me 👍
Just a few minor questions but approved from my end 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
This change will allow instant to operate on packages concurrently, which dramtically speeds up the time taken to deploy multiple packages. It takes into account dependencies and only those packages it can safely deploy while waiting for dependencies to be available before allowing dependant packages to execute. The default concurreny is set to 5. This can be configred but requires changes in the go CLI for this configuration to be exposed.