Skip to content

Refactor code around scheduling cluster installations

What does this MR do?

Refactors code around scheduling cluster installations

Are there points in the code the reviewer needs to double check?

no

Why was this MR needed?

To simplify schedule service for the cluster applications. I believe it better if schedule_installation_service is about scheduling rather than creating application records. Benefits are next:

  1. It simplifies the code a bit.
  2. It allows creating applications in the controller which means we can pass arguments to it.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Extracted from https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/19019

Edited by Dmytro Zaporozhets (DZ)

Merge request reports