Bug 7676 - It's easy to miss that subcluster association limits session startup in the documentation
Summary: It's easy to miss that subcluster association limits session startup in the d...
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Documentation (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.13.0
Assignee: Niko Lehto
URL:
Keywords: ossman_tester, prosaic, samuel_tester
Depends on:
Blocks:
 
Reported: 2021-04-08 10:26 CEST by Niko Lehto
Modified: 2021-04-14 07:30 CEST (History)
1 user (show)

See Also:
Acceptance Criteria:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Niko Lehto cendio 2021-04-08 10:26:53 CEST 
We should be more explicit with the information that subcluster association rules apply when creating a new session. An user can reconnect to a session outside their cluster if the association were changed after startup.
Comment 2 Pierre Ossman cendio 2021-04-09 11:06:04 CEST 
Looks good, but since we use the very similar concepts "cluster" and "subcluster" it is important we are strict about using them correctly. And the new text uses "cluster" in a few places where it really means "subcluster".
Comment 3 Pierre Ossman cendio 2021-04-09 11:18:38 CEST 
Some minor nitpicking that may or may not be changed:

> ... the user will reconnect ...

I'd use "is able to" as it is not certain if the user will actually reconnect.

> ... the correct cluster.

As a general rule I try to avoid subjective terms like "correct" in favour of more precise phrasings like "configured". A typo might cause the two to be different things. :)
Comment 4 Niko Lehto cendio 2021-04-13 09:29:12 CEST 
(In reply to Pierre Ossman from comment #3)
I agree. I'll change these parts of the documentation.
Comment 6 Samuel Mannehed cendio 2021-04-13 11:32:30 CEST 
Looks good now.
Comment 8 Niko Lehto cendio 2021-04-13 16:38:55 CEST 
(In reply to Pierre Ossman from comment #2)
> Looks good, but since we use the very similar concepts "cluster" and
> "subcluster" it is important we are strict about using them correctly. And
> the new text uses "cluster" in a few places where it really means
> "subcluster".

I had missed this comment earlier when fixing the documentation. This is fixed now.
Comment 9 Samuel Mannehed cendio 2021-04-14 07:30:47 CEST 
Looks good.
Note You need to log in before you can comment on or make changes to this bug.