Skip to content

Cache layers when building image in GHA#1687

Merged
theseanything merged 1 commit into
mainfrom
theseanything/cache-image-builds
Oct 21, 2025
Merged

Cache layers when building image in GHA#1687
theseanything merged 1 commit into
mainfrom
theseanything/cache-image-builds

Conversation

@theseanything
Copy link
Copy Markdown
Contributor

This improves the build process by caching layers in GitHub Actions, which speeds up build times and improves resilience to image repository outages.

This GHA workflow is only used during CI, to ensure an image can build without errors. Not the final image used in deployments.

This improves the build process by caching layers in GitHub Actions,
which speeds up build times and improves resilience to image repository
outages.

This GHA workflow is only used during CI, to ensure an image can build
without errors. Not the final image used in deployments.
@theseanything theseanything requested a review from whi-tw October 21, 2025 15:50
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown
Contributor

🎉 A review copy of this PR has been deployed! It is made of up two components

  1. A review copy of forms-runner
  2. A production copy of forms-admin

Important

Not all of the functionality of forms-runner is present in review apps.
Functionality such as sending emails, file upload, and S3 submission types are
deliberately disabled for the sake of simplifying review apps.

You should use the full dev environment to test the functionality which is disabled here.

It may take 5 minutes or so for the application to be fully deployed and working. If it still isn't ready
after 5 minutes, there may be something wrong with the ECS task. You will need to go to the integration AWS account
to debug, or otherwise ask an infrastructure person.

For the sign in details and more information, see the review apps wiki page.

@theseanything theseanything merged commit 3f8c752 into main Oct 21, 2025
8 checks passed
@theseanything theseanything deleted the theseanything/cache-image-builds branch October 21, 2025 16:05
lfdebrux added a commit that referenced this pull request Oct 28, 2025
Since we started using docker buildx in CI [[1]], we've been getting
check annotations with linting warnings for the Dockerfile in all PRs
(even ones which don't change the Dockerfile), which is very annoying.

This commit fixes all the warnings:

  - UndefinedVar
    - NODE_ENV needs to be declared as a build argument
    - RAILS_ENV needs to be declared as a build argument in all
      independent stages
    - There is unlikely to be a need to set REDIS_URL to anything
      specific, always set it to the same dummy value

[1]: #1687
lfdebrux added a commit that referenced this pull request Oct 31, 2025
Since we started using docker buildx in CI [[1]], we've been getting
check annotations with linting warnings for the Dockerfile in all PRs
(even ones which don't change the Dockerfile), which is very annoying.

This commit fixes all the warnings:

  - UndefinedVar
    - NODE_ENV needs to be declared as a build argument
    - RAILS_ENV needs to be declared as a build argument in all
      independent stages
    - There is unlikely to be a need to set REDIS_URL to anything
      specific, always set it to the same dummy value

[1]: #1687
lfdebrux added a commit that referenced this pull request Oct 31, 2025
Since we started using docker buildx in CI [[1]], we've been getting
check annotations with linting warnings for the Dockerfile in all PRs
(even ones which don't change the Dockerfile), which is very annoying.

This commit fixes all the warnings:

  - UndefinedVar
    - NODE_ENV needs to be declared as a build argument
    - RAILS_ENV needs to be declared as a build argument in all
      independent stages
    - There is unlikely to be a need to set REDIS_URL to anything
      specific, always set it to the same dummy value

[1]: #1687
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