influxdata/telegraf

Migrate from couchbase/go-couchbase to couchbase/gocb

Open

#9 813 ouverte le 23 sept. 2021

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Go (4 161 forks)batch import
area/couchbasehelp wantedsize/l

Métriques du dépôt

Stars
 (9 892 stars)
Métriques de merge PR
 (Merge moyen 2j 12h) (136 PRs mergées en 30 j)

Description

Couchbase provides two go client modules. couchbase/gocb is the current official module and couchbase/go-couchbase is an older unofficial module.

The telegraf couchbase input uses couchbase/go-couchbase. We have recently had to work around missing functionality (#9032) and insecure code (#9485) in it.

Telegraf should switch to gocb because it is the official supported module.

There is an old closed issue to switch to gocb (#2418). At the time of that issue the gocb module was new and didn't have functionality telegraf needed. We should revisit this now that time has past and gocb has matured.

Guide contributeur

Direction de recherche
Commencez par examiner l'issue fermée #2418 pour le contexte historique. Ensuite, examinez le plugin d'entrée couchbase actuel dans le dépôt telegraf (plugins/inputs/couchbase/). Comprenez l'API gocb en lisant sa documentation. La migration implique de remplacer les appels couchbase/go couchbase par des équivalents gocb, en veillant à ce que toutes les métriques soient collectées correctement. Vérifiez s'il y a des changements cassants ou des fonctionnalités manquantes dans gocb par rapport à l'ancien client.
Stack technique
go
Domaine
backenddeveloper experience
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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
GoCouchbase
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60