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

Upload *.timing files on failure #2012

Merged
merged 6 commits into from
Feb 5, 2025
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions .github/workflows/coq-alpine.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,13 +80,21 @@ jobs:
shell: alpine.sh {0}
run: etc/ci/github-actions-make.sh -j2 generated-files
- run: tar -czvf generated-files.tgz fiat-*/
if: ${{ failure() }}
if: failure()
- name: upload generated files
uses: actions/upload-artifact@v4
with:
name: generated-files-${{ matrix.alpine }}
path: generated-files.tgz
if: ${{ failure() }}
if: failure()
- run: find . -name "*.timing" | xargs tar -czvf timing-files.tgz
if: failure()
- name: upload generated timing files
uses: actions/upload-artifact@v4
with:
name: timing-files-${{ matrix.alpine }}
path: timing-files.tgz
if: failure()
- name: standalone-haskell
shell: alpine.sh {0}
run: etc/ci/github-actions-make.sh -j1 standalone-haskell GHCFLAGS='+RTS -M7G -RTS'
Expand Down
12 changes: 10 additions & 2 deletions .github/workflows/coq-archlinux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,13 +45,21 @@ jobs:
- name: generated-files
run: etc/ci/github-actions-make.sh -j2 generated-files
- run: tar -czvf generated-files.tgz fiat-*/
if: ${{ failure() }}
if: failure()
- name: upload generated files
uses: actions/upload-artifact@v4
with:
name: generated-files-archlinux
path: generated-files.tgz
if: ${{ failure() }}
if: failure()
- run: find . -name "*.timing" | xargs tar -czvf timing-files.tgz
if: failure()
- name: upload generated timing files
uses: actions/upload-artifact@v4
with:
name: timing-files-archlinux
path: timing-files.tgz
if: failure()
- name: install-standalone-unified-ocaml
run: etc/ci/github-actions-make.sh install-standalone-unified-ocaml BINDIR=dist
#- name: standalone-js-of-ocaml
Expand Down
12 changes: 10 additions & 2 deletions .github/workflows/coq-debian.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,13 +50,21 @@ jobs:
- name: generated-files
run: etc/ci/github-actions-make.sh -j2 generated-files
- run: tar -czvf generated-files.tgz fiat-*/
if: ${{ failure() }}
if: failure()
- name: upload generated files
uses: actions/upload-artifact@v4
with:
name: generated-files-${{ matrix.debian }}
path: generated-files.tgz
if: ${{ failure() }}
if: failure()
- run: find . -name "*.timing" | xargs tar -czvf timing-files.tgz
if: failure()
- name: upload generated timing files
uses: actions/upload-artifact@v4
with:
name: timing-files-${{ matrix.debian }}
path: timing-files.tgz
if: failure()
- name: install-standalone-unified-ocaml
run: etc/ci/github-actions-make.sh install-standalone-unified-ocaml BINDIR=dist
- name: standalone-js-of-ocaml
Expand Down
12 changes: 10 additions & 2 deletions .github/workflows/coq-docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,14 @@ jobs:
with:
name: build-outputs-docker-coq-${{ matrix.env.DOCKER_COQ_VERSION }}-ocaml-${{ matrix.env.DOCKER_OCAML_VERSION }}
path: fiat-crypto-build.tar.gz
- run: find . -name "*.timing" | xargs tar -czvf timing-files.tgz
if: failure()
- name: upload generated timing files
uses: actions/upload-artifact@v4
with:
name: timing-files-${{ matrix.env.COQ_VERSION }}
path: timing-files.tgz
if: failure()
- name: install
uses: coq-community/docker-coq-action@v1
with:
Expand Down Expand Up @@ -371,13 +379,13 @@ jobs:
run: etc/ci/github-actions-make.sh --warnings -f Makefile.examples -j2 generated-files
if: github.event_name == 'pull_request'
- run: tar -czvf generated-files.tgz fiat-*/
if: ${{ failure() }}
if: failure()
- name: upload generated files
uses: actions/upload-artifact@v4
with:
name: generated-files-${{ matrix.coq-version }}
path: generated-files.tgz
if: ${{ failure() }}
if: failure()

standalone-haskell:
needs: build
Expand Down
8 changes: 8 additions & 0 deletions .github/workflows/coq-macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,14 @@ jobs:
run: opam exec -- etc/ci/github-actions-make.sh -j2 all
if: ${{ matrix.os.arch != 'arm64' }}
# js_of_ocaml is too heavy for M1 GH Action runners which have only 7GB RAM, cf https://github.com/ocsigen/js_of_ocaml/issues/1612, https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories
- run: find . -name "*.timing" | xargs tar -czvf timing-files.tgz
if: failure()
- name: upload generated timing files
uses: actions/upload-artifact@v4
with:
name: timing-files-${{ matrix.os.runs-on }}
path: timing-files.tgz
if: failure()
- name: install-standalone-unified-ocaml
run: opam exec -- etc/ci/github-actions-make.sh install-standalone-unified-ocaml BINDIR=dist
- name: install-standalone-js-of-ocaml
Expand Down
9 changes: 9 additions & 0 deletions .github/workflows/coq-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,15 @@ jobs:
run: opam exec -- bash etc/ci/github-actions-make.sh -j${{ env.NJOBS }} c-files lite-generated-files
- name: only-test-amd64-files-lite
run: opam exec -- bash etc/ci/github-actions-make.sh -j${{ env.NJOBS }} only-test-amd64-files-lite SLOWEST_FIRST=1
- run: Get-ChildItem -Recurse -Filter *.timing | %{ $_.FullName } | & { tar -czvf timing-files.tgz -InputObject $_ }
shell: pwsh
# if: failure()
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left if: failure() commented out because I'm not sure whether or not the PowerShell syntax is right (it was written by copilot), and I want CI to test it

- name: upload generated timing files
uses: actions/upload-artifact@v4
with:
name: timing-files-windows
path: timing-files.tgz
if: failure()
- name: upload OCaml files
uses: actions/upload-artifact@v4
with:
Expand Down
Loading