Review app cleanup on schedule not on tags and in master
What does this MR do?
In https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/6665
we missed that automated cleanup job was gated at master branch. Since it runs on schedule, it is ok for it to run in master.
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary -
Documentation created/updated -
Tests added for this feature/bug -
Conforms to the code review guidelines -
Conforms to the merge request performance guidelines -
Conforms to the style guides -
Conforms to the database guides -
EE specific content should be in the top level /ee
folder -
For a paid feature, have we considered GitLab.com plans, how it works for groups, and is there a design for promoting it to users who aren't on the correct plan?