Skip to content

Commit 968fb65

Browse files
ebrMillu
andauthored
Report ci disk space + minor docker fixes (#5461)
* ci: add docker build timout; log free space on runner before and after build * docker: bump frontend builder to node=20.x; skip linting on build * chore: gitignore .pnpm-store * update code owners for docker and CI --------- Co-authored-by: Millun Atluri <[email protected]>
1 parent 80ec9f4 commit 968fb65

File tree

4 files changed

+11
-7
lines changed

4 files changed

+11
-7
lines changed

.github/CODEOWNERS

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# continuous integration
2-
/.github/workflows/ @lstein @blessedcoolant @hipsterusername
2+
/.github/workflows/ @lstein @blessedcoolant @hipsterusername @ebr
33

44
# documentation
55
/docs/ @lstein @blessedcoolant @hipsterusername @Millu
@@ -10,7 +10,7 @@
1010

1111
# installation and configuration
1212
/pyproject.toml @lstein @blessedcoolant @hipsterusername
13-
/docker/ @lstein @blessedcoolant @hipsterusername
13+
/docker/ @lstein @blessedcoolant @hipsterusername @ebr
1414
/scripts/ @ebr @lstein @hipsterusername
1515
/installer/ @lstein @ebr @hipsterusername
1616
/invokeai/assets @lstein @ebr @hipsterusername
@@ -26,9 +26,7 @@
2626

2727
# front ends
2828
/invokeai/frontend/CLI @lstein @hipsterusername
29-
/invokeai/frontend/install @lstein @ebr @hipsterusername
29+
/invokeai/frontend/install @lstein @ebr @hipsterusername
3030
/invokeai/frontend/merge @lstein @blessedcoolant @hipsterusername
3131
/invokeai/frontend/training @lstein @blessedcoolant @hipsterusername
3232
/invokeai/frontend/web @psychedelicious @blessedcoolant @maryhipp @hipsterusername
33-
34-

.github/workflows/build-container.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,14 @@ jobs:
4040
- name: Free up more disk space on the runner
4141
# https://github.com/actions/runner-images/issues/2840#issuecomment-1284059930
4242
run: |
43+
echo "----- Free space before cleanup"
44+
df -h
4345
sudo rm -rf /usr/share/dotnet
4446
sudo rm -rf "$AGENT_TOOLSDIRECTORY"
4547
sudo swapoff /mnt/swapfile
4648
sudo rm -rf /mnt/swapfile
49+
echo "----- Free space after cleanup"
50+
df -h
4751
4852
- name: Checkout
4953
uses: actions/checkout@v3
@@ -91,6 +95,7 @@ jobs:
9195
# password: ${{ secrets.DOCKERHUB_TOKEN }}
9296

9397
- name: Build container
98+
timeout-minutes: 40
9499
id: docker_build
95100
uses: docker/build-push-action@v4
96101
with:

docker/Dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ RUN --mount=type=cache,target=/root/.cache/pip \
5959

6060
# #### Build the Web UI ------------------------------------
6161

62-
FROM node:18-slim AS web-builder
62+
FROM node:20-slim AS web-builder
6363
ENV PNPM_HOME="/pnpm"
6464
ENV PATH="$PNPM_HOME:$PATH"
6565
RUN corepack enable
@@ -68,7 +68,7 @@ WORKDIR /build
6868
COPY invokeai/frontend/web/ ./
6969
RUN --mount=type=cache,target=/pnpm/store \
7070
pnpm install --frozen-lockfile
71-
RUN pnpm run build
71+
RUN npx vite build
7272

7373
#### Runtime stage ---------------------------------------
7474

invokeai/frontend/web/.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ pnpm-debug.log*
88
lerna-debug.log*
99

1010
node_modules
11+
.pnpm-store
1112
# We want to distribute the repo
1213
dist
1314
dist/**

0 commit comments

Comments
 (0)