Skip to content

Jf/validate minio #13

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.

Sign up for GitHub

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

Merged
BetoFandino merged 13 commits into from
Jan 27, 2026
Merged

Jf/validate minio #13

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
c7ed2de
feat: A new variable has been added to validate whether minio is used…
BetoFandino Jan 15, 2026
c99dce2
feat: validated with snipper value
BetoFandino Jan 19, 2026
aa533d0
fix: decorator of plugin loaded
BetoFandino Jan 19, 2026
2988e77
fix: deleted variable not used minio_enabled
BetoFandino Jan 20, 2026
07ae962
fix: deleted return config in plugin
BetoFandino Jan 20, 2026
2094716
fix: validation of ilm on rules
BetoFandino Jan 21, 2026
c5d6b35
feat: taked action and filter to validate list
BetoFandino Jan 22, 2026
77e23a3
feat: taked filter to validate list
BetoFandino Jan 22, 2026
aba6187
fix: return list of plugins
BetoFandino Jan 22, 2026
84b7b84
feat: added comments in minio init
BetoFandino Jan 23, 2026
14c1425
feat: use actions and take plugins list by filter
BetoFandino Jan 23, 2026
cdacb84
fix: comments in init minio
BetoFandino Jan 23, 2026
1403beb
fix: format of comments
BetoFandino Jan 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 9 additions & 7 deletions drydock_backups/plugin.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -107,10 +107,12 @@
) as fi:
tutor_hooks.Filters.CLI_DO_INIT_TASKS.add_item(("mysql", fi.read()), priority=tutor_hooks.priorities.HIGH)

@tutor_hooks.Actions.PLUGIN_LOADED.add()
def _add_minio_init(_name: str) -> None:
with open(
str(importlib_resources.files("drydock_backups") / "templates" / "drydock_backups" / "task" / "minio" / "init"),
encoding="utf-8",
) as fi:
tutor_hooks.Filters.CLI_DO_INIT_TASKS.add_item(("minio", fi.read()), priority=tutor_hooks.priorities.HIGH)
@tutor_hooks.Actions.PLUGINS_LOADED.add()
def _add_minio_init() -> None:
_plugins_list = tutor_hooks.Filters.PLUGINS_LOADED.apply([])
if "minio" in _plugins_list:
with open(
str(importlib_resources.files("drydock_backups") / "templates" / "drydock_backups" / "task" / "minio" / "init"),
encoding="utf-8",
) as fi:
tutor_hooks.Filters.CLI_DO_INIT_TASKS.add_item(("minio", fi.read()), priority=tutor_hooks.priorities.HIGH)
4 changes: 4 additions & 0 deletions drydock_backups/templates/drydock_backups/task/minio/init
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
mc config host add minio http://minio:9000 {{ OPENEDX_AWS_ACCESS_KEY }} {{ OPENEDX_AWS_SECRET_ACCESS_KEY }} --api s3v4
mc mb --ignore-existing minio/{{ BACKUP_BUCKET_NAME }}
{%- if BACKUP_MINIO_EXPIRATION_DAYS > 0 %}
# Validated if already exists expiration rule in backup bucket
if ! mc ilm ls minio/{{ BACKUP_BUCKET_NAME }} | grep -q "Expiration"; then
# Add expiration rule if it doesn't exist
mc ilm add --expiry-days "{{ BACKUP_MINIO_EXPIRATION_DAYS }}" minio/{{ BACKUP_BUCKET_NAME }}
fi
{%- endif %}