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

allow disabling elementary version check #1674

Merged
merged 5 commits into from
Sep 4, 2024

Conversation

slaupster
Copy link
Contributor

checking the version is nice and all, but when running in production it just creates firewall log spam - we don't let traffic out anywhere, everything is explicitly allowed in kubernetes overlay network policy as well as an overarching firewall policy. Everytime elementary runs we see the spam trying to contact pypi to check for a newer version.

This PR creates a simple config to allow disabling the check.

checking the version is nice and all, but when running in production it just creates firewall log spam - we don't let traffic out anywhere, everything is explicitly allowed in kubernetes overlay network policy as well as an overarching firewall policy. Everytime elementary runs we see the spam trying to contact pypi to check for a newer version.

This PR creates a simple config to allow disabling the check.
@slaupster slaupster changed the title disable elementary version check allow disabling elementary version check Aug 19, 2024
@elongl
Copy link
Member

elongl commented Aug 28, 2024

Thanks a lot for this contribution, makes a lot of sense!
Left one small comment, will merge afterwards.

elementary/cli/upgrade.py Outdated Show resolved Hide resolved
@slaupster slaupster requested a review from elongl September 2, 2024 09:15
Copy link
Collaborator

@haritamar haritamar left a comment

Choose a reason for hiding this comment

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

looks good!
merging

@haritamar haritamar merged commit 70e4a00 into elementary-data:master Sep 4, 2024
2 of 3 checks passed
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