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: update apko shas for v0.0.0 #16

Merged
merged 1 commit into from
Oct 12, 2023
Merged

chore: update apko shas for v0.0.0 #16

merged 1 commit into from
Oct 12, 2023

Conversation

kawfong
Copy link
Contributor

@kawfong kawfong commented Oct 11, 2023

I'm currently experiencing this error when I tried setting up rules_apko locally.

ERROR: User/Dev/bazel-monorepo/WORKSPACE.bazel:487:25: fetching apko_repositories rule //external:apko_darwin_arm64: Traceback (most recent call last):
        File "/private/var/tmp/_bazel_user/82451c28c0803a6357e2fc0181c91a49/external/rules_apko/apko/repositories.bzl", line 56, column 40, in _apko_repo_impl
                repository_ctx.download_and_extract(
Error in download_and_extract: java.io.IOException: Error downloading [https://github.com/thesayyn/apko/releases/download/v0.0.0/apko_0.0.0_darwin_arm64.tar.gz] to /private/var/tmp/_bazel_User/82451c28c0803a6357e2fc0181c91a49/external/apko_darwin_arm64/temp13831384609312796342/apko_0.0.0_darwin_arm64.tar.gz: Checksum was 397f12f138e255e9ef73bd4231646a64cfc928e22d4de2b12d8938f074d8f26d but wanted 902668329a29e1554f88554925556178bd5a5969813b5d6edcb8942e6713f77a
ERROR: User/Dev/bazel-monorepo/tools/oci_images/node_js_base_image/BUILD.bazel:8:11: //tools/oci_images/node_js_base_image:node_js_base_image depends on @apko_darwin_arm64//:apko_toolchain in repository @apko_darwin_arm64 which failed to fetch. no such package '@apko_darwin_arm64//': java.io.IOException: Error downloading [https://github.com/thesayyn/apko/releases/download/v0.0.0/apko_0.0.0_darwin_arm64.tar.gz] to /private/var/tmp/_bazel_User/82451c28c0803a6357e2fc0181c91a49/external/apko_darwin_arm64/temp13831384609312796342/apko_0.0.0_darwin_arm64.tar.gz: Checksum was 397f12f138e255e9ef73bd4231646a64cfc928e22d4de2b12d8938f074d8f26d but wanted 902668329a29e1554f88554925556178bd5a5969813b5d6edcb8942e6713f77a

I believe the issue happened because the SHAs in version.bzl are not matching with the SHAs in the release page of APKO.

I believe that the real fix should be on Apko side to release different versions instead of updating a SHA. Given that both projects are still at v0, I'm not sure what's the best way of doing this instead of creating a PR and update the SHA here.

Can you please provide a recommended solution, so updating version.bzl is no longer needed when APKO updates? Thank you

@kawfong kawfong marked this pull request as ready for review October 11, 2023 04:24
@thesayyn thesayyn merged commit 3380f98 into chainguard-dev:main Oct 12, 2023
3 of 8 checks passed
@kawfong kawfong deleted the chore/update_apko_versions branch October 13, 2023 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants