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

Restore remote execution of unit tests by fixing firewall #8162

Merged
merged 1 commit into from Aug 12, 2019

Conversation

@Eric-Arellano
Copy link
Contributor

commented Aug 11, 2019

In #8138, we turned off remote execution because we mysteriously started getting 403 errors. Turns out that Travis had updated one of its IP addresses without us realizing, so the Google App Engine firewall was giving 403 errors.

This was fixed by updating the Google Cloud Console at https://console.cloud.google.com/appengine/firewall?project=pants-remoting-beta to match the list at https://docs.travis-ci.com/user/ip-addresses/.

@Eric-Arellano Eric-Arellano requested a review from stuhood Aug 11, 2019

@Eric-Arellano Eric-Arellano changed the title Restore remote execution of unit tests by fixing Firewall Restore remote execution of unit tests by fixing firewall Aug 11, 2019

@jsirois
Copy link
Member

left a comment

Do you mind adding me to the the GCP project? I think we should have all leads added, but I'm happy to take up that bit if you get me in there.

@Eric-Arellano

This comment has been minimized.

Copy link
Contributor Author

commented Aug 11, 2019

Do you mind adding me to the the GCP project? I think we should have all leads added, but I'm happy to take up that bit if you get me in there.

I'm no longer an owner, only an editor. @jsirois do you have access to pantsbuild@gmail.com? That's the only owner atm. CC @stuhood.

@jsirois

This comment has been minimized.

Copy link
Member

commented Aug 11, 2019

@jsirois do you have access to pantsbuild@gmail.com?

I do, that did the trick. Thank you.

@Eric-Arellano Eric-Arellano merged commit 9264c16 into pantsbuild:master Aug 12, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Eric-Arellano Eric-Arellano deleted the Eric-Arellano:restore-remoting branch Aug 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.