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

Test the new total storage size is reflected in the UI post resize osd #9975

Conversation

yitzhak12
Copy link
Contributor

No description provided.

@yitzhak12 yitzhak12 added Test Case A test case PR Squad/Brown Squad/Black Covers UI specific test cases such as ODF Dashboard etc. labels Jun 23, 2024
@yitzhak12 yitzhak12 requested review from a team as code owners June 23, 2024 13:17
@pull-request-size pull-request-size bot added the size/M PR that changes 30-99 lines label Jun 23, 2024
@yitzhak12 yitzhak12 force-pushed the create-ui-test-storage-size-post-resize-osd branch from 41f4fb5 to 67e67d3 Compare June 27, 2024 12:08
Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation on existing cluster

Cluster Name: ikave-ibm9-416
Cluster Configuration:
PR Test Suite: tier1
PR Test Path: tests/functional/z_cluster/cluster_expansion/test_resize_osd.py
Additional Test Params:
OCP VERSION: 4.16
OCS VERSION: 4.16
tested against branch: master

Job UNSTABLE (some or all tests failed).

@yitzhak12 yitzhak12 force-pushed the create-ui-test-storage-size-post-resize-osd branch from 67e67d3 to 7742f5b Compare July 1, 2024 07:28
Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation on existing cluster

Cluster Name: ikave-ibm9-416
Cluster Configuration:
PR Test Suite: tier1
PR Test Path: tests/functional/z_cluster/cluster_expansion/test_resize_osd.py
Additional Test Params:
OCP VERSION: 4.16
OCS VERSION: 4.16
tested against branch: master

Job PASSED.

OdedViner
OdedViner previously approved these changes Jul 1, 2024
AviadP
AviadP previously approved these changes Jul 1, 2024
@yitzhak12 yitzhak12 force-pushed the create-ui-test-storage-size-post-resize-osd branch from 7742f5b to df8768e Compare July 1, 2024 15:12
@openshift-ci openshift-ci bot removed the lgtm label Jul 1, 2024
…he 'create_pvcs_and_pods' from the 'setup' method to 'prepare_data_before_resize_osd' method

Signed-off-by: Itzhak Kave <[email protected]>
@yitzhak12 yitzhak12 dismissed stale reviews from OdedViner and AviadP via aae6292 July 1, 2024 15:18
@yitzhak12 yitzhak12 force-pushed the create-ui-test-storage-size-post-resize-osd branch from df8768e to aae6292 Compare July 1, 2024 15:18
UI 'ocs-storagecluster-storagesystem' page.

"""
block_and_file = (
Copy link
Contributor

Choose a reason for hiding this comment

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

nice!

Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation on existing cluster

Cluster Name: ikave-ibm9-416
Cluster Configuration:
PR Test Suite: tier1
PR Test Path: tests/functional/z_cluster/cluster_expansion/test_resize_osd.py
Additional Test Params:
OCP VERSION: 4.16
OCS VERSION: 4.16
tested against branch: master

Job PASSED.

ebondare
ebondare previously approved these changes Jul 2, 2024
DanielOsypenko
DanielOsypenko previously approved these changes Jul 2, 2024
AviadP
AviadP previously approved these changes Jul 2, 2024
@yitzhak12 yitzhak12 dismissed stale reviews from AviadP, DanielOsypenko, and ebondare via 44181ae July 2, 2024 12:39
@openshift-ci openshift-ci bot removed the lgtm label Jul 2, 2024
@openshift-ci openshift-ci bot added the lgtm label Jul 2, 2024
Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation on existing cluster

Cluster Name: ikave-ibm10-416
Cluster Configuration:
PR Test Suite: tier4b
PR Test Path: tests/functional/z_cluster/cluster_expansion/test_resize_osd.py::TestResizeOSD::test_resize_osd_with_node_restart tests/functional/z_cluster/cluster_expansion/test_resize_osd.py::TestResizeOSD::test_ui_storage_size_post_resize_osd
Additional Test Params:
OCP VERSION: 4.16
OCS VERSION: 4.16
tested against branch: master

Job FAILED (installation failed, tests not executed).

Copy link

@ocs-ci ocs-ci left a comment

Choose a reason for hiding this comment

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

PR validation on existing cluster

Cluster Name: ikave-ibm10-416
Cluster Configuration:
PR Test Suite: tier4b
PR Test Path: tests/functional/z_cluster/cluster_expansion/test_resize_osd.py::TestResizeOSD::test_resize_osd_with_node_restart tests/functional/z_cluster/cluster_expansion/test_resize_osd.py::TestResizeOSD::test_ui_storage_size_post_resize_osd
Additional Test Params:
OCP VERSION: 4.16
OCS VERSION: 4.16
tested against branch: master

Job PASSED.

@yitzhak12 yitzhak12 added the Verified Mark when PR was verified and log provided label Jul 3, 2024
Copy link

openshift-ci bot commented Jul 3, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: AviadP, DanielOsypenko, ebondare, PrasadDesala, yitzhak12

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

@PrasadDesala PrasadDesala merged commit 28b36aa into red-hat-storage:master Jul 3, 2024
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm size/M PR that changes 30-99 lines Squad/Black Covers UI specific test cases such as ODF Dashboard etc. Squad/Brown Test Case A test case PR Verified Mark when PR was verified and log provided
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants