Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(provider/azure): Fix UI for Azure Load Balancer in server group c… #7055

Merged
merged 2 commits into from
May 23, 2019

Conversation

michaeljqzq
Copy link
Contributor

This pull request fix the issues:

  • Load balancer type may be recognized incorrectly
  • Delete load balancer can be clicked even if it is disabled
  • The Enable in bound NAT options will be displayed when user choose Azure Load Balancer as load balancer type

We hope this PR could be applied to v1.14.0 as a hotfix, thanks!

@michaeljqzq
Copy link
Contributor Author

linked with spinnaker/clouddriver#3713

@mybayern1974
Copy link
Contributor

@maggieneterval, please possibly help apply this hotfix PR to v1.14.0. To supplement the background, though there are multiple bug fixes included, the cause could be regarded as unique that we brought in the 2nd Spinnaker Load Balancer type for Azure: Azure Load Balancer (ALB). Between the 2 types of Azure Load Balancers (Load Balancer and Application Gateway), there is compatibility issue which resulted in the above symptoms.

Copy link
Contributor

@maggieneterval maggieneterval left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Happy to cherry-pick into 1.14 as well.

@anotherchrisberry anotherchrisberry merged commit 265946b into spinnaker:master May 23, 2019
@@ -38,7 +38,7 @@ <h3 class="horizontal middle space-between flex-1" select-on-dbl-click>
class="disabled"
tooltip="You must detach all server groups before you can delete this load balancer."
>
<a href ng-click="ctrl.deleteLoadBalancer()">Delete Load Balancer</a>
<a href>Delete Load Balancer</a>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean to remove the click handler?

@maggieneterval
Copy link
Contributor

@spinnakerbot cherry-pick 1.14

spinnakerbot pushed a commit that referenced this pull request May 23, 2019
@spinnakerbot
Copy link
Contributor

Cherry pick successful: #7057

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants