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

Packaging : rend l'installation indépendante de librados #50

Merged
merged 6 commits into from
Aug 31, 2023

Conversation

Guts
Copy link
Contributor

@Guts Guts commented Jul 18, 2023

  • ne plante pas si le projet est installé alors que librados n'est pas disponible
  • gérer le cas au niveau du setup - a priori pas besoin
  • modulariser les stockages pour en implémenter d'autres de façon cohérente - remis à plus tard

Désolé @Dolite, j'ai pas le temps de pousser le reste. Faut que je remette mon dpéôt local au propre après les autres PRs.

@Guts Guts changed the base branch from master to develop August 29, 2023 14:50
@Guts Guts force-pushed the packaging/make-it-independent-from-ceph branch from 8a3d0a1 to ab170b6 Compare August 29, 2023 15:08
@github-actions github-actions bot added the enhancement New feature or request label Aug 29, 2023
@Guts Guts marked this pull request as ready for review August 29, 2023 15:12
@Guts
Copy link
Contributor Author

Guts commented Aug 29, 2023

@Dolite tu peux tester en local sur une VM sans RADOS, ça fonctionne chez moi.

@sylvainbeo ou @vicente23 si vous pouvez tester la branche 🙏

@Dolite
Copy link
Member

Dolite commented Aug 30, 2023

On va aussi en profiter pour repasser l'installation de psycopg via pip plutôt que paquet debian seulement.

@Dolite
Copy link
Member

Dolite commented Aug 30, 2023

Est ce qu'on peut juste ajouter à cette PR le test sur la disponibilité de rados pour qu'un appel concernant du stockage CEPH sorte avec une exception NotImplementedError ou StorageError ?

@Guts
Copy link
Contributor Author

Guts commented Aug 30, 2023

On va aussi en profiter pour repasser l'installation de psycopg via pip plutôt que paquet debian seulement.

Comme ceci ? https://gitlab.gpf-tech.ign.fr/geoplateforme/gpf-entrepot-toolbelt-py/-/blob/main/docs/usage/installation.md#installer-les-d%C3%A9pendances-li%C3%A9es-au-client-postgresql

Est ce qu'on peut juste ajouter à cette PR le test sur la disponibilité de rados pour qu'un appel concernant du stockage CEPH sorte avec une exception NotImplementedError ou StorageError ?

Oui, je peux faire ça. Côté installation je n'ai pas trop de doute (jai viré rados de ma machine). J'ai des doutes au moment de l'utilisation de la lib, d'où ma demande de tests auprès de vous

@Guts Guts force-pushed the packaging/make-it-independent-from-ceph branch from aa180f4 to 18fe810 Compare August 31, 2023 12:47
@Guts
Copy link
Contributor Author

Guts commented Aug 31, 2023

Tâches à suivre post-merge :

  • créer une exception dédiée héritée de NotImplementedError
  • la lever à chaque fois qu'un appel à rados/ceph est nécessaire

@Dolite Dolite merged commit 73c5a7b into rok4:develop Aug 31, 2023
1 check passed
@Guts Guts deleted the packaging/make-it-independent-from-ceph branch August 31, 2023 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants