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
lets just try dockah #15585
lets just try dockah #15585
Conversation
This reverts commit f03fd35.
sudo apt -y install libcurl4-openssl-dev libssl-dev pkg-config libxml2-dev libxslt-dev | ||
- uses: actions/setup-python@v5 | ||
- name: Expose GitHub Runtime | ||
uses: crazy-max/ghaction-github-runtime@v3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exposes the necessary variables for the gha
cache type to work for docker layers, see https://docs.docker.com/build/cache/backends/gha/#authentication
cache-source: var-lib-apt | ||
cache-target: /var/lib/apt | ||
skip-extraction: ${{ steps.cache-var-lib-apt.outputs.cache-hit }} | ||
- name: inject root-cache-pip into docker |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these combined with the normal "cache" steps help us persist RUN --mount=type=cache
caches between runs improving our pip/apt/npm install times.
- name: Dependencies | ||
command: bin/github-actions-deps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
something goes wrong trying to run git in the container. without the gha specific deps job... we burn an extra minute of CI every run :/
@@ -39,7 +39,7 @@ build-mos: compile-pot | |||
translations: compile-pot | |||
ifneq ($(GITHUB_ACTIONS), false) | |||
cd ../../; \ | |||
git diff --quiet ./warehouse/locale/messages.pot || (echo "There are outstanding translations, run 'make translations' and commit the changes."; exit 1) | |||
cmp -s ./warehouse/locale/messages.pot /opt/warehouse/src/warehouse/locale/messages.pot || (echo "There are outstanding translations, run 'make translations' and commit the changes."; exit 1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this leverages the fact that we have a copy of the messages.pot from the build step since git diff doesn't work in the container.
everything else involved (dev, prod) is docker, why not ci?
yeee, fucking, haw.
closes #15583.