mu-semtech / mu-python-template

Template for running Python/Flask microservices
MIT License
4 stars 8 forks source link

Migrate template to Python3 #2

Closed MikiDi closed 3 years ago

MikiDi commented 3 years ago

This PR aims to migrate the Python template to Python3 + adds various major improvements. Currently marked as draft, allowing for discussion prior to formal review.

Bugfixes

features

Discussion

MikiDi commented 3 years ago

Preliminary conclusions of above mentioned discussion-points:

  • Is a production WSGI-server actually needed? ("We count on other layers of the stack for security. There is little need for concurrent request handling.")

Yes, a production server is needed. By default however, we configure the amount of workers by 1, which makes the default setup behave in a similar way as the (current) dev server does. The template consumer can configure the amount of spawned workers to their own liking.

  • Linting: This PR includes a .pylintrc configured for developing microservices based on this template. See README for rationale & See discussion here

Providing linting config for a local setup encourages a way of development that goes against the mu.semte.ch-idea that development should happen in a dockerized environment. Despite the fact that the provided linting configuration lowers the bar for novice (mu.semte.ch-) developers, we will not include the linting config in this PR, but rather discuss it separately.