82 feature request suggestion create GitHub actions workflow for automating the build and deployments for feature branches #87

lmcdonough · 2024-12-13T19:14:03Z

This pull request introduces a new GitHub Actions workflow to build and deploy Docker containers for Rust and Next.js projects. The workflow includes steps for building, testing, and pushing Docker images to the GitHub Container Registry.

GitHub Issue: [Closes] #82

Changes

Added a new Github Actions workflow configuration file .github/workflows/build_and_deploy_containers.yml to automate the build and deployment process for feature branches and pull requests to the main branch.
Defined two main jobs: build_test_run for building and testing the project, and build_and_push_docker for building and pushing Docker images. The latter job depends on the successful completion of the former.
Configured steps to checkout the repository, install the Rust toolchain, use cached dependencies, install seaORM CLI, build the project, and run tests.
Configured steps to checkout the repository, log in to the GitHub Container Registry, set up Docker Buildx, cache Docker layers, and build and push Docker images for both the Rust backend and the Next.js frontend.
Implemented a caching mechanism for Docker layers to optimize the build process, including steps## Description

Testing Strategy

Manually use the workflow_dispatch to iterate and view any errors from the workflow run.

Concerns

Managing environment variables, and secrets can be improved upon in the future.

… with updated actions, improved environment variable handling, and added artifact attestations for images.

…rontend images

…rams. Also accept a list of CORS allowed origin URLs instead of hardcoding them.

…he Rust backend expects

…actor_platform_rs binary

…tions are set and passed from the backend/docker-compose.yaml

…ions

also fixes test

There is a lot of duplicated code here. The plan is to defer refactoring for now and take a look at a better error handling system.

Improving the bug reporting template.

From README: Layer of abstraction above `entity_api` and intended to encapsulate most business logic. Ex. interactions between `entity_api` and network calls to the outside world.

I will be extending this pattern and using code does not quite fit in my mind

Considering a record not found as an Error to improve upstream ergonomics

Co-authored-by: Jim Hodapp <[email protected]>

Clap was complaining and just having a -t seems not super expressive

lmcdonough added 2 commits December 13, 2024 13:43

Add GitHub Actions workflow for building and deploying containers

a367882

fixed error with pr to main.

f9fd508

lmcdonough linked an issue Dec 13, 2024 that may be closed by this pull request

[Feature Request / Suggestion]: Create Github Actions Workflow for automating the build and deployments for feature branches #82

Open

4 tasks

lmcdonough added 2 commits December 13, 2024 14:45

updates env vars to be injected into images on build

b027a13

updates database url

18bd87d

lmcdonough self-assigned this Dec 13, 2024

lmcdonough requested a review from jhodapp December 13, 2024 19:56

lmcdonough added the feature work Specifically implementing a new feature label Dec 13, 2024

jhodapp added this to the 0.10 - Container deployment to a staging or production server milestone Dec 13, 2024

lmcdonough and others added 21 commits December 14, 2024 19:01

Enhance GitHub Actions workflow for building and deploying containers…

a38b35c

… with updated actions, improved environment variable handling, and added artifact attestations for images.

correcting attestations

4300b78

Fix subject-name formatting in artifact attestation for backend and f…

a03bd08

…rontend images

Enable the use of .env for setting runtime variables alongside CLI pa…

a4790fa

…rams. Also accept a list of CORS allowed origin URLs instead of hardcoding them.

Tweak some of the environment variable names to align with the ones t…

32c03cd

…he Rust backend expects

Display the DB URI and make sure the env variables make it to the ref…

26cf882

…actor_platform_rs binary

All environment variables are now being passed through correctly.

00e402b

cargo fmt

6ae1c55

Update the container README to include a full .env example

c2594e8

All environment variables for the dockerized backend/frontend applica…

1b37624

…tions are set and passed from the backend/docker-compose.yaml

Update example .env files in the runbook README

f7c518c

Fix the invalid type 'status' when inserting OverarchingGoals and Act…

1c78d2a

…ions

scope coaching_relationships query by user_id

3629ce5

basic scaffolding for custom authorization functions

2873053

add permissions for coaching_relationships by organization

ac40f17

add coaching_relationship::find_by_id

c29aec4

add protect::coaching_relationships::index

9405557

fix uuid parsing

4af8909

add coaching_session::find_by_id

4e1a27a

refactor how query params are deserialized

6128ba5

add protect::overarching_goals::index

d1b0355

also fixes test

calebbourg and others added 29 commits February 7, 2025 21:00

add index authz for coaching session resources

449523a

There is a lot of duplicated code here. The plan is to defer refactoring for now and take a look at a better error handling system.

apply authorization to specific routes

bb1e734

Update bug_report.yaml

cb44998

Improving the bug reporting template.

add slug columns

9b97b79

add domain crate

3b21caa

From README: Layer of abstraction above `entity_api` and intended to encapsulate most business logic. Ex. interactions between `entity_api` and network calls to the outside world.

add domain::coaching_session

32c40ac

set versions to 1.0.0-beta1

2727681

update SQL comments

2627036

add and update reqwest

445f4c3

change EntityErrorCode to EntityErrorKind

9fe8f2e

I will be extending this pattern and using code does not quite fit in my mind

refactor error tree structure and conversions

ed90732

change find_by_id semantics

76ed0f1

Considering a record not found as an Error to improve upstream ergonomics

improve TipTap document creation code

524695d

successful creation of tiptap document

d248e3c

add error documentation

fe5782b

improve error logging

7256aaa

fix failing test

438f2e7

use Config struct for Tiptap secrets

586f9c5

Update service/src/config.rs

52d53b5

Co-authored-by: Jim Hodapp <[email protected]>

Update service/src/config.rs

e8d1ebd

Co-authored-by: Jim Hodapp <[email protected]>

append v0 to tiptap document names

3e6cdb2

extract tiptap code into simple module

08c34b2

remove short form args for tiptap related config

9a3eddc

Clap was complaining and just having a -t seems not super expressive

truncate seconds from coaching_sessions dates

19ba3a8

add collab_document_name to coaching_sessions

a9b9331

set default log level to info

350b174

update dbml comments

ab5da53

fixes docker compose and build_and_run script.

022a039

update docker-compose to use environment variables for backend command

2461703

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

82 feature request suggestion create GitHub actions workflow for automating the build and deployments for feature branches #87

82 feature request suggestion create GitHub actions workflow for automating the build and deployments for feature branches #87

lmcdonough commented Dec 13, 2024

82 feature request suggestion create GitHub actions workflow for automating the build and deployments for feature branches #87

Are you sure you want to change the base?

82 feature request suggestion create GitHub actions workflow for automating the build and deployments for feature branches #87

Conversation

lmcdonough commented Dec 13, 2024

GitHub Issue: [Closes] #82

Changes

Testing Strategy

Concerns