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

Init migration Python 3.9 #3945

Merged
merged 32 commits into from
Feb 28, 2023
Merged

Init migration Python 3.9 #3945

merged 32 commits into from
Feb 28, 2023

Conversation

azime
Copy link
Contributor

@azime azime commented Feb 22, 2023

  • Debian11 (Python3.9)
Component Debian8(Python2.7) Debian11(Python3.9)
Jormungandr 481.5 MB 153.09 MB
Tyr-web 464.79 MBl 90.98 MB

@sonarqubecloud
Copy link

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

No Coverage information No Coverage information
0.0% 0.0% Duplication

# Redirect apache log output to docker log collector
RUN ln -sf /proc/self/fd/1 /var/log/apache2/access.log \
&& ln -sf /proc/self/fd/2 /var/log/apache2/error.log \
&& echo "net.ipv4.tcp_tw_reuse=1" >> /etc/sysctl.d/10-ipv4.conf
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a comment explaining why we set net.ipv4.tcp_tw_reuse=1 ?
Otherwise we will quickly forget why :)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tcp_tw_reuse allows the reuse of a client port immediately after the connection has been closed, even though the connection is still in TIME_WAIT state. It's important to test the configuration before to push it in production.

@xlqian xlqian changed the title Init migration Init migration Python 3.9 Feb 28, 2023
Copy link
Member

@xlqian xlqian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well done

greenlet==0.4.15 ; python_version <= "3.6"
pickle-mixin==1.0.2
Flask-Caching==1.7.2 ; python_version < "3.9"
Flask-Caching==1.8.0 ; python_version >= "3.9"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@azime azime merged commit eafd1b2 into dev Feb 28, 2023
@azime azime deleted the init_migration branch February 28, 2023 17:11
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.

4 participants