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

Update image to use uWSGI #301

Merged
merged 4 commits into from
Aug 12, 2022
Merged

Update image to use uWSGI #301

merged 4 commits into from
Aug 12, 2022

Conversation

santi-eidu
Copy link
Contributor

Requirements

  • Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
  • All new code requires tests to ensure against regressions
  • Remember to set idealista:develop as base branch;

Description of the Change

Docker image will use the same deploy method than prom2teams_role.

  • Use of bin/wsgi.py to run the app, launched with uWSGI module.
  • Use of prom2teams user instead of root.
  • Use processes and threads variable with uWSGI.
  • Added variable loglevel to config file.
  • Update fix helm deployment configuration and default values.

Benefits

Docker image high load ready for production environments.

Possible Drawbacks

This new image needs new environment variables and may broke a deployment with the old image.

Applicable Issues

#300

santi-eidu and others added 2 commits June 27, 2022 18:05
Merge develop with master to add deploy in Dockerhub with travis
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 6, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@santi-eidu santi-eidu merged commit f2d1664 into develop Aug 12, 2022
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.

3 participants