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

[Archery] Crossbow refuses to run on the main branch of a fork #39996

Closed
pitrou opened this issue Feb 8, 2024 · 1 comment · Fixed by #39997 or #40002
Closed

[Archery] Crossbow refuses to run on the main branch of a fork #39996

pitrou opened this issue Feb 8, 2024 · 1 comment · Fixed by #39997 or #40002

Comments

@pitrou
Copy link
Member

pitrou commented Feb 8, 2024

Describe the bug, including details regarding any error messages, version, and platform.

When invoking @github-actions crossbow submit ... on a PR made from a fork's main branch, Archery fails triggering the builds. Example:
#39878 (comment)

Result:
https://github.com/apache/arrow/actions/runs/7828116460/job/21357317509#step:5:18

Run archery trigger-bot \
  archery trigger-bot \
    --event-name issue_comment \
    --event-payload /home/runner/work/_temp/_github_workflow/event.json
  shell: /usr/bin/bash -e {0}
  env:
    pythonLocation: /opt/hostedtoolcache/Python/3.8.18/x64
    PKG_CONFIG_PATH: /opt/hostedtoolcache/Python/3.8.18/x64/lib/pkgconfig
    Python_ROOT_DIR: /opt/hostedtoolcache/Python/3.8.18/x64
    Python2_ROOT_DIR: /opt/hostedtoolcache/Python/3.8.18/x64
    Python3_ROOT_DIR: /opt/hostedtoolcache/Python/3.8.18/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.8.18/x64/lib
    ARROW_GITHUB_TOKEN: ***
    CROSSBOW_GITHUB_TOKEN: ***
fatal: not a git repository (or any of the parent directories): .git
Cloning into '/tmp/tmpze7d9_8y/arrow'...
fatal: refusing to fetch into branch 'refs/heads/main' checked out at '/tmp/tmpze7d9_8y/arrow'
ERROR:archery:Command '['git', '-C', '/tmp/tmpze7d9_8y/arrow', 'fetch', 'origin', 'pull/39878/head:main']' returned non-zero exit status 128.

Component(s)

Archery, Continuous Integration

pitrou added a commit to pitrou/arrow that referenced this issue Feb 8, 2024
pitrou added a commit that referenced this issue Feb 8, 2024
…nch (#39997)

### Rationale for this change

### What changes are included in this PR?

### Are these changes tested?

### Are there any user-facing changes?

* Closes: #39996

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
@pitrou pitrou added this to the 16.0.0 milestone Feb 8, 2024
@pitrou
Copy link
Member Author

pitrou commented Feb 8, 2024

Reopening as the PR didn't fix the issue.

@pitrou pitrou reopened this Feb 8, 2024
pitrou added a commit to pitrou/arrow that referenced this issue Feb 8, 2024
…in branch

Continued from PR apache#39997, which wasn't sufficient (the `git fetch` that works for me doesn't seem to work on GHA).
pitrou added a commit to pitrou/arrow that referenced this issue Feb 12, 2024
…in branch

Instead of doing an intermediate bare clone, just fix the locally created branch name when fetching.

Amended from PR apache#39997, which wasn't sufficient (the `git fetch` that works for me doesn't seem to work on GHA).
pitrou added a commit to pitrou/arrow that referenced this issue Feb 12, 2024
…in branch

Instead of doing an intermediate bare clone, just fix the locally created branch name when fetching.

Amended from PR apache#39997, which wasn't sufficient (the `git fetch` that works for me doesn't seem to work on GHA).
pitrou added a commit that referenced this issue Feb 12, 2024
…nch (#40002)

Instead of doing an intermediate bare clone, just fix the locally created branch name when fetching.

Amended from PR #39997, which wasn't sufficient (the `git fetch` that works for me doesn't seem to work on GHA).

* Closes: #39996

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
…in branch (apache#39997)

### Rationale for this change

### What changes are included in this PR?

### Are these changes tested?

### Are there any user-facing changes?

* Closes: apache#39996

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
…in branch (apache#40002)

Instead of doing an intermediate bare clone, just fix the locally created branch name when fetching.

Amended from PR apache#39997, which wasn't sufficient (the `git fetch` that works for me doesn't seem to work on GHA).

* Closes: apache#39996

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
zanmato1984 pushed a commit to zanmato1984/arrow that referenced this issue Feb 28, 2024
…in branch (apache#39997)

### Rationale for this change

### What changes are included in this PR?

### Are these changes tested?

### Are there any user-facing changes?

* Closes: apache#39996

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
zanmato1984 pushed a commit to zanmato1984/arrow that referenced this issue Feb 28, 2024
…in branch (apache#40002)

Instead of doing an intermediate bare clone, just fix the locally created branch name when fetching.

Amended from PR apache#39997, which wasn't sufficient (the `git fetch` that works for me doesn't seem to work on GHA).

* Closes: apache#39996

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
thisisnic pushed a commit to thisisnic/arrow that referenced this issue Mar 8, 2024
…in branch (apache#39997)

### Rationale for this change

### What changes are included in this PR?

### Are these changes tested?

### Are there any user-facing changes?

* Closes: apache#39996

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
thisisnic pushed a commit to thisisnic/arrow that referenced this issue Mar 8, 2024
…in branch (apache#40002)

Instead of doing an intermediate bare clone, just fix the locally created branch name when fetching.

Amended from PR apache#39997, which wasn't sufficient (the `git fetch` that works for me doesn't seem to work on GHA).

* Closes: apache#39996

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment