Skip to content

Commit

Permalink
sagemathgh-39003: CI Fix: Do not upload logs artifact for the default…
Browse files Browse the repository at this point in the history
… job in CI Linux

<!-- ^ Please provide a concise and informative title. -->
<!-- ^ Don't put issue numbers in the title, do this in the PR
description below. -->
<!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method
to calculate 1 + 2". -->
<!-- v Describe your changes below in detail. -->
<!-- v Why is this change required? What problem does it solve? -->
<!-- v If this PR resolves an open issue, please link to it here. For
example, "Fixes sagemath#12345". -->

fixes the failure seen in https://github.com/sagemath/sage/actions/runs/
11869791506/job/33080484685

The cause is duplicate uploading of artifacts of the same name.

Test:
https://github.com/kwankyu/sage/actions/runs/11906298470/job/33178163324

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation and checked the documentation
preview.

### ⌛ Dependencies

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->

URL: sagemath#39003
Reported by: Kwankyu Lee
Reviewer(s):
  • Loading branch information
Release Manager committed Nov 26, 2024
2 parents a1ea128 + 8377403 commit 52d4bdf
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 5 deletions.
1 change: 1 addition & 0 deletions .github/workflows/ci-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ jobs:
tox_packages_factors: >-
["standard"]
docker_push_repository: ghcr.io/${{ github.repository }}/
logs_artifact: false

# All platforms. This duplicates the default platform, but why not,
# it makes it more robust regarding random timeouts.
Expand Down
8 changes: 6 additions & 2 deletions .github/workflows/docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,9 @@ on:
description: 'Elapsed time (seconds) at which to kill the build'
default: 20000
type: number
logs_artifact:
default: true
type: boolean
#
# Publishing to GitHub Packages
#
Expand Down Expand Up @@ -260,11 +263,12 @@ jobs:
cp -r .tox/$TOX_ENV/* "artifacts/$LOGS_ARTIFACT_NAME"
rm -rf "artifacts/$LOGS_ARTIFACT_NAME"/{bin,lib,pyvenv.cfg}
if: always()
- uses: actions/upload-artifact@v4
- name: Upload logs artifact
uses: actions/upload-artifact@v4
with:
path: artifacts
name: ${{ env.LOGS_ARTIFACT_NAME }}
if: always()
if: always() && inputs.logs_artifact
- name: Print out logs for immediate inspection
# and markup the output with GitHub Actions logging commands
run: |
Expand Down
4 changes: 2 additions & 2 deletions build/pkgs/configure/checksums.ini
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
tarball=configure-VERSION.tar.gz
sha1=4b08e1c0e6b812bfbade59ef7c751619d8e44f81
sha256=9ae6ed35a2391f1af02ed2c7b855670d3db0bb421a69064a2c907b4f3a584eb4
sha1=cc1009b3adb05d4b3acfed07a82fb0485ecdaca1
sha256=4ef5e672d76f9a826d18b59444eab2cad92eb24acd3690b2b2007f04a919b915
2 changes: 1 addition & 1 deletion build/pkgs/configure/package-version.txt
Original file line number Diff line number Diff line change
@@ -1 +1 @@
95eb9b8c8642831f849b5fec82a0b545a95e9ca6
3ae56adac5fb11f7e448761ecf4cc992d5ddb928

0 comments on commit 52d4bdf

Please sign in to comment.