Skip to content

ui: Fix create kubernetes cluster with ha enabled#6416

Merged
nvazquez merged 1 commit intoapache:mainfrom
shapeblue:fix-create-ha-kube-cluster
May 30, 2022
Merged

ui: Fix create kubernetes cluster with ha enabled#6416
nvazquez merged 1 commit intoapache:mainfrom
shapeblue:fix-create-ha-kube-cluster

Conversation

@davidjumani
Copy link
Copy Markdown
Contributor

Description

Fixes the issue of ha kubernetes clusters not coming up with the specified control plane nodes

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)

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

Before :

Screenshot from 2022-05-30 16-01-22

After :

Screenshot from 2022-05-30 15-59-48

@davidjumani davidjumani added this to the 4.17.0.0 milestone May 30, 2022
@acs-robot
Copy link
Copy Markdown

Found UI changes, kicking a new UI QA build
@blueorangutan ui

Copy link
Copy Markdown
Contributor

@nvazquez nvazquez left a comment

Choose a reason for hiding this comment

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

LGTM - manually tested. Control nodes are passed correctly and running as specified
Screen Shot 2022-05-30 at 08 30 06

Copy link
Copy Markdown
Member

@weizhouapache weizhouapache left a comment

Choose a reason for hiding this comment

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

code lgtm

Copy link
Copy Markdown
Contributor

@Pearl1594 Pearl1594 left a comment

Choose a reason for hiding this comment

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

LGTM

@nvazquez nvazquez merged commit c249e25 into apache:main May 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants