-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
Added cal.com as template #2119
base: next
Are you sure you want to change the base?
Conversation
Scheduling infrastructure for absolutely everyone. Docker-Compose instructions used here: https://github.com/calcom/docker
@kunumigab
Unfortunately, I can't test this right now since my main server is ARM. |
@andrasbacsai any chance you could review this PR this week? |
Love, the idea, could really use the template! I've tried your file, but encountered some hiccups. What I've doneI've used your The I've tried both to set a custom url for the 'Calcom' service (https://cal.<domain.com>) or to leave it empty (trying to reach it with IP address). The services are successfully spinning on start up, even if the Calcom service is flagged as 'unhealthy'. Here are the deployement logs:
My faced issuesCalcom service logsI have issues with the prismaClient, something to do with the base user table not being properly created?
Calcom service target URLI've tried both to leave empty the domain or to fill it with a subdomain, and both have the same result: No cert is created. Same as @RayBB, I'm no maintainers or nothing to the project. After some time on the Discord, (thanks a lot again @iamEvanYT for your time and explanations!), we couldn't find any workaround so I figure here maybe someone has one, or it can help improve the template. Hope this helps, |
Hey i have just solved this issue. I will also help you with the next issue you will be facing :) I really have no idea why this is the case. I found the scripts from https://github.com/calcom/docker/blob/main/scripts/start.sh that is run during build.
I have run this from command line that is present in coolify dashboard. It gave me the error that DATABASE_DIRECT_URL is not set. Eventhough read me from calcom docker repo says it is optional. I think in this case it is required. You need to add the following to the docker compose as well for the calcom.
Then after you create a admin account. You will not be able to get admin privileges, because you can't set a 2fa. This problem is because of these lines: If you inspect console logs it will say "ERR_CRYPTO_INVALID_KEYLEN". Because CALENDSO_ENCRYPTION_KEY is too long for this. I have used the following command that i found in original calcom/cal example .env file. When i set this new secret for CALENDSO_ENCRYPTION_KEY. Everything seems to be working fine. I am trying to set my instance up. If i face anything i will let you know. |
@SamAmann @kunumigab sorry forgot to mention you guys :) |
Hey!, Can you test this again?, I don't have a x86 server right now |
Nice job @dylancetin! I've tested it in 2 different condition:
TEST RESULTWith your fix to the EDIT! After few tries, the container is back to a 'unhealthy' status... Here are the logs of the container, it seems there is lots of progress, as it started to install apps and run the next.js script to start.
Still some troubles with env var like Unfortunately, I still can't manage to reach it, still having problem obtaining the cert for that docker when using a custom subdomain. On my extra server, can't even reach raw IP of server and gives me either 404 or too long request. But I think we're on the good track here :) |
https://github.com/calcom/docker does have NEXT_PUBLIC_API_V2_URL as an important env variable. I will be guessing that warning from log is unnecessary. But it is set on the .env.example. (I don't have this set, and my instance runs perfectly. My guess will be this is part of the enterprise API. If you need that you probably should set this.)
https://github.com/calcom/docker?tab=readme-ov-file#client_fetch_error [next-auth][error][CLIENT_FETCH_ERROR] I think email is kinda "optional", but on most instances is used. I have used resend because i find it easiest to set. We might want to add these to the template. @kunumigab what do you say? For mail i have the following variables.
@SamAmann please check the log with a bigger number of lines like "10000". If you have the CLIENT_FETCH_ERROR do the NEXTAUTH_URL patch. And on first 10 lines there needs to be successful database migrations. Then you might need to set DATABASE_DIRECT_URL. |
Added hostname and mail variables, again need testing Edit: Healthcheck was added too, so the unhealthy status could be fixed |
Scheduling infrastructure for absolutely everyone.
Docker-Compose instructions used here: https://github.com/calcom/docker