Skip to content

Add a development setup#29

Open
tomasr8 wants to merge 2 commits intoindico:masterfrom
tomasr8:dev-container
Open

Add a development setup#29
tomasr8 wants to merge 2 commits intoindico:masterfrom
tomasr8:dev-container

Conversation

@tomasr8
Copy link
Member

@tomasr8 tomasr8 commented Mar 1, 2023

Depends on #28

Adds a new containerized development setup which is similar to the production one.

Extra features:

  • Pulls from master
  • Includes maildump
  • Creates a default admin user to skip the manual setup
  • Adds an announcement saying not to use this setup in production
  • Pulls translations from Transifex (either using a public mirror or directly when given an API token)
  • Alpine base image to cut down the size (still more than 1GB)

@tomasr8 tomasr8 force-pushed the dev-container branch 8 times, most recently from 8be0ad1 to f9d432b Compare March 7, 2023 14:32
@tomasr8 tomasr8 force-pushed the dev-container branch 5 times, most recently from e7b7ce1 to f9ddb59 Compare March 10, 2023 15:02
@ThiefMaster
Copy link
Member

  • We probably shouldn't use uwsgi for this but the dev server (to get the reloader)
  • "Indico will be accessible at localhost:8080. You can also access the wsgi app directly at localhost:9090 which skips the nginx proxy" is something best to get rid of since w/ base url enforcement it won't be possible to access indico via different URLs at the same time, and it's generally a bad idea anyway

/root/.local/bin/maildump -n --http-ip 0.0.0.0 --http-port 60000 --db /tmp/maildump.sqlite --smtp-ip 127.0.0.1 --smtp-port 25 &

echo 'Starting Indico...'
uwsgi /etc/uwsgi.ini
Copy link

Choose a reason for hiding this comment

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

Suggested change
uwsgi /etc/uwsgi.ini
indico run -h 0.0.0.0 -p 59999 -u http://localhost:8080 --reloader watchfiles

Co-authored-by: Alex Iribarren <alex.iribarren@gmail.com>
@tomasr8
Copy link
Member Author

tomasr8 commented Nov 15, 2024

Thanks for the comments @alexiri! I'll push some updates.
(I also rebased the PR on the latest master)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants