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.

Sign up for GitHub

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

Jump to bottom

Verify build fix with prisma field encryption inclusion #65

Merged
merged 14 commits into from
Sep 18, 2024
Merged

Verify build fix with prisma field encryption inclusion #65

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
c30edd1
Temp: Verify build fix with prisma field encryption inclusion
BMartinos Sep 17, 2024
a0582fa
Fix tagged build
BMartinos Sep 17, 2024
1976b1c
Fix indentation issue
BMartinos Sep 17, 2024
7b71328
Fix typo
BMartinos Sep 17, 2024
a87f6ff
Sanatise the branch name for docker tag usage
BMartinos Sep 17, 2024
31927f5
Add updated workflow for GHA
BMartinos Sep 17, 2024
15ce199
Update Production Dockerfile process
BMartinos Sep 17, 2024
1c4201f
Temp: test the docker build in GHA workflow
BMartinos Sep 17, 2024
51df2d5
Merge branch 'main' of github.com:jembi/smart-health-links-portal int…
BMartinos Sep 17, 2024
69012dd
Merge branch 'fix/production-build-prisma-field-encryption' into upda…
BMartinos Sep 17, 2024
bce4425
Merge pull request #69 from jembi/update-docker-production-image-build
BMartinos Sep 17, 2024
69c7a02
Remove old GHA workflow files
BMartinos Sep 17, 2024
790467e
only one plaform needed for build test
BMartinos Sep 17, 2024
739005d
Fix incorrect branch name refernce
BMartinos Sep 17, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .env.production.example
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,5 @@ SHLP_POSTGRES_DB=
SHLP_POSTGRES_USERNAME=
SHLP_POSTGRES_PASSWORD=
POSTGRES_PRISMA_URL=postgresql://${SHLP_POSTGRES_USERNAME}:${SHLP_POSTGRES_PASSWORD}@${SHLP_POSTGRES_HOST}:5432/${SHLP_POSTGRES_DB}
PRISMA_FIELD_ENCRYPTION_KEY=
PRISMA_FIELD_ENCRYPTION_HASH_SALT=
PRISMA_FIELD_ENCRYPTION_KEY=k1.aesgcm256.Wmy6koJaJpjB0H25qBccLfQ84AvpmCjexanqCHJ5Hr0=
PRISMA_FIELD_ENCRYPTION_HASH_SALT=1a4671c31ff22834a56d4f3741aeac8cc8662a11c73802083ea5a118e5ee600e
BMartinos marked this conversation as resolved.
Show resolved Hide resolved
37 changes: 37 additions & 0 deletions .github/workflows/run-tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ on:
pull_request:
branches:
- main
- fix/production-build-prisma-field-encryption

jobs:
test:
Expand All @@ -27,6 +28,42 @@ jobs:
- name: Run Jest Tests
run: yarn test --collectCoverage=false

build-and-push:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4

- name: Set up QEMU
uses: docker/setup-qemu-action@v3

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

- name: Login to DockerHub
uses: docker/login-action@v3
with:
username: ${{ secrets.DOCKERHUB_USERNAME }}
password: ${{ secrets.DOCKERHUB_TOKEN }}

- name: Build and push tag
if: ${{ github.ref_name != 'main' }}
uses: docker/build-push-action@v5
with:
platforms: linux/amd64,linux/arm64
push: true
file: docker/production/Dockerfile
tags: jembi/smart-health-links-portal:${{ github.ref_name }}
BMartinos marked this conversation as resolved.
Show resolved Hide resolved

- name: Build and push latest
if: ${{ github.ref_name == 'main' }}
uses: docker/build-push-action@v5
with:
platforms: linux/amd64,linux/arm64
push: true
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we not check if it is nit on pull request before we push the image?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@jacob-khoza-symb Yup thats correct, I still need to make some updates on this action script.
The current changes are just intended for testing purposes to have an image build and sent to dockerhub to verify the build works as intended (without having to merge into master multiple times)

Side note: The entire GHA needs a bit of refactoring and improvement

file: docker/production/Dockerfile
tags: jembi/smart-health-links-portal:latest

# coverage:
# runs-on: ubuntu-latest
# needs: test
Expand Down
Loading