chore(earthbuild): fix docker-test-misc-* jobs #34
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
janishorsts
changed the title
janishorsts
changed the title
janishorsts
force-pushed
the
33-choreearthbuld-fix-docker-test-misc--in-docker-ci-ubuntu
branch
from
1eb65d0 to
93b5329
Compare
gilescope
reviewed
Aug 2, 2025
janishorsts
deleted the
33-choreearthbuld-fix-docker-test-misc--in-docker-ci-ubuntu
branch
kmannislands
added a commit
that referenced
this pull request
Aug 6, 2025
* explore structure and dependencies of the initial CI build * remove dependency on private earthly mirror, move to earthbuild actions setup. TODO: switch to ECR public or google public mirror instead or just removing mirroring entirely? * fix +lint-docs * attempt to solve container registry issues blocking all CI jobs (other than doc lint) that have run so far * weird issue where earthly is using docker.io/earthly/buildkitd:dev-get-ci-working as the buildkitd image--unclear why. perhaps a sequencing logic issue where the CI builds attempt to use the previous binary and cannot handle a failure of the previous job? restore cache from to see if that is somehow related * figure out root of the wrong bootstrap image problem * try adding permissions to the github actions definition. TODO: see if settings in the UI need adjustment * move all SAVE IMAGE calls to ghcr for now, using a global variable * get another place that I missed with --DEFAULT_BUILDKITD_IMAGE * another place where implicit docker.io was missed 🤦 * try to fix stage-2 build * --wip-- try to get build post stage-2 working * add back mirror settting, switching to gcr mirror instead of earthly corp * apply giles/janis review feedback: align arg names used to build image registry paths with dind repo terms * resolve action lint issues by adding back workflow params for compatibility (for now) * try switching to env.GITHUB_TOKEN which claude believes to automatically be available * attempt to fix where we source the earthly binary from * fix reference format to the moved buildkitd image as well * comment out auto-skip for now since it requires earthly cloud to work * align actual image name with what the CI pipeline expects--still using a single repository for now * simple update, conmtinuing to use the main repos GHCR for the buildkit intermediary img * chore(earthbuild): fix +lint-all (#22) * chore(earthbuild): fix +lint-all * chore: run Podman CI Ubuntu on any pull_request * remove unused verbose field * undo Podman CI Ubuntu workflow changes * chore: fix docker-test-misc job (#26) * ci: update race-test image name (#25) Currently, the race-test image for a CI build step is generated the step before with the new EarthBuild name: ghcr.io/earthbuild/earthbuild:buildkitd-race-test. The next step references the original Earthly name: earthly/buildkitd:race-test. This causes a build failure. Update to the new name. Co-authored-by: Squirrel <[email protected]> * chore(earthbuild): fix docker-tests-no-qemu-group9 job in Docker CI Ubuntu (#29) * chore(earthbuild): fix +docker-tests-no-qemu-group4 (#27) * fix podman teardown (#30) * feat: don't login to docker if not using docker * fix examples1 (#31) * feat: don't login to docker if not using docker * fix: examples1/grpc example * chore(earthbuild): fix docker-test-misc-* jobs (#34) * chore: fix docker-earthly-image-test / earthly-image-tests (#53) --------- Co-authored-by: Janis Horsts <[email protected]> Co-authored-by: John Moon <[email protected]> Co-authored-by: Squirrel <[email protected]> Co-authored-by: Giles Cope <[email protected]>
kmannislands
added a commit
that referenced
this pull request
Aug 20, 2025
* explore structure and dependencies of the initial CI build * remove dependency on private earthly mirror, move to earthbuild actions setup. TODO: switch to ECR public or google public mirror instead or just removing mirroring entirely? * fix +lint-docs * attempt to solve container registry issues blocking all CI jobs (other than doc lint) that have run so far * weird issue where earthly is using docker.io/earthly/buildkitd:dev-get-ci-working as the buildkitd image--unclear why. perhaps a sequencing logic issue where the CI builds attempt to use the previous binary and cannot handle a failure of the previous job? restore cache from to see if that is somehow related * figure out root of the wrong bootstrap image problem * try adding permissions to the github actions definition. TODO: see if settings in the UI need adjustment * move all SAVE IMAGE calls to ghcr for now, using a global variable * get another place that I missed with --DEFAULT_BUILDKITD_IMAGE * another place where implicit docker.io was missed 🤦 * try to fix stage-2 build * --wip-- try to get build post stage-2 working * add back mirror settting, switching to gcr mirror instead of earthly corp * apply giles/janis review feedback: align arg names used to build image registry paths with dind repo terms * resolve action lint issues by adding back workflow params for compatibility (for now) * try switching to env.GITHUB_TOKEN which claude believes to automatically be available * attempt to fix where we source the earthly binary from * fix reference format to the moved buildkitd image as well * comment out auto-skip for now since it requires earthly cloud to work * align actual image name with what the CI pipeline expects--still using a single repository for now * simple update, conmtinuing to use the main repos GHCR for the buildkit intermediary img * chore(earthbuild): fix +lint-all (#22) * chore(earthbuild): fix +lint-all * chore: run Podman CI Ubuntu on any pull_request * remove unused verbose field * undo Podman CI Ubuntu workflow changes * chore: fix docker-test-misc job (#26) * ci: update race-test image name (#25) Currently, the race-test image for a CI build step is generated the step before with the new EarthBuild name: ghcr.io/earthbuild/earthbuild:buildkitd-race-test. The next step references the original Earthly name: earthly/buildkitd:race-test. This causes a build failure. Update to the new name. Co-authored-by: Squirrel <[email protected]> * chore(earthbuild): fix docker-tests-no-qemu-group9 job in Docker CI Ubuntu (#29) * chore(earthbuild): fix +docker-tests-no-qemu-group4 (#27) * fix podman teardown (#30) * feat: don't login to docker if not using docker * feat: OIDC not needed. Also remove EARTHLY_TOKEN * feat: remove secret integration * fix examples1 (#31) * feat: don't login to docker if not using docker * fix: examples1/grpc example * fix: make compile * feat: web test used earthly specific tech so we don't need to run it * feat: test not needed * fix: remove account login * fix: remove earthly cloud secret refs * fix: lint errors * fix: remove ref to earthly secret * fix: fix lint * fix: remove cloud test * fix: update docker2earth test * fix: analytics removed * fix: use EarthBuild cache * fix: point to EarthBuild image-test * fix: let's just delete the cloud docs * fix: restore some of aws-flag.earth * chore(earthbuild): fix docker-test-misc-* jobs (#34) * fix: go.sum not found * fix: restore oidc code * fix: stray fi from merge * fix: skip az test until we setup a login * fix: skip warn test * fix: skip aws test as no login * fix: skip gcp test as no login * putting back oidc bits * we intend to bring back this test * Cloud secrets will still be a thing. * Put back OIDC and cloud secret docs * Update docs/guides/best-practices.md * fix: trailing newlines * fix: ci * fix: debian stable v13 doesn't have software-properties-common --------- Co-authored-by: Kieran Mann <[email protected]> Co-authored-by: Janis Horsts <[email protected]> Co-authored-by: John Moon <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WIP
readaccess to dindtest private repo https://github.com/orgs/EarthBuild/packages/container/dindtest/settings