Skip to content

Fixes: expected docker context is increased due to WF merge#1893

Merged
sadiqkhoja merged 1 commit into
getodk:nextfrom
sadiqkhoja:fixes/docker-context
May 14, 2026
Merged

Fixes: expected docker context is increased due to WF merge#1893
sadiqkhoja merged 1 commit into
getodk:nextfrom
sadiqkhoja:fixes/docker-context

Conversation

@sadiqkhoja

@sadiqkhoja sadiqkhoja commented May 14, 2026

Copy link
Copy Markdown
Contributor

Size of git db (objects) has increase quite a bit after merging web-forms code into central-frontend, therefore increasing expected file size of docker context.

I am wondering if there is a way not to include git objects in the docker context but we rely on them to generate version.txt. Maybe once we start publishing built images we can get rid of them.

PS: Found this issue while working on a separate issue. To understand the root cause I rerun GH Actions on next and master branches even those were failing with everything (OS, runner, git version, docker version) being same e.g. https://github.com/getodk/central/actions/runs/25595119221/job/76010520926. Only difference I could see was git/objects/pack/pack*.

Before submitting this PR, please make sure you have:

  • branched off and targeted the next branch OR only changed documentation/infrastructure (master is stable and used in production)
  • verified that any code or assets from external sources are properly credited in comments or that everything is internally sourced

@sadiqkhoja
sadiqkhoja requested a review from lognaturel May 14, 2026 15:51

@lognaturel lognaturel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approving to fix CI in the short term and because I agree we should review tagging and version generation as part of what @alxndrsn and @garethbowen are working on to pre-build images.

An alternate small fix would be to set the fetch depth to something like 50 instead of 0 (unlimited). That should pull the tagged commits we need without increasing the download size so much.

@sadiqkhoja
sadiqkhoja merged commit 448b96a into getodk:next May 14, 2026
6 checks passed
@lognaturel

Copy link
Copy Markdown
Member

We've talked about this plenty but it's another example of how our installs are not exactly reproducible. CC @yanokwa

@alxndrsn

Copy link
Copy Markdown
Contributor

Seems like the new file count window is very big.

@sadiqkhoja

Copy link
Copy Markdown
Contributor Author

Talked with @alxndrsn on Slack, and he is suggesting that we increase the min-size value as well so that expected size window is not too large. I'll create a new PR to do that.

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.

3 participants