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

v0.6.15 coveralls-checksums.txt has incorrect path prefix (release/) #165

Closed
adityasaky opened this issue Oct 2, 2024 · 7 comments
Closed
Labels
bug Something isn't working

Comments

@adityasaky
Copy link

What was wrong?

We noticed coverage runs via the github action failing because the path specified in the latest release's checksums file doesn't match the location of the binary in question. It looks like the latest release includes a release/ prefix which isn't present in at the very least the last release.

Logs

Our CI run is at https://github.com/gittuf/gittuf/actions/runs/11147824919/job/30983463583?pr=578#step:5:33

  mkdir -p ~/bin/
  cd ~/bin/
  if [ $COVERAGE_REPORTER_VERSION == "latest" ]
  then
    asset_path=latest/download
  else
    asset_path="download/${COVERAGE_REPORTER_VERSION}"
  fi
  curl -sLO "[https://github.com/coverallsapp/coverage-reporter/releases/${asset_path}/coveralls-linux.tar.gz](https://github.com/coverallsapp/coverage-reporter/releases/$%7Basset_path%7D/coveralls-linux.tar.gz)"
  curl -sLO "[https://github.com/coverallsapp/coverage-reporter/releases/${asset_path}/coveralls-checksums.txt](https://github.com/coverallsapp/coverage-reporter/releases/$%7Basset_path%7D/coveralls-checksums.txt)"
  cat coveralls-checksums.txt | grep coveralls-linux.tar.gz | sha256sum --check
  tar -xzf coveralls-linux.tar.gz
  rm coveralls-checksums.txt
  echo ~/bin >> $GITHUB_PATH
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    COVERAGE_REPORTER_VERSION: latest
sha256sum: release/coveralls-linux.tar.gz: No such file or directory
sha256sum: WARNING: 1 listed file could not be read
release/coveralls-linux.tar.gz: FAILED open or read
Error: Process completed with exit code 1.
@afinetooth
Copy link
Member

@adityasaky Thanks for the report. We are working on it now. Will update you here.

@randseay
Copy link

randseay commented Oct 2, 2024

+1 this is also affecting our team today.

Run mkdir -p ~/bin/
  mkdir -p ~/bin/
  cd ~/bin/
  curl -sLO https://github.com/coverallsapp/coverage-reporter/releases/latest/download/coveralls-linux.tar.gz
  curl -sLO https://github.com/coverallsapp/coverage-reporter/releases/latest/download/coveralls-checksums.txt
  cat coveralls-checksums.txt | grep coveralls-linux.tar.gz | sha[2](https://github.com/dispel/dispel/actions/runs/11148756317/job/30986336387#step:6:2)56sum --check
  tar -xzf coveralls-linux.tar.gz
  rm coveralls-checksums.txt
  echo ~/bin >> $GITHUB_PATH
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
sha256sum: release/coveralls-linux.tar.gz: No such file or directory
release/coveralls-linux.tar.gz: FAILED open or read
sha256sum: WARNING: 1 listed file could not be read
Error: Process completed with exit code 1.

@afinetooth
Copy link
Member

afinetooth commented Oct 2, 2024

Workarounds for now

For users of coverage-reporter directly:

Instead of this install step:

curl -L https://coveralls.io/coveralls-linux.tar.gz | tar -xz -C /usr/local/bin

Download and install the latest working version (v0.6.14):

curl -L https://github.com/coverallsapp/coverage-reporter/releases/download/v0.6.14/coveralls-linux.tar.gz | tar -xz -C /usr/local/bin

Suggestion for users of the Coveralls GitHub Action:

set fail-on-error input option to false for now:

fail-on-error: false

Suggestion for users of the Coveralls Orb for CircleCI:

set fail_on_error input option to false for now:

fail_on_error: false

@afinetooth
Copy link
Member

@adityasaky We believe we've patched this for now by updating the coveralls-checksums.txt file.

Can you please try v0.6.15 again to see if it works for you now?

@adityasaky
Copy link
Author

Looks good! https://github.com/gittuf/gittuf/actions/runs/11147824919/job/30988431835

@afinetooth
Copy link
Member

Awesome. Thank you! ❤️

We're now fixing the issue for subsequent releases.

@afinetooth
Copy link
Member

afinetooth commented Oct 2, 2024

Root cause analysis (by request) here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants