Bug 208681

Summary: [patch] CARP preemption explanation is misleading
Product: Documentation Reporter: Philipp Erbelding <philipp>
Component: Books & ArticlesAssignee: Sevan Janiyan <sevan>
Status: Closed FIXED    
Severity: Affects Some People CC: girgen, sevan
Priority: --- Keywords: patch
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://reviews.freebsd.org/D15519
Attachments:
Description Flags
clarified proposal for the preemption note none

Description Philipp Erbelding 2016-04-10 19:25:23 UTC 
Created attachment 169161 [details]
clarified proposal for the preemption note

The handbook states in a note on CARP:
> If preemption has been enabled, hostc.example.org might not release the virtual IP
> address back to the original master server.

The preemption setting does the opposite of what I would interpret this sentence to convey. Preemption needs to be enabled in order for the system with a lower advskew value (the original master) to take the address back.

In writing this, I wonder if it also needs a sentence on where to enable preemption (master, backup or both...). I'd assume there are very few scenarios, where different settings on nodes in the same group would be useful or necessary.

I have attached a proposed change for this note.
Comment 1 Palle Girgensohn freebsd_committer freebsd_triage 2016-11-29 21:40:35 UTC 
I agree fully, the man page's text is the exact opposite, or it is very hard to interpret, which is just as bad. Please commit this, someone. :)
Comment 2 commit-hook freebsd_committer freebsd_triage 2018-05-22 14:34:16 UTC 
A commit references this bug:

Author: sevan
Date: Tue May 22 14:33:42 UTC 2018
New revision: 51703
URL: https://svnweb.freebsd.org/changeset/doc/51703

Log:
  CARP preemption explanation is misleading

  PR:		208681
  Submitted by:	philipp AT copythat de
  Approved by:	bcr (mentor)
  Differential Revision:	https://reviews.freebsd.org/D15519

Changes:
  head/en_US.ISO8859-1/books/handbook/advanced-networking/chapter.xml
Comment 3 Sevan Janiyan freebsd_committer freebsd_triage 2018-05-22 14:34:45 UTC 
Committed, thank you for the patch.