influxdata/telegraf

Migrate from couchbase/go-couchbase to couchbase/gocb

Open

#9,813 opened on Sep 23, 2021

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Go (4,161 forks)batch import
area/couchbasehelp wantedsize/l

Repository metrics

Stars
 (9,892 stars)
PR merge metrics
 (Avg merge 2d 12h) (136 merged PRs in 30d)

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.

Contributor guide

Research direction
Examine the current Couchbase input plugin's code that uses couchbase/go couchbase and understand how to replace it with couchbase/gocb. Study the gocb documentation and API differences, then plan a step by step migration.
Tech stack
go
Domain
backenddeveloper experience
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GoCouchbase
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60