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

Added dropdown with accounts for advanced cluster settings #1495

Merged

Conversation

vitalii-honchar
Copy link
Contributor

@vitalii-honchar vitalii-honchar commented Mar 5, 2024

Context

  • Added dropdown with accounts for advanced cluster settings
  • User can choose from what account cluster should be provisioned

Test Plan

  • Provision cluster from specified account in UI.
  • Test update cluster from UI.
Test Details

Provision cluster from specified account in UI

Create cluster: basic settings

Unchanged ✅

Screenshot 2024-03-05 at 12 08 13 PM

Create cluster: advanced settings

Default account ✅

Screenshot 2024-03-05 at 2 53 42 PM

Accounts dropdown ✅

Screenshot 2024-03-05 at 2 55 08 PM

Cluster provision for default account from UI works ✅

Cluster provision not works due to a bug in Rodimus, but UI works ✅

Screenshot 2024-03-05 at 4 09 32 PM

Test Results

UI works as expected, but cluster provisioning from UI not works due to the bug in the cluster provisioning service. AWS account has a lack of some policies:
Screenshot 2024-03-05 at 4 09 32 PM

The bug will be fixed in the Jira task CDP-7717.

Links and ids have removed from screenshots due to security reasons and open source nature of the project.

@github-actions github-actions bot added the deploy-board Includes changes to deploy-board label Mar 5, 2024
@vitalii-honchar vitalii-honchar force-pushed the vhonchar/provision-cluster-from-specified-account branch 2 times, most recently from cfcf170 to e6b0648 Compare March 5, 2024 17:57
@vitalii-honchar vitalii-honchar force-pushed the vhonchar/provision-cluster-from-specified-account branch from d384951 to f230cd1 Compare March 5, 2024 21:15
@vitalii-honchar vitalii-honchar marked this pull request as ready for review March 5, 2024 21:25
@vitalii-honchar vitalii-honchar requested a review from a team as a code owner March 5, 2024 21:25
@vitalii-honchar vitalii-honchar merged commit b9c9743 into master Mar 5, 2024
5 checks passed
@vitalii-honchar vitalii-honchar deleted the vhonchar/provision-cluster-from-specified-account branch March 5, 2024 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy-board Includes changes to deploy-board
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants