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.

Sign up for GitHub

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

Jump to bottom

[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
Assignees
@pitrou
Labels
Component: Archery Component: Continuous Integration Type: bug
Milestone
16.0.0

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
apacheGH-39996: [Archery] Fix Crossbow build on a PR from a fork's ma…
eb21b9f 
…in branch
@github-actions github-actions bot mentioned this issue Feb 8, 2024
Merged
pitrou added a commit that referenced this issue Feb 8, 2024
@pitrou
GH-39996: [Archery] Fix Crossbow build on a PR from a fork's main bra…
30f6fdb 
…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 <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
@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
@pitrou
apacheGH-39996: [Archery] Fix Crossbow build on a PR from a fork's ma…
17455a9 
…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).
@github-actions github-actions bot mentioned this issue Feb 8, 2024
Merged
pitrou added a commit to pitrou/arrow that referenced this issue Feb 12, 2024
@pitrou
apacheGH-39996: [Archery] Fix Crossbow build on a PR from a fork's ma…
3dcf932 
…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
@pitrou
apacheGH-39996: [Archery] Fix Crossbow build on a PR from a fork's ma…
d6d661d 
…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
@pitrou
GH-39996: [Archery] Fix Crossbow build on a PR from a fork's main bra…
52e5603 
…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 <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
@pitrou @dgreiss
apacheGH-39996: [Archery] Fix Crossbow build on a PR from a fork's ma…
6c5a37c 
…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 <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
@pitrou @dgreiss
apacheGH-39996: [Archery] Fix Crossbow build on a PR from a fork's ma…
bf804bb 
…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 <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
zanmato1984 pushed a commit to zanmato1984/arrow that referenced this issue Feb 28, 2024
@pitrou @zanmato1984
apacheGH-39996: [Archery] Fix Crossbow build on a PR from a fork's ma…
0b62a7b 
…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 <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
zanmato1984 pushed a commit to zanmato1984/arrow that referenced this issue Feb 28, 2024
@pitrou @zanmato1984
apacheGH-39996: [Archery] Fix Crossbow build on a PR from a fork's ma…
963dae0 
…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 <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
thisisnic pushed a commit to thisisnic/arrow that referenced this issue Mar 8, 2024
@pitrou @thisisnic
apacheGH-39996: [Archery] Fix Crossbow build on a PR from a fork's ma…
6eec17f 
…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 <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
thisisnic pushed a commit to thisisnic/arrow that referenced this issue Mar 8, 2024
@pitrou @thisisnic
apacheGH-39996: [Archery] Fix Crossbow build on a PR from a fork's ma…
38658bd 
…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 <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pitrou pitrou

Labels
Component: Archery Component: Continuous Integration Type: bug
Projects
None yet
Milestone
16.0.0
1 participant
@pitrou