Skip to content

Fix CI project include error

Furkan Ayhan requested to merge 406813-fa-fix-project-include into master

What does this MR do and why?

This MR fixes the CI project-include error defined in #406813 (closed) which was introduced by !114279 (merged). Sentry: https://sentry.gitlab.net/gitlab/gitlabcom/issues/4119804

There is no test in this MR because I couldn't reproduce the error and did not find the real cause of it. It is interesting that the can_access_local_content? method returns true but project is nil in the sha method.

Whether it fixes the problem or not, having the project.nil? check in the condition seems fine.

If this does not solve the problem, then we can try logging it.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Furkan Ayhan

Merge request reports