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

fix(docker): compose dev and prod #9982

Merged
merged 2 commits into from
Feb 8, 2024
Merged

fix(docker): compose dev and prod #9982

merged 2 commits into from
Feb 8, 2024

Conversation

Josh-Walker-GM
Copy link
Collaborator

Problem
By default when once you have run the experimental setup command for docker, the compose setup for dev and prod doesn't quite work.

Changes

  1. We add missing fs-extra dependency for the @redwoodjs/web-server, it is a runtime dependency.
  2. We add a REDWOOD_API_HOST=0.0.0.0 env var for the dev setup. This is required for the vite dev server to proxy requests correctly.

Notes
@jtoar I'm not overly sure how to test the compose with the updated dependency of the web-server package. Adding the dependency to the web side itself did work so I would imagine this works fine too.

@Josh-Walker-GM Josh-Walker-GM added the release:fix This PR is a fix label Feb 8, 2024
@Josh-Walker-GM Josh-Walker-GM added this to the next-release-patch milestone Feb 8, 2024
@Josh-Walker-GM Josh-Walker-GM self-assigned this Feb 8, 2024
@jtoar jtoar enabled auto-merge (squash) February 8, 2024 18:56
@jtoar jtoar disabled auto-merge February 8, 2024 20:52
@jtoar jtoar merged commit 60d031d into main Feb 8, 2024
40 checks passed
@jtoar jtoar deleted the jgmw-fix/docker-compose branch February 8, 2024 20:52
jtoar pushed a commit that referenced this pull request Feb 9, 2024
**Problem**
By default when once you have run the experimental setup command for
docker, the compose setup for dev and prod doesn't quite work.

**Changes**
1. We add missing `fs-extra` dependency for the `@redwoodjs/web-server`,
it is a runtime dependency.
2. We add a `REDWOOD_API_HOST=0.0.0.0` env var for the dev setup. This
is required for the vite dev server to proxy requests correctly.

**Notes**
@jtoar I'm not overly sure how to test the compose with the updated
dependency of the web-server package. Adding the dependency to the web
side itself did work so I would imagine this works fine too.
dac09 added a commit to dac09/redwood that referenced this pull request Feb 9, 2024
…d-cookies-dbauth

* 'main' of github.com:redwoodjs/redwood:
  chore: update rsc fixture (redwoodjs#9986)
  fix(server): use file extension in import, fix graphql route registering (redwoodjs#9984)
  chore(deps): update babel monorepo (redwoodjs#9983)
  fix: unpin react types (redwoodjs#9727)
  fix(docker): compose dev and prod (redwoodjs#9982)
  fix(deps): update prisma monorepo to v5.9.1 (redwoodjs#9980)
  fix(cli): use fetch instead of `yarn npm info` (redwoodjs#9975)
  fix(test): Update createServer test to use a different port to normal (redwoodjs#9977)
  fix(docker): corepack permissions fix and style updates (redwoodjs#9976)
@jtoar jtoar modified the milestones: next-release-patch, v7.0.0 Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:fix This PR is a fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants