planetscale / planetscale/vitess-operator

Inclusive naming

Open
#214 0 comments 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Go
Stars
372
Forks
102
Avg merge
3d 5h
Merged PRs (30d)
15

Description

Work is in progress on the vitess repo to deprecate/rename/fix comments to not use oppressive terminology.
As part of this, tablet_type MASTER will change to PRIMARY.
On the next vitess upgrade of operator after that is merged, code changes will be needed for operator code to compile.
As part of that all usage of master from code and comments should be replaced with primary.
Also, there are (will be) new RPCs available from vitess that should be used in place of deprecated ones.

Refs:
https://github.com/vitessio/vitess/issues/7113
https://github.com/vitessio/vitess/issues/7114

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by searching the operator code and comments for master, then review the Vitess upgrade and the new RPCs referenced in issues 7113 and 7114. Done means the operator compiles against the upgraded Vitess version, uses primary terminology, and no longer depends on the deprecated RPCs.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, kubernetes
Domain
infrastructure
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.