Skip to content

Avoid API limit by querying Docker Hub unauthenticated in Github Actions, and build only if needed#442

Merged
Quetzacoalt91 merged 4 commits into
PrestaShop:masterfrom
Quetzacoalt91:build-then-push
Sep 8, 2025
Merged

Avoid API limit by querying Docker Hub unauthenticated in Github Actions, and build only if needed#442
Quetzacoalt91 merged 4 commits into
PrestaShop:masterfrom
Quetzacoalt91:build-then-push

Conversation

@Quetzacoalt91

@Quetzacoalt91 Quetzacoalt91 commented Aug 27, 2025

Copy link
Copy Markdown
Member
Questions Answers
Description? Building all the PrestaShop images fails because of the number of API calls allowed reached in a limited period of time. Pulls from Docker Hub could be allowed without limit when we are not authenticated on GitHub Actions environments. This PR limits the risk of quota exceeded by:
  • pulling without being authenticated,
  • building images only if a change has been found.
Type? improvement
BC breaks? Nope
Deprecations? Nope
Fixed ticket? /
Sponsor company @PrestaShopCorp
How to test? Builds succeed on a fork
image

@Quetzacoalt91
Quetzacoalt91 force-pushed the build-then-push branch 3 times, most recently from 9da5839 to 5c15fd0 Compare August 29, 2025 13:57
@Quetzacoalt91
Quetzacoalt91 force-pushed the build-then-push branch 7 times, most recently from 62f8241 to 989f07e Compare September 5, 2025 15:25
@Quetzacoalt91 Quetzacoalt91 changed the title Avoid API limit by querying Docker Hub unauthenticated in Github Actions Avoid API limit by querying Docker Hub unauthenticated in Github Actions, and build only if needed Sep 5, 2025
@Quetzacoalt91
Quetzacoalt91 merged commit 12e305e into PrestaShop:master Sep 8, 2025
2 checks passed
@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Sep 8, 2025
@Quetzacoalt91
Quetzacoalt91 deleted the build-then-push branch September 8, 2025 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants