-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
NoMongo: Production Docker Installation Fails #3243
Comments
Please assign |
I am able to reproduce this error |
https://palisadoes-foundation.slack.com/archives/CSWH4PN0M/p1738240831597849 Sir this error is not due to recent PR's, I faced this when I just started using talawa API. To fix this run start server in corepack enabled environment, I remember maybe this fixed this, do you want me to debug this ? |
As you can see i am already working on this issue. |
I hve already assigned this issue. Kindly ensure that the terms and conditions of Palisadoes are upheld. I would truly appreciate your assistance in this matter. |
@VanshikaSabharwal Yes please continue your work, I am referring to my latest PR |
Can you please share you PR link? |
I believe that has nothing to do with this, also it has been reverted for now, refer to this message please https://palisadoes-foundation.slack.com/archives/CSWH4PN0M/p1738240831597849 |
@JaiPannu-IITI there is no problem even if the error has changed. I'll fix it. @palisadoes was just mentioning the error. |
If you want to debug you can come and assist me. You can contact me on slack. |
Also about the error occuring i saw one issue is consistent from beginning the MINIO setup one. And other issues are not consistent. Docker is vast and there can be issues even if the version is different. |
@VanshikaSabharwal It's really fine carry on with the task, don't panic you have all the time you want to fix this it's assigned to you. I didn't see this in the beginning I didn't wanted to be rude was just trying to make clear that adding pnpm to all shells has nothing to do with this |
Yes @JaiPannu-IITI Thanks |
For some reason, pnpm isn't being installed in the container. Look at the startup console messages to see where the Another option would be to see how you can log messages related to container initialization behind the scenes within docker versus externally via stdio/stderr to the screen. |
@palisadoes pnpm is a package manager, a package manager is only required in non production environments, therefore it is not included in the production docker image of talawa api |
What could be causing this error? The most recent error messages are related to pnpm. |
@palisadoes How this makes sense? ![]() |
@palisadoes use this It will show some progress |
@palisadoes Node is not getting installed in production environment, fnm is trying to use default |
This should fix |
Yes @JaiPannu-IITI i am updating the dockerfile and trying to remove this error. I am installing pnpm but it shows the same error |
@JaiPannu-IITI it should but it is not getting started that is the issue we have to fix. Why is it not getting started |
I tried your approach but it is not working |
Have you tries it locally for yourself? If yes please share ss |
Are you able to visit |
Clear docker volumes and rebuild before using docker-compose -f compose.yaml up
New issue will be node version by fnm. That even makes sense, pnpm issue is just misleading |
Can we connect on slack? |
Sure |
Great Yes i am texting you |
Fnm is not used in anywhere except the All other docker build stages are based off of
Each step in a docker image's build cycle contributes to its final size. All this is done so that the size of the production image is as minimal as possible. By copying the relevant stuff from other build stages into the |
You guys are probably having a problem with your It all works for me when I follow the video that is uploaded on youtube for the latest commit on |
Okay what is the solution to this. @xoldd |
Can you please share link of that video? @xoldd |
It is on talawa docs. |
Is it this one. https://www.youtube.com/watch?v=10Zi2srGPHM |
The video shows this:
Why use |
I did what you suggested @JaiPannu-IITI but i still see the same error. |
now api is not starting, before it was failing, some difference. |
Sir I am sure that pnpm error will come when docker is setup using compose.testing.yaml but in production environment since it does not have pnpm, Ideally we should use compose.yaml to setup api in production, I totally agree with @xoldd that production env clearly mentions COMPOSE_FILE=./compose.yaml in 71st line, but still somehow compose.testing.yaml is being used. |
Try to force compose.yaml and you'll se that pnpm disappears completely from logs and environment as expected. But then when we get error that node is not available After fixing that too, comes worse, api exits with code 0 directly, no logs !! |
I have not tested but I can predict that the command like pnpm start_development_server which is used to run api in container in development, this has something equivalent (not this) for production which might not be executed after container is run. Also I haven't gone through that video, I just recreated the issue as above and tried to fix that, maybe some pipeline is already set to run api in production which is not updated In docs. |
@JaiPannu-IITI either delete your existing talawa api image or rebuild it using |
Yes @JaiPannu-IITI i told you the same on slack.
|
@xoldd Id possible can you make another video and let us know if the app is working because i tried this but still i see the error
|
Already done. |
it works for peter now, he can explain the rest of the stuff also |
Okkay @xoldd Thanks for your guidance |
Alright @xoldd , glad to hear |
@xoldd assisted in getting this fixed. There were mistakes made and mutual assumptions made that contributed the the issue. The various containers are designed to operate separately so that their services can be hosted on any combination of suitable hosted remote servers or SaaS providers. This means that some passwords will need to be manually set to match, some are more obvious than others.
You have to use the The documentation has been updated This file works and will replace
|
Describe the bug
To Reproduce
Expected behavior
Actual behavior
Screenshots
Additional details
Potential internship candidates
Please read this if you are planning to apply for a Palisadoes Foundation internship
The text was updated successfully, but these errors were encountered: