kou opened a new issue, #36218: URL: https://github.com/apache/arrow/issues/36218
### Describe the enhancement requested > Is there any reason we are raising an AssertionError here? Yes, this code is not supposed to execute in GHA for a non-default branch: https://github.com/apache/arrow/blob/e6221c45de7b4c675bcedfac9afb87f6a131dad9/ci/scripts/go_bench_adapt.py#L46 ``` else: # Assume that the environment is not GitHub Actions CI. Error out if that # assumption seems to be wrong. assert os.getenv("GITHUB_ACTIONS") is None # This is probably a local dev environment, for testing. In this case, it # does usually not make sense to provide commit information (not a # controlled CI environment). Explicitly keep `github_commit_info=None` to # reflect that (to not send commit information). ``` In that sense I think the failure is expected and does / should not block merge. If the redness is a problem then we can also sys.exit(0) instead. Although that might result in mean false-positive green signal in the future. _Originally posted by @jgehrcke in https://github.com/apache/arrow/issues/36195#issuecomment-1600599390_ --- Ok, I think the problem is that the PR from dependabot is coming from a branch on the apache/arrow repository instead of a fork hence those are executed which are not if they are from a fork branch due to: `if: success() && github.event_name == 'push' && github.repository == 'apache/arrow'` Probably we should not error and just do a `sys.exit(0)` but now I understand why this is failing here. _Originally posted by @raulcd in https://github.com/apache/arrow/issues/36195#issuecomment-1600624989_ ### Component(s) Continuous Integration, Go -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
