Remove implicit `, -> { success }` from `deployments` relations in `Project` and `Environment` model
Currently, environment.deployments
searches only successful deployments. Although, ideally, this should be explicitly scoped in each invocation, instead of setting it as a default scope.
As the consequence, we had a problem https://gitlab.com/gitlab-org/gitlab-ce/issues/53870.