JetSki icon indicating copy to clipboard operation
JetSki copied to clipboard

Boost deployment time

Open rsevilla87 opened this issue 3 years ago • 0 comments

Description

In order to cut down deployment time of new clusters, several tasks, that take a few minutes, can be executed in parallel with the help of the async statement, and then we can poll them with async_status.

Deployment time of a 6 node cluster (3 masters + 3 workers) seen with the original code was: Thursday 06 October 2022 21:17:21 +0000 (0:00:00.402) 1:32:37.115 ******

With the patch was: Monday 10 October 2022 12:15:55 +0000 (0:00:00.397) 1:15:51.220 ********

That means a time saving of ~15minutes

Also updating ansible.cfg to enable pipelining and replace the deprecated callback_whitelist by callbacks_enabled to suppress deprecation warnings in the output.

Type of change

Please select the appropriate options:

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] This change requires a documentation update
  • [ ] This change is a documentation update

Checklist

  • [x] I have performed a self-review of my own code
  • [ ] If a change is adding a feature, it should require a change to the README.md and the review should catch this.
  • [ ] If the change is a fix, it should have an issue. The review should make sure the comments state the issue (not just the number) and it should use the keywords that will close the issue on merge.
  • [ ] A change should not be merged unless it passes CI or there is a comment/update saying what testing was passed.
  • [ ] PRs should not be merged unless positively reviewed.

rsevilla87 avatar Oct 13 '22 11:10 rsevilla87