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

[Python3 and Django2.2 upgrade] part 7 - Updated docker image with non-root user #633

Closed

Conversation

noliveleger
Copy link
Contributor

@noliveleger noliveleger commented Jul 23, 2020

Tests pass. Image can be build and container should start
Related to #602

@noliveleger noliveleger requested a review from jnm July 23, 2020 20:36
@noliveleger noliveleger marked this pull request as draft July 23, 2020 20:39
@noliveleger noliveleger changed the title Updated docker image with non-root user [Python3 and Django2.2 upgrade] part 6 - Updated docker image with non-root user Jul 23, 2020
@noliveleger noliveleger changed the title [Python3 and Django2.2 upgrade] part 6 - Updated docker image with non-root user [Python3 and Django2.2 upgrade] part 7 - Updated docker image with non-root user Jul 28, 2020
@noliveleger noliveleger changed the title [Python3 and Django2.2 upgrade] part 7 - Updated docker image with non-root user [Python3 and Django2.2 upgrade] part 8 - Updated docker image with non-root user Aug 11, 2020
@noliveleger noliveleger marked this pull request as ready for review August 19, 2020 21:24
- Remove non-root user's password
- Use uWSGI and Celery `--uid` and `--gid` option to start processes
- Use gosu for other processes that don't have such option
@noliveleger noliveleger changed the title [Python3 and Django2.2 upgrade] part 8 - Updated docker image with non-root user [Python3 and Django2.2 upgrade] part 7 - Updated docker image with non-root user Dec 18, 2020
@noliveleger
Copy link
Contributor Author

Closed in favor of #681

@noliveleger noliveleger closed this Mar 4, 2021
@noliveleger noliveleger deleted the kobokitten-py3-new-docker-image branch March 11, 2021 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants