Skip to content

Conversation

aruniiird
Copy link
Contributor

When we detect a 2 Nodes + 1 Arbiter cluster, we should only show, '2-way Replication' in 'Data protection policy' list

How is '2 Nodes + 1 Arbiter' cluster detected?
There is an entry in 'Infrastructure' CR's controlPlaneTopology status, value should be HighlyAvailableArbiter.

@aruniiird aruniiird force-pushed the storage-pool-creation-ui-changes-for-2node-1arb-cluster-setup branch from 7970644 to 259e6c7 Compare July 20, 2025 14:30
Copy link
Contributor

openshift-ci bot commented Jul 20, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: aruniiird
Once this PR has been reviewed and has the lgtm label, please assign alfonsomthd for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@aruniiird aruniiird force-pushed the storage-pool-creation-ui-changes-for-2node-1arb-cluster-setup branch from 259e6c7 to cd0d893 Compare July 22, 2025 08:35
@aruniiird aruniiird force-pushed the storage-pool-creation-ui-changes-for-2node-1arb-cluster-setup branch from cd0d893 to 72d3d9b Compare July 22, 2025 12:02
@aruniiird aruniiird force-pushed the storage-pool-creation-ui-changes-for-2node-1arb-cluster-setup branch 4 times, most recently from c1f916f to d6f2525 Compare July 23, 2025 11:10
@aruniiird
Copy link
Contributor Author

Currently I have TWO issues,
a. The green arbiter label does not have the expected color and not aligned according to the UX design

UX design

image

Actual UI

image

b. Second issue is when you click on the Sortable arrow alonge the Name label (see the above actual UI screenshot), it is going in to a rotating state with a message PersistentVolumes are being provisioned on the selected nodes.

image

@aruniiird aruniiird force-pushed the storage-pool-creation-ui-changes-for-2node-1arb-cluster-setup branch 4 times, most recently from f6047e7 to de9b29f Compare July 29, 2025 12:13
@agarwal-mudit
Copy link
Member

/hold
Pls finish the reviews and keep it ready to merge but don't merge till next update

@aruniiird aruniiird force-pushed the storage-pool-creation-ui-changes-for-2node-1arb-cluster-setup branch 9 times, most recently from b63901d to e21d994 Compare August 4, 2025 11:21
@aruniiird
Copy link
Contributor Author

One of the TWO issues I was facing is resolved now (the first issue, where Arbiter label was not alligned properly).
Thanks to @bipuladh
Additionally, the table footer text is also fixed. Please see the below screen shot

Screenshot from 2025-08-04 16-48-12

@aruniiird
Copy link
Contributor Author

aruniiird commented Aug 5, 2025

@SanjalKatiyar , @bipuladh , please take a look at the e2e-aws test failure.
Error message:
{ error occurred handling build odf-console-amd64: the build odf-console-amd64 failed after 2m49s with reason DockerBuildFailed: Dockerfile build strategy has failed.}

Don't think this is relevant to this PR

EDIT: above was a different error. now I could see that we have test failures related to getDeviceSetReplica() function
fixing them

@aruniiird aruniiird force-pushed the storage-pool-creation-ui-changes-for-2node-1arb-cluster-setup branch from e21d994 to 5bfd053 Compare August 5, 2025 11:00
@aruniiird aruniiird requested a review from SanjalKatiyar August 5, 2025 11:00
@aruniiird aruniiird force-pushed the storage-pool-creation-ui-changes-for-2node-1arb-cluster-setup branch 3 times, most recently from 8f1af47 to e98bbaf Compare August 5, 2025 16:38
When we detect a 2 Nodes + 1 Arbiter cluster, we should only show,
'2-way Replication' in 'Data protection policy' list

How is '2 Nodes + 1 Arbiter' cluster detected?
There is an entry in 'Infrastructure' CR's `controlPlaneTopology` status,
value should be `HighlyAvailableArbiter`.

Added changes for 'StorageSystem' creation steps.
Made changes to types as well.

Signed-off-by: Arun Kumar Mohan <[email protected]>
Selected Nodes table footer sample (for a TNA cluster setup)
Before Change (this included arbiter resources):
'3 node selected (36 CPUs and 156.9 GiB on 0 zone)'

After Change (now we exclude the arbiter resources):
'2 nodes selected (32 CPUs and 125.6 GiB on 0 zone) + 1 arbiter detected'

Change description: previously we had included the arbiter resources as
well, but now we include only resources from nodes other than arbiter
node.

Signed-off-by: Arun Kumar Mohan <[email protected]>
@aruniiird aruniiird force-pushed the storage-pool-creation-ui-changes-for-2node-1arb-cluster-setup branch from e98bbaf to 1673ffb Compare August 20, 2025 14:08
Copy link
Contributor

openshift-ci bot commented Aug 20, 2025

@aruniiird: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/odf-console-e2e-aws 1673ffb link true /test odf-console-e2e-aws

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@openshift-merge-robot
Copy link
Contributor

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

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.

8 participants