-
Notifications
You must be signed in to change notification settings - Fork 10
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
Making changes to the metadata service to use the new base image #99
Conversation
The style in this PR agrees with This formatting comment was generated automatically by a script in uc-cdis/wool. |
…or metadata to work as expected
Please find the ci env pod logs here |
Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the ci env pod logs here |
Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
pyproject.toml
Outdated
@@ -9,7 +9,7 @@ license = "Apache-2.0" | |||
python = "^3.9" | |||
fastapi = "^0.65.2" | |||
uvicorn = {extras = ["standard"], version="^0.18.2"} | |||
gunicorn = ">=20.0.4" | |||
gunicorn = "^22.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like to keep all our deps >= until we hit a version that doesn't work, then we create a ticket, pin, and try to fix soon
We're not consistent with this, but in newer services, we've tried to keep every >=
Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
Please find the detailed integration test report here Please find the ci env pod logs here |
…cdis#99) Improvements Update to use new Amazon Linux base image and use the same structure as our other python services. Utilizing "gen3" user instead of "root" for more secure containers Moving to Poetry to manage our virtual environments Multi-stage Docker builds for smaller images Move to Gunicorn
Improvements