Skip to content
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

feat: Create and push docker images to hub #198

Merged
merged 10 commits into from
Apr 17, 2024
42 changes: 42 additions & 0 deletions .github/workflows/push-to-hub.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,42 @@
name: Build and push images to docker hub on merge to dev
run-name: ${{ github.actor }} is building and pushing images to docker hub

on: [push]
# on:
# push:
# branches:
# - "dev"

jobs:
Build-Push-Images-To-Docker-Hub:
runs-on: ubuntu-latest

strategy:
matrix:
directory: [agents-api, model-serving, gateway, memory-store]

steps:
Copy link
Contributor

Choose a reason for hiding this comment

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

The matrix strategy is commented out and not used in the workflow. This means that the workflow will not be able to build and push images for multiple services as intended. Please uncomment and properly use the matrix strategy.

- uses: actions/checkout@v4

- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3

- name: Login to Docker Hub
uses: docker/login-action@v3
with:
username: julepai
password: "${{ secrets.DOCKER_HUB_PASSWORD }}"

- name: Build images
run: |
cd ${{ matrix.directory }}
docker compose build

- name: Push images
run: |
cd ${{ matrix.directory }}
docker compose push

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
4 changes: 4 additions & 0 deletions agents-api/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ include:
# TODO: figure out why this doesn't work
services:
agents-api:
image: julepai/agents-api:dev
env_file:
- path: .env
required: false
Expand Down Expand Up @@ -40,6 +41,7 @@ services:
path: Dockerfile

worker:
image: julepai/worker:dev
env_file:
- path: .env
required: false
Expand Down Expand Up @@ -87,6 +89,7 @@ services:
capabilities: [gpu]

temporal:
image: julepai/temporal:dev
container_name: temporal
env_file:
- path: .env
Expand All @@ -108,6 +111,7 @@ services:
path: Dockerfile.temporal

cozo-migrate:
image: julepai/cozo-migrate:dev
env_file:
- path: .env
required: false
Expand Down
1 change: 1 addition & 0 deletions gateway/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ include:

services:
gateway:
image: julepai/gateway:dev
env_file:
- path: .env
required: false
Expand Down
1 change: 1 addition & 0 deletions memory-store/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ version: "3"

services:
memory-store:
image: julepai/memory-store:dev
env_file:
- path: .env
required: false
Expand Down
1 change: 1 addition & 0 deletions model-serving/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ name: julep-model-serving

services:
model-serving:
image: julepai/model-serving:dev
env_file:
- path: .env
required: false
Expand Down
Loading