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

chore(kernel_crawler): updated archlinux arm64 mirror. #196

Merged
merged 2 commits into from
Apr 3, 2024

Conversation

FedeDP
Copy link
Collaborator

@FedeDP FedeDP commented Apr 3, 2024

What type of PR is this?

/kind cleanup

Any specific area of the project related to this PR?

/area crawler

What this PR does / why we need it:

The old mirror is often down and it is not updated anymore (latest update 2022, kernel 5.19.8).

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Signed-off-by: Federico Di Pierro <nierro92@gmail.com>
Signed-off-by: Federico Di Pierro <nierro92@gmail.com>
@@ -35,16 +35,16 @@ runs:
shell: bash
working-directory: ${{ github.action_path }}
run: |
kernel-crawler crawl --distro=${{ inputs.distro }} --arch=${{ inputs.arch }} > kernels_${{ inputs.arch }}.json
kernel-crawler crawl --distro=${{ inputs.distro }} --arch=${{ inputs.arch }} > ${{ runner.temp }}/kernels_${{ inputs.arch }}.json
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This fixes the test.yml CI, since before it was failing with:


test kernel-crawler
Invalid pattern '/home/runner/work/kernel-crawler/kernel-crawler/.//kernels_x86_64.json'. Relative pathing '.' and '..' is not allowed.

because ${{ github.action_path }} was relative to the root folder.
Using runner temporary folder yelds better results.

@poiana poiana added the lgtm label Apr 3, 2024
@FedeDP FedeDP added the approved label Apr 3, 2024
@poiana poiana merged commit 2fecb92 into main Apr 3, 2024
5 checks passed
@poiana poiana deleted the chore/update_archlinuxarm_mirror branch April 3, 2024 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants