Project

General

Profile

Actions
Copy link

Feature #11164

closed
SW DZ

Input validation to prevent setting a load balancing gateway group as default

Feature #11164: Input validation to prevent setting a load balancing gateway group as default

Added by Steve Wheeler over 5 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Low
Assignee:
Danilo Zrenjanin
Category:
Gateways
Target version:
2.5.2
Start date:
12/15/2020
Due date:
% Done:

0%

Estimated time:
Plus Target Version:
21.05
Release Notes:
Default

Description

Currently it's possible to set a load-balance gateway group as the system default gateway in System > Routing > Gateways.

That is an invalid setting though, only individual gateways or failover groups are valid there.

It would be nice for the input validation to prevent that but that would mean code to categorise gateway groups. That could get complex quickly.

Failing that a note on that page such as changing the text to:

Select a gateway or failover gatewaygroup to use as the default gateway.

Files

f_o.png (6.64 KB) f_o.png Alhusein Zawi, 04/21/2021 09:17 PM

SW Updated by Steve Wheeler over 5 years ago Actions
Copy link
 #1

Reviewing it appears the intended behaviour here is that a load-balance group can be selected but behaves as a failover group with whichever gateway is first acrting as the higher tier. Importantly it cannot actually load-balance.

So a note to that effect on the page would probably suffice here.

JT Updated by Jim Thompson over 5 years ago Actions
Copy link
 #2

  • Target version changed from Future to CE-Next

Seems simple enough to fix (with the message) and the GUI is misleading.

JP Updated by Jim Pingle over 5 years ago Actions
Copy link
 #4

  • Status changed from New to Pull Request Review

RB Updated by Renato Botelho over 5 years ago Actions
Copy link
 #5

  • Status changed from Pull Request Review to Feedback
  • Assignee set to Danilo Zrenjanin

PR has been merged. Thanks!

RB Updated by Renato Botelho over 5 years ago Actions
Copy link
 #6

  • Target version changed from CE-Next to 2.6.0

AZ Updated by Alhusein Zawi over 5 years ago Actions
Copy link
 #7

note is added (attached)

2.6.0-DEVELOPMENT (amd64)
built on Wed Apr 21 01:03:55 EDT 2021
FreeBSD 12.2-STABLE

VG Updated by Viktor Gurov over 5 years ago Actions
Copy link
 #8

  • Status changed from Feedback to Resolved

JP Updated by Jim Pingle over 5 years ago Actions
Copy link
 #9

  • Plus Target Version set to 21.05

Already present on 21.05 builds.

JP Updated by Jim Pingle over 5 years ago Actions
Copy link
 #10

  • Subject changed from Prevent setting a load-balance gateway group as default. to Input validation to prevent setting a load balancing gateway group as default

Updating subject for release notes.

JP Updated by Jim Pingle over 5 years ago Actions
Copy link
 #11

  • Target version changed from 2.6.0 to 2.5.2
Actions
Copy link

Also available in: Atom