canonical/cos-lib

allow using cosl.coordinated_workers.S3ConnectionInfo outside of tempo-loki-mimir

Offen

#119 geöffnet am 27.01.2025

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (6 Forks)auto 404
Status: TriageType: Buggood first issue

Repository-Metriken

Stars
 (0 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 5h) (1 gemergte PR in 30 T)

Beschreibung

Bug Description

Right now attempting to import that module from outside the HA charms (e.g. Parca) will result in import errors because of the check-charm-libs-installed thing we do on import.

we should either refactor cosl to run that check on-demand rather than on import, or lift anything that needs not be under that check outside of the coordinated_workers module so other charms can use them without ugly workarounds.

To Reproduce

n/a

Environment

n/a

Relevant log output

n/a

Additional context

No response

Contributor Guide

Research-Richtung
Finde die Import Prüfung in cosl.coordinated workers, die den Importfehler verursacht. Refaktoriere sie, um sie bei Bedarf auszuführen, oder verschiebe S3ConnectionInfo und verwandte Dienstprogramme in ein separates Modul außerhalb von coordinated workers, um Importe ohne die Prüfung zu ermöglichen.
Tech Stack
python
Domain
backendinfrastructure
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
PythonGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70