rust-lang-deprecated / rust-buildbot

Buildbot configs for the Rust project
http://buildbot.rust-lang.org/
Apache License 2.0
36 stars 27 forks source link

Consider moving from master/slave terminology #2

Closed cllns closed 9 years ago

cllns commented 9 years ago

This seems like an internal tool, but it might be a good idea to change from 'master/slave' to something less problematic.

I'm not familiar with this tool, but primary/replica might work. (or leader/follower)

@steveklabnik @brson, thoughts?

steveklabnik commented 9 years ago

Very, very +1

(and primary/replica is my preferred, but just anything other than master/slave)

brson commented 9 years ago

'master' and 'slave' are the buildbot terminology.

bheesham commented 9 years ago

Yeah, that would be tricky. You can always fork Buildbot and hope a simple text replacement would work

tbu- commented 9 years ago

@steveklabnik Primary/replica doesn't seem to fit, as there's a controller (the current "master") which completely controls the other instances (the current "slaves"), i.e. they're not replicas of the primary.

cllns commented 9 years ago

@tbu- What about leader/follower?

tbu- commented 9 years ago

That sounds better.

trevrosen commented 9 years ago

I think "leader/follower" works as a pretty much universal replacement for "master/slave". I don't do any Rust at work, but we've started to use this nomenclature for multi-server environments there.

eevee commented 9 years ago

uwsgi uses emperor/vassal for some similar feature.

"peon" is a great word too. but then i'm inclined towards colorful names :)

tonylukasavage commented 9 years ago

leader/follower is pretty popular among projects making this particular change.

fables-tales commented 9 years ago

@steveklabnik leader/follower seems correct in this case for me.

coderanger commented 9 years ago

For CI stuff in the past I've used "server/worker" or "server/builder".

jedahan commented 9 years ago

leader/follower seems more general than leader/builder

though I like both

runlevel5 commented 9 years ago

Oh my, please check https://github.com/django/django/pull/2692

alex commented 9 years ago

coordinator/worker is probably what I'd use if I were building a CI system.

effigies commented 9 years ago

Would suggest controller/worker, but +1 to coordinator/worker, too.

timplunkett commented 9 years ago

Drupal decided on using primary/replica, following Django's lead.

alex commented 9 years ago

A build system's naming scheme isn't describing the same thing as a database system; I do think primary/replica is the right naming for database replication schemes.

On Wed, Feb 4, 2015 at 3:25 PM, Tim Plunkett notifications@github.com wrote:

Drupal decided on using primary/replica, following Django's lead.

— Reply to this email directly or view it on GitHub https://github.com/rust-lang/rust-buildbot/issues/2#issuecomment-72962526 .

"I disapprove of what you say, but I will defend to the death your right to say it." -- Evelyn Beatrice Hall (summarizing Voltaire) "The people's good is the highest law." -- Cicero GPG Key fingerprint: 125F 5C67 DFE9 4084

tomburns commented 9 years ago

+1 for leader / follower, given the context.

jjjjw commented 9 years ago

primary/secondary also works, if primary/replica seems weird.

reaperhulk commented 9 years ago

http://trac.buildbot.net/ticket/2340 may also be of interest.

mikeedwards commented 9 years ago

primary/secondary? That's sort of what mongo does: http://docs.mongodb.org/manual/core/replication/

Primary and replica would work, too, but that might be too specific in terms of function of the "non-primary" machine. Secondary is nicely general, if that matters.

tbu- commented 9 years ago

@zjjw Your terminology doesn't quite fit, it might fit for databases, but not for build servers.

server/worker might fit.

cllns commented 9 years ago

@reaperhulk I was in the process of writing a comment there when you posted the link :+1:

tomjakubowski commented 9 years ago

We should avoid using "leader" in upper case or any other context where it could be construed as a title.

kamalmarhubi commented 9 years ago

slave -> worker above the rest of the suggestions.

Perhaps master is all right with that change, trying to decide for myself.

For the master:

Edit: ha! Marx's comments on the buildbot ticket are my exact potential issue with master / worker.

colingourlay commented 9 years ago

I'm throwing in gru / minion.

O-I commented 9 years ago

I think primary/replica has precedent in replacing master/slave terminology like in the aforementioned Django issue mentioned by @joneslee85, but I find myself more in the leader/follower camp.

If it didn't make things even more confusing, I'd suggest something like yang/yin. :smiley:

locks commented 9 years ago

CouchDB also switched to primary / replica, if I remember correctly: https://issues.apache.org/jira/browse/COUCHDB-2248

crazymykl commented 9 years ago

Although leader/follower looks to be in the lead, I humbly propose exemplar/aspirant.

Manishearth commented 9 years ago

primary/replica sounds too database-y, and replication isn't exactly what's happening here.

I prefer leader/follower

lukemayes commented 9 years ago

@tbi- describing the current "master" as a controller made me think controller/drone or controller/worker might be good.

tomjakubowski commented 9 years ago

"Drone" is equally problematic in parts of the world ravaged by US military intervention.

steveklabnik commented 9 years ago

Thank you everyone for your suggestions.

Given that we're not a database, I agree that primary/replica doesn't make a lot of sense. But regardless, this really needs to be fixed upstream. Therefore, I'm doing two things:

  1. Closing this in favor of http://trac.buildbot.net/ticket/2340
  2. Working with said upstream to land a patch. I have been emailing with one of the committers about it, and they're favorable to the change, the work just needs done. I plan on implementing leader/follower.

If upstream ends up rejecting this, we can re-open and see what we can do, but for now, dealing with them is the correct course of action.

dhuseby commented 9 years ago

How about "gun" and "shovel"?

"You see in this world there's two kinds of people, my friend. Those with loaded guns, and those who dig. You dig." https://www.youtube.com/watch?v=6YnryHbJqC4&t=3m34s