cmis: add a file watch mechanism (#73466) #54

Open
ecazenave wants to merge 2 commits from wip/73466-cmis-watch into main
Owner

Par rapport à la description du ticket, j'ai choisi l'option faire en sorte que "monitor" ne notifie qu'une fois charge au trigger d'avoir en première action de placer à nouveau un appel vers l'API "monitor" si nécessaire.

La vérification du fichier ne s'arrête que si l'appel qu callback réussit. En cas d'URL de callback foireux, ça va continuer être appelé indéfiniment. J'imaginais ajouter dans un deuxième temps un mécanisme de nettoyage, genre tous ce qui est plus vieux que X mois, on supprime. Ouvert aux suggestions.

J'ai appelé le endpoint 'watch' plutôt que monitor (souvenirs de watchdog, et je trouve le terme monitor moche).

Un endpoint check_object pour pouvoir forcer la vérification d'un objet, j'imagine ça utile uniquement pour des tests. En régime de croisière c'est la tache 'daily' qui doit faire le taf.

Et d'ailleurs j'utilise le terme 'object' ici et là plutôt que 'file' parce que ça me semble pouvoir servir à surveiller un dossier aussi bien qu'un fichier.

Par rapport à la description du ticket, j'ai choisi l'option faire en sorte que "monitor" ne notifie qu'une fois charge au trigger d'avoir en première action de placer à nouveau un appel vers l'API "monitor" si nécessaire. La vérification du fichier ne s'arrête que si l'appel qu callback réussit. En cas d'URL de callback foireux, ça va continuer être appelé indéfiniment. J'imaginais ajouter dans un deuxième temps un mécanisme de nettoyage, genre tous ce qui est plus vieux que X mois, on supprime. Ouvert aux suggestions. J'ai appelé le endpoint 'watch' plutôt que monitor (souvenirs de watchdog, et je trouve le terme monitor moche). Un endpoint check_object pour pouvoir forcer la vérification d'un objet, j'imagine ça utile uniquement pour des tests. En régime de croisière c'est la tache 'daily' qui doit faire le taf. Et d'ailleurs j'utilise le terme 'object' ici et là plutôt que 'file' parce que ça me semble pouvoir servir à surveiller un dossier aussi bien qu'un fichier.
ecazenave added 2 commits 2023-01-24 11:42:18 +01:00
gitea-wip/passerelle/pipeline/pr-main This commit looks good Details
gitea/passerelle/pipeline/pr-main Something is wrong with the build of this commit Details
gitea/passerelle/pipeline/head There was a failure building this commit Details
9bf91ae7a5
cmis: add a file watch mechanism (#73466)
Owner

La vérification du fichier ne s'arrête que si l'appel qu callback réussit. En cas d'URL de callback foireux, ça va continuer être appelé indéfiniment. J'imaginais ajouter dans un deuxième temps un mécanisme de nettoyage, genre tous ce qui est plus vieux que X mois, on supprime. Ouvert aux suggestions.

Oui encore un objet qui va traîner je serai d'avis d'immédiatement prévoir une durée de vie fixe pour ces objets, genre 3 mois et une autre fournie à la création pour rallonger sa durée de vie d'autant.

> La vérification du fichier ne s'arrête que si l'appel qu callback réussit. En cas d'URL de callback foireux, ça va continuer être appelé indéfiniment. J'imaginais ajouter dans un deuxième temps un mécanisme de nettoyage, genre tous ce qui est plus vieux que X mois, on supprime. Ouvert aux suggestions. Oui encore un objet qui va traîner je serai d'avis d'immédiatement prévoir une durée de vie fixe pour ces objets, genre 3 mois et une autre fournie à la création pour rallonger sa durée de vie d'autant.
Some checks reported errors
gitea-wip/passerelle/pipeline/pr-main This commit looks good
gitea/passerelle/pipeline/pr-main Something is wrong with the build of this commit
gitea/passerelle/pipeline/head There was a failure building this commit
This pull request has changes conflicting with the target branch.
  • tests/test_cmis.py
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#54
No description provided.