Skip to content
This repository was archived by the owner on Oct 29, 2025. It is now read-only.

fix: Docker builds#359

Merged
keithwillcode merged 19 commits intomainfrom
fix/separate-run-commands
May 9, 2024
Merged

fix: Docker builds#359
keithwillcode merged 19 commits intomainfrom
fix/separate-run-commands

Conversation

@keithwillcode
Copy link
Copy Markdown
Contributor

@keithwillcode keithwillcode commented May 9, 2024

This fixes our Docker builds by

  • Including the files needed from API to properly build web
  • Including an action to free disk space - ideally we wouldn't need this but for the moment is required.

@keithwillcode keithwillcode self-assigned this May 9, 2024
CALENDSO_ENCRYPTION_KEY: '0zfLtY99wjeLnsM7qsa8xsT+Q0oSgnOL'


# - name: Push image
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alishaz-polymath Should we not actually use this instead so we don't need to rebuild the image from scratch again like it's doing now?

Copy link
Copy Markdown
Collaborator

@emrysal emrysal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@keithwillcode keithwillcode merged commit 7924c20 into main May 9, 2024
@keithwillcode keithwillcode deleted the fix/separate-run-commands branch May 9, 2024 16:13
@ivancarlosti
Copy link
Copy Markdown

I do not get it, docker build keeps failing...

@ivancarlosti
Copy link
Copy Markdown

docker build keeps failing... https://github.com/calcom/docker/actions/runs/14186960112

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants