When running the prow presubmit check locally, it fails on the test-multi-module check because the environment variables REPO_OWNER, REPO_NAME, PULL_NUMBER are not set.
My current workaround is to manually set these variables in my local environment.
@runewake2
@monopole the multi-repo check is using prow ENV variables. These are not available when the prow-presubmit-check is run directly from the Makefile. I can see a few ways to approach this:
PULL_NUMER not set).prow-presubmit-check into two make commands. One for local runs excluding prow specific jobs and another for prow which is inclusive of these jobs and the previous job.Do you have thoughts on which approach would be preferred? Is there another approach that isn't listed?
In meantime providing REPO_OWNER, REPO_NAME, PULL_NUMBER during local runs can get around this but is inconvenient.
I will vote for #1. If the multi-repo check requires a PR exists, then it doesn't make sense to run it locally.
Most helpful comment
I will vote for #1. If the multi-repo check requires a PR exists, then it doesn't make sense to run it locally.