canonical/cos-lib

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

Ouverte

#119 ouverte le 27 janv. 2025

Voir sur GitHub
 (7 commentaires) (0 réaction) (0 personne assignée)Python (6 forks)auto 404
Status: TriageType: Buggood first issue

Métriques du dépôt

Stars
 (0 étoile)
Métriques de merge PR
 (Merge moyen 1j 5h) (1 PR mergée en 30 j)

Description

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

Guide contributeur

Direction de recherche
Localisez la vérification lors de l'importation dans cosl.coordinated workers qui cause l'erreur d'importation. Refactorez la pour qu'elle s'exécute à la demande, ou déplacez S3ConnectionInfo et les utilitaires associés dans un module séparé en dehors de coordinated workers pour permettre les importations sans la vérification.
Stack technique
python
Domaine
backendinfrastructure
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70