Skip to content

Commit

Permalink
GH-44422: [Packaging][Release][Linux] Upload artifacts before test (#…
Browse files Browse the repository at this point in the history
…44425)

### Rationale for this change

It seems that the current `quay.io/centos/centos:stream9` for `linux/aarch64` is broken.

We use it for building and testing but testing is only broken for now. Because we use cached (a bit old) `quay.io/centos/centos:stream9` for now. Building will be failed when the cache is expired.

### What changes are included in this PR?

Change upload phase to after the build from after the test. Test is still failing but we can proceed our release process by this.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: #44422

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
  • Loading branch information
kou authored and raulcd committed Oct 16, 2024
1 parent 70a2245 commit 5d7987b
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions dev/tasks/linux-packages/github.linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,10 @@ jobs:
APT_TARGETS: {{ target }}
REPO: {{ '${{ secrets.REPO }}' }}
YUM_TARGETS: {{ target }}

{% set patterns = upload_extensions | format_all("arrow/dev/tasks/linux-packages/*/*/repositories/**/*{}") %}
{{ macros.github_upload_releases(patterns)|indent }}

- name: Set up test
run: |
sudo apt install -y \
Expand Down Expand Up @@ -123,6 +127,3 @@ jobs:
APT_TARGETS: {{ target }}
ARROW_VERSION: {{ arrow.version }}
YUM_TARGETS: {{ target }}

{% set patterns = upload_extensions | format_all("arrow/dev/tasks/linux-packages/*/*/repositories/**/*{}") %}
{{ macros.github_upload_releases(patterns)|indent }}

0 comments on commit 5d7987b

Please sign in to comment.