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

build(dev): docker compose using a prebuilt image with latest gems and node packages #2288

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

PiTrem
Copy link
Member

@PiTrem PiTrem commented Jan 16, 2025

  • set env variables in the docker-compose.dev.yml to allow using

    • prebuild images for the app and skip the building process
    • distinct postgres images
    • distinct named volume for the db and homedir(asdf, gems, etc)
  • add .dockerenv.example file to help settings those variables
    example

docker compose -f docker-compose.dev.yml --env-file .dockerenv.example up
  • rework the preparation scripts to better separate node packages
    installation from ruby env installation. so that yarn install
    is only run by the js container

  • introduce FULL_BUILD BRANCH args for preparing the dev docker
    image with preinstalled gems and nodejs packages for a given
    branch from complat/chemotion_ELN

    • add optional var RAKE_DB_MIGRATE to run db:migrate instead of db:setup
      (see .dockerenv.example)

    NB the path for the node modules is set in the image outside the app directory to NODE_PATH

@PiTrem PiTrem changed the base branch from main to shakapacker_config_upd January 16, 2025 11:24
@PiTrem PiTrem marked this pull request as draft January 16, 2025 11:24
Copy link

LCOV of commit 9d9e751 during Continuous Integration #4320

Summary coverage rate:
  lines......: 66.8% (14413 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@PiTrem PiTrem force-pushed the dev-docker-compose branch 2 times, most recently from 46cebcc to 8c6122f Compare January 21, 2025 13:40
@PiTrem PiTrem changed the base branch from shakapacker_config_upd to main January 21, 2025 13:41
Copy link

LCOV of commit 46cebcc during Continuous Integration #4333

Summary coverage rate:
  lines......: 66.8% (14411 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

Copy link

LCOV of commit 8c6122f during Continuous Integration #4334

Summary coverage rate:
  lines......: 66.8% (14411 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@PiTrem PiTrem force-pushed the dev-docker-compose branch from 8c6122f to d98b2f9 Compare January 21, 2025 14:23
Copy link

LCOV of commit d98b2f9 during Continuous Integration #4335

Summary coverage rate:
  lines......: 66.9% (14427 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@PiTrem PiTrem force-pushed the dev-docker-compose branch 2 times, most recently from 478d0d9 to 4899f89 Compare January 21, 2025 15:18
Copy link

LCOV of commit 478d0d9 during Continuous Integration #4336

Summary coverage rate:
  lines......: 66.8% (14411 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

Copy link

LCOV of commit 4899f89 during Continuous Integration #4337

Summary coverage rate:
  lines......: 66.8% (14411 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@PiTrem PiTrem force-pushed the dev-docker-compose branch from 4899f89 to 4a9a7d9 Compare January 21, 2025 16:36
Copy link

LCOV of commit 4a9a7d9 during Continuous Integration #4338

Summary coverage rate:
  lines......: 66.8% (14413 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@PiTrem PiTrem force-pushed the dev-docker-compose branch 4 times, most recently from fc27fef to f58f0bc Compare January 22, 2025 13:20
Copy link

LCOV of commit bb7cbef during Continuous Integration #4342

Summary coverage rate:
  lines......: 66.8% (14411 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

Copy link

LCOV of commit 40549a2 during Continuous Integration #4343

Summary coverage rate:
  lines......: 66.9% (14427 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

Copy link

LCOV of commit fc27fef during Continuous Integration #4344

Summary coverage rate:
  lines......: 66.8% (14411 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

Copy link

LCOV of commit f58f0bc during Continuous Integration #4345

Summary coverage rate:
  lines......: 66.8% (14413 of 21564 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

- set env variables in the docker-compose.dev.yml to allow using
  - prebuild images for the app and skip the building process
  - distinct postgres images
  - distinct named volume for the db and homedir(asdf, gems, etc)

- add .dockerenv.example file to help settings those variables
example
```
docker compose -f docker-compose.dev.yml --env-file .dockerenv.example up
```

- rework the preparation scripts to better separate node packages
  installation from ruby env installation. so that `yarn install`
  is only run by the js container

- introduce FULL_BUILD BRANCH args for preparing the dev docker
  image with preinstalled gems and nodejs packages for a given
  branch from complat/chemotion_ELN

- add optional var RAKE_DB_MIGRATE to run db:migrate instead of db:setup
   (see .dockerenv.example)

- vs task to start docker compose with --env-file .dockerenv and without
  build
as prepare-ruby--dev is already called thr run-ruby
@PiTrem PiTrem force-pushed the dev-docker-compose branch from f58f0bc to b6c4d24 Compare January 23, 2025 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant