-
Notifications
You must be signed in to change notification settings - Fork 101
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
ballistica in docker :o #698
Conversation
so I have to write my fix into changelog with PR? |
yeah u should |
@efroemling https://github.com/efroemling/ballistica/actions/runs/9140752345/job/25134432953?pr=698#step:5:23 |
The src/assets dir is a bit of a complicated beast as far as spinoff rules and things like that. Can you stick the Dockerfile in |
Okay sure just gimme few minutes I'll push |
Also @vishal332008 requests you to come on discord |
If it's still giving you errors after that, don't worry about it; I can fix the spinoff stuff on my end after I pull the PR if that's the case. |
Also just as an FYI: the error I was seeing is that spinoff didn't know whether or not to filter the Dockerfile. Check out config/spinoffconfig.py line 142 for where it determines that. If you relocate Dockerfile to under config/docker/ then it will be covered by one of those rules (we do want it to be filtered I think). |
filtered means not included in spinoff project? |
Thanks for this! |
No; filtered just means that all instances of 'BallisticaKit' in the file are replaced with 'BombSquad' (or whatever else the project name is), and optionally other filtering is done too. Generally we want this for most things. |
Steps
make preflight
completes successfully.Description
Added build system for docker which builds docker image with bombsquad server which could be deployed easily with docker on any system
Added
docker-build
anddocker-run
targets to make fileType of Changes