Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CircleCI doesn't not build for Forked PRs #409

Open
hdost opened this issue Mar 31, 2020 · 3 comments
Open

CircleCI doesn't not build for Forked PRs #409

hdost opened this issue Mar 31, 2020 · 3 comments

Comments

@hdost
Copy link
Contributor

hdost commented Mar 31, 2020

@Dieterbe so I know we've mostely been communicating about this in the PRs #406 #407

I think it might be good to just have it in an issue. What we know:

  1. Setting for forked PRs is Turned on.
  2. PRs from forked Repositories not building.

I have performed a couple of tests on https://github.com/hdost/goran I created a fork in to criteo-forks I even had someone else in my team create a PR with a random change to it. I had set an environment variable in the project before creating any PRs.

  • My PRs from a forked repo are building
  • His PRs from a forked repo are building build
    ** I also asked him to create his own fork of the repo, and it still starts building.

Two possibilities that I can think of:

  1. There is in the grafana org setup. My guess would be that maybe old style Github Services are being used instead of Github Apps. Check https://github.com/grafana/carbon-relay-ng/settings/installations and possible re-configure it, even with the same settings.
  2. Unless you're using the variable somewhere in the repo it doesn't do restrictions. (Not sure how they would determine this.)
@Dieterbe
Copy link
Contributor

There is in the grafana org setup. My guess would be that maybe old style Github Services are being used instead of Github Apps. Check https://github.com/grafana/carbon-relay-ng/settings/installations and possible re-configure it

this shows the "CircleCI Checks", no services. There's no config options there for the App, but I went back to the circleCI config, and disabled and re-enabled "Build forked pull requests", and also enabled "GitHub Status updates" which was previously off.
"Pass secrets to builds from forked pull requests" is off (of course).

this option is also on:

"Only build pull requests
By default, we will build all the commits for this project. Once turned on, we will only build branches that have associated pull requests open."

@Dieterbe
Copy link
Contributor

Let me see if i can contact support, as we have a paid account.

@hdost
Copy link
Contributor Author

hdost commented Apr 26, 2020

@Dieterbe any hope?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants