Bug 280 - Not possible to configure load balancing in tlwebadm
Summary: Not possible to configure load balancing in tlwebadm
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Web Administration (show other bugs)
Version: 1.1
Hardware: PC Linux
: P2 Enhancement
Target Milestone: 4.19.0
Assignee: Samuel Mannehed
URL:
Keywords: focus_loadbalancer, linma_tester, relnotes
Depends on: 8566
Blocks:
  Show dependency treegraph
 
Reported: 2003-04-29 10:37 CEST by Kjell Enblom
Modified: 2025-05-05 15:50 CEST (History)
3 users (show)

See Also:
Acceptance Criteria:
MUST: * Agent weights must be configurable from Web Admin. SHOULD: * It should be easy to reset agent weights back to their default values.


Attachments
Duplicate agents in tlwebadm weights list (45.78 KB, image/png)
2025-04-23 13:17 CEST, Tobias
Details

Description Kjell Enblom cendio 2003-04-29 10:37:17 CEST
There should be a module configuration for load balance in tladm.
Comment 1 Peter Åstrand cendio 2003-08-26 10:23:05 CEST
Need to be able to configure:

* configurable limits (when fields should be marked red)

Currently, the value is so high that fields will *never* be marked red. 
Comment 2 Samuel Mannehed cendio 2025-04-03 14:54:18 CEST
As of bug 4429, there is no longer any configuration parameters pertaining to the load balancer. I will put a dependency on bug 8566 where the idea is to add the possibility of configuring weights for different agents.
Comment 8 Samuel Mannehed cendio 2025-04-22 10:03:05 CEST
Done.

> MUST:
> 
> * Agent weights must be configurable from Web Admin.
They are, under Cluster/Agents.

> SHOULD:
> 
> * It should be easy to reset agent weights back to their default values.
If the value is removed from the weight input field, a placeholder containing the default value is shown. If the form is saved with an empty input, the default value is used.
Comment 9 Tobias cendio 2025-04-23 13:17:20 CEST
Created attachment 1270 [details]
Duplicate agents in tlwebadm weights list

If an agent is associated with multiple subclusters, it will show up multiple times in the load balancer's weight table.

Saving different weights for the same agent in different entries appears to result in it picking the first weight in the list.
Comment 12 Samuel Mannehed cendio 2025-04-23 17:57:35 CEST
(In reply to Tobias from comment #9)
> Created attachment 1270 [details]
> Duplicate agents in tlwebadm weights list
> 
> If an agent is associated with multiple subclusters, it will show up
> multiple times in the load balancer's weight table.
> 
> Saving different weights for the same agent in different entries appears to
> result in it picking the first weight in the list.

Fixed now.
Comment 15 Linn cendio 2025-04-29 09:31:20 CEST
While looking through the documentation, I noticed the screenshots of Web Admin were not updated. I also saw some sentences that were hard to read.

I'll go ahead and fix these small issues. Reopening.
Comment 21 Samuel Mannehed cendio 2025-05-02 11:00:46 CEST
The documentation and screenshots have now been updated. The things that were cause for reopen have been fixed.

Verified build 4034 on Fedora 41. The weight page does not crash when there are no subclusters when saving. The default weight (100) is used when the input field is empty when saving.
Comment 22 Linn cendio 2025-05-05 09:38:25 CEST
Tested server build 4029 on Ubuntu 22.04, and did not find any issues. Tested the following:

* Setting new weights for the agent (changing both one value and multiple values at once)
* Saving an empty value, which sets the value to default value
* Agent in multiple clusters only show up once
* Cannot save a value less than 1 or non-integer
* Loading a bad value from conf results in the following error message being shown. One error message is shown for each faulty variable.
> Invalid weight configured for agent 'agent2', using default weight of 100

Also looked through the commits and release notes, look good. 

> MUST:
> ✅ Agent weights must be configurable from Web Admin.
They are, see above for test scenarios.

> SHOULD:
> ✅ It should be easy to reset agent weights back to their default values.
There is no built-in way to explicitly reset the values, but there is a text stating the default value. The weight field also presents the default value as a suggestion when the field is empty.

Note You need to log in before you can comment on or make changes to this bug.