-
Notifications
You must be signed in to change notification settings - Fork 35
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
Doesn't work on custom containers ubuntu18:04 #71
Comments
Does this work on another configuration? Also, can you try with an older release? |
Thank you for your reply. |
I've tried with 1.0.1 but same issue. |
Can you share your workflow file and the full log as well? Also, can you tell me more about this custom container setup? |
Thank you for the reply. Unfortunately I can't disclose the rest of the workflow file, but I made a test repository here: https://github.com/emiliencartesoft/test-apt-cache/actions As you can see the full logs are here: https://github.com/emiliencartesoft/test-apt-cache/actions/runs/3581373128/jobs/6024358104 |
Found the issue, in https://github.com/awalsh128/cache-apt-pkgs-action/blob/master/action.yml Changing from |
Workaround for #71. Due to a bug in the runner actions/runner#716
Nice find. I updated the dev branch with the workaround so you can use that as |
Well it works with the example packages (ca-certificates build-essential) and when I did test with other packages like Anyway with the commit 343fbfd it works in custom containers, at least when |
Sorry for the lag here. RE: Action path - Looks like this was mentioned before. Let me know if the situation changes make the container option modification. |
Well for me the changes in the commit 343fbfd should be merged into the master branch because it fixes the issue with custom containers. |
I agree. I am going to put in that change as well and issue a patch tonight. I meant the container specific one but I think I misunderstood and this solved it completely for you. |
Issued a patch update in v1.2.1. Let me know how it works and I will rev up the tag aliases for general use (i.e. |
* Workaround for #71. Due to a bug in the runner actions/runner#716 (#75) * docs: fix link to actions/cache (#77) Co-authored-by: Kévin Dunglas <kevin@dunglas.fr>
Going to assume this worked and close out this issue since it has been awhile. |
name: Test webrtc thread leak
on: [push, pull_request]
jobs:
test-webrtc-thread-leak:
runs-on: ubuntu-22.04
container: ubuntu:24.04
env:
DEBIAN_FRONTEND: noninteractive
steps:
- name: Install build dependencies
uses: awalsh128/cache-apt-pkgs-action@v1
with:
packages: libunwind-dev
- name: Checkout
uses: actions/checkout@v4 output: Run awalsh128/cache-apt-pkgs-action@v1
with:
packages: libunwind-dev
execute_install_scripts: false
debug: false
env:
DEBIAN_FRONTEND: noninteractive
Run ${GITHUB_ACTION_PATH}/pre_cache_action.sh \
${GITHUB_ACTION_PATH}/pre_cache_action.sh \
~/cache-apt-pkgs \
"$VERSION" \
"$EXEC_INSTALL_SCRIPTS" \
"$DEBUG" \
"$PACKAGES"
echo "CACHE_KEY=$(cat ~/cache-apt-pkgs/cache_key.md5)" >> $GITHUB_ENV
shell: bash --noprofile --norc -e -o pipefail {0}
env:
DEBIAN_FRONTEND: noninteractive
VERSION:
EXEC_INSTALL_SCRIPTS: false
DEBUG: false
PACKAGES: libunwind-dev
22:06:54.398 Normalizing package list...
22:06:54.426 done
22:06:54.428 Validating action arguments (version='', packages='')...
22:06:54.430 aborted
22:06:54.431 Packages argument cannot be empty.
Error: Process completed with exit code 3. |
My bad, my problem was another thing. |
Hello,
I'm unable to make this project working when using a custom container.
Here is an example yaml file:
And here is the error:
Could you help me in making it work with a custom container?
Thank you.
The text was updated successfully, but these errors were encountered: