https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342019-01-22T09:51:09ZRedmine Entr’ouvertPasserelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1520602019-01-22T09:51:09ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-6 priority-4 priority-default closed" href="/issues/28637">Development #28637</a>: Logguer quand même en DB lorsqu'un connecteur est 'down' </i> ajouté</li></ul> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1520662019-01-22T09:57:17ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><blockquote>
<p>Lorsque le test de disponibilité s'avère instable, il serait commode de pouvoir débrayer le test de disponibilité dans l'UI et ainsi obtenir des logs.</p>
</blockquote>
<p>Pas tellement envie de charger l'UI avec ça; ça ne pourrait pas simplement être un settings global ?</p> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1521672019-01-22T13:29:12ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul></ul><p>Je trouverais ça beaucoup moins pratique.</p> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1521742019-01-22T13:40:10ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><blockquote>
<p>Je trouverais ça beaucoup moins pratique.</p>
</blockquote>
<p>Parce que tu es la personne qui sur le moment a besoin de cette option. Pour toutes les autres personnes qui ajouteraient/modifieraient un connecteur, ça ferait une case à cocher en plus.</p> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1521852019-01-22T14:00:08ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul></ul><p>En fait je n'imaginais pas ça sur la page d'édition principale du connecteur, mais dans une popup qui s'ouvrait à partir d'un lien figurant à droite de 'paramètres de journalisation'.</p>
<p>Pour aller jusqu'au bout de l'idée, je pensais un nouveau champ booléen <code>force_up</code> sur <code>ResourceStatus</code>, par défaut à <code>False</code>, qui peut facilement influer les méthodes <code>up</code> and <code>down</code> et être utilisé dans <code>availability</code> pour zapper le <code>check_status</code>.</p>
<p>Et donc la popup serait une simple checkbox d'édition de ce champ.</p> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1524202019-01-23T10:47:14ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul></ul><p>Loupé le fait qu'on écrit successivement des ResourcesStatus, je pensais qu'il y en avait qu'un par connecteur, donc le flag serait sur l'objet connecteur, ce qui n'empêche pas de l'éditer ailleurs que sur la page d'édition principale.</p> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1545682019-02-05T14:53:05ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/31506">0001-manage-availability-check-through-the-UI-29965.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/31506/0001-manage-availability-check-through-the-UI-29965.patch">0001-manage-availability-check-through-the-UI-29965.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/31508">Screenshot-2019-2-5 Passerelle.png</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/31508/Screenshot-2019-2-5%20Passerelle.png">Screenshot-2019-2-5 Passerelle.png</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/31507">Screenshot-2019-2-5 Passerelle(1).png</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/31507/Screenshot-2019-2-5%20Passerelle(1).png">Screenshot-2019-2-5 Passerelle(1).png</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>Finalement je stocke ce flag <code>disable_check</code> dans un modèle à part <code>AvailibityParameters</code>, dans l'idée qu'on pourra vouloir y rajouter un pilotage plus fin de la disponibilité, genre périodicité des tests ou que sais-je.</p>
<p>Si le flag est mis à <code>True</code>, plus de tests de dispos, et le connecteur passe à 'up' si il était 'down' (on pourrait aussi plus tard donner le choix à l'utilisateur sur le statut cible en cas de débrayage du suivi).</p>
<p>Coté UI, cf captures, une nouvelle entrée de le menu, qui ne s'affiche que sur les connecteurs où des tests de disponibilité ont déjà eu lieu. Je voulais faire un truc plus joli avec un couple d'icônes à la place du texte 'manage availability' qui aurait indiqué tout de suite l'état actif ou non du suivi de disponibilité, mais je me suis perdu dans font-awesome et gadjo.</p> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1545752019-02-05T15:04:11ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Plutôt "availability check" comme libellé de bouton; et plutôt formuler l'option dans le sens inverse (option qui s'appellerait "Run regular availability checks", cochée par défaut); peut-être avec un help_text qui dit que ça tourne toutes les x minutes ?</p>
<p>Histoire de ne pas trainer ça pendant des années, corriger dès maintenant le nom du modèle (availibity vs availability).</p>
<blockquote>
<p>qui ne s'affiche que sur les connecteurs où des tests de disponibilité ont déjà eu lieu</p>
</blockquote>
<p>Je l'afficherais sur les connecteurs où les tests de disponibilité sont possibles. (genre ajouter un attribut à la méthode check_status de la classe de base, pour différencier facilement cette méthode de celles définies pour faire vraiment quelque chose).</p>
<pre>
resource.logger.info(
u'connector "%s" (%s) put back up by user', resource, resource.__class__.__name__)
</pre>
<p>Je logguerais simplement "availability reports enabled (ou disabled)", peu importe si c'était connu down, ou pas.</p> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1555282019-02-11T16:18:24ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/31680">0001-manage-availability-check-through-the-UI-29965.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/31680/0001-manage-availability-check-through-the-UI-29965.patch">0001-manage-availability-check-through-the-UI-29965.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/31679">0002-remove-default-check_status-29965.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/31679/0002-remove-default-check_status-29965.patch">0002-remove-default-check_status-29965.patch</a> ajouté</li><li><strong>Assigné à</strong> mis à <i>Emmanuel Cazenave</i></li></ul><p>Toutes les remarques prises en compte dans <code>0001-manage-availability-check-through-the-UI-29965.patch</code>.</p>
<p><code>0002-remove-default-check_status-29965.patch</code> optionnel : simplifie la discrimination entre connecteurs qui implémentent <code>check_status</code> et les autres (à la lecture du patch de <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: endpoint /check (ou autre) pour vérif minimale du bon fonctionnement du service distant (Fermé)" href="https://dev.entrouvert.org/issues/9416">#9416</a>, je ne vois pas la nécessite de check_status sur <code>BaseResource</code>, donc autant simplifier plutôt que de rajouter un attribut sur la méthode me suis-je dit).</p> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1555342019-02-11T16:31:07ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><blockquote>
<p>0002-remove-default-check_status-29965.patch optionnel : simplifie la discrimination entre connecteurs qui implémentent check_status et les autres (à la lecture du patch de <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: endpoint /check (ou autre) pour vérif minimale du bon fonctionnement du service distant (Fermé)" href="https://dev.entrouvert.org/issues/9416">#9416</a>, je ne vois pas la nécessite de check_status sur BaseResource, donc autant simplifier plutôt que de rajouter un attribut sur la méthode me suis-je dit).</p>
</blockquote>
<p>De mon côté j'aime bien que la classe de base puisse décrire les méthodes attendues; ailleurs (combo), pour des raisons de performances, on fait sans mais on garde quand même un attribut, à None, exemple :</p>
<pre>
# get_badge(self, context); set to None so cell types can be skipped easily
get_badge = None
</pre>
<p>(ce qui retirerait la nécessité d'hasattr, ce qui est toujours mieux)</p>
<p>Bref, sur 0002, j'aime bien sans, que la méthode soit clairement présente; mais si on veut la retirer qu'un attribut None prenne sa place, que ça permettre de retirer de l'hasattr.</p> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1555432019-02-11T17:58:59ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul></ul><p>Ok merci pour les explications.</p>
<p>Restons en à l'essentiel, oublions <code>0002-remove-default-check_status-29965.patch</code>.</p> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1571202019-02-19T13:25:38ZChristophe Siraut
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1572702019-02-20T10:09:37ZEmmanuel Cazenaveecazenave@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>
commit fa76fbc8d7a0576eb94ff16cc5f6161e1c8fec68
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Tue Jan 22 14:34:56 2019 +0100
manage availability check through the UI (#29965)
</pre> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1573832019-02-20T14:16:03ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> Passerelle - Development #29965: Débrayer la vérification de disponibilité d'un connecteurhttps://dev.entrouvert.org/issues/29965?journal_id=1672442019-04-12T08:27:28ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution déployée</i> à <i>Fermé</i></li></ul>