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.

Sign up for GitHub

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

Jump to bottom

Move to using docker for dependencies and deployments #537

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

Move to using docker for dependencies and deployments #537

wants to merge 60 commits into from

Conversation

nicwest
Copy link

@nicwest nicwest commented Jun 17, 2024

No description provided.

glbert-does and others added 25 commits November 17, 2023 14:35
@glbert-does
fix deprecation warnings
ab5328e
@glbert-does
fix deprecated rexgexp
86345bd
@glbert-does
fix one test, temporarily turn failing test off
57d4fd1
@glbert-does
providing_args are documentational and are deprecated
a852a92
@glbert-does
add migration that was not done for 3rd alt option
70d208e
@glbert-does
remove install_cacheops()
548ba3c
@glbert-does
default_app_config in __init__.py is deprecated
d9cd10c
@glbert-does
migrate deprecated fields
361460a
@glbert-does
conf.urls.url() is deprecated in favour of urls.re_path()
4f8aa65
@glbert-does
change some urls to paths
5c5e931
@glbert-does
change re_path to more readable path
0b49e75
@glbert-does
ifequals is deprecated
13f282c
@glbert-does
update recaptcha to version 4
2b14357
@glbert-does
update removed django.utils.http functions
a02c11a
@glbert-does
re-enable cacheops, move to django 4.2
9ce14d4
@glbert-does
the app is called tournament, not heltour
f006834
@glbert-does
disable cacheops for testing
89598e3
@glbert-does
add ad-hoc replacement for deactivated test
5787737
@glbert-does
enable cacheops in default settings
f3a368d
@glbert-does
fix django 5 deprecation warnings
5a5b9b0
@glbert-does
fix broken url for webcal
ae22207
@glbert-does
upgrade to python 3.9
70a1edc
@nicwest
update gitignore
baafb7a 
We are now going to ignore all files in local settings this stuff
shouldn't be being commited in my opinion
@nicwest
update some out of date dependencies
b7a2acc
@nicwest
simple docker file
5246a61
nicwest added 4 commits June 24, 2024 01:04
@nicwest
remove loading of local settings based on hostname
0cd6cdd 
piping a hostname into exec doesn't seem like a great idea. Hopefully
with a nice docker based development enviroment we won't need this kind
of config any more.
@nicwest
add static files as it's own image
76cc77e 
this will work as a stop gap until static files can move to a cdn
@nicwest
fix static docker things
6474bb6
@nicwest
add simple docker compose
0d2e829
@nicwest
Copy link
Author

nicwest commented Dec 10, 2024 •
edited
Loading

summary of stuff that changes.

  • settings rationalised into base, development and production
  • settings mostly will have env vars that can be used to overwrite config.
  • docker files
    • heltour:latest: contains the python env (the webapp, celery, and eventually the api worker)
    • heltour-static: a static files server. at somepoint we should talk about moving this stuff to a CDN.
  • docker compose to replace vagrant. I've been using this locally and with the new docker compose watch feature, this feels viable.
  • includes glbert's update django to 4.2 #527

still todo:

  • api worker
  • secrets
  • figuring out what to do with database stuff. I'm playing around with django fixtures and feels viable.

@nicwest nicwest marked this pull request as ready for review December 15, 2024 19:03
@nicwest nicwest mentioned this pull request Dec 15, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nicwest @glbert-does