Skip to content

Conversation

@ravening
Copy link
Member

Description

Ported from apache/cloudstack-primate#778

This is the UI change needed for confgiuring load balancer
rules using haproxy and also adding ssl certificated to
load balancer rules

Load balancer configs are avaiable at two scopes.
one at the network level under "LB Configs" tab and
another under each load balancer rule and can be configured
by clicking on "Configuration" button.

The SSL certificates can be added only for load balancer
rules with ssl protocols

To apply global load balancing configurations, navigate to "Networks" -> LB Configs

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

Ported from apache/cloudstack-primate#778

This is the UI change needed for confgiuring load balancer
rules using haproxy and also adding ssl certificated to
load balancer rules

Load balancer configs are avaiable at two scopes.
one at the network level under "LB Configs" tab and
another under each load balancer rule and can be configured
by clicking on "Configuration" button.

The SSL certificates can be added only for load balancer
rules with ssl protocols

To apply global load balancing configurations, navigate to "Networks" -> LB Configs
@DaanHoogland
Copy link
Contributor

@ravening this is for 4.16?

@onitake
Copy link
Contributor

onitake commented Jan 27, 2021

This needs to merged together with (or after) #4141 , so it should depend on that.

@ravening
Copy link
Member Author

ravening commented Feb 1, 2021

@ravening this is for 4.16?

@DaanHoogland Wei might use this change along with his backend pr only. so i will close this once he updates the pr

@weizhouapache
Copy link
Member

@ravening this is for 4.16?

@DaanHoogland Wei might use this change along with his backend pr only. so i will close this once he updates the pr

@ravening
I have updated pr #4141 to include your changes.

@ravening
Copy link
Member Author

ravening commented Feb 1, 2021

@ravening this is for 4.16?

@DaanHoogland Wei might use this change along with his backend pr only. so i will close this once he updates the pr

@ravening
I have updated pr #4141 to include your changes.

Closing this pr

@ravening ravening closed this Feb 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants