Skip to content

Adds cluster_for_group factory for convenience

Thong Kuah requested to merge group_with_cluster_factory into master

What does this MR do?

Also means we don't have to resort to an update statement to set parent for child groups who also have clusters.

create(:cluster_for_group, groups: [group])

is much shorter than

create(:cluster, :provided_by_gcp, :group, groups: [group])

Does this MR meet the acceptance criteria?

Conformity

Performance and testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading