Skip to content

Commit

Permalink
apacheGH-43449: [CI][Conan] Don't push used images (apache#43470)
Browse files Browse the repository at this point in the history
### Rationale for this change

Because they aren't managed by us.

### What changes are included in this PR?

Don't push used images for Conan related jobs.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Jacob Wujciak-Jens <jacob@wujciak.de>
  • Loading branch information
kou authored Jul 30, 2024
1 parent 95c6d77 commit 96a6c45
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions dev/tasks/docker-tests/github.linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@ jobs:
{% if arrow.is_default_branch() %}
{{ macros.github_login_dockerhub()|indent }}
- name: Push Docker Image
if: {{ push|default("true") }}
shell: bash
run: archery docker push {{ image }}
{% endif %}
2 changes: 2 additions & 0 deletions dev/tasks/tasks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -207,6 +207,7 @@ tasks:
template: docker-tests/github.linux.yml
params:
image: conan
push: false

conan-maximum:
ci: github
Expand All @@ -224,6 +225,7 @@ tasks:
-e ARROW_CONAN_WITH_SNAPPY=True
-e ARROW_CONAN_WITH_ZSTD=True
image: conan
push: false

########################### Python Minimal ############################

Expand Down

0 comments on commit 96a6c45

Please sign in to comment.