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

WARNING Could not get OrganizationID while constructing locator #321

Closed
grosser opened this issue Oct 5, 2018 · 11 comments
Closed

WARNING Could not get OrganizationID while constructing locator #321

grosser opened this issue Oct 5, 2018 · 11 comments

Comments

@grosser
Copy link
Contributor

grosser commented Oct 5, 2018

did not change our setup and suddenly it's failing with this error https://travis-ci.org/zendesk/samson/jobs/437513874

@microsoftly
Copy link
Contributor

@grosser We're looking into the issue. Could you please provide your .fossa.yml?

@grosser
Copy link
Contributor Author

grosser commented Oct 5, 2018

@elldritch
Copy link
Member

This is probably related to #281 cc @zlav.

@microsoftly
Copy link
Contributor

@grosser could you retry the build?

@grosser
Copy link
Contributor Author

grosser commented Oct 5, 2018

priority: low since it's not a blocker, just failing on forks ... I guess this was the only PR we ever had that did not come from our base repo :(

WARNING Unrecognized locator
WARNING Unrecognized locator
FATAL Error during upload: you do not have permission to upload builds for this project

@grosser
Copy link
Contributor Author

grosser commented Oct 5, 2018

also can't retry now since the ref is gone ... and now the logs are gone too since I restarted it :(

@grosser
Copy link
Contributor Author

grosser commented Oct 5, 2018

yeah according to travis docs secrets are not shared with fork PRs ... which kinda makes sense ...
but the error message should be nicer ... like "Api key is missing" or so :)

@grosser
Copy link
Contributor Author

grosser commented Oct 5, 2018

... and analyze should exit 1 when upload failed or key was not given

@grosser
Copy link
Contributor Author

grosser commented Oct 5, 2018

hmm it might already do the exit status, I think travis might continue if a step fails ... checking

@grosser
Copy link
Contributor Author

grosser commented Oct 5, 2018

FYI changing it to fail nicely and pass when using a foreign repository to make a PR
zendesk/samson#2980

... that snipped might be nice for your readme in a travis section, since it's not very obvious :)

@elldritch
Copy link
Member

Closing in favor of #226.

meghfossa pushed a commit that referenced this issue Nov 12, 2021
* Run monorepo OR normal, not both

* Forgot the last bit

* fmt

* Remove --monorepo flag

* update user-guide

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

No branches or pull requests

3 participants