Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace usages of whitelist/blacklist and master/slave #1287

Open
letzya opened this issue Jun 15, 2020 · 3 comments
Open

Replace usages of whitelist/blacklist and master/slave #1287

letzya opened this issue Jun 15, 2020 · 3 comments
Labels
bug Error Error in the docs Wrong

Comments

@letzya
Copy link
Collaborator

letzya commented Jun 15, 2020

Branch/Environment/Version

All releases up to today (2.9.4.1)

Relevant Document

Describe the error

Replace usages of whitelist/blacklist and master/slave.
This probably needs a bit of research before changing the docs.

My personal choice, based on the term I practice on a daily basis:

  1. Master/slave ==> Primary/worker
    Off docs, we already use:
  • primary DC, primary redis, primary gw.
  • worker DC, worker GW. Redis-secondary
  1. Blacklist/white list ==> blocked list/allowed list or blocklist/allowlist

Screenshots/Video

Additional context

this doc can offer some options https://tools.ietf.org/id/draft-knodel-terminology-00.html#rfc.section.1.1

@letzya letzya changed the title all: replace usages of whitelist/blacklist and master/slave Replace usages of whitelist/blacklist and master/slave Jun 15, 2020
@letzya letzya added bug Error Error in the docs labels Jun 15, 2020
@marksou
Copy link
Contributor

marksou commented Jun 15, 2020

We could use Allow List and Deny List instead of White and Blacklist.

@joshblakeley
Copy link
Member

While we're here change referring to master branches with main since that will change too

@davegarvey
Copy link
Member

For the data centre stuff we could use data plane and control plane.

@marksou marksou added the Wrong label Jul 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Error Error in the docs Wrong
Projects
None yet
Development

No branches or pull requests

4 participants