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

autoprune: add initial setup for the autoprune feature (PROJQUAY-6094) #2277

Merged
merged 3 commits into from
Oct 5, 2023

Conversation

bcaton85
Copy link
Contributor

@bcaton85 bcaton85 commented Oct 2, 2023

Adds the auto-prune worker, database models, and feature flag.

@bcaton85 bcaton85 requested review from syed, dmage and kleesc October 2, 2023 20:10
@codecov
Copy link

codecov bot commented Oct 2, 2023

Codecov Report

Attention: 29 lines in your changes are missing coverage. Please review.

Files Coverage Δ
conf/init/supervisord_conf_create.py 44.44% <ø> (ø)
config.py 93.45% <100.00%> (+0.02%) ⬆️
data/database.py 90.61% <100.00%> (+0.07%) ⬆️
initdb.py 93.83% <ø> (+0.02%) ⬆️
util/config/schema.py 100.00% <ø> (ø)
workers/autopruneworker.py 0.00% <0.00%> (ø)

... and 7 files with indirect coverage changes

📢 Thoughts on this report? Let us know!.

from app import app
from workers.gunicorn_worker import GunicornWorker
from workers.worker import Worker
from util.log import logfile_path
Copy link
Member

Choose a reason for hiding this comment

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

nit: this file isn't isort'ed

Copy link
Contributor

@Sunandadadi Sunandadadi left a comment

Choose a reason for hiding this comment

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

lgtm

@bcaton85 bcaton85 merged commit 0e496b4 into quay:master Oct 5, 2023
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants