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.

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

Automatically refresh upstream and create Docker image #541

Open
wants to merge 20 commits into
base: dev
Choose a base branch
from

Conversation

DavidHenryThoreau
Copy link

No description provided.

@DavidHenryThoreau
Copy link
Author

@SamR1 I'm working on docker images i'm looking at https://github.com/SamR1/FitTrackee/blob/master/db/create.sql , why do you drop database, if container restart it'll drop the DB.

I've also created a repository in order to build fittrackee-db https://github.com/DavidHenryThoreau/fittrackee-db add all steps in the Dockercompose.yaml f49fc18

@DavidHenryThoreau
Copy link
Author

TODO : Run ftcli db upgrade when fittrackee docker container starts in order to populate the databse

@DavidHenryThoreau
Copy link
Author

Images are built form source, need some testers :
docker compose pull && docker compose up -d

Have fun !

@DavidHenryThoreau
Copy link
Author

DavidHenryThoreau commented May 15, 2024

TODO :

docker python:3:10 image size : 1.46GB
docker python:3.10-slim image size : 902MB
docker python:3.10-alpineimage size : 737MB

  • Do not build PostgreSQL image (not necessary at the moment)

  • Add Redis alternative

@SamR1
Copy link
Owner

SamR1 commented May 15, 2024

Hi,

I'm working on docker images i'm looking at https://github.com/SamR1/FitTrackee/blob/master/db/create.sql , why do you drop database, if container restart it'll drop the DB.

The SQL script allows to recreate databases easily for tests during developpement.

About its execution (in docker-entrypoint-initdb.d):

Warning: scripts in /docker-entrypoint-initdb.d are only run if you start the container with a data directory that is empty; any pre-existing database will be left untouched on container startup.

source: https://hub.docker.com/_/postgres/

As I said in the other MR, docker files present in this repository are intended for evaluation or development only. So the configuration is not suitable for production. Other changes or steps may be necessary to secure deployment in production.

@DavidHenryThoreau
Copy link
Author

DavidHenryThoreau commented May 17, 2024

I'm trying to import gpx file I'm getting this error Erreur lors du traitement du fichier gpx.

2f332d7b38db:/usr/src/app# tail -f fittrackee.log
2024/05/17 15:47:17 - fittrackee - ERROR - could not download 4 tiles: [(525, 358, ''), (525, 359, ''), (526, 358, ''), (526, 359, '')]
2024/05/17 15:47:17 - werkzeug - INFO - 192.168.1.2 - - [17/May/2024 15:47:17] "POST /api/workouts HTTP/1.1" 500 -

Fixed by removing empty variables :

      - TILE_SERVER_URL=
      - STATICMAP_SUBDOMAINS=
      - MAP_ATTRIBUTION=
      - DEFAULT_STATICMAP=False
      - WEATHER_API_PROVIDER=
      - WEATHER_API_KEY=

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.

None yet

2 participants