apihub icon indicating copy to clipboard operation
apihub copied to clipboard

feat(containerization): enhance docker compose config

Open yntpdotme opened this issue 9 months ago • 3 comments

Description

This PR addresses issue #153, aiming to enhance developer experience with Docker Compose improvements.

Changes Made:

  • Implement Docker Compose watch feature for automatic sync and rebuild of backend service.
  • Optimize MongoDB service to reduce log clutter.

Testing:

  • Manually tested synching and rebuilding actions.
  • Checked for any unintended side effects or regressions.

Additional Notes:

Related Issue:

  • Issue #153

Screenshots:

After running docker compose up --build:

  • Press w to Enable Watch

    Screenshot 2024-05-02 at 11 27 50

  • Syncing backend service after changes in code

    Screenshot 2024-05-02 at 12 28 47

  • Rebuilding backend service after changes in dependencies

    Screenshot 2024-05-02 at 11 31 16

yntpdotme avatar May 05 '24 10:05 yntpdotme

@wajeshubham It's woking on my side.

  • Here is my docker version

    image

Irony is that we are using Docker and still facing "It's Working on my Machine" 🥶

  • Here are logs

    image

Please share your docker version and logs for that I can look into issue.

yntpdotme avatar May 14 '24 16:05 yntpdotme

@wajeshubham --watch is available after Docker compose 2.25.0 release

Ref: https://docs.docker.com/compose/release-notes/#2250

yntpdotme avatar May 14 '24 17:05 yntpdotme

We want this project to be backward compatible and must work on majority of versions of the tools that we are using. I guess --watch flag is not needed and we can just use --attach to ignore db logs. For now, I'm closing this PR but we will consider this PR in future. Thank you

wajeshubham avatar May 17 '24 04:05 wajeshubham

@wajeshubham

I appreciate your consideration of my PR, and I’m happy to revisit these changes in the future when it aligns better with our project goals.

Thank you for reviewing and providing your insights.

yntpdotme avatar May 17 '24 06:05 yntpdotme